Resolve all 69 open Dependabot security alerts - #42
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR focuses on resolving Dependabot security alerts by updating direct and transitive dependencies across the pnpm workspace, and includes a small compatibility update for Cloudflare Workers AI request payloads required by the newer wrangler tooling.
Changes:
- Added two targeted
pnpm.overrides(yamlrange pin andesbuild@0.18→0.25.12) to address remaining transitive vulnerabilities not fixed by parent upgrades. - Upgraded key direct dependencies (
wrangler,vite,hono,fast-xml-parser) and refreshedpnpm-lock.yamlaccordingly. - Updated
apps/jobsWorkers AI request field frommax_output_tokenstomax_tokens, along with its unit test.
Reviewed changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| pnpm-lock.yaml | Captures dependency graph updates plus applied overrides and patched transitive resolutions. |
| package.json | Adds root pnpm.overrides for yaml and esbuild@0.18 while keeping existing tooling config intact. |
| apps/ui/package.json | Bumps vite to a patched version. |
| apps/api/package.json | Bumps hono and wrangler to patched versions. |
| apps/jobs/package.json | Bumps fast-xml-parser and wrangler to patched versions. |
| apps/jobs/src/clients/ai-client.ts | Renames Workers AI request field to max_tokens to match updated schema expectations. |
| apps/jobs/src/tests/ai-client.test.ts | Updates request-shape assertions to validate max_tokens. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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
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.
What
pnpm.overridesfor the only two transitives not fixable via a stable parent upgrade:yaml(>=2.8.3 <3 — vite's optional peer, resolved out entirely) andesbuild@0.18→0.25.12 (pinned by the deprecated@esbuild-kit/core-utilsthat drizzle-kit still requires).apps/jobs:max_output_tokens→max_tokens— required by wrangler 4.107.0's regenerated AI schema for@cf/meta/llama-3.1-8b-instruct-fast(same value/semantics), plus its test.How to test
Security review
Dependency security patches only (69 CVEs); the sole code change is a Workers AI request-field rename. No application auth, input-handling, or data-flow logic changed.
Notes
pnpm.overridesare temporary — remove them once the parent packages ship compatible patched versions (drizzle-kit dropping@esbuild-kit; vite's optional yaml peer).mainauto-deploys all three Workers; recommend a post-deploy smoke check of the jobs AI path (themax_tokensrename) and a revert plan, per the Codex plan review.🤖 Generated with Claude Code