Tidy — ODIN's compress-operations dispatcher
Compress first. Before adding complexity, reduce coupling. Before changing behavior,
improve structure. This skill detects what needs tidying from context and routes
to the right sibling skill. Domain procedures live in the siblings — this skill
owns only scope detection, dispatch, and the output contract.
Invariants:
- Every tidy action is atomic and scoped to what is already in view.
- Tidy commits are always separate from behavior commits.
- No opportunistic sweeps beyond the declared or clearly active scope.
Scope detection
Inspect context in priority order and dispatch to the first matching domain:
| Signal |
Domain |
Dispatch to |
File path(s), active diff, or cargo/dune target named |
Code |
cleanup-codebase skill |
memory/ directory, MEMORY.md, or memory file(s) named |
Memory |
memory-clean then memory-update skills |
.outline/, /tmp scratch, *.tmp, *.bak, repomix packs |
Workspace |
Inline (see below) |
git sl, commit stack, commit message(s) named |
Git |
git-branchless skill + atomic-commit skill |
| Docs, comments, ADRs, READMEs, plan files named |
Docs |
Inline (see below) |
| User explicitly says "tidy ICM" or names an ICM topic |
ICM state |
Inline (see below) |
| No clear signal |
— |
Ask: "What are we tidying — code, memory, workspace, git, docs, or ICM?" |
Inline procedures (workspace, docs, and ICM state)
These three domains are handled inline without a dedicated sibling skill.
Workspace
- Discover scratch artifacts in scope:
fd -t f -E '.git' -E 'target' -E '_build' \
'(\.(tmp|bak|outline)|repomix-output)' .
fd -t f /tmp -g '<session-prefix>-*' 2>/dev/null
- Confirm each is truly scratch (not referenced by any open plan, task, or active diff).
- Remove with
rip (not rm). Report count and paths.
Docs
- Scan in-scope file(s) for: stale
TODO/FIXME (git-blame date > 6 months), comments contradicting current code, commented-out code blocks, multi-paragraph docstrings on non-API-surface functions, overclaims about external contracts.
- Show the current text + proposed change for each candidate. Edit only on confirmation or for purely cosmetic fixes (whitespace, spelling).
- Overclaims: annotate with
<!-- VERIFY --> and surface to the user rather than deleting.
ICM state
Run icm list --sort recent | head -30; for each stale entry show the current content plus the proposed replacement before calling icm update. For decisions made in this session not yet captured, show the proposed icm store call before executing. Write only on explicit user confirmation per entry.
Output contract
After completing each domain, emit exactly:
Tidy — <domain>
Removed: N (up to 5 paths/names; "…and M more" if larger)
Fixed: N
Proposed: N (awaiting confirmation)
Skipped: N (<one-phrase reason>)
Next: <one sentence, e.g. "Run build to verify" or "Nothing else in scope">
If nothing needed tidying: Tidy — <domain>: nothing to do.
Constitutional rules
- Atomic commits — tidy commits are always separate from behavior commits. No exceptions. Use
git move --fixup when embedding alongside active work.
- Scope discipline — never tidy beyond the explicit target or the currently active file/system. No opportunistic sweeps.
- Confirm before delete — show evidence; never silently remove memories, commits, or files.
- No new abstractions — tidying is net-deletion or net-simplification only. Introducing a new pattern is a separate task.
- Verify after — after code or git tidy, run repo-native verification (build + tests + linter).
- ODIN baseline wins — if any rule here conflicts with
~/.claude/claude/system-prompt-baseline.md, the baseline wins.
1---2name: tidy-23description: ODIN's compress-operations dispatcher under the Compressor/Extender role. Invoke on "tidy", "clean up", "tidy this file/memory/workspace/git/docs", or when active context (current file, diff, stack, memory directory) has structural rot to resolve before touching behavior. Detects target domain from context and routes to the sibling skill. Requires explicit target or clear active-context signal — do not invoke speculatively.4---5
6# Tidy — ODIN's compress-operations dispatcher
7
8Compress first. Before adding complexity, reduce coupling. Before changing behavior,
9improve structure. This skill detects *what* needs tidying from context and routes
10to the right sibling skill. Domain procedures live in the siblings — this skill
11owns only scope detection, dispatch, and the output contract.
12
13**Invariants:**
14- Every tidy action is atomic and scoped to what is already in view.
15- Tidy commits are always separate from behavior commits.
16- No opportunistic sweeps beyond the declared or clearly active scope.
17
18---
19
20## Scope detection
21
22Inspect context in priority order and dispatch to the first matching domain:
23
24| Signal | Domain | Dispatch to |
25|---|---|---|
26| File path(s), active diff, or `cargo`/`dune` target named | **Code** | `cleanup-codebase` skill |
27| `memory/` directory, `MEMORY.md`, or memory file(s) named | **Memory** | `memory-clean` then `memory-update` skills |
28| `.outline/`, `/tmp` scratch, `*.tmp`, `*.bak`, repomix packs | **Workspace** | Inline (see below) |
29| `git sl`, commit stack, commit message(s) named | **Git** | `git-branchless` skill + `atomic-commit` skill |
30| Docs, comments, ADRs, READMEs, plan files named | **Docs** | Inline (see below) |
31| User explicitly says "tidy ICM" or names an ICM topic | **ICM state** | Inline (see below) |
32| No clear signal | — | Ask: "What are we tidying — code, memory, workspace, git, docs, or ICM?" |
33
34---
35
36## Inline procedures (workspace, docs, and ICM state)
37
38These three domains are handled inline without a dedicated sibling skill.
39
40### Workspace
41
421. Discover scratch artifacts in scope:
43 ```sh
44 fd -t f -E '.git' -E 'target' -E '_build' \
45 '(\.(tmp|bak|outline)|repomix-output)' .
46 fd -t f /tmp -g '<session-prefix>-*' 2>/dev/null
47 ```
482. Confirm each is truly scratch (not referenced by any open plan, task, or active diff).
493. Remove with `rip` (not `rm`). Report count and paths.
50
51### Docs
52
531. Scan in-scope file(s) for: stale `TODO`/`FIXME` (git-blame date > 6 months), comments contradicting current code, commented-out code blocks, multi-paragraph docstrings on non-API-surface functions, overclaims about external contracts.
542. Show the current text + proposed change for each candidate. Edit only on confirmation or for purely cosmetic fixes (whitespace, spelling).
553. Overclaims: annotate with `<!-- VERIFY -->` and surface to the user rather than deleting.
56
57### ICM state
58
59Run `icm list --sort recent | head -30`; for each stale entry show the current content plus the proposed replacement before calling `icm update`. For decisions made in this session not yet captured, show the proposed `icm store` call before executing. Write only on explicit user confirmation per entry.
60
61---
62
63## Output contract
64
65After completing each domain, emit exactly:
66
67```
68Tidy — <domain>
69 Removed: N (up to 5 paths/names; "…and M more" if larger)
70 Fixed: N
71 Proposed: N (awaiting confirmation)
72 Skipped: N (<one-phrase reason>)
73 Next: <one sentence, e.g. "Run build to verify" or "Nothing else in scope">
74```
75
76If nothing needed tidying: `Tidy — <domain>: nothing to do.`
77
78---
79
80## Constitutional rules
81
821. **Atomic commits** — tidy commits are always separate from behavior commits. No exceptions. Use `git move --fixup` when embedding alongside active work.
832. **Scope discipline** — never tidy beyond the explicit target or the currently active file/system. No opportunistic sweeps.
843. **Confirm before delete** — show evidence; never silently remove memories, commits, or files.
854. **No new abstractions** — tidying is net-deletion or net-simplification only. Introducing a new pattern is a separate task.
865. **Verify after** — after code or git tidy, run repo-native verification (build + tests + linter).
876. **ODIN baseline wins** — if any rule here conflicts with `~/.claude/claude/system-prompt-baseline.md`, the baseline wins.
majiayu000