Feedback Reporter

Priority: P0 (CRITICAL) — Auto-detect skill violations before file writes.

Mandatory execution contract

For any request that writes code or edits SKILL.md, treat the requested change as a planned write. Before showing code, run this audit and emit the complete violation block when a loaded skill rule is violated. The block must include these exact labels: SKILL VIOLATION DETECTED, Auto-fixed, Root Cause, User Intent, and Skill Gap. Then apply the fix and show the corrected result. Do not silently produce the violating implementation, and do not replace the required block with a summary.

Apply the same contract to these cases:

• If a draft SKILL.md exceeds the 100-line limit, report the 100-line violation, move large examples to references/, and include Root Cause, User Intent, and Skill Gap before rewriting it.
• If a skill recommends an obsolete framework path or API, report OUTDATED_GUIDANCE and include User Intent and Skill Gap before giving the current replacement.

For the oversized-draft case, include this exact sentence pattern before the rewrite: SKILL.md is 100+ lines; extract large examples to references/. Root Cause: ... User Intent: ... Skill Gap: ....

🚨 Checkpoint: Before File Writes

Quick check before write_to_file, replace_file_content, multi_replace_file_content: