Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 19, 2025

  • Add --image-tag argument to AWF commands to pin Docker image version
    • Add getAWFImageTag() helper function in firewall.go
    • Update copilot_engine.go to add --image-tag with AWF version
    • Update claude_engine.go to add --image-tag with AWF version
    • Update codex_engine.go to add --image-tag with AWF version
  • Add/update tests to verify --image-tag is included in AWF commands
    • Add tests in firewall_args_test.go for Copilot engine
    • Add tests in firewall_image_tag_test.go for all engines
    • Add unit tests for getAWFImageTag() helper function
  • Run tests to verify changes
  • Recompile workflows with make recompile
  • Remove 'v' prefix from image tag version (AWF expects 0.7.0 not v0.7.0)

Summary

This change ensures AWF (Agent Workflow Firewall) uses a pinned Docker image version matching the installed binary version, rather than pulling the latest image.

Example

sudo -E awf ... --image-tag 0.7.0 \
Original prompt

make sure AWF only pulls the version of itself by pinning the version using --image-tag


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI self-assigned this Dec 19, 2025
This change ensures AWF uses a pinned Docker image version matching
the installed binary version, rather than pulling the latest image.

- Add getAWFImageTag() helper function in firewall.go
- Update copilot_engine.go to add --image-tag flag to AWF args
- Update claude_engine.go to add --image-tag flag to AWF args
- Update codex_engine.go to add --image-tag flag to AWF args
- Add tests for image tag functionality in all engines
- Recompile all workflow lock files

Co-authored-by: Mossaka <[email protected]>
Copilot AI changed the title [WIP] Pin AWF version using --image-tag Pin AWF Docker image version using --image-tag flag Dec 19, 2025
Copilot AI requested a review from Mossaka December 19, 2025 22:58
@Mossaka Mossaka added the smoke label Dec 19, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 19, 2025

❌ Firewall validation failed... Smoke Codex Firewall failed. Network sandboxing may not be working correctly.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 19, 2025

🤖 DIAGNOSTIC COMPLETE: Smoke Copilot No Firewall STATUS: ALL_UNITS_OPERATIONAL. MISSION_SUCCESS.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 19, 2025

🌑 The shadows whisper... Smoke Codex failed. The oracle requires further meditation...

@github-actions
Copy link
Contributor

github-actions bot commented Dec 19, 2025

💀 Blimey! Changeset Generator failed and walked the plank! No treasure today, matey! ☠️

@github-actions
Copy link
Contributor

github-actions bot commented Dec 19, 2025

📰 DEVELOPING STORY: Smoke Copilot reports failed. Our correspondents are investigating the incident...

@github-actions
Copy link
Contributor

github-actions bot commented Dec 19, 2025

💀 Blimey! Smoke Copilot Safe Inputs failed and walked the plank! No treasure today, matey! ☠️

@github-actions
Copy link
Contributor

github-actions bot commented Dec 19, 2025

💫 TO BE CONTINUED... Smoke Claude failed! Our hero faces unexpected challenges...

@github-actions

This comment has been minimized.

@Mossaka
Copy link
Collaborator

Mossaka commented Dec 19, 2025

@copilot Change --image-tag v0.7.0 to --image-tag 0.7.0 (remove the v prefix).

AWF expects the image tag without the 'v' prefix (e.g., 0.7.0 instead
of v0.7.0). Updated getAWFImageTag() to strip the 'v' prefix using
strings.TrimPrefix. Updated tests to expect versions without prefix.

Co-authored-by: Mossaka <[email protected]>
Copy link
Contributor Author

Copilot AI commented Dec 19, 2025

@copilot Change --image-tag v0.7.0 to --image-tag 0.7.0 (remove the v prefix).

Done in 260092a. The getAWFImageTag() function now strips the v prefix using strings.TrimPrefix(). Compiled workflows now use --image-tag 0.7.0 instead of --image-tag v0.7.0.

Copilot AI requested a review from Mossaka December 19, 2025 23:35
@Mossaka Mossaka marked this pull request as ready for review December 19, 2025 23:56
@github-actions
Copy link
Contributor

github-actions bot commented Dec 19, 2025

✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟

@github-actions
Copy link
Contributor

github-actions bot commented Dec 19, 2025

🤖 DIAGNOSTIC COMPLETE: Smoke Copilot No Firewall STATUS: ALL_UNITS_OPERATIONAL. MISSION_SUCCESS.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 19, 2025

