Skip to content

Remove gate artifacts and publish coordinator updates - #8

Merged
alanhoff merged 7 commits into
mainfrom
codex/remove-gates
Aug 26, 2026
Merged

alanhoff merged 7 commits into
mainfrom
codex/remove-gates

Conversation

@alanhoff

Copy link
Copy Markdown
Owner

Summary

  • remove the repository ignore entry for GATES.md; the branch tree contains no GATES.md, ./gates, or related references
  • publish the five previously local coordinator commits together with the cleanup commit

Validation

  • ruff check --no-cache src skill/scripts tests
  • python -m unittest discover -s tests -v (67 tests)
  • python -m compileall -q src skill/scripts tests
  • docker compose config -q

@alanhoff
alanhoff marked this pull request as ready for review August 26, 2026 21:57
Copilot AI lite review requested due to automatic review settings August 26, 2026 21:57
@alanhoff
alanhoff merged commit 0c62b61 into main Aug 26, 2026
5 checks passed
@alanhoff
alanhoff deleted the codex/remove-gates branch August 26, 2026 21:58

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Needs a closer look

It introduces sweeping, security- and correctness-critical changes (schema v6 validation, filesystem fingerprinting/locking, and recovery semantics) that warrant careful human review beyond automated checks.

Pull request overview

This PR publishes a major Coordinator state/protocol update (schema v6) and removes obsolete gate-related ignore behavior, along with adding a Docker-based demo harness and updated public contracts to document and enforce the new invariants.

Changes:

  • Upgrades workflow state storage/validation to schema v6 (assessment + refinement/splitting, provenance obligations, scope fingerprinting, revised locking/recovery, and updated CLI commands).
  • Updates the public protocol/docs and tests to reflect the new “fixed-point planning → route → claim → evidence” contract and removal of VCS coupling.
  • Adds a Docker Compose demo runner plus supporting prompt/analysis content, and starts ignoring generated /data/.
File summaries
File Description
tests/test_contract.py Extends contract tests to assert new protocol references, compose invariants, and removal of git fields.
src/coordinator/state/store.py Implements schema v6 state model, validation, planning gates, scope fingerprinting, locking, and updated mutation/CLI behaviors.
src/coordinator/cli/state.py Updates CLI surface for schema v6 (new node fields, new commands, revised status choices).
skill/SKILL.md Updates the controller protocol documentation for schema v6 workflows and execution rules.
skill/references/workflow-protocol.md Refreshes the workflow protocol reference to match fixed-point planning and execution semantics.
skill/references/state-schema.md Documents schema v6 structure, invariants, diagnostics, and evidence model.
skill/references/model-routing.md Updates routing guidance to use schema v6 assessment mapping and explicit routing rules.
skill/references/complexity-accounting.md Adds rubric v2 details plus refine/split contract examples and rules.
README.md Updates high-level project claims to match schema v6 evidence and planning gates; adds demo guidance.
prompt.md Adds the demo prompt used by the Compose-based run.
compose.yaml Adds a reproducible demo harness for running Coordinator + generated backend in containers.
ANALYZES.md Adds a detailed postmortem/analysis describing the demo failure mode and the schema v6 repair.
.gitignore Drops obsolete /GATES.md ignore and ignores /data/ demo output.
Review details
  • Files reviewed: 11/14 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread skill/SKILL.md
Comment on lines +69 to +70
planning conventions. Use `node-refine --node-id ...
(--refinement-json|--refinement-file)` to atomically persist clarified or reassessed inputs.
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