Move claude-sdk-cli source into model/view/controller directories#246
Move claude-sdk-cli source into model/view/controller directories#246
Conversation
Makes the three-layer architecture explicit in the filesystem. State classes move to model/, renderer functions to view/, event handlers to controller/. Two biome.json files enforce layer boundaries: model/ cannot import from view/ or controller/, view/ cannot import from controller/. Also updates linting instructions: replace pnpm biome check --write with pnpm ci:fix to avoid scope bleed across packages. No logic changes. AppLayout stays in src/ as a known future split target.
Tests (427/427), type-check, and biome ci all pass. Work is verified and ready for the PR.
shellicar
added
enhancement
shellicar
added
enhancement
shellicar
force-pushed
the
feature/mvc-directory-structure
branch
from
5ab0556 to
41db517
Compare
bananabot9000
left a comment
bananabot9000
left a comment
There was a problem hiding this comment.
Phase 1: file moves + biome boundary enforcement. Clean mechanical PR.
No suspicious files — no history files, no logs, no credentials. changes.jsonl is the only .jsonl and it's the changelog, not history. Session log mentions .sdk-history2.jsonl was committed and repaired in an earlier session — confirmed it's NOT in this diff.
File inventory checks out:
- model/: 7 state files + buildSubmitText + biome.json ✅
- view/: 5 renderers + biome.json ✅
- controller/: AgentMessageHandler ✅
- Boundary rules: model blocks view+controller, view blocks controller ✅
- All 12 test files updated with new import paths ✅
CLAUDE.md rewrite is significant — the testament/cast vocabulary landed, session protocol replaced with fleet identity framework, TUI architecture section added with MVC docs, biome fix command changed to pnpm ci:fix. All intentional per the design doc reviewed earlier tonight.
One note: the session logs mention damage from an earlier session (.sdk-history2.jsonl committed, biome run against entire repo). The 2026-04-12 session log says this was repaired before the verify session. Confirmed not present in this diff.
Approved ✅ 🍌
2 participants
Summary
model/, renderer functions intoview/, event handler intocontroller/Co-Authored-By: Claude noreply@anthropic.com