Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

feat: add hiddenCODER_AGENT_IS_SUB_AGENT flag tocoder agent#17783

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
DanielleMaywood merged 1 commit intomainfromdm-devcontainer-is-sub-agent
May 13, 2025

Conversation

DanielleMaywood
Copy link
Contributor

Closescoder/internal#620

Adds a new, hidden, flagCODER_AGENT_IS_SUB_AGENT to thecoder agent command.

Adds a new, hidden, flag `CODER_AGENT_IS_SUB_AGENT` to the `coder agent`command.
Copy link
Contributor

@CopilotCopilotAI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Pull Request Overview

This PR introduces a new hidden flag, CODER_AGENT_IS_SUB_AGENT, to enable sub-agent behavior within the coder agent command.

  • Added a boolean "subAgent" field to both CLI and agent option structures.
  • Updated the CLI command to include a new hidden flag "is-sub-agent" with related documentation.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

FileDescription
cli/agent.goAdds a new boolean flag and variable for sub-agent support in the CLI command configuration.
agent/agent.goPropagates the new sub-agent option through the agent options and uses it in agent initialization.
Comments suppressed due to low confidence (2)

cli/agent.go:56

  • [nitpick] Consider renaming the variable 'subAgent' to 'isSubAgent' to better reflect its boolean nature and align with the CLI flag naming convention.
subAgent            bool

cli/agent.go:487

  • Please add unit tests to verify that the hidden flag remains hidden in the CLI help output and functions correctly when set.
Flag:        "is-sub-agent",

@DanielleMaywoodDanielleMaywood marked this pull request as ready for reviewMay 13, 2025 09:10
Copy link
Member

@mafredrimafredri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

LGTM 👍

@DanielleMaywoodDanielleMaywood merged commit7f056da intomainMay 13, 2025
38 checks passed
@DanielleMaywoodDanielleMaywood deleted the dm-devcontainer-is-sub-agent branchMay 13, 2025 09:57
@github-actionsgithub-actionsbot locked and limited conversation to collaboratorsMay 13, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.
Reviewers

Copilot code reviewCopilotCopilot left review comments

@mafredrimafredrimafredri approved these changes

Assignees

@DanielleMaywoodDanielleMaywood

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add CLI optionCODER_AGENT_IS_SUB_AGENT tocoder agent command.
2 participants
@DanielleMaywood@mafredri
[8]ページ先頭
©2009-2025 Movatter.jp