Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 27, 2025

When AWF firewall is enabled for Claude engine, the claude CLI isn't accessible inside the container because npm global binaries aren't mounted.

Changes

  • Use npx -y @anthropic-ai/claude-code@version when firewall is enabled, matching Copilot's pattern
  • Keep direct claude invocation for non-firewall mode (global npm install)
if isFirewallEnabled(workflowData) {
    // npx handles package resolution inside container
    commandParts := []string{fmt.Sprintf("npx -y @anthropic-ai/claude-code@%s", claudeVersion)}
} else {
    // Use globally installed claude
    commandParts := []string{"claude"}
}

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Nov 27, 2025

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

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/user
    • Triggering command: /usr/bin/gh gh api user --jq .login (http block)

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

Copilot AI changed the title [WIP] Fix firewall issues on Claude engine and update AWF fix: use npx to run Claude CLI when AWF firewall is enabled Nov 27, 2025
Copilot AI requested a review from Mossaka November 27, 2025 01:09
Copilot finished work on behalf of Mossaka November 27, 2025 01:09
@Mossaka Mossaka marked this pull request as ready for review November 28, 2025 00:12
@Mossaka Mossaka merged commit d32156c into firewall-claude Nov 28, 2025
@Mossaka Mossaka deleted the copilot/sub-pr-4792-again branch November 28, 2025 00:12
Mossaka added a commit that referenced this pull request Nov 28, 2025
* Initial plan

* fix: use npx to run Claude CLI when AWF firewall is enabled

Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants