Skip to content

feat: add desktop localization and memory view - #395

Merged
SudoJacky merged 2 commits into
masterfrom
feat/desktop-localization-memory-ui
Aug 9, 2026
Merged

feat: add desktop localization and memory view#395
SudoJacky merged 2 commits into
masterfrom
feat/desktop-localization-memory-ui

Conversation

@SudoJacky

Copy link
Copy Markdown
Owner

Summary

  • add persistent English and Simplified Chinese app language settings using an extensible i18n resource structure
  • add a desktop Memory page backed by a native memory snapshot command
  • remove the obsolete stdio worker transport and other unreachable Rust APIs while retaining active diagnostics

Why

Tinybot needed a device-local language preference and a way to inspect current long-term memory. The Rust backend also retained a complete stdio worker path after the desktop runtime moved in-process, which produced widespread dead-code warnings and increased maintenance and test compilation overhead.

Impact

  • users can switch the desktop UI between English and Simplified Chinese
  • users can review user and workspace memory from the app
  • default and all-feature Rust builds are warning-free
  • the native source and test surface is reduced by about 2.8k lines without removing active diagnostics

Validation

  • npm test -- --run (82 files, 530 tests)
  • npm run build
  • cargo test --manifest-path src-tauri/Cargo.toml --lib (884 tests)
  • cargo check --manifest-path src-tauri/Cargo.toml --all-features --all-targets
  • cargo build --manifest-path src-tauri/Cargo.toml --release

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f03223802c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +2195 to 2196
<span className="react-session-row__title">{displaySessionTitle(session.title, t)}</span>
<small>{formatRelativeUpdatedTime(session.updatedAtMs, now())}</small>

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Pass the selected locale to sidebar timestamps

When the user selects Chinese, session titles and controls in this sidebar are translated, but this call still omits the locale and localized no-date label. formatRelativeUpdatedTime therefore uses its English fallback strings such as 4 min, 2 hr, and No date, leaving every sidebar session timestamp untranslated; pass the active i18n language and translated no-date text here as the search results do.

Useful? React with 👍 / 👎.

@SudoJacky
SudoJacky merged commit 549cf82 into master Aug 9, 2026
1 check passed
@SudoJacky
SudoJacky deleted the feat/desktop-localization-memory-ui branch August 9, 2026 10:19
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