fix: Pass 0b — Duplicate Guard, Severity-Kalibrierung, Title-Truncation
All checks were successful
CI/CD / go-lint (push) Has been skipped
CI/CD / python-lint (push) Has been skipped
CI/CD / nodejs-lint (push) Has been skipped
CI/CD / test-go-ai-compliance (push) Successful in 55s
CI/CD / test-python-backend-compliance (push) Successful in 36s
CI/CD / test-python-document-crawler (push) Successful in 23s
CI/CD / test-python-dsms-gateway (push) Successful in 20s
CI/CD / validate-canonical-controls (push) Successful in 11s
CI/CD / Deploy (push) Successful in 4s
All checks were successful
CI/CD / go-lint (push) Has been skipped
CI/CD / python-lint (push) Has been skipped
CI/CD / nodejs-lint (push) Has been skipped
CI/CD / test-go-ai-compliance (push) Successful in 55s
CI/CD / test-python-backend-compliance (push) Successful in 36s
CI/CD / test-python-document-crawler (push) Successful in 23s
CI/CD / test-python-dsms-gateway (push) Successful in 20s
CI/CD / validate-canonical-controls (push) Successful in 11s
CI/CD / Deploy (push) Successful in 4s
1. Duplicate Guard: merge_hint-Lookup vor INSERT in _write_atomic_control() verhindert semantisch identische Controls unter demselben Parent. 2. Severity-Kalibrierung: action_type-basiert statt blind vom Parent. define/review/test → max medium, implement/monitor → max high. 3. Title-Truncation: Schnitt am Wortende statt mitten im Wort. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -1418,6 +1418,18 @@ _OBJECT_SYNONYMS: dict[str, str] = {
|
||||
}
|
||||
|
||||
|
||||
def _truncate_title(title: str, max_len: int = 80) -> str:
|
||||
"""Truncate title at word boundary to avoid mid-word cuts."""
|
||||
if len(title) <= max_len:
|
||||
return title
|
||||
truncated = title[:max_len]
|
||||
# Cut at last space to avoid mid-word truncation
|
||||
last_space = truncated.rfind(" ")
|
||||
if last_space > max_len // 2:
|
||||
return truncated[:last_space]
|
||||
return truncated
|
||||
|
||||
|
||||
def _normalize_object(object_raw: str) -> str:
|
||||
"""Normalize object text to a snake_case key for merge hints.
|
||||
|
||||
@@ -1613,11 +1625,11 @@ def _compose_deterministic(
|
||||
# ── Title: "{Object} {Zustand}" ───────────────────────────
|
||||
state = _ACTION_STATE_SUFFIX.get(action_type, "umgesetzt")
|
||||
if object_:
|
||||
title = f"{object_.strip()} {state}"[:80]
|
||||
title = _truncate_title(f"{object_.strip()} {state}")
|
||||
elif action:
|
||||
title = f"{action.strip().capitalize()} {state}"[:80]
|
||||
title = _truncate_title(f"{action.strip().capitalize()} {state}")
|
||||
else:
|
||||
title = f"{parent_title} {state}"[:80]
|
||||
title = _truncate_title(f"{parent_title} {state}")
|
||||
|
||||
# ── Objective = obligation text (the normative statement) ─
|
||||
objective = obligation_text.strip()[:2000]
|
||||
@@ -1678,7 +1690,7 @@ def _compose_deterministic(
|
||||
requirements=requirements,
|
||||
test_procedure=test_procedure,
|
||||
evidence=evidence,
|
||||
severity=_normalize_severity(parent_severity),
|
||||
severity=_calibrate_severity(parent_severity, action_type),
|
||||
category=parent_category or "governance",
|
||||
)
|
||||
# Attach extra metadata (stored in generation_metadata)
|
||||
@@ -2877,10 +2889,31 @@ class DecompositionPass:
|
||||
"""Insert an atomic control and create parent link.
|
||||
|
||||
Returns the UUID of the newly created control, or None on failure.
|
||||
Checks merge_hint to prevent duplicate controls under the same parent.
|
||||
"""
|
||||
parent_uuid = obl["parent_uuid"]
|
||||
candidate_id = obl["candidate_id"]
|
||||
|
||||
# ── Duplicate Guard: skip if same merge_hint already exists ──
|
||||
merge_hint = getattr(atomic, "source_regulation", "") or ""
|
||||
if merge_hint:
|
||||
existing = self.db.execute(
|
||||
text("""
|
||||
SELECT id::text FROM canonical_controls
|
||||
WHERE parent_control_uuid = CAST(:parent AS uuid)
|
||||
AND generation_metadata->>'merge_group_hint' = :hint
|
||||
AND release_state != 'rejected'
|
||||
LIMIT 1
|
||||
"""),
|
||||
{"parent": parent_uuid, "hint": merge_hint},
|
||||
).fetchone()
|
||||
if existing:
|
||||
logger.debug(
|
||||
"Duplicate guard: skipping %s — merge_hint %s already exists as %s",
|
||||
candidate_id, merge_hint, existing[0],
|
||||
)
|
||||
return existing[0]
|
||||
|
||||
result = self.db.execute(
|
||||
text("""
|
||||
INSERT INTO canonical_controls (
|
||||
@@ -3475,4 +3508,39 @@ def _normalize_severity(val: str) -> str:
|
||||
return "medium"
|
||||
|
||||
|
||||
# Action-type-based severity calibration: not every atomic control
|
||||
# inherits the parent's severity. Definition and review controls are
|
||||
# typically medium, while implementation controls stay high.
|
||||
_ACTION_SEVERITY_CAP: dict[str, str] = {
|
||||
"define": "medium",
|
||||
"review": "medium",
|
||||
"document": "medium",
|
||||
"report": "medium",
|
||||
"test": "medium",
|
||||
"implement": "high",
|
||||
"configure": "high",
|
||||
"monitor": "high",
|
||||
"enforce": "high",
|
||||
}
|
||||
|
||||
# Severity ordering for cap comparison
|
||||
_SEVERITY_ORDER = {"low": 0, "medium": 1, "high": 2, "critical": 3}
|
||||
|
||||
|
||||
def _calibrate_severity(parent_severity: str, action_type: str) -> str:
|
||||
"""Calibrate severity based on action type.
|
||||
|
||||
Implementation/enforcement inherits parent severity.
|
||||
Definition/review/test/documentation caps at medium.
|
||||
"""
|
||||
parent = _normalize_severity(parent_severity)
|
||||
cap = _ACTION_SEVERITY_CAP.get(action_type)
|
||||
if not cap:
|
||||
return parent
|
||||
# Return the lower of parent severity and action-type cap
|
||||
if _SEVERITY_ORDER.get(parent, 1) <= _SEVERITY_ORDER.get(cap, 1):
|
||||
return parent
|
||||
return cap
|
||||
|
||||
|
||||
# _template_fallback removed — replaced by _compose_deterministic engine
|
||||
|
||||
Reference in New Issue
Block a user