fix(ucca): Cross-Reg 0070 — beide Domaenen im Router-Top-K (Known Defects 0) #47

Merged

Benjamin_Boenisch merged 2 commits from fix/multi-reg-0070 into main 2026-06-30 13:42:29 +00:00

Conversation 0 Commits 2 Files Changed 4

+138 -3

Benjamin_Boenisch commented 2026-06-30 13:08:58 +00:00

Owner

Copy Link

Copy Source

Was & Warum

Der einzige offene Retrieval-Härtefall (Known Defects = 1 → 0): eine Query mit ≥2 explizit genannten Regelwerken („Wie greifen CRA und Maschinenverordnung ineinander?") lieferte nur die keyword-dominante Domäne (CRA), MaschVO fiel komplett aus dem Top-8. Drei zusammenwirkende Ursachen — alle behoben:

1. CodeValues-Mismatch: MaschVO heißt je Collection anders — Slice MASCHVO · gesetze MVO · ce MACHINERY/MASCHINENVO. Der Catalog hatte nur ["MASCHVO","MaschVO"] → der exakte per-Reg-Filter fand MaschVO nur in der Slice. Jetzt alle Varianten als CodeValues. (CRA/NIS2/DORA/AI ACT/DATA ACT gegen den dev-Korpus auditiert — stimmen überein.)
2. Per-Collection-Truncation: Router gab perColl=3 → searchMultiRegulation holte 3+3=6, schnitt auf 3 → konnte eine Domäne je Collection verlieren. Multi-Reg-Queries bekommen jetzt perColl = routerPerCollectionTopK * len(regs).
3. Router-Score-Merge starvte die nicht-dominante Domäne im finalen Top-K. Neue balanceByRegulation() gruppiert den gemergten Pool per Regelwerk (exakter regulation_code-Match gegen CodeValues) und nimmt round-robin über die genannten Domänen → jede Domäne mit Treffern ist garantiert im Top-K. Generisch über jede genannte Menge; Single-Domain-Pfad unverändert.

Validierung

• Go-Unit: TestBalanceByRegulation (dominante CRA verdrängt MaschVO NICHT mehr; round-robin-Reihenfolge).
• 0070-e2e gegen dev (echter Retrieve()): top-8 = [CRA MVO CRA MVO CRA MVO CRA MASCHINENVO] = beide Domänen (vorher nur CRA).
• CB-100-Stichprobe (stride 5): OLD 11/20 → NEW 15/20, GAIN 4, REGR 0 — Gain-Profil unverändert ggü. dem Router-Baseline.
• go build/vet/gofmt grün.

Scope / Sicherheit

Nur internal/ucca (multi_regulation.go + authority_router.go + Tests), disjunkt iace/Registry/Obligation/backend. Ändert ausschließlich den Multi-Reg-Pfad (≥2 genannte Regelwerke); Single-Domain-Queries laufen byte-identisch. Folgt auf PR #46 (Authority Router).

🤖 Generated with Claude Code

## Was & Warum Der **einzige offene Retrieval-Härtefall** (Known Defects = 1 → 0): eine Query mit **≥2 explizit genannten Regelwerken** („Wie greifen **CRA** und **Maschinenverordnung** ineinander?") lieferte nur die keyword-dominante Domäne (CRA), MaschVO fiel komplett aus dem Top-8. Drei zusammenwirkende Ursachen — alle behoben: 1. **CodeValues-Mismatch:** MaschVO heißt je Collection anders — Slice `MASCHVO` · gesetze `MVO` · ce `MACHINERY`/`MASCHINENVO`. Der Catalog hatte nur `["MASCHVO","MaschVO"]` → der exakte per-Reg-Filter fand MaschVO **nur in der Slice**. Jetzt alle Varianten als CodeValues. (CRA/NIS2/DORA/AI ACT/DATA ACT gegen den dev-Korpus auditiert — stimmen überein.) 2. **Per-Collection-Truncation:** Router gab `perColl=3` → `searchMultiRegulation` holte 3+3=6, schnitt auf 3 → konnte eine Domäne je Collection verlieren. Multi-Reg-Queries bekommen jetzt `perColl = routerPerCollectionTopK * len(regs)`. 3. **Router-Score-Merge** starvte die nicht-dominante Domäne im finalen Top-K. Neue **`balanceByRegulation()`** gruppiert den gemergten Pool per Regelwerk (exakter `regulation_code`-Match gegen CodeValues) und nimmt **round-robin** über die genannten Domänen → jede Domäne mit Treffern ist garantiert im Top-K. Generisch über jede genannte Menge; **Single-Domain-Pfad unverändert**. ## Validierung - **Go-Unit:** `TestBalanceByRegulation` (dominante CRA verdrängt MaschVO NICHT mehr; round-robin-Reihenfolge). - **0070-e2e gegen dev** (echter `Retrieve()`): `top-8 = [CRA MVO CRA MVO CRA MVO CRA MASCHINENVO]` = **beide Domänen** (vorher nur CRA). - **CB-100-Stichprobe (stride 5):** OLD 11/20 → NEW 15/20, GAIN 4, **REGR 0** — Gain-Profil unverändert ggü. dem Router-Baseline. - `go build`/`vet`/`gofmt` grün. ## Scope / Sicherheit Nur `internal/ucca` (multi_regulation.go + authority_router.go + Tests), disjunkt iace/Registry/Obligation/backend. Ändert ausschließlich den **Multi-Reg-Pfad** (≥2 genannte Regelwerke); Single-Domain-Queries laufen byte-identisch. Folgt auf PR #46 (Authority Router). 🤖 Generated with [Claude Code](https://claude.com/claude-code)

Benjamin_Boenisch added 1 commit 2026-06-30 13:09:01 +00:00

fix(ucca): Cross-Reg 0070 — beide Regelwerk-Domaenen im Router-Top-K (Known Defects 0) ... f2d445b891

Der einzige offene Retrieval-Haertefall: eine Query mit >=2 genannten Regelwerken
("CRA und Maschinenverordnung") lieferte nur die keyword-dominante Domaene (CRA),
MaschVO fiel raus. Drei zusammenwirkende Ursachen, alle behoben:

1. CodeValues-Mismatch: MaschVO heisst je Collection anders (Slice MASCHVO ·
   gesetze MVO · ce MACHINERY/MASCHINENVO), der Catalog hatte nur ["MASCHVO","MaschVO"]
   → Filter fand MaschVO nur in der Slice. Jetzt alle Varianten als CodeValues.
2. Per-Collection-Truncation: der Router gab perColl=3 → searchMultiRegulation holte
   3+3=6, schnitt auf 3 → konnte eine Domaene je Collection verlieren. Multi-Reg-Queries
   bekommen jetzt perColl = 3*len(regs).
3. Router-Score-Merge starvte die nicht-dominante Domaene. Neue balanceByRegulation()
   gruppiert den gemergten Pool per Regelwerk (exakter regulation_code-Match) und nimmt
   round-robin ueber die genannten Domaenen → jede Domaene mit Treffern ist im Top-K.
   Generisch ueber jede genannte Menge; Single-Domain-Pfad unveraendert.

Validierung: Go-Unit (balanceByRegulation: dominante CRA verdraengt MaschVO NICHT mehr);
0070-e2e gegen dev (Retrieve() → [CRA MVO CRA MVO CRA MVO CRA MASCHINENVO] = beide
Domaenen, vorher nur CRA); CB-100-Stichprobe REGR 0 (Gain-Profil unveraendert).

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Benjamin_Boenisch added 1 commit 2026-06-30 13:31:04 +00:00

style(ucca): gocritic equalFold in balanceByRegulation (go-lint gruen) ... 1d65d99d5f

strings.EqualFold(code, cv) statt code==strings.ToUpper(cv) — behebt den einzigen
gocritic-Befund auf der neuen Zeile (CI go-lint, new-from-merge-base). Verhalten
unveraendert (case-insensitive exakter regulation_code-Match); Unit + 0070-e2e bleiben gruen.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Benjamin_Boenisch merged commit f0120b237e into main 2026-06-30 13:42:29 +00:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

config

Env vars, secrets management, startup validation

data-integrity

Transactions, indexes, pagination

frontend

Next.js, client-side concerns

observability

Logging, audit trails, health checks

reliability

Error handling, retries, circuit breakers

security

Auth, secrets, injection, tenant isolation

severity: critical

System-level risk, data breach or full outage

severity: high

Significant risk, must fix before go-live

severity: medium

Important but not blocking go-live

testing

Test coverage, integration tests

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

sharang (Sharang Parnerkar) Benjamin_Boenisch

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Benjamin_Boenisch/breakpilot-compliance#47