fix(profiles): a Codex apply_patch counts as an edit in the style observation (#727, slice 1) - #882

Summary

First slice of #727. The incremental style observation (core/profile_builder._build_style_observation) recognized Claude tool names only (Edit/Write as edits, Read/Grep as reads). A Codex rollout records its edits as apply_patch (infrastructure/transcript_rollout.py), so the issue's replay holds: a one-tool session with duration=None gave activeReflective=0.3 for Edit and an empty observation for apply_patch.

Root cause: the operation an observation reads was inferred from a Claude tool name, not from a host-independent operation. Fix: new pure module core/tool_operation.py maps each native name to the operation it makes explicit (apply_patch → edit, as hooks/host_patch.py already does on the hook path). Native names stay as recorded in toolsUsed (provenance). exec (an arbitrary program) stays unclassified (None): it is not guessed into a read.

The function's final clamp to [-1, 1] is removed: activeReflective is one of {0.5, 0.0, -0.5} plus at most +0.3 and -0.3, so it stays within [-0.8, 0.8]. The clamp never acted, and its 4 mutants were equivalent; update_style_ema clamps the blended value anyway.

Refs #727

Remaining slices (not in this PR): persona_vector (reads toolPreferences keyed by native name), sparse_dictionary_activation and style_classifier (full rebuild path), with a cross-client test on transcript-derived EMA, persona and feature activation.

Type of change

• Bug fix (non-breaking change which fixes an issue)

Test plan

• pytest tests_py/core/test_tool_operation.py tests_py/core/test_profile_builder.py: 31 passed. The new test test_apply_patch_session_observation_equals_edit_session fails on main (checked with the fix stashed) and passes here.
• pytest tests_py/core/: 3729 passed, 1 failed (test_wiki_coverage_dashboard.py::test_unreadable_page_is_skipped_not_fatal). It fails the same way on main in this container, which runs as root, so a chmod'd file stays readable.
• Diff-scoped mutation gate (ADR-1122), GITHUB_BASE_REF=main python scripts/ci_mutation_diff.py: 2 file(s), 0 failed, 0 untested (_build_style_observation: 0 survivors in scope; tool_operation.py: 5/5 killed).
• ruff check / ruff format --check on the changed files: clean. python scripts/check_craftsmanship.py: OK.
• Not verified here: the full suite with PostgreSQL and the torch-dependent extras (the container cannot reach download-r2.pytorch.org). No retrieval benchmark applies: no retrieval or memory path changed.

Audit notes

• No-source check: the edit/read groups are the ones ADR-0228 already used; apply_patch → edit follows hooks/host_patch.py; exec is left unknown, as the issue requires.

Coding-standards compliance

• §2.2 Layer dependency direction preserved (core/tool_operation.py imports nothing).
• §4.1 / §4.2 within limits (craftsmanship gate OK).
• §9 No dead code (the unreachable clamp is removed).

Reviewer checklist

• CHANGELOG.md updated under Unreleased / Fixed.
• Documentation updated (docs/module-inventory.md).
• No secrets / credentials / PII in the diff.

Generated by Claude Code