-
Notifications
You must be signed in to change notification settings - Fork 34
Use install-awmg.sh script for MCP gateway binary downloads in release mode #9079
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
|
@copilot generateMCPGatewayDownloadStep should generate specialized shell script code based on the release mode. It should change the generated code base on the mode. |
Co-authored-by: pelikhan <[email protected]>
Co-authored-by: pelikhan <[email protected]>
The release tag should be detected at runtime in the shell script based on GITHUB_REF, not at compile time in Go using GetCurrentGitTag(). This ensures that the same compiled workflow can adapt to different release contexts when it actually runs. Co-authored-by: pelikhan <[email protected]>
Fixed in 3fab956. The function now generates shell script code that detects the release tag at runtime using |
|
@copilot use the release tag stored in the cli GO code itself, not GITHUB_REF. Use the helper to determine if isRelease |
The release tag is now determined from the CLI's embedded version (GetVersion() and IsRelease()) rather than detecting it from GITHUB_REF at runtime. This ensures that workflows compiled with gh-aw v1.2.3 will download awmg v1.2.3, maintaining version consistency between the CLI and MCP gateway binary. Co-authored-by: pelikhan <[email protected]>
Fixed in 6a8a36a. The function now uses |
|
@copilot check if awmg is already available before using the install script. |
Added a check for $HOME/.local/bin/awmg before downloading the install-awmg.sh script. If awmg is already installed at that location, the workflow will skip the download and use the existing binary, saving time and bandwidth. Co-authored-by: pelikhan <[email protected]>
|
📰 VERDICT: Smoke Copilot Playwright has concluded. All systems operational. This is a developing story. 🎤 |
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
|
🎉 Yo ho ho! Smoke Copilot Safe Inputs found the treasure and completed successfully! ⚓💰 |
|
✅ Firewall validation complete... Smoke Codex Firewall confirmed network sandboxing is operational. 🛡️ |
Smoke Test Results✅ File Writing Test: Successfully created Overall Status: Note: safeinputs-gh tool may need to be configured in workflow definition
|
Smoke Test Results - Copilot EngineLast 2 merged PRs:
✅ GitHub MCP Testing - Retrieved PR data successfully Overall Status: PASS cc: @pelikhan
|
|
PRs: [WIP] Split daily-news shell script into multiple steps; Replace manual prompt handling with console.ConfirmAction in trial command
|
Smoke Test Results
Status: FAIL Details: Playwright module not found, safe-input gh tool not configured in workflow.
|
Claude Smoke Test ResultsLast 2 Merged PRs:
Test Results: Overall Status: PASS (5/6 tests successful, 1 test N/A)
|
|
Smoke test results:
|
The MCP gateway download step in release mode now uses the
install-awmg.shscript from the main branch instead of manually implementing platform detection and gh CLI downloads.Changes
Modified
pkg/workflow/gateway.go:install-awmg.shfrom main branch via curlGetVersion()andIsRelease()helpers to determine which version to download$HOME/.local/bin/awmgbefore downloading to avoid unnecessary downloadsAWMG_CMDto$HOME/.local/bin/awmgafter installationKey Implementation
The function checks for existing awmg installations before downloading:
The version selection uses the CLI's embedded Go code:
Examples
When compiled with gh-aw v1.2.3 (release build):
# Download awmg for CLI release version: v1.2.3 /tmp/install-awmg.sh v1.2.3When compiled with gh-aw dev build:
# Download latest awmg release (CLI is not a release build) /tmp/install-awmg.shThis approach ensures:
This centralizes installation logic in the script rather than duplicating platform detection and download logic in the compiled workflow.
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.
Changeset
install-awmg.shfrommainto download theawmgMCP gateway binary in release mode; checks for existing installations before downloading and passes the CLI release version when available to ensure version consistency.