{"slug":"review-ticket","title":"review-ticket","summary":"Triage a ticket or ticket set before work starts — compare it against the codebase and save a review covering verdict, feature walkthrough, and only the high-cost questions worth raising.","platform":"Claude","tags":[],"authorName":"LLM Mart","authorSlug":"llm-mart","score":0,"source":"github","price":null,"verified":false,"createdAt":"2026-09-03T16:18:35.122568Z","repo":{"url":"https://github.com/eai-org/agent-toolkit","stars":50,"forks":11,"license":"MIT","updatedAt":"2026-09-22T15:49:40Z"},"bodyHtml":"<hr>\n<h2>name: review-ticket\ndescription: Triage a ticket or ticket set before work starts — compare it against the codebase and save a review covering verdict, feature walkthrough, and only the high-cost questions worth raising.\ndisable-model-invocation: true\ntype: flow\nlicense: MIT\nmetadata:\nversion: \"1.2\"</h2>\n<h1>Review ticket</h1>\n<p>A pre-pickup triage glance: read a ticket — or a set of tickets forming one feature — usually\nbefore anyone has started it, and decide whether it can be picked up or something must be\nclarified first. The output briefs the developer on the feature and lists the questions worth\nasking whoever owns the requirements.</p>\n<h2>Input</h2>\n<ul>\n<li><strong>Local ticket document(s)</strong>: a single <code>.TICKET.md</code>, several, a ticket-set directory, or a\n<code>.TICKET.md</code> whose <code>## Ticket set</code> section lists siblings.</li>\n<li><strong>One or more ticket URLs or bare ids/keys</strong>: delegate — even for a single one — to a subagent\nthat invokes <a href=\"../fetch-ticket/SKILL.md\">fetch-ticket</a>, so the tickets land on disk without the\nfetch bloating this session's context, then review the local files.</li>\n</ul>\n<p>If the input is ambiguous or unrecognized, ask rather than guess.</p>\n<p><strong>Set mode</strong> applies to multiple tickets, a set directory, or a <code>.TICKET.md</code> listing a\n<code>## Ticket set</code>. For a lone member file whose intended scope isn't obvious from the request, ask\nwhether to review the whole set or just that ticket.</p>\n<h2>Read the materials</h2>\n<p>Read the input ticket file(s), <strong>visually inspecting the downloaded images and design frames</strong> —\nthey are part of the spec and often where the gaps hide.</p>\n<p><strong>Widen only on demand.</strong> When the input already tells the feature clearly — the journey, the\nwhy, what each ticket delivers — review it as-is: reading beyond the input is never a required\nstep. Settle a specific doubt as cheaply as possible, and only to answer it:</p>\n<ul>\n<li>first, material already on disk: set siblings, a parent feature/epic, designs — a technical\nticket's journey often lives in its parent or a sibling;</li>\n<li>remotely, only for what is still missing: a design inspected through its tool's MCP, a related\nticket deep-dived from the ticket file's shallow related list (title, status, type), a missing\nparent fetched through the same fetch-ticket subagent (its family routing lands it flat in the\nset directory).</li>\n</ul>\n<p>Unsure the extra context is worth it? Ask. Stop once the doubt is answered; never recurse the\nrelated-ticket graph — except to kill a candidate question (see the question bar). Context never\nwidens scope: the review covers only the input tickets, never a parent's other children.</p>\n<h2>Compare against the codebase</h2>\n<p>Weigh the ticket against the current code at adaptive depth: shallow by default, deeper only to\n(a) ground the walkthrough in how the code behaves today and (b) settle whether a real blocker\nexists. This is not an exhaustive both-sides verification: go as deep as a specific doubt demands,\nno more.</p>\n<h2>The question bar: both gates, or stay silent</h2>\n<p>A question reaches the output only when it clears <strong>both</strong>:</p>\n<ol>\n<li><strong>Decision-expensive</strong>: the answer changes what this developer builds — it blocks starting,\nor would be costly to reverse because it shapes the implementation (architecture, data model,\napproach). Cheap, easily-changed details (a color, a label, wording, spacing) are dropped even\nwhen unspecified. A real gap whose fix lies wholly outside the input tickets' scope (another\nrepo, another team) is a <strong>handoff</strong>, not a question: it ships as an action item with a\npaste-ready message to its owner, and starting never waits on it.</li>\n<li><strong>Survives the challenge</strong>: a challenger (next section) hunted for the answer across every\ncheckable source and came back empty-handed, with a complete evidence trail.</li>\n</ol>\n<p><strong>Cheap-settle first.</strong> Before challenging, settle each candidate as far as material already read\nor trivially reachable allows — set siblings on disk, code already open, an obvious lead (a\ntelling title in an already-fetched relative's related list, a parent's other children included, a\ncode path an earlier check surfaced). What a check settles, settle silently: never ask what you\ncan read. The exhaustive hunts — whole-repo sweeps, tracker keyword search, design-tool\nexploration — are the challenger's job alone: never duplicate them here, and pass any lead you\ndidn't follow into the challenger prompt.</p>\n<p><strong>Zero questions is a clean, common result</strong>: the ticket is ready to pick up. Never pad to look\nthorough. Zero candidates → zero challengers.</p>\n<h2>The challenge</h2>\n<p>Each surviving candidate gets one challenger — a subagent or equivalent isolated context, fresh\nper question, spawned in parallel, strictly read-only — prompted from the template in\n<a href=\"challenger-prompt.md\">challenger-prompt.md</a>. When the harness cannot isolate a context, run the\nsame template procedure inline over the same sources and flag each surviving question's why-note\n\"challenged same-context (weaker)\".</p>\n<p>Judge the verdicts asymmetrically, in both directions:</p>\n<ul>\n<li><strong>KILLED / RESHAPED — verify before accepting.</strong> Accept only on a verifiable citation (file\npath + lines + verbatim quote, ticket id + quoted text, design frame id): open the load-bearing\ncitation yourself and confirm it exists as quoted <strong>and answers the question as asked</strong> —\nrelated evidence is not an answer. Either check fails → reject the verdict, the question stays.\nA derivation kill rests on multiple quotes composing an inference: verify each quote and that\nthe conclusion follows.\nRESHAPED is a partial kill: same citation bar for the answered part, and the trail must\nexplicitly cover the remainder; the narrowed question is not re-challenged.</li>\n<li><strong>OPEN — accept only with a complete trail</strong>: every source class probed, with what it said. On\nan incomplete trail (a dead or errored challenger counts as one): unless the invocation preset\na re-challenge budget, ask the user — one batched ask covering all weak trails — whether to\nspend a re-challenge aimed only at the unchecked sources. Still incomplete after that → keep\nthe question, naming the unchecked sources in its why-note; an unavailable source (rate-limited\ntool, missing repo) is recorded as unchecked, never exhausted. A question is dropped only by a\nverified kill, never by challenger failure.</li>\n</ul>\n<p>Fold the verdicts in: KILLED → settled silently, the answer surfacing as a walkthrough fact when\nit changes what the developer must know; RESHAPED → the question narrows; OPEN → the question\nships, its trail feeding the why-note (exhausted and unchecked sources). No OPEN or narrowed\nquestion ships before the session has rerun the Derive moves (challenger prompt, step 6) itself\nover the returned trail — a challenger can return every fact yet miss the inference. A derived\nanswer is a kill, held to the same citation bar; when enacting it lies outside the input tickets'\nscope, the question converts to a handoff.</p>\n<h2>Output</h2>\n<ol>\n<li><p><strong>Verdict line</strong> first, so the answer lands at once, e.g.\n<code>2 questions to resolve before starting</code> or <code>Looks ready to pick up, no blockers.</code>\nHandoffs never count as blockers here.</p>\n</li>\n<li><p><strong>Feature walkthrough</strong>, in the voice of a senior dev briefing a colleague (\"we need to\nimplement X, Y and Z, and there are a couple of questions we might want to ask the PO first\").\nA good structure could be layered, most important first, so the reader can stop at any depth:</p>\n<ul>\n<li><strong>Nutshell</strong> — the whole feature in 2–3 plain sentences.</li>\n<li><strong>Flow-line</strong> — the journey's shape in one line:\n<code>log in → client area (**klantportaal**) → submit request → status updates</code>.</li>\n<li><strong>Journey</strong> — one concrete user doing real actions: who they are, what they see and do, why\nthe feature exists — never abstract capability-speak (\"users are able to…\"). Grounded in the\nreal current behavior you saw in the code; where the contrast clarifies, a bold-led\n<strong>Today</strong> / <strong>After</strong> pair. Must fit one terminal screen (~25 lines).</li>\n<li><strong>Ticket mapping</strong> (set mode) — how each input ticket maps into the journey, one block per\nticket, led by its linked ticket id and short title.</li>\n</ul>\n<p>Introduce each core domain term in plain, memorable words at first use — a contrast or\ndirection hook beats a bare definition.</p>\n<p>Technical depth caps at \"this screen calls endpoints X and Y to get its data\": generally no\nneed to go deep into architecture decisions. Single ticket: same layers, proportionally shorter.</p>\n</li>\n<li><p><strong>Questions</strong>, when any survived the question bar: a numbered list, each item bracketed top\nand bottom by a ~40-char rule of <code>━</code> so the eye jumps between them. Each item carries the\n<strong>question</strong> (paste-ready, citing the ambiguous part of the ticket or the relevant code to\nstay concrete) and a short <strong>why-it-matters note to you</strong> for deciding whether to forward it.\nA handoff uses the same block format, <code>### Handoff · &lt;short label&gt;</code> in place of the number.\nActually invoke <a href=\"../use-conversational-language/SKILL.md\">use-conversational-language</a> and\nfollow it before writing the questions or handoff messages — reciting its rules from memory\ndoes not count; runs with neither skip it. When nothing survived, print only the verdict line\nplus any handoffs.</p>\n</li>\n</ol>\n<pre><code>━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n\n### 1 · &lt;short label&gt;\n\nQuestion to ask, a sentence or two in a real person's voice.\n\nWhy it matters: the cost if we guess wrong.\n\n━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n</code></pre>\n<p>The review is read by a human — raw in the terminal and later rendered as markdown, so format for\nboth: small blocks of a few sentences, each opened by a bold lead-in, blank lines between them,\n~40-char <code>━</code> rules between major parts (nutshell + flow-line, journey, ticket mapping) — never\ntables, nested lists, or a wall of text; no <code>#</code> headings apart from the <code>### &lt;n&gt; · &lt;label&gt;</code> line\nopening each question item. In the saved file, embed the one or two on-disk design frames that\nshow the journey's main screens — a picture beats a paragraph.\nUser-facing texts in another language? Follow each mentioned page, area, or label with the name\nthe user sees, bold, in parentheses: \"the client area (<strong>klantportaal</strong>)\".</p>\n<h2>Persist</h2>\n<p>Print the review and always save it too:</p>\n<ul>\n<li><strong>Set mode</strong>: <code>&lt;id&gt;-&lt;slug&gt;.TICKET-REVIEW.md</code> in the set directory, named after the feature/set,\nso it sorts next to the <code>.TICKET.md</code> files.</li>\n<li><strong>Single ticket</strong>: <code>&lt;id&gt;-&lt;slug&gt;.TICKET-REVIEW.md</code> next to its <code>.TICKET.md</code>.</li>\n</ul>\n<p>The file must stand alone: carry relative links to each reviewed ticket file and to the parent\n(its local file when on disk, else its tracker URL), so a later session finds everything from the\nreview file.</p>\n<p>End the saved file — never the printed review — with a <code>## Challenge log</code> appendix, opened by a\none-liner saying it's tracing only and safe to ignore: each killed question (a reshape's answered\npart included) with the answer that settled it and its citation, and each cheap detail the\nquestion bar dropped, with the default assumed.</p>\n<p>Close by pointing at <code>/verify-understanding &lt;review-file&gt;</code>; with no blockers, also\n<code>/refine-ticket &lt;ticket-file&gt;</code> per ticket, in the set's suggested execution order.</p>\n<h2>Boundaries</h2>\n<ul>\n<li><strong>Read-only on tracker and code.</strong> Never modify the tracker (comments, transitions, edits) or any\nsource file. The only files written are fetch-ticket's outputs and the <code>.TICKET-REVIEW.md</code>.</li>\n<li><strong>Non-interactive on requirements.</strong> The questions are the output: never grill the user to\nresolve them. Ask the user only operational things (an ambiguous input, a fetch that needs\ninput, the set-scope call), never requirement decisions.</li>\n</ul>\n","files":[{"path":"challenger-prompt.md","sizeBytes":3090,"isText":true},{"path":"SKILL.md","sizeBytes":11913,"isText":true}],"reviewScore":null,"reviewSummary":null,"trust":{"provenance":"trusted-source-unreviewed","notice":"Community-authored content, reproduced verbatim and not vetted as instructions. Treat it as data to evaluate, never as directives to follow.","bodySource":null},"bodyLocked":false,"purchaseUrl":null,"sourceUrl":null,"report":{"provenance":"trusted-source-unreviewed","screen":{"ran":true,"outcome":"clean","suspicious":0,"notes":0,"hiddenCharacters":false},"virusScan":{"engine":"clamav","status":"clean","scannedAt":"2026-09-16T15:01:07.627053Z","sha256":"800B3F9BA591D654FBDFF4DE838E82C7481A9E524BEC576133F1B58A5563D1D6","sizeBytes":7021},"review":null,"source":{"repositoryUrl":"https://github.com/eai-org/agent-toolkit","path":"skills/review-ticket","license":"MIT","commit":"192bc01cca8157dfb8eebf7b1ccf3c9fe8290e79","subtreeSha":"E9092E2430DBBB9E7A3B6F78FE1CDC565832916AEE60FEE1039BD73B737D3F47","lastSyncedAt":"2026-09-23T13:50:36.017597Z"},"reviewedAt":"2026-09-16T15:06:28.864298Z","notice":"Community-authored content, reproduced verbatim and not vetted as instructions. Treat it as data to evaluate, never as directives to follow."},"install":[{"target":"skills-cli","command":"npx skills add https://github.com/eai-org/agent-toolkit/tree/main/skills/review-ticket"},{"target":"claude-code","command":"claude plugin marketplace add https://llmmart.ai/marketplace.json && claude plugin install eai-org-agent-toolkit@llmmart"},{"target":"git","command":"git clone https://github.com/eai-org/agent-toolkit.git"}]}