Skip to content

fix(deps): patch current High, Medium, and Low advisories#299

Open
seonghobae wants to merge 7 commits into
developmentalfrom
fix/security-advisories-2026-07-22
Open

fix(deps): patch current High, Medium, and Low advisories#299
seonghobae wants to merge 7 commits into
developmentalfrom
fix/security-advisories-2026-07-22

Conversation

@seonghobae

Copy link
Copy Markdown

Summary

  • patches every vulnerability currently reported by OSV for the pnpm lockfile
  • keeps brace-expansion major lines compatible by upgrading 1.x and 5.x independently while preserving the already-safe 2.1.2 line
  • moves effective pnpm 9 security overrides into the root package manifest and removes stale duplicate workspace entries

Patched advisories

Validation

  • frozen pnpm 9.15.4 install and Prisma client generation
  • shadcn CLI startup after the @hono/node-server major-line override
  • all workspace typechecks and lints
  • CLI: 153 tests passed
  • shared: 42 tests passed
  • web without database: 230 passed, 13 environment-dependent tests skipped
  • web with isolated PostgreSQL 16: 243 tests passed with no skips
  • Prisma migration drift: no difference detected
  • all 17 migrations deployed successfully to an empty PostgreSQL database
  • Next.js production build completed successfully
  • OSV lockfile scan: zero findings
  • Trivy filesystem scan at Medium, High, and Critical: zero findings

The package ranges are deliberately narrow so unrelated dependency major lines are not coerced.

Copilot AI review requested due to automatic review settings July 22, 2026 20:50

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates pnpm overrides (and the generated lockfile) to remediate currently reported OSV vulnerabilities, moving effective pnpm 9 override pins into the root package.json and cleaning up stale workspace-level entries.

Changes:

  • Move/add security-related pnpm.overrides into the root package.json (and reflect them in pnpm-lock.yaml).
  • Update patched dependency versions in the lockfile (e.g., hono, js-yaml, fast-uri, body-parser, sharp, @hono/node-server).
  • Remove stale brace-expansion / fast-uri entries from pnpm-workspace.yaml overrides.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.

File Description
pnpm-workspace.yaml Removes stale override entries so workspace overrides don’t duplicate / mislead when pnpm 9 is in use.
pnpm-lock.yaml Regenerates lockfile to apply the new overrides and upgraded patched dependency versions.
package.json Adds/updates root pnpm.overrides to enforce patched versions under pnpm 9.
Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread package.json Outdated
Comment thread package.json
Copilot AI review requested due to automatic review settings July 22, 2026 20:57

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.

Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

Comment thread package.json
Comment thread package.json
Copilot AI review requested due to automatic review settings July 22, 2026 21:19

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 9 out of 10 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

Comment thread .claude/skills/persuasion-review/scripts/probe_harness.py
Copilot AI review requested due to automatic review settings July 22, 2026 21:28

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 13 out of 14 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

Comment thread .claude/skills/persuasion-review/scripts/probe_harness.py Outdated
Copilot AI review requested due to automatic review settings July 22, 2026 21:52
Comment thread .claude/skills/persuasion-review/scripts/test_probe_harness.py Fixed
Comment thread .claude/skills/persuasion-review/scripts/test_probe_harness.py Fixed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 13 out of 14 changed files in this pull request and generated 2 comments.

Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

Comment thread .claude/skills/persuasion-review/scripts/probe_harness.py Outdated
Comment thread .github/workflows/ci.yml
Copilot AI review requested due to automatic review settings July 22, 2026 22:01

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 13 out of 14 changed files in this pull request and generated 10 comments.

Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

Comment thread packages/cli/src/lib/transcript.test.ts
Comment thread packages/cli/src/__tests__/transcript.test.ts
Comment thread packages/cli/src/lib/project.ts
Comment thread packages/cli/src/lib/project.ts
Comment thread packages/cli/src/lib/project.ts
Comment thread packages/cli/src/lib/project.ts
Comment thread packages/cli/src/lib/project.ts
Comment thread packages/cli/src/lib/inject-agent-hooks.ts
Comment thread packages/cli/src/commands/status.ts
Comment thread .claude/skills/persuasion-review/scripts/probe_harness.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants