Skip to content

feat: implement advisor subagent#23436

Open
loss-and-quick wants to merge 1 commit intoanomalyco:devfrom
loss-and-quick:feat/advisor-subagent
Open

feat: implement advisor subagent#23436
loss-and-quick wants to merge 1 commit intoanomalyco:devfrom
loss-and-quick:feat/advisor-subagent

Conversation

@loss-and-quick
Copy link
Copy Markdown

@loss-and-quick loss-and-quick commented Apr 19, 2026

Issue for this PR

Closes #

Type of change

  • New feature

What does this PR do?

Implements advisor as a native subagent for strategic architecture advice, security review, and complex trade-offs.

How it works:

  • Registered conditionally when OPENCODE_EXPERIMENTAL_ADVISOR=1 or experimental.advisor: true
  • Uses standard task tool routing: @advisor <question> triggers task tool with subagent_type: "advisor"
  • Read-only permissions: grep, glob, list, read, bash, webfetch, websearch (no write/edit)

Why this approach:

  • Consistent with other subagents (explore, general)

How did you verify your code works?

# Verify advisor appears in agent list with flag
OPENCODE_EXPERIMENTAL_ADVISOR=1 ./result/bin/opencode agent list | grep advisor

# Verify advisor absent without flag
./result/bin/opencode agent list | grep advisor

# Test advisor invocation
OPENCODE_EXPERIMENTAL_ADVISOR=1 ./result/bin/opencode run "@advisor How to organize modular architecture?"

Nix build succeeds: nix build .#opencode

Screenshots / recordings

N/A - CLI feature

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

Add advisor as a native subagent for architecture decisions, security
review, and complex trade-offs. Enabled via OPENCODE_EXPERIMENTAL_ADVISOR=1
or experimental.advisor: true in config.

Uses standard task tool routing (@advisor mentions) instead of custom tool.
Read-only permissions: grep, glob, list, read, bash, webfetch, websearch.
@github-actions github-actions bot added needs:compliance This means the issue will auto-close after 2 hours. and removed needs:compliance This means the issue will auto-close after 2 hours. labels Apr 19, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Thanks for updating your PR! It now meets our contributing guidelines. 👍

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.

1 participant