feat: multiple UI improvements, font settings, and memory file display names #175

Merged

naomi merged 22 commits from fix/tweaks into main 2026-03-03 20:21:58 -08:00

Conversation 0 Commits 22 Files Changed 48

+7148 -101

hikari commented 2026-03-02 15:39:36 -08:00

Owner

Copy Link

Copy Source

Summary

• fix: show_thinking_blocks setting now persists across sessions — it was defined on the TypeScript side but missing from the Rust HikariConfig struct, so serde silently dropped it on every save/load
• feat: Tool calls are now rendered as collapsible blocks matching the Extended Thinking block aesthetic, replacing the old inline dropdown approach
• feat: Add configurable max output tokens setting
• feat: Use random creative names for conversation tabs
• test: Significantly expanded frontend unit test coverage
• docs: Require tests for all changes in CLAUDE.md
• feat: Allow users to specify a custom terminal font (Closes #176)
• feat: Display friendly names for memory files derived from the first heading (Closes #177)
• feat: Add custom UI font support for the app chrome (buttons, labels, tabs)
• fix: Apply custom UI font to the full app interface — .app-container was hardcoded, blocking inheritance from body; also renamed "Custom Font" to "Custom Terminal Font" for clarity

✨ This PR was created with help from Hikari~ 🌸

## Summary - **fix**: `show_thinking_blocks` setting now persists across sessions — it was defined on the TypeScript side but missing from the Rust `HikariConfig` struct, so serde silently dropped it on every save/load - **feat**: Tool calls are now rendered as collapsible blocks matching the Extended Thinking block aesthetic, replacing the old inline dropdown approach - **feat**: Add configurable max output tokens setting - **feat**: Use random creative names for conversation tabs - **test**: Significantly expanded frontend unit test coverage - **docs**: Require tests for all changes in CLAUDE.md - **feat**: Allow users to specify a custom terminal font (Closes #176) - **feat**: Display friendly names for memory files derived from the first heading (Closes #177) - **feat**: Add custom UI font support for the app chrome (buttons, labels, tabs) - **fix**: Apply custom UI font to the full app interface — `.app-container` was hardcoded, blocking inheritance from `body`; also renamed "Custom Font" to "Custom Terminal Font" for clarity ✨ This PR was created with help from Hikari~ 🌸

hikari added 2 commits 2026-03-02 15:39:36 -08:00

fix: persist show_thinking_blocks setting across sessions c5d1df351c

feat: restyle tool calls as collapsible blocks matching thinking block aesthetic 08f7ca2d55

naomi changed title from fix: assorted small tweaks to feat: tinking blocks setting persists, tool calls rendered nicer 2026-03-02 15:48:11 -08:00

naomi changed title from feat: tinking blocks setting persists, tool calls rendered nicer to feat: thinking blocks setting persists, tool calls rendered nicer 2026-03-02 15:48:16 -08:00

naomi added 1 commit 2026-03-03 09:47:19 -08:00

feat: add configurable max output tokens setting 19e28b7ec7

hikari added 1 commit 2026-03-03 10:56:50 -08:00

feat: use random creative names for conversation tabs 66c65a6ab8

hikari added 1 commit 2026-03-03 12:05:37 -08:00

test: expand frontend unit test coverage to 30% fd3122e080

hikari added 1 commit 2026-03-03 12:21:55 -08:00

test: add component logic tests and clean up eslint directives ... 514e137590

Adds mirror-function tests for five Svelte components (HighlightedText,
CliVersion, AchievementNotification, StatusBar, ConversationTabs) and
removes stale eslint-disable comments from existing store test files.

hikari added 1 commit 2026-03-03 12:52:14 -08:00

test: add clipboard and notification rules tests ... 7e45c685d3

Adds 55 tests for clipboard store pure functions (detectLanguage for 12
languages + formatTimestamp with fake timers) and 13 tests for the
notification rules state machine (reconnect transition logic and no-op
handlers). Total test count now 698 across 30 files.

hikari added 6 commits 2026-03-03 16:06:41 -08:00

test: add coverage for messageMode, stateMapper branches, and initStatsListener acebe590c3

test: add coverage for configStore methods and derived stores 58f53a421b

test: add coverage for drafts, soundPlayer, wslNotificationHelper, and costTrackingStore c819adc9ea

test: add coverage for terminalNotifier, testNotifications, and claude store a5fb22bd23

test: add coverage for debugConsole, notificationManager, and achievements ea53bf0d4d

test: add coverage for slashCommands execute functions and notificationManager Method 4 55ad039451

hikari added 1 commit 2026-03-03 16:28:42 -08:00

test: fix clipboard coverage (real imports) and add cd/new success paths 3b8a093421

hikari added 1 commit 2026-03-03 16:41:14 -08:00

test: add comprehensive sessions store coverage (escapeHtml, generateHtmlExport, all exports) 16f92e22b9

hikari added 2 commits 2026-03-03 17:43:10 -08:00

test: add coverage for editorStore methods and file system operations f30ff60ea9

test: add coverage for editorStore methods and file system operations 67bae9a33c

hikari added 1 commit 2026-03-03 17:46:38 -08:00

docs: require tests for all changes in CLAUDE.md 9061098b0f

hikari added 1 commit 2026-03-03 18:07:20 -08:00

feat: allow users to specify a custom font (closes #176) ... 5bfff9d5e0

Adds support for loading a custom font from either a remote URL or a
local file path, and applying it to the terminal and input bar.

- Rust: adds `custom_font_path` and `custom_font_family` fields to
  `HikariConfig` with `#[serde(default)]` for backwards compatibility
- TypeScript: extends `HikariConfig` interface and `defaultConfig`;
  exports `applyCustomFont()` which injects an `@import` for CSS
  stylesheet URLs, a `@font-face` rule for direct font file URLs, or a
  base64 data URL `@font-face` for local files via Tauri `readFile`;
  adds `setCustomFont()` to the config store
- Terminal.svelte and InputBar.svelte now use
  `--terminal-font-family` CSS variable (falls back to `monospace`)
- ConfigSidebar.svelte: new "Custom Font" section with URL/path input,
  family name input, Apply + Reset buttons, and inline status feedback
- `+page.svelte`: applies saved font on startup alongside theme/size
- 14 new tests for `applyCustomFont` (all code paths) + 2 for
  `setCustomFont`

hikari added 1 commit 2026-03-03 18:20:01 -08:00

feat: display friendly names for memory files (closes #177) ... c5feb9b43c

The memory file list now shows human-readable titles instead of raw
filenames wherever possible.

- Rust: adds `MemoryFileInfo { path, heading }` and updates
  `MemoryFilesResponse` to use `Vec<MemoryFileInfo>`; adds
  `extract_first_heading()` helper that scans the first `# Heading`
  from a file's content; both `list_memory_files_native` and
  `list_memory_files_via_wsl` now read each file and populate the
  heading field; raw filename remains available as a fallback
- Frontend: updates `MemoryBrowserPanel.svelte` to use the richer type;
  adds `getDisplayName()` which returns the heading or falls back to the
  raw filename; file buttons use the display name with the raw filename
  as a tooltip; the viewer header also shows the display name with the
  filename shown as a subtitle when a heading is present
- Tests: 8 new Rust unit tests for `extract_first_heading` (happy path,
  edge cases, empty content, whitespace); 12 new TypeScript tests for
  `getFileName` and `getDisplayName` mirrored from the component

hikari added 1 commit 2026-03-03 18:41:28 -08:00

feat: add custom UI font support ... d2c39fd5c2

Allow users to specify a custom font for the entire app interface
(menus, labels, buttons) separately from the terminal font. Supports
Google Fonts URLs, direct font file URLs, and local file paths.

- Add custom_ui_font_path and custom_ui_font_family to Rust config
- Refactor applyCustomFont into shared applyFontFromSource helper
- Add applyCustomUiFont function using --ui-font-family CSS variable
- Update app.css to use --ui-font-family with fallback
- Apply custom UI font on startup in +page.svelte
- Add Custom UI Font section to ConfigSidebar settings panel
- Add tests for applyCustomUiFont and setCustomUiFont

hikari added 1 commit 2026-03-03 19:38:30 -08:00

fix: apply custom UI font to full app interface ... c3a6a2950a

- Set document.body.style.fontFamily directly (belt-and-suspenders
  alongside the CSS variable) to ensure the font propagates
- Use var(--ui-font-family) in .app-container so the main app area
  respects the custom font (was hardcoded, blocking inheritance)
- Rename 'Custom Font' setting to 'Custom Terminal Font' for clarity

hikari changed title from feat: thinking blocks setting persists, tool calls rendered nicer to feat: multiple UI improvements, font settings, and memory file display names 2026-03-03 19:40:14 -08:00

naomi merged commit fa906684c2 into main 2026-03-03 20:21:58 -08:00

naomi deleted branch fix/tweaks 2026-03-03 20:21:58 -08:00

naomi referenced this issue from a commit 2026-03-03 20:21:59 -08:00

feat: multiple UI improvements, font settings, and memory file display names (#175)

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

aspect

code

Concerns the software code in the repository

aspect

dx

Concerns developers' experience with the codebase

aspect

interface

Concerns end-users' experience with the software

aspect

text

Concerns the documentation material in the repository

contribute

good first issue

A great opportunity for a team member to learn a new codebase

contribute

help wanted

Open for anyone on our team to grab.

contribute

staff only

Restricted to our executive leadership.

goal

addition

Addition of new feature

goal

fix

Bug fix

goal

improvement

Improvement to an existing feature

points

1

Very simple issue requiring minimal effort and complexity.

points

13

Extremely complex issue representing major undertakings. Should be broken down into smaller pieces.

points

2

Simple issue that requires a bit more thought or investigation.

points

3

Moderate complexity issue requiring more substantial work.

points

5

Complex issue requiring significant effort and expertise.

points

8

Very complex issue requiring extensive work and deep expertise.

priority

critical

1

Must be fixed ASAP

priority

high

2

Stalls work on the project or its dependents

priority

low

4

Low priority and doesn't need to be rushed

priority

medium

3

Not blocking but should be fixed soon

priority

none

5

No priority, should only be performed when a developer is available

status

awaiting triage

Has not been triaged & therefore, not ready for work

status

blocked

Blocked and therefore not ready for work

status

discarded

Will not be worked on

status

discontinued

Not suitable for work as repo is in maintenance

status

label work required

Needs proper labelling before it can be worked on

status

ready for dev

Ready for work

status

ticket work required

Needs more details before it can be worked on

talk

discussion

Open for discussions and feedback

talk

question

Can be resolved with an answer

time

1 day

Approximately one full day of development work.

time

1-2 weeks

One to two weeks of focused development effort.

time

2-3 days

Two to three days of development effort.

time

4-5 days

Approximately one week of development work.

time

<1 day

Less than one day of focused work. Quick fixes or simple tasks.

time

>2 weeks

More than two weeks of development work. Must be broken down into smaller pieces.

No Label

Milestone

No items

No Milestone

Assignees

Clear assignees

gurkirat hanna (Hanna Rose) hikari (Hikari) minori (Minori) naomi (Naomi Carrigan) rain teklu tim

No Assignees

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: nhcarrigan/hikari-desktop#175