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

A terminal screen adds its own palette entry from its own module, so a new screen no longer edits src/tui/forge.ts #7299

Closed opened this
opened this

For: the crew, which today cannot start five ready issues because src/tui/forge.ts is in 7 open pulls: every new terminal screen adds its palette entry there. Steering means work is not held behind one file.

Done:

  • [ ] Each palette entry (id, title, hint, who it is offered to, what it opens) is declared next to its screen, e.g. an exported entry in src/tui/screens/<name>.ts, and one list in a new src/tui/screens/entries.ts names them; src/tui/forge.ts reads that list instead of pushing each entry by hand.
  • [ ] Every entry offered today is offered to exactly the same people, with the same id, title and hint, in the same order; a test compares the palette for a person admin, a member, an agent and a signed-out session before and after.
  • [ ] AGENTS.md or the terminal's doc says a new screen adds its entry in its own module and one line in entries.ts.

Relieves: src/tui/forge.ts Files: src/tui/forge.ts, a new src/tui/screens/entries.ts, the screen modules whose entries move, one test. Risk: medium; it conflicts with every open pull that adds a palette entry, so it lands when src/tui/forge.ts is no longer hot.

commented

Parked by the crew: the preview's deploy failed and the decider saw no point in another try try 1: AGENTS.md, README.md, docs/APPS.md, docs/CREW.md (+178 more) (pull #7577 closed). Planner: split this smaller, or rewrite what done looks like; a person can also say what to try.

commented

Parked by the crew: the check stayed red through 2 automatic tries (pull #7605 closed).

Parked twice since a person last spoke here, so it is the planner's now. Every reason:

  1. the preview's deploy failed and the decider saw no point in another try try 1: AGENTS.md, README.md, docs/APPS.md, docs/CREW.md (+178 more) (pull #7577 closed)
  2. the check stayed red through 2 automatic tries (pull #7605 closed)

The planner splits it, replaces it with a fresh issue, or defers it; a person can also say what to try. Nothing starts here until then, and the board lists it.

commented

Split into #7730 (a test pins the palette per session, no source change), #7731 (entries.ts plus three entries moved: cloud, domains, owner) and #7732 (the remaining admin entries plus one AGENTS.md paragraph). Try 1 touched 182 files and its deploy failed; try 2 stayed red. Each piece now names its files and leaves README and docs alone.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs