Repository navigation
docs(plan): record user decision E20 (option A) - #390
Conversation
The user chose option A for E20 on 2026-10-04 in the orchestrator session: v1.5.0 is an app release through PR -> gates -> Mergify queue, continuous mode ships off and not release-eligible, and the user attests acceptance himself. PR #372 review B blocked because docs/PLAN.md still listed E20 as open. No-Test: documentation only Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 695c9d80da
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| | E18 | Matrixzeile 3 (riskante Übergänge, `resume`): Für v1.5.0 einengen? Vorschlag (Empfehlung): `resume` ist in v1.5.0 nicht verfügbar; ein Laufzeittest mit echtem Store belegt, dass `resume` ohne, mit falschem und mit aktuellem Urteil abgelehnt wird und nichts verändert, auch nach Neustart (Paket M4-ROW3-A eingereiht). Alternative: urteilsgebundenes `resume` bauen (5 Pakete, davon 2 auf der st-Spur, v1.5.0 ca. 1–2 Tage später). Bis zur Antwort gilt der Vorschlag als Arbeitsannahme. | Hohe Priorität; v1.5.0-Pfad, Empfehlung annehmen. | Nutzer | offen, hoch | | ||
| | E19 | Matrixzeilen 15 und 17 eingeengt statt neu gebaut (Berater Fable + Astra, 04.10., beide Empfehlung: einengen). Echte Policy-Revision je Kandidat (DF-13) und echte Freigabe-Widerrufung (W5-02d) werden in M5 nachgeprüft. Gilt als Beraterentscheid; der Nutzer kann widersprechen. | Hohe Priorität; v1.5.0-Pfad. | Nutzer | entschieden (Berater), Nutzer kann widersprechen | | ||
| | E20 | Zeilen 18 und 27 blockieren v1.5.0 in der heutigen Fassung: `releaseEligible` verlangt Review-Befugnis (W5-02d, nach M4 geparkt) und zugleich eingeschalteten Dauerbetrieb, obwohl E5 sagt, der Dauerbetrieb bleibt bis nach der Abnahme aus. Optionen: (A, Empfehlung beider Berater) v1.5.0 = App-Release über den menschlich kontrollierten Weg PR -> Gates -> Merge-Queue; Dauerbetrieb wird aus und nicht releasefähig ausgeliefert; Zeile 18/27 und das Audit werden getrennt (App-Release vs. Dauerbetrieb), der Nutzer bestätigt die Abnahme selbst in einer Datei; (B) Review-Befugnis jetzt bauen (mehrere Stufe-A-Pakete an Nahtstellen, v1.5.0 deutlich später); (C) Release verschieben. | Höchste Priorität; v1.5.0-Blocker. | Nutzer | offen, Nutzer | | ||
| | E20 | Zeilen 18 und 27 blockieren v1.5.0 in der heutigen Fassung: `releaseEligible` verlangt Review-Befugnis (W5-02d, nach M4 geparkt) und zugleich eingeschalteten Dauerbetrieb, obwohl E5 sagt, der Dauerbetrieb bleibt bis nach der Abnahme aus. Optionen: (A, Empfehlung beider Berater) v1.5.0 = App-Release über den menschlich kontrollierten Weg PR -> Gates -> Merge-Queue; Dauerbetrieb wird aus und nicht releasefähig ausgeliefert; Zeile 18/27 und das Audit werden getrennt (App-Release vs. Dauerbetrieb), der Nutzer bestätigt die Abnahme selbst in einer Datei; (B) Review-Befugnis jetzt bauen (mehrere Stufe-A-Pakete an Nahtstellen, v1.5.0 deutlich später); (C) Release verschieben. | Höchste Priorität; v1.5.0-Blocker. | Nutzer | ✓ entschieden (Nutzer 04.10.): Option A — v1.5.0 = App-Release, Dauerbetrieb aus und nicht releasefähig, Abnahme per Nutzer-Attestierung | |
There was a problem hiding this comment.
At this commit, a repo-wide search finds no M4-R27-03 package, while the canonical plan still says Continuous Mode is enabled with v1.5.0 (lines 25–27 and 37) and .pa/continuous_acceptance_matrix.md:37 still requires releaseEligible: true; this new status instead says v1.5.0 ships with Continuous Mode off and not release-eligible. The blocker therefore appears resolved without any tracked work to reconcile the mutually exclusive release criteria. Add the follow-up package and dependencies here, update the criteria now, or keep E20 open until that work lands.
AGENTS.md reference: AGENTS.md:L52-L53
Useful? React with 👍 / 👎.
|
Queued — the merge queue status continues in this comment ↓. |
Cuarroc
added
the
review-ok
|
@Mergifyio queue |
Merge Queue Status
This pull request spent 12 minutes 36 seconds in the queue, including 12 minutes 5 seconds running CI. Required conditions to merge
|
Deine Entscheidung „Option A“ für E20 steht jetzt auch in docs/PLAN.md. Vorher stand dort noch „offen“, deshalb hat ein Prüfer #372 blockiert. Sonst ändert sich nichts.
Report
What changed:
docs/PLAN.mddecision inbox: E20 marked✓ entschieden (Nutzer 04.10.): Option Aand removed from the "Offen sind …" sentence. The user gave this decision on 2026-10-04 in the orchestrator session (answer "Option A" to the E20 question). Option A: v1.5.0 is an app release via PR -> gates -> Mergify queue; continuous mode ships off and is not release-eligible; the user attests acceptance himself.Why now: PR #372 review B (B1, high) blocked M4-R27-01 because
origin/main:docs/PLAN.md:489still readoffen, Nutzer. This PR provides the missing evidence. The remaining E20 doc work (matrix rows 18/27, decisions.md, README) stays in M4-R27-03.Evidence: pre-commit lane green at 695c9d8 (exit 0); push via
PA_PREPUSH=light(branchpush lane, exit 0);git ls-remoteshows 695c9d8.Tier: C (documentation only,
No-Test: documentation only). No external review needed.NICHT ABGEDECKT: no Rust suite or clippy run locally (docs-only); the linux lane runs on this PR because
docs/PLAN.mdis a heavy doc.🤖 Generated with Claude Code