Skip to content

fix(deps): raise the undici override to 8.12.2 to close GHSA-xjj6-f33q-929c - #201

Merged
GeiserX merged 1 commit into
mainfrom
fix/undici-8.12.2
Oct 26, 2026
Merged

GeiserX merged 1 commit into
mainfrom
fix/undici-8.12.2

Conversation

@GeiserX

@GeiserX GeiserX commented Oct 26, 2026 •

Copy link
Copy Markdown
Owner

Dependabot alert #142 (high, GHSA-xjj6-f33q-929c) flags undici 8.11.2 in the lockfile. Dependabot can't fix it on its own because our overrides entry holds undici at ^8.10.2.

undici only reaches us through test tooling (jsdom, testcontainers), so this is a dev-only dependency and nothing in the shipped app changes.

The fix raises the override to ^8.12.2. The lockfile now resolves undici 8.12.3 from registry.npmjs.org, and no other package moved.

Summary by CodeRabbit

  • No user-facing changes.

@coderabbitai

coderabbitai Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
Commits

Reviewing files that changed from the base of the PR and between 6a06417 and f69b4ab.

Files ignored due to path filters: 1
  • package-lock.json is excluded by !**/package-lock.json, !**/package-lock.json
Files selected for processing: 1
  • package.json

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Walkthrough

The undici override in package.json changes from ^8.10.2 to ^8.12.2.

Changes: Dependency override update

Layer / File(s) Summary
Update undici override
package.json
The undici override changes from ^8.10.2 to ^8.12.2.

Priority: Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Merge Risk: Minimal · up to f69b4

The dependency update appears mergeable with no identified current-head risk.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title names the change: raising the undici override from ^8.10.2 to 8.12.2 to close GHSA-xjj6-f33q-929c. The raw summary confirms the single-line package.json change to the undici override (^8.10.…
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Title check

Explanation

The title names the change: raising the undici override from ^8.10.2 to 8.12.2 to close GHSA-xjj6-f33q-929c. The raw summary confirms the single-line package.json change to the undici override (^8.10.2 to ^8.12.2). The objectives state the same intent, so the title matches the main change. The title is 75 characters, which is within a normal length for a conventional commit title.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@GeiserX
GeiserX merged commit 8e79490 into main Oct 26, 2026
11 checks passed
@GeiserX
GeiserX deleted the fix/undici-8.12.2 branch October 26, 2026 15:19
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.

1 participant