js: misc yarn cleanup (#93474)

Author: joshuarli

.github/actions/setup-sentry/action.yml

@@ -17,9 +17,6 @@ inputs:
     default: 'default'
 
 outputs:
-  yarn-cache-dir:
-    description: 'Path to yarn cache'
-    value: ${{ steps.config.outputs.yarn-cache-dir }}
   matrix-instance-number:
     description: 'The matrix instance number (starting at 1)'
     value: ${{ steps.config.outputs.matrix-instance-number }}
@@ -94,7 +91,6 @@ runs:
         MATRIX_INSTANCE: ${{ matrix.instance }}
       shell: bash --noprofile --norc -eo pipefail -ux {0}
       run: |
-        echo "yarn-cache-dir=$(yarn cache dir)" >> "$GITHUB_OUTPUT"
         echo "matrix-instance-number=$(($MATRIX_INSTANCE+1))" >> "$GITHUB_OUTPUT"
         echo "matrix-instance-total=$((${MATRIX_INSTANCE_TOTAL:-}))" >> "$GITHUB_OUTPUT"
 

Brewfile

@@ -9,7 +9,7 @@ brew 'qemu'
 brew 'docker'
 brew 'docker-buildx'
 
-# required for yarn test -u
+# required for pnpm test -u
 brew 'watchman'
 
 # direnv isn't defined here, because we have it configured to check for a bootstrapped environment.

rspack.config.ts

@@ -457,7 +457,7 @@ const appConfig: Configuration = {
       stream: false,
       // Node crypto is imported in @sentry-internal/global-search but not used here
       crypto: false,
-      // `yarn why` says this is only needed in dev deps
+      // `pnpm why` says this is only needed in dev deps
       string_decoder: false,
       // For framer motion v6, might be able to remove on v11
       'process/browser': require.resolve('process/browser'),
@@ -653,7 +653,7 @@ if (IS_UI_DEV_ONLY) {
     return slug;
   };
 
-  // Try and load certificates from mkcert if available. Use $ yarn mkcert-localhost
+  // Try and load certificates from mkcert if available. Use $ pnpm mkcert-localhost
   const certPath = path.join(__dirname, 'config');
   const httpsOptions = fs.existsSync(path.join(certPath, 'localhost.pem'))
     ? {

src/sentry/charts/chartcuterie.py

@@ -80,7 +80,7 @@ def generate_chart(self, style: ChartType, data: Any, size: ChartSize | None = N
 
             if resp.status_code == 503 and settings.DEBUG:
                 logger.info(
-                    "You may need to build the chartcuterie config using `yarn build-chartcuterie-config`"
+                    "You may need to build the chartcuterie config using `pnpm build-chartcuterie-config`"
                 )
 
             if resp.status_code != 200:

static/app/utils/profiling/profile/profile.benchmark.ts

@@ -1,7 +1,7 @@
 // Benchmarks allow us to make changes and evaluate performance before the code gets shipped to production.
 // They can be used to make performance improvements or to test impact of newly added functionality.
 
-// Run with: yarn run ts-node --project ./config/tsconfig.benchmark.json -r tsconfig-paths/register static/app/utils/profiling/profile/profile.benchmark.ts
+// Run with: pnpm run ts-node --project ./config/tsconfig.benchmark.json -r tsconfig-paths/register static/app/utils/profiling/profile/profile.benchmark.ts
 
 // @ts-expect-error TS(7016): Could not find a declaration file for module 'benc... Remove this comment to see the full error message
 import benchmarkjs from 'benchmark';

static/app/utils/profiling/renderers/flamegraphTextRenderer.benchmark.ts

@@ -1,6 +1,6 @@
 // Benchmarks allow us to make changes and evaluate performance before the code gets shipped to production.
 // They can be used to make performance improvements or to test impact of newly added functionality.
-// Run with: yarn run ts-node --project ./config/tsconfig.benchmark.json -r tsconfig-paths/register static/app/utils/profiling/renderers/textRenderer.benchmark.ts
+// Run with: pnpm run ts-node --project ./config/tsconfig.benchmark.json -r tsconfig-paths/register static/app/utils/profiling/renderers/textRenderer.benchmark.ts
 // @ts-expect-error TS(7016): Could not find a declaration file for module 'benc... Remove this comment to see the full error message
 import benchmarkjs from 'benchmark';
 import maxBy from 'lodash/maxBy';