Skip to content

test: add valid/invalid fixtures for checkpoint and fork schemas - #204

Merged
telleroutlook merged 2 commits into
mainfrom
fix/169-checkpoint-fork-schemas
Aug 5, 2026
Merged

test: add valid/invalid fixtures for checkpoint and fork schemas#204
telleroutlook merged 2 commits into
mainfrom
fix/169-checkpoint-fork-schemas

Conversation

@telleroutlook

Copy link
Copy Markdown
Contributor

Fixes conformance.py failure introduced when checkpoint/fork schemas were added in #203 without accompanying fixtures.

  • tests/fixtures/valid/checkpoint/{minimal,full}.json
  • tests/fixtures/invalid/checkpoint/{missing-required,extra-property}.json
  • tests/fixtures/valid/fork/{minimal,full}.json
  • tests/fixtures/invalid/fork/{missing-required,bad-enum}.json

All 19 schemas now pass conformance.py. bun test 18/18.

Operator added 2 commits August 6, 2026 07:22
Implements #169. Adds:
- schemas/aep/checkpoint.schema.json — snapshot of execution state
  (checkpoint_id, state_digest, created_at, optional parent_run_id/fork_of)
- schemas/aep/fork.schema.json — new lineage from an existing checkpoint
  (fork_id, source_checkpoint_id, fork_reason, created_at, optional target_branch)
- docs/checkpoint-fork-architecture.md — concept definitions, field semantics,
  relationship to CheckpointEvidence
- schemas/index.json — registers both schemas (checkpoint/v0.1, fork/v0.1)

bun test: 18/18 pass. npm run drift: no drift.
conformance.py requires at least one valid and one invalid fixture per
registered schema. Adds:
- tests/fixtures/valid/checkpoint/{minimal,full}.json
- tests/fixtures/invalid/checkpoint/{missing-required,extra-property}.json
- tests/fixtures/valid/fork/{minimal,full}.json
- tests/fixtures/invalid/fork/{missing-required,bad-enum}.json

All 19 schemas now pass conformance.py (was: 4 errors for missing fixtures).
@telleroutlook
telleroutlook merged commit d5def8e into main Aug 5, 2026
4 of 5 checks passed
@telleroutlook
telleroutlook deleted the fix/169-checkpoint-fork-schemas branch August 5, 2026 23:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant