Open
Conversation
- Switch from npm to Yarn 4 (corepack, nodeLinker: node-modules) - Restructure into proper monorepo layout: - packages/sentry-testkit/ (main publishable package) - packages/sentry-testkit-docs/ (Docusaurus site, was website/) - apps/expo-react-native-test-app/ (RN test harness) - Replace legacy browser/ subpath hack with proper exports map in package.json - Delete old Docsify docs/ folder - Remove all package-lock.json files; yarn.lock at root - Update all GitHub Actions workflows: - npm → yarn (--immutable, corepack enable) - workspace-scoped build/test/publish commands - Fix test-deploy-docs.yml targeting main instead of master - Use YARN_NPM_AUTH_TOKEN for publish Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Import sentry-testkit via workspace package instead of relative ../../src - Import commonTests via correct relative path to packages/sentry-testkit/__tests__ - Add sentry-testkit workspace:* to expo app devDependencies Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
packageManager: yarn@4.13.0,nodeLinker: node-modules) — replaces npm and allpackage-lock.jsonfiles with a singleyarn.lockat the rootpackages/sentry-testkit/— main publishable package (moved from root)packages/sentry-testkit-docs/— Docusaurus site (waswebsite/)apps/expo-react-native-test-app/— React Native test harness (wasexpo-react-native-test-app/)browser/package.jsonsubpath hack with a proper"exports"map inpackages/sentry-testkit/package.json—sentry-testkit/browsernow resolves via the standard Node.js exports field; the manualbrowser.d.tsis removed (TypeScript auto-generatesdist/browser.d.ts)docs/(old Docsify site, superseded by Docusaurus inpackages/sentry-testkit-docs/)npm ci→yarn install --immutablewithcorepack enablenpm run build/test→yarn workspace sentry-testkit build/testnpm publish→yarn workspace sentry-testkit npm publish+YARN_NPM_AUTH_TOKENdeploy-docs/test-deploy-docspoint topackages/sentry-testkit-docs/test-deploy-docs.ymlwhich was incorrectly targetingmaininstead ofmasterTest plan
yarn installat repo root — should generateyarn.lockand install all workspacesyarn workspace sentry-testkit build— should compilepackages/sentry-testkit/src/→dist/yarn workspace sentry-testkit test— all existing tests should passrequire('sentry-testkit/browser')resolves via new exports map (nobrowser/dir needed)yarn workspace expo-react-native-test-app test— Expo RN tests should passyarn workspace sentry-testkit-docs build— Docusaurus site should build cleanly🤖 Generated with Claude Code