Skip to content

Commit 14e3e55

Browse files
committed
fix(ci): remove redundant pnpm install from test-setup-script
The setup-and-install action already runs pnpm install --frozen-lockfile, so running it again in test-setup-script is redundant and may cause issues with the build output. Simplified test-setup-script to: - Clean dist/ and .vite cache with del-cli - Build the CLI This should resolve remaining 'Cannot find module dist/cli.js' errors by ensuring the build output is not affected by a second install.
1 parent 9810700 commit 14e3e55

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ jobs:
4040
name: Run CI Pipeline
4141
uses: SocketDev/socket-registry/.github/workflows/ci.yml@d8ff3b0581d799466cfbf150f715c1a4bf9f84a5 # 2025-10-23
4242
with:
43-
test-setup-script: 'pnpm exec del-cli packages/cli/node_modules/.vite packages/cli/dist && pnpm install --frozen-lockfile && pnpm --filter @socketsecurity/cli run build'
43+
test-setup-script: 'pnpm exec del-cli packages/cli/node_modules/.vite packages/cli/dist && pnpm --filter @socketsecurity/cli run build'
4444
lint-script: 'pnpm --filter @socketsecurity/cli run check'
4545
type-check-script: 'pnpm --filter @socketsecurity/cli run type'
4646
test-script: ${{ inputs.skip-tests && 'echo "Tests skipped"' || 'pnpm --filter @socketsecurity/cli run test:unit' }}

0 commit comments

Comments
 (0)