Skip to content

chore: upgrade Sentry dependencies#599

Merged
kentcdodds merged 5 commits into
mainfrom
cursor/sentry-dependencies-upgrade-f018
Apr 28, 2026
Merged

chore: upgrade Sentry dependencies#599
kentcdodds merged 5 commits into
mainfrom
cursor/sentry-dependencies-upgrade-f018

Conversation

@kentcdodds
Copy link
Copy Markdown
Member

@kentcdodds kentcdodds commented Apr 28, 2026

Summary

  • sync this branch onto the latest main, including the merged NX/workspace tooling, MCP/auth, and app platform dependency baselines
  • upgrade the Sentry dependency group together across the app, utils, CLI, and MCP packages
  • refresh the root and example lockfiles for the merged baselines plus the aligned Sentry SDK and Vite plugin versions
  • keep MCP tests from enabling Sentry instrumentation in NODE_ENV=test, which avoids a child-process regression introduced by the upgraded MCP wrapper path

Test Plan

  • npm run validate
  • inspect the Sentry runtime initialization and Vite integration entrypoints in packages/workshop-app/instrument.js, packages/workshop-app/app/utils/monitoring.client.ts, packages/workshop-app/vite.config.ts, packages/workshop-cli/src/utils/sentry-cli.ts, and packages/workshop-mcp/src/index.ts
Open in Web Open in Cursor 

@nx-cloud
Copy link
Copy Markdown

nx-cloud Bot commented Apr 28, 2026

View your CI Pipeline Execution ↗ for commit bf3ee1e

Command Status Duration Result
nx run-many --target typecheck ✅ Succeeded 15s View ↗
nx run-many --target build ✅ Succeeded 50s View ↗
nx lint ✅ Succeeded 33s View ↗

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

@kentcdodds kentcdodds marked this pull request as ready for review April 28, 2026 18:09
@kentcdodds kentcdodds merged commit e139852 into main Apr 28, 2026
10 checks passed
@kentcdodds kentcdodds deleted the cursor/sentry-dependencies-upgrade-f018 branch April 28, 2026 19:04
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