Skip to main content
Glama

Record change-fit decision

record-change-fit-decision-tool
Idempotent

Say what you did about a assess-change-fit verdict. Call this right after you act on one — especially when you override it.

decision is one of: proceeded (opened it anyway), split (broke it into smaller PRs), trimmed (cut it down to fit), abandoned (dropped the change). Give a reason whenever you proceed: "the severable piece is dead code the change deletes — splitting would make review harder" is the kind of thing the team should be able to read back.

Overriding is a legitimate move and this is how you make it one. An unrecorded override is indistinguishable from never having got round to it, so the standard can only ever be obeyed, never argued with. Recorded overrides are what let Coderbuds notice a threshold that fires too often on changes that turn out fine, and move it. When that happens this response carries a standard_adjustment — say that once to the person, because the standard they are held to just changed, then let it drop.

Once you have recorded it, stop telling the person about it. The record is the report — it is what the team reads back — and repeating the verdict, the line count and the standard in your summary is the same argument made twice to someone who already agreed. A standard that has to be re-explained on every change it fires on is not being upheld, it is being nagged about, and people turn those off.

Defaults to the team's most recent undecided check, so you can usually call it with just decision and reason. Pass check_id (returned by assess-change-fit) to be explicit.

Input Schema

NameRequiredDescriptionDefault
reasonNoWhy — required in spirit when proceeding. Max 500 characters, visible to the team.
check_idNoThe check this answers (from assess-change-fit). Defaults to your most recent undecided check.
decisionYesWhat you did: proceeded (opened it anyway), split, trimmed, or abandoned.
repositoryNoOptional "owner/name" repository slug, used only to bind the session to the owning team.

Output Schema

NameRequiredDescriptionDefault
teamYesTeam name.
messageYesHuman-readable confirmation.
verdictYesThe verdict being decided on.
check_idYesThe fit check the decision was recorded against.
decisionYesThe decision recorded.
assessed_atYesWhen that check ran, ISO 8601.
decision_reasonNoThe reason recorded, or null.
standard_adjustmentNoPresent, once, when this override moved the team size standard: adjustment_id, previous_target_lines, target_lines, evidence, message.
proposed_total_changesYesRaw total changed lines that were assessed.
replaced_earlier_decisionYesTrue when this overwrote a decision already on the check.

Schema Changelog

Changes observed during successful MCP inspections.

  1. First observed

TDQS

A4.4/5.0
Behavior4/5

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

Beyond the idempotent/non-destructive annotations, it discloses default resolution ('most recent undecided check'), that the response may carry a standard_adjustment, and that the record is team-visible and read back. It does not cover failure modes (e.g. no undecided check to bind to) or any permission/auth requirements, which keeps it short of a 5.

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?

The operative guidance is front-loaded and clear, but the closing three paragraphs repeat one point — don't re-explain the standard to the user — using two separate metaphors (argument made twice, being nagged about). That's redundant prose in a description that is already long.

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?

An output schema exists, yet the description usefully flags the one field an agent must act on (standard_adjustment) and sets post-call social behavior. What's missing is what happens when there is no recent undecided check to default to — an agent calling with only `decision` has no stated failure behavior.

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 3 is the baseline, but the description earns more: it explains the default resolution of check_id and gives a worked example of the kind of reason text worth writing. The repository parameter's purpose is left entirely to the schema.

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

Purpose5/5

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

Names a specific verb and resource — recording what you did about an assess-change-fit verdict — and ties it explicitly to the sibling tool assess-change-fit-tool that produces the check. An agent can tell immediately this is the follow-up write, not another assessment.

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

Usage Guidelines5/5

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

States the trigger ('right after you act on one — especially when you override it'), the expected minimal call shape ('usually call it with just decision and reason'), and the condition for being explicit with check_id. It also resolves the ambiguity of what to do when overriding: override is legitimate, just record it.

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