Skip to content

Commit 9810700

Browse files
committed
fix(ci): replace rm -rf with cross-platform del-cli command
The rm -rf command fails on Windows PowerShell, causing test-setup-script to fail and preventing dist/cli.js from being built. This resulted in 'Cannot find module dist/cli.js' errors across all Windows test runs. Changed from: rm -rf packages/cli/node_modules/.vite packages/cli/dist To: pnpm exec del-cli packages/cli/node_modules/.vite packages/cli/dist This uses the del-cli package which works consistently across all platforms (Windows, macOS, Linux).
1 parent e344956 commit 9810700

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: 'rm -rf 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 install --frozen-lockfile && 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)