Repository navigation
chore(deps-dev): bundle the vitest, vite, undici and source-map-js bumps - #35
Conversation
Combines the lockfile changes from #19, #20, #22 and #34. _Generated with [`xum`](https://github.com/coder/xum) • Model: `anthropic:claude-opus-5-5` • Thinking: `high`_ Change-Id: I35dd0d39bef79f8df8cc90378719980e69016681 Signed-off-by: Thomas Kosiewski <tk@coder.com>
|
@codex review |
|
@codex security review |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Codex Review: Didn't find any major issues. Chef's kiss. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
🛡️ Codex Security ReviewSecurity review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
Change-Id: If1a9dc5eb1ba35cbcdfa27ab86fb80a8f6b9d8b0 Signed-off-by: Thomas Kosiewski <tk@coder.com>
Why
Four Dependabot PRs each changed only
package-lock.json, and they conflict with each other. This PR applies all of them in one lockfile so they land together.The same packages carry all 13 open Dependabot alerts on
main(5 high). Every alert's patched version is at or below the version in this lockfile.What changed
Scope
npm update --package-lock-only undici source-map-js. Every package version that changed matches a version one of the four PRs proposed. No package is added or removed.package.jsonis unchanged. All bumps are dev dependencies.Tradeoffs
npm update vitealone would also move rolldown to 1.2.16 and bump postcss and @types/estree. No Dependabot PR proposed those versions, so I left them out.Blast Radius
Dev and test tooling only. The published package does not ship these dependencies.
Verification
mise run cisteps on Node 26.12.1: npm ci, build:libghostty, build:native, build, 40/40 tests, smoke all pass.Generated with
xum• Model:anthropic:claude-opus-5-5• Thinking:high