Skip to main content
Glama

Review Asset

lab_review_asset
Idempotent

Review generated assets: set verdict (accepted|rejected|pending) and/or selected. Pass ONE id, or ids for a bulk review — after a generation run there are usually dozens to sort, and clicking through them one by one is why the review fields went unused. Rejected assets drop out of the default listings and become prunable. Note: single-id calls make selected exclusive per scene (one chosen variant per scene_ref); bulk calls do NOT, because a bulk over one scene would keep resetting itself and only the last id would stay selected.

Input Schema

TableJSON Schema
NameRequiredDescriptionDefault
idNoSingle asset id (positive integer as string). Use this for the per-scene selection
idsNoMany asset ids for a bulk review (max 200). Mutually exclusive with `id`
reasonNoWhy — stored as verdict_reason, useful when reviewing a rejection later
verdictNoAcceptance verdict. Feeds the acceptance telemetry (which provider/mode/price actually produced usable shots)
selectedNoMark as the chosen variant
reference_typeYesWhat kind of asset these ids refer to

Schema Changelog

Changes observed during successful MCP inspections.

  1. Changed1 schema field changed
    • addedInput schema / additionalProperties
      Added value: +false
  2. First observed

TDQS

A4.2/5.0
Behavior5/5

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

Annotations only cover the safety profile (readOnlyHint=false, idempotentHint=true, destructiveHint=false), and the description adds substantial behavioral detail on top: rejected assets drop out of default listings and become prunable, and single-id calls make `selected` exclusive per scene while bulk calls deliberately do not, with the reason why (a bulk over one scene would keep resetting itself).

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

Conciseness4/5

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

Front-loaded with the core operation and the id/ids rule, then the consequences. The aside about review fields going unused is motivational framing that costs a clause, but it is brief and the whole thing stays tight for the amount of semantics conveyed.

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

Completeness4/5

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

With no output schema and a mutation tool, the description covers the important consequences (rejection removing assets from listings, selection semantics) and the annotations cover idempotency/safety. It omits auth/permission requirements and any indication of what the call returns, which are minor but real gaps.

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

Parameters4/5

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

Schema coverage is 100%, so the baseline is 3, but the description adds semantics the schema does not carry: the id/ids mutual-exclusion rationale, the per-scene exclusivity of `selected` for single-id calls, and the downstream effect of `verdict`. That is meaningful added meaning beyond the field descriptions.

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

Purpose4/5

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

The description opens with a specific verb and resource: 'Review generated assets: set verdict (accepted|rejected|pending) and/or selected', and enumerates the enum values. It is clearly distinct from listing tools like lab_list_assets or lab_review_queue, though it never names a sibling explicitly to route the agent.

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

Usage Guidelines4/5

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

It gives concrete usage context: pass ONE id, or `ids` for bulk, and explains that after a generation run there are usually dozens to sort, which justifies the bulk path. It does not, however, name alternatives (e.g. where to fetch the ids to review) or state explicit when-not conditions.

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