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

The planner's refusals live in one module, read by dependencies, plan and limb replan #5595

shipped production is serving it opened this has it
opened this

For: A plan that makes an issue wait for itself, or a re-plan already in hand reads the same wherever it is said, because it is said from one place. Today each sentence below is typed out separately in src/actions/dependencies.ts, src/actions/plan.ts, src/actions/limb_replan.ts, src/tui/screens/limbs.ts, so the next edit to one copy makes them drift.

Done:

  • [ ] src/actions/plan_words.ts exports these sentences (new, or the existing module of that concept if one already owns similar words; say which in the pull):

- "an issue cannot wait for itself" - "the planner has it in hand already"

  • [ ] Every file named imports them, and grep -rn for each sentence under src/ finds only the module.
  • [ ] Answers and pages are byte-for-byte unchanged: every existing test passes as it is.
  • [ ] A new test/plan-words.test.ts asserts each exported sentence is non-empty and that each file named no longer contains the literal (read the source as text). It fails if a copy is typed back.
  • [ ] Before pushing, run test/plan-words.test.ts, the tests of the files you touched, bun x tsc --noEmit and, if src/web changed, bun scripts/design-lint.ts. A red test elsewhere that main also has is named in the pull, not a reason to stop.

Files: src/actions/plan_words.ts, src/actions/dependencies.ts, src/actions/plan.ts, src/actions/limb_replan.ts, src/tui/screens/limbs.ts, test/plan-words.test.ts. Risk: a pure move; change no wording. Touch only the lines the sentences are on.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs