Skip to content

fix: bug sweep — profile, mentions, deep links, channel links, doctor page#480

Merged
wesbillman merged 2 commits intomainfrom
bug-sweep
May 5, 2026
Merged

fix: bug sweep — profile, mentions, deep links, channel links, doctor page#480
wesbillman merged 2 commits intomainfrom
bug-sweep

Conversation

@wesbillman
Copy link
Copy Markdown
Collaborator

Summary

  • Hide "Message" button on own profile (mobile) — added currentPubkeyProvider check to conditionally hide the button when viewing your own profile sheet
  • Fix multi-word @mention highlighting (mobile) — pre-process content to replace spaces with NBSP inside known mention names so gpt_markdown's combined regex matches multi-word names as single tokens; normalize back on lookup
  • Fix channel link not highlighted at message start (mobile) — prepend ZWS when content starts with #[word-char] to prevent gpt_markdown block processing from swallowing the channel link pattern
  • Force workspace reinit on deep link to same relay (desktop) — bump reinitKey in switchWorkspace when id === activeId so deep links properly reconnect even when the workspace is already active
  • Clean up Doctor settings page (desktop) — remove binary paths section and "Setup help" section that aren't useful in production builds

Test plan

  • dart format clean, dart analyze clean
  • tsc --noEmit clean, biome check clean
  • All pre-commit hooks pass (desktop, mobile, web, rust)
  • 201 unit tests pass (148 sprout-core + 53 sprout-auth)
  • Manual: verify own-profile sheet hides Message button
  • Manual: verify multi-word @mention highlights fully in message list
  • Manual: verify #channel-name highlights when it's the only message content
  • Manual: verify deep link to already-active relay reconnects properly
  • Manual: verify Doctor page shows only relevant info

🤖 Generated with Claude Code

wesbillman and others added 2 commits May 5, 2026 08:48
…annel links, doctor page

- Hide "Message" button on own profile sheet (mobile)
- Fix multi-word @mention highlighting via NBSP preprocessing (mobile)
- Fix channel link not highlighted at message start via ZWS guard (mobile)
- Force workspace reinit on deep link to same relay (desktop)
- Clean up Doctor settings page: remove binary paths and setup help sections (desktop)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The Doctor panel no longer renders the sprout-acp / sprout-mcp-server
binary path rows, so the matching expectations in profile.spec.ts were
asserting against testids that no longer exist.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@wesbillman wesbillman merged commit 89f4b5d into main May 5, 2026
14 checks passed
@wesbillman wesbillman deleted the bug-sweep branch May 5, 2026 19:08
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