Skip to content

feat(migrations): add atomic checkpoint storage operator - #95

Closed
seonghobae wants to merge 3 commits into
agent/checkpoint-audit-trail-v3from
agent/checkpoint-migration-operator-v3
Closed

feat(migrations): add atomic checkpoint storage operator#95
seonghobae wants to merge 3 commits into
agent/checkpoint-audit-trail-v3from
agent/checkpoint-migration-operator-v3

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Linearized replacement for #80

This Draft replays the atomic checkpoint-migration operator onto the current audit predecessor #94 without rewriting #94 or discarding the later #60 durable-store DSN boundary.

RCA and replacement boundary

#80 targeted the superseded #79 branch. Current #94 is the linearized audit replacement on #92 and differs from the former #79 baseline only through the current predecessor chain plus the preserved checkpoint-DSN CHANGELOG entry. Before publication, every #80-modified path that overlaps #94 was checked against the former #79 result. .github/workflows/ci.yml, AGENTS.md, ARCHITECTURE.md, CLAUDE.md, pg_llm_batch/__init__.py, and tests/test_checkpoint_audit_ci_contract.py were byte-identical between old #79 and current #94; only CHANGELOG.md had the expected newer DSN entry.

The replacement therefore reuses the exact #80 result blobs for all non-CHANGELOG paths and applies the same migration-operator CHANGELOG delta to #94's current CHANGELOG. No conflict-side selection, force-push, feature rewrite, migration rewrite, workflow weakening, release action, or evidence transfer is used.

Exact replacement evidence

Historical #80 checks, reviews, and approvals do not transfer. Successful staged CI/release-acceptance is not final protected-main security/review/policy evidence.

Dependency and merge boundary

Required order is .github#790 -> pg-llm-batch#53 -> #57 -> #58 -> #59 -> #60 -> #92 -> #94 -> this replacement -> #96 -> #97.

Keep Draft while earlier prerequisites remain unintegrated. After prerequisite integration, reconcile/retarget only when integration correctness requires it and regenerate every applicable exact-head/exact-base CI, live PostgreSQL rollback/concurrency, security, dependency, SBOM/provenance, packaging, automated-review, branch-protection, repository-policy, and Release Acceptance gate. Merge only after the unchanged final head has zero valid unresolved findings, all required repository/security/release gates terminal-success, branch policy satisfied, and a qualifying independent non-author formal APPROVED review where required. Queued, pending, cancelled, skipped-required, absent, neutral-required, stale-head/base, predecessor, status-only, synthetic-only, author-only, rate-limited or failed evidence is not acceptance.

@coderabbitai

coderabbitai Bot commented Aug 9, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e03e374b-0023-42c8-86c8-9e316e327bbc

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Aug 9, 2026

Copy link
Copy Markdown
⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Aug 9, 2026

Copy link
Copy Markdown
⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

Please review exact current replacement head 7ada507901cbba6497348812a8c015403a5f38aa against exact #94 predecessor cc7c862cff76521ead8696c7d6bfe1a2c4b31780. Focus on bounded canonical migration reads, advisory-lock/all-or-nothing transaction behavior, live rollback/concurrency semantics, CLI/API authority, and preservation of the current predecessor chain. Do not transfer evidence from superseded #80.

@coderabbitai

coderabbitai Bot commented Aug 9, 2026

Copy link
Copy Markdown

@seonghobae I will review 7ada507901cbba6497348812a8c015403a5f38aa against cc7c862cff76521ead8696c7d6bfe1a2c4b31780. I will assess the requested migration, transaction, rollback/concurrency, CLI/API, and predecessor-chain boundaries. I will not use evidence from superseded #80.

⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

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