-
Notifications
You must be signed in to change notification settings - Fork 2
feat: add --agent-base-image act for pre-built GitHub Actions parity #396
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Add support for `--agent-base-image act` which uses the pre-built `agent-act` image from GHCR without requiring `--build-local`. Changes: - CLI accepts 'act' as a special value for --agent-base-image - When 'act' is specified, uses ghcr.io/.../agent-act:latest - Custom images still require --build-local (now errors instead of warns) - Added useAgentActImage flag to WrapperConfig - Updated docker-manager to select agent-act image when flag is set - Added tests for new functionality Usage: awf --agent-base-image act --allow-domains github.com -- your-command Closes #395 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
|
| Metric | Base | PR | Delta |
|---|---|---|---|
| Lines | 77.63% | 77.42% | 📉 -0.21% |
| Statements | 77.71% | 77.50% | 📉 -0.21% |
| Functions | 77.54% | 77.54% | ➡️ +0.00% |
| Branches | 70.44% | 70.31% | 📉 -0.13% |
📁 Per-file Coverage Changes (2 files)
| File | Lines (Before → After) | Statements (Before → After) |
|---|---|---|
src/cli.ts |
35.9% → 35.3% (-0.65%) | 36.2% → 35.5% (-0.66%) |
src/docker-manager.ts |
79.3% → 79.4% (+0.06%) | 78.6% → 78.7% (+0.06%) |
Coverage comparison generated by scripts/ci/compare-coverage.ts
|
Smoke Test Results
Status: PASS - @Mossaka
|
Security Review: Supply Chain Validation BypassI've identified a security concern in this PR that introduces a supply chain security validation bypass. Issue: Skipped Validation for
|
Smoke Test Results - Claude EngineLast 2 Merged PRs:
Test Results:
Status: PASS
|
|
Closing PR to reduce backlog. The work is tracked in issue #395 (which was closed as completed). Will revisit if needed. |
Summary
Add support for
--agent-base-image actwhich uses the pre-builtagent-actimage from GHCR without requiring--build-local.Changes
actas a special value for--agent-base-imageactis specified, usesghcr.io/githubnext/gh-aw-firewall/agent-act:latestghcr.io/catthehacker/ubuntu:runner-22.04) still require--build-local(now errors instead of silently warning)useAgentActImageflag toWrapperConfigdocker-managerto selectagent-actimage when flag is setUsage
Test plan
useAgentActImagein docker-managerawf --agent-base-image act --allow-domains github.com -- curl https://github.com--build-localCloses #395
🤖 Generated with Claude Code