fix(hosted): harden provider delivery contracts - #1016
Merged
Conversation
Greptile SummaryThe PR hardens hosted provider execution and GitHub delivery boundaries while aligning lifecycle and terminal-result behavior across Codex, Claude, and OMP.
Confidence Score: 5/5The PR appears safe to merge based on the available follow-up review evidence. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| zeroshot-rust/hosted-node/workspace-delivery-github.js | Adds bounded auto-merge retries and authoritative PR-state reconciliation before reporting delivery success. |
| zeroshot-rust/hosted-node/workspace-delivery-retry.js | Introduces the shared bounded operation-and-reconciliation retry policy for uncertain GitHub effects. |
| zeroshot-rust/hosted-node/workspace-ship.js | Verifies both the merge revision and delivered provider revision against the refreshed target branch. |
| src/agent-cli-provider/single-agent-runtime.ts | Propagates validated provider execution context and registry-owned runtime configuration through command preparation. |
| src/agent-cli-provider/omp/sdk-environment-policy.ts | Restricts OMP SDK child environments to minimal process variables and registry-declared non-secret configuration. |
| src/agent/agent-lifecycle.js | Makes delayed starts cancellable and incorporates normalized provider terminal failures into lifecycle handling. |
| task-lib/runner.js | Aligns detached watcher ownership, prompt transport, execution context, and Codex writable Git metadata paths. |
| private/hosted-cli-candidate/credentials.js | Preserves bounded provider credentials and runtime authority in the private hosted runtime bundle. |
| docker/zeroshot-oecp/Dockerfile | Updates the private capsule image to include the hardened runtime dependencies and writable execution layout. |
Sequence Diagram
sequenceDiagram
participant CLI as Hosted CLI
participant Capsule as OECP Capsule
participant Provider as Provider Runtime
participant Delivery as GitHub Delivery
participant GitHub as GitHub Authority
CLI->>Capsule: Submit RunIntent + private runtime bundle
Capsule->>Provider: Launch with validated execution context
Provider-->>Capsule: Bounded terminal result
Capsule->>Delivery: Ship verified workspace mutation
Delivery->>GitHub: Push deterministic branch and create PR
Delivery->>GitHub: Merge or enable auto-merge
GitHub-->>Delivery: Authoritative PR/merge state
Delivery-->>Capsule: Verified delivery receipt
Capsule-->>CLI: Bounded RunIntent result
Reviews (2): Last reviewed commit: "fix(hosted): harden provider delivery co..." | Re-trigger Greptile
mkceichelbeck
force-pushed
the
codex/fix-provider-matrix-linear-20260813
branch
from
August 14, 2026 00:29
42c6cfd to
f3ed3fe
Compare
|
🎉 This PR is included in version 6.39.2 🎉 The release is available on: Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Deployed validation
Validated through
dev.theopenengine.comwith real provider calls, real work, and real--shipdelivery tothe-open-engine/zeroshot-matrix-demo-20260812-3ebd. Every shipped PR merged and its repository CI passed.019ffd84-61e7-7d70-b900-f74eab6a6a5c019ffd8d-6159-70a3-ac33-cc3a1eeb049d019ffd9c-e8f8-7f30-a6b1-d142a54238b6019ffd9c-ef80-7901-b892-4b1fdaf0d244019ffd9c-f8e8-7070-9b16-781bba765fab019ffd9c-ff78-7791-950c-c003ba111378The targeted Azure failure rows were proven first; only then were the four previously green rows rerun as regression guards. The accumulated demo repository suite passes 543/543 tests. No delivery fixtures or simulated provider results were used.
The full runtime tree was tested at
42c6cfd; this PR was then recomposed as one conventional commit. The only tree delta inf3ed3feis two test-fixture files (no production/runtime files). Zero-cloud main pinsf3ed3fe, and focused exact-head deployment run31757758450completed successfully with image scan, IaC apply, and public health checks.Validation
npm run check: green (0 errors; existing warnings only)