Repository navigation
docs(plan): mark merged PRs Erledigt in stale status cells (AP-5 step 2) - #1439
Conversation
No-Test: documentation only (PLAN status cells) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Cuarroc
added
the
review-ok
|
Root: tier C (docs / deletion of unreferenced scripts, no runtime effect), gates suffice (AGENTS.md rule 5). review-ok bound to head ec4b854. |
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: ec4b854b1f
ℹ️ 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".
| | DR-11 | UI-Harness auf `@playwright/test` portieren (nur relative Pfade), permanenter D4-Fall | UX-Architekt | DR-10/#665 und HOTFIX-RF-PW/#741 | Server | C bei reinem Testdiff; höhere Risiken nach Regel 5 | Erledigt #718 (merged 2026-10-09T13:38:46Z); ersetzt: In Prüfung: Draft #718 | B1-Harness und D4-Test: fehlende/leere Root-ID sperrt alle POST-Aktionen mit Grundtext und ohne Writes; Harness läuft im Gate auf dem Server | 55ea21ce | | ||
| | DR-11-FU | Denkraum-UI Review-Nachzüge nach #718 | DR-11-Owner | DR-11/#718 gemergt | Server | B | Erledigt #870 (merged 2026-10-09T17:35:03Z) | Review-Follow-ups gemergt | – | | ||
| | DR-FU-0910g | Review-Nachzüge 09.10. Abend (R870-LOWS, R833-LOWS, R814-G1, R824-G1, R871-A7, R877-LOWS, R887-FU) | Root/Orchestrator | diverse Vorgänger-PRs | Server | B/C | R870-LOWS Erledigt #882 (merged 2026-10-09T18:24:36Z); R833-LOWS Erledigt #878 (merged 2026-10-09T18:38:04Z); R814-G1 Erledigt #877 (merged 2026-10-09T18:10:57Z); R824-G1 Erledigt #875 (merged 2026-10-09T18:17:58Z); R871-A7 Ready #890; R877-LOWS Ready #887; R887-FU Draft #892 | beobachtete PR-Zustände via gh; keine neuen Tabellen | PLAN-SYNC-0910g | | ||
| | DR-FU-0910g | Review-Nachzüge 09.10. Abend (R870-LOWS, R833-LOWS, R814-G1, R824-G1, R871-A7, R877-LOWS, R887-FU) | Root/Orchestrator | diverse Vorgänger-PRs | Server | B/C | R870-LOWS Erledigt #882 (merged 2026-10-09T18:24:36Z); R833-LOWS Erledigt #878 (merged 2026-10-09T18:38:04Z); R814-G1 Erledigt #877 (merged 2026-10-09T18:10:57Z); R824-G1 Erledigt #875 (merged 2026-10-09T18:17:58Z); R871-A7 Erledigt #890 (merged 2026-10-09T19:56:03Z); R877-LOWS Erledigt #887 (merged 2026-10-09T19:41:45Z); R887-FU Erledigt #892 (merged 2026-10-09T20:09:42Z) | beobachtete PR-Zustände via gh; keine neuen Tabellen | PLAN-SYNC-0910g | |
There was a problem hiding this comment.
When HQ parses this updated row, scripts/lib/hq-parse.mjs::goalStatus only recognizes a defined status at the beginning of the cell; because the cell still begins with R870-LOWS, DR-FU-0910g remains invalid even though every listed follow-up is now complete. Prefix the cell with Erledigt: so HQ counts and displays this canonical-plan package as completed.
AGENTS.md reference: AGENTS.md:L60-L66
Useful? React with 👍 / 👎.
| | DR-15a-BASE | Z1: exakt geprüftes Startmodul `5f501f1` erhalten; lexikalische Prüfung ohne Dateisystemauflösung | vorhandener Implementierer · Codex | DR-01; gemeinsame Abnahme mit DR-15a-CLI | Server | A | Erledigt: #647 `5f501f1` durch Mergify am 08.10. 07:26:49 UTC gemergt (`cd3bffa`); Lieferung zusammen mit CLI, DR-15b offen | unveränderter Head `5f501f1`, alter eigener prepush 22/188s Exit 0 und Reviewbelege erhalten; Draft auf #635; Lieferung erst mit CLI-Korrektur erfüllt | 868dbc83 | | ||
| | DR-15a-CLI | Z1: vorhandenen Alias-Entry-Fix getrennt abnehmen; bestehende Commits erhalten, keine Wiederholung | derselbe Implementierer · Codex | DR-15a-BASE | Server | A | Erledigt: #639 `3bf5a8b` durch Mergify am 08.10. 08:09:55 UTC gemergt (`7ac5985`); kombinierte Source-Abnahme erhalten, DR-15b offen | #639 auf Baseline retargetet, gleicher Head; Symlink/Junction und Node-24.0-Einstieg benannt rot→grün; eigener prepush am neuen Head Exit 0; Node >= 24 erhalten; zwei Vendoren außerhalb OpenAI am neuen Delta und Root-Abnahme | f1a2ade9 | | ||
| | DR-15b | Startintegration nach Server; bestehendes Startmodul/Entry aus 15a, konkreten Dateischnitt vor Dispatch pinnen | Chief weist einen Codex-Owner zu | DR-06a und integrierte DR-15a-BASE/CLI | eigener Worktree | A | In Prüfung: Draft #750 | loadStartConfig bei fehlender ROOT_AGENT_ID an 06a angleichen: Start verweigern statt nur Benachrichtigungen abschalten; physischer externer STATE-Pfad/H2 und Secret-Manager-Vertrag erhalten; Rot, eigene prepush/zwei Fremdreviews | a463151a | | ||
| | DR-15b | Startintegration nach Server; bestehendes Startmodul/Entry aus 15a, konkreten Dateischnitt vor Dispatch pinnen | Chief weist einen Codex-Owner zu | DR-06a und integrierte DR-15a-BASE/CLI | eigener Worktree | A | Erledigt #750 (merged 2026-10-09T01:52:45Z); ersetzt: In Prüfung: Draft #750 | loadStartConfig bei fehlender ROOT_AGENT_ID an 06a angleichen: Start verweigern statt nur Benachrichtigungen abschalten; physischer externer STATE-Pfad/H2 und Secret-Manager-Vertrag erhalten; Rot, eigene prepush/zwei Fremdreviews | a463151a | |
There was a problem hiding this comment.
Marking DR-15b complete here leaves the surrounding canonical statuses contradictory: DR-15 and both DR-15a rows on lines 99–101 still explicitly state that DR-15b is open. Update those references in the same status synchronization—while retaining any genuinely open overall user acceptance—so agents do not treat already merged work as outstanding.
AGENTS.md reference: AGENTS.md:L60-L66
Useful? React with 👍 / 👎.
| | DR-SEC-01g-G4 | Server-Lifecycle / Admission | Root bindet einen Owner im Ticket | DR-SEC-01g-G3 | Server | A | Aufgeteilt G4a Signal-Stop / G4b Admission+Drain; ersetzt: Geplant (Design-Bericht) | zweiter Listener mutiert/sendet nicht; Drain vor Release; Interface für DR-16a | b0221f4a | | ||
| | DR-SEC-01g-G4a | Signal-Stop: Ownership freigeben | Root bindet einen Owner im Ticket | DR-SEC-01g-G3 | Server | A | Erledigt #872 (merged 2026-10-09T18:24:41Z); ersetzt: In Prüfung: Draft #872 | Signal-Stop gibt Ownership frei | b0221f4a | | ||
| | DR-SEC-01g-G4b | Admission+Drain | Root bindet einen Owner im Ticket | DR-SEC-01g-G4a | Server | A | In Prüfung: #889 (Runde 2); ersetzt: Geplant, Spec in Arbeit | zweiter Listener mutiert/sendet nicht; Drain vor Release | b0221f4a | | ||
| | DR-SEC-01g-G4b | Admission+Drain | Root bindet einen Owner im Ticket | DR-SEC-01g-G4a | Server | A | Erledigt #889 (merged 2026-10-09T19:45:58Z); ersetzt: In Prüfung: #889 (Runde 2) | zweiter Listener mutiert/sendet nicht; Drain vor Release | b0221f4a | |
There was a problem hiding this comment.
Completing G4b means both split children G4a and G4b are now terminal, but the parent DR-SEC-01g-G4 row still begins with Aufgeteilt, which scripts/lib/hq-parse.mjs::goalStatus classifies as invalid; DR-16a also already records G4 as merged. Give the parent a valid terminal prefix such as Erledigt: or Entfallen: and retain the split explanation afterward so HQ no longer reports this canonical-plan package as invalid.
AGENTS.md reference: AGENTS.md:L60-L66
Useful? React with 👍 / 👎.
Merge Queue Status
This pull request spent 17 minutes 30 seconds in the queue, including 10 minutes 9 seconds running CI. Required conditions to merge
|
Ich habe in
docs/PLAN.mdsieben veraltete Statuszellen auf „Erledigt" gesetzt, weil die zugehörigen Pull Requests laut GitHub längst gemergt sind. Betroffen sind #741, #887, #890, #892, #750, #889, #725, #784 und #780, jeweils mit PR-Nummer und Merge-Zeitpunkt. Es ist reine Statuspflege (AP-5 Schritt 2): keine Umstrukturierung, kein Kopfbereich, keine Code-Änderung.Report
Was geändert:
docs/PLAN.md, 7 Zeilen (je 1 Zeile ersetzt): HOTFIX-RF-PW (#741), DR-FU-0910g (#890/#887/#892), DR-15b (#750), DR-SEC-01g-G4b (#889), Z2-RAM-SLOTS (#725), FLAKE-Q-TMP (#784), KI25-SETUP-XDEAD (#780). Format wie die übrigen Zellen:Erledigt #N (merged <UTC>); ersetzt: <alter Status>. Nur Statuszellen, sonst nichts angefasst. Geprüfte Zeilennummern: 92, 95, 102, 114, 216, 321, 322 (Auftragsliste stimmte).Belege:
gh pr view <n> --json state,mergedAtfür 741, 887, 890, 892, 750, 889, 725, 784, 780: alleMERGED(Zeitstempel stehen in den Zellen).bash scripts/ci/gates.sh lane prepush: Lauf 1 Exit 1 (Gate fe-test: Zeitüberschreitung inSettingsView.test.tsxbei Serverlast, 1307/1307 Tests grün, nur ein unbehandelter Timer-Fehler). Danachgates.sh --from fe-test lane prepush: Exit 0 (fe-test, hq-test, denkraum, clippy, rust-suite grün).git push(Pre-push-Hook führt prepush aus): Versuch 1 Exit 1 (denkraum, Playwright D4 Timeout 30 s), Versuch 2 Exit 1 (fe-test:LeitstandRoute.test.tsx,renderBudget.test.tsxTimeout 5 s), Versuch 3 Exit 0 nach Abklingen der Serverlast (Load 37 auf 8 Kernen → 13). Alle drei Fehlschläge sind Zeitlimits bei Serverlast in Dateien, die dieser PR nicht berührt; derselbe Kopf war in anderen Läufen grün. Kein Test wurde verändert.git ls-remote origin claude/ap5-plan-status=ec4b854b…Red-first: No-Test: reine Dokumentation (PLAN-Statuszellen).
Architecture drift: keine Änderung.
Findings
SettingsView.test.tsx, Playwright D4,LeitstandRoute.test.tsx,renderBudget.test.tsxbei Serverlast ~37NICHT ABGEDECKT:
#[cfg(windows)]-Tests und Windows-Arme von clippy (nur Linux gelaufen; Windows-Bahn läuft in der Merge-Queue); Browser-Smoke/Frontend-Build/Workflow-Gates der Bahnlinux(nurprepushlokal gelaufen). Offen und nicht geändert: ältere „In Prüfung"-Zellen anderer PRs (AP-5 Schritte 1/3/5) wurden nicht geprüft.Review: ausstehend (Stufe C: kein externes Review, die Gates genügen).
PONYTAIL: applied (kleinste vollständige Änderung: nur die 7 Zellen, kein Skript im Repo).
🤖 Generated with Claude Code