Revert Whiteboard rename stack and subsequent changes - #500
Merged
Merged
Conversation
Restore the tracked tree exactly to 5d5181f. Reverts #478, #479, #480, #481, #482, #475, #494, and #495. Branding and selected later changes will be reapplied separately. Agent-Session: 01a0cf60-069f-7a21-a156-c605ad79dfec Agent-Session: 01a0cf8e-5b2b-7253-9d6c-2d77ac6485b8 Agent-Session: 01a0cf90-a513-7740-a23f-6f5ab7b22320 Agent-Session: b320a5ae-50a4-4472-a324-515d964c5f5a
thesiti92
approved these changes
Sep 23, 2026
thesiti92
pushed a commit
that referenced
this pull request
Sep 23, 2026
Restore the Whiteboard agent-facing surface after #500 without renaming internal Review packages, paths or stored data. This selectively combines #479, #480 and Alex’s #475 setup redesign. - Expose the `whiteboard` CLI/MCP registration and `session_*` tools with `sessionId` arguments through the public adapter. Authored content remains opaque; internal HTTP/store contracts and the legacy CLI alias remain. - Restore connection prompts, pointer skills/plugins and MCP-served authoring guidance, replacing automatic skill/config installation. - Follow #475’s launcher migration: retarget enabled repositories’ managed hooks to `whiteboard`, even when the old executable still works, before removing the app-owned old launcher. Recognize both executable names for hook ownership. - Keep the npm package named `@dev.fast/review`. Remove the unnecessary root convenience shortcut; `pnpm dev` remains the development command. Validation: the packed npm artifact installed outside the workspace and passed trace-hook installation/removal plus headless authoring through `whiteboard` and `session_*`, without Desktop. All 250 trace tests, 38 installer tests and 8 historical-copy canvas tests pass, along with lint and trace typechecking. Earlier validation covered desktop build/launch, workspace typechecks, canvas/native suites and public-adapter integration. The separate visible-branding layer is #507. Both Whiteboards are updated to the current stack.
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
2 participants
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.
Restore the repository exactly to
5d5181f252011727621013b44a0f4cffeafaa5cf, before the Whiteboard rename stack, so the user-facing rename can be rebuilt separately with a narrower scope.This reverts #478, #479, #480, #481, #482 and the subsequently merged #475, #494, #495. No branding, migrations, or later fixes are selectively retained; reapplication belongs in follow-up PRs. No new release was cut from these changes.
Validation: the resulting Git tree is identical to the target commit (
a6e9beb4849072e538563e89449422f690c57fd8), and the working tree is clean. The target tree previously passed Review Desktop CI. Tests have not been rerun locally; this PR will run CI again.