feat(plans): implement checkpoint creation for plan state snapshots #8738

Closed
HAL9000 wants to merge 4 commits from feat/v3.3.0-checkpoint-creation into master

4 Commits

Author SHA1 Message Date
CleverAgents Bot 5ee8ae51ad ci: stop master workflow on PR updates
CI / lint (pull_request) Has been cancelled
CI / typecheck (pull_request) Has been cancelled
CI / security (pull_request) Has been cancelled
CI / quality (pull_request) Has been cancelled
CI / unit_tests (pull_request) Has been cancelled
CI / integration_tests (pull_request) Has been cancelled
CI / e2e_tests (pull_request) Has been cancelled
CI / coverage (pull_request) Has been cancelled
CI / build (pull_request) Has been cancelled
CI / docker (pull_request) Has been cancelled
CI / helm (pull_request) Has been cancelled
CI / push-validation (pull_request) Has been cancelled
CI / status-check (pull_request) Has been cancelled
Remove the stale pull_request trigger from master.yml so PR branch commits do not launch the master workflow.

Maintenance patch for PR #8738.
2026-06-10 20:23:54 -04:00
HAL9000 f4d31e8f31 fix(plans): fix quoted-parameter bugs in Behave step patterns
CI / benchmark-publish (pull_request) Has been skipped
CI / helm (pull_request) Successful in 1m13s
CI / build (pull_request) Successful in 1m45s
CI / push-validation (pull_request) Successful in 34s
CI / lint (pull_request) Successful in 2m4s
CI / quality (pull_request) Successful in 2m12s
CI / typecheck (pull_request) Successful in 2m34s
CI / benchmark-regression (pull_request) Failing after 1m11s
CI / security (pull_request) Successful in 2m45s
CI / e2e_tests (pull_request) Successful in 5m11s
CI / integration_tests (pull_request) Successful in 5m36s
CI / unit_tests (pull_request) Failing after 9m35s
CI / coverage (pull_request) Has been skipped
CI / docker (pull_request) Has been skipped
CI / status-check (pull_request) Failing after 3s
Resolved two remaining instances of the quoted-parameter bug in
checkpoint_creation_steps.py causing unit_tests CI failures. Bug 1: file
content parameter lacked surrounding quotes (line 93). Bug 2: non-existent
plan_id parameter lacked surrounding quotes (line 279).

ISSUES CLOSED: #8555

Both patterns passed double-quoted strings from Gherkin feature files.
Without surrounding quotes in step decorator patterns, Behave captures
the quote characters as part of the parameter value, causing mismatched
step resolution and CI test failures.
2026-05-09 14:50:00 +00:00
HAL9000 1b5779cc4e fix(plans): resolve CONTRIBUTORS.md merge conflict markers
CI / benchmark-publish (pull_request) Has been skipped
CI / helm (pull_request) Successful in 48s
CI / lint (pull_request) Successful in 53s
CI / build (pull_request) Successful in 54s
CI / quality (pull_request) Successful in 1m10s
CI / typecheck (pull_request) Successful in 1m21s
CI / benchmark-regression (pull_request) Failing after 58s
CI / security (pull_request) Successful in 1m37s
CI / push-validation (pull_request) Successful in 33s
CI / integration_tests (pull_request) Successful in 3m8s
CI / e2e_tests (pull_request) Successful in 4m26s
CI / unit_tests (pull_request) Failing after 5m27s
CI / coverage (pull_request) Has been skipped
CI / docker (pull_request) Has been skipped
CI / status-check (pull_request) Failing after 3s
Remove unresolved <<<<<< HEAD / >>>>>>> bae2b2f3 conflict markers from
CONTRIBUTORS.md that were introduced during automated rebasing. The file
now correctly contains all master entries plus the new checkpoint creation
feature entry for issue #8555.

ISSUES CLOSED: #8555
2026-05-08 16:50:13 +00:00
HAL9000 bae2b2f377 feat(plans): implement checkpoint creation for plan state snapshots
CI / benchmark-publish (pull_request) Has been skipped
CI / lint (pull_request) Successful in 55s
CI / quality (pull_request) Successful in 1m3s
CI / typecheck (pull_request) Successful in 1m19s
CI / security (pull_request) Successful in 1m36s
CI / helm (pull_request) Successful in 32s
CI / push-validation (pull_request) Successful in 52s
CI / build (pull_request) Successful in 1m8s
CI / benchmark-regression (pull_request) Failing after 1m29s
CI / unit_tests (pull_request) Failing after 4m17s
CI / coverage (pull_request) Has been skipped
CI / docker (pull_request) Has been skipped
CI / integration_tests (pull_request) Failing after 4m25s
CI / e2e_tests (pull_request) Successful in 4m43s
CI / status-check (pull_request) Failing after 4s
Checkpoint creation functionality for capturing plan state at specific points during execution. Checkpoints are created manually via CheckpointService.create_checkpoint() or automatically at configurable intervals (on tool write, after tool execution, on subplan spawn, on error). Each checkpoint stores a sandbox reference (git commit hash), metadata (reason, source tool, phase), and optional decision/resource associations.

Key features:
- Manual checkpoint creation via service API and CLI
- Automatic checkpoints triggered at configurable intervals
- Database persistence with retention policy enforcement (default max 50)
- Domain event emission on checkpoint creation for observability
- Configurable retention policy injection into CheckpointService
- Comprehensive BDD test coverage across lifecycle, persistence, and integration scenarios

ISSUES CLOSED: #8555
2026-05-08 09:12:53 +00:00