fix(wiki): seed-codebase emits modern kind tags the classifier reads (ADR-2244 Phase 6.2)

[cdeust]

Summary

Producer audit follow-up — closes the second leak flagged out-of-scope in #38. wiki_seed_codebase was emitting kind hints in a tag shape (kind:<value>) that the classifier never read; the kind hint flowed nowhere.

What was wrong

wiki_seed_codebase ingests markdown files (README, ADR, spec, convention, lesson) from a repo into Cortex memory. The _kind_for(rel_path) helper inferred a kind from the path, then the call-site wrote it as a kind:<value> tag along with seed:codebase and file:<rel>.

Two problems:

1. Legacy kind names: _kind_for returned spec / convention / lesson / note — none of these match any modern kind tag alias in wiki_axis_registry._DEFAULT_KINDS.
2. Wrong tag shape: even if _kind_for had returned modern names, kind:adr is a different string than adr. The classifier's tag_aliases set intersection requires exact match.

Net: every seed-imported markdown page routed via legacy → modern fallback to kind=explanation, ignoring the path hint that said "this is an ADR" or "this is architecture".

What changed

_kind_for returns modern kinds

Path pattern	Before	After (modern)
ADR / decision	adr	adr
architecture	spec	rfc
convention / style	convention	explanation
lesson / postmortem	lesson	explanation
README / default	note	explanation

The new values are themselves registered tag aliases (or kind names) in _DEFAULT_KINDS, so they flow through the classifier.

Call-site tag shape

# Before
"tags": ["seed:codebase", f"kind:{kind}", f"file:{rel}"]

# After
"tags": ["seed:codebase", "imported", kind, f"file:{rel}"]

The bare kind tag (e.g. adr, rfc, explanation) is now a registered alias. imported tag flips provenance to imported — correct for bulk-imported markdown.

Tests

tests_py/handlers/test_wiki_seed_codebase.py (NEW) — 8 tests:

Test	Surface
test_adr_path_routes_to_adr	path → kind
test_decision_path_routes_to_adr	path → kind
test_architecture_path_routes_to_rfc	path → kind
test_convention_path_routes_to_explanation	path → kind
test_lesson_path_routes_to_explanation	path → kind
test_readme_routes_to_explanation	path → kind
test_unknown_path_defaults_to_explanation	path → kind
test_all_returned_kinds_are_registered	contract pin: every returned value must be a registered tag alias

The contract-pin test is the load-bearing one: if a future refactor returns a value that isn't a registered alias, the kind hint silently flows nowhere again. The test catches that class of bug.

Test plan

• pytest tests_py/handlers/test_wiki_seed_codebase.py — 8 passed
• ruff format --check and ruff check clean
• CI on this PR

Out of scope

• End-to-end test through remember() → wiki_sync → on-disk page. The seed handler delegates to remember() which kicks off the broader pipeline; an integration test would need a full DB fixture. The unit test on the tag-shape invariant catches the producer bug; downstream behavior is covered by existing wiki_sync tests.

🤖 Generated with Claude Code