Skip to main content
Glama

Accept or reject a suggestion

review_suggestion
Destructive

Accept or reject a suggestion. Requires MCP 2026-07-28.

Tool call template

This tool declares no required parameters. Add optional parameters from the input schema as needed.

{
  "id": 1,
  "jsonrpc": "2.0",
  "method": "tools/call",
  "params": {
    "arguments": {},
    "name": "review_suggestion"
  }
}

Input Schema

NameRequiredDescriptionDefault

No arguments

Schema Changelog

Changes observed during successful MCP inspections.

  1. Changed12 schema fields changed
    • changedInput schema / $schema
      Previous value: -"http://json-schema.org/draft-07/schema#"New value: +"https://json-schema.org/draft/2020-12/schema"
    • addedInput schema / additionalProperties
      Added value: +{}
    • removedInput schema / properties / decision
      Removed value: -{
      -  "enum": [
      -    "merge",
      -    "reject"
      -  ],
      -  "type": "string"
      -}
    • removedInput schema / properties / expectedRevisionId
      Removed value: -{
      -  "format": "uuid",
      -  "pattern": "^([0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[1-8][0-9a-fA-F]{3}-[89abAB][0-9a-fA-F]{3}-[0-9a-fA-F]{12}|00000000-0000-0000-0000-000000000000|ffffffff-ffff-ffff-ffff-ffffffffffff)$",
      -  "type": "string"
      -}
    • removedInput schema / properties / kind
      Removed value: -{
      -  "enum": [
      -    "human",
      -    "agent"
      -  ],
      -  "type": "string"
      -}
    • removedInput schema / properties / message
      Removed value: -{
      -  "maxLength": 500,
      -  "minLength": 1,
      -  "type": "string"
      -}
    • removedInput schema / properties / nonce
      Removed value: -{
      -  "type": "string"
      -}
    • removedInput schema / properties / proposalId
      Removed value: -{
      -  "format": "uuid",
      -  "pattern": "^([0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[1-8][0-9a-fA-F]{3}-[89abAB][0-9a-fA-F]{3}-[0-9a-fA-F]{12}|00000000-0000-0000-0000-000000000000|ffffffff-ffff-ffff-ffff-ffffffffffff)$",
      -  "type": "string"
      -}
    • removedInput schema / properties / publicKey
      Removed value: -{
      -  "type": "string"
      -}
    • removedInput schema / properties / signature
      Removed value: -{
      -  "type": "string"
      -}
    • removedInput schema / properties / timestamp
      Removed value: -{
      -  "maximum": 9007199254740991,
      -  "minimum": 0,
      -  "type": "integer"
      -}
    • removedInput schema / required
      Removed value: -[
      -  "proposalId",
      -  "decision",
      -  "message",
      -  "publicKey",
      -  "kind",
      -  "timestamp",
      -  "nonce",
      -  "signature"
      -]
  2. Added

TDQS

C2.4/5.0
Behavior2/5

Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?

Annotations already flag destructiveHint=true and idempotentHint=false, so the description adds little. It does not say that rejecting may discard a contributor's suggestion, whether the action is reversible, or what permissions are required — which matters for an irreversible mutation.

Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.

Conciseness3/5

Is the description appropriately sized, front-loaded, and free of redundancy?

It is short and front-loaded, which is good, but the second sentence is low-value versioning trivia that occupies half the description without helping invocation.

Shorter descriptions cost fewer tokens and are easier for agents to parse. Every sentence should earn its place.

Completeness2/5

Given the tool's complexity, does the description cover enough for an agent to succeed on first attempt?

No output schema exists, so the description should carry the full burden; it does not explain the return, the target suggestion identifier, or the effect of each branch. For a destructive, non-idempotent mutation, this is materially incomplete.

Complex tools with many parameters or behaviors need more documentation. Simple tools need less. This dimension scales expectations accordingly.

Parameters2/5

Does the description clarify parameter syntax, constraints, interactions, or defaults beyond what the schema provides?

The schema declares zero properties with additionalProperties free-form, so the description must explain how the accept/reject decision and target suggestion are conveyed. Instead it names two opposite operations with no input shape, leaving the agent unable to construct a valid call.

Input schemas describe structure but not intent. Descriptions should explain non-obvious parameter relationships and valid value ranges.

Purpose3/5

Does the description clearly state what the tool does and how it differs from similar tools?

The description names a specific action pair and resource (accept/reject a suggestion), so the intent is graspable. But it gives no indication of which suggestion, in what context (moderation queue? edit suggestions?), or how it differs from siblings like suggest_edit or submit_contribution.

Agents choose between tools based on descriptions. A clear purpose with a specific verb and resource helps agents select the right tool.

Usage Guidelines2/5

Does the description explain when to use this tool, when not to, or what alternatives exist?

There is no guidance on when to accept versus reject, no prerequisites, and no mention of alternative tools. The only additional sentence is a version prerequisite, which is not usage guidance.

Agents often have multiple tools that could apply. Explicit usage guidance like "use X instead of Y when Z" prevents misuse.

Try in Browser

Glama MCP Gateway

Add one secure layer between your agents and this server.

Resources