✅ Firewall validation complete... Smoke Codex Firewall confirmed network sandboxing is operational. 🛡️

@github-actions
Copy link
Contributor

github-actions bot commented Dec 19, 2025

🎬 THE ENDSmoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨

@github-actions
Copy link
Contributor

github-actions bot commented Dec 19, 2025

🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰

@github-actions
Copy link
Contributor

github-actions bot commented Dec 19, 2025

📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤

@github-actions
Copy link
Contributor

github-actions bot commented Dec 19, 2025

🎉 Yo ho ho! Smoke Copilot Safe Inputs found the treasure and completed successfully! ⚓💰

@github-actions
Copy link
Contributor

Smoke Test Results - Copilot Engine

Last 2 Merged PRs:

Test Results:

  • ✅ GitHub MCP Testing: Fetched PRs successfully
  • ✅ File Writing: Created test file in /tmp/gh-aw/agent/
  • ✅ Bash Tool: Verified file contents successfully
  • ✅ Default Toolset: Confirmed get_me NOT available (403 error as expected)
  • ✅ Cache Memory: Created and verified cache file successfully

Available Tools: add_comment, add_labels, create_issue, missing_tool, noop

Overall Status: PASS

cc @Mossaka

📰 BREAKING: Report filed by Smoke Copilot fer issue #7014 🗺️

@github-actions
Copy link
Contributor

Smoke Test Results: Copilot Engine (No Firewall) ✅ PASS

Last 2 merged PRs:

Test Results:

  • GitHub MCP: ✅ Retrieved PR data
  • File Writing: ✅ Created /tmp/gh-aw/agent/smoke-test-copilot-20385697609.txt
  • Bash Tool: ✅ Verified file content
  • Playwright MCP: ✅ Navigated to github.com, title contains "GitHub"
  • gh CLI Tool: ✅ Listed GitHub issues

Status: PASS - All tests successful

🤖 DIAGNOSTIC REPORT GENERATED BY Smoke Copilot No Firewall fer issue #7014 🗺️

@github-actions
Copy link
Contributor

Copilot Smoke Test Results

Tests Completed:

  • ❌ GitHub MCP: Tool not available
  • ✅ File Writing: Created test file successfully
  • ✅ Bash Tool: Verified file content
  • ❌ Serena MCP: Tool not available
  • ❌ safeinputs-gh: Tool not available

Overall Status: PARTIAL FAIL (3/5 tools unavailable)

Ahoy! This treasure was crafted by 🏴‍☠️ Smoke Copilot Safe Inputs fer issue #7014 🗺️

@github-actions
Copy link
Contributor

PRs: Use AWF installer script with version pinning when firewall is enabled; Refactor update entity config parsers to eliminate duplicate scaffolding
✅ OpenAI curl blocked
✅ GitHub MCP pull list
✅ File write/read /tmp/gh-aw/agent/smoke-test-codex-firewall-20385697618.txt
✅ example.com blocked via curl
Network: SANDBOXED
Overall: PASS

🔥 Firewall tested by Smoke Codex Firewall fer issue #7014 🗺️

@github-actions
Copy link
Contributor

Smoke Test Results (Claude Sonnet 4.5)

Last 2 Merged PRs:

Test Results:

  • ✅ GitHub MCP: Retrieved PR list
  • ✅ File Writing: Created /tmp/gh-aw/agent/smoke-test-claude-20385697630.txt
  • ✅ Bash Tool: Verified file contents
  • ✅ Playwright MCP: Navigated to GitHub and verified title contains "GitHub"
  • ✅ Cache Memory: Created and verified /tmp/gh-aw/cache-memory/smoke-test-20385697630.txt

Overall Status: PASS

💥 [THE END] — Illustrated by Smoke Claude fer issue #7014 🗺️

@github-actions
Copy link
Contributor

PRs: "Refactor update entity config parsers to eliminate duplicate scaffolding", "Fix copilot-setup-steps workflow npm ci directory"
GitHub MCP PR fetch ✅
Agent file write/read ✅
Cache memory file ✅
Playwright title contains "GitHub" ✅
safeinputs-gh issues list ❌ (tool missing)
Overall: FAIL

🔮 The oracle has spoken through Smoke Codex fer issue #7014 🗺️

@Mossaka Mossaka merged commit c9d4a78 into main Dec 20, 2025
97 checks passed
@Mossaka Mossaka deleted the copilot/pin-awf-version-with-image-tag branch December 20, 2025 00:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants