Skip to content

Tighten gate checkpoints and skill validation#14

Open
sw1pp3r wants to merge 1 commit into
Intelligent-Internet:mainfrom
sw1pp3r:fix/gate-checkpoints-skill-validation
Open

Tighten gate checkpoints and skill validation#14
sw1pp3r wants to merge 1 commit into
Intelligent-Internet:mainfrom
sw1pp3r:fix/gate-checkpoints-skill-validation

Conversation

@sw1pp3r

@sw1pp3r sw1pp3r commented Jul 2, 2026

Copy link
Copy Markdown

Summary

  • make passing gate checkpoints self-describing and clarify continue semantics
  • reject unknown task skill names during submit_plan and patch application
  • refresh tests/smoke helpers for current v5 bucket paths and end_mission lifecycle

Verification

  • uv run ruff check .
  • uv run pytest -q
  • deterministic controller E2E smoke for unknown_skill and gate checkpoint continue
  • real Codex ACP smoke reached state=done; stale smoke assertions were updated to bucket paths

@TueVNguyen

Copy link
Copy Markdown
Collaborator

Thanks, @sw1pp3r . We still need additional feedback before moving forward with this PR.

The gate checkpoint is quite complicated, so we may need to brainstorm more before making a decision here. For example, we might consider removing this type of node or exploring other approaches in the next-release.

Thanks for your contribution, but we won’t merge this into the main branch for now.

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.

2 participants