Repository navigation
docs: close KI-30 with queue evidence, record KI-34 - #874
Conversation
No-Test: docs only (KNOWN_ISSUES.md) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Merge Protections🟢 All 2 merge protections satisfied — ready to merge. Show 2 satisfied protections🟢 Paket-PR bringt seinen Bericht mitPackage branches (w<N>-, df<N>, ki-<N>, hq2-, dr-, z<N>-) must carry their report in the PR body, in a section that starts with "## Report".
🟢 🚦 Auto-queueWhen all merge protections are satisfied and these conditions match, this pull request will be queued automatically.
|
R874-R1: the run 37910399142 failure (goal_task_audit_control_uses_one_connection) was fixed by hotfix #843; a new KI id duplicated it. KI-30 closing text now names it as a non-KI-30 failure. No-Test: docs only (KNOWN_ISSUES.md) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Cuarroc
added
the
review-ok
![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: 0eb83b0c9f
ℹ️ 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".
| | KI-28 | Der Capture-Host (`bin/pa-capture-host.rs`) läuft nur unter Windows | — (dokumentierte Grenze) | Entscheidung 16.09. (W3-05, `docs/decisions.md`; `pa-capture-host.rs:142`). Aus `STAND.md` übernommen 2026-09-24. Die frühere Einschränkung „acht native Tests laufen in CI nie" gilt nicht mehr: W3-06 (PR #75) fährt sie im Gate `native-tests` der Windows-Bahn. Linux-CI prüft nur die Kompilation. | | ||
| | KI-29 | F-SEC-4, Restrisiko des OmniRoute-Schlüssel-Syncs: wer den Opt-in einschaltet, schickt die Vault-Schlüssel an den Listener auf dem OmniRoute-Port, ohne dessen Identität zu prüfen | niedrig (bewusst in Kauf genommen) | Default ist seit W1-24 (PR #62) „kein Push"; W1-24b (PR #98) hat daraus ein ausdrückliches Opt-in-Setting gemacht, das im UI das Restrisiko benennt (`.pa/report_w1-24b.md`, „Hinweise und Folgearbeiten"). Eine Listener-Identität gibt es nicht: `/api/version` bräche Builds, die es weglassen, und ein Management-Token als Bearer gäbe es einem Horcher mit (PLAN, frühere Entscheidung Nr. 13). Solange der Sync aus bleibt, besteht kein Risiko. Paket W1-24c. | | ||
| | KI-30 | Windows-Gate `native-tests`: vier der acht `real_native_*`-Tests scheitern sporadisch in Merge-Queue-Läufen, jedes Mal nach grüner `rust-suite` | mittel (Flake, blockiert die Queue) | Stand 2026-10-05 (V16-01): Ursache für den Early-Exit-Fall belegt und behoben, die serielle Minderung ist entfernt. Offen bleibt der Beleg über die nächsten 40 Queue-Läufe und die Ursache der übrigen drei Tests. Details im Abschnitt „KI-30“ unten. | | ||
| | KI-30 | Windows-Gate `native-tests`: vier der acht `real_native_*`-Tests scheitern sporadisch in Merge-Queue-Läufen, jedes Mal nach grüner `rust-suite` | geschlossen 2026-10-09 (KI-30-CLOSE): 42 abgeschlossene Queue-Läufe, kein `real_native_*`-Fehler | Beleg und Methode im Abschnitt „KI-30“ unten (Schluss). 1 non-KI-30 failure (run 37910399142, goal_task_audit_control_uses_one_connection), already fixed by #843. Tritt ein `real_native_*`-Test wieder rot auf, ist das ein neuer Befund. | |
There was a problem hiding this comment.
Am geprüften Head führt docs/PLAN.md:292 den 40-Läufe-Nachweis noch als laufend; docs/PLAN.md:303-306 verlangt ausdrücklich, das bestehende Beobachtungsfenster nicht neu zu starten und KI-30 nicht vorzeitig zu schließen. Diese Zeile erklärt KI-30 dagegen anhand eines erst nach #737 beginnenden Fensters für geschlossen. Damit widerspricht der Status der einzigen kanonischen Planung und kann den Release-Vorgänger fälschlich als erfüllt erscheinen lassen; KI-30 sollte offen bleiben, bis der Plan durch den zuständigen Owner synchronisiert und die dortige Abnahme erfüllt wurde.
AGENTS.md reference: AGENTS.md:L53-L56
Useful? React with 👍 / 👎.
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. |
Merge Queue Status
This pull request spent 15 minutes 6 seconds in the queue, including 14 minutes 49 seconds running CI. Required conditions to merge
|
KI-30 ist in
KNOWN_ISSUES.mdgeschlossen: In 42 abgeschlossenen Merge-Queue-Läufen seit dem Fix gab es keinen einzigenreal_native_*-Fehler. Der eine rote Lauf (37910399142) betraf einen anderen Test, der bereits durch den Hotfix #843 behoben ist; ein neuer KI-Eintrag war deshalb falsch und wurde entfernt. Das ist reine Dokumentation, es ändert sich kein Code.Report
Geändert:
KNOWN_ISSUES.md(nur die KI-30-Zeile und der Abschluss-Absatz; kein KI-34).Nachzählung (
gh run list --workflow ci.yml --limit 200 --json databaseId,headBranch,createdAt,status,conclusion, Exit 0; FilterheadBranchbeginnt mitmergify/merge-queue,createdAt> 2026-10-09T01:11:29Z): 43 gestartet, 42 abgeschlossen (41 success, 1 failure), 1 in_progress (nicht gezählt). Erster abgeschlossener Lauf 37868576516, letzter 37958364523. Basisorigin/main= 23da083.Fehlschlag: Lauf 37910399142 (Windows): 1 non-KI-30 failure,
goal_task_audit_control_uses_one_connection(continuous.rs:1598), already fixed by #843 (merged 2026-10-09 10:40Z). 0 Treffer aufreal_native.Prepush:
bash scripts/ci/gates.sh lane prepush→ Exit 0 (CARGO_TARGET_DIR=slot2, CARGO_BUILD_JOBS=1), auch nach der Nachbesserung.Commit:
No-Test: docs only (KNOWN_ISSUES.md).Nötige PLAN-Änderung (Chief, nicht Teil dieses PR):
docs/PLAN.mdZeile 25 — KI-30 als geschlossen (09.10.2026, 42 Läufe, siehe KNOWN_ISSUES) eintragen.NICHT ABGEDECKT:
#[cfg(windows)]-Tests und Windows-Clippy (Linux-Lauf; Windows-Bahn läuft in der Merge-Queue). Der Beleg ist statistisch (keine Wiederholung in 42 Läufen), kein Ursachenbeweis für die drei übrigen Tests. Zählung ist ein Stand von 09.10. ~17:00Z; später abgeschlossene Läufe nicht enthalten.Review: ausstehend (Stufe C, keine externe Review; Gates genügen).
🤖 Generated with Claude Code
Root-Abnahme (09.10.2026 17:50Z, head 0eb83b0)
Tier C (KNOWN_ISSUES.md only, +18/−2): no external review; gates suffice. Root checked: diff only KNOWN_ISSUES.md, no
KI-34left,No-Test:trailer on both commits (test-first.sh greps the whole message), counts match Root's own count at 17:00Z (43 started / 42 completed / 1 failure = run 37910399142, not real_native, fixed by #843).Ready +
review-okbound to 0eb83b0.