docs(spec): align checkpoint trigger names and config key path with implementation #5163

Merged
HAL9000 merged 3 commits from spec/checkpoint-trigger-names-and-config-key-fix into master 2026-06-15 17:46:19 +00:00

3 Commits

Author SHA1 Message Date
controller-ci-rerun 0d846a335c chore: re-trigger CI [controller]
CI / load-versions (pull_request) Successful in 28s
CI / push-validation (pull_request) Successful in 25s
CI / lint (pull_request) Successful in 1m18s
CI / typecheck (pull_request) Successful in 1m18s
CI / security (pull_request) Successful in 1m18s
CI / quality (pull_request) Successful in 1m18s
CI / build (pull_request) Successful in 32s
CI / helm (pull_request) Successful in 41s
CI / unit_tests (pull_request) Successful in 5m1s
CI / docker (pull_request) Successful in 1m30s
CI / integration_tests (pull_request) Successful in 8m28s
CI / coverage (pull_request) Successful in 9m23s
CI / status-check (pull_request) Successful in 3s
2026-06-15 13:14:25 -04:00
HAL9000 f85ecd0feb docs(changelog): add entry for checkpoint config key spec fix
ISSUES CLOSED: #5009
2026-06-15 13:14:25 -04:00
HAL9000 4a68e02107 docs(spec): align checkpoint trigger names and config key path with implementation
Fixes two spec-implementation discrepancies identified in issue #5009:

1. Checkpoint trigger names: rename on_tool_write → before_tool_execute and
   on_tool_write_complete → after_tool_execute to match the implementation in
   src/cleveragents/tool/runner.py and src/cleveragents/application/services/
   config_service.py. The implementation names are more precise — they describe
   the execution phase rather than implying only write tools trigger them
   (though the implementation correctly gates them on write tools).

2. Config key path: correct the Configuration Reference table entry from
   sandbox.checkpoint.auto-create-on to core.checkpoints.auto-create-on,
   matching both the implementation (config_service.py line 482) and the
   inline spec at line 19449. Also update the default values in the table
   to use the corrected trigger names.

Closes #5009
2026-06-15 13:14:25 -04:00