feat: add desktop localization and memory view - #395
Conversation
There was a problem hiding this comment.
💡 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".
| <span className="react-session-row__title">{displaySessionTitle(session.title, t)}</span> | ||
| <small>{formatRelativeUpdatedTime(session.updatedAtMs, now())}</small> |
There was a problem hiding this comment.
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 👍 / 👎.
Summary
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
Validation
npm test -- --run(82 files, 530 tests)npm run buildcargo test --manifest-path src-tauri/Cargo.toml --lib(884 tests)cargo check --manifest-path src-tauri/Cargo.toml --all-features --all-targetscargo build --manifest-path src-tauri/Cargo.toml --release