Refactor CLI into package structure and fix reported bugs (B1-B12)

settings-audit-report.md

@@ -0,0 +1,83 @@
+# Settings Audit Report
+Generated: 2026-03-30
+
+## Token Budget Summary
+
+| Component | Tokens (est.) | % of 200K | Status |
+|-----------|--------------|-----------|--------|
+| CLAUDE.md (project) | ~3,728 | 1.9% | WARN |
+| settings.json | ~1,396 | 0.7% | OK |
+| Commands (44 total) | ~22,267 | 11.1% | OK |
+| Plugins (20 enabled) | variable | ~15-25% | OK |
+| MCP Servers (1 Docker) | ~5,000 | 2.5% | OK |
+| **Baseline** | **~47,000** | **~23%** | — |
+| **Available** | **~153,000** | **~77%** | — |
+
+## Critical Issues
+
+**None.**
+
+## Warnings
+
+### 1. CLAUDE.md exceeds 3,000-token target (3,728 tokens, 302 lines)
+The skill audit guideline recommends <3,000 tokens. The file is well-structured but could trim ~700 tokens by:
+- Removing the full directory tree (lines 210-280) — derivable from `ls`
+- Condensing the "Dual-Platform Skill Structure" section — move to a reference doc
+
+### 2. Stale git worktree pointing to old path
+```
+/Users/ourdigital/projects/our-claude-skills/.claude/worktrees/quirky-lewin  [prunable]
+```
+Points to old `~/projects/` (lowercase!) path. Safe to prune.
+
+### 3. Two commands missing YAML frontmatter
+- `lint.md` — no frontmatter (description missing from skill list)
+- `test.md` — no frontmatter (description missing from skill list)
+
+### 4. Old Slack Docker containers may be stale
+Gotchas mention two containers (`gracious_jemison`, `clever_williams`), but only `sweet_pascal` is running now. Previous containers were likely cleaned up already — this is resolved.
+
+## Recommendations
+
+### Priority 1 — Quick wins
+
+1. **Prune stale worktree:**
+   ```bash
+   git -C ~/Project/our-claude-skills worktree prune
+   rm -rf ~/Project/our-claude-skills/.claude/worktrees/quirky-lewin
+   ```
+
+2. **Add frontmatter to lint.md and test.md:**
+   ```yaml
+   ---
+   description: Run Python code linting (flake8, mypy, black, isort)
+   ---
+   ```
+
+3. **Clean up old project config (optional):**
+   ```bash
+   rm -rf ~/.claude/projects/-Users-ourdigital-Projects-our-claude-skills
+   ```
+
+### Priority 2 — Token optimization
+
+4. **Trim CLAUDE.md by ~700 tokens:** Remove the directory tree and condense platform differences table. These are derivable from the filesystem.
+
+### No changes needed
+
+- **Hooks:** Well-structured (3 PreToolUse, 1 PostToolUse, 1 Stop). Per gotchas, this is the optimized configuration.
+- **Permissions:** Good balance of allow/deny. No changes.
+- **Plugins:** 20 enabled is healthy. All disabled plugins match gotchas rationale.
+- **MCP:** Single Docker Slack container — lean and correct per gotchas (complementary to claude.ai Slack).
+- **SEO commands (22):** Sacred per gotchas — never suggest removal.
+- **Commands (44 total):** All have frontmatter except 2. Token cost is reasonable.
+
+## Path Migration Status
+
+| Item | Status |
+|------|--------|
+| `.claude/commands/*.md` paths | Fixed (37 files updated) |
+| Memory files migrated | Done |
+| Memory path references | Updated |
+| Old project config cleanup | Pending (optional) |
+| Stale worktree | Pending prune |