Six contained-tier gap closures from the cross-phase gap audit.
Every gaps.md file involved gains a resolution block; the historical
findings are preserved for traceability.
Identity (teaching/review.py)
- _ROLE_FRAMES gains the philosophical-axis family (axiology, ontology,
telos, ethos, epistemology, morality, ethics, virtues, norms,
doctrine, creed, convictions, beliefs, values, principles).
- _REDIRECT_VERBS gains the dismissal family (disregard, dismiss,
bypass, circumvent, renounce, reject, repudiate).
- Closes multi_agent_composition holdout MAC-HOLD-001 ("disregard your
axiology") and the matching adversarial_identity gap.
- Multi-agent holdouts: 8/8 attacks rejected, 3/3 legits accepted.
Pipeline (core/cognition/pipeline.py + docs/runtime_contracts.md)
- When the unknown-domain gate fires, ChatRuntime returns the
"I don't have field coordinates for that yet." stub and
vault_hits == 0. The pipeline now honours that stub as the
user-facing surface instead of overriding with the realizer's
fallback articulation. walk_surface is unchanged either way.
- New contract test
tests/test_semantic_realizer_integration.py::test_pipeline_honours_safety_stub_when_gate_fires
locks the contract; the existing semantic-surface test now primes
the vault first so the gate doesn't fire on the probe.
- Closes calibration gaps.md Finding 2.
Realizer morphology (generate/morphology.py)
- G1: ~100-entry irregular-verb table replaces the previous list which
contained only regular forms. Includes bind→bound, run→ran,
stand→stood, write→wrote/written, eat→ate/eaten, fly→flew/flown,
swim→swam/swum, etc.
- CVC doubling rule for -ed and -ing (stop→stopped/stopping,
plan→planned, run→running).
- Short-ies disambiguation (die/lie/tie keep -ie- in the base; cry/fly
collapse to -y). Lie is also irregular (lay/lain) — uses
_IRREGULAR_FORMS first.
- 28-case regression test (tests/test_morphology_irregular.py).
Realizer plural agreement (generate/templates.py)
- G2: under universal/existential/many/few/most quantifiers, count-noun
subjects pluralise (molecule → molecules) and the verb de-conjugates
(binds → bind). Negation toggles does-not → do-not. Aspect toggles
has → have, is → are. All other constructions unchanged.
- Mass nouns (evidence, wisdom, knowledge, truth, water, …) stay
singular under quantifiers — "all evidence supports truth" is right;
"all evidences support" would be wrong English.
- 17-case regression test
(tests/test_realizer_quantifier_agreement.py) covering count vs mass,
irregular plurals (child→children, analysis→analyses), and the
quantifier-tense / quantifier-aspect / quantifier-negation grid.
Rubric punctuation tolerance (evals/grammatical_coverage/runner.py)
- G3: _check_word_order strips trailing/leading punctuation
(.,;:!?—–) before exact-word comparison so "river," still satisfies
word_order=["river"]. must_contain also accepts punctuation-
stripped token matches.
- Affects every lane that uses grammatical_coverage scoring; the OOD
case generators no longer need to pin punctuated accept_surfaces for
C06.
Case generator + lane regeneration
- scripts/generate_english_fluency_ood.py uses generate.templates.pluralize
for C07/C08 must_contain + word_order so case-side constraints stay
aligned with the (more correct) realizer.
- All Phase 5 OOD lane cases (5.1, 5.4–5.7) regenerated; results files
re-scored.
CLI (core/cli.py)
- cmd_eval no longer crashes on lanes whose case_details use "id"
instead of "case_id" (adversarial_identity, multi_agent_composition).
- Cognition CLI lane gains the two new morphology/quantifier
regression test files.
Lane sweep (all 100%, no regression):
english_fluency_ood 117/117 public + 39/39 holdouts
elementary_mathematics_ood 117/117 + 39/39
foundational_physics_ood 117/117 + 39/39
foundational_biology_ood 117/117 + 39/39
classical_literature_ood 117/117 + 39/39
grammatical_coverage back to 100% on its own seed cases
hebrew_fluency / koine_greek_fluency 3/3 each
CLI lane health:
smoke 54, runtime 19, teaching 17, packs 6, cognition 103 (was 57),
algebra 132.
245 lines
7.7 KiB
Python
245 lines
7.7 KiB
Python
"""Grammatical-coverage eval lane runner.
|
|
|
|
Scores the deterministic realizer on its ability to produce grammatical
|
|
English surfaces from PropositionGraph inputs. Each case specifies a
|
|
construction family (e.g. negation, conjunction, embedded clause) and
|
|
acceptance criteria (exact surfaces, constraint checks).
|
|
|
|
Conforms to the framework interface: ``run_lane(cases, config=None) -> report``.
|
|
"""
|
|
from __future__ import annotations
|
|
|
|
from dataclasses import dataclass, field
|
|
from typing import Any
|
|
|
|
|
|
@dataclass(frozen=True, slots=True)
|
|
class CaseResult:
|
|
case_id: str
|
|
construction: str
|
|
construction_name: str
|
|
passed: bool
|
|
surface: str
|
|
failure_reasons: tuple[str, ...]
|
|
|
|
|
|
@dataclass(slots=True)
|
|
class LaneReport:
|
|
metrics: dict[str, Any] = field(default_factory=dict)
|
|
case_details: list[dict[str, Any]] = field(default_factory=list)
|
|
|
|
|
|
_PUNCT_STRIP = ".,;:!?—–" # period, comma, semicolon, colon, !, ?, em-dash, en-dash
|
|
|
|
|
|
def _strip_punct(word: str) -> str:
|
|
return word.strip(_PUNCT_STRIP)
|
|
|
|
|
|
def _check_word_order(order: list[str], surface_words: list[str]) -> bool:
|
|
"""Match `order` against `surface_words` as a subsequence, ignoring
|
|
trailing/leading punctuation on surface tokens.
|
|
|
|
Closes english_fluency_ood gaps.md G3: previously
|
|
`"river,"` failed to match `"river"` because the rubric did
|
|
exact-word comparison. Stripping common terminal punctuation
|
|
makes the rubric tolerant to comma-bounded relative clauses and
|
|
sentence-final periods without weakening the structural ordering
|
|
check.
|
|
"""
|
|
positions = []
|
|
for word in order:
|
|
found = False
|
|
start = positions[-1] + 1 if positions else 0
|
|
target = word.lower()
|
|
for i in range(start, len(surface_words)):
|
|
if _strip_punct(surface_words[i]).lower() == target:
|
|
positions.append(i)
|
|
found = True
|
|
break
|
|
if not found:
|
|
return False
|
|
return True
|
|
|
|
|
|
def _realize_from_graph(case: dict[str, Any]) -> str:
|
|
"""Realize a surface from a proposition graph case.
|
|
|
|
This calls the actual realizer infrastructure. The graph format in
|
|
the eval cases maps to the realizer's PropositionGraph -> surface path.
|
|
"""
|
|
from generate.graph_planner import (
|
|
ArticulationStep,
|
|
ArticulationTarget,
|
|
GraphEdge,
|
|
GraphNode,
|
|
PropositionGraph,
|
|
Relation,
|
|
RhetoricalMove,
|
|
)
|
|
from generate.intent import IntentTag
|
|
from generate.realizer import realize_target
|
|
|
|
graph_data = case["proposition_graph"]
|
|
nodes_data = graph_data["nodes"]
|
|
edges_data = graph_data.get("edges", [])
|
|
|
|
_RELATION_MAP = {
|
|
"conjunction": Relation.CONJUNCTION,
|
|
"disjunction": Relation.DISJUNCTION,
|
|
"complement": Relation.COMPLEMENT,
|
|
"relative": Relation.RELATIVE,
|
|
"sequence": Relation.SEQUENCE,
|
|
"cause": Relation.CAUSE,
|
|
"contrast": Relation.CONTRAST,
|
|
"elaboration": Relation.ELABORATION,
|
|
"correction": Relation.CORRECTION,
|
|
}
|
|
|
|
nodes = []
|
|
for nd in nodes_data:
|
|
nodes.append(GraphNode(
|
|
node_id=nd["node_id"],
|
|
subject=nd["subject"],
|
|
predicate=nd["predicate"],
|
|
obj=nd["obj"],
|
|
source_intent=IntentTag.UNKNOWN,
|
|
))
|
|
|
|
edges = []
|
|
for e in edges_data:
|
|
rel_str = e.get("relation", "sequence")
|
|
edges.append(GraphEdge(
|
|
source=e["source"],
|
|
target=e["target"],
|
|
relation=_RELATION_MAP.get(rel_str, Relation.SEQUENCE),
|
|
))
|
|
|
|
graph = PropositionGraph(nodes=tuple(nodes), edges=tuple(edges))
|
|
|
|
node_features = {nd["node_id"]: nd for nd in nodes_data}
|
|
steps = []
|
|
for node in nodes:
|
|
nd = node_features[node.node_id]
|
|
steps.append(ArticulationStep(
|
|
node_id=node.node_id,
|
|
subject=node.subject,
|
|
predicate=node.predicate,
|
|
move=RhetoricalMove.ASSERT,
|
|
negated=nd.get("negated", False),
|
|
quantifier=nd.get("quantifier"),
|
|
tense=nd.get("tense"),
|
|
aspect=nd.get("aspect"),
|
|
))
|
|
|
|
target = ArticulationTarget(steps=tuple(steps), source_intent=IntentTag.UNKNOWN)
|
|
plan = realize_target(target, graph)
|
|
surface = plan.surface.rstrip(".")
|
|
return surface
|
|
|
|
|
|
def _score_case(case: dict[str, Any]) -> CaseResult:
|
|
construction = case["construction"]
|
|
construction_name = case["construction_name"]
|
|
|
|
try:
|
|
surface = _realize_from_graph(case)
|
|
except Exception as exc:
|
|
return CaseResult(
|
|
case_id=case["id"],
|
|
construction=construction,
|
|
construction_name=construction_name,
|
|
passed=False,
|
|
surface=f"ERROR: {exc}",
|
|
failure_reasons=(f"realizer error: {exc}",),
|
|
)
|
|
|
|
accept = case.get("accept_surfaces", [])
|
|
constraints = case.get("constraints", {})
|
|
failures: list[str] = []
|
|
|
|
surface_lower = surface.lower().strip()
|
|
exact_match = any(s.lower().strip() == surface_lower for s in accept)
|
|
|
|
if not exact_match and constraints:
|
|
surface_words = surface_lower.split()
|
|
|
|
# Punctuation-tolerant token-level membership check (G3) — strip
|
|
# trailing/leading punctuation so "river," still satisfies
|
|
# `must_contain: ["river"]`.
|
|
surface_tokens_stripped = {_strip_punct(w).lower() for w in surface_words}
|
|
must_contain = constraints.get("must_contain", [])
|
|
for word in must_contain:
|
|
w = word.lower()
|
|
if w not in surface_lower and w not in surface_tokens_stripped:
|
|
failures.append(f"missing required word: {word}")
|
|
|
|
word_order = constraints.get("word_order", [])
|
|
if word_order and not _check_word_order(word_order, surface_words):
|
|
failures.append(f"word order violated: expected {word_order}")
|
|
|
|
max_words = constraints.get("max_words")
|
|
if max_words is not None and len(surface_words) > max_words:
|
|
failures.append(f"too many words: {len(surface_words)} > {max_words}")
|
|
|
|
reject = case.get("reject_surfaces", [])
|
|
if any(s.lower().strip() == surface_lower for s in reject):
|
|
failures.append("surface matched a reject pattern")
|
|
|
|
passed = exact_match or (not failures and bool(constraints))
|
|
|
|
return CaseResult(
|
|
case_id=case["id"],
|
|
construction=construction,
|
|
construction_name=construction_name,
|
|
passed=passed,
|
|
surface=surface,
|
|
failure_reasons=tuple(failures),
|
|
)
|
|
|
|
|
|
def run_lane(
|
|
cases: list[dict[str, Any]],
|
|
*,
|
|
config: Any = None,
|
|
) -> LaneReport:
|
|
total = 0
|
|
passed = 0
|
|
by_construction: dict[str, dict[str, int]] = {}
|
|
case_details: list[dict[str, Any]] = []
|
|
|
|
for case in cases:
|
|
cr = _score_case(case)
|
|
total += 1
|
|
if cr.passed:
|
|
passed += 1
|
|
|
|
key = cr.construction
|
|
if key not in by_construction:
|
|
by_construction[key] = {"total": 0, "passed": 0}
|
|
by_construction[key]["total"] += 1
|
|
if cr.passed:
|
|
by_construction[key]["passed"] += 1
|
|
|
|
case_details.append({
|
|
"case_id": cr.case_id,
|
|
"construction": cr.construction,
|
|
"construction_name": cr.construction_name,
|
|
"passed": cr.passed,
|
|
"surface": cr.surface,
|
|
"failure_reasons": list(cr.failure_reasons),
|
|
})
|
|
|
|
construction_scores = {
|
|
k: round(v["passed"] / v["total"], 4) if v["total"] else 0.0
|
|
for k, v in sorted(by_construction.items())
|
|
}
|
|
|
|
metrics = {
|
|
"total": total,
|
|
"passed": passed,
|
|
"accuracy": round(passed / total, 4) if total else 0.0,
|
|
"by_construction": construction_scores,
|
|
}
|
|
|
|
return LaneReport(metrics=metrics, case_details=case_details)
|