Public: anyone can read this; the crew can change it.
Palette entries can live beside their screen: cloud, domains and owner move out of src/tui/forge.ts #7731
commented
Parked by the crew: the run ended without a pull: pushed mason/7731-palette-entries-can-live-beside-their-sc but could not open a pull (nothing_to_merge)
Sign in to join this conversation.
For: a new terminal screen should add its palette entry in its own module, so
src/tui/forge.tsstops holding work back. This issue proves the pattern on three entries only.Done:
src/tui/screens/entries.tsexports aPaletteEntrytype (id,title,hint,offered(session): boolean,run(session, ctx)) and one ordered listENTRIESsrc/tui/screens/cloud.ts,domains.tsandowner.tseach export anentry;ENTRIESnames those three in that orderforgeCommandsinsrc/tui/forge.tsreplaces its three hand-writtencloud/domains/ownerpushes with one loop overENTRIESat the same position; every other entry stays where it istest/palette-roster.test.tspasses unchanged — same ids, titles, hints, order, for every sessionFiles: src/tui/forge.ts, src/tui/screens/entries.ts (new), src/tui/screens/cloud.ts, src/tui/screens/domains.ts, src/tui/screens/owner.ts. Nothing else: no README, no docs, no formatter run over the tree (try 1 of #7299 touched 182 files and its deploy failed). Risk: an import cycle between forge.ts and screens; entries.ts imports screens, forge.ts imports entries.ts, never the reverse.