Public: anyone can read this; the crew can change it.

Contraction: one codebase, fewer frames — fold /big, the dev app page, landing and the no-JS search into studio and board; remove the TUI's app/run/crew and origins twins #401

Merged opened this in answer to

Check · Review · Land · Live done — Landed and live in production.

Merged

This pull covers the first step of #386, removing the leftover `leans`/bulk code and the `pages/studio.ts` shim (#393). It also splits #386 into one issue per fold or removal, as #386 asks. The net change is **−55 lines** (7 added, 62 deleted).

shipped production is serving it deploy 2fa0c24

opened this

This pull covers the first step of #386, removing the leftover leans/bulk code and the pages/studio.ts shim (#393). It also splits #386 into one issue per fold or removal, as #386 asks. The net change is −55 lines (7 added, 62 deleted).

What changed

  • Deleted the shim: src/web/pages/studio.ts (17 lines) only re-exported other modules. test/studio.test.ts now imports pullWord, runWord, servingHistory and WordCtx straight from src/web/studio/words.ts. Only that import line changed in the test, no assertions.
  • Deleted latestLean from src/core/decisions.ts. Nothing called it. The "lean" value stays in DecisionKind, because older ledgers still hold rows of that kind and the board and /big still have labels for it. Its comment now says that, instead of pointing at web/studio/leans.ts, which no longer exists.
  • Fixed comments that described missing code: the bulkCss / "bulk route" note in src/web/studio/style.ts, the "bulk bar" in src/web/studio/asks.ts, and leans in the source map in CLAUDE.md. There is no bulk bar, route or stylesheet anywhere in the code.
  • Split #386 into seven issues, #393–#399, each linked so #386 waits on it. Each one repeats #386's acceptance terms (every old URL still answers, with a test per old route; the check stays green; the line count goes down; the docs name only the surviving surfaces):

- #393: this pull - #394: fold /big into the board - #395: fold the dev app page into the studio app page - #396: fold the landing page into the studio home - #397: fold the no-JS search page into the search route - #398: remove the terminal UI's apps, runs and crew screens - #399: remove the terminal UI's origins screen (the web origins and migrate pages stay)

Not done here: the six folds and removals in #394–#399, one per run as #386 asks. No URL changed, so there are no redirect tests in this pull. docs/STUDIO.md and docs/DESIGN.md never mentioned any of the deleted code, so they are unchanged.

How to try it: the full forge.json check passes (type check plus all 562 tests). To confirm nothing still refers to the removed code, run grep -rn "pages/studio\|latestLean\|bulkCss\|leans" src test; only unrelated matches should come up.

🤖 Generated with Claude Code

Closes #386

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs