[Birfy]

Refs #197 (items 2, 4, 5). Items 1, 3 and 6 stay open, see below.

Fixed

• Composer under a read-only past room (item 2). When a face-to-face room is left or closed, its "You left this room." / "… left the room. This room is closed." line already sits in the composer stack; a CSS rule now hides the message box whenever that line is shown. Open rooms are unchanged.
• Own handle contrast (item 4). The account-menu handle uses --bt-ink at 13px instead of --bt-ink-2 at 12px.
• "Friends' Bots" wording (item 5). The relationship_exists message (client and host) now says "You're already friends. Their Bot is back in your list." The held-message notice to a friend's Bot no longer points to a Friends' Bots section and names the setting the UI really shows: "Sent, but it waits for your approval in this chat. Turn on Auto-send for this friend to skip that next time." (It said "Auto-accept" before.) The small marker label "Friend's Bot" stays, as it names the Bot, not a section.

Not in this PR

• Item 1 (Past rooms stale until reload): the host already moves a left room to pastFaceRooms and social/face-rooms returns it on the next 5 s poll, so the cause is not visible from code. Needs a live repro (one guess to check: the slow pollFaceRooms() call to Bot Social blocking the reply).
• Item 3 (+ menu shown as a "New… ×" tab): needs the compact popover rework from Social UI polish round 3: truncation, duplicate Add friend/Accept invite, Manage access depth, wording (#166 follow-up) #173.
• Item 6 (Chief greeting in Chinese): the kickoff already says "Write in English" when no locale was reported, and the client reports the active DSH locale; needs a live repro to tell a model slip from a zh locale.

Ran: ARCH_DOCS_BASE=upstream/main npm run verify with TZ=UTC on Node 24: pass (core 750, social 161, tree 56, scripts 56, interop). New test #197: … in plugins/social/test/social.test.js. Changelog fragment changelog.d/197-social-ui-nits.fixed.md.
Not run: browser check of the CSS rule (it relies on DSH's _composerStack / _root + [data-composer-card] markup, the same hooks core/src/client/styles.js already uses).

[chatgpt-codex-connector]

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed 2026-10-17T16:27:22.374374Z	b68ca02	PR opened

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b68ca02af4

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Update the social subsystem doc or add the waiver

When checked against this commit's parent, cd core && ARCH_DOCS_BASE=129752c^ node scripts/check-arch-docs.mjs fails: these plugins/social/ edits are owned by docs/subsystems/social.md, but the commit changes neither that page nor includes a Docs-not-needed: trailer. Since npm run lint runs this gate, required CI cannot pass for this commit; document the changed UI contract or add the trailer if it is truly docs-neutral.

AGENTS.md reference: AGENTS.md:L94-L94

Useful? React with 👍 / 👎.