Skip to content

fix(web): align chat with the current Core contract - #25

Merged
sam2tom merged 2 commits into
codex/issue-22-openai-ui-alignmentfrom
codex/issue-24-execution-readiness
Sep 16, 2026
Merged

sam2tom merged 2 commits into
codex/issue-22-openai-ui-alignmentfrom
codex/issue-24-execution-readiness

Conversation

@sam2tom

@sam2tom sam2tom commented Sep 16, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • restore the documented agents=v1 Session message, steering, cancellation, and function-result event flow
  • require exact HTTP 204 admission for event writes, keep uncertain payload/idempotency state for explicit unchanged manual retry, and never retry automatically
  • remove the private execution-readiness gate that blocked the current public chat contract
  • remove the top-level Environments destination and Environment template/key entries because Core exposes no corresponding public list or management APIs
  • retain the exact read-only Environment projection for an already selected self_hosted Session
  • refresh the immutable Parsar compatibility audit to d91ba48

Why the original gate changed

The live Core audit established that Session event submission is the current versioned public contract. The absence of a separate readiness resource does not make those documented writes unsupported; runtime, model, provider, or credential failures remain visible through durable Session/Turn/Item state and are never treated as readiness proof.

This supersedes the original fail-closed plan in #24; it does not claim that plan's negative-write acceptance criteria were implemented.

Validation

  • pnpm check (67 root tests, 50 agents-client tests, 344 Web tests, typecheck, and production build)
  • pnpm test:acceptance (41/41 passed against the isolated fixture Core)
  • git diff --check
  • real local Parsar d91ba48 flow: Web -> Agents API -> daemon -> Codex 0.153.4 -> provider -> assistant output, confirmed interactively; provider credentials remained server-side and were not committed

Scope

Web/client/docs only. No Parsar source change, deployment, branch deletion, or credential material.

Restore documented Session event writes with exact HTTP 204 admission, remove unsupported top-level Environment surfaces, and refresh the pinned Parsar compatibility coverage.
@sam2tom sam2tom changed the title feat(web): gate execution writes on proven compatibility fix(web): align chat with the current Core contract Sep 16, 2026
@sam2tom
sam2tom marked this pull request as ready for review September 16, 2026 14:16
@sam2tom
sam2tom merged commit 021353c into codex/issue-22-openai-ui-alignment Sep 16, 2026
1 check 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