Correct the terminal, notepad, and clipboard specs; fix a right-click ending a mouse override

[nedtwigg]

Spec audit, group 2 (terminal-state / terminal-context / terminal-escapes / mouse-and-clipboard / notepad). Stacked on #708 (the spec-audit PRs merge in order #708 → #715); independent of the alert series.

Commit 1 — docs and comments. Twenty-four inaccuracies corrected against the code (stale helper-status strings, an OSC 8 rule whose implementing module was not named, a dragDropEnabled claim that predates pointer-based pane drag — in the spec and two code comments and a third copy in standalone.md — wrong Source of truth: pointers for the notepad quit gate, the WSL -d gate, bash argv replacement, the 260 ms Cancel flash, IS_MAC's real reach, six rules carrying both a why-clause and a (rationale) marker, and more). Condensed: the helper lifecycle table → one rule + pointer, keystroke-fallback mechanism → the store's comments, per-shell integration mechanics → the scripts, duplicate statements across mouse/notepad/terminal-context.

Commit 2 — code.

• Bug: under a temporary mouse override, a right-click started a pendingDrag and its release read as the click that ends the override, while contextmenu stayed swallowed — the user got nothing. Non-primary buttons no longer start a pending drag; test added, spec §2 now says "primary click".
• Deleted write-only TerminalPaneState.title, derived VolatileSurfaceNotes.terminalId, dead shellName options, the dragBlock module latch (folded into Selection), two unreachable guards (proved by test), and a double control-strip on every output chunk.
• Mirrored-constants tests now pin DEFAULT_HELPER_COMMAND and the helper-command validator against the sidecar.

Budgets: mouse 3800→3750, notepad 4000→3850, terminal-escapes 3850→3800, standalone 10700→10650.

Deferred (reported, not done): the second outside-release finalizer, one shared CWD-refresh helper, moving bracketedPaste off the selection store, headerPrimary returning the winning title, the double beginClosing.

🤖 Generated with Claude Code

[cloudflare-workers-and-pages]

Deploying mouseterm with    Cloudflare Pages

Latest commit:	4fc9aeb
Status:	⚡️  Build in progress...

View logs

[dormouse-bot]

Feedback on work in progress, not a merge verdict — mark the PR ready for review when you want the full pass.

Two spec-accuracy points in docs/specs/notepad.md, both as inline suggestions.

The scope claim. notepad-store.ts is a per-webview module instance, and a VS Code window can hold several Dormouse webviews at once — the sidebar WebviewView plus any editor panels, each with its own attachRouter call in extension.ts. "Per window" therefore undercounts in exactly the host the "VS Code lifecycle" section below it is about; the sentence's own explanation already says "in the webview". The same wording went into NotepadPanel.test.tsx's doc comment, so there's a third suggestion for that line.

The window-close.test.ts citation. That test sets mocks.notepadSurfaceIds.mockReturnValue(["pane-a"]), so the "even after their last note is deleted" half of the rule is stubbed out — what it pins is that the discard path iterates the id set at all. quit-notepad.test.ts pins the archiving half for real: it deletes the last note and asserts the landed batch is still removed on the next attempt.