Skip to content

chore: upgrade app platform dependencies#597

Merged
kentcdodds merged 3 commits into
mainfrom
cursor/app-platform-dependencies-3986
Apr 28, 2026
Merged

chore: upgrade app platform dependencies#597
kentcdodds merged 3 commits into
mainfrom
cursor/app-platform-dependencies-3986

Conversation

@kentcdodds
Copy link
Copy Markdown
Member

@kentcdodds kentcdodds commented Apr 28, 2026

Summary

  • upgrade the app platform dependency group together across the workspace and example manifests
  • preserve the merged NX/formatting/tooling changes from main and the later MCP/auth dependency updates from main while keeping this PR scoped to React Router, Vite, TypeScript, Tailwind, Vitest, Playwright, and esbuild
  • remove the deprecated baseUrl setting from packages/workshop-app/tsconfig.json so the TypeScript 6 upgrade typechecks cleanly
  • regenerate the root and example lockfiles after each main sync so they reflect the latest tooling/auth baseline plus the app platform upgrade

Test Plan

  • npm install
  • npm run validate
  • npm run test
  • merge latest main after the NX/workspace tooling PR and rerun npm run validate
  • rerun npm run test after the NX/workspace tooling merge
  • merge latest main again after the MCP/auth dependency PR and rerun npm run validate
  • rerun npm run test after the MCP/auth merge

Notes

  • this branch now includes the latest main through merge commit 7eab68e8, after absorbing both 84173232 (NX/workspace tooling) and e882eead (MCP/auth dependency group)
  • on the first full-suite run after the MCP/auth merge, npm run test transiently failed in packages/workshop-mcp/src/mcp-client.test.ts; the isolated MCP test file passed immediately after, and a full npm run test rerun also passed without code changes, so I did not widen scope into that upstream area
  • npm run validate still reports existing warning-level lint findings in packages/workshop-mcp/src/utils.test.ts, but validation succeeds and this PR does not expand into that separate cleanup

Artifacts

Open in Web Open in Cursor 

@kentcdodds kentcdodds marked this pull request as ready for review April 28, 2026 17:49
@nx-cloud
Copy link
Copy Markdown

nx-cloud Bot commented Apr 28, 2026

View your CI Pipeline Execution ↗ for commit 7eab68e

Command Status Duration Result
nx run-many --target typecheck ✅ Succeeded 19s View ↗
nx run-many --target build ✅ Succeeded 39s View ↗
nx lint ✅ Succeeded 35s View ↗

☁️ Nx Cloud last updated this comment at 2026-04-28 18:25:46 UTC

@kentcdodds kentcdodds merged commit 8f0efd5 into main Apr 28, 2026
10 checks passed
@kentcdodds kentcdodds deleted the cursor/app-platform-dependencies-3986 branch April 28, 2026 18:27
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