Skip to content

Desktop content polish (tracking) - #2216

Draft
delkc wants to merge 2 commits into
mainfrom
claydelk/system-content-polish
Draft

Desktop content polish (tracking)#2216
delkc wants to merge 2 commits into
mainfrom
claydelk/system-content-polish

Conversation

@delkc

@delkc delkc commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Tracking PR for the desktop content polish work. The original single diff reached 71 files spanning five unrelated changes, so it's being split into reviewable slices.

This branch is the holding pen for whatever hasn't been split yet. Each child PR is cut off main; as one merges, this branch gets rebased and those commits drop out of the diff automatically. So the diff below is always exactly "what's left." When it's empty, this closes.

Slices

  • 1. Chat timeline system messagesfix(desktop): channel topic and membership metadata cleanup #3642
    SystemMessageRow.tsx. Cleared topic/purpose rendering as empty quotes (a real bug), and "was added by" → "added by". Extracted the copy to lib/systemEventCopy.ts for unit coverage.

  • 2. "relay" / "space" → "community"
    identity-archive/hooks.ts, community-members/ui/ConfirmRemoveDialog.tsx, profile/ui/ArchiveConfirmDialog.tsx, agents/ui/PersonaDeleteDialog.tsx.
    A product-vocabulary decision, not a copy tweak — wants a reviewer who can approve the word itself. Incomplete in this branch: 7 files still show "relay" to users, including onboarding/ui/ProfileStep.tsx ("reconnect to the relay") and communities/ui/CommunityEditForm.tsx ("Can't reach this relay"). Finish or scope explicitly.

  • 3. Destructive confirmation pattern — ~15 files
    Title becomes a question ("Delete template?"), description states the consequence in future tense ("X will be removed"), confirm button names the object ("Delete team", not "Delete"), and cannotcan't. One pattern applied consistently.

  • 4. Toast voice sweep — ~40 files
    "Failed to X" → "Couldn't X", trailing periods dropped from toast fragments. Purely mechanical.
    Incomplete in this branch: 96 files still contain "Failed to" (180 instances), and the projects/ feature was left behind entirely — it kept "Failed to merge pull request", "Failed to post comment", "Failed to approve" while everything else moved to "Couldn't". Either finish it or state the boundary in the PR, because as-is it reads as an oversight.

  • 5. Toasts that needed new data — 8 files
    Not copy — this group changes code. MembersSidebarMemberCard.tsx + useMembersSidebarModeration.ts thread a new durationLabel through two layers so the toast can say "timed out for 10 minutes"; channel-templates/useApplyTemplate.ts adds a new formatFailedAgentNames helper. Neither has a test. Most likely of the six to draw a real review comment, and currently buried under 60 files of string edits.

  • 6. Date and time formattingfeat(desktop): one relative date ladder across chat and the Inbox #3769
    Three divergent implementations against one standard:

    • messages/lib/dateFormatters.ts formatDayHeading (chat day divider) — has Today/Yesterday, but every older date renders as "Monday, March 31st" with an ordinal, instead of the standard's weekday → "June 20" → "Aug 2022" progression.
    • home/ui/InboxDetailPane.tsx:239 and HomeView.tsx:842 — the visible header timestamp is formatInboxFullTimestamp, which is always absolute and always carries the year ("Jul 8, 2026, 2:34 PM"), even for a message from this morning. Never relative, at any distance. (An earlier version of this item said clock-time-only; that was wrong — formatTime feeds only the continuation-row hover gutter.) The pane has no day divider, so whatever it shows has to carry its own date.
    • home/lib/inbox.ts — a third implementation; shows "Yesterday" but never "Today".

    Standard is shared-foundations-numbers → Relative Dates and Timestamps. One open question: the standard's oldest band drops the day ("Aug 2022"), which is ambiguous for a day divider that must uniquely identify a day.

Sequencing

  • 2 and 3 overlap on PersonaDeleteDialog, ConfirmRemoveDialog, and ArchiveConfirmDialog — send them sequentially or as a small stack.
  • 1, 5, and 6 are disjoint and can go in parallel.
  • 4 last — it's the slice that rots against main.

Also noted, not scheduled

  • Membership rows are the only passive-voice status lines. 10 of the 12 system-event captions are active ("created this channel", "left the channel"). The 2 exceptions are members_added and member_joined where actor ≠ target, which read "added by Alice Chen" — because those rows are titled with the person who arrived, not the person who added them. That's also why they get the dual avatar leading with the new member's face.

    fix(desktop): channel topic and membership metadata cleanup #3642 dropped "was" rather than making it agree with the subject ("You was added by" is broken). Agreement would key on user-controlled display names, fail visibly when it guesses wrong, and need re-deriving for every future passive line. The durable fix is to remove the passive voice instead: re-title those rows with the actor and go active — "Alice added Marcia to the channel, along with Peter, Jordan, and 2 others" — which matches member_removed exactly and makes the title slot always mean "the actor". Cost: it changes whose avatar and name lead the row, so it's a design call, and it touches isMembershipArrival plus the mentions.spec.ts membership assertions.

  • "Added by You" in the persona catalog. agents/ui/PersonaAddedBy.tsx has the same mid-sentence capitalization mismatch fix(desktop): channel topic and membership metadata cleanup #3642 fixed for channel system rows (toInlineName in messages/lib/systemEventCopy.ts). One-word change plus two assertions in agents.spec.ts, but it's the agents feature rather than the chat timeline.

  • "the channel" vs "this channel" — joined/left/removed say one, created/archived/unarchived say the other.

  • This branch's 7fb85c878 carries an unrelated .gitignore addition (.pnpm-store/) that should ride with something else or go on its own.

delkc and others added 2 commits July 27, 2026 11:31
Sweep the desktop toast surface for a consistent voice:
- error toasts use "Couldn't …" instead of "Failed to …" (git/PR
  service toasts keep "Failed" as more accurate for that surface)
- drop terminal periods on single-clause toasts; keep them only on
  multi-sentence messages
- name the member in relay promote/demote/remove confirmations
- archive toasts say "in this community" instead of "on this relay"
- clearer copy for repos-directory, DM agent-mention, storage-full,
  and stop-turn toasts
- "added by" (not "was added by") in membership rows, fixing the
  "You was added by" grammar bug
- template apply names the agents that failed; member timeout echoes
  the chosen duration

error.message/err.message branches are preserved throughout; only
fallback literals changed.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Standardize confirmation dialogs: community terminology, question-form
titles, sentence-case buttons, specific "Delete {noun}" actions, contracted
"can't be undone", and consequence-led bodies. Also add cleared-topic and
cleared-purpose system messages.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: Clay Delk <clay.delk@gmail.com>
@delkc
delkc force-pushed the claydelk/system-content-polish branch from 403b17a to d56ee64 Compare July 27, 2026 17:00
@delkc delkc changed the title Polish desktop system message, toast, and confirmation dialog copy Desktop content polish (tracking) Jul 30, 2026
tellaho pushed a commit that referenced this pull request Jul 30, 2026
## Why

The Profile settings action still says “Sign Out,” while its
confirmation action says “Delete My Data.” Both buttons trigger the same
destructive local-data wipe and should name it consistently.

## What

- Label both destructive actions “Delete my data”
- Assert the matching section and confirmation labels in the existing
Playwright coverage

## Risk Assessment

Low — copy and test assertions only; sign-out behavior is unchanged.

## References

- Follow-up to #2208
- #2216 also touches this copy and should preserve “Delete my data” when
rebased
- `just desktop-check`
- `just desktop-test` (3,275 tests)
- Desktop E2E build and sign-out Playwright spec (2 tests)

Generated with Codex

Signed-off-by: Bradley Axen <baxen@squareup.com>
delkc added a commit that referenced this pull request Jul 30, 2026
First slice of #2216, scoped to the system/status lines in the chat
timeline.

## Why

Two problems on the same surface.

**Clearing a channel topic renders as empty quotes.** The relay reports
a clear as a `topic_changed` event carrying an empty string — there's no
separate "cleared" event type. So the timeline printed:

> Alice
> changed the topic to “”

which reads as if the topic were *set to* two quote marks. Same for
purpose.

**The membership caption reads like a headline, not a metadata line.**
`title` and `action` render on separate lines — the member's name sits
in the header row with the avatar and timestamp, and the caption sits
beneath it. So the caption was "was added by Alice Chen" standing alone
under a name, while its siblings on that same line are "joined the
channel" and "left the channel".

## What

- Blank, missing, or whitespace-only topic/purpose now reads **"cleared
the channel topic"** / **"cleared the channel purpose"**.
- Membership captions drop "was": **"added by Alice Chen"**, matching
"joined the channel" and "left the channel".
- The wording moves to `lib/systemEventCopy.ts` as a pure function, so
it's assertable in a unit test instead of only reachable through the
DOM. That also removes two JSX fragments from `SystemMessageRow.tsx`,
taking it 911 → 900 lines.

## Two E2E assertions this exposed

Both were measuring something other than what they claimed, and the copy
change tipped them over. Neither is a product bug, but both would have
failed the next person too.

1. **`mentions.spec.ts:1245`** asserted a button was un-underlined while
the mouse was still parked from a previous `hover()`. Any reflow — new
rows, scroll-to-bottom, a different text wrap — can slide that button
under the stationary pointer, so the assertion measured *where the mouse
happened to be* rather than the resting style. Dropping four characters
changed the text wrap, changed the row height, changed the scroll
offset, and the pointer landed on it. Now parks the pointer off-target
first.
2. **`mentions.spec.ts:1253`** used a bare `role=tooltip` lookup. Once
the first tooltip animates out while the second opens, two elements
match and strict mode trips. Now scopes to the open tooltip via
`:not([data-state="closed"])`.

## Deliberately out of scope

- **Timestamps.** The day divider, per-message clock times, the Inbox
thread pane, and the inbox list have three divergent date
implementations and none fully match the writing standard's
Today/Yesterday/weekday/date progression. That's its own slice of #2216.
- **Whose avatar shows.** An addition puts the *added* member in the
header; a removal puts the *remover* there. Possibly intentional, but
it's a design question, not copy.
- **`the channel` vs `this channel`.** joined/left/removed say "the
channel"; created/archived/unarchived say "this channel". Worth
normalizing, but it touches lines this PR otherwise leaves alone.

## Validation

- `pnpm check`, `pnpm typecheck` — clean
- Unit: **3781/3781**, including 6 new tests in
`systemEventCopy.test.mjs` covering set/blank/undefined/null/whitespace
for both fields, plus a guard that no variant can emit empty quotes
- Smoke E2E `mentions` + `messaging`: **85/85**
- The previously fragile test run with `--repeat-each=5`: **5/5**

Signed-off-by: Clay Delk <clay.delk@gmail.com>
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
calvadev pushed a commit to shopstr-eng/buzz that referenced this pull request Aug 3, 2026
## Why

The Profile settings action still says “Sign Out,” while its
confirmation action says “Delete My Data.” Both buttons trigger the same
destructive local-data wipe and should name it consistently.

## What

- Label both destructive actions “Delete my data”
- Assert the matching section and confirmation labels in the existing
Playwright coverage

## Risk Assessment

Low — copy and test assertions only; sign-out behavior is unchanged.

## References

- Follow-up to block#2208
- block#2216 also touches this copy and should preserve “Delete my data” when
rebased
- `just desktop-check`
- `just desktop-test` (3,275 tests)
- Desktop E2E build and sign-out Playwright spec (2 tests)

Generated with Codex

Signed-off-by: Bradley Axen <baxen@squareup.com>
calvadev pushed a commit to shopstr-eng/buzz that referenced this pull request Aug 3, 2026
First slice of block#2216, scoped to the system/status lines in the chat
timeline.

## Why

Two problems on the same surface.

**Clearing a channel topic renders as empty quotes.** The relay reports
a clear as a `topic_changed` event carrying an empty string — there's no
separate "cleared" event type. So the timeline printed:

> Alice
> changed the topic to “”

which reads as if the topic were *set to* two quote marks. Same for
purpose.

**The membership caption reads like a headline, not a metadata line.**
`title` and `action` render on separate lines — the member's name sits
in the header row with the avatar and timestamp, and the caption sits
beneath it. So the caption was "was added by Alice Chen" standing alone
under a name, while its siblings on that same line are "joined the
channel" and "left the channel".

## What

- Blank, missing, or whitespace-only topic/purpose now reads **"cleared
the channel topic"** / **"cleared the channel purpose"**.
- Membership captions drop "was": **"added by Alice Chen"**, matching
"joined the channel" and "left the channel".
- The wording moves to `lib/systemEventCopy.ts` as a pure function, so
it's assertable in a unit test instead of only reachable through the
DOM. That also removes two JSX fragments from `SystemMessageRow.tsx`,
taking it 911 → 900 lines.

## Two E2E assertions this exposed

Both were measuring something other than what they claimed, and the copy
change tipped them over. Neither is a product bug, but both would have
failed the next person too.

1. **`mentions.spec.ts:1245`** asserted a button was un-underlined while
the mouse was still parked from a previous `hover()`. Any reflow — new
rows, scroll-to-bottom, a different text wrap — can slide that button
under the stationary pointer, so the assertion measured *where the mouse
happened to be* rather than the resting style. Dropping four characters
changed the text wrap, changed the row height, changed the scroll
offset, and the pointer landed on it. Now parks the pointer off-target
first.
2. **`mentions.spec.ts:1253`** used a bare `role=tooltip` lookup. Once
the first tooltip animates out while the second opens, two elements
match and strict mode trips. Now scopes to the open tooltip via
`:not([data-state="closed"])`.

## Deliberately out of scope

- **Timestamps.** The day divider, per-message clock times, the Inbox
thread pane, and the inbox list have three divergent date
implementations and none fully match the writing standard's
Today/Yesterday/weekday/date progression. That's its own slice of block#2216.
- **Whose avatar shows.** An addition puts the *added* member in the
header; a removal puts the *remover* there. Possibly intentional, but
it's a design question, not copy.
- **`the channel` vs `this channel`.** joined/left/removed say "the
channel"; created/archived/unarchived say "this channel". Worth
normalizing, but it touches lines this PR otherwise leaves alone.

## Validation

- `pnpm check`, `pnpm typecheck` — clean
- Unit: **3781/3781**, including 6 new tests in
`systemEventCopy.test.mjs` covering set/blank/undefined/null/whitespace
for both fields, plus a guard that no variant can emit empty quotes
- Smoke E2E `mentions` + `messaging`: **85/85**
- The previously fragile test run with `--repeat-each=5`: **5/5**

Signed-off-by: Clay Delk <clay.delk@gmail.com>
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
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.

1 participant