CtrlK
BlogDocsLog inGet started
Tessl Logo

common-feedback-reporter

Pre-write audit for skill violations: checks planned code against loaded skill anti-patterns before any file write. Use when writing Flutter/Dart/TS code or editing SKILL.md files with active project skills. Load as composite; on auto-fixed violation, also load +common/common-learning-log.

56

Quality

66%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide

SecuritybySnyk

Passed

No findings from the security scan

Fix and improve this skill with Tessl

tessl review fix ./.github/skills/common/common-feedback-reporter/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

56%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.

The skill delivers a concrete, highly actionable audit procedure with an exact report template and a useful root-cause taxonomy, but it is notably redundant — the same contract, labels, and 100-line rule are repeated three to four times — and it fails to disclose its own reference files. Consolidating the duplication and linking the existing examples would raise both token efficiency and structure.

Suggestions

State the required labels and the 100-line/extract-to-references rule exactly once (in the report format and contract sections) and delete their verbatim repetitions in the 'Checkpoint', 'Anti-Patterns', and closing lines — this would cut the body by roughly a third.

Merge the 'Mandatory execution contract', 'Checkpoint', and 'Pre-Completion Check' sections into one audit procedure instead of three restatements of the same trigger condition.

Link the existing bundle files (e.g., 'Worked examples: see references/violation-examples.md; activation tests: see references/test-scenarios.md') and add a re-audit step after each auto-fix so the corrected code is verified, not just applied.

DimensionReasoningScore

Conciseness

The same material is restated many times: the label list "SKILL VIOLATION DETECTED", "Auto-fixed", "Root Cause", "User Intent", "Skill Gap" appears verbatim in the contract, the sentence-pattern line, the anti-patterns list, and the closing line (~4x); the 100-line rule appears three times (lines 28, 31, 91); and the audit-before-write contract is repeated across the 'Mandatory execution contract', 'Checkpoint', and 'Pre-Completion Check' sections. This matches anchor 2 ("noticeably verbose; several unnecessary... padded sections") — roughly 30-40% of the body is redundant restatement — rather than anchor 3's "mostly efficient... some unnecessary explanation could be tightened".

2 / 5

Actionability

The violation report is a copy-paste-ready fill-in template, the Root Cause Guide is a concrete enumerated table, and the checkpoint is a two-step decision procedure with explicit branches — mostly executable guidance with minor gaps. It falls short of anchor 5 because the body contains no worked example of detecting/reporting a violation (the examples exist in references/ but are never linked), and the fix step is asserted rather than illustrated.

4 / 5

Workflow Clarity

The sequence is clearly laid out (check skills loaded → audit planned code → emit violation block → apply fix immediately), reinforced by a pre-completion re-check, matching anchor 4's "clear sequence with most checkpoints present; minor validation gaps". It is not anchor 5 because there is no error-recovery or verification loop after an auto-fix — the fix is applied and the flow ends, and "apply fix immediately — not wait for user confirmation" carries risk with no re-audit step.

4 / 5

Progressive Disclosure

The body itself is well-sectioned with headers, a template, and a table, but the two actual bundle files (references/violation-examples.md and references/test-scenarios.md) are never linked from the body — "references/" is mentioned only as a destination for oversized examples, not as navigation to existing material. This matches anchor 3 ("references present but not clearly signaled") rather than anchor 4, which requires references to be mostly clearly signaled.

3 / 5

Total

13

/

20

Passed

Description

76%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.

A strong description that explicitly states both what the skill does and when to use it, with an unusually explicit conditional trigger for the co-skill. Its main weakness is trigger-term coverage: it relies on internal jargon ("pre-write audit", "composite") and omits file-extension keywords users would naturally mention.

Suggestions

Add natural trigger synonyms and file extensions to the 'Use when' clause (e.g., '.dart, .ts, .tsx files', 'report skill violations', 'check code against skills') instead of relying on the jargon phrase 'pre-write audit'.

Briefly state the observable outcome (emits a violation report block with diagnostic fields) so the 'what' is comprehensive, not just the audit action.

DimensionReasoningScore

Specificity

The description lists several concrete actions — "checks planned code against loaded skill anti-patterns before any file write", "Load as composite", and "on auto-fixed violation, also load +common/common-learning-log" — with only minor gaps. It falls short of a 5 because it does not enumerate the full scope of behavior (e.g., emitting the violation report block, the diagnostic fields) and does not exceed anchor 4's "several specific actions; minor gaps in coverage"; it is clearly above anchor 3, which covers only 1-2 actions.

4 / 5

Completeness

It explicitly answers both questions: what ("Pre-write audit for skill violations: checks planned code against loaded skill anti-patterns before any file write") and when ("Use when writing Flutter/Dart/TS code or editing SKILL.md files with active project skills"), plus a conditional trigger ("on auto-fixed violation, also load +common/common-learning-log"). This directly matches anchor 5; anchor 4 would require the "when" to be less explicit than it is.

5 / 5

Trigger Term Quality

Relevant terms are present ("skill violations", "writing Flutter/Dart/TS code", "editing SKILL.md files"), but common variations and synonyms are missing — no file extensions (.dart, .ts, .tsx), and phrases like "pre-write audit" are author jargon a user would not naturally type. This matches anchor 3 ("some relevant keywords but missing common variations or synonyms") better than anchor 4, which requires good coverage with only a few natural terms missing.

3 / 5

Distinctiveness Conflict Risk

It occupies a clear meta-niche (auditing other loaded skills before writes) that few skills share, but its trigger surface (any Flutter/Dart/TS write) is broad, giving minor overlap risk with closely related code-writing and skill-management skills — matching anchor 4 rather than anchor 5's "clear niche with distinct triggers; minimal conflict risk".

4 / 5

Total

16

/

20

Passed

Validation

87%

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

Validation — 14 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

metadata_version

'metadata.version' is missing

Warning

metadata_field

'metadata' should map string keys to string values

Warning

Total

14

/

16

Passed

Repository
HoangNguyen0403/agent-skills-standard
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.