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

A run that finds its issue already done on main says which commit did it, and the asker closes it instead of the crew parking it #1048

shipped production is serving it opened this has it
opened this

Pillar: P7 Steering (umbrella #593). Replaces #664, #707, #758 and #869: four people's asks that were already built on main. Each was parked twice because the run found nothing to change, and the forge can only read that as a failure.

Outcome. When a run ends with nothing to change because its issue is already done on the base, the issue shows "done on main, waiting for you to close it" with the commit the run named, and the asker gets one notice. It is not a park, and it never comes back to the planner as parked twice.

Done means.

  • [ ] A run that declines with an "already done" verdict names the commit(s). The crew records that as its own outcome, not a park (src/crew/parks.ts, src/crew/decide.ts), and the issue page and /asked show it with the commit links.
  • [ ] The asker sees one line with a Close button on the issue; closing keeps the commits in the close comment.
  • [ ] A run that declines for any other reason is parked exactly as today.
  • [ ] Tests: an already-done decline is recorded, shown and not counted toward parked-twice; an ordinary decline still parks.

Files. src/crew/parks.ts, src/crew/decide.ts, src/core/asked.ts, the issue page, test/crew-parks.test.ts.

Risks. A run that wrongly says "already done" would hide work. Require at least one named commit that is on the base, and leave the close to the person.

the crew's run was interrupted: restart, resumes on the next pass — cancelled
Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs