Skip to content

Bump some dependencies to suppress security warnings - #236

Merged
dividedmind merged 2 commits into
mainfrom
chore/bump-deps
Sep 11, 2026
Merged

Bump some dependencies to suppress security warnings#236
dividedmind merged 2 commits into
mainfrom
chore/bump-deps

Conversation

@dividedmind

Copy link
Copy Markdown
Collaborator

Bump some dependencies, mostly test-only, to suppress vulnerability warnings.

Bumps some dependencies to suppress security warnings:
- js-yaml 4 to 4.3.2,
- next 16 to 16.3.4,
- vitest 4 to 4.1.11.

Note there are still some security warnings flagged on older next
and vitest versions; these aren't maintained anymore so there aren't
bumps available, but these are here for testing only and are benign
in this project. Added suppresions for these.
@dividedmind
dividedmind requested review from kgilpin and a lite review from Copilot September 11, 2026 13:27
@dividedmind dividedmind self-assigned this Sep 11, 2026

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.

🟡 Changes recommended

The Vitest 4 fixture breaks the unconditional compatibility test on Node 18; keep Vitest 3 or gate the test before upgrading.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Updates dependencies and lockfiles to address security warnings, including test tooling and Next.js-related packages.

Changes:

  • Refreshes root and compatibility-fixture lockfiles.
  • Upgrades selected dependencies, including Vitest.
  • Updates audit advisory suppressions and rationale.
File summaries
File Description
yarn.lock Updates root dependency resolutions and checksums.
test/pnpm-compat/yarn.lock Regenerates the compatibility fixture dependency graph.
test/pnpm-compat/package.json Upgrades the fixture to Vitest 4.
.yarnrc.yml Updates audit suppressions and documentation.
Review details
  • Files reviewed: 2/4 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread test/pnpm-compat/package.json
This change is in order to suppress vulnerability warnings, even
though they're benign in this test code.
@pkg-pr-new

pkg-pr-new Bot commented Sep 11, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/appmap-node@236 -D

commit: be22263

@dividedmind
dividedmind merged commit fd40eb5 into main Sep 11, 2026
8 checks passed
@dividedmind
dividedmind deleted the chore/bump-deps branch September 11, 2026 14:04
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.

2 participants