Skip to content

fix(desktop): use sentence case in trace - #4734

Open
Phoenix500526 wants to merge 1 commit into
apache:mainfrom
Phoenix500526:fix/usage-recorded-time-copy
Open

fix(desktop): use sentence case in trace#4734
Phoenix500526 wants to merge 1 commit into
apache:mainfrom
Phoenix500526:fix/usage-recorded-time-copy

Conversation

@Phoenix500526

@Phoenix500526 Phoenix500526 commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Summary

Normalize the Time breakdown labels to sentence case so they match the
adjacent Token usage labels.

  • Recorded Time becomes Recorded time
  • LLM Calls becomes LLM calls
  • Tool Runs becomes Tool runs

Verification

  • npm exec -- biome check apps/desktop/src/renderer/locales/conversation-copy.ts
  • npm --workspace @maka/desktop run typecheck
  • Verified the hot-reloaded Trace panel in the actual Desktop; all three labels
    render in sentence case without changing the existing layout
  • Automated behavioral tests were not added because this is a copy-only change

Before

image

After

image

AI use

Select exactly one:

  • No generative tool made a substantive contribution
  • Generative tooling made a substantive contribution

Tool(s) and scope: OpenAI Codex assisted with the copy edit, verification,
commit, and PR preparation.

Checklist

  • Tests cover the change and fail without it
  • Lint, format, typecheck and the affected suites pass locally

Does this PR entail a change in behavior?

  • Yes, the English labels now use sentence case
  • No

@github-actions github-actions Bot added the effort/XS Under 10 readable lines label Sep 4, 2026
Mixed title casing made adjacent usage labels look inconsistent.

Generated-by: OpenAI Codex
Signed-off-by: Jiawei Zhao <Phoenix500526@163.com>
@Phoenix500526
Phoenix500526 force-pushed the fix/usage-recorded-time-copy branch from 62de60b to 8ca76ae Compare September 4, 2026 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

effort/XS Under 10 readable lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant