loki-mode 9.17.0 → 9.17.2

package/dashboard/server.py

@@ -8354,93 +8354,6 @@ async def get_trust_trajectory():
     return traj
 
 
-# =============================================================================
-# Gate policy API: which gates block here, and which only advise?
-# =============================================================================
-
-_GATE_POLICY_MODULE = None  # cached import of autonomy/lib/gate_policy.py
-
-
-def _load_gate_policy_module():
-    """Import the shared gate-policy reporter (single source of truth).
-
-    Same shape as _load_trust_module above: the derivation lives in
-    autonomy/lib/gate_policy.py so this endpoint, the bash `loki gates`, and the
-    test suite all agree, and it loads via importlib because autonomy/lib is not
-    an importable package. Cached after first load, None when unavailable.
-    """
-    global _GATE_POLICY_MODULE
-    if _GATE_POLICY_MODULE is not None:
-        return _GATE_POLICY_MODULE
-    repo_root = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
-    mod_path = os.path.join(repo_root, "autonomy", "lib", "gate_policy.py")
-    if not os.path.isfile(mod_path):
-        return None
-    try:
-        import importlib.util as _ilu
-        spec = _ilu.spec_from_file_location("gate_policy", mod_path)
-        if spec is None or spec.loader is None:
-            return None
-        mod = _ilu.module_from_spec(spec)
-        spec.loader.exec_module(mod)
-        _GATE_POLICY_MODULE = mod
-        return mod
-    except Exception:
-        return None
-
-
-@app.get("/api/gate-policy", dependencies=[Depends(auth.require_scope("read"))])
-async def get_gate_policy():
-    """Which quality gates block on this repo, and which only advise.
-
-    Read-only by construction. There is deliberately no POST counterpart:
-    promoting an advisory gate to blocking stays an explicit operator act via
-    the environment variable named in each record's `promote_with`, because a
-    surface that can silently start blocking is the thing operators most
-    reasonably fear. This endpoint reports; it never promotes.
-
-    Honest-data rule, and the reason `audit_hits` is nullable: an unmeasured
-    gate reports null, NEVER 0. Zero is the positive claim that the gate ran and
-    never fired; an absent ledger is not evidence of that. See the shape
-    contract in the gate_policy module docstring.
-
-    Fails open like /api/trust/trajectory: a missing module returns a renderable
-    available=False payload rather than a 500. Unlike that sibling the assess()
-    call is also wrapped, because it reads a ledger file that an unrelated
-    writer can leave malformed, and a broken ledger must not take down a
-    read-only report.
-
-    Response keys are the module shape contract plus `available` (bool). On the
-    fail-open path `available` is False, `status` is "unavailable" (a value the
-    module contract does not itself define, since assess() never returned), an
-    `error` string explains why, and `gates` is empty. A UI should branch on
-    `available`, not on a non-empty `gates`.
-    """
-    mod = _load_gate_policy_module()
-    if mod is None:
-        return {
-            "schema_version": 1,
-            "available": False,
-            "error": "gate_policy module not found",
-            "status": "unavailable",
-            "ledger": "absent",
-            "gates": [],
-        }
-    try:
-        res = mod.assess(str(_get_loki_dir()))
-    except Exception as e:
-        return {
-            "schema_version": 1,
-            "available": False,
-            "error": f"gate policy assessment failed: {e}",
-            "status": "unavailable",
-            "ledger": "absent",
-            "gates": [],
-        }
-    res["available"] = True
-    return res
-
-
 # =============================================================================
 # Pricing API
 # =============================================================================
@@ -10270,71 +10183,6 @@ async def remove_checklist_waiver(item_id: str):
 # Council Hard Gate Endpoint (Phase 4)
 # =============================================================================
 
-# Receipts name gates in snake_case; the UI rows are display names. Mapping is
-# explicit rather than derived (a lower()/replace() heuristic would silently
-# mis-map "Test Suite" <-> "test_coverage" and "Test Mutation" <->
-# "mutation_integrity", which are different gates).
-_RECEIPT_GATE_NAMES = {
-    "static_analysis": "Static Analysis",
-    "test_coverage": "Test Suite",
-    "code_review": "Blind Code Review",
-    "anti_sycophancy": "Anti-Sycophancy",
-    "mock_integrity": "Mock Integrity",
-    "mutation_integrity": "Test Mutation",
-    "doc_coverage": "Documentation Coverage",
-    "magic_debate": "Magic Modules Debate",
-}
-
-
-# Receipts say "passed"/"failed"; the UI switches on "pass"/"fail"
-# (dashboard-ui/components/loki-quality-gates.js:51). Passing the receipt's word
-# through unchanged matched nothing, so every gate fell to the "pending" default
-# and the page still read "0 Pass, 0 Fail, 8 Pending" while showing real
-# timestamps -- half-fixed, and only visible by loading the page.
-#
-# An UNRECOGNISED status maps to nothing and the gate stays pending. Guessing
-# (e.g. treating any unknown word as a pass) is how a future status string would
-# silently become a green badge.
-_RECEIPT_GATE_STATUS = {
-    "passed": "pass", "pass": "pass", "ok": "pass",
-    "failed": "fail", "fail": "fail", "blocked": "fail",
-}
-
-
-def _receipt_backed_gates():
-    """_DEFAULT_QUALITY_GATES, with results filled in from the newest receipt.
-
-    Fail-open by design: any error returns the plain defaults, so a malformed
-    receipt degrades to "pending" rather than breaking the Quality page.
-
-    Gates absent from the receipt keep status "pending" and get NO last_checked.
-    An absent measurement is never rendered as a result.
-    """
-    gates = [dict(g) for g in _DEFAULT_QUALITY_GATES]
-    try:
-        proofs = sorted((_get_loki_dir() / "proofs").glob("*/proof.json"))
-        if not proofs:
-            return gates
-        receipt = json.loads(proofs[-1].read_text())
-        checked_at = receipt.get("generated_at") or ""
-        by_display = {}
-        for entry in receipt.get("quality_gates", {}).get("gates", []) or []:
-            display = _RECEIPT_GATE_NAMES.get(entry.get("name", ""))
-            status = _RECEIPT_GATE_STATUS.get(str(entry.get("status", "")).lower())
-            if display and status:
-                by_display[display] = status
-        for gate in gates:
-            status = by_display.get(gate.get("name"))
-            if status:
-                gate["status"] = status
-                if checked_at:
-                    gate["last_checked"] = checked_at
-                gate["source"] = "receipt"
-    except (OSError, ValueError, KeyError, TypeError, AttributeError):
-        return [dict(g) for g in _DEFAULT_QUALITY_GATES]
-    return gates
-
-
 _DEFAULT_QUALITY_GATES = [
     {"name": "Static Analysis", "description": "CodeQL, ESLint/Pylint, type-checker findings on the diff", "status": "pending"},
     {"name": "Test Suite", "description": "Project test runner pass/fail (red blocks)", "status": "pending"},
@@ -10373,22 +10221,7 @@ async def get_council_gate():
         except (json.JSONDecodeError, IOError):
             data = {"blocked": False, "gates": _DEFAULT_QUALITY_GATES, "error": "Failed to read gate file"}
     else:
-        # No live gate file. Fall back to the most recent RECEIPT, which already
-        # records per-gate results -- rather than serving eight rows that all
-        # say "Last checked: Never" while .loki/proofs/ holds the real answers.
-        #
-        # That "Never" was not a display bug: /api/council/gate served
-        # _DEFAULT_QUALITY_GATES verbatim, and those entries carry no
-        # last_checked, so the UI honestly rendered "Never" for every gate on a
-        # repo with 9 receipts.
-        #
-        # ONLY gates the receipt actually names are filled in. Measured across
-        # all 9 receipts here, that is 3 of 8 (static_analysis, code_review,
-        # doc_coverage); the other five stay "pending" with no timestamp,
-        # because a gate that never ran must not inherit a neighbour's result.
-        # Filling all eight from a 3-gate receipt is the exact false-green this
-        # codebase exists to prevent.
-        data = {"blocked": False, "gates": _receipt_backed_gates()}
+        data = {"blocked": False, "gates": _DEFAULT_QUALITY_GATES}
 
     # Verified-completion evidence gate (additive).
     if evidence_file.exists():