Condense Relay and Pocket reference contracts - #903
Merged
nedtwigg merged 5 commits intoOct 2, 2026
Merged
Conversation
Deploying mouseterm with
|
| Latest commit: |
120ca96
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://a574ce8a.mouseterm.pages.dev |
| Branch Preview URL: | https://spec-cleanup-relay-pocket.mouseterm.pages.dev |
dormouse-bot
approved these changes
Oct 2, 2026
nedtwigg
requested a deployment
to
hosted-preview
October 2, 2026 06:19 — with
GitHub Actions
Waiting
nedtwigg
added this pull request to stack #896
October 2, 2026 13:23
The PR collapsed relay.md's `checkRegistration` step list into "its code owns check ordering", losing the one ordering fact a careful editor could break: the challenge is redeemed before the origin check, so a wrong-origin registration still burns its challenge. It also added a mouse-and-clipboard rationale paragraph restating the Windows clipboard ACL Known gap that security-local.md already owns. Restore the step list from base, checked against `checkRegistration` in remote-lib-common/src/remote/relay-common.ts (adding the optional `label` the body also carries), state the redeem-before-origin rule, and drop the duplicate rationale paragraph. Ratchet relay.md's budget by 50. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
# Conflicts: # scripts/spec-word-budgets.json
dormouse-bot
approved these changes
Oct 2, 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Correct Relay, Pocket, self-hosting, and pairing-story references against the shipped code. Pocket scans pairing codes in its own storage context; rows have a primary action rather than exactly one action. Canonical Phase, RuntimeInfo, enrollment/status types, and registration validation own their inventories.
Qualify Unix file modes and Windows inherited permissions, point host persistence to its owning specs, and use the local-security pointer for clipboard storage limits. This layer changes only prose, source comments, and word budgets.
Validation: spec lint and 23 mutations; all 15 public-doc checks. Complete section/rationale audit with targeted source-symbol and message-direction checks. Stacked on #902.
Trimmed in review. Restored the
checkRegistrationstep list in relay.md. Its order matters, because the challenge is redeemed before the origin check, so a wrong-origin registration still burns it. Dropped a rationale paragraph that duplicated security-local.md's Known gap.🤖 Generated with Claude Code