fix: add --ignore-scripts to security-guard Claude Code install#2963
Conversation
The security-guard.lock.yml was missing --ignore-scripts on the npm install command for Claude Code CLI, causing the workflow-engine-install-security test to fail. Closes #2959 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
✅ Coverage Check PassedOverall Coverage
📁 Per-file Coverage Changes (1 files)
Coverage comparison generated by |
|
Smoke Test Results
Status: PARTIAL (3/4 tests passed - gh auth expected in this context)
|
Smoke Test: Copilot BYOK (Offline) Mode
Running in BYOK offline mode ( Overall: FAIL — template variables not resolved; pre-step data unavailable for verification.
|
🔬 Smoke Test Results — PR #2963 (branch:
|
| Test | Result |
|---|---|
| GitHub MCP connectivity | ❌ 401 Bad credentials |
| GitHub.com HTTP connectivity | |
File write/read (/tmp/gh-aw/agent/smoke-test-copilot-25705487680.txt) |
✅ File exists and readable |
Overall: FAIL — GitHub MCP returned 401; pre-step smoke data template variables were not substituted.
📰 BREAKING: Report filed by Smoke Copilot
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Pull request overview
Updates the Security Guard locked workflow to install the Claude Code CLI with npm --ignore-scripts, aligning with the repository’s engine-install security test and unblocking the failing workflow-engine-install-security check that was breaking the Test Coverage Reporter on main.
Changes:
- Add
--ignore-scriptsto the globalnpm installcommand for@anthropic-ai/claude-codeinsecurity-guard.lock.yml.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/security-guard.lock.yml | Adds --ignore-scripts to the Claude Code CLI install step in the Security Guard locked workflow. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 1/1 changed files
- Comments generated: 1
| run: bash "${RUNNER_TEMP}/gh-aw/actions/install_awf_binary.sh" v0.25.41 | ||
| - name: Install Claude Code CLI | ||
| run: npm install -g @anthropic-ai/claude-code@2.1.126 | ||
| run: npm install --ignore-scripts -g @anthropic-ai/claude-code@2.1.126 |
🏗️ Build Test Suite Results
Overall: 8/8 ecosystems passed — ✅ PASS
|
Smoke Test Results — FAIL
|
|
Smoke Codex: FAIL Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "registry.npmjs.org"See Network Configuration for more information.
|
The
security-guard.lock.ymlwas missing--ignore-scriptson thenpm installcommand for Claude Code CLI, causing theworkflow-engine-install-securitytest to fail on main.This was the root cause of the Test Coverage Reporter failure (#2959) — the coverage workflow runs
npm testagainst main and this test was failing.Closes #2959