[danielgap]

Related issue / Issue relacionada

Refs #66

ODD: fase L2/L3, núcleo: catálogos y selector global. Sigue a #152 (L1) y #159 (ids estables). Gate de #66: "Tests de claves/fallback/sincronización y selección persistente".

Scope / Alcance

El idioma se elige en Ajustes → Apariencia → Idioma (Español / English). Se guarda al momento y cambia sin reiniciar:

• en las ventanas de la app, el menú lateral de Ajustes y los títulos de sus páginas;
• en Rust, el menú de la bandeja, los títulos nativos de las ventanas (Ajustes, detalle y efectos) y el diálogo de instalación de Codex.

El resto de textos se migra por áreas en los siguientes PRs (T2c). Con el idioma en es, todo se ve igual que hoy.

Son dos commits revisables por separado:

1. Rust: feat(i18n): switch tray, window titles and Codex dialog by locale

• src-tauri/src/i18n.rs: Locale (cualquier valor desconocido se trata como es) y enum Msg con match exhaustivo es/en. Los textos en español son los literales actuales, copiados tal cual, y un test lo comprueba.
• Al guardar un idioma distinto, save_config y patch_config reconstruyen la bandeja (set_menu, mismos ids y handlers) y re-titulan las ventanas abiertas (set_title). Al arrancar y al crear ventanas se usa el idioma configurado.
• La decisión (si cambió el idioma, qué etiquetas y qué títulos) es una función pura, locale_relabel, y se ejecuta con efectos inyectados (run_locale_relabel). Así se testea sin Tauri.
• Seguimientos de feat(i18n): ids estables de medidores entre Rust y JS (L2 #66) #159:
  • (1) Meter.id queda documentado como obligatorio. El historial hace debug_assert! si falta, y el fallback a la etiqueta para datos antiguos vive en una función pura, sample_key.
  • (2) Los medidores de anthropic_api, openai_api y copilot salen de funciones puras con test (budget_meter, premium_meter).

2. TS: feat(i18n): typed es/en catalogs and language selector in Settings

• src/i18n/es.ts: hecho con los literales existentes, sin retraducir (matiz 4 de L0). en.ts usa satisfies Messages, así que una clave que falte o sobre no compila.
• t(locale, key, params?) acepta solo claves válidas, interpola {placeholder} y, si falta un texto, cae así: locale → es → la propia clave. intlLocale es un mapa cerrado es→es-ES, en→en-US.
• Tests de runtime: todas las claves existen en los dos idiomas y los placeholders coinciden.
• useLocale() (Ajustes, detalle): lee la config y escucha config-changed. Vuelve a leer la config, porque el payload a veces es (). El widget usa su propia config, que ya recibe ese evento.
• Selector: guarda por la misma cola que el resto de Ajustes. El borrador del formulario también adopta el idioma; si no, aparecía "cambios sin guardar" y "Guardar cambios" devolvía el idioma anterior. Está cubierto con test.
• docs/i18n-inventory.md regenerado.

Fuera de alcance: el resto de textos de React y Rust (T2c), los formatos y los errores propios (L4), y message/plan/periodLabel de los proveedores (L4).

Tamaño: unas 1050 líneas escritas con los tests (Rust ~550, TS ~560), más el inventario generado. Lo mantengo en un PR porque el cambio de idioma solo se puede probar completo con las dos partes. Si prefieres revisarlo en dos, lo separo.

Coordination / Coordinación

• feat: interfaz bilingüe ES/EN con español por defecto #66 asignada a @danielgap; L0 aprobado, L1 (feat(i18n): persistir el idioma de la interfaz e inventario reproducible de textos (L1 #66) #152) y los ids estables (feat(i18n): ids estables de medidores entre Rust y JS (L2 #66) #159) fusionados.

Contribuidor: @danielgap

Acceptance and validation / Aceptación y validación

• Criterios cumplidos, de feat: interfaz bilingüe ES/EN con español por defecto #66: "Selector en Ajustes actualiza todas las ventanas y textos nativos" (para los textos ya migrados) y la selección es persistente.
• Comprobaciones ejecutadas (Windows 10):
  • Tests primero:
    • Rust: los tests de i18n, del historial y de los medidores extraídos fallaron antes de implementar. Los de locale_relabel no compilaban (E0425) antes de existir la función.
    • TS: los tests de catálogos, del hook y de la navegación fallaron porque los módulos no existían. changeLocale falló con "is not a function".
  • cargo test: 251/251.
  • npx vitest run --dir src: 651/651.
  • npx tsc --noEmit: OK.
  • node scripts/i18n-inventory.mjs --check: OK.
  • Prueba manual con npm run app:dev. Al cambiar a English:
    • el menú lateral y el título de Ajustes cambian al momento;
    • la bandeja (clic derecho) muestra "Show / hide widget", "Refresh now", "Settings…", "Quit";
    • "Guardar cambios" no muestra el aviso de cambios sin guardar ni revierte el idioma;
    • vuelta a Español correcta.
• Comprobaciones no ejecutadas y motivo:
  • Que Tauri pinte el texto nuevo no tiene test automático. En tauri 2.12.1, MockRuntime::set_title no hace nada y title() devuelve "", y mock_context arranca sin bandeja. Por eso se testea la decisión, y lo pintado se probó a mano.
  • El onChange del <select> no tiene test de componente, porque el repo no usa jsdom ni testing-library. La lógica está en changeLocale, que es pura y tiene test. Si te parece bien añadir esas devDependencies, puedo proponerlo aparte.
  • Prueba en Linux y macOS: no disponible.
• Bloqueos pendientes: ninguno.

[Copilot]

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

[egdev6]

Gracias, @danielgap. He revisado 0764048 y veo dos puntos que conviene corregir antes de fusionar:

1. P1 · CI bloqueada en Windows, Linux y macOS. Los fixtures de src-tauri/src/i18n.rs:257–264 incluyen Quorb Dev y quorb-dev, que detecta el guard de retirada de ese canal. Lo reproduje con node --test scripts/dev-retirement.test.mjs; es también el fallo de las tres CI. Usa una identidad ficticia distinta para probar las sustituciones, sin relajar el guard.
2. P2 · Idioma pendiente invisible si falla el guardado. changeLocale (src/i18n/use-locale.ts:28–31) cambia el borrador antes de persistir, pero el selector (src/Settings.tsx:2225) muestra el locale del backend y el error solo llega a consola (:1377). Inyectando un fallo de persistencia, el borrador queda en en y el backend en es: el selector muestra Español, pero «Guardar cambios» puede aplicar después English. Conviene reconciliar el selector y el borrador al fallar, mostrar el error y cubrir ese caso con un test.

Validación: 651 tests frontend, TypeScript, build y 7 tests aislados del catálogo Rust pasan. La suite de scripts pasa 220/221, con el guard anterior como único fallo. No ejecuté cargo test completo ni pruebas GUI.

[danielgap]

¡Gracias por la revisión tan completa! Ambos puntos quedaron resueltos en 7bc6ab3.

• P1: los fixtures del test de Rust usan ahora la identidad ficticia «Quorb Beta» / .quorb-beta, así que el guard de retirada pasa sin relajarse: node --test scripts/dev-retirement.test.mjs 5/5 y cargo test i18n 7/7.
• P2: changeLocale devuelve el borrador al locale que realmente persistió el backend cuando el guardado falla (conservando el resto de ediciones) y relanza el error; Ajustes lo muestra junto al selector con la clave nueva settings.language.error en ambos catálogos. Cubierto con dos tests de unidad en use-locale.test.ts.

Verifiqué además la suite completa en local: vitest 653/653, tsc --noEmit limpio y suite de scripts verde salvo setup-signing.test.mjs, que ya fallaba en el commit base por EPERM de symlinks en Windows (limitación del entorno local, no del cambio). La CI del PR está corriendo ahora.

[egdev6]

Verificado en local sobre 7bc6ab3 (rama actualizada a pr-163):

• P1 ✓ — Los fixtures del test de Rust usan ahora «Quorb Beta» / .quorb-beta (identidad ficticia, no la retirada Dev) y el guard no se relaja: node --test scripts/dev-retirement.test.mjs 5/5 y cargo test i18n 7/7 en verde.
• P2 ✓ — changeLocale relee el locale persistido del backend al fallar el guardado, devuelve el borrador a ese locale conservando el resto de ediciones y relanza el error; Ajustes lo muestra junto al selector con settings.language.error en ambos catálogos. Los dos tests nuevos de use-locale.test.ts cubren la reconciliación y el rethrow del doble fallo. tsc --noEmit limpio y suite completa verde (653/653 antes del merge).

Un detalle nuevo tras el merge con main (e2cf99b): docs/i18n-inventory.md quedó desactualizado — node scripts/i18n-inventory.mjs --check sale con código 1 en local (los tree hashes del encabezado no coinciden con los árboles del merge) y de hecho el job Tests and build (Windows) de la CI falló justo en ese paso. Basta regenerarlo en la rama:

node scripts/i18n-inventory.mjs --out docs/i18n-inventory.md

Con ese commit el PR quedaría listo para aprobar por mi parte.

[egdev6]

Regenerado docs/i18n-inventory.md en 6eb163e tras el merge con main: node scripts/i18n-inventory.mjs --check pasa en local y la CI del PR está verde en los cuatro workflows (CI, Linux PR CI, Code safety y macOS CI, además de Security). Por mi parte, el PR queda listo para fusionar.