Skip to content

test(l10n): add en.json coverage guard (test:l10n) - #49

Merged
rubenvdlinde merged 1 commit into
developmentfrom
wip/l10n-ci-guard
Jul 26, 2026
Merged

test(l10n): add en.json coverage guard (test:l10n)#49
rubenvdlinde merged 1 commit into
developmentfrom
wip/l10n-ci-guard

Conversation

@rubenvdlinde

Copy link
Copy Markdown
Contributor

Adds the self-configuring l10n coverage guard (tests/l10n/check-l10n.js, copied verbatim from docudesk) to bring hermiq to parity with openconnector, softwarecatalog, procest, docudesk.

  • New test:l10n npm script: node tests/l10n/check-l10n.js
  • Auto-derives app id from package.json name; scans src/ for t()/n()/$t/this.t calls and asserts every literal key exists in l10n/en.json
  • info.xml patch version bump (0.1.95 -> 0.1.96)

NOTE: The guard currently reports 8 genuine missing keys — plural n() source strings never extracted to l10n/en.json (both singular and plural forms entirely absent). These are pre-existing coverage gaps the guard correctly surfaces; per task scope no l10n JSON was touched here. Misses:

  • "%n node" / "%n nodes" (src/views/GraphSidebar.vue:198)
  • "%n connection" / "%n connections" (src/views/GraphSidebar.vue:199)
  • "%n tool" / "%n tools" (src/components/AgentSelector.vue:63)
  • "Exported %n record." / "Exported %n records." (src/widgets/ComplianceOperations.vue:97)

@rubenvdlinde
rubenvdlinde merged commit cce9b3c into development Jul 26, 2026
8 of 18 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

Quality Report — ConductionNL/hermiq @ 0e35ff9

Check PHP Vue Security License Tests
lint
phpcs
phpmd
psalm
phpstan
phpmetrics
eslint
stylelint
composer ❌ 1/117 denied
npm ❌ 1/659 denied
PHPUnit ⏭️
Newman ⏭️
Playwright ⏭️

❌ Denied composer licenses

Package Version License
smalot/pdfparser v2.12.5 LGPL-3.0

❌ Denied npm licenses

Package Version License
vue3-apexcharts 1.11.1 Custom: https://apexcharts.com/media/vue-apexcharts.png

Quality workflow — 2026-07-26 21:58 UTC

Download the full PDF report from the workflow artifacts.

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