Feedback Reporter
Priority: P0 - Auto-detect skill violations before file writes
π¨ Checkpoint: Before File Writes
Quick check before write_to_file, replace_file_content, multi_replace_file_content:
- Check - Any skills loaded for this file extension?
- NO β β Proceed silently
- YES β Continue to step 2
- Audit - planned code violate loaded skill rules?
- NO β β Proceed silently β ** NOT submit feedback report**
- YES β Output violation block below, then fix immediately
Detection Flow
Before file write?
ββ Check file extension β Identify loaded skills
ββ Review skill anti-patterns/rules
ββ Code matches anti-pattern?
β ββ YES β π¨ VIOLATION β feedback command
β ββ NO β β
Proceed
ββ No skills loaded β β
Proceed
Examples (Quick Reference)
Flutter: color: Colors.blue β Rule: No hardcoded colors β Report + use theme
React: class MyComponent extends... β Rule: Use functions β Report + convert
SKILL.md: 105 lines β Rule: β€100 lines max β Report + extract to references/
Violation Report Format
When violation detected, output this block before fixing:
π¨ SKILL VIOLATION DETECTED
Skill: [skill-id]
File: [relative/path/to/file.ext]:[line-range]
Rule: [exact rule text from SKILL.md]
Violation: [what planned code does β up to 5-line offending snippet]
Fix: [corrected approach β up to 5-line corrected snippet]
Auto-fixed: YES / NO
Root Cause: [AMBIGUOUS_RULE | MISSING_COVERAGE | OUTDATED_GUIDANCE | COMPETING_RULES | PATTERN_MISMATCH]
User Intent: [1 sentence: what the user was trying to achieve]
Skill Gap: [1β2 sentences: what change to the SKILL.md would prevent this next time]
Co-skills: [other active skill IDs, comma-separated, or 'none']
Root Cause Guide
| Code | When to use |
|------|-------------|
| AMBIGUOUS_RULE | Rule wording permits multiple interpretations |
| MISSING_COVERAGE | Common pattern not addressed anywhere in skill |
| OUTDATED_GUIDANCE | Skill references deprecated API or framework version |
| COMPETING_RULES | Two loaded skills gave contradictory guidance |
| PATTERN_MISMATCH | AI misread or misapplied anti-pattern definition |
Then apply fix immediately β not wait for user confirmation.
Pre-Completion Check
Before notify_user or task completion:
** I write code?** YES β ** I audit skills?** NO β Audit now
Anti-Patterns
- No "I'll check later": Check before writing, not after
- No "minor change skip": Every write needs check
- No "user waiting skip": 10-second check > pattern violation
- No "clean-pass report": If no violation found, proceed silently β NOT submit report
- No "shallow report": Always populate Root Cause, User Intent, and Skill Gap β these drive improvement