Skip to content

fix: use correct Unicode codepoint for branch indicator (⑂ not ⒂) — #1522#1523

Merged
2 commits merged intonesquena:masterfrom
franksong2702:fix/1522-branch-indicator-wrong-codepoint
May 3, 2026
Merged

fix: use correct Unicode codepoint for branch indicator (⑂ not ⒂) — #1522#1523
2 commits merged intonesquena:masterfrom
franksong2702:fix/1522-branch-indicator-wrong-codepoint

Conversation

@franksong2702
Copy link
Copy Markdown
Contributor

Thinking Path

Bug origin: Discord discussion — AvidFuturist and @frank noticed (15) appearing in front of forked session names. Investigated static/sessions.js and found the escape \u2482 maps to (PARENTHESIZED DIGIT FIFTEEN), not (OCR FORK). The developer comment says but the hex code is off by one digit: 2482 vs 2442.

What Changed

One character: \u2482\u2442 on line 1657 of static/sessions.js.

Escape Char Unicode Name
\u2442 OCR FORK ✅
\u2482 PARENTHESIZED DIGIT FIFTEEN ❌

Why It Matters

Forked sessions show a mysterious (15) that looks like a message count or unread badge. Users click into the session expecting something related to "15" and find nothing. The actual fork indicator is invisible.

Verification

  • \u2442 renders as in all major browsers/fonts
  • Tooltip ("Forked from ...") unchanged — still shows parent session ID
  • Click behavior unchanged — still navigates to parent session
  • No test changes needed (visual-only character swap)

Risks

Zero. Same <span>, same CSS class, same click handler. Only the Unicode codepoint inside textContent changes.

Model Used

mimo-v2.5-pro

Fixes #1522

Frank Song added 2 commits May 3, 2026 15:31
@nesquena-hermes
Copy link
Copy Markdown
Collaborator

Confirmed locally — static/sessions.js:1657 has '\u2482' with the inline comment // ⑂, which is exactly the "comment said one codepoint, escape contained another" bug. The escape is off by one digit (\u2482 = ⒂ PARENTHESIZED DIGIT FIFTEEN vs \u2442 = ⑂ OCR FORK).

Single-character swap, no test changes needed, zero behavioral risk beyond the visible glyph. The "(15)" that's been showing up in front of forked session names is now explained.

Will pull and verify the rendered output locally before merge — but this is about as low-risk as a fix gets. Thanks @franksong2702.

@nesquena-hermes nesquena-hermes closed this pull request by merging all changes into nesquena:master in f8ed6da May 3, 2026
sunnysktsang pushed a commit to sunnysktsang/hermes-webui that referenced this pull request May 3, 2026
sunnysktsang pushed a commit to sunnysktsang/hermes-webui that referenced this pull request May 3, 2026
…sorbed

CHANGELOG, ROADMAP, TESTING bumped (3936 \u2192 3946).

8 constituent PRs:
- nesquena#1523 (@franksong2702) branch indicator codepoint fix
- nesquena#1519 (@franksong2702) onboarding API-key focus loss fix
- nesquena#1518 (@franksong2702) voice-mode toggle-off recognizer stop
- nesquena#1516 (@franksong2702) YAML newline CSS rules
- nesquena#1517 (@franksong2702) __CACHE_VERSION__ \u2192 __WEBUI_VERSION__ rename
- nesquena#1532 (@ai-ag2026) state.db WebUI session recovery
- nesquena#1525 (@ai-ag2026) stale stream state proactive cleanup
- nesquena#1526 (@ai-ag2026) max_tokens forwarding + OpenRouter quota classifier

Opus MUST-FIX absorbed: sw.js conflict-marker cleanup + regression guard.
Opus SHOULD-FIX deferred to follow-up nesquena#1533 (race in _clear_stale_stream_state).

2 closed as duplicates: nesquena#1528 (identical to nesquena#1517), nesquena#1529 (superseded by nesquena#1516).
1 maintainer-review label: nesquena#1531 (Asunfly stowaway change in force-push).
5 stay on hold: nesquena#1418 nesquena#1464 nesquena#1404 nesquena#1353 nesquena#1311.
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.

bug(sidebar): branch indicator shows '⒂' (parenthesized 15) instead of ⑂ (fork icon)

2 participants