mirror of
https://github.com/block/buzz.git
synced 2026-11-12 12:22:06 +00:00
Blind review found the guard-refusal branch untested, and a refutation pass disagreed. The refutation tested the wrong mutation: DELETING the `allow()` call fails three ordering tests, so it looked covered. Consulting the guard and discarding its answer passed all ten. That is the branch that stops a channel switch while a thread edit is unsaved, so a regression there would have shipped silently. Two tests now pin it — refused, and refused-while-forced — and the ignore-the-answer mutation fails both. Also removes residue the same review found in the tracer teardown: - a stranded `append_switch_perf_log` arm in the e2e mock bridge, for a Tauri command deleted two commits ago - MessageTimeline's marker comment, which justified its placement by a tracer that no longer exists; its readers are now the perf benchmark and the e2e readiness specs - ChannelScreenLoadingFallback's `data-render-pending` wrapper, added solely for the tracer. Measured benchmark switches are warm, so the lazy pane never suspends there and nothing asserts the marker. Reverted to main rather than left as an unowned production DOM node. Signed-off-by: Max Lampert <maxwell@squareup.com>
Buzz
Desktop chat shell with:
- Tauri + React + TypeScript + Vite
- Tailwind CSS
- shadcn/ui-ready shared components
- Biome (lint/format/check)
- Feature-driven frontend structure
Scripts
pnpm dev- run the web frontendpnpm tauri dev- run the desktop apppnpm build- typecheck and build frontendpnpm typecheck- TypeScript checkspnpm lint- Biome lintpnpm format- Biome format (write)pnpm check- Biome check
Structure
src/shared- reusable app-wide code (ui,lib,styles)src/features- feature modules (vertical slices)src/app- top-level app composition