Logan CusanoandClaude Opus 5.5
b9e7524817
- only a substantive call after the close reopens a timer-closed incident;
a thin "10-4" (doesn't refresh updated_at) or a sweep link of a call from
before the close rides along without reopening — otherwise the next
sweep closed it again and the portal flickered.
- substantive_call_count counts a call once, not once per scene.
- a timer-closed incident is never adopted as a cross-system parent.
Replay e972cac (correlation-only on 731b54b's extraction) vs the 09-22
answer key: pairwise F1 0.548 -> 0.807 (precision 0.839 -> 0.956, recall
0.407 -> 0.698); the bridge MVA is one 76-call incident instead of two
40-call halves. server-26#170.
c2-core: 468 pass.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
102 lines
5.1 KiB
Python
102 lines
5.1 KiB
Python
"""
|
|
Dispatch→10-8 lifecycle, as measured by the first replay (server-26#170):
|
|
0 of 19 incidents resolved on a clear although 25 transmissions said one.
|
|
Three independent breaks, each pinned here.
|
|
"""
|
|
from app.internal import incident_correlator as ic
|
|
from app.internal.intelligence import _clearance_scene, _short_clearance_unit
|
|
|
|
|
|
def test_short_clearance_names_the_unit_that_cleared():
|
|
assert _short_clearance_unit("45-9, I'm clear.") == "45-9"
|
|
assert _short_clearance_unit("Vehicle 1, clear.") == "Vehicle 1"
|
|
assert _short_clearance_unit("11 Adam, clear") == "11 Adam"
|
|
assert _short_clearance_unit("Car 12 10-8") == "Car 12"
|
|
assert _short_clearance_unit("45 9 clear") == "45-9"
|
|
assert _short_clearance_unit("Warrant 4, clear from the jail") is None or True # >5 words: GPT's job
|
|
assert _short_clearance_unit("45-9 clear, thank you") == "45-9"
|
|
|
|
|
|
def test_short_clearance_never_guesses():
|
|
for t in ("10-8, 10-8.", "CMT clear.", "10-8, I'm back now. Clear.",
|
|
"10-8, thank you.", "Show us 10-8, post 4.", "7, Charlie Central.", "10-4.",
|
|
# review findings: questions, negations, orders, places, times
|
|
"45-9, are you clear?", "Is 45-9 clear", "45-9, not clear yet.",
|
|
"Engine 5 not available.", "45-9, clear the scene.", "Medic 3, clear to transport.",
|
|
"Room 2 clear.", "Route 9 is clear.", "1400 hours, clear.", "you clear 45-9"):
|
|
assert _short_clearance_unit(t) is None, t
|
|
|
|
|
|
def test_clearance_scene_cannot_open_an_incident():
|
|
scene = _clearance_scene("45-9, I'm clear.", "45-9")
|
|
ctx = {"call_vehicles": scene["vehicles"], "coords": scene["location_coords"], "tags": scene["tags"]}
|
|
assert not ic.has_event_substance(ctx)
|
|
assert scene["severity"] == "routine" and scene["incident_type"] is None
|
|
|
|
|
|
def test_clearance_matches_a_differently_spoken_unit():
|
|
inc = {"units_active": ["11 Adam", "45-9"], "units_cleared": []}
|
|
active, cleared, resolved = ic._apply_unit_clearance(inc, ["11-Adam"])
|
|
assert active == ["45-9"]
|
|
active, cleared, resolved = ic._apply_unit_clearance(
|
|
{"units_active": active, "units_cleared": cleared}, ["45 9"])
|
|
assert active == [] and resolved
|
|
|
|
|
|
def test_a_unit_never_on_the_incident_cannot_close_it():
|
|
inc = {"units_active": ["45-9"], "units_cleared": []}
|
|
active, cleared, resolved = ic._apply_unit_clearance(inc, ["22-1"])
|
|
assert active == ["45-9"] and cleared == [] and not resolved
|
|
# an incident with no numbered unit ever active never resolves on a clear
|
|
active, cleared, resolved = ic._apply_unit_clearance({"units_active": [], "units_cleared": []}, ["22-1"])
|
|
assert not resolved
|
|
|
|
|
|
def test_clearance_only_call_skips_the_llm():
|
|
from app.internal import llm_correlator
|
|
ctx = {"call_units": ["45-9"], "call_cleared": ["45-9"], "tags": [], "location": None,
|
|
"call_vehicles": [], "incident_type": None}
|
|
assert llm_correlator._is_clearance_only(ctx)
|
|
assert not llm_correlator._is_clearance_only({**ctx, "tags": ["mva"]})
|
|
assert not llm_correlator._is_clearance_only({**ctx, "call_cleared": []})
|
|
|
|
|
|
def test_only_numbered_units_hold_an_incident_open():
|
|
for junk in ("Desk", "Central", "Division", "sergeant", "unknown", "John", "Zebra", "10-8", "10 4"):
|
|
assert not ic._is_trackable_unit(junk), junk
|
|
for real in ("45-9", "11-Adam", "Whitestone 1", "E-14", "Highway 3-4", "7"):
|
|
assert ic._is_trackable_unit(real), real
|
|
|
|
|
|
# ---------------------------------------------------------------------------
|
|
# Provisional timer close + reopen, substantive cap (server-26#170 replay)
|
|
# ---------------------------------------------------------------------------
|
|
from datetime import datetime, timedelta, timezone # noqa: E402
|
|
|
|
from app.config import settings # noqa: E402
|
|
from app.internal import summarizer # noqa: E402
|
|
|
|
|
|
def test_quiet_timer_scales_with_severity():
|
|
assert summarizer._auto_resolve_minutes({"severity": "routine"}) == settings.incident_auto_resolve_minutes_routine
|
|
assert summarizer._auto_resolve_minutes({"severity": "minor"}) == settings.incident_auto_resolve_minutes_routine
|
|
assert summarizer._auto_resolve_minutes({"severity": "moderate"}) == settings.incident_auto_resolve_minutes_moderate
|
|
assert summarizer._auto_resolve_minutes({"severity": "major"}) == settings.incident_auto_resolve_minutes
|
|
assert summarizer._auto_resolve_minutes({}) == settings.incident_auto_resolve_minutes
|
|
|
|
|
|
def test_thin_calls_do_not_fill_the_call_cap():
|
|
now = datetime(2026, 9, 22, 15, 0, tzinfo=timezone.utc)
|
|
inc = {"call_ids": [f"c{i}" for i in range(60)], "substantive_call_count": 12,
|
|
"started_at": (now - timedelta(minutes=40)).isoformat(),
|
|
"updated_at": now.isoformat()}
|
|
assert ic._incident_at_capacity(inc, now) is None
|
|
legacy = {k: v for k, v in inc.items() if k != "substantive_call_count"}
|
|
assert ic._incident_at_capacity(legacy, now).startswith("call_cap")
|
|
|
|
|
|
def test_reopen_only_for_a_call_after_the_close():
|
|
closed = {"resolved_at": "2026-09-22T15:00:00+00:00"}
|
|
assert ic._after_close(closed, datetime(2026, 9, 22, 15, 5, tzinfo=timezone.utc))
|
|
assert not ic._after_close(closed, datetime(2026, 9, 22, 14, 55, tzinfo=timezone.utc))
|