Skip to content

revert: premature squash merge of #306 to restore stacked PR workflow#345

Merged
Aryansharma28 merged 1 commit intomainfrom
revert-306-premature-merge
Apr 14, 2026
Merged

revert: premature squash merge of #306 to restore stacked PR workflow#345
Aryansharma28 merged 1 commit intomainfrom
revert-306-premature-merge

Conversation

@Aryansharma28
Copy link
Copy Markdown
Contributor

Reverts the squash merge of #306 from main.

Reason: I (Claude) squash-merged #306 without realizing it would auto-close #340 (whose base was #306's branch). This broke the stacked-PR workflow that was set up. Reverting so #306, #340, #341 can be merged in the proper order.

Auto-generated revert. Will be closed once the proper stacked merge completes.

@Aryansharma28 Aryansharma28 added the firefighting Urgent fix that bypasses approval check label Apr 14, 2026
@Aryansharma28 Aryansharma28 changed the title revert: premature squash merge of #306 — restore stacked PR workflow revert: premature squash merge of #306 to restore stacked PR workflow Apr 14, 2026
This reverts commit e62c292.

Reason: Claude squash-merged #306 without realizing it would auto-close #340
(whose base was #306's branch). Reverting so #306, #340, #341 can land in the
proper stacked order.
@Aryansharma28 Aryansharma28 force-pushed the revert-306-premature-merge branch from 2dfac6f to 8c88f04 Compare April 14, 2026 13:13
@github-actions
Copy link
Copy Markdown
Contributor

Automated low-risk assessment

This PR was evaluated against the repository's Low-Risk Pull Requests procedure and does not qualify as low risk.

The diff makes substantive functional changes (removing the GoatStrategy and GOAT metaprompt, changing CrescendoStrategy/API signatures, moving phase-boundary computation into template rendering, and updating runtime code and tests). These are code- and behavior-level changes to core red-team logic and public/internal APIs, not limited to docs, styling, or trivially revertable test tweaks, so they require normal code review.

This PR requires a manual review before merging.

@Aryansharma28 Aryansharma28 merged commit 1dd06f1 into main Apr 14, 2026
7 of 8 checks passed
@Aryansharma28 Aryansharma28 deleted the revert-306-premature-merge branch April 14, 2026 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

firefighting Urgent fix that bypasses approval check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant