Skip to content

fix: bound release proof head polling - #787

Merged
jason931225 merged 6 commits into
mainfrom
codex/release-proof-consistency
Aug 16, 2026
Merged

fix: bound release proof head polling#787
jason931225 merged 6 commits into
mainfrom
codex/release-proof-consistency

Conversation

@jason931225

@jason931225 jason931225 commented Aug 16, 2026

Copy link
Copy Markdown
Owner

Summary

  • tolerate only the exact stale pre-push lease tip while the Pulls API converges
  • validate the full initial PR snapshot before any write-capable work, then revalidate every stable field after transport
  • bound head polling to 20 reads / 19 waits and the protected producer job to 10 minutes
  • fail closed on metadata drift, third SHAs, API errors, sleep errors, or producer timeout
  • add RED/GREEN regressions and custody-indexed design and rollout records

Evidence

  • full npm run verify with pinned DotSlash 0.5.7: exit 0, verify(fast) passed
  • focused protected release authority suite: 49/49 passed, zero skipped or failed
  • independent security review: READY, no Critical or Important findings
  • independent test/contract review: READY, no Critical or Important findings
  • reviewed HEAD: 4d07b6f1c8281eee5f35d255ff61e2bdb26a2959
  • reviewed tree: 992cb2ff0a5d4ade2c7952e14d6a74027febe91d
  • base-relative patch SHA-256: adea2726b692666d8b39ea7fdabef8cfb95bc5c1be4123d74e9683a642dfba0b

Review fixes

  • pre-push snapshot validation now binds draft/base/head/repository/ref/creator/title/body fields before worktree creation, token acquisition, commit construction, or push; post-push validation remains
  • release-please producer now has timeout-minutes: 10, bounding a stalled GitHub API call without changing triggers, permissions, secrets, or proof identity

Safety and rollout

This does not change branch protection, required context names, packages, images, or production. After the exact amended head passes all three app-bound required contexts, merge this repair, observe a fresh protected Release Please run on that exact main SHA, and merge PR #760 only after its regenerated exact head has native authority proof plus Required / CI and Required / Security. Image/package/production actions remain HOLD.

Bead: console-l23.

@cursor

cursor Bot commented Aug 16, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4d5f8d3490

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread scripts/console/converge-release-please-doc-custody.mjs
Comment thread scripts/console/converge-release-please-doc-custody.mjs
@jason931225
jason931225 merged commit 0e7cfc0 into main Aug 16, 2026
25 checks passed
@jason931225
jason931225 deleted the codex/release-proof-consistency branch August 16, 2026 17:46
jason931225 pushed a commit that referenced this pull request Aug 16, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.3.7](v0.3.6...v0.3.7)
(2026-08-16)


### Features

* **console:** tracked lane-receipt validator wired into CI preflight
(W1.0 pilot) ([#770](#770))
([d29be0d](d29be0d))
* **db:** 0218 employee_leave_balances schema (hee2 PR-1)
([#783](#783))
([97a45cf](97a45cf))
* **hr:** L5-HR UUID attributes + ReassignOrgUnit→hr.transfer
(console-y0n)
([#761](#761))
([681540d](681540d))
* **identity:** role-set compare, no-op replay conflict,
deactivated-region 409
([#776](#776))
([de8fc64](de8fc64))
* **p4:** retarget Employment owner to the canonical adapter
([#775](#775))
([86c35b1](86c35b1))
* **peas:** employment port routing
([#781](#781))
([fb9ae31](fb9ae31))
* wave-3 admit — 7sx L5-ORG + xxp L5-JOB
([#756](#756))
([9e29b62](9e29b62))
* **wave4:** ADR-0030 §7 backend — aggregate path + OpenAPI generator
([#755](#755))
([84655ce](84655ce))


### Bug Fixes

* bound release proof head polling
([#787](#787))
([0e7cfc0](0e7cfc0))
* **cursor:** ban cross-agent git process/lock races
([#757](#757))
([111e65a](111e65a))
* **employment:** freeze gate + append-only backdating on the port
(rte/r25) ([#780](#780))
([6a7b27b](6a7b27b))
* **gates:** decouple G004 policy contract prose
([#771](#771))
([dea1f91](dea1f91))
* **gates:** prototype-chain lookup census (console-g14a)
([#762](#762))
([ae699da](ae699da))
* **hardening:** fail closed on source/./timeout argv0 wrappers of real
executors ([#778](#778))
([f0f8c1d](f0f8c1d))
* **hr:** B-EMP-A employment containment (31e/2kd/0hf)
([#779](#779))
([0da6c2f](0da6c2f))
* **hr:** clear exit_date on reactivation (console-90h)
([#766](#766))
([b2acd80](b2acd80))
* **hr:** route employee-row creation through the Employment port (u6ih
part-1) ([#782](#782))
([7fc04e3](7fc04e3))
* **identity:** gate replace_user_branches on actor BranchScope
(console-o498)
([#759](#759))
([5770c04](5770c04))
* **logistics:** POD corpus from enforced scheme (console-5yn)
([#767](#767))
([4417bb3](4417bb3))
* **orgchange:** refuse CreateBranch/RenameBranch under deactivated
region (console-k6wm)
([#763](#763))
([950b3b1](950b3b1))
* **payroll:** refuse changed provenance on PayRun conflict; recheck
drain period gate
([#777](#777))
([f9a88ed](f9a88ed))
* residual admit — kwm §60 pin, cvh UI sight (+clippy), gc9z
LifecycleError, 0lj branch-removal
([#754](#754))
([e07fffd](e07fffd))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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