Skip to content

Comments

add bigbonk.yml#6146

Merged
elithrar merged 1 commit intomainfrom
elithrar-patch-1
Feb 24, 2026
Merged

add bigbonk.yml#6146
elithrar merged 1 commit intomainfrom
elithrar-patch-1

Conversation

@elithrar
Copy link
Contributor

@elithrar elithrar commented Feb 24, 2026

Max thinking variant for bigger tasks.

Invoked with /bigbonk

Cc @southpolesteve

Max thinking variant for bigger tasks. 

Invoked with `/``bigbonk`
@elithrar elithrar requested review from a team as code owners February 24, 2026 00:31
Copy link
Contributor

@ask-bonk ask-bonk bot left a comment

Choose a reason for hiding this comment

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

I'm Bonk, and I've done a quick review of your PR.

Adds a new bigbonk.yml workflow for a max-thinking variant of the Bonk agent, triggered by /bigbonk.

Issues

  1. Shared concurrency group (medium): Both bonk.yml and bigbonk.yml set name: Bonk, and the concurrency group key uses github.workflow which resolves to the workflow name, not the filename. This means both workflows share the same concurrency group and can cancel/block each other. Since cancel-in-progress: false, a /bonk run would prevent a /bigbonk run on the same PR from starting until the first finishes (and vice versa). Give this workflow a distinct name so the two can run independently.

@ask-bonk
Copy link
Contributor

ask-bonk bot commented Feb 24, 2026

I'm Bonk, and I've done a quick review of your PR.

Adds a new bigbonk.yml workflow for a max-thinking variant of the Bonk agent, triggered by /bigbonk.

One issue found: Both bonk.yml and bigbonk.yml have name: Bonk, and the concurrency group uses github.workflow which resolves to the workflow name. This means they share the same concurrency group and will contend with each other. I posted a suggestion to rename to BigBonk to disambiguate.

github run

@ask-bonk
Copy link
Contributor

ask-bonk bot commented Feb 24, 2026

@elithrar Bonk workflow was cancelled.

View workflow run · To retry, trigger Bonk again.

@elithrar elithrar enabled auto-merge February 24, 2026 01:16
@elithrar elithrar merged commit a1eaf44 into main Feb 24, 2026
29 checks passed
@elithrar elithrar deleted the elithrar-patch-1 branch February 24, 2026 01:20
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