Skip to content

don't change current agent#564

Merged
backnotprop merged 1 commit intobacknotprop:mainfrom
andreineculau:patch-1
Apr 15, 2026
Merged

don't change current agent#564
backnotprop merged 1 commit intobacknotprop:mainfrom
andreineculau:patch-1

Conversation

@andreineculau
Copy link
Copy Markdown
Contributor

agent_cycle is not the correct command as the user may have several agents defined, not only build and plan.

I do think that the changes introduced in #40 make sense, that is "not only run the plan with the build agent, but also switch the agent to build" - but in its current format, it is simply buggy as it assumes only a build and plan agent.

I checked the opencode sourcecode and I'm not sure what is the alternative. Cycle and validate that the current agent is the target agent, if not repeat?!

agent_cycle is not the correct command as the user may have several agents defined, not only build and plan
@backnotprop
Copy link
Copy Markdown
Owner

There is a setting in the UI, does that not work for you?

You can disable agent switching

@andreineculau
Copy link
Copy Markdown
Contributor Author

@backnotprop it's 2 different things. I want that option enabled, so that an approval triggers the build agent to implement the plan. Disabling the option disables the agent cycle but also the automatic implementation in
https://github.com/andreineculau/plannotator/blob/f893a9d9874d4738bd1938b5c01354d0cf9c1438/apps/opencode-plugin/index.ts#L467-L475

@backnotprop
Copy link
Copy Markdown
Owner

I understand. That's an oversight on my part.

@backnotprop backnotprop merged commit a8dc9e8 into backnotprop:main Apr 15, 2026
7 checks passed
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