docs(policy-engine): link to tools reference for tool names and args#22081
docs(policy-engine): link to tools reference for tool names and args#22081Aaxhirrr wants to merge 4 commits intogoogle-gemini:mainfrom
Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request enhances the policy engine documentation by providing direct links to the Tools Reference page for Highlights
Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request enhances the policy engine documentation. It introduces direct links to the tools reference page for toolName and argsPattern, which will help users look up tool names and their corresponding arguments. Additionally, it clarifies that commandPrefix and commandRegex are shorthands for creating policy rules, not arguments for the run_shell_command tool itself. The changes are accurate and improve the documentation's clarity. I have no further comments.
Note: Security Review has been skipped due to the limited scope of the PR.
|
This should do it. |
|
@g-samroberts Is there any context on this or could we review? :) |
g-samroberts
left a comment
There was a problem hiding this comment.
Thanks for your contribution, @Aaxhirrr! If you could just add these suggestions, and then do npm run format to make sure the linter will pass, I can merge this in.
|
Thanks, @g-samroberts . I’ve applied the final suggested wording changes and pushed them to this PR. Always happy to contribute. |
|
Build failed: Build process failed with link errors: |
Hi @chrstnb , thank you for flagging this. I think i've found the root cause. That was an oversight on my part. I had actually added the new links in Policy Engine links using Markdown-style relative paths. That looked fine in the repo, but the docs preview/build expects site-route links instead, so those links ended up failing validation there. I’ve now updated those links to use the site route format already used elsewhere in the docs and pushed the fix to this PR. Could you please rerun/recheck the docs build when you get a chance? and lmk if everything looks fine, or if there still is an issue, and ill get right on to it. Thanks again. |
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request updates the docs/reference/policy-engine.md file to improve its clarity and navigability. It adds several internal links to the "Tools reference" document, providing readers with direct access to lists of built-in tool names, argument keys for argsPattern, and details on toolName values and argument structures within the TOML rule schema. Additionally, it clarifies the distinction between policy-rule shorthands and tool invocation arguments for run_shell_command by linking to the "Shell tool" and "Tools reference". A minor correction was also made to the "Customizing Plan Mode Policies" link by removing the .md extension. There is no feedback to provide from the review comments.
Note: Security Review has been skipped due to the limited scope of the PR.
Summary
Update the Policy Engine docs to reference the Tools Reference page for valid tool names and argument structures, instead of duplicating a tool list in the policy page.
Details
./tools.md#available-tools) for validtoolNamevalues.argsPattern.toolNamevalues and argument structures.run_shell_commandthatcommandPrefixandcommandRegexare policy-rule shorthands, notrun_shell_commandinvocation arguments.../tools/shell.md)./tools.md#available-tools)Related Issues
How to Validate
docs/reference/policy-engine.md../tools.md#available-tools../tools.md#available-tools.toolNamevalues and argument structures via the Tools Reference.run_shell_command, verify it explicitly sayscommandPrefix/commandRegexare policy shorthands (not shell tool args) and links to:../tools/shell.md./tools.md#available-toolsPre-Merge Checklist
Just a doc issue.