core/teaching/discovery.py
Shay 07d35c0f54 feat(adr-0055): Phase B — DiscoveryCandidate emission from turn loop
Lands the first deterministic trigger of the discovery → reviewed-
memory loop. Candidates are structured evidence; emission is
opt-in via attach_discovery_sink and NEVER mutates the active
teaching corpus.

- teaching/discovery.py: DiscoveryCandidate dataclass + pure
  extract_discovery_candidates(turn_event, intent, subject) rule
  firing. Phase B fires only the would_have_grounded trigger:
    grounding_source == "none"
    AND intent ∈ {CAUSE, VERIFICATION}
    AND subject lemma in ratified cognition pack
    AND (subject, intent) NOT in active corpus
  candidate_id = SHA-256 of canonical JSON payload — replay-stable.
  Other DiscoveryTrigger literals (successful_comparison,
  hedge_acknowledged, oov_resolved_via_decomp) are reserved for
  later phases.

- teaching/discovery_sink.py: DiscoveryCandidateSink protocol,
  DiscoveryBufferSink (in-memory), DiscoveryMonthlyFileSink
  (append-only JSONL, <root>/<YYYY>/<YYYY-MM>.jsonl rollover,
  injectable clock).

- chat/runtime.py: opt-in attach_discovery_sink, post-turn
  emission inside _stub_response only when caller threads
  classified intent forward (gate-fire fall-through site).
  Intent classification at the call site reuses the same
  deterministic classifier already invoked by
  _maybe_pack_grounded_surface for the empty-vault English path.

Trust boundary: candidates write to a separate sink/file path
only; the active corpus on disk is never touched. Tests
explicitly assert corpus bytes are byte-identical before and
after a candidate-emitting turn.

Tests: tests/test_discovery_candidates.py — 24 tests covering
pure-predicate rule firing, every short-circuit path,
deterministic candidate_id, sink opt-in, runtime parity with no
sink, monthly rollover semantics, append-only behaviour, no
corpus mutation.

Lanes: smoke 67, cognition 121, runtime 19, teaching 17, packs 6
— all green. Cognition eval metrics unchanged on dev / public /
holdout splits. versor_condition < 1e-6 invariant untouched.
2026-05-18 08:26:04 -07:00

232 lines
7.9 KiB
Python

"""ADR-0055 Phase B — DiscoveryCandidate emission from the turn loop.
A ``DiscoveryCandidate`` is **structured evidence** — never a
mutation. When a turn's audit trail satisfies a deterministic
predicate, an entry is emitted to the discovery candidate stream.
Candidates **never** load into the active teaching corpus; the only
path to corpus extension is the review-gated
``TeachingChainProposal`` (Phase C, not yet built).
Trigger set (Phase B lands the first; the others are stubbed in the
``Literal`` so the structure is stable when later phases add them):
- ``would_have_grounded`` — the turn fell through to the universal
"insufficient grounding" disclosure, the classified intent was
``CAUSE`` or ``VERIFICATION``, the subject lemma is in the
ratified cognition pack, and no active chain matched
``(subject, intent)``. A reviewed chain of that subject/intent
would have grounded the turn.
- ``successful_comparison`` — open question §5 in ADR-0055; not
fired in Phase B.
- ``hedge_acknowledged`` — open question §5 in ADR-0055; not
fired in Phase B.
- ``oov_resolved_via_decomp`` — not fired in Phase B.
Determinism contract:
- ``extract_discovery_candidates`` is a pure function of its
inputs.
- ``candidate_id`` is a SHA-256 hash of a canonical JSON encoding
of the candidate's load-bearing fields; identical inputs always
produce the identical id.
- No LLM, no stochastic sampling, no clock-time read.
Trust boundary:
- This module reads pack + corpus indices and a ``TurnEvent``.
It never writes to the corpus, the pack, or runtime state.
- The ``source_turn_trace`` is the upstream ``TurnEvent.trace_hash``
when present; absent that, the empty string. Tying every
candidate to a replayable turn is the load-bearing audit
property.
"""
from __future__ import annotations
import hashlib
import json
from dataclasses import dataclass
from typing import Any, Literal
from chat.pack_grounding import _pack_index
from chat.teaching_grounding import _corpus_index
from generate.intent import IntentTag
DiscoveryTrigger = Literal[
"would_have_grounded",
"successful_comparison",
"hedge_acknowledged",
"oov_resolved_via_decomp",
]
@dataclass(frozen=True, slots=True)
class DiscoveryCandidate:
"""Structured evidence that a reviewed chain would have helped.
``proposed_chain`` is *partial* by design: Phase B can only see
that a chain of a given ``(subject, intent)`` would have grounded
the turn — it cannot infer the connective or object. Phase C's
``TeachingChainProposal`` is the place where a complete proposed
entry is constructed and gated through review + replay.
"""
candidate_id: str
proposed_chain: dict[str, Any]
trigger: DiscoveryTrigger
source_turn_trace: str
pack_consistent: bool
boundary_clean: bool
review_state: Literal["unreviewed"] = "unreviewed"
def as_dict(self) -> dict[str, Any]:
return {
"candidate_id": self.candidate_id,
"proposed_chain": self.proposed_chain,
"trigger": self.trigger,
"source_turn_trace": self.source_turn_trace,
"pack_consistent": self.pack_consistent,
"boundary_clean": self.boundary_clean,
"review_state": self.review_state,
}
_TEACHING_INTENT_NAME: dict[IntentTag, str] = {
IntentTag.CAUSE: "cause",
IntentTag.VERIFICATION: "verification",
}
def _hash_candidate_id(payload: dict[str, Any]) -> str:
"""Deterministic SHA-256 over a canonical JSON encoding.
Sorted keys + tight separators keep the hash stable across
Python runtimes and dict-insertion order. This is the
``candidate_id`` — used both as the on-disk JSONL line key and
by Phase C to look up the originating candidate.
"""
blob = json.dumps(payload, sort_keys=True, separators=(",", ":"))
return hashlib.sha256(blob.encode("utf-8")).hexdigest()
def _boundary_clean(turn_event: Any) -> bool:
"""Return True iff the source turn produced no safety/ethics
refusal and no hedge injection.
Tolerates events that pre-date the bundled-verdicts era (ADR-0039
onward) by reading the canonical fields directly.
"""
refusal_emitted = bool(getattr(turn_event, "refusal_emitted", False) or False)
hedge_injected = bool(getattr(turn_event, "hedge_injected", False) or False)
if refusal_emitted or hedge_injected:
return False
verdicts = getattr(turn_event, "verdicts", None)
if verdicts is not None:
if bool(getattr(verdicts, "refusal_emitted", False) or False):
return False
if bool(getattr(verdicts, "hedge_injected", False) or False):
return False
return True
def _trace_hash(turn_event: Any) -> str:
value = getattr(turn_event, "trace_hash", "") or ""
return str(value)
def extract_discovery_candidates(
turn_event: Any,
intent_tag: IntentTag | None,
intent_subject_lemma: str | None,
*,
grounding_source: str | None = None,
) -> tuple[DiscoveryCandidate, ...]:
"""Return zero or more DiscoveryCandidates for a single turn.
Phase B only fires the ``would_have_grounded`` trigger. All
other triggers in the ``DiscoveryTrigger`` Literal are reserved
for later phases.
Fires when **every** condition holds (deterministic predicate):
1. ``grounding_source`` is ``"none"`` or absent — the turn
fell through to the universal disclosure.
2. ``intent_tag`` is ``CAUSE`` or ``VERIFICATION`` — the
intent set the teaching-grounded surface answers.
3. ``intent_subject_lemma`` is a non-empty pack lemma in the
ratified cognition pack.
4. ``(subject_lemma, intent_name)`` is **not** in the active
corpus — a chain of that shape would have grounded the
turn but does not exist.
Order of conditions matters for tests: short-circuit on the
cheapest predicate first.
"""
source = (grounding_source or getattr(turn_event, "grounding_source", "none") or "none").lower()
if source != "none":
return ()
if intent_tag is None or intent_tag not in _TEACHING_INTENT_NAME:
return ()
if not intent_subject_lemma or not isinstance(intent_subject_lemma, str):
return ()
lemma = intent_subject_lemma.strip().lower()
if not lemma:
return ()
pack = _pack_index()
if lemma not in pack:
return ()
intent_name = _TEACHING_INTENT_NAME[intent_tag]
if (lemma, intent_name) in _corpus_index():
return ()
# The candidate's proposed_chain is intentionally partial: Phase B
# can only assert that a chain of this (subject, intent) would
# have helped. Connective and object remain null; Phase C is
# where a complete proposed entry is constructed and review-gated.
proposed_chain = {
"subject": lemma,
"intent": intent_name,
"connective": None,
"object": None,
}
trace_hash = _trace_hash(turn_event)
boundary_clean = _boundary_clean(turn_event)
trigger: DiscoveryTrigger = "would_have_grounded"
hash_payload = {
"proposed_chain": proposed_chain,
"trigger": trigger,
"source_turn_trace": trace_hash,
}
candidate_id = _hash_candidate_id(hash_payload)
candidate = DiscoveryCandidate(
candidate_id=candidate_id,
proposed_chain=proposed_chain,
trigger=trigger,
source_turn_trace=trace_hash,
pack_consistent=True, # subject is in pack; object is null pending Phase C
boundary_clean=boundary_clean,
review_state="unreviewed",
)
return (candidate,)
def format_candidate_jsonl(candidate: DiscoveryCandidate) -> str:
"""Serialise to one JSONL line (sorted keys, no trailing newline)."""
return json.dumps(candidate.as_dict(), sort_keys=True, separators=(",", ":"))
__all__ = [
"DiscoveryCandidate",
"DiscoveryTrigger",
"extract_discovery_candidates",
"format_candidate_jsonl",
]