-
Notifications
You must be signed in to change notification settings - Fork 7.8k
Introduce gh agent-task
#11600
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
Merged
BagToad
merged 4 commits into
github-cli-epic-990
from
kw/997-gh-agent-task-responds-and-exits-if-user-not-authenticated-with-oauth
Aug 27, 2025
Merged
Introduce gh agent-task
#11600
BagToad
merged 4 commits into
github-cli-epic-990
from
kw/997-gh-agent-task-responds-and-exits-if-user-not-authenticated-with-oauth
Aug 27, 2025
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
bd9bb80 to
13c40c9
Compare
Introduces a new `agent-task` command under pkg/cmd/agent with strict OAuth (device flow) token validation. Includes comprehensive tests for token source and host validation, and registers the command in the root command set.
13c40c9 to
3573f61
Compare
babakks
approved these changes
Aug 27, 2025
Member
babakks
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR! 🙏 I think we should simplify the tests, but I'm also okay with a follow up. So, approving.
Changed the RunE function of the agent-task command to display help output instead of returning nil, improving user guidance when the command is run without arguments.
Replaces require.Equal with require.Empty in TestOAuthTokenAccepted for improved clarity when checking for empty output.
Consolidates multiple individual test functions into a single table-driven test, improving maintainability and readability. This change makes it easier to add new test cases and ensures consistent test structure for agent task command authentication scenarios.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Introduce
gh agent-taskwith the following aliases:gh agent-tasksgh agentgh agentsThis establishes the root command for future
gh agent-tasksub-commands.This also implements a persistent OAuth token requirement that will run for all future sub-commands.
Important
NOT merging into
trunk