Skip to content

Correct stale comment counts, a misplaced doc comment, and the Burrow raise's restart scope - #1308

Merged
nedtwigg merged 1 commit into
mainfrom
docs/stale-pointers-and-counts
Oct 21, 2026
Merged

nedtwigg merged 1 commit into
mainfrom
docs/stale-pointers-and-counts

Conversation

@dormouse-bot

Copy link
Copy Markdown
Collaborator

This PR fixes comments and one spec sentence that no longer match the code. None of it changes behavior.

  • docs/specs/remote-security-model.md, Passkeys → "Presence, or verification": the spec said "A raise outlives a restart even when it could not be saved." That holds only for a Burrow restart within the same process. If the save failed, a relaunch starts from the stored, lower demand until the Relay's next policy frame, as the #withRaisedPolicy doc comment in lib/src/host/remote/service.ts and S181 both state. The sentence now says "a Burrow restart in the same process".
  • lib/src/components/wall/workspace-lifecycle.ts: the JSDoc describing the Labs delayed Workspace close sat above helpersHaveWork, not pendWorkspace, so editors showed it on the wrong function. It now sits on pendWorkspace, and helpersHaveWork has a one-line comment of its own.
  • website/src/components/DorCommandReference.tsx: the comment said "eleven subcommands", but dor --help now lists 18. The count is dropped.
  • lib/.storybook/main.ts: the comment said the pairing walkthrough opens on "the four trust layers". docs/stories/pairing.mdx is headed "Three parties, five layers", so the comment now says five.
  • lib/src/host/agent-browser-host.test.ts: a comment line was duplicated verbatim as the first line of the comment below it. The duplicate is removed.

pnpm lint:specs passes locally. No regression test, since only comments and spec text change.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying mouseterm with  Cloudflare Pages  Cloudflare Pages

Latest commit: b7d872a
Status: ✅  Deploy successful!
Preview URL: https://8b9565a0.mouseterm.pages.dev
Branch Preview URL: https://docs-stale-pointers-and-coun.mouseterm.pages.dev

View logs

@nedtwigg
nedtwigg merged commit 65ad6d5 into main Oct 21, 2026
17 of 18 checks passed
@nedtwigg
nedtwigg deleted the docs/stale-pointers-and-counts branch October 21, 2026 18:16

This branch was successfully deployed

No deployments
hosted-preview — b7d872a9 Deployed Oct 21, 2026 by dormouse-bot via deploy #1790
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.

2 participants