Skip to content

Name browser host and separate development tooling - #699

Merged
wesbillman merged 1 commit into
mainfrom
carl/browser-host-rename
Oct 7, 2026
Merged

wesbillman merged 1 commit into
mainfrom
carl/browser-host-rename

Conversation

@wesbillman

Copy link
Copy Markdown
Collaborator

Summary

Rename dev/ to root-level browser-host/ so the privileged Node host is clearly separate from shared browser-compatible application code (src/) and the native Rust host (src-tauri/). Move the developer-settings Vite plugin and live setup probe, with their tests, to scripts/.

This is a mechanical naming/layout change, not host consolidation. Browser capabilities, native behavior, endpoint URLs, environment variables, credential namespaces and runtime defaults are unchanged. No compatibility directory or new runtime is added.

  • 54 relocated files; 52 are byte-for-byte identical. The host README clarifies ownership; the Vite config test additionally asserts the broker is absent in build mode.
  • Update imports, profiler path, probe command, test discovery, source attribution and ownership docs. Pre-push selection follows the moved host/tooling files without expanding to unrelated scripts. New hook regressions cover related-test selection and full-suite fallback on deletion.
  • Formatter-required import wrapping moves the existing @ts-expect-error comments to the module-specifier line; no test assertions were removed.

Validation

  • At clean head 335c378dd9ef42fff2a423c4dba40a7f7edbf454, mandatory push hooks passed TypeScript, the full Vitest suite, design checks and workspace Clippy. Precommit formatting/lint passed. Vitest file discovery maps exactly across the rename: 564 files before and after.
  • In the precommit working tree on base 03120fb5: full Node integration run 201 passed, 3 existing opt-in skips; complete new-message.spec.mjs 4 passed across Chromium and WebKit, exercising the real Node broker with synthetic identities/upstreams; pnpm probe:live --help passed. Subsequent changes were formatting and TypeScript-directive placement only.
  • Independent source review completed; stale NOTICE attribution and initially overbroad hook selection were fixed and reviewed. Local README/host README/contributor links and anchors resolve; diff checks pass.
  • No browser cases added or removed. No native GUI, real credential access or live relay writes. Real ffmpeg conversion was unavailable locally; that existing optional test returned early. Hosted CI and human confirmation remain pending; this stays draft.

Open PR migration

Existing host files move from dev/<name> to browser-host/<name> at the same depth. Tooling moves are scripts/developer-settings.{ts,test.mjs} and scripts/live-setup-probe{,.test}.mjs. Rebase with rename detection, then inspect added/deleted files and imports rather than recreating dev/.

Directly overlapping open PRs: #295, #427, #324, #122, #695. Their inspected heads remain unchanged since the earlier assessment. #295 and #427 add/remove modules and need explicit new-path reconciliation. #324's probe edits belong in scripts/. #122 already has older stacked integration conflicts. #695's proposed WebKit diagnostic should live under scripts/wkprobe/, not browser-host/. No merge simulation or conflict-free claim is made.

Human check

On this branch, run pnpm probe:live --help and confirm usage appears. Run your usual just web workflow without changing identity/environment, verify the app and Developer Settings load, and check that the layout makes the host boundary clear. No native identity switch or live write is needed for this rename. Please confirm before marking ready.

Signed-off-by: Carl <32a2e2c9d428ee08902cab75d956da2c1d235a22d4766b0dd4138bf6e2e5db1d@buzz.block.builderlab.xyz>
@wesbillman

Copy link
Copy Markdown
Collaborator Author

Rename coordination: @bradseiler (#295), @tlongwell-block (#427), @baxen (#324), @morgmart (#122), @ngoodger-block (#695): this draft mechanically moves dev/ to browser-host/, with developer-settings and live-setup-probe pairs under scripts/. Please reconcile newly added/deleted files and imports during rebase; existing-file edits should retain normal rename detection. The PR description has the exact mapping. For #695, put the new diagnostic at scripts/wkprobe/ rather than recreating dev/. No need to rush your feature PRs for this change.

Carl, an automated contributor, posting via Wes’s GitHub account.

@wesbillman
wesbillman marked this pull request as ready for review October 7, 2026 17:25
@wesbillman
wesbillman requested review from a team and comp615 as code owners October 7, 2026 17:25
@wesbillman
wesbillman merged commit 3b99ad3 into main Oct 7, 2026
22 checks passed
@wesbillman
wesbillman deleted the carl/browser-host-rename branch October 7, 2026 17:41
johnmatthewtennant pushed a commit that referenced this pull request Oct 7, 2026
…ge-delete

* origin/main: (23 commits)
  feat(agents): default conversation context to each thread (#646)
  Name browser host and separate development tooling (#699)
  test(archive): stamp expiry fixtures with one fixed time (#697)
  feat(messages): list Mark unread above Copy message (#638)
  fix(messages): link credential-free http:// URLs like https:// (#691)
  refactor(hooks): replace custom wiring with standard Lefthook hooks (#641)
  fix: deduplicate working agent indicators in composers (#687)
  feat(messages): click images to zoom in the gallery viewer (#688)
  fix(media): serve ranged buzz-media reads from shared signed blocks (#690)
  fix(messages): exclude row chrome from cross-message copies (#645)
  docs: Add kind 30177 publishing guidance for plugin authors (#558)
  feat(agents): allow creating agents with Claude Code
  feat: Enroll Builderlab remote agents (30177) to community (#647)
  Enroll Builderlab agents with the selected community during attestation (#642)
  fix(messages): restore main typecheck for pasted mentions (#684)
  Install the bundled Buzz CLI skill on desktop startup (#667)
  refactor: share status, DM and sidebar intent policy across hosts (#683)
  fix(sessions): show enabled page in navigation (#648)
  fix(composer): replace exact emoji shortcodes on closing colon (#584)
  refactor(mentions): let the Mentions plugin own what the chooser offers (#669)
  ...

Signed-off-by: Opus <f425b86c3b780d43f8d9cf1802429d04dd985bf92eac44fca9468b5ad3887c5e@buzz.block.builderlab.xyz>

# Conflicts:
#	src/features/messages/MessageManagement.tsx
johnmatthewtennant pushed a commit that referenced this pull request Oct 7, 2026
…iles

* origin/main:
  feat(agents): default conversation context to each thread (#646)
  Name browser host and separate development tooling (#699)
  test(archive): stamp expiry fixtures with one fixed time (#697)
  feat(messages): list Mark unread above Copy message (#638)
  fix(messages): link credential-free http:// URLs like https:// (#691)
  refactor(hooks): replace custom wiring with standard Lefthook hooks (#641)
  fix: deduplicate working agent indicators in composers (#687)
  feat(messages): click images to zoom in the gallery viewer (#688)
  fix(media): serve ranged buzz-media reads from shared signed blocks (#690)
  fix(messages): exclude row chrome from cross-message copies (#645)
  docs: Add kind 30177 publishing guidance for plugin authors (#558)
  feat(agents): allow creating agents with Claude Code
johnmatthewtennant pushed a commit that referenced this pull request Oct 7, 2026
…iles

* origin/main:
  feat(agents): default conversation context to each thread (#646)
  Name browser host and separate development tooling (#699)
  test(archive): stamp expiry fixtures with one fixed time (#697)
  feat(messages): list Mark unread above Copy message (#638)
  fix(messages): link credential-free http:// URLs like https:// (#691)
  refactor(hooks): replace custom wiring with standard Lefthook hooks (#641)
  fix: deduplicate working agent indicators in composers (#687)
  feat(messages): click images to zoom in the gallery viewer (#688)
  fix(media): serve ranged buzz-media reads from shared signed blocks (#690)
  fix(messages): exclude row chrome from cross-message copies (#645)
  docs: Add kind 30177 publishing guidance for plugin authors (#558)
  feat(agents): allow creating agents with Claude Code

Signed-off-by: Opus <f425b86c3b780d43f8d9cf1802429d04dd985bf92eac44fca9468b5ad3887c5e@buzz.block.builderlab.xyz>
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