docs(crdt): recover durable link intent decision - #196
christian-byrne wants to merge 2 commits into
Conversation
|
Warning Review paused — included plan limit reachedKeep your review moving with free on-demand reviews.
On-demand reviews are free for the next 23 days.
Reviews can continue after your included limit without a manual trigger. An admin must approve usage-based billing. Promotion and pricing detailsOn-demand reviews are free for the next 23 days. After that, they cost $0.25 per reviewed file. Review limit detailsOr wait 28 minutes for your next included review. Limit details: You’ve used the included review currently available. Your 113 included PR review attempts over the past 7 days set your current allowance at 1 review per hour. Review configuration: ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (1)
Comment |
|
Exact-head Fallow audit for |
Preserves the proposed durable-link design from frontend PR #16819. Design and human review history only; runtime remains separate.
Full context for agent readers
Scope
main.removed_links, kind-specific concrete/promoted/autogrow eligibility, host-owned v2 to v3 migration, compaction retention, and the permanent test matrix.Review carryover
The original approval remains attributable historical evidence only and does not approve this replacement. DrJKL's approval reviewed original commit
72952823f37fe0c3d1242551ea907024e4f49ac6, not this PR.All six human inline records from three unresolved, outdated source threads are accounted for against original reviewed commit
72952823f37fe0c3d1242551ea907024e4f49ac6:PRRC_kwDOMIrOxs7qPGJz, DrJKL, 2026-09-04: imported v2 baseline and malformed-only refusal. Fixed in the source final and carried into the migration clauses here; replacement text verified.PRRC_kwDOMIrOxs7qQIGp, christian-byrne, 2026-09-04: source response documenting that fix. Context-only provenance; its claims were rechecked against this ADR.PRRC_kwDOMIrOxs7qPGJ5, DrJKL, 2026-09-04: model concrete, promoted, and autogrow eligibility separately. Fixed in the source final and carried into the eligibility table here; replacement text verified.PRRC_kwDOMIrOxs7qQIH7, christian-byrne, 2026-09-04: source response documenting that fix. Context-only provenance; its claims were rechecked against this ADR.PRRC_kwDOMIrOxs7qPGJ_, DrJKL, 2026-09-04:clearcannot create link tombstones without explicit link IDs. Fixed in the source final and carried into thecleartransition here; replacement text verified.PRRC_kwDOMIrOxs7qQIJd, christian-byrne, 2026-09-04: source response documenting that fix. Context-only provenance; its claims were rechecked against this ADR.The source threads remain unresolved and outdated. No thread state or approval is fabricated here. The later runtime alternative in frontend PR #16866 remains a dependency/resume record; this bounded checkpoint does not implement or adopt that runtime patch.
Provenance
Source final:
0fa26a3908f6b7fd74bdb8d8bf097ff4a73804f8; source base:0e3d52bae389c36f0698a0d588e30f4f3b8dd9ab. The source file was transferred alone, not with its frontend parent. After correcting the status inference, its content is byte-identical to the source final. Raw collaboration exports remain private.Runtime completion still requires the ADR's full schema-v3 implementation gates, migration and compaction behavior, packed-package/consumer validation, and affected QA reruns. No package is published or deployed by this PR.
Verification
git diff --checknpm run buildnpm run check:puritynpm run check:importsnpm run check:pinsnpm run check:profile-claimsnpm run check:coderabbitnpm run verify:corpusnpm testThe full suite of 985 tests ran at the initial recovery head. The follow-up status-only correction received an exact archived-file comparison and diff check; hosted checks must be evaluated at the new head.