Skip to content

Add Hindi (hi) locale - #351

Merged
sox8502 merged 5 commits into
alphaXiv:mainfrom
SID-6921:add-hindi-locale
Sep 16, 2026
Merged

sox8502 merged 5 commits into
alphaXiv:mainfrom
SID-6921:add-hindi-locale

Conversation

@SID-6921

@SID-6921 SID-6921 commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Adds Hindi (hi) across the dashboard, including browser-language detection and the हिन्दी option in Settings. Includes the complete translated catalog and regenerated embedded UI assets, while preserving the Spanish locale added in #347.

Verification

  • Four independent Codex 5.6 Sol reviews: no blockers
  • Localization catalog/placeholder checks and style-token checks
  • Dev-slot helper tests (9 passed), UI typecheck, UI tests (163 passed)
  • Production UI build and regenerated ui/dist
  • Rust formatting, clippy, locked build and tests (855 passed, 2 ignored)
  • Release build; debug/release binaries report the development telemetry channel
  • In-app browser: switch from Español to हिन्दी, reload persistence, settings, chat, experiments, files, artifacts and projects
  • Source/built startup scripts and generated runtime: Hindi variants, preference ordering, English fallback and saved-language precedence
  • GitHub Actions, including Windows
  • User acceptance of the Hindi UI

- ui/messages/hi.json: full translation of all 1175 UI strings,
  validated against scripts/check-i18n.mjs (key parity, placeholder
  parity, no empty values, no HTML entities) and compiled cleanly
  with @inlang/paraglide-js 2.25.0.
- Register hi in project.inlang/settings.json.
- Add the Hindi option to the language picker (SettingsPage.tsx).
- Keep ui/index.html's early locale-detection list in sync, per its
  own comment.
Copilot AI lite review requested due to automatic review settings September 16, 2026 02:30

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

pnpm install was blocked in my first pass by a jsr-registry
resolution issue; that's now fixed (see PR discussion), so this
rebuilds the dashboard bundle the same way the merged Arabic
locale PR (alphaXiv#344) did.
@SID-6921

Copy link
Copy Markdown
Contributor Author

Update: found the root cause of the local build failure and pushed the missing piece.

It wasn't this repo or a sandboxing artifact — pnpm 12's new built-in "supply-chain policy" check falls back to re-verifying package metadata against the default registry.npmjs.org, even for packages resolved through a scoped custom registry (here, @clo/react-markdown's jsr: specifier, mapped to npm.jsr.io). Since JSR packages aren't mirrored onto the default registry under that translated scope name, the cross-check 404s and aborts the whole install. pnpm install --trust-lockfile skips that check and installs cleanly; from there it's a normal pnpm approve-builds esbuild for esbuild's native postinstall, then pnpm build succeeds end to end.

Pushed a follow-up commit with a freshly built ui/dist (same two files the merged Arabic PR, #344, touched), so this should now be a complete, working locale contribution rather than partial.

@SID-6921

Copy link
Copy Markdown
Contributor Author

@sox8502 @myles332 — this is my first PR here, so the CI/Release workflow runs are sitting at `action_required` and need a maintainer to approve them before they'll execute. Whenever you get a chance to approve the run (or let me know if anything else is needed), happy to address feedback.

SID-6921 and others added 3 commits September 15, 2026 23:39
# Conflicts:
#	ui/dist/assets/index-C11jNpxS.js
#	ui/dist/index.html
#	ui/index.html
#	ui/project.inlang/settings.json
#	ui/src/components/SettingsPage.tsx
@sox8502
sox8502 merged commit 325eb50 into alphaXiv:main Sep 16, 2026
11 checks passed
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.

3 participants