Skip to content

Consolidate Dependabot dependency bumps (#122–#128)#129

Open
enricopiovesan wants to merge 1 commit into
mainfrom
cursor/deps-consolidated
Open

Consolidate Dependabot dependency bumps (#122–#128)#129
enricopiovesan wants to merge 1 commit into
mainfrom
cursor/deps-consolidated

Conversation

@enricopiovesan

Copy link
Copy Markdown
Contributor

Summary

Consolidates the seven open Dependabot PRs into one maintainer PR that passes repo CI gates.

  • Bump GitHub Actions: actions/checkout v4→v7, actions/setup-node v4→v6
  • Bump npm dev deps across all web workspaces: vitest/@vitest/coverage-v8 3→4, jsdom 26→29, @types/node 24→26, typescript-eslint 8.62→8.63
  • Replace global with globalThis in unit tests so typecheck passes with Vitest 4 and @types/node 26

No UI or runtime behavior changes — dependency and test typing only. Architecture boundary unchanged: UI remains a rendering layer with no business logic added.

Supersedes #122, #123, #124, #125, #126, #127, #128.

Definition of Done

Validation

npm install
npm run typecheck
npm run lint
npm run test
bash scripts/ci/coverage_gate.sh
bash scripts/ci/repository_checks.sh

All commands pass locally.

Checklist

  • Issue linked and Project 2 item updated (maintenance — no feature issue)
  • Updated AGENTS.md platform table if this changes platform status (N/A)
  • No business logic added to UI layer
  • CI gates pass locally where applicable

Made with Cursor

Supersedes #122#128 with Vitest 4/jsdom 29 compatibility fixes so CI gates pass.

Co-authored-by: Cursor <cursoragent@cursor.com>
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