Skip to content

docs: reconcile fixed known issues with merge evidence (Z2-DOKSYNC) - #670

Merged
mergify[bot] merged 1 commit into
mainfrom
codex/z2-doksync-known-issues
Oct 8, 2026
Merged

mergify[bot] merged 1 commit into
mainfrom
codex/z2-doksync-known-issues

Conversation

@Cuarroc

@Cuarroc Cuarroc commented Oct 8, 2026

Copy link
Copy Markdown
Owner

Die veralteten offenen Einträge KI-15, KI-16 und KI-20 sind mit PR-Nummern und Merge-SHAs bereinigt.
Für KI-33 belegt ein Windows-Queue-Lauf den erfolgreichen Test in 1,812 Sekunden, deshalb steht auch dieser Eintrag unter „Behoben“.
Die Grenzen zu ConPTY und zum fehlenden Live-Kimi-Test bleiben ausdrücklich dokumentiert.

Report

Only KNOWN_ISSUES.md changed: 4 insertions, 5 deletions. KI-15 and KI-16 moved to the fixed table; the stale KI-20 open row was removed and its existing fixed row retained. KI-33 moved after observing the required Windows queue evidence. No runtime or dependency changes.

Issue Fix PR Merge SHA
KI-15 #116 c3de8410ff4e4335fc24a5149d0ecbc5bb515928
KI-16 #141 / #438 376f75f55406d813732dc8a638c5a94d06e90e02 / cd7ec3207e84b3cafd7947d92ae691701aa24a0e
KI-20 #140 56266cce31a905b6f3e946e774bfd3ad1ccd3044
KI-33 #543 04ea1a4313649d8882b70e0bd11643e7ee0ac525

Evidence observed 2026-10-08:

  • gh pr list --state merged --search 'KI-15' --json number,title,mergeCommit,mergedAt — exit 0; equivalent searches for KI-16 and W1-27 — exit 0 each. gh pr view 438 --json number,title,mergeCommit,mergedAt — exit 0. Cross-checked against git log origin/main --oneline --regexp-ignore-case --grep='ki-15\|ki-16\|w1-27\|retention\|#438\|#141' — exit 0.
  • gh run list --workflow ci --limit 60 --json databaseId,headBranch,headSha,event,conclusion,createdAt — exit 0. Queue run 37767556830, head 729393d684f1fa94b00439ccfd5af80233a466f0, Windows job 113278804180: success.
  • gh run view 37767556830 --job 113278804180 --log — exit 0; retention::tests::deletion_runs_in_chunks_until_the_table_is_clean: PASS [1.812s].
  • gh api repos/Cuarroc/ProjectA/compare/04ea1a4313649d8882b70e0bd11643e7ee0ac525...729393d684f1fa94b00439ccfd5af80233a466f0 --jq '{status,merge_base: .merge_base_commit.sha}' — exit 0; ahead, merge base equals the Batch retention chunk-test inserts; record KI-33 Windows timeout #543 merge SHA, proving the queue candidate contains the change.
  • rg -n KNOWN_ISSUES scripts src-tauri src — exit 0. Matches are path classification, fixture data and output text; no content reader found. bash scripts/test-lane-plan.sh — exit 0, including the real-repository light-doc classification.
  • git diff --check — exit 0.
  • bash scripts/ci/gates.sh lane precommit — exit 0, including the staged secret scan; Rust checks correctly skipped for the docs-only staged diff.
  • bash scripts/ci/gates.sh lane prepush — exit 0 at candidate e13d0032e849bd9c2b8325dd3fceca8d4c357ff0; 1873 Rust tests passed, 7 skipped. The push hook repeated the prepush lane successfully.
  • git push -u origin codex/z2-doksync-known-issues — exit 0. git ls-remote origin codex/z2-doksync-known-issues — exit 0, remote head e13d0032e849bd9c2b8325dd3fceca8d4c357ff0.

Process deviation: the initial npm run dev:start-check and the check before the explicit prepush lane exited 0. The check immediately before push exited 1 because two cargo builds were running (10.6 GB available); the sequential command batch incorrectly proceeded to push instead of waiting. Both prepush runs completed successfully. Provider usage and actual model identity were not verified by the start check.

NICHT ABGEDECKT

Linux does not compile or run the #[cfg(windows)] tests (including ConPTY and DPAPI) or Windows clippy branches. The candidate's Windows lane remains for the merge queue; the observed KI-33 run is historical evidence for that fix only. One passing sample does not prove long-term flake freedom. KI-16 still lacks a dedicated spawn_pty test and a ConPTY observation; KI-20 retains the fixed 1;1 response and missing live Kimi smoke. Browser smoke, frontend build and workflow gates belong to the full linux lane; external service state is not covered.

Review: Stufe C (Gates genügen)

Handover: base a167486c37c3947a9faeaa7c699937db85285fa0, candidate e13d0032e849bd9c2b8325dd3fceca8d4c357ff0. Native goal tools returned goal: null initially, then created and re-read the bounded active Z2-DOKSYNC worker goal. Worker acceptance is the verified draft handover; acceptance, Ready and merge remain with Root/Chief. Chief follow-up: Z2-DOKSYNC → Erledigt in docs/PLAN.md; this package does not edit PLAN.

Commit trailer: No-Test: documentation-only known-issues bookkeeping.

🤖 Generated with Codex on the ProjectA server worker

No-Test: documentation-only known-issues bookkeeping
@Cuarroc
Cuarroc marked this pull request as ready for review October 8, 2026 14:18
@Cuarroc Cuarroc added the review-ok Pipeline: review disposition complete, no open high finding label Oct 8, 2026
@mergify mergify Bot added the queued label Oct 8, 2026
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-08T14:21:00.659515Z e13d003 Draft marked ready
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@mergify
mergify Bot merged commit 010f62f into main Oct 8, 2026
17 checks passed
@mergify

mergify Bot commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Merge Queue Status

This pull request spent 15 minutes 14 seconds in the queue, including 12 minutes 40 seconds running CI.

Required conditions to merge

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