Skip to content

Generate completion reports deterministically#175

Merged
chubes4 merged 4 commits into
mainfrom
fix/deterministic-completion-report
Jul 20, 2026
Merged

Generate completion reports deterministically#175
chubes4 merged 4 commits into
mainfrom
fix/deterministic-completion-report

Conversation

@chubes4

@chubes4 chubes4 commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • generate the completion artifact from caller inputs, Git state, writable scope, and bootstrap filesystem checks
  • remove model-authored completion JSON and transcript parsing from the acceptance path
  • retain only machine-observed report facts and harden canonical artifact staging against traversal, symlinks, and oversized output

Fixes #174

How to test

  1. Run php tests/validate-docs-agent-packages.php.
  2. Run php tests/validate-docs-agent-completion-contract.php.
  3. Check out WP Codebox at a6fe2d208e990a8d04104aa74aacbb8d1539fbc1, then run WP_CODEBOX_DIR=/path/to/wp-codebox php tests/validate-wp-codebox-run-agent-task-contract.php.
  4. Run php tests/validate-external-native-package-sources.php.
  5. Run actionlint .github/workflows/*.yml and git diff --check.

Compatibility

This intentionally removes the unstable model-authored DOCS_AGENT_COMPLETION_REPORT response contract and revises docs-agent/completion-report/v1 to contain host-observed facts only. Accepted consumers will be repinned atomically; no backwards-compatibility parser is included.

Control-plane evidence

Homeboy Lab finalization was attempted but blocked before provider execution by Extra-Chill/homeboy#9115. Direct PR finalization was explicitly authorized after the committed candidate passed the checks above.

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode with GPT-5.6-sol
  • Used for: Implemented and reviewed the deterministic completion contract, schema, workflow integration, documentation, and tests with Chris reviewing and owning the change.

@chubes4
chubes4 merged commit b136f9b into main Jul 20, 2026
3 checks passed
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.

Generate completion reports deterministically from runner evidence

1 participant