Skip to content

docs(coil): fix seam-ledger rows the #71 rename left pointing at t3x names - #116

Merged
radroid merged 1 commit into
mainfrom
coil/seams-stale-rename-refs
Aug 15, 2026
Merged

docs(coil): fix seam-ledger rows the #71 rename left pointing at t3x names#116
radroid merged 1 commit into
mainfrom
coil/seams-stale-rename-refs

Conversation

@radroid

@radroid radroid commented Aug 15, 2026

Copy link
Copy Markdown
Owner

Four rows in docs/coil/SEAMS.md still described the update-delivery seam using its pre-rename
identifiers. None of them exist in the tree any more:

Row Claimed Actually
packages/contracts/src/ipc.ts t3xUpdate, src/t3x/updateDelivery.ts coilUpdate, src/coil/updateDelivery.ts
apps/desktop/src/preload.ts t3xUpdate bridge object coilUpdate
apps/desktop/src/ipc/channels.ts four t3x:update-* constants coil:update-*
apps/desktop/src/ipc/DesktopIpcHandlers.ts three t3xUpdate handlers coilUpdate handlers, now named

The ledger is the fork's map of its own upstream surface. A row naming a symbol that no longer
exists is worse than no row at all, because it reads as verified.

Deliberately not a blanket rename. scripts/t3x/**, .github/workflows/t3x-*.yml and the
t3x-sync label all kept the old name through #71, and line 49's note about the rename itself is
correct history. Only the source namespaces and docs moved, so t3x in a path is not automatically
stale.

Every identifier the file now claims was grepped against the tree before committing: coilUpdate,
coil/updateDelivery.ts, coil:update-state, getCoilUpdateState, restartIntoUpdate,
dismissCoilUpdate — all present.

Header total

Also corrects +2621 to +2622. Regenerated against origin/main per the file's own Regenerating
section (merge-base 196c8ea0d, matching the header; HEAD == origin/main): 51 files,
+2622 / -1093
. File count and deletions were already right.

That is the third time these totals have been wrong when checked — +1616 against a real +1627
on 2026-08-05, now this. The file's instruction to regenerate before trusting it keeps being
correct.

Verification

Docs-only; no code paths touched. vp fmt ran clean via the pre-commit hook.

Model: Claude Opus 5 (1M context). Harness: Claude Code.

🤖 Generated with Claude Code

…names

Four rows in the ledger still described the update-delivery seam with its
pre-rename identifiers: a `t3xUpdate` bridge object, four `t3x:update-*`
channel constants, three `t3xUpdate` handlers, and interfaces at
`src/t3x/updateDelivery.ts`. None of those exist — the code calls them
`coilUpdate`, `coil:update-*`, `getCoilUpdateState` / `restartIntoUpdate` /
`dismissCoilUpdate`, and `src/coil/updateDelivery.ts`.

The ledger is the fork's map of its own upstream surface, so a row naming a
symbol that no longer exists is worse than no row: it reads as verified.

Not a blanket rename. `scripts/t3x/**`, `.github/workflows/t3x-*.yml` and the
`t3x-sync` label all kept the old name through #71, and line 49's note about
the rename itself is correct history. Only source namespaces and docs moved.

Also corrects the header total, which read +2621 against a measured +2622.
That is the third time the totals have been wrong when checked (+1616 vs
+1627 on 08-05), which is the file's own "regenerate before trusting it"
advice earning its place.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Aug 15, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. 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: ASSERTIVE

Plan: Pro Plus

Run ID: 01f4151c-588d-4e6c-bd5c-820b751424a2

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.

@radroid
radroid merged commit df027ec into main Aug 15, 2026
2 checks passed
@radroid
radroid deleted the coil/seams-stale-rename-refs branch August 15, 2026 00:10
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