fix(ce-commit-push-pr): construct retrospective stacks - #1338
Conversation
PR SummaryMedium Risk Overview
Reviewed by Cursor Bugbot for commit 195b100. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 13a8b1cb6b
ℹ️ 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".
Address PR review feedback (#1338) - defer stack submission until Step 5 - return a pipeline residual for confirmation-gated rewrites
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2018e86363
ℹ️ 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".
Address PR review feedback (#1338) - compose metadata against each immediate parent - keep feature-branch commits out of the bottom layer
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit e7c4691. Configure here.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e7c4691a75
ℹ️ 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".
Address PR review feedback (#1338) - keep retrospective work focused on topology construction - describe new stack PRs by explicit URL after submit - preserve existing stack PR metadata without rewrite intent
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cf78c27b86
ℹ️ 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".
Address PR review feedback (#1338) Stop before stack submission when explainer archival would require post-submit branch mutations.

Summary
Stack intent now works for completed work, not only branches already registered with
gh stack. When no topology exists,ce-commit-push-prderives the smallest useful linear dependency layers and constructs them before the ordinary single-branch commit flow. It proceeds when one safe topology is clear and asks before ambiguous review boundaries, hunk-level splits, or published-history rewrites.Existing managed-stack submission and the
ce-babysit-prhandoff remain unchanged.Design decisions
Validation
bun run test— 2,908 passed, 0 failed.bun run release:validate— release metadata is in sync.bun run plugin:validate— strict marketplace and plugin validation passed.gh stack init→ bottom commit →gh stack add→ top commit →gh stack view --jsonwithout pushing.New concepts
Retrospective stack construction
A PR stack can be designed after implementation by partitioning completed work into dependency-ordered review units. The bottom PR provides a coherent foundation; each later PR adds one independently reviewable outcome on its immediate parent.
This skill derives boundaries from the complete change set rather than a fixed PR count. It does not use retrospective stacking when the work is one logical change or when only artificial, hunk-level slices are possible.
Security Disclosure
This changes skill guidance for local
gitandgh stackmutations. Automatic construction is limited to safe local topology; published-history rewrites require explicit confirmation. No credential, permission, dependency, or executable-code changes.Agent Disclosure