scriptoriumadmin/strix
1
0
Fork 0
mirror of https://github.com/usestrix/strix.git synced 2026-10-15 04:18:10 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

Fix coverage completeness: include 'failed' agent status and use word-boundary matching

Browse source 
- Add 'failed' to _INCOMPLETE_AGENT_STATUSES so agents that exited failed
  correctly mark the coverage record as incomplete (fixes #1458)
- Fix _entry_is_about substring matching bug: use word-boundary set
  membership instead of substring 'in' so terms like 'rce' don't match
  inside 'force' (fixes #1447: 'brute force' was falsely counted as RCE coverage)
This commit is contained in:
dajiaohuang 2026-10-07 14:47:50 +08:00
parent f1386cad37
commit e97112a88d
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
strix/report/coverage.py
View file

@ -55,7 +55,7 @@ OUTCOME_LABELS: dict[str, str] = {
}
#: Statuses that mean the agent stopped early rather than finishing its task.
_INCOMPLETE_AGENT_STATUSES = frozenset({"crashed", "stopped", "running", "waiting"})
_INCOMPLETE_AGENT_STATUSES = frozenset({"crashed", "failed", "stopped", "running", "waiting"})
#: Run statuses that mean the scan itself did not run to completion.
_INCOMPLETE_RUN_STATUSES = frozenset({"failed", "interrupted", "stopped", "running"})
@ -224,7 +224,8 @@ def _skill_phrasings(skill: str) -> list[list[str]]:
def _entry_is_about(entry: dict[str, Any], phrasings: list[list[str]]) -> bool:
"""True when a ledger row plausibly concerns any phrasing of a risk class."""
haystack = _normalized(f"{entry.get('risk_area', '')} {entry.get('surface', '')}")
return any(all(term in haystack for term in terms) for terms in phrasings)
haystack_words = set(haystack.split())
return any(all(term in haystack_words for term in terms) for terms in phrasings)
def skill_coverage_gaps(