CtrlK
BlogDocsLog inGet started
Tessl Logo

address-feedback

Respond to an automated reviewer's findings on the current feature PR — a narrow, headless responder. Triages each reviewer finding into Clear / Ambiguous / Complex / Out-of-PRD-Scope, then acts — fixes the cause and pushes (diff-only) for Clear, posts one in-thread reply for the rest. Skips findings it already handled. Invoked headless by the dispatcher's feedback gate; the dispatcher owns the round counter and the STUCK escalation. Triggers - address-feedback, address review comments, respond to PR review, handle reviewer findings, PR feedback loop (project)

74

Quality

93%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Low

Low-risk findings worth noting

The canonical home for this skill is address-feedback in tdg-ninja/context-specs-factory-ai

SKILL.md
Quality
Evals
Security

Quality

Content

86%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

A well-engineered instruction skill: crisp workflow with genuine idempotency checkpoints, highly actionable bucket actions with enumerated forbidden directives, and exemplary one-level-deep reference splitting. The main costs are a redundant 'Hard nevers' checklist that restates in-line rules and a missing post-fix verification step for Clear fixes before pushing.

Suggestions

Cut the duplication between the 'Hard nevers' checklist and the in-line rule statements — keep the checklist and drop the prose restatements (or vice versa) — trimming roughly 10-15 lines without losing any rule.

Add an explicit verification step for Clear fixes before push, e.g. 'after the fix, re-run ./prds/<feature>/run-prd-test.sh and only commit/push when it exits 0', closing the workflow's only validation gap.

Tighten the 'one-way model' bullets, which pre-explain the dispatcher's STUCK escalation that the 'Contract with the dispatcher' section already states authoritatively.

DimensionReasoningScore

Conciseness

The body is dense and imperative with no padding of concepts Claude already knows, but the 'Hard nevers' section (~10 lines) restates rules already given in-line ('Never reply to a Clear finding', the suppression list, the skip rule, 'never create a stub', round tracking), and the one-way-model bullets repeat dispatcher/STUCK escalation details covered again in 'Contract with the dispatcher'. This matches 'Efficient; minor instances of over-explanation that could be trimmed' rather than the every-token-earns-its-place of score 5.

4 / 5

Actionability

Fully actionable: exact paths and commands are cited ('prds/<feature>/prd.md', 'see references/gh-mechanics.md for the exact commands' — verified to contain copy-paste-ready gh/GraphQL/git commands), each bucket gets an exact action, and the forbidden-suppression list is enumerated down to specific directives ('eslint-disable*', '@ts-ignore', '# noqa', '.skip', 'xfail'). Per the rubric's instruction-skill note, absence of inline code is not penalized when guidance is this concrete.

5 / 5

Workflow Clarity

The four-step sequence (re-derive with idempotency checks → ground in PRD/Expert → triage → act per bucket → exit) is clear with real checkpoints (two idempotency checks, the never-silence/skip rules, one-pass exit), matching 'Clear sequence with most checkpoints present'. Score 5 is withheld because a Clear fix has no explicit post-fix verification before push — nothing says re-run './prds/<f>/run-prd-test.sh' after the change — a minor validation gap in a code-modifying workflow.

4 / 5

Progressive Disclosure

Textbook structure: the SKILL.md body keeps decision logic and rules inline, while the two external files exist in the bundle, are one level deep, and are clearly signaled with their purpose ('The taxonomy, boundary heuristics, and examples live in references/triage.md — read it'; gh-mechanics.md holds 'the fiddly gh / GraphQL / REST commands... kept out of the skill's narrative'). Both referenced files were verified present and hold exactly the promised content.

5 / 5

Total

18

/

20

Passed

Description

100%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

An exemplary description: concrete third-person actions covering the full capability set, explicit trigger phrases including natural synonyms, and a sharply bounded niche (automated reviewer findings on the current feature PR) that minimizes conflict with general review skills. No fluff or over-claims.

DimensionReasoningScore

Specificity

Lists multiple concrete, comprehensive actions in third person: 'Triages each reviewer finding into Clear / Ambiguous / Complex / Out-of-PRD-Scope', 'fixes the cause and pushes (diff-only) for Clear, posts one in-thread reply for the rest', 'Skips findings it already handled'. Every capability of the skill is explicitly named with no vague filler — this matches the anchor 'Lists multiple specific concrete actions; comprehensive coverage' and exceeds the score-4 example, which would leave gaps in coverage.

5 / 5

Completeness

Explicitly answers both questions: the 'what' is concrete (triage into four named buckets, fix-and-push for Clear, in-thread reply for the rest, skip handled findings) and the 'when' is explicit trigger guidance ('Triggers - ...'), the equivalent of a 'Use when...' clause, so the completeness cap of 3 for missing triggers does not apply.

5 / 5

Trigger Term Quality

The trigger list 'address-feedback, address review comments, respond to PR review, handle reviewer findings, PR feedback loop' covers the natural synonyms a user would actually say for this task. This matches the comprehensive-with-synonyms anchor; score 4 would require natural terms to be visibly missing, and none are.

5 / 5

Distinctiveness Conflict Risk

Clear niche with distinct triggers: 'an automated reviewer's findings on the current feature PR — a narrow, headless responder', bounded further by 'Invoked headless by the dispatcher's feedback gate'. It is explicitly scoped to bot-reviewer findings (not human comments, not general code review), giving minimal overlap risk with related review skills.

5 / 5

Total

20

/

20

Passed

Validation

93%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 15 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

referenced_paths_exist

Referenced path issues: 2 missing

Warning

Total

15

/

16

Passed

Repository
tdg-ninja/context-specs-claude-code
Reviewed

Table of Contents

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.