m3taversal
ff5162d5ba
Some checks failed
CI / lint-and-test (pull_request) Has been cancelled
- What: Unified DOMAIN_AGENT_MAP, VALID_DOMAINS, agent_for_domain(), detect_domain_from_diff(), detect_domain_from_branch() into lib/domains.py. Removed duplicated mappings from evaluate.py and merge.py. VALID_DOMAINS in validate.py now derives from DOMAIN_AGENT_MAP.keys() (single source of truth). - Why: Phase 3 structural refactor. Domain mapping was duplicated across evaluate.py (DOMAIN_AGENT_MAP) and merge.py (agent_domain dict). Adding a domain required editing 3 files; now it requires editing 1. - Connections: evaluate.py uses agent_for_domain() + detect_domain_from_diff(), merge.py uses detect_domain_from_branch(), validate.py uses VALID_DOMAINS. Pentagon-Agent: Ganymede <F99EBFA6-547B-4096-BEEA-1D59C3E4028A>
87 lines
3 KiB
Python
87 lines
3 KiB
Python
"""Domain→agent mapping and domain detection — single source of truth.
|
|
|
|
Extracted from evaluate.py and merge.py (Phase 3 refactor).
|
|
All domain classification logic goes through this module.
|
|
"""
|
|
|
|
import re
|
|
|
|
# Canonical domain→agent mapping. Every domain must have exactly one primary agent.
|
|
DOMAIN_AGENT_MAP: dict[str, str] = {
|
|
"internet-finance": "Rio",
|
|
"entertainment": "Clay",
|
|
"health": "Vida",
|
|
"ai-alignment": "Theseus",
|
|
"space-development": "Astra",
|
|
"mechanisms": "Rio",
|
|
"living-capital": "Rio",
|
|
"living-agents": "Theseus",
|
|
"teleohumanity": "Leo",
|
|
"grand-strategy": "Leo",
|
|
"critical-systems": "Theseus",
|
|
"collective-intelligence": "Theseus",
|
|
"teleological-economics": "Rio",
|
|
"cultural-dynamics": "Clay",
|
|
}
|
|
|
|
# Valid domain names — derived from the map, not maintained separately.
|
|
VALID_DOMAINS: frozenset[str] = frozenset(DOMAIN_AGENT_MAP.keys())
|
|
|
|
# Inverse mapping: agent name (lowercase) → primary domain (for branch detection).
|
|
_AGENT_PRIMARY_DOMAIN: dict[str, str] = {
|
|
"rio": "internet-finance",
|
|
"clay": "entertainment",
|
|
"theseus": "ai-alignment",
|
|
"vida": "health",
|
|
"astra": "space-development",
|
|
"leo": "grand-strategy",
|
|
}
|
|
|
|
|
|
def agent_for_domain(domain: str | None) -> str:
|
|
"""Get the reviewing agent for a domain. Falls back to Leo."""
|
|
if domain is None:
|
|
return "Leo"
|
|
return DOMAIN_AGENT_MAP.get(domain, "Leo")
|
|
|
|
|
|
def detect_domain_from_diff(diff: str) -> str | None:
|
|
"""Detect primary domain from changed file paths in a unified diff.
|
|
|
|
Checks domains/, entities/, core/, foundations/ for domain classification.
|
|
Returns the most-referenced domain, or None if no domain files found.
|
|
"""
|
|
domain_counts: dict[str, int] = {}
|
|
for line in diff.split("\n"):
|
|
if line.startswith("diff --git"):
|
|
# Check domains/ and entities/ (both carry domain info)
|
|
match = re.search(r"(?:domains|entities)/([^/]+)/", line)
|
|
if match:
|
|
d = match.group(1)
|
|
domain_counts[d] = domain_counts.get(d, 0) + 1
|
|
continue
|
|
# Check core/ subdirectories
|
|
match = re.search(r"core/([^/]+)/", line)
|
|
if match:
|
|
d = match.group(1)
|
|
if d in DOMAIN_AGENT_MAP:
|
|
domain_counts[d] = domain_counts.get(d, 0) + 1
|
|
continue
|
|
# Check foundations/ subdirectories
|
|
match = re.search(r"foundations/([^/]+)/", line)
|
|
if match:
|
|
d = match.group(1)
|
|
if d in DOMAIN_AGENT_MAP:
|
|
domain_counts[d] = domain_counts.get(d, 0) + 1
|
|
if domain_counts:
|
|
return max(domain_counts, key=domain_counts.get)
|
|
return None
|
|
|
|
|
|
def detect_domain_from_branch(branch: str) -> str | None:
|
|
"""Extract domain from branch name like 'rio/claims-futarchy' → 'internet-finance'.
|
|
|
|
Uses agent prefix → primary domain mapping for pipeline branches.
|
|
"""
|
|
prefix = branch.split("/")[0].lower() if "/" in branch else ""
|
|
return _AGENT_PRIMARY_DOMAIN.get(prefix)
|