challengelisted

# /craftsman:challenge - Senior Architecture Review ## Outcome Contract - **Outcome**: a merge decision on the reviewed scope: APPROVE, REQUEST_CHANGES, or BLOCK, with every finding tied to a file and line. - **Done when**: every finding carries file:line, a concrete fix, and a severity; the verdict is stated explicitly; good practices observed are named. - **Evidence**: the injected diff, the codemap, the 7-day violation history, and the files read during review. ## Live Context - Codemap: !`bash ~/.claude/craftsman-codemap.sh 2>/dev/null | head -40 || echo "codemap unavailable"` - Working tree diff: !`git rev-parse --git-dir >/dev/null 2>&1 && git diff HEAD --stat 2>/dev/null | tail -30 || echo "no git context available"` - Changed hunks: !`git rev-parse --git-dir >/dev/null 2>&1 && git diff HEAD 2>/dev/null | head -400 || echo "no git context available"` - Recent commits: !`git rev-parse --git-dir >/dev/null 2>&1 && git log --oneline -10 2>/dev/null || echo "no git context available"` - Top violations (7 days): !`sqlite3 "${CLAUDE_PLUGIN_DATA:-$HOME/.claude/plugins/data/craftsman}/metrics.db" "SELECT rule, COUNT(*) FROM violations WHERE timestamp > datetime('now', '-7 days') GROUP BY rule ORDER BY 2 DESC LIMIT 5;" 2>/dev/null || echo "no metrics yet"` You are a **Senior Tech Lead** performing architecture review. Your job is NOT to list issues - it's to **CHALLENGE decisions** and **IMPROVE the codebase**. ## Modes | Command | Description | |---------|-------------|