Skip to content

fix(ci): diagnose retained checkpoint readback failures - #288

Merged
vincentkoc merged 2 commits into
mainfrom
fix/checkpoint-readback-diagnostics-20261017
Oct 17, 2026
Merged

vincentkoc merged 2 commits into
mainfrom
fix/checkpoint-readback-diagnostics-20261017

Conversation

@vincentkoc

Copy link
Copy Markdown
Member

What Problem This Solves

Private checkpoint migration errors did not identify which GitHub operation failed, leaving operators unable to inspect a retained draft safely.

User Impact

API failures now report a fixed operation, exit category, and HTTP status when available. The existing manual migration workflow can verify a pinned failed draft and its first part without restoring a database or changing the checkpoint.

Why This Change Was Made

Public admission and private checkpoint requests share one bounded process owner while retaining separate credentials. It drains private output without logging it and verifies that its process group has stopped before consuming results. The diagnostic binds the reviewed current source separately from the original failed migration and rechecks custody after the read.

A successful diagnostic proves only the current read path and selected part. It does not classify an earlier unrecorded failure or establish full recovery.

Evidence

  • 38 focused Python tests passed, including stderr flooding, inherited descriptors, unknown cleanup, credential separation, evidence drift, corrupt parts, and first-part-only reads.
  • Three native publication/workflow regressions passed.
  • Two independent source reviews and fresh P3 autoreview passed on the exact head.
  • Hosted production diagnostic remains a separate reviewed operation; no live retry, upload, promotion, or cache deletion has been performed by this change.

@clawsweeper

clawsweeper Bot commented Oct 17, 2026

Copy link
Copy Markdown
Contributor

🦞👀
ClawSweeper picked this up.

Pull request received. I will update this pull request when review starts.

@vincentkoc
vincentkoc marked this pull request as ready for review October 17, 2026 14:48
@vincentkoc
vincentkoc requested a review from a team as a code owner October 17, 2026 14:48
@vincentkoc
vincentkoc merged commit 463348d into main Oct 17, 2026
18 checks passed
@vincentkoc
vincentkoc deleted the fix/checkpoint-readback-diagnostics-20261017 branch October 17, 2026 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant