Skip to content

fix(app): restore ChatView code block styling - #25

Merged
Itsnotaka merged 1 commit into
mainfrom
daniel/chat-code-block-style
Aug 4, 2026
Merged

Itsnotaka merged 1 commit into
mainfrom
daniel/chat-code-block-style

Conversation

@Itsnotaka

@Itsnotaka Itsnotaka commented Aug 4, 2026 •

Copy link
Copy Markdown
Contributor

What changed

  • Restored the former ChatView fenced-code geometry through Honk's Prose tokens: 12px inset, 8px radius, and 12px/18px mono type.
  • Removed the assistant code-line gutter and its per-line number nodes.
  • Made Pierre-backed read/write previews use the same Honk-owned frame and inset while leaving patch diff gutters unchanged.
  • Added focused structural and geometry tests plus a dev-time performance review.

Why

The current implementation mixed an editor-style line-number gutter into assistant prose and let Pierre choose source-preview spacing. Former ChatView used Pierre only for highlighted token spans and owned the code-block frame itself.

Performance

  • A 100-line streaming fence drops from 200 line-related spans to zero.
  • The eager startup graph stays at 105 modules and 869.9 KiB.
  • First-load assets are 53 raw bytes and 29 gzip bytes smaller.
  • The lazy thread chunk is 402 raw bytes and 212 gzip bytes smaller.

Checks

  • 23 focused app tests pass.
  • 4 focused Prose tests pass.
  • App production build passes.
  • UI typecheck passes.
  • Exact root pnpm dev starts Electron, the renderer, and a healthy sidecar.
  • React Doctor reports no new changed-file issue.

App typecheck, design lint, and the architecture check still report the known main-branch onboarding, dormant v2, and oversized-file debt. None is in this change.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@cursor

cursor Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@vercel

vercel Bot commented Aug 4, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
honk-marketing Ready Ready Preview Aug 4, 2026 8:16pm

Request Review

@Itsnotaka
Itsnotaka merged commit ad57fe4 into main Aug 4, 2026
4 of 5 checks passed
@Itsnotaka
Itsnotaka deleted the daniel/chat-code-block-style branch August 6, 2026 05:46

This branch was successfully deployed

1 active deployment
Preview — de65f45e Deployed Aug 4, 2026 by vercel[bot]
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