chore(i18n): mejoras de seguimiento al check del inventario (avisos no bloqueantes de #158)

Đã đóng
  • Có gì không ổn?
#160 2 bình luận 0 reaction 1 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@danielgap đang làm issue này rồi.

Từ ngày 25/10/2026.

  • #186 của @danielgap — đã merge

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
42/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
github-actions, javascript, typescript
Lĩnh vực
ci-cd, developer-experience

Hướng nghiên cứu

Read scripts/i18n-inventory.mjs, the CI workflow ci.yml, and the relevant CONTRIBUTING guidance. Start by choosing one of the three independent improvements: strengthen the workflow contract test, reduce inventory diff noise, or clarify the merge-ref check behavior. Check the existing tests and CI setup to locate the current behavior; done means the selected improvement works and its check or documentation reflects the intended behavior.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

type:chore
Pre-flight checks
  • I searched existing issues and found no duplicate.
  • I understand that this proposal requires product review before implementation.

Seguimiento de los avisos no bloqueantes declarados en la PR #158 (CI del inventario i18n de #155). Nada de esto bloquea #158; se recoge para no perderlo.

Problem to solve

La revisión automática independiente de #158 aprobó el cambio con tres avisos que quedaron como trabajo posterior y no tienen issue propia.

Proposed solution

Tres mejoras, priorizables por separado:

  1. Test de contrato más robusto: el test que verifica que ci.yml ejecuta scripts/i18n-inventory.mjs --check busca strings con indexOf y no parsea el YAML. Sustituirlo por un parseo real del workflow (o una aserción estructural) para que reordenes cosméticos del job no lo rompan ni lo den por bueno en falso.

  2. Reducir el ruido de diffs del inventario: cada fila del inventario lleva número de línea, así que cualquier desplazamiento de líneas en src/ o src-tauri/src/ regenera diffs grandes (y fuerza regenerar en casi toda PR que toque esas carpetas). Valorar si el generador puede emitir el check sobre contenido estable (p. ej. orden/conteno de textos sin números de línea, o números solo en un apéndice) para que el diff sea relevante solo cuando cambia un texto.

  3. Efecto merge-ref en PRs: CI hace checkout del merge con main; si main cambia src/ mientras una PR está abierta, hay que actualizar la rama y regenerar de nuevo. Documentado en CONTRIBUTING, pero se puede suavizar (mensaje del check más explícito, o verificación sobre el árbol de la rama en lugar del merge ref).

Affected area

CI / developer experience

Scope and non-goals

Mismo alcance que #155: el generador scripts/i18n-inventory.mjs, su check en CI y la convención en CONTRIBUTING. Fuera de alcance: catálogos y selector (L2/L3 de #66) y cualquier cambio de comportamiento visible de la app.

Cada mejora es una unidad de trabajo independiente; no es necesario hacerlas juntas.

Alternatives considered

Dejarlo como comentarios en #155: la issue se cierra con #158, así que los avisos se perderían de vista. Issue propia garantiza trazabilidad.

Ngôn ngữ chính
TypeScript
Star
10
Fork
2
Merge trung bình
7 giờ 51 phút
Pull request đã merge (30 ngày)
129

Chuẩn bị môi trường

Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của egdev6/quorb

Tất cả issue của egdev6/quorb

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.