Skip to content

fix: validate screened candidate IDs without totals#352

Merged
Stahl-G merged 1 commit into
mainfrom
codex/screened-universe-id-validation
Jul 2, 2026
Merged

fix: validate screened candidate IDs without totals#352
Stahl-G merged 1 commit into
mainfrom
codex/screened-universe-id-validation

Conversation

@Stahl-G

@Stahl-G Stahl-G commented Jul 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • Fix the fix: close topology review gaps #350 post-merge review gap where screened candidate ID validation only ran when total_candidates / candidate count metadata was declared.
  • When candidate_claims.json has stable IDs, selected/excluded/deprioritized screened entries must now carry known, non-duplicate candidate_id values even if the screening policy only has fields such as max_items.
  • Keep full universe coverage checks tied to declared totals, so this does not broaden the optional total/count contract.

Boundary

This is a validator hardening PR only. It does not change topology, stage order, runtime handoff guidance, Claim Ledger behavior, gates, delivery, or release authority.

Validation

  • python3 -m pytest -q tests/test_runtime_state.py -k 'screened_candidates or candidate_universe' -> 22 passed
  • python3 -m pytest -q tests/test_runtime_state.py -> 300 passed
  • python3 scripts/check_release_consistency.py --no-tag
  • git diff --check

@Stahl-G Stahl-G marked this pull request as ready for review July 2, 2026 09:32
@Stahl-G Stahl-G merged commit ef639a1 into main Jul 2, 2026
13 checks passed
@Stahl-G Stahl-G deleted the codex/screened-universe-id-validation branch July 2, 2026 09:32
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