test(config): unify coverage floors into a single coverage-thresholds.json

The CI gate hard-coded floors in a bash array while the vitest configs
carried their own numbers — two ledgers that could drift apart (local
green, CI red). Move the per-workspace line-floors into a root
coverage-thresholds.json read by both: ci.yml via jq, and the workspace
vitest/jest configs via import.

packages/api-client (73) and routine-domain (74) gained local
enforcement for the first time — previously vitest exited 0 at any
coverage and only the CI bash array caught regressions.

(audit P2 — double-bookkeeping coverage thresholds)

Author: Skords-01

.github/workflows/ci.yml

@@ -411,32 +411,22 @@ jobs:
 
       - name: Check coverage threshold (≥75%)
         # Fails CI if any workspace drops below its floor.
-        # Default floor: 75%. Per-workspace overrides align the gate with
-        # the floors documented in each package's vitest.config / jest.config:
-        #   apps/web:              39  (vitest.config.js  lines: 39 — legacy idb/shared-lib-ui debt)
-        #   apps/mobile:           30  (jest.config.js    lines: 30 — React Native coverage gaps)
-        #   packages/api-client:   73  (approaching 75%; see api-client/vitest.config.ts)
-        #   packages/routine-domain: 74 (approaching 75%; see routine-domain/vitest.config.ts)
-        # Raise these floors as coverage improves in each package.
+        # Floors live in coverage-thresholds.json (single source of truth) —
+        # the same file is read by the workspace vitest/jest configs, so the
+        # local `pnpm test:coverage` run and this gate can't drift apart.
+        # Raise floors in that JSON as coverage improves.
         run: |
           set -euo pipefail
-          THRESHOLD=75
+          THRESHOLD=$(jq -r '.default' coverage-thresholds.json)
           FAILED=0
 
-          declare -A WORKSPACE_THRESHOLDS=(
-            ["apps/web"]=39
-            ["apps/mobile"]=30
-            ["packages/api-client"]=73
-            ["packages/routine-domain"]=74
-          )
-
           for summary in apps/*/coverage/coverage-summary.json packages/*/coverage/coverage-summary.json; do
             if [ ! -f "$summary" ]; then
               continue
             fi
 
             WORKSPACE=$(echo "$summary" | sed 's|/coverage/coverage-summary.json||')
-            WS_THRESHOLD=${WORKSPACE_THRESHOLDS[$WORKSPACE]:-$THRESHOLD}
+            WS_THRESHOLD=$(jq -r --arg ws "$WORKSPACE" '.workspaces[$ws] // .default' coverage-thresholds.json)
             COVERAGE=$(jq -r '.total.lines.pct' "$summary")
 
             if [ "$COVERAGE" = "null" ] || [ -z "$COVERAGE" ]; then

apps/mobile/jest.config.js

@@ -6,6 +6,10 @@
  * and a jsdom-free RN runtime. We run only tests under `src/**` to keep
  * Expo Router's app-directory out of Jest's test discovery.
  */
+// Line-floor — з кореневого coverage-thresholds.json (single source of truth,
+// той самий файл читає CI-гейт у ci.yml). Решта метрик — локальні.
+const sharedThresholds = require("../../coverage-thresholds.json").workspaces;
+
 module.exports = {
   preset: "jest-expo",
   testMatch: [
@@ -110,7 +114,7 @@ module.exports = {
   coverageReporters: ["text-summary", "lcov", "json", "json-summary"],
   coverageThreshold: {
     global: {
-      lines: 30,
+      lines: sharedThresholds["apps/mobile"],
       branches: 25,
       functions: 30,
       statements: 30,

apps/web/vitest.config.js

@@ -1,8 +1,19 @@
+import { readFileSync } from "node:fs";
 import { defineConfig } from "vitest/config";
 import react from "@vitejs/plugin-react";
 import { resolve } from "path";
 import { baseCoverageConfig } from "@sergeant/config/vitest.base";
 
+// Line-floor — з кореневого coverage-thresholds.json (single source of truth,
+// той самий файл читає CI-гейт у ci.yml). branches/functions/statements
+// лишаються локальними — CI гейтить тільки lines.
+const sharedThresholds = JSON.parse(
+  readFileSync(
+    new URL("../../coverage-thresholds.json", import.meta.url),
+    "utf8",
+  ),
+).workspaces;
+
 export default defineConfig({
   plugins: [react()],
   test: {
@@ -83,8 +94,9 @@ export default defineConfig({
         // Floors set ~2pp below current actuals — same pattern as
         // `apps/server/vitest.config.ts`. Raise per sprint as the idb /
         // shared-lib-ui tests land (see docs/testing/2026-05-05-tests-
-        // pr-plan.md → PR-T03 / PR-T04).
-        lines: 39,
+        // pr-plan.md → PR-T03 / PR-T04). `lines` приходить з кореневого
+        // coverage-thresholds.json — піднімай floor там.
+        lines: sharedThresholds["apps/web"],
         branches: 32,
         functions: 29,
         statements: 38,

coverage-thresholds.json

@@ -0,0 +1,10 @@
+{
+  "$comment": "Single source of truth для per-workspace coverage line-floors. Читається CI-гейтом (.github/workflows/ci.yml :: 'Check coverage threshold') і конфігами workspace-ів: vitest (apps/web, packages/api-client, packages/routine-domain) та jest (apps/mobile). Піднімай floors тут у міру росту покриття.",
+  "default": 75,
+  "workspaces": {
+    "apps/web": 39,
+    "apps/mobile": 30,
+    "packages/api-client": 73,
+    "packages/routine-domain": 74
+  }
+}

docs/90-work/audits/2026-06-11-fable5-independent-audit.md

@@ -86,15 +86,20 @@
 - ✅ Monobank `historyFetch.ts`/`privat.ts` без тестів → додано 24 unit-тести (`privat.test.ts` — guard/path-allowlist/CRLF/upstream-mapping; `historyFetch.test.ts` — schema/buildMemoryContent/fetchAccountStatement; pure-helpers експортовані).
 - ✅ Postgres-skew: Testcontainers pg16 vs CI pg17 → усі 16 тест-контейнерів + прозові коментарі підняті до pg17 (збіг з CI service + docker-compose).
 
+**P2 — закриті у другій хвилі 2026-06-12:**
+
+- ✅ Hard Rule #1 без механізму → глобальний `pg.types.setTypeParser(int8)` у `apps/server/src/lib/pgInt8.ts` (install у db.ts; safe-integer guard — fail loud замість мовчазної втрати точності; 6 unit-тестів). Серіалізатори лишаються другим рубежем.
+- ✅ Coverage-пороги двобухгалтерні → single source `coverage-thresholds.json` у корені: ci.yml-гейт читає його через jq, vitest-конфіги (web, api-client, routine-domain) і jest (mobile) імпортують `lines`-floor звідти. api-client (73) і routine-domain (74) уперше отримали локальний enforcement (раніше — тільки CI-bash). Web-floor 39→50 — окремий burn-down, не закритий.
+- ✅ Orphan `syncV2Types.ts` → видалено (#3522, chip `task_abed59b4`).
+
 **P2 — далі:**
 
 - Freshness-механіка міряє churn, не review (`bump-last-validated` штампує будь-який staged .md; 53% корпусу проштамповано одним link-rewrite комітом). Потрібен дизайн: ручний validate-маркер vs churn-bump.
 - SLO/alert-стек: фантомні Alertmanager-згадки прибрані + wiring-статус задокументований (ws-15, #3519). Лишається: UptimeRobot (founder) + рішення про Grafana Cloud rules sync чи видалення 24 design-правил.
 - Merge-серіалізація: ≥3 колізії номерів міграцій на main; GitHub merge queue або timestamp-префікси. (= ws-14, founder-gated)
-- Coverage-пороги двобухгалтерні (vitest-конфіги vs bash-масив у ci.yml) + web-floor 39%/32% проти цілі 50/40.
+- Web coverage-floor 39%/32% проти цілі 50/40 — burn-down тестами, не конфігом.
 - pnpm audit critical-gate без exception-path (audit-exceptions ledger не читається гейтом; escape — лише PR-label `audit-exception` для high, не для critical).
-- Hard Rule #1 — конвенція без механізму: нема глобального pg `setTypeParser` для int8.
-- Orphan-схема: `syncV2Types.ts` (0 імпортерів; шит-маркер цитує неіснуючий ADR-0062-decomposition — номер зайнятий OpenAPI-ADR → chip `task_abed59b4`), m047/m070-072 billing-орфани (two-phase DROP post-launch), db-schema pg-runner з розбіжним ledger-default.
+- Orphan-схема: m047/m070-072 billing-орфани (two-phase DROP post-launch), db-schema pg-runner з розбіжним ledger-default.
 - i18n: en.ts 215 рядків vs uk.ts 847; allowlist 243 файли без ratchet.
 - ESLint baselines без дедлайнів: react-hooks ~152 off-порушень, ~96 non-null assertions (burn-down «2026-Q3» без enforcement дати).
 

packages/api-client/vitest.config.ts

@@ -1,6 +1,17 @@
+import { readFileSync } from "node:fs";
 import { defineConfig } from "vitest/config";
 import { baseCoverageConfig } from "@sergeant/config/vitest.base";
 
+// Coverage line-floor — з кореневого coverage-thresholds.json (single source
+// of truth, той самий файл читає CI-гейт у ci.yml). Лишається тільки lines:
+// решту метрик CI не гейтить.
+const thresholds = JSON.parse(
+  readFileSync(
+    new URL("../../coverage-thresholds.json", import.meta.url),
+    "utf8",
+  ),
+) as { workspaces: Record<string, number> };
+
 export default defineConfig({
   test: {
     environment: "node",
@@ -9,6 +20,9 @@ export default defineConfig({
     coverage: {
       ...baseCoverageConfig,
       include: ["src/**/*.ts"],
+      thresholds: {
+        lines: thresholds.workspaces["packages/api-client"],
+      },
     },
   },
 });

packages/routine-domain/vitest.config.ts

@@ -1,6 +1,17 @@
+import { readFileSync } from "node:fs";
 import { defineConfig } from "vitest/config";
 import { baseCoverageConfig } from "@sergeant/config/vitest.base";
 
+// Coverage line-floor — з кореневого coverage-thresholds.json (single source
+// of truth, той самий файл читає CI-гейт у ci.yml). Лишається тільки lines:
+// решту метрик CI не гейтить.
+const thresholds = JSON.parse(
+  readFileSync(
+    new URL("../../coverage-thresholds.json", import.meta.url),
+    "utf8",
+  ),
+) as { workspaces: Record<string, number> };
+
 export default defineConfig({
   test: {
     environment: "node",
@@ -9,6 +20,9 @@ export default defineConfig({
     coverage: {
       ...baseCoverageConfig,
       include: ["src/**/*.ts"],
+      thresholds: {
+        lines: thresholds.workspaces["packages/routine-domain"],
+      },
     },
   },
 });