Skip to content

Conversation

@spoons-and-mirrors
Copy link
Contributor

@spoons-and-mirrors spoons-and-mirrors commented Jan 11, 2026

What does this PR do?

feat: Adds subagent_tools to the experimental config to allow specifying tools that should only be available to subagents, complementing primary_tools

refactor: moved the primary_tools filtering to the tool registry for consistency (as subagent_tools needs to be constructed at the registry level)

@github-actions
Copy link
Contributor

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@github-actions
Copy link
Contributor

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

I found a potentially related PR that may be relevant to your feature:

Related PR:

This PR addresses subagent tool permissions and inheritance, which is conceptually related to your PR that adds subagent_tools configuration. They may be complementary or address overlapping concerns around controlling which tools are available to subagents.

Otherwise, no other duplicate PRs found with the specific subagent_tools and primary_tools terminology.

@spoons-and-mirrors
Copy link
Contributor Author

unrelated

@spoons-and-mirrors spoons-and-mirrors marked this pull request as draft January 26, 2026 03:21
@spoons-and-mirrors
Copy link
Contributor Author

spoons-and-mirrors commented Jan 26, 2026

Reworking the PR, added session-level permission handling for subagent tool visibility. Also makes this supersede #7473 - i think this is a win-win ?

@spoons-and-mirrors spoons-and-mirrors changed the title feat(config): add subagent_tools to complement primary_tools feat(config): add subagent_tools to complement primary_tools and unifies tool visibility under permission system Jan 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant