feat(inbox): preview recent Inbox activity when hovering the sidebar row - #2347

…-nip-fi

* origin/main:
  feat(inbox): preview recent Inbox activity when hovering the sidebar row (#2347)
  Add collapsible Solo and Team sidebar sections (#2296)
  fix(codex): open long Codex transcripts a page of items at a time (#2259)
  Keep Solo model and effort choices per chat (#2342)
  feat(sidebar): smooth channel reordering with bounded motion (#2108)
  Connect legacy agents from DMs and channel mentions (#2423)
  fix: exclude muted channels from sidebar unread scroll indicators (#2335)
  fix(claude-code): open long Claude transcripts a page at a time (#2428)
  feat(solo): share chats as view only (#2375)

# Conflicts:
#	src/app/shell/PrimarySidebar.tsx

…-clone-auth-ab8b97

* origin/main: (38 commits)
  feat(identity): decide a missing backup choice from community membership (block#2378)
  Offer Builderlab decisions in FI builds (block#2227)
  feat(inbox): preview recent Inbox activity when hovering the sidebar row (block#2347)
  Add collapsible Solo and Team sidebar sections (block#2296)
  fix(codex): open long Codex transcripts a page of items at a time (block#2259)
  Keep Solo model and effort choices per chat (block#2342)
  feat(sidebar): smooth channel reordering with bounded motion (block#2108)
  Connect legacy agents from DMs and channel mentions (block#2423)
  fix: exclude muted channels from sidebar unread scroll indicators (block#2335)
  fix(claude-code): open long Claude transcripts a page at a time (block#2428)
  feat(solo): share chats as view only (block#2375)
  fix(inbox): show Solo responses under All, read live replies in the open conversation, describe toolbar badges (block#2275)
  AIDA-2434: Show saved teams in mentions (block#2339)
  AIDA-2455: Restore channel scrollbar dragging beside threads (block#2436)
  feat(agent-protection): show the owner why a protection provider denied input (block#2105)
  fix(search): show slow lookups beside their section headings (block#2442)
  Align Bestie DM layout and unread state; speed up thinking shimmer (block#2418)
  Send Claude, Pi and Goose history as exact records (block#2088)
  fix(agent-activity): align the stopped note and show it after the agent's reply (block#2441)
  fix(agents): stop orphaned Attention commands and prevent duplicate runs (block#2448)
  ...

Signed-off-by: damienrj <[email protected]>

…2395)

## What users see

Someone mentions you in a channel you haven't opened this session, and
their profile hasn't loaded yet. You leave any channel and open the
**Inbox**. The row shows a shortened key (`npub…srl`) and a letter
avatar instead of their name. It stays that way while the Inbox is open,
and nothing asks again. Their name appears only after you open their
channel.

This is a regression from #2269 ("reserve profile room for an open
channel's roster", by Logan Johnson, `loganj`). A bisect over 248
commits, with no skips, found its merge commit `fe6eaa222` as the first
bad one. Its parent `589c2a489` names the author right away. The channel
being left doesn't need to be large: in the repro, #Alpha has one
member.


https://github.com/user-attachments/assets/e1e24cfb-14c1-4a4c-ba65-993443e28b5c

Left: the branch base 8668dbd. Right: this branch at b3d1f09. Both
sides click Inbox on the same frame. On the left, the Inbox's read for
Priya is cancelled 6 ms after it's sent and the row stays `npub…srl`. On
the right, the read returns 200 in 16 ms and the row shows "Priya Shah"
from the first Inbox frame.

![Three seconds after clicking Inbox: the base build (red, left) still
shows npub…srl with a letter avatar; the fix (green, right) shows Priya
Shah with her
avatar](https://github.com/user-attachments/assets/f378e271-14b3-4921-bd45-516340b00c47)

## Cause

1. An open channel reserves profile-cache room for its roster
(`Conversation.tsx`, a layout effect). Leaving releases that
reservation, and the release schedules the shrink one task later
(`shrinking ??= setTimeout(resize)`).
2. In the same commit, the Inbox's mount effect asks for the names it
shows, as a background read.
3. When `resize()` runs, it treats the shrink (1025 → 1024 in the repro)
as a reason to cancel **every** pending background read that isn't a
reconnect refresh. That includes the read the Inbox just started.
4. The cancel is silent: `ensure()` resolves normally, and only
`evictions()` moves. The mention menu is the only consumer of
`evictions()`. The Inbox's effect has no dependency that changes, so it
never asks again.

The cancel in #2269 was right for the departing view's reads, for
example the mention menu's 500-author roster pages, which would
otherwise land late and push the next view's names out of the
1,024-entry cache. It was wrong for reads the **next** view started
while the shrink was waiting.

## The fix

The fix is in one production file,
`src/features/relay/profile-directory.ts` (+28/−8):

- **Stamp at creation.** A counter goes up each time a reservation is
released. A read records that count when it is created, and the stamp
never changes (`readonly releases`).
- **One predicate.** `shrinkCancels(batch)` is true for a background
read that isn't a reconnect refresh and was created before the latest
release. `resize()` uses it to choose what to cancel. `ensure()` uses it
to decide whether a request may join a pending read.
- **No joining a read that's about to be cancelled.** While a shrink is
waiting, a request doesn't join a pending read that the shrink would
cancel. That name gets its own read, and the shrink still cancels the
rest of the departing page. Joins on foreground and refresh reads, and
any join while no shrink is waiting, work as before.
- A remount or a roster-count change is unaffected. `reserve()` clears
the waiting shrink before the children's effects ask for names.
- The `evictions()` doc comment, the `resize()` comment and one line of
`docs/channels.md` now describe the narrower cancel. `docs/buzz-help.md`
is unchanged, because this restores the earlier behaviour.

**Why the stamp isn't updated on a join.** The first commit (8429c27)
updated a pending read's stamp whenever a later request joined it.
Review showed that this kept a whole departing roster page alive for the
sake of one shared author. The reader runs one background job at a time,
so those pages also landed before the Inbox's own read. In a
9,000-member scenario, 3 joined pages loaded 1,500 roster profiles into
the 1,024-entry cache. That pushed out 5 names the Inbox had already
shown, and those rows went back to `npub…`, which is the case #2269 was
written to prevent. With the creation-time stamp and the join skip, the
same scenario cancels 19 pages and keeps all the Inbox's names and all
200 earlier names, with the same 20 reads.

**Rejected alternatives**
- **Make the Inbox re-read when `evictions()` moves.** That fixes one
page, and every switch from a channel to the Inbox would send the read,
cancel it and send it again.
- **Bump `generation()` when a shrink cancels reads.** That changes what
`generation()` means: it marks a full cache purge, which other code keys
on.
- **Drop the cancel, or skip it for small shrinks.** Either one brings
back the eviction case #2269 fixed.

**Accepted costs.** These were noted in review and don't block:
- A release that leaves a larger reservation open can split a join into
a second, redundant read until the next task.
- If the next view asks again for every author on a departing page, that
page isn't cancelled. It loads only names the next view asked for, so
the cost is one redundant read and no names are pushed out.

## Surfaces covered

The directory change covers every page that replaces a channel and asks
for names in its mount effect without watching `evictions()`:
- the Inbox (tested and recorded);
- Drafts, opened straight from a channel (`DraftsView.tsx`);
- Reminders (`RemindersPage.tsx`);
- Workflows owner names (`WorkflowLanding.tsx`);
- Agents → Watching (`watching/Tab.tsx`);
- the Inbox hover preview in open PR #2347, which copies the Inbox's
effect.

These were found by reading the code. Only the Inbox has a browser case
or a recording. The others use the same directory rule, which the unit
test covers.

**Out of scope.** Views that stay mounted when you leave a channel are
still exposed if their read happens to be in flight at that moment:
- sidebar DM labels (`useChannelLabels.ts`);
- Inbox names for edited-message authors (`unread.ts`);
- channel previews loaded in the background (`store.ts`), which are
re-read when the channel opens.

The window is one relay round trip at the moment you leave. None of
these was reproduced, so a follow-up is warranted only if one is.

## Tests

### Unit test (Vitest), `src/features/relay/profile-details.test.ts`

The new test, "keeps background reads asked for after a release when the
shrink runs", sits next to #2269's cancellation test, which is
unchanged.
- **Setup:** reserve room. Start two departing background reads: one for
a `stale` author, and one page holding `joined` and `other`. Release.
Before the timer fires, the next view asks for `fresh` and `joined`.
- **Assertions:**
- how each read is split: `[stale]`, `[joined, other]`, `[fresh,
joined]`;
- `evictions()` moves by exactly 2, because both departing reads are
cancelled;
- every read is answered with all four profiles, and only `fresh` and
`joined` end up in the cache.
- **Mutants:** in the round 2 review, four mutants were run against this
test at b01fe78, and each failed:
1. the first commit's stamp update on join: evictions +1, and `other` is
named;
  2. no join skip: `joined` stays unnamed;
3. a shrink that never cancels: evictions +0, and `stale` and `other`
are named;
4. main's behaviour: `fresh` and `joined` stay unnamed, with evictions
+3.

Mutants 1 and 2 issued 3 reads either way, so b3d1f09 added the
read-split assertion. With the join skip removed, the third read holds
only `fresh`, and the test now fails on the split too.

### Browser case, `tests/browser/inbox.spec.mjs`

One case was added, "Inbox names after leaving a channel › names an
author seen only in the Inbox", and none were removed.
- **The flow:** open on #Alpha, click Inbox, and expect the row for an
author seen only in #Launch to contain "Priya Shah". Also expect exactly
one kind-0 query naming her.
- **Why it needs a browser:** the bug depends on ordering across two
surfaces: the channel's layout-effect release, then the Inbox's
mount-effect read, then the deferred shrink. The unit test proves the
directory rule, not that the real Inbox's read starts after the release
and before the shrink.
- **Fail, then pass:** it fails without the fix in both Chromium and
WebKit, where the row shows `npub…`. It passed 5/5 per engine at
8429c27 and passed again in both engines after the join change.
Production code hasn't changed since then; b3d1f09 changes only the
unit test and a comment.
- **Why CI missed this:** no journey checked the name of an author seen
only in the Inbox.

### Other checks during development

These were not re-run for this push:
- `profile-details.test.ts`: 25/25 pass.
- The 16 unit test files that import the profile directory: 630/630
pass.
- `profile-roster-scale.spec.mjs`, #2269's 9,000-member check: passes in
both measurement projects.
- `typecheck`, `lint` and `git diff --check`: clean.
- Pre-commit hooks: passed on each commit. Pre-push hooks were skipped
for this push (`--no-verify`).

## Review rounds

- **Round 1 (8429c27):** one blocker. Updating the stamp on a join
kept departing roster pages alive (the 9,000-member scenario above).
Fixed in b01fe78, along with the optional items: a clearer field
comment, a `stale` assertion that can actually fail (the test now
answers every read with all profiles), and a minimal `docs/channels.md`
diff.
- **Round 2 (b01fe78):** the blocker is fixed, with no new defects,
and the change was judged at the 9/10 bar. Its optional items, asserting
how each read splits and a comment wording, are in b3d1f09.

## Status

This is a draft until a human tries it in the app (AGENTS.md "Before
opening a PR", step 3). To test: open the app on a channel, then click
**Inbox**. An author who mentioned you in a channel you haven't opened
this session should show their name and avatar right away, without
opening their channel.

Deferred:
- the human test above;
- hosted CI on this head;
- a WebKit recording (the browser case covers WebKit);
- recordings of the other covered pages.

Logan Johnson (`loganj`): this narrows #2269's cancel-on-shrink and
changes the `evictions()` contract wording. Your review is welcome.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

---------

Signed-off-by: Matt Toohey <[email protected]>
Co-authored-by: Claude Opus 5.5 <[email protected]>

Resolve conflicts with block#2347, which moved hasUnread into InboxRow.tsx, and
give its new inbox-preview browser journey the shared INBOX_BUTTON locator.

Co-authored-by: DeviousMachine <8aa903583ddc111fb259d28d298a0b05685db29a3082a57a99c98522ae044622@buzz.block.builderlab.xyz>
Co-Authored-By: Claude Opus 5.5 <[email protected]>
Co-authored-by: Jonathan Hollingsworth <[email protected]>
Signed-off-by: Jonathan Hollingsworth <[email protected]>

* origin/main: (38 commits)
  fix(browser): load scheme-less address input over https, falling back to http (#2467)
  fix(identity): stop one-click key unlinks and show the linked backed-up key (#2486)
  fix(test): observe coordination transition events (#2297)
  fix(profiles): keep the next view's profile reads through a shrink (#2395)
  fix(channels): show Add and Send message errors above the Manage members list (#2421)
  fix(threads): keep reply hooks off agent activity bubbles (#2422)
  fix(channels): offer only Copy link in a not-joined preview's sidebar actions (#2425)
  test: scope row menu placement checks to active section rows (#2493)
  feat(relay): detach channel windows to a point in history (#2042)
  feat(identity): decide a missing backup choice from community membership (#2378)
  Offer Builderlab decisions in FI builds (#2227)
  feat(inbox): preview recent Inbox activity when hovering the sidebar row (#2347)
  Add collapsible Solo and Team sidebar sections (#2296)
  fix(codex): open long Codex transcripts a page of items at a time (#2259)
  Keep Solo model and effort choices per chat (#2342)
  feat(sidebar): smooth channel reordering with bounded motion (#2108)
  Connect legacy agents from DMs and channel mentions (#2423)
  fix: exclude muted channels from sidebar unread scroll indicators (#2335)
  fix(claude-code): open long Claude transcripts a page at a time (#2428)
  feat(solo): share chats as view only (#2375)
  ...

…r-load-perf

* origin/main: (31 commits)
  test: fix invitation and coordination checks on main (#2517)
  perf(search): name searched people from the profile search response (#2353)
  Use the merge queue for pull requests that require it (#2452)
  fix(browser): load scheme-less address input over https, falling back to http (#2467)
  fix(identity): stop one-click key unlinks and show the linked backed-up key (#2486)
  fix(test): observe coordination transition events (#2297)
  fix(profiles): keep the next view's profile reads through a shrink (#2395)
  fix(channels): show Add and Send message errors above the Manage members list (#2421)
  fix(threads): keep reply hooks off agent activity bubbles (#2422)
  fix(channels): offer only Copy link in a not-joined preview's sidebar actions (#2425)
  test: scope row menu placement checks to active section rows (#2493)
  feat(relay): detach channel windows to a point in history (#2042)
  feat(identity): decide a missing backup choice from community membership (#2378)
  Offer Builderlab decisions in FI builds (#2227)
  feat(inbox): preview recent Inbox activity when hovering the sidebar row (#2347)
  Add collapsible Solo and Team sidebar sections (#2296)
  fix(codex): open long Codex transcripts a page of items at a time (#2259)
  Keep Solo model and effort choices per chat (#2342)
  feat(sidebar): smooth channel reordering with bounded motion (#2108)
  Connect legacy agents from DMs and channel mentions (#2423)
  ...

* origin/main: (31 commits)
  test: fix invitation and coordination checks on main (#2517)
  perf(search): name searched people from the profile search response (#2353)
  Use the merge queue for pull requests that require it (#2452)
  fix(browser): load scheme-less address input over https, falling back to http (#2467)
  fix(identity): stop one-click key unlinks and show the linked backed-up key (#2486)
  fix(test): observe coordination transition events (#2297)
  fix(profiles): keep the next view's profile reads through a shrink (#2395)
  fix(channels): show Add and Send message errors above the Manage members list (#2421)
  fix(threads): keep reply hooks off agent activity bubbles (#2422)
  fix(channels): offer only Copy link in a not-joined preview's sidebar actions (#2425)
  test: scope row menu placement checks to active section rows (#2493)
  feat(relay): detach channel windows to a point in history (#2042)
  feat(identity): decide a missing backup choice from community membership (#2378)
  Offer Builderlab decisions in FI builds (#2227)
  feat(inbox): preview recent Inbox activity when hovering the sidebar row (#2347)
  Add collapsible Solo and Team sidebar sections (#2296)
  fix(codex): open long Codex transcripts a page of items at a time (#2259)
  Keep Solo model and effort choices per chat (#2342)
  feat(sidebar): smooth channel reordering with bounded motion (#2108)
  Connect legacy agents from DMs and channel mentions (#2423)
  ...

Signed-off-by: Michael Neale <[email protected]>

## Why

People said Inbox's filter bar was crowded and Archive didn't make
sense. They liked the tabs and the Unread toggle, and they wanted the
page called **Activity**, like Slack's. This PR keeps those parts and
removes the rest, so the toolbar fits on one row for as long as
possible.

## What

- **Rename:** Inbox is now **Activity**, in the page title, the sidebar
row, the shortcut titles ("Open … in Activity") and the help docs. The
bell icon stays. Plugin id `buzz.inbox`, page id `inbox` and storage
keys are unchanged.
- **Toolbar:** one left-aligned row.
- Type tabs come first: All, Threads, Mentions, DMs. The first tab lines
up with the title-bar bell.
- A text-only **From** menu (Anyone / People / Agents) follows the tabs.
It gets a subtle fill while it's narrowing the list.
- The **Unread** switch sits right after From, so it stays in view on a
full-width page.
- When the window is too narrow, Unread wraps to the next line first,
then From.
- **Archive removed:** every conversation stays in the list. The row ✓
is now **Mark as read**. It only shows on unread rows.
- With Unread on, the row leaves the list and an **Undo** toast appears.
- When nothing unread is left, the list says **All caught up** and
offers **Show all**.
- **Drafts removed:** a draft stays in the conversation where it was
written.
- **Focus:** after Mark as read, focus moves to the next row only if
focus is still in the list. If someone moves to the composer while the
read is still saving, their cursor stays where they put it.
- **Sidebar hover card (#2347):** now titled Activity, with an **Open
Activity** button. It no longer hides conversations that were archived
before, so it lists the same conversations as the page.
- **Filter routes (#2394):** the route contract drops `show` and
`drafts`. Older links that still carry them open the page with its saved
filters instead of failing: `show` is ignored, and `{ drafts: true }`
alone opens the saved filters.

## Screenshots

<!-- Default toolbar · From menu · From: People · Marked as read + Undo
· All caught up · Narrow (560px) · Hover card -->

## Not in this PR

- The Inbox badge and bold label: #2337.
- Which items count as unread: #2401.
- The unread logic from #2275.
- "Retry inbox" and "Inbox history unavailable" in
`src/features/relay/session.ts` and `inbox-feed.ts`. Those files are
shared relay code, so the strings are unchanged.
- `archive.ts` no longer has a caller in the page or the card. Removing
it is a follow-up, so this PR doesn't touch the files #2275 edited.

## Testing

- **Unit:** `vitest run src/bundled/inbox src/app src/features/pages
src/plugins` passes, 1347 tests after the latest rebase.
- New: Mark as read via the ✓ and via the context menu, each when the
save succeeds and when it fails. In all four, focus stays in the
composer the user moved to. All four fail without the fix.
- New: route cases for sender-only routes and links that still carry
`show` or `drafts`.
- **Browser, Chromium and WebKit:** `tests/browser/inbox*` and
`sidenav-polish` pass, 74 tests after the latest rebase.
- New `inbox-mark-read.spec.mjs` covers the one-row toolbar layout, bell
alignment, Mark as read with Undo, and All caught up.
- **Static checks:** `tsc`, `lint` and `design:check` are clean.
- **Agent review:** Crack Catcher's blocking finding was a focus jump
during a pending read. It's fixed, and Crack Catcher confirmed it.
- **Human testing:** tulsi tried it in the desktop dev app and confirmed
it all works.

buzz-review-completed

🤖 Generated with [Claude Code](https://claude.com/claude-code)

<img width="1280" height="800" alt="1 Default toolbar"
src="https://github.com/user-attachments/assets/4e3d23ac-5529-48b6-ab91-594ce3933e52"
/>
<img width="1280" height="800" alt="2 From menu open"
src="https://github.com/user-attachments/assets/7e2c86df-998a-467d-9ae8-10ea8c73bcbc"
/>
<img width="1280" height="800" alt="3 From People (dark)"
src="https://github.com/user-attachments/assets/3dd58da4-204b-46de-b4e5-cc531f7bd580"
/>
<img width="1280" height="800" alt="4 Marked as read with Undo"
src="https://github.com/user-attachments/assets/d54214a4-2990-4080-bf1d-15d6cacfd4fc"
/>
<img width="1280" height="800" alt="5 All caught up"
src="https://github.com/user-attachments/assets/215850bb-8ed3-49c4-a4a7-a7cf6e2530ff"
/>
<img width="560" height="800" alt="6 Narrow window"
src="https://github.com/user-attachments/assets/e4a1fa0c-a19e-45de-b082-43c5295f7cba"
/>
<img width="1280" height="800" alt="7 Hover card"
src="https://github.com/user-attachments/assets/63947445-59ae-4c49-97b4-d04ae12c6875"
/>

---------

Signed-off-by: tulsi <[email protected]>
Co-authored-by: BasilBot <399581b6165ab01343e3a1324774b1024d76942aea563468f93c975e5ee5193c@buzz.block.builderlab.xyz>
Co-authored-by: Claude Opus 5.5 <[email protected]>

Activity moves out of the dock and back into a row at the top of the
Messages sidebar, where it was before #2626. Drafts & sent sits below it
when that plugin is on.

- **Hover preview:** resting the pointer on the row opens the same
Activity card as before (#2347). The shell already supports previews on
sidebar rows, so the card works unchanged.
- **Reaction emoji:** the row's bell swaps for the latest reaction
emoji, as the dock tile did. `ActivityNavigationIcon` is now sized for
the row's 16px icon slot, and the dock's Activity-only icon code is
gone.
- **Selection:** opening Activity keeps Messages selected in the dock
and keeps its sidebar, like other pages listed there (#2779).

The change is host policy in `pagePlacement()`
(`src/app/shell/presentation.ts`): `buzz.inbox/inbox` resolves to
`"sidebar"` whatever placement it registers, and it's dropped from the
dock order. No FOUNDATION files are touched.

Docs (`docs/inbox.md`, `docs/buzz-help.md`, `docs/shell-design.md`, the
Activity README) and Inbox code comments now say "Messages sidebar"
where they said "app dock".

## Tradeoffs

- **Fewer places to see it.** The row lives in the Messages sidebar.
With another dock app open, the sidebar collapsed (below 650px), or a
plugin narrowing the sidebar, Activity takes an extra step (Messages,
Show navigation, or Search/shortcut), and the reaction-emoji swap isn't
visible until the sidebar is.
- Activity is no longer a dock tab. This reverses that part of #2626, so
the dock's owner should sign off.

## Screenshots

Synthetic browser-fixture data only.

Hovering the Activity row, light:

![Activity hover card beside its Messages sidebar row, light
theme](https://github.com/block/buzz-app/raw/26a45ad84c2494d21fff0399adff249f76095b1d/activity-sidebar-preview-light.png)

Dark:

![Activity hover card beside its Messages sidebar row, dark
theme](https://github.com/block/buzz-app/raw/26a45ad84c2494d21fff0399adff249f76095b1d/activity-sidebar-preview-dark.png)

The row's bell swapped for a new reaction (✅):

![Activity row showing a reaction emoji in place of its
bell](https://github.com/block/buzz-app/raw/26a45ad84c2494d21fff0399adff249f76095b1d/activity-sidebar-reaction.png)

## Tests

- New `PageNavigation.test.tsx`: the sidebar Activity row swaps its bell
for the emoji and keeps its accessible name.
- `presentation.test.ts`: bundled Activity resolves to the sidebar
whatever placement it registers.
- `navigation-owner.test.ts`: Activity now belongs to Messages.
- `entity-navigation.test.tsx` also checks that Activity has no dock
tab.
- AppShell and entity-navigation tests that used Activity as an example
dock app now use Bestie, Reminders or Workflows. Entity-navigation tests
find the Activity row only after the sidebar's one startup remount, so
they never click a detached row.
- Browser specs find Activity with a shared `activityRow()` helper in
`tests/browser/navigation.mjs`. No browser cases were added or removed.
`inbox-preview.spec.mjs` now covers hovering the sidebar row; on a
selected row it checks for the row's title hint where it used to check
for the dock's tooltip. `inbox-tabs.spec.mjs` expects the 16px row emoji
instead of the 18px dock one.

Validation on this branch:
- `pnpm check` (biome, typecheck, design checks) passed.
- Full `vitest run`: 20,129 passed, 1 failed. The failure,
`MemoryPanel.test.tsx > copies a memory from its context menu`, passes
when run on its own on both this branch and main.
- Chromium browser specs (`--project chromium`) covering Activity or the
dock (41 files): after the fixes, every case touched passes. Three cases
fail identically on the base commit without this change, so they're out
of scope here: `layout.spec.mjs` "Alt+ArrowUp from a message link
preserves native scroll ownership", and both `typing.spec.mjs` "a
growing … composer lifts history" cases (React error #185).
- **Narrow windows:** below 650px the Messages sidebar is collapsed, so
getting to Activity takes Show navigation or Search. `inbox.spec.mjs`'s
390px case opens Activity through Show navigation and checks that the
drawer closes.
- After review fixes (`916e24d3e`): the vitest files touched (30 files,
689 tests) and `tsc` pass; the Chromium specs rerun with 45 passed and 1
failed, the Alt+ArrowUp case that also fails on main. `layout.spec.mjs`
now uses Workflows as its non-Messages dock destination.
- tulsi tried the running desktop app: the row, hover preview and emoji
swap all work.

Not run locally: WebKit browser projects and native builds. CI covers
those.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

---------

Signed-off-by: tulsi <[email protected]>
Co-authored-by: BasilBot <399581b6165ab01343e3a1324774b1024d76942aea563468f93c975e5ee5193c@buzz.block.builderlab.xyz>
Co-authored-by: Claude Opus 5.5 <[email protected]>