Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 19, 2025

AWF-enabled workflows should install via the official one-liner installer script instead of manual binary download, while preserving explicit version pinning.

  • Implementation
    • Switch AWF installation step to run the upstream installer one-liner, passing AWF_VERSION so requested versions remain pinned and logged.
    • Keep custom AWF commands bypassing installation unchanged.
  • Tests
    • Update AWF installation tests to assert installer usage and version logging/pinning.

Example installation step:

- name: Install awf binary
  run: |
    echo "Installing awf via installer script (requested version: v0.7.0)"
    curl -sSL https://raw.githubusercontent.com/githubnext/gh-aw-firewall/main/install.sh | sudo AWF_VERSION=v0.7.0 bash
    which awf
    awf --version
Original prompt

Use this one liner installation to install awf (if awf is enabled)

curl -sSL https://raw.githubusercontent.com/githubnext/gh-aw-firewall/main/install.sh | sudo bash


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Install awf using one liner script Use AWF installer script when firewall is enabled Dec 19, 2025
Copilot AI requested a review from Mossaka December 19, 2025 08:28
@Mossaka Mossaka added the smoke label Dec 19, 2025
@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! Changeset Generator found the treasure and completed successfully! ⚓💰

@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

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 ENDSmoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨

@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. 🌟

@Mossaka
Copy link
Collaborator

Mossaka commented Dec 19, 2025

the downside of this is that it won't be able to pin the version anymore...

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@pelikhan
Copy link
Contributor

The install script should support a version number as argument

@pelikhan
Copy link
Contributor

@Mossaka the installer script should support pre-downloading the docker image so that we don't get the docker download output in the agent console output.

@Mossaka
Copy link
Collaborator

Mossaka commented Dec 19, 2025

@copilot please apply version pinning:

usage

# Install specific version via argument
curl -sSL .../install.sh | sudo bash -s -- v1.0.0

# Or via environment variable
curl -sSL .../install.sh | sudo AWF_VERSION=v1.0.0 bash

# Default: latest (unchanged behavior)
curl -sSL .../install.sh | sudo bash

Copy link
Contributor Author

Copilot AI commented Dec 19, 2025

the downside of this is that it won't be able to pin the version anymore...

Implemented AWF_VERSION pinning through the installer call; workflows now install the requested version via AWF_VERSION, keeping determinism. Commit 8309477.

The install script should support a version number as argument

Installer call now passes the version via AWF_VERSION (and logs it) so a specific AWF release can be installed using the one-liner. Commit 8309477.

@copilot please apply version pinning:

usage
...

Version pinning applied: the installer is invoked with AWF_VERSION set to the requested version, matching the documented usage you shared. Commit 8309477.

@Mossaka the installer script should support pre-downloading the docker image so that we don't get the docker download output in the agent console output.

Can you share the exact image name/flag you want pre-downloaded? The current installer doesn’t pull a container image, so I need those details to wire in a pre-pull step without guessing.

Copilot AI changed the title Use AWF installer script when firewall is enabled Use AWF installer script with version pinning when firewall is enabled Dec 19, 2025
@Mossaka
Copy link
Collaborator

Mossaka commented Dec 19, 2025

Copy link
Contributor Author

Copilot AI commented Dec 19, 2025

@copilot investgiate why https://github.com/githubnext/gh-aw/actions/runs/20383542822/job/58579634999?pr=6925 failed

Looked at run 20383542822: only job copilot-setup-steps failed at step “Install JavaScript dependencies” (npm ci); all earlier steps passed and later steps were skipped. I can’t fetch the job logs from Actions (download URL returned a non-zip/403), but npm ci succeeds locally on this branch with Node 24. Likely a transient npm/network hiccup—can you rerun that workflow to confirm?

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/githubnext/gh-aw/actions/jobs/58579634999/logs
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (http block)

If you need me to access, download, or install something from one of these locations, you can either:

@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

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

📰 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

✨ 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

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

@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

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

@github-actions
Copy link
Contributor

PRs: #7009 Enable strict mode and awf firewall for all agentic workflows; #7013 Fix copilot-setup-steps workflow npm ci directory
OpenAI curl blocked ✅
GitHub MCP merged PR fetch ✅
File write/read ✅
example.com blocked ✅
Network: SANDBOXED
Overall: PASS

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

@github-actions
Copy link
Contributor

Copilot Smoke Test Results

Last 2 merged PRs:

Test Results:

  • ✅ GitHub MCP: Retrieved 2 merged PRs
  • ✅ File Writing: Created /tmp/gh-aw/agent/smoke-test-copilot-20385263400.txt
  • ✅ Bash Tool: Verified file contents
  • ✅ GitHub MCP get_me tool: Confirmed NOT available with default toolsets (expected 403)
  • ✅ Cache Memory: Created /tmp/gh-aw/cache-memory/smoke-test-20385263400.txt

Available Tools: add_comment, add_labels, create_issue, missing_tool, noop, github-list_pull_requests, github-pull_request_read, github-issue_read, github-list_issues, bash, view, create, edit, grep, glob

Overall Status: PASS

cc: @Mossaka

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

@github-actions
Copy link
Contributor

Smoke Test Results

Last 2 Merged PRs:

Test Results:

  • ✅ GitHub MCP - Retrieved PR data
  • ✅ File Writing - Created test file successfully
  • ✅ Bash Tool - Verified file contents
  • ✅ Playwright MCP - Navigated to github.com, title contains "GitHub"
  • ❌ Safe Input gh Tool - Tool not available (missing_tool will be reported)

Status: PARTIAL PASS (4/5 tests passed)

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

@github-actions
Copy link
Contributor

Smoke Test Results

Last 2 Merged PRs: Unable to access (GitHub MCP not available in this environment)
File Writing: ✅ Created /tmp/gh-aw/agent/smoke-test-copilot-20385263412.txt
Bash Tool: ✅ Verified file content successfully
Serena MCP: Unable to test (Serena MCP not available in this environment)
Safe Input gh: Unable to test (safeinputs-gh tool not available)

Overall Status: PARTIAL (3 of 5 tests could not run due to tool availability)

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

@github-actions
Copy link
Contributor

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

@github-actions
Copy link
Contributor

Smoke Test Results (Claude Engine)

Recent PRs:

Test Results:

  • ✅ GitHub MCP: Fetched 2 recent merged PRs
  • ✅ File Writing: Created /tmp/gh-aw/agent/smoke-test-claude-20385263397.txt
  • ✅ Bash Tool: Verified file creation successful
  • ✅ Playwright MCP: Navigated to GitHub, title contains "GitHub"
  • ✅ Cache Memory: Created and verified /tmp/gh-aw/cache-memory/smoke-test-20385263397.txt
  • ❌ Safe Input gh Tool: Tool not available for issue listing (expected limitation)

Overall Status: PASS

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

@Mossaka Mossaka merged commit 885f961 into main Dec 19, 2025
97 checks passed
@Mossaka Mossaka deleted the copilot/install-awf-if-enabled branch December 19, 2025 23:56
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.

3 participants