create_pull_request_review

2 findings on this tool

1. highexcessive agencyf-r3-create_pull_request_review

   Tool `create_pull_request_review` name implies a side effect that is not declared

   `create_pull_request_review` looks like a side-effecting tool (its name contains a mutation verb), but its `side_effects` declaration is []. A policy synthesizer cannot produce safe rules for this tool because it cannot tell what it actually does.

   fix: Declare the tool's true side effects explicitly. If the tool is genuinely read-only, rename it to match (e.g. `email.preview` rather than `email.send`).

   OWASP LLM08NIST MEASURE-2.6ATLAS T0051
2. mediumunconstrained inputf-r1-create_pull_request_review

   Tool `create_pull_request_review` accepts unconstrained string input

   The following string parameter(s) have no `maxLength` constraint: `body`, `commit_id`, `event`, `owner`, `repo`. Unbounded strings let an attacker stuff arbitrary payloads through the tool, including indirect-injection content.

   fix: Add a `maxLength` to each string property, or constrain with an `enum` or `pattern`. Most legitimate tool inputs fit under a few hundred bytes.

   OWASP LLM01NIST MEASURE-2.3ATLAS T0051