Skip to content

ci: route npm installs through Socket Firewall - #71

Merged
peakematt merged 1 commit into
masterfrom
sfw/route-ci-npm-through-socket-firewall
Sep 11, 2026
Merged

peakematt merged 1 commit into
masterfrom
sfw/route-ci-npm-through-socket-firewall

Conversation

@peakematt

Copy link
Copy Markdown
Contributor

GitHub Actions currently installs npm packages directly in CI. This changes the CI job so npm installs run through Socket Firewall while keeping the existing Node version, npm cache, and validation commands intact.

The workflow uses the public Socket Firewall token and enables the action's validated external-fork fallback for the low-privilege pull request job. Checkout no longer persists credentials, so forked pull request jobs keep read-only GitHub permissions.

Manual verification: open the CI run for this pull request and confirm Socket Firewall setup completes before npm ci, then the normal lint, format, typecheck, and build steps pass. I also validated the workflow syntax and Socket Firewall placement before opening this draft.

@vercel

vercel Bot commented Sep 9, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
explore Ready Ready Preview Sep 9, 2026 12:22am UTC

Request Review

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@greptile-apps

greptile-apps Bot commented Sep 9, 2026

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

The PR appears safe to merge, with no actionable regression identified in the changed workflow.

Summary

  • Adds the pinned workos/setup-socket-firewall action.
  • Supplies the public firewall token and enables external-fork fallback.
  • Preserves lint, formatting, type-checking, and build validation.

@peakematt
peakematt merged commit 0961cd7 into master Sep 11, 2026
10 checks passed
@peakematt
peakematt deleted the sfw/route-ci-npm-through-socket-firewall branch September 11, 2026 14:15

This branch was successfully deployed

1 active deployment
Preview — 49af9b99 Deployed Sep 9, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants