Skip to content

fix(tactician): align deterministic rule metadata#26

Open
Fartis wants to merge 8 commits into
developfrom
feature/sprint12-2-judge-tool-gateway
Open

fix(tactician): align deterministic rule metadata#26
Fartis wants to merge 8 commits into
developfrom
feature/sprint12-2-judge-tool-gateway

Conversation

@Fartis

@Fartis Fartis commented Jul 17, 2026

Copy link
Copy Markdown
Owner

Summary

  • align deterministic Judge answers with Tactician completeness, confidence, and evidence-verification metadata
  • replace brittle exact-phrase checks with reusable basic-land-type and mana-result concepts
  • support both a and an in Oracle patterns such as Each land is an Island...
  • add a fictional-card regression using different land types to prove the renderer is Oracle-pattern-driven rather than card-name-driven
  • document Sprint 12.3a2 as the closure of the land-type investigation hardening line

Why

The Blood Moon / Urborg / Dryad answer was factually correct and source-grounded, but the public metadata still reported answer_complete: false, judge_verified: false, low confidence, and judge:evidence_incomplete. The contract checker depended on exact sentence fragments and pluralization even though the deterministic renderer had satisfied every requested obligation.

Validation

  • python -m pytest -q tests — 320 passed
  • focused land-type regressions — 7 passed
  • conversational gauntlet — 40/40 scenarios, 58/58 turns
  • repository health — 5/5
  • release packaging — 1/1
  • all focused CI modules passed; executed in chunks because the local sequential runner exceeded the sandbox command limit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant