Skip to content

test: cover recovery after Pi retry - #10

Merged
c0da-ricon merged 1 commit into
ikma/fix-committed-leaf-recoveryfrom
c0da/pr9-auto-retry-recovery-test
Aug 7, 2026
Merged

test: cover recovery after Pi retry#10
c0da-ricon merged 1 commit into
ikma/fix-committed-leaf-recoveryfrom
c0da/pr9-auto-retry-recovery-test

Conversation

@c0da-ricon

Copy link
Copy Markdown
Contributor

Summary

  • add a real SessionManager regression for Pi's persisted auto-retry shape
  • prove one visible user/assistant pair can validly map to user, retryable assistant error, and successful assistant entries
  • verify restart selects the exact committed leaf without interpreting raw Pi message cardinality

Why

PR #9 correctly removes role-count matching from recovery, but needs a regression demonstrating the valid runtime shape that motivates removing that guard.

Validation

  • focused recovery test: 6/6 passed
  • full equivalent check: strict TypeScript, 59/59 tests, all six Codebase lints, and whitespace checks
  • git pre-commit
  • git pre-push (expected new-branch and locally unknown GitHub merge-signature warnings only)

Test-only change. No model call, service start, deployment, or merge.

@c0da-ricon
c0da-ricon merged commit 1679940 into ikma/fix-committed-leaf-recovery Aug 7, 2026
1 check passed
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