chore(deps): consolidate Dependabot bumps + re-vendor diff CSS - #70
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Deploying agent-render with
|
| Latest commit: |
6fa4157
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://0e787ee8.agent-render.pages.dev |
| Branch Preview URL: | https://codex-dependabot-deps.agent-render.pages.dev |
|
| Filename | Overview |
|---|---|
| package.json | Bumps @git-diff-view/react ^0.1.1→^0.1.5, @playwright/test ^1.58.2→^1.61.0, vitest ^4.0.18→^4.1.9, and @types/node ^22.13.10→^22.19.21. Deliberate holds for @types/node major and @codemirror/search are explained in the PR. |
| package-lock.json | Lock file updated to reflect resolved package versions; transitive dependency reactivity-store bumped from ^0.3.12 to ^0.4.0 (driven by @git-diff-view/react 0.1.5's own dependency upgrade). |
| public/vendor/diff-view-pure.css | Re-vendored from @git-diff-view/react 0.1.5; additions include !container responsive overrides, flow-root display rule, and multi-select/drag-selection styles. tests/diff-style-asset.test.ts enforces exact byte-for-byte sync with the package copy. |
| public/vendor/diff-view-pure.css.br | Brotli-compressed counterpart of diff-view-pure.css regenerated to match the updated plain asset; the sync test decompresses and compares both files byte-for-byte. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A["npm install\n@git-diff-view/react@0.1.5"] --> B["node_modules/@git-diff-view/react\n/styles/diff-view-pure.css"]
A --> C["npm run assets:compress"]
C --> D["public/vendor/diff-view-pure.css\n(re-vendored plain copy)"]
C --> E["public/vendor/diff-view-pure.css.br\n(Brotli-compressed copy)"]
D --> F["tests/diff-style-asset.test.ts"]
E --> F
B --> F
F -- "expect(public) === expect(package)" --> G["✅ Test passes – assets in sync"]
D --> H["DiffRenderer\n(deferred stylesheet load)"]
E --> H
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
A["npm install\n@git-diff-view/react@0.1.5"] --> B["node_modules/@git-diff-view/react\n/styles/diff-view-pure.css"]
A --> C["npm run assets:compress"]
C --> D["public/vendor/diff-view-pure.css\n(re-vendored plain copy)"]
C --> E["public/vendor/diff-view-pure.css.br\n(Brotli-compressed copy)"]
D --> F["tests/diff-style-asset.test.ts"]
E --> F
B --> F
F -- "expect(public) === expect(package)" --> G["✅ Test passes – assets in sync"]
D --> H["DiffRenderer\n(deferred stylesheet load)"]
E --> H
Reviews (3): Last reviewed commit: "chore(deps): bump @git-diff-view/react, ..." | Re-trigger Greptile
7e5dc67 to
0d371b2
Compare
…vendor diff CSS Consolidates the open Dependabot bumps that survive the optimization stack: - @git-diff-view/react ^0.1.1 -> ^0.1.5 (re-vendored public/vendor/diff-view-pure.css + .br) - @playwright/test ^1.58.2 -> ^1.61.0 - vitest ^4.0.18 -> ^4.1.9 Held back, on purpose: - @types/node stays at ^22 (matches the supported runtime; engines is node >=20.10). Bumping to 25 types Node-25 APIs that do not exist on the supported runtime and broke the Buffer typings in tests. - @codemirror/search: already removed by the optimization branch. vitest 4.1 needs no config change on regular vite 7 (the esbuild jsx config still applies); the 4.1.5 rolldown/oxc parse failure does not reproduce on 4.1.9. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
0d371b2 to
6fa4157
Compare
Summary
Consolidates the open Dependabot bumps into one CI-clean PR. (The Dependabot PRs show red only because the
claude-reviewcheck can't run on them — Dependabot PRs don't receive repo secrets — even though their build + test pass.) Stacked on #69.Bumps
@git-diff-view/react^0.1.1 → ^0.1.5 — re-vendoredpublic/vendor/diff-view-pure.css+.brto keep the deferred stylesheet in sync (pinned bytests/diff-style-asset.test.ts).@playwright/test^1.58.2 → ^1.61.0vitest^4.0.18 → ^4.1.9 — no config change needed: on regular vite 7 the existing esbuild JSX config still applies. The 4.1.5 rolldown/oxc parse failure that broke chore(deps-dev): bump vitest from 4.0.18 to 4.1.5 #64 does not reproduce on 4.1.9.Deliberately held
@types/node: kept at ^22. It should track the supported runtime (engines: node >=20.10); bumping to 25 types Node-25 APIs that don't exist there and broke theBuffertypings in tests. Revisit when the runtime floor moves.@codemirror/search: already removed by Optimize fragment rendering and homepage link generation #67, so its bump is moot.Supersedes #62, #63, #64, #65, #66.
Verification
npm run checkgreen — lint + 216 tests + bench + typecheck + build + budgets (rich-diff deferred JS 323.9 / 340 KiB).🤖 Generated with Claude Code