Skip to content

Conversation

@dsyme
Copy link
Contributor

@dsyme dsyme commented Jan 27, 2026

Summary

  • Introduced new RunGH and RunGHCombined methods for executing GitHub CLI commands with interactive spinners
  • Updated multiple files to use the new command execution methods
  • Improved logging and user feedback during GitHub CLI command execution

Key Changes

  • Added spinner support for CLI commands in interactive terminals
  • Simplified command execution across various workflow-related functions
  • Enhanced error handling and output capturing for GitHub CLI commands

Context

These changes improve the user experience by providing visual feedback during network operations and standardizing GitHub CLI command execution across the project.

@dsyme dsyme merged commit b45ffcc into main Jan 27, 2026
84 checks passed
@dsyme dsyme deleted the twe branch January 27, 2026 03:03
Mossaka pushed a commit that referenced this pull request Jan 27, 2026
Mossaka added a commit that referenced this pull request Jan 27, 2026
#11969)

* Initial plan

* Investigate firewall 0 requests issue

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

* Fix AWF command quoting to ensure agent runs inside firewall container

The compound command after -- was not properly quoted, causing the &&
operator to be interpreted by the outer shell. This resulted in:
1. AWF container running only the PATH export (which exits immediately)
2. Copilot agent running on the host AFTER the firewall container stopped
3. Firewall reporting 0 requests

Now the entire command is wrapped in single quotes so it's passed to AWF
as a single argument, ensuring Copilot runs inside the firewall container.

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

* Add changeset [skip-ci]

* 🔧 Improve interactive workflow add experience (#11966)

* interactive add

* interactive add

* fix lint

* review

* fix quick start

* fix quick start

* fix line break

* Add upgrade, import, and debug sections to agentic authoring docs (#11987)

* various minor tweaks to ux (#11988)

* Fix release workflow: Clarify MCP tool call vs bash file operations (#11991)

* chore: rm schedule triggers from security alert burndown campaign workers (#11993)

* Merge main and regenerate workflows

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Add changeset [skip-ci]

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
Co-authored-by: Jiaxiao Zhou <duibao55328@gmail.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Don Syme <dsyme@users.noreply.github.com>
Co-authored-by: Don Syme <dsyme@github.com>
Co-authored-by: Claude Opus 4.5 <noreply@anthropic.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