Skip to content

Reuse existing pull requests during publication - #539

Open
omkhar wants to merge 3 commits into
mainfrom
fix/publish-existing-pr
Open

Reuse existing pull requests during publication#539
omkhar wants to merge 3 commits into
mainfrom
fix/publish-existing-pr

Conversation

@omkhar

@omkhar omkhar commented Jul 30, 2026

Copy link
Copy Markdown
Owner

Summary

  • reuse an exact matching open pull request after a verified push
  • anchor GitHub operations to the single effective origin push destination without exposing embedded credentials
  • preserve draft and certified-adapter policy invariants, including create-race handling
  • document and test the idempotent publication contract

Validation

  • live existing-PR reuse against PR ^F Freeze the v1 public contract #537
  • three clean peer-review lenses after iterative fixes
  • Workcell quick validation
  • exact-head pr-parity shared validation
  • exact-head pre-merge pr-parity evidence, including host invariants and container smoke

…reuse, and peer review pass; user-visible publication bug fix)
@omkhar

omkhar commented Jul 30, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@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: 0ccb7ea5ab

ℹ️ 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 docs/safe-path-expectations.md
@omkhar
omkhar marked this pull request as ready for review July 30, 2026 20:42
@omkhar

omkhar commented Jul 30, 2026

Copy link
Copy Markdown
Owner Author

@codex review

Please perform a final source-correctness review of exact signed head 0ccb7ea5ab45a44808acb56cfc11d71c9219daef. The prior signature thread cited unrelated object 2837b2ca5d067d0c513cf22661eb907d7a887a64; the actual PR head verifies locally with the maintainer key and GitHub reports verification.verified=true, reason=valid. Review the implementation and tests at the exact PR head.

@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: 0ccb7ea5ab

ℹ️ 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 internal/publishpr/publish_pr_cli.go Outdated
…reuse, quick, and peer review pass; hosted review fix)
@omkhar

omkhar commented Jul 30, 2026

Copy link
Copy Markdown
Owner Author

@codex review

Please re-review exact signed head a28c8b0f86f52c53f523eb2870ce6449e0fe0205. The deleted-foreign-fork finding is addressed with explicit null-vs-missing parsing, origin-only metadata validation, focused and scenario regressions, three CLEAN peer lenses, live GitHub reuse, exact-head parity, and clean-snapshot pre-merge parity.

@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: a28c8b0f86

ℹ️ 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 internal/publishpr/publish_pr_cli.go Outdated
…ce, live reuse, quick, and peer review pass; hosted review fix)
@omkhar

omkhar commented Jul 30, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@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: 98703e7159

ℹ️ 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 docs/safe-path-expectations.md
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