chore: sync package-lock.json #23
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Co-authored-by: jmbish04 <26469722+jmbish04@users.noreply.github.com>
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
core-github-api | 72423be | Nov 27 2025, 06:08 PM |
Copilot
AI
changed the title
[WIP] Fix package-lock.json to sync with package.json
chore: sync package-lock.json
Nov 27, 2025
jmbish04
pushed a commit
that referenced
this pull request
Dec 6, 2025
Comprehensive improvements to landing page generator addressing all unresolved code review feedback. **JSONC Parsing (#20)** - Replace regex-based comment stripping with proper JSON5 parser - Add json5 as devDependency in package.json - Update generate.ts to use JSON5.parse() for wrangler.jsonc **Tailwind JIT Safety (#16, #23)** - Replace all dynamic class construction with static mappings - Add colorClasses map in generateMetrics() for safe color variants - Add statusColorClasses map in generateRoadmap() with complete class sets - Ensures Tailwind JIT can properly detect and include all classes **Remove `any` Types** - Define WranglerConfig interface matching Cloudflare schema - Define PackageJson interface for package.json structure - Define OpenAPISpec interface for API specs - Update all analyzer methods to use proper interfaces - Change `any` to `unknown` for truly dynamic values **CTAs Configurable (#9)** - Add `links` property to WorkerAnalysis (primary, secondary, footer) - Update generateHero() to use analysis.links with sensible defaults - Pass configured links through to template generator **Branding Configurable (#15)** - Add `branding` property to WorkerAnalysis (icon, displayName) - Update generateNav() and generateFooter() to use branding - Remove hardcoded "CloudflareWorker" and ⚡ emoji **Footer Links Configurable (#17)** - Update generateFooter() to accept footerLinks parameter - Default to /doc, /openapi.json, /openapi.yaml if not provided - Support custom footer link arrays via analysis.links.footer **Colors Configurable (#14)** - Add ColorScheme interface to types - Add colors property to WorkerAnalysis - Foundation for future gradient customization **Highlights Dynamic (#10)** - Add generateHighlights() method that analyzes components - Generate highlights based on Durable Objects, D1, AI, Queues presence - Add tech stack-based highlights (OpenAPI, Zod, Hono) - Replace hardcoded highlights array with dynamic generation **Architecture Diagrams Dynamic (#13, #22)** - Completely rewrite generateArchitectureDiagram() - Build diagram dynamically based on available components - Support 0, 1, 2+ components with proper ASCII layout - Add edge latency info to header **Use Cases Dynamic (#11)** - Update generateUseCases() to infer from architecture - Generate use cases based on Durable Objects, AI, Queues presence - Fallback to generic case if insufficient components **extractTechStack Refactor (#21)** - Replace repetitive if-statements with wranglerTechMap array - Add dependencyMap for cleaner package.json scanning - More maintainable and extensible approach **Workflows Schema Alignment (#6)** - Verify workflows binding matches Cloudflare schema - Add proper fallback: w.name || w.class_name - Add comment documenting schema expectations **OpenAPI Feature Extraction (#7)** - Prioritize OpenAPI tags over path segments - Extract from operation summaries as secondary fallback - Only use path segments as last resort - Much more semantic feature detection **Pain Points Documentation (#8)** - Add JSDoc comment to identifyPainPoints() - Document that method returns generic fallback - Direct users to override via customAnalysis **CLI Description Clarity (#2)** - Clarify generate.ts is for generating landing pages - Not a CLI for the Worker itself **README Improvements (#1, #3, #4)** - Add usage examples for asset-based JSON parsing in Workers - Clarify that features come from WorkerAnalysis.features - Document customization via customAnalysis instead of editing templates - Add comprehensive examples of all new configuration options **Reduce Need for customAnalysis** - Better OpenAPI tag extraction reduces manual feature specification - Dynamic highlights generation reduces boilerplate - Dynamic diagram generation adapts to architecture - Dynamic use cases infer from components - Users still need to customize purpose/pain points (domain-specific) **Component Scope** - mobileMenuOpen already scoped via x-data on body - No global Alpine.store() usage - working as intended - types.ts: +90 lines (new interfaces, ColorScheme, typed configs) - analyzer.ts: Proper types, improved extractFeatures, refactored extractTechStack - blueprint.ts: Dynamic highlights, improved diagram, configurable CTAs - template.ts: Static Tailwind classes, branding params, footer links - generate.ts: JSON5 parser, branding config, footer links config - index.ts: Pass branding and footer links to template - package.json: Add json5 dependency - Generated landing page verified with all improvements - Static Tailwind classes ensure JIT compatibility - Branding (🔮 Core GitHub API) applied successfully - Custom footer links rendered correctly - Dynamic architecture diagram shows all 5 component types
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
CI deployment failing due to
npm cierror from out-of-syncpackage-lock.json.Changes
package-lock.jsonfrom scratch to matchpackage.jsonmanifestThe lockfile was stale and missing resolved versions for some transitive dependencies. Fresh generation ensures CI can install deterministically.
Original prompt
This pull request was created as a result of the following prompt from Copilot chat.
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.