You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This branch established the right additive direction and valuable PostgreSQL tests, but concurrent branch-only repair writers left multiple temporary workflows/scripts and repeatedly changing structural contracts on the same head. It also began with the analysis knowledge cutoff on the reusable snapshot, nullable requester identity, globally scoped idempotency, and an append-only event bag without a serialized legal state machine.
PR #85 is the clean canonical replacement on the same reviewed #74 product line. It preserves the unique value of this branch while resolving the product contracts in one coherent schema:
snapshot-owned evidence availability and capture time;
run-owned knowledge cutoff;
non-null real user_account requester;
idempotency scoped to the authenticated account;
immutable snapshot, aggregate, and run-request evidence;
one shared snapshot-row lock order for count freeze versus first run;
pending-first, contiguous, monotonic, legal status transitions and terminal finality;
fail-closed rollback, current-status view, 3NF, and descriptive database-object names;
no temporary repair workflow or script in the intended final diff.
This PR is closed without deleting its branch so its research, tests, rollback concepts, and repair history remain available for audit. Do not merge or reopen it alongside #85; that would create competing migration 0018 and ADR 0013 authorities.
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: 4dc04151-4164-455a-b82e-ea55bcadf479
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.
Please perform an independent review-only assessment of exact head 899f99d4dd2ddd61032e8091cc48975fc9795f29. Verify migrations 0018/0019, temporal leakage, snapshot/count locking, account-scoped idempotency, run/scope immutability, lifecycle transition enforcement, rollback replay safety, 3NF/naming, public-content exclusions, and the current claim boundary. Do not push changes or treat stacked-parent evidence as merge approval.
seonghobae
changed the title
feat: add normalized temporal analysis-run registry (v0.79.0)
feat: retain analysis-run registry prototype as superseded evidence
Aug 15, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Superseded by clean canonical delivery in #85
This branch established the right additive direction and valuable PostgreSQL tests, but concurrent branch-only repair writers left multiple temporary workflows/scripts and repeatedly changing structural contracts on the same head. It also began with the analysis knowledge cutoff on the reusable snapshot, nullable requester identity, globally scoped idempotency, and an append-only event bag without a serialized legal state machine.
PR #85 is the clean canonical replacement on the same reviewed #74 product line. It preserves the unique value of this branch while resolving the product contracts in one coherent schema:
user_accountrequester;This PR is closed without deleting its branch so its research, tests, rollback concepts, and repair history remain available for audit. Do not merge or reopen it alongside #85; that would create competing migration 0018 and ADR 0013 authorities.
Superseded by #85. Refs #79.