Skip to content

docs: mark tools config as deprecated#17951

Merged
rekram1-node merged 6 commits intoanomalyco:devfrom
OpeOginni:fix/inherit-top-level-permission
Mar 17, 2026
Merged

docs: mark tools config as deprecated#17951
rekram1-node merged 6 commits intoanomalyco:devfrom
OpeOginni:fix/inherit-top-level-permission

Conversation

@OpeOginni
Copy link
Contributor

@OpeOginni OpeOginni commented Mar 17, 2026

Issue for this PR

Closes ##16331 (comment)

Related to conversations in #16331 (comment)

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

It adds support for extended permissions when users make use of the legacy tools feild when setting up agents. Alerting them that its better to use permissions for more fine-grained control. Also adding the logic that takes permissions set in the top layer to be applied to these tools set.

So if a permission is set for the edit tool, and the user sets this tool to true in the agent, it make use of the permission from above instead of just an allow.

I also added tests to show this addition works in different cases.

How did you verify your code works?

Tests pass

Screenshots / recordings

Screenshot 2026-03-17 at 16 05 33

Checklist

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

@OpeOginni OpeOginni changed the title feat(opencode): Inherit top level permissions when using legacy tools setting of agents docs(agents): Display tools config as depricated Mar 17, 2026
@rekram1-node rekram1-node changed the title docs(agents): Display tools config as depricated docs: mark tools config as deprecated Mar 17, 2026
@rekram1-node rekram1-node merged commit 6708c3f into anomalyco:dev Mar 17, 2026
10 checks passed
AvatarGanymede pushed a commit to AvatarGanymede/opencode-dev that referenced this pull request Mar 19, 2026
demostanis pushed a commit to demostanis/opencode that referenced this pull request Mar 19, 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.

2 participants