
Improve Architecture
Architecture check-in for a growing macOS Swift codebase.
Install
npx skills add daveebbelaar/bama --skill improve-architecture -g -a claude-code -y- Repository
- daveebbelaar/bama1.3k
- Category
- Mobile
- Updated
- 3d ago
- License
- MIT
- Context cost
- Lightabout 1,324 tokens
- Runs scripts
- No, instructions only
What it does
Architecture check-in for a growing macOS Swift codebase. Finds hot spots, proposes deeper modules, and refactors one step per commit with behavior proven unchanged. Use at the end of a milestone, before a big feature, when a file passes 600 lines, when one feature touches many files, when bugs repeat in one area, or when the user asks to refactor or clean up the code.
Improve architecture
Features pile up, and an app that was easy to change gets hard to change. A check-in catches that early: survey, propose, let the user pick, then refactor in small steps that keep behavior the same. The target shape for apps that grow is in architecture.md.
When to run
Any one signal is enough:
- The end of a milestone (the
improve-loopskill calls this skill there). - Before a big feature. Ask: "What change to the code would make this feature easy?"
- A Swift file over 600 lines (act at 1,000), a type over 400 lines, or "unable to type-check this expression".
- One feature touched more than 6 files or more than 2 targets.
- The same area shows up in 3 fix-log entries, or the same kind of bug happens twice.
- State that changes more than 10 times a second sits on a model that many views observe.
- A dependency cannot be replaced in a test.
Vocabulary
Use these words exactly, so proposals stay comparable:
- Module: anything with an interface and an implementation (a type, a file group, a target).
- Interface: everything a caller must know: signatures, ordering, error modes, threading, performance.
- Depth: how much a module does per unit of interface. Deep modules hide a lot behind a small interface. Shallow modules are pass-throughs.
- Seam: a place where you can swap behavior without editing callers.
- Adapter: one implementation behind a seam. One adapter is a hypothetical seam. Two (production and test) make it real.
- Locality: a change touches one place. Leverage: callers get a lot for little.
1. Survey
Change no code in this step.
-
Read
CLAUDE.md,ARCHITECTURE.md,docs/adr/, anddocs/FIXLOG.mdwhere they exist. -
Find hot spots:
git log --since=30.days --name-only --format= | sort | uniq -c | sort -rn | head -20 find Sources -name '*.swift' | xargs wc -l | sort -n | tail -20
