Skip to content

docs: fix permission system documentation in agents section - #7652

Merged
rekram1-node merged 2 commits into
anomalyco:devfrom
IdrisGit:update-agents-permission-section
Jan 15, 2026
Merged

docs: fix permission system documentation in agents section#7652
rekram1-node merged 2 commits into
anomalyco:devfrom
IdrisGit:update-agents-permission-section

Conversation

@IdrisGit

Copy link
Copy Markdown
Contributor

What does this PR do?

Updates the agents section in the permissions documentation to reflect the new granular rules system. The previous examples showed mismatched patterns (e.g., "git status" without wildcards) that didn't properly demonstrate how pattern matching works for commands with arguments. The fix adds "grep *" examples to clearly demonstrate when pattern matching is essential, updates the explanatory tip to show both scenarios (commands that work without arguments vs. commands that require pattern matching).

How did you verify your code works?

Ran the docs development server locally to verify the changes render correctly and the examples are properly formatted. The documentation now accurately reflects how the granular permission system works with clear examples showing when to use pattern matching for commands that accept arguments like grep, versus commands that work with basic usage like git status.

@github-actions

Copy link
Copy Markdown
Contributor

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

No duplicate PRs found

```

:::tip
Use pattern matching for commands with arguments. `"grep *"` allows `grep pattern file.txt`, while `"grep"` alone would block it. Commands like `git status` work for default behavior but require explicit permission (like `"git status*"`) when arguments are passed.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

i think u can use git status * instead of git status*

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

removed this one instead of the one in permissions.mdx, as I think it makes more sense there than in agents.mdx.

Only analyze code and suggest changes.
```

:::tip

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

duplicate tip

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

kept this one and removed the duplicate one in agents.mdx

removed it from agents.mdx, as I think it makes more sense in the
permissions page rather than in agents documentation page
@IdrisGit
IdrisGit requested a review from rekram1-node January 15, 2026 01:57
@rekram1-node
rekram1-node merged commit 6b019a1 into anomalyco:dev Jan 15, 2026
3 checks passed
xywsxp pushed a commit to xywsxp/opencode that referenced this pull request Apr 24, 2026
Rwanbt pushed a commit to Rwanbt/unifia that referenced this pull request May 5, 2026
AIALRA-0 pushed a commit to AIALRA-0/opencode-turn-engine that referenced this pull request Jun 10, 2026
AIALRA-0 pushed a commit to AIALRA-0/opencode-turn-engine that referenced this pull request Jun 10, 2026
avion23 pushed a commit to avion23/opencode that referenced this pull request Jun 10, 2026
Rwanbt pushed a commit to Rwanbt/unifia that referenced this pull request Jun 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants