An Architect idea pauses Architect runs until it is triaged #326

Parent

#311

What to build

The Architect idea pauses Architect runs until the Day shift triages it.

When an Architecture review finds no Strong candidate and ends on an issue rather than a plan, thirdshift labels that issue an Architect idea: architect-idea and needs-triage, in one request, keeping its other labels. thirdshift adds the label, never the agent, and creates architect-idea in the repository if it is missing, as it does for architect-plan. This applies to both outcomes:

• "Idea filed": the issue the review filed.
• "Idea already filed": the open issue that already covers the top recommendation. needs-triage goes back on even if that issue was already triaged (e.g. ready-for-human), because the factory again takes it for the best next move.

If thirdshift can't label the issue, the Architect run fails as a failed review, and its Run notification carries the cause. The plan path is unchanged.

While any Architect idea is open and still labelled needs-triage (both labels compared ignoring case, as every label rule is), every later Architect run on the repository is skipped before any review. The check comes after the lock and the open-plan check. The skip prints each such idea's URL on stdout, one per line, and names them on stderr as waiting for triage. Any triage decision releases the pause: needs-triage taken off, whatever replaces it, or the issue closed.

Acceptance criteria

• A review that files an idea leaves the idea labelled architect-idea and needs-triage.
• A review whose idea is already filed puts architect-idea on the covering issue, puts needs-triage back on one that had been triaged, and keeps its other labels.
• A failure to label the idea fails the Architect run, and its one notification carries the cause.
• An open Architect idea labelled needs-triage skips the next Architect run before any review: exit 0, the idea's URL on stdout, the reason on stderr.
• The same idea with needs-triage swapped for another triage label, or closed, does not skip it.
• Labels in another case (e.g. Architect-Idea, Needs-Triage) count the same.
• With both an open Architect plan and a waiting Architect idea, the skip reports the plan.
• The idea half of the test asserting that an open idea issue does not skip the Architect run is rewritten to the new rule.
• The README's Architect run sections (no Strong candidate, one Architect plan at a time, the steps) describe the label and the pause.

Blocked by

None (can start immediately)