Skip to content

docs(known-issues): move closed KI-30 to Behoben and condense its narrative - #1438

Merged
mergify[bot] merged 1 commit into
mainfrom
claude/known-issues-ki30
Oct 14, 2026
Merged

mergify[bot] merged 1 commit into
mainfrom
claude/known-issues-ki30

Conversation

@Cuarroc

@Cuarroc Cuarroc commented Oct 14, 2026 •

Copy link
Copy Markdown
Owner

KI-30 ist seit dem 9. Oktober geschlossen, stand aber noch in der Tabelle „Offen“ und füllte rund 175 Zeilen Erzählung in KNOWN_ISSUES.md. Diese Änderung verschiebt die Zeile nach „Behoben“ und kürzt den Abschnitt auf Ursache, Fix und den Abschlussbeleg (42 Queue-Läufe, Lauf 37910399142). Den vollen alten Text findest du weiter über git show 42ed6709:KNOWN_ISSUES.md; am Code ändert sich nichts.

Report

Was geändert wurde (nur KNOWN_ISSUES.md, +5 / −175):

  • KI-30 aus „Offen“ entfernt, in „Behoben“ eingetragen (Ursache behoben mit V16-01, PR fix(capture): drain checkpoints before an undelivered exit settles (KI-30) #518; geschlossen mit PR docs: close KI-30 with queue evidence, record KI-34 #874).
  • Abschnitt „KI-30“ von rund 177 auf 7 Zeilen gekürzt: Ursache und Fix mit Wächtertest, Abschlusszählung (42 Läufe, 41 success, 1 failure; der eine Fehlschlag ist ein anderer Test), Hinweis „statistisch, kein Ursachenbeweis“, Zeiger auf den letzten Commit mit dem vollen Text (42ed6709).
  • Keine anderen KI-Einträge und keine Versionszeile angefasst. Kein Verweis auf den Abschnitt von außerhalb der Datei (grep über den Baum, nur Code-Kommentare mit „KI-30“ ohne Anker).

Red-first: No-Test, reine Dokumentation (Tier C).

Belege

Befehl Exit
bash scripts/ci/gates.sh lane prepush (Gates bis lint grün; fe-test einmal rot) 1
npx vitest run src/shell/Shell.test.tsx (9 Tests) 0
bash scripts/ci/gates.sh --from fe-test lane prepush (fe-test, hq-test, denkraum, clippy, rust-suite grün) 0
PA_PREPUSH=light git push -u origin claude/known-issues-ki30, danach git ls-remote zeigt a9a18316 0

Die Last auf dem Server lag bei 20 bis 35 (parallele Worker). Dadurch liefen im Hook-Lauf 5-s-Tests im Frontend (Shell.test.tsx, LeitstandRoute.test.tsx, renderBudget.test.tsx) in Zeitüberschreitungen; mit dem Diff (nur Markdown) haben sie nichts zu tun. Kein Test wurde angepasst. Der Push gelang erst im vierten Versuch: die ersten beiden mit dem vollen Hook-Lauf und der dritte mit PA_PREPUSH=light (dokumentierter Schalter CI-02, keine --no-verify-Umgehung) scheiterten an den Last-Timeouts in fe-test; der vierte, mit demselben Schalter und Last unter 10, war grün.

Befundtabelle

ID Quelle Schwere Befund Disposition
F-1 Hook-Lauf fe-test niedrig Drei Frontend-Tests laufen unter Serverlast in 5-s-Timeouts follow-up (nicht Teil dieses Pakets, am unveränderten Code einzeln grün)

NICHT ABGEDECKT

  • Die #[cfg(windows)]-Tests und die Windows-Arme von clippy (Linux-Lauf); die Windows-Bahn läuft in der Merge-Queue.
  • Browser-Smoke, Frontend-Build und Workflow-Gates (laufen erst in der Bahn linux).
  • Ob der gekürzte Text jede Aussage des alten Abschnitts bewahrt: der Abgleich ist von Hand erfolgt, ohne Werkzeug.

Review: ausstehend (Stufe C, kein externes Review; die Gates genügen).

🤖 Generated with Claude Code

…rative

The KI-30 row was closed on 2026-10-09 but still sat in the Offen table and its
narrative filled ~175 lines. Move the row to Behoben, keep cause, fix and the
closing evidence (42 queue runs, run 37910399142) in a short section and point
to the last commit holding the full text.

No-Test: documentation only

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@Cuarroc
Cuarroc marked this pull request as ready for review October 14, 2026 14:07
@Cuarroc Cuarroc added the review-ok Pipeline: review disposition complete, no open high finding label Oct 14, 2026
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@Cuarroc

Cuarroc commented Oct 14, 2026

Copy link
Copy Markdown
Owner Author

Root: tier C (docs / deletion of unreferenced scripts, no runtime effect), gates suffice (AGENTS.md rule 5). review-ok bound to head a9a1831.

@mergify mergify Bot added the queued label Oct 14, 2026
@mergify mergify Bot mentioned this pull request Oct 14, 2026
30 of 46 tasks
@mergify
mergify Bot merged commit 4e47c78 into main Oct 14, 2026
20 checks passed
@mergify

mergify Bot commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

Merge Queue Status

  • ✅ Entered queue — 2026-10-14 14:08 UTC · Rule: default · triggered by merge protections
  • ✅ Checks passed · on draft merge queue: checking #1438 on main (67dc46b) #1441
  • ✅ Merged — 2026-10-14 14:25 UTC · at 4e47c782fc805fe1169eabf327f0e6beb517a2bf · merge

This pull request spent 17 minutes 8 seconds in the queue, including 16 minutes 25 seconds running CI.

Required conditions to merge
  • check-success = gates (linux)
  • check-success = gates (windows)
  • check-success = red-first
  • github-review-approved [🛡 GitHub repository ruleset rule main-pr-required-checks]
  • label != do-not-merge
  • any of [🛡 GitHub repository ruleset rule main-pr-required-checks]:
    • check-success = @github-actions/gates (linux)
    • check-neutral = @github-actions/gates (linux)
    • check-skipped = @github-actions/gates (linux)
  • any of [🛡 GitHub repository ruleset rule main-pr-required-checks]:
    • check-success = @github-actions/gates (windows)
    • check-neutral = @github-actions/gates (windows)
    • check-skipped = @github-actions/gates (windows)
  • any of [🛡 GitHub repository ruleset rule main-pr-required-checks]:
    • check-success = @github-actions/red-first
    • check-neutral = @github-actions/red-first
    • check-skipped = @github-actions/red-first
  • any of [🛡 GitHub repository ruleset rule main-pr-required-checks]:
    • check-neutral = @mergify/Mergify Merge Protections
    • check-skipped = @mergify/Mergify Merge Protections
    • check-success = @mergify/Mergify Merge Protections

@mergify
mergify Bot deleted the claude/known-issues-ki30 branch October 14, 2026 14:25
@mergify mergify Bot removed the queued label Oct 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review-ok Pipeline: review disposition complete, no open high finding

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant