Skip to content

fix(gui): repair log tooltip encoding - #1935

Draft
olddonkey wants to merge 1 commit into
lidge-jun:devfrom
olddonkey:codex/fix-log-tooltip-encoding
Draft

fix(gui): repair log tooltip encoding#1935
olddonkey wants to merge 1 commit into
lidge-jun:devfrom
olddonkey:codex/fix-log-tooltip-encoding

Conversation

@olddonkey

@olddonkey olddonkey commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Replace malformed UTF-8 byte escapes in Logs model and token tooltips with a single Unicode middle dot, eliminating visible · mojibake.
  • Extract tooltip formatters for direct regression coverage.
  • Add a GUI source encoding hygiene test that rejects invalid UTF-8, common mojibake, and adjacent multibyte \xNN escape sequences.

Screenshots

Model tooltip (before)

Model tooltip showing mojibake separators

Token tooltip (before)

Token tooltip showing mojibake separators

Verification

  • bun run typecheck
  • bun run test — 12,753 pass / 10 skip / 0 fail
  • cd gui && bun test tests — 939 pass / 0 fail
  • cd gui && bun run lint
  • cd gui && bun run build
  • bun run privacy:scan
  • git diff --check

Checklist

  • Scope stays focused and avoids unrelated cleanup.
  • Docs or release notes were updated when needed. (Not needed: no configuration or workflow change.)
  • Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults. (No security-sensitive surface changed.)

Review readiness checklist

This PR stays in draft until every box below is ticked. Tick all four boxes once the requirements are met:

  • All CI tests are green on my local testing.

  • I pushed my PR to the latest dev commit.

  • I resolved all correct Codex and CodeRabbit findings.

  • My PR is ready for review.

@coderabbitai

coderabbitai Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 1b13f7bc-a85f-4026-965f-26575d18e4a1

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added the bug Something isn't working label Aug 17, 2026
@github-actions

Copy link
Copy Markdown
Contributor

⏳ DRAFT

  • review readiness checklist open (0/4 boxes ticked).

What to do

  • Tick all four boxes in the PR description once you're done (currently 0/4).

Review readiness checklist

  • ⬜ All CI tests are green on my local testing.
  • ⬜ I pushed my PR to the latest dev commit.
  • ⬜ I resolved all correct Codex and CodeRabbit findings.
  • ⬜ My PR is ready for review.

0/4 boxes ticked.

This PR stays in draft until every box above is ticked.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant