-
Notifications
You must be signed in to change notification settings - Fork 36
Clarify Copilot agent assignment methods in documentation #11122
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 review official GitHub documentation as well. Cross reference |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
@copilot, review the implementation of create-issue and ensure that it still uses the same token. It appears to create Copilot specially and may use GraphQL. |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Cross-referenced with official GitHub documentation and updated all terminology to match. Changes in commit 6507eae: Official Docs Reviewed: Key Updates:
All documentation now uses consistent terminology with official GitHub docs while explaining both the UI workflow (manual assignment) and the programmatic workflow methods ( 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:
If you need me to access, download, or install something from one of these locations, you can either:
|
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Reviewed the implementation and corrected the documentation. Key findings in commit eb3ebdb: Both methods use the SAME token and GraphQL API:
Token precedence for both: Documentation now accurately reflects that both methods use identical token chain and GraphQL implementation. 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:
If you need me to access, download, or install something from one of these locations, you can either:
|
Documentation used ambiguous
@copilotsyntax without distinguishing between two distinct assignment mechanisms. After code review, corrected critical documentation errors about token usage and GraphQL implementation.Changes
Terminology corrections
@copilotGitHub mention syntax with accurate method references inresearchplanassign.mdand blog postCross-references to official GitHub documentation
tokens.md: Enhanced with links to official Copilot coding agent docssafe-outputs.md: Added notes that both methods mirror standard GitHub UI assignment workflowCorrected token and GraphQL implementation documentation
GH_AW_AGENT_TOKEN(NOT different tokens as previously documented)replaceActorsForAssignablemutation viaassign_agent_helpers.cjsassignees: copilotin create-issue filters copilot and assigns it in a separate GraphQL steptokens.mdandsafe-outputs.mdwith accurate token informationTwo Assignment Methods (Corrected)
Both methods use the same token (
GH_AW_AGENT_TOKEN) and same GraphQL API:Method 1:
create-issue: assignees: copilotGH_AW_AGENT_TOKENMethod 2:
assign-to-agent:safe outputToken precedence for both:
GH_AW_AGENT_TOKEN || GH_AW_GITHUB_TOKEN || GITHUB_TOKENBoth methods use the same underlying implementation (
replaceActorsForAssignableGraphQL mutation) and result in the same outcome as manually assigning issues to Copilot through the GitHub UI.Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.