-
Notifications
You must be signed in to change notification settings - Fork 34
Remove legacy token GH_AW_COPILOT_TOKEN #6045
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
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
📰 BREAKING: Smoke Copilot Playwright is now investigating this pull request. Sources say the story is developing... |
|
💥 WHOOSH! Smoke Claude springs into action on this pull request! [Panel 1 begins...] |
|
🔮 The ancient spirits stir... Smoke Codex awakens to divine this pull request... |
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
|
📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing... |
Copilot Smoke Test ResultsPR Titles (last 2 merged):
Test Results:
Overall Status: PASS
|
Smoke Test ResultsRecent PRs:
Test Results:
Overall: PASS (5/6 tests successful)
|
Smoke Test Results (Run 20107380584)✅ Playwright MCP: Successfully navigated to https://github.com - Page title: "GitHub · Change is constant. GitHub keeps you ahead. · GitHub" Overall Status: PARTIAL PASS - 2/3 tests passed. Safe input gh tool issue is infrastructure-related (HTTP/HTTPS protocol mismatch).
|
Smoke Test ResultsGitHub MCP: ❌ (HTTP/HTTPS protocol error) Overall Status: FAIL 📰🔥📋 https://github.com/githubnext/gh-aw/actions/runs/20107380581 fer issue #6045 🗺️ |
|
Last merged PRs: Fix linting errors and update action pin SHA; Sort safe output tool names for deterministic compilation
|
Removes all support for the
GH_AW_COPILOT_TOKENsecret name. Users must migrate toCOPILOT_GITHUB_TOKEN(recommended) orGH_AW_GITHUB_TOKEN(legacy).Changes
Token fallback chain updated:
COPILOT_GITHUB_TOKEN→GH_AW_COPILOT_TOKEN→GH_AW_GITHUB_TOKENCOPILOT_GITHUB_TOKEN→GH_AW_GITHUB_TOKENCode:
pkg/workflow/github_token.go- Removed fromgetEffectiveCopilotGitHubToken()pkg/workflow/safe_outputs.go- Removed from inline script fallbackpkg/workflow/js/create_agent_task.cjs- Updated error messagepkg/workflow/github_token_test.go- Updated test expectationsDocumentation:
docs/reference/tokens.md- Updated precedence lists and migration guidedocs/reference/safe-outputs.md- Updated safe output token requirementsdocs/reference/engines.md- Updated legacy token noticesskills/gh-agent-task/SKILL.md- Updated token precedenceSpecs:
specs/template-injection-prevention.md- Updated examplespecs/security_review.md- Updated exampleMigration
Context
Follows the pattern established when
COPILOT_CLI_TOKENwas removed in v0.26+. All 106 workflow lock files regenerated with updated token chain.Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.
Changeset
GH_AW_COPILOT_TOKEN. The token precedence now isCOPILOT_GITHUB_TOKEN→GH_AW_GITHUB_TOKEN. See.changeset/minor-remove-legacy-gh-aw-copilot-token.mdfor migration steps.