Skip to content

fix(risk-fork): preserve cleanup and ambiguity evidence custody - #488

Merged
rhein1 merged 2 commits into
mainfrom
codex/risk-fork-worker-boundary-observations-20261006
Oct 9, 2026
Merged

rhein1 merged 2 commits into
mainfrom
codex/risk-fork-worker-boundary-observations-20261006

Conversation

@rhein1

@rhein1 rhein1 commented Oct 6, 2026 •

Copy link
Copy Markdown
Collaborator

Source-only Risk Fork cleanup now survives hostile thrown values, and ambiguity records retain bounded internal recovery provenance without accepting caller-forged authority. Cleanup runs once for each resource, unknown outcomes stay unknown, and diagnostics use controller-owned labels.

The branch is refreshed against main 42dad6ae1ae9c28df9e6abdda16aaef22db7792b. Both generated conflicts were resolved through the canonical hosted builder; the final eight-path diff preserves the merged #498, #499, and #500 security repairs.

Validation of tree bf02821d0df443c54c15c43ff8c29fd934bdb667:

  • Targeted crash/activation regressions: 59/59, zero skips.
  • Full Windows core suite: 647 passed, 151 platform/database skips, zero failures.
  • Full Windows managed suite: 393 passed, 165 database skips, zero failures.
  • Hosted artifact, integrity, and offline packed-consumer tests: 16/16, zero skips.
  • Canonical source integrity: sha256:e9042558b3ea295e0fafc1c40246dcf6cce7e470a22e1ba2028a2bdda22ee8e5.
  • Source/privacy review and fresh diff verification: no blocking finding. Fresh exact-head hosted Linux/PostgreSQL checks and eligible approval remain required.

Tracking issues #375, #301, #302, and #303 remain incomplete. Provider use, production qualification, publication, deployment, staging, canary, and activation remain default-off and separately gated.

Workflow: Fable deep-review lenses with real inherited Codex subagents; the requested Sol Ultra/Luna profile was not active. Root owns repairs and GitHub actions. Reviews and tests establish source behavior, not independent operational certification.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-06T23:58:14.544868Z e909d15 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@jmborden1112-beep jmborden1112-beep left a comment •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM for exact head e909d15, tested/reviewed tree a54849387c159aca640a10fe2e1e5d67d0d20df5, eight public paths against deb37e1.

Actual source/privacy and operations/test review contexts examined the final diff, callers, regressions, docs and generated custody. Hostile thrown values cannot bypass cleanup, inspect exception hooks or cause duplicate destruction. Private internal recovery provenance retains bounded real authority state/operation references without forwarding public-constructor evidence. Unknown cleanup remains unknown; persisted ambiguous effects are not retried. Final canonical hosted bundle/manifest matches source.

Final local proof: focused105/105; core1016/1020 with four existing platform skips; mandatory real PG16.15 CA-TLS/distinct-role managed568/568, no skips; hosted16/16; packed consumer141files593233bytes. Correct archive24b80d4c719eef332a9847436745825c034f023f18be14492df7b4c4ebbc89cb/log binds exacta548, labko5aplpS/exit0/cleanuptrue with independent local resource-absence checks. An earlier operations log reference to old#487 was withdrawn and rechecked against this final candidate. All27 applicable exact-head PR checks now succeeded; local/GitHub head signature valid. Rejected candidates and test failures retained.

This is the authorized second owner-controlled account's eligible source review, not third-party operational certification. No provider qualification, billing/cost proof, managed HA/PITR, deployed edge, release publication, staging, canary, enrollment, activation or live protection is established or authorized by this approval. Original production gate parents remain incomplete. No admin/protection bypass.

Correction to the issue-association statement: GitHub initially interpreted a negated closing phrase in the PR prose as an issue-closing instruction. The wording was changed without altering this source head, and closing-issue references were verified empty before merge. Completion issues remain OPEN. This correction supersedes the earlier premature no-association assertion.

@jmborden1112-beep jmborden1112-beep left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

Needs Fixing

None found in refreshed head f9f3341e7d4116dd0d4a6c72ec258391bbcf5ea7, tree bf02821d0df443c54c15c43ff8c29fd934bdb667.

Requires Human Review

None for this authorized source-only merge. Production and provider gates remain separate.

Recommended Optional

None.

Create Follow-up Issue

Existing #375, #301, #302 and #303 retain operational qualification requirements.

Cleanup/private recovery provenance was reviewed against current main. The final eight-path diff retains #498/#499/#500 changes; both generated conflicts were rebuilt canonically. Targeted tests59/59; full Windows core647passed/151skipped, managed393passed/165skipped; hosted artifact16/16, zero skips. Hosted Linux/PostgreSQL results must still be green before merge.

Workflow Trace: real inherited Codex subagent source/privacy and operations/test review; root repaired and verified the final tree and source attestation e9042558. This is the user's authorized second owner-controlled account approval, not third-party operational certification. No provider call, spend, deployment, publication, activation or protection claim. Normal branch protection applies.

@rhein1
rhein1 merged commit 1d1d346 into main Oct 9, 2026
27 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.

2 participants