fix(ci): Remove container from E2E jobs — they need host Docker access

E2E tests use Docker-in-Docker (Verdaccio registry) which doesn't work
inside a GHA container due to network isolation. On develop these jobs
run directly on the host. Only browser Playwright, loader, and Remix
tests need the Playwright container image.

Revert registrySetup.ts path conversion and temp directory changes for
E2E jobs since they now run on the host again.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: mydea

.github/workflows/build.yml

@@ -922,15 +922,8 @@ jobs:
     # See: https://github.com/actions/runner/issues/2205
     if:
       always() && needs.job_e2e_prepare.result == 'success' && needs.job_e2e_prepare.outputs.matrix != '{"include":[]}'
-      && needs.job_playwright_image.result == 'success'
-    needs: [job_get_metadata, job_build, job_e2e_prepare, job_playwright_image]
+    needs: [job_get_metadata, job_build, job_e2e_prepare]
     runs-on: ubuntu-24.04
-    container:
-      image: ${{ needs.job_playwright_image.outputs.image }}
-      credentials:
-        username: ${{ github.actor }}
-        password: ${{ secrets.GITHUB_TOKEN }}
-      options: --ipc=host --user root
     timeout-minutes: 15
     env:
       # We just use a dummy DSN here, only send to the tunnel anyhow
@@ -1003,18 +996,18 @@ jobs:
           E2E_TEST_PUBLISH_SCRIPT_NODE_VERSION: ${{ steps.versions.outputs.node }}
 
       - name: Copy to temp
-        run: yarn ci:copy-to-temp ./test-applications/${{ matrix.test-application }} ../tmp-test-application
+        run: yarn ci:copy-to-temp ./test-applications/${{ matrix.test-application }} ${{ runner.temp }}/test-application
         working-directory: dev-packages/e2e-tests
 
       - name: Build E2E app
-        working-directory: dev-packages/tmp-test-application
+        working-directory: ${{ runner.temp }}/test-application
         timeout-minutes: 7
         run: ${{ matrix.build-command || 'pnpm test:build' }}
         env:
           SENTRY_E2E_WORKSPACE_ROOT: ${{ github.workspace }}
 
       - name: Run E2E test
-        working-directory: dev-packages/tmp-test-application
+        working-directory: ${{ runner.temp }}/test-application
         timeout-minutes: 10
         run: ${{ matrix.assert-command || 'pnpm test:assert' }}
         env:
@@ -1053,16 +1046,9 @@ jobs:
     if:
       always() && needs.job_get_metadata.outputs.is_release != 'true' && needs.job_e2e_prepare.result == 'success' &&
       needs.job_e2e_prepare.outputs.matrix-optional != '{"include":[]}' && (github.event_name != 'pull_request' ||
-      github.event.pull_request.head.repo.full_name == github.repository) && github.actor != 'dependabot[bot]' &&
-      needs.job_playwright_image.result == 'success'
-    needs: [job_get_metadata, job_build, job_e2e_prepare, job_playwright_image]
+      github.event.pull_request.head.repo.full_name == github.repository) && github.actor != 'dependabot[bot]'
+    needs: [job_get_metadata, job_build, job_e2e_prepare]
     runs-on: ubuntu-24.04
-    container:
-      image: ${{ needs.job_playwright_image.outputs.image }}
-      credentials:
-        username: ${{ github.actor }}
-        password: ${{ secrets.GITHUB_TOKEN }}
-      options: --ipc=host --user root
     timeout-minutes: 15
     env:
       E2E_TEST_AUTH_TOKEN: ${{ secrets.E2E_TEST_AUTH_TOKEN }}
@@ -1122,16 +1108,16 @@ jobs:
           E2E_TEST_PUBLISH_SCRIPT_NODE_VERSION: ${{ steps.versions.outputs.node }}
 
       - name: Copy to temp
-        run: yarn ci:copy-to-temp ./test-applications/${{ matrix.test-application }} ../tmp-test-application
+        run: yarn ci:copy-to-temp ./test-applications/${{ matrix.test-application }} ${{ runner.temp }}/test-application
         working-directory: dev-packages/e2e-tests
 
       - name: Build E2E app
-        working-directory: dev-packages/tmp-test-application
+        working-directory: ${{ runner.temp }}/test-application
         timeout-minutes: 7
         run: ${{ matrix.build-command || 'pnpm test:build' }}
 
       - name: Run E2E test
-        working-directory: dev-packages/tmp-test-application
+        working-directory: ${{ runner.temp }}/test-application
         timeout-minutes: 10
         run: ${{ matrix.assert-command || 'pnpm test:assert' }}
 

.github/workflows/canary.yml

@@ -180,16 +180,16 @@ jobs:
           E2E_TEST_PUBLISH_SCRIPT_NODE_VERSION: ${{ steps.versions.outputs.node }}
 
       - name: Copy to temp
-        run: yarn ci:copy-to-temp ./test-applications/${{ matrix.test-application }} ../tmp-test-application
+        run: yarn ci:copy-to-temp ./test-applications/${{ matrix.test-application }} ${{ runner.temp }}/test-application
         working-directory: dev-packages/e2e-tests
 
       - name: Build E2E app
-        working-directory: dev-packages/tmp-test-application
+        working-directory: ${{ runner.temp }}/test-application
         timeout-minutes: 7
         run: yarn ${{ matrix.build-command }}
 
       - name: Run E2E test
-        working-directory: dev-packages/tmp-test-application
+        working-directory: ${{ runner.temp }}/test-application
         timeout-minutes: 15
         run: yarn test:assert
 

dev-packages/e2e-tests/registrySetup.ts

@@ -4,15 +4,7 @@ import * as path from 'path';
 import { PUBLISH_PACKAGES_DOCKER_IMAGE_NAME, TEST_REGISTRY_CONTAINER_NAME, VERDACCIO_VERSION } from './lib/constants';
 
 const publishScriptNodeVersion = process.env.E2E_TEST_PUBLISH_SCRIPT_NODE_VERSION;
-
-// When running inside a GHA container, paths resolve to /__w/... which only
-// exists inside the container. Docker-in-Docker mounts need host paths.
-// The host maps /home/runner/work to /__w, so we convert back.
-function toHostPath(p: string): string {
-  return p.replace(/^\/__w\//, '/home/runner/work/');
-}
-
-const repositoryRoot = toHostPath(path.resolve(__dirname, '../..'));
+const repositoryRoot = path.resolve(__dirname, '../..');
 
 // https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#grouping-log-lines
 function groupCIOutput(groupTitle: string, fn: () => void): void {
@@ -43,7 +35,7 @@ export function registrySetup(): void {
         '-p',
         '4873:4873',
         '-v',
-        `${toHostPath(__dirname)}/verdaccio-config:/verdaccio/conf`,
+        `${__dirname}/verdaccio-config:/verdaccio/conf`,
         `verdaccio/verdaccio:${VERDACCIO_VERSION}`,
       ],
       { encoding: 'utf8', stdio: 'inherit' },