…ircuit + state-holders

Four sub-plugin extractions in one PR, following the PR #185
KR-PLUGIN-COST-LADDER template (constants + <logic> + plugin
file shape; backward-compat shim with identity-against-canonical
drift guard).

# Canonical locations created

  kora_cli/reasoning/kora_hermes_plugin/
    audit/         — Deliverable A; _emit_tool_called_audit
                      writer (moved from anthropic_engine);
                      post_tool_call + post_llm_call handlers
    caching/       — Deliverable B; _wrap_system/tools_as_cacheable
                      markers (moved from anthropic_engine);
                      standalone caching_hook (not registered;
                      bundled with cost-ladder hook in v1)
    short_circuit/ — Deliverable C; PhrasebookEntry +
                      ShortCircuitMatch + load_phrasebook +
                      match_message + render_reply +
                      try_short_circuit (moved from
                      kora_cli/short_circuit/dm_phrasebook.py)
    state_holders/ — Deliverable D; on_session_start handler +
                      STATE_HOLDER_ACCESSORS registry +
                      holder_liveness probe

# Backward-compat shims

  kora_cli/reasoning/anthropic_engine.py
    Re-imports _emit_tool_called_audit (Deliverable A) +
    _wrap_system/tools_as_cacheable (Deliverable B) from their
    canonical sub-plugin locations. The 3 in-file callers of
    the audit helper + the engine's _make_request_kwargs caching
    wrap keep resolving the symbols.

  kora_cli/short_circuit/dm_phrasebook.py (333 → 62 lines)
    Re-exports the full public surface from the canonical
    matcher module. slack_dm_handler + web_server + phrasebook
    editor (PR #177) keep working unchanged. Bundled YAML
    phrasebook stays at default_slack_dm_phrasebook.yml in this
    package — phrasebook editor reads/writes it through
    importlib.resources.files("kora_cli.short_circuit").

# Cross-dep cleanup (Deliverable B's stated value)

The cost-ladder plugin's hook handler (added in PR #185)
previously imported the caching wrap helpers from
kora_cli.reasoning.anthropic_engine. This PR cuts that
cross-dep — cost_ladder/plugin.py now imports
_wrap_system/tools_as_cacheable from
kora_hermes_plugin.caching.markers (the canonical sub-plugin
location). Verified by grep: zero remaining anthropic_engine
imports in cost_ladder/plugin.py.

# Orchestrator topology (post-batch)

  KoraHermesPlugin.register walks 5 sub-registers in order
  (cost_ladder, audit, caching, short_circuit, state_holders)
  then attaches 3 remaining orchestrator-resident handlers
  (pre_tool_list_finalized + pre_tool_call + the ST2B tool
  bridge). 7 total hooks wired against KORA_ROUTES.

# Tests

  tests/kora_cli/reasoning/kora_hermes_plugin/audit/test_writer.py
    7 tests: identity-against-canonical (engine shim,
    orchestrator re-export, discovery shim re-export) + writer
    end-to-end JSONL emit + dual-write log preservation +
    register-wires-2-hooks + handler-noop-on-non-Kora-routes.

  tests/kora_cli/reasoning/kora_hermes_plugin/caching/test_markers.py
    8 tests: engine shim identity + cost-ladder hook proves it
    imports from canonical location (via patched-canonical
    sentinel) + marker shape + last-tool-only + empty input +
    register-is-noop + standalone caching_hook end-to-end.

  tests/kora_cli/reasoning/kora_hermes_plugin/short_circuit/test_matcher.py
    Moved 1:1 from tests/kora_cli/short_circuit/test_dm_
    phrasebook.py with import path updated to canonical.

  tests/kora_cli/reasoning/kora_hermes_plugin/state_holders/test_plugin.py
    7 tests: orchestrator + discovery shim identity + handler
    no-op gate + register-wires-on-session-start + accessors-
    registry-shape + holder_liveness-returns-bool-per-accessor.

  tests/kora_cli/short_circuit/test_dm_phrasebook.py
    Converted to 3-test shim-verification suite (identity-
    against-canonical for public + private re-exports +
    package-level resolves + canonical-module attribution).

  tests/kora_cli/test_phrasebook_endpoints.py
    test_placeholder_regex_matches_dm_phrasebook_source updated
    to source-grep the canonical matcher.py location (the
    regex moved with the module; the drift-guard intent
    preserved).

# Scoped regression (xdist)

  tests/kora_cli/reasoning/kora_hermes_plugin/   passing
  tests/kora_cli/short_circuit/                  passing
  tests/kora_cli/reasoning/test_anthropic_engine_caching.py  passing
  tests/kora_cli/audit/test_jsonl_sink.py        passing
  tests/plugins/test_kora_hermes_plugin*.py      passing
  tests/kora_cli/handlers/test_slack_dm_short_circuit.py  passing
  tests/kora_cli/test_phrasebook_{editor,endpoints}.py  passing
  tests/kora_cli/reasoning/test_anthropic_engine_router.py  passing
  tests/kora_cli/snapshot/test_state_snapshot.py passing
  Total directly-affected surface: 337/337 passing.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>