Skip to content

Conversation

@observerw
Copy link

Fix #4443.

This pr is intend to replace #4447 since it seems buggy.

@observerw observerw changed the title feat(config): Add defaultAgent field feat(config): Add default_agent field in config Dec 10, 2025
🤔 Maybe some AI-generated error code
I've searched through the repo for the string "build" and replaced instances with the default agent from the global
config. The `cfg.default_agent` is marked as non-null (with `!`) since a default value is set.
@observerw observerw marked this pull request as draft December 10, 2025 08:41
@observerw observerw changed the title feat(config): Add default_agent field in config feat: Support custom default agent Dec 10, 2025
@observerw observerw marked this pull request as ready for review December 10, 2025 15:12
@observerw
Copy link
Author

observerw commented Dec 10, 2025

@rekram1-node I added a default_agent field in config, and a default field in Agent.Info to indicate if an agent is assigned to default. Some related logic in TUI and ACP had been updated. If I missed anything pls let me know.

@rekram1-node
Copy link
Collaborator

thanks! ill have to give it a test

@rekram1-node
Copy link
Collaborator

/review

observerw and others added 2 commits December 10, 2025 23:47
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@rekram1-node
Copy link
Collaborator

/review

@github-actions
Copy link

lgtm

return merged
}

function firstPrimaryAgent(agents: Record<string, Agent>): string {
Copy link
Collaborator

Choose a reason for hiding this comment

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

shouldn't it still default to build if it is enabled tho, rather than arbitrary first agent

Copy link
Author

Choose a reason for hiding this comment

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

done

@observerw
Copy link
Author

Wait I found several places (e.g. desktop app) need to be updated

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.

Is it possible to start in plan mode by default?

2 participants