Skip to content

feat(acp): include subagents in available modes response#12305

Open
YichenLi00 wants to merge 1 commit intoanomalyco:devfrom
YichenLi00:feat/acp-subagent-support
Open

feat(acp): include subagents in available modes response#12305
YichenLi00 wants to merge 1 commit intoanomalyco:devfrom
YichenLi00:feat/acp-subagent-support

Conversation

@YichenLi00
Copy link

Summary

  • Include subagents in ACP available modes response

Closes #11576

Changes

Remove subagent filter from loadAvailableModes and add mode field to response, letting clients decide what to display.

- .filter((agent) => agent.mode !== "subagent" && !agent.hidden)
+ .filter((agent) => !agent.hidden)
  .map((agent) => ({
    id: agent.name,
    name: agent.name,
    description: agent.description,
+   mode: agent.mode,
  }))

Backward Compatibility

Same data shape, just with more entries and an additional mode field. Existing clients can filter by mode if needed.

- Remove subagent filter from loadAvailableModes
- Add mode field to response for client-side filtering
- Maintains backward compatibility (same data shape, more entries)
@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

The following comment was made by an LLM, it may be inaccurate:

No duplicate PRs found

id: agent.name,
name: agent.name,
description: agent.description,
mode: agent.mode,
Copy link
Collaborator

Choose a reason for hiding this comment

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

ModeOption doesnt even have a mode field

also I dont understand, why would a subagent be in the selectable list. That seems inconsistent

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE]: Support subagent information in ACP

2 participants