Skip to content

Bind work-item question gates and expose safe native MCP failures - #623

Merged
adrianwebb merged 9 commits into
stagingfrom
codex/work-item-question-gates553
Oct 25, 2026
Merged

adrianwebb merged 9 commits into
stagingfrom
codex/work-item-question-gates553

Conversation

@adrianwebb

@adrianwebb adrianwebb commented Oct 25, 2026 •

Copy link
Copy Markdown
Contributor

Outcome

Blocking inbox questions now gate only their related proposal work items. Managed answers require exact immutable read-back; a reopen or changed source invalidates the proof, and agent replies remain evidence for managed evaluation. Native MCP exposes safe typed Problem responses so coordinators can diagnose the same backend failures seen over REST.

Work authority

Contributor mode (select one):

  • Human-authored
  • Agent-assisted under human authority
  • Agent-authored under human authority

Plan

Completed the shared authoring/gateway repairs, question-to-graph connection, exact answer provenance and serialized lifecycle transitions. Reused the existing graph-readiness observation for typed graph_not_ready and the existing REST Problem formatter for MCP. No new runner, status store, schema, migration or agent-specific proposal targeting.

Changes and commits

Authoring permissions retain governed schema/source authority. Question proof writes lock the inbox row and condition the state transition on the exact source and lifecycle sequence. Advisory, foreign and unknown relationships attach no work-item gate. Existing ownership and review behavior remain unchanged.

The graph query observes exact dependency readiness only after the typed not-ready response and re-queries the same authority. MCP catches typed ControlPlaneOperationError only, returning safe code/status/detail/requestId/instance as structuredContent and JSON text with isError=true. Causes and credentials are excluded; resources and signed confirmation receipts retain their existing behavior.

Verification

Exact-head Actions40159462985 passed complete owner verification, native TreeDX prerequisites, PostgreSQL backup/identity/concurrent-allocation checks, artifact packing, coded component scenes and cleanup. Local combined focused verification passed99cases/15files, strict build and policies.

Real native MCP on the qualified85eff candidate returned409/discussion_agent_chat_profile_missing for the retained group request, confirming structured diagnosis. Source inspection identifies a separate group-alias admission bug now tracked through real API proposal8bc60c93. The Deployment question was observed gating only Releaser while Writer was admitted. Full portfolio simulated release remains unproven; failed review/Reporter observations are retained in docs/agent-log.md.

  • I ran the narrowest relevant package verification and documented any checks that could not be run.

Risk and rollback

No migration. Blocking questions require managed approval; a normal comment cannot release their authority. Roll back through the existing exact-ref delivery process to c7e9bab, preserving all Git/TreeDX products, history and accounting. Group-alias admission, interrupted finalization and full simulation parity are separate unresolved work.

Completion summary

Exact candidate verification passed and supported coached adoption succeeded; final protected staging integration follows the checked PR. This is shared-backend bootstrap delivery, not a claim of a complete native-MCP simulated release.

AGPL committer authorization

Existing base-owned authorization remains authoritative. No license or grant change.

Submission checklist

  • The change is bounded to the stated work item and target repository.
  • Exact base and head refs are recorded and the branch is ready for review.
  • Verification and compatibility evidence are recorded above.
  • No plaintext secrets, credentials, machine state, or unrelated residue are included.
  • Plan, status, commits, and completion summary form a complete durable record.
  • Rollback or recovery steps are documented and executable.

@adrianwebb
adrianwebb force-pushed the codex/work-item-question-gates553 branch from 23d2c29 to bb66a4b Compare October 25, 2026 02:20
@adrianwebb adrianwebb changed the title fix(api): connect work-item questions to verified execution gates Bind work-item question gates and expose safe native MCP failures Oct 25, 2026
@adrianwebb
adrianwebb marked this pull request as ready for review October 25, 2026 05:26
@adrianwebb
adrianwebb merged commit 2793e2a into staging Oct 25, 2026
7 of 9 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.

1 participant