Skip to content

Conversation

@olimorris
Copy link
Owner

Description

Reading the orchestrator class, cold, yesterday, the method names didn't feel as explicit as they should. This PR fixes that and moves more towards the args pattern.

Checklist

  • I've read the contributing guidelines and have adhered to them in this PR
  • I've added test coverage for this fix/feature
  • I've run make all to ensure docs are generated, tests pass and my formatting is applied
  • (optional) I've updated CodeCompanion.has in the init.lua file for my new feature
  • (optional) I've updated the README and/or relevant docs pages

@olimorris olimorris merged commit 68e9939 into main Dec 19, 2025
4 checks passed
@olimorris olimorris deleted the refactor/orchestrator branch December 19, 2025 07:43
georgeharker pushed a commit to georgeharker/codecompanion.nvim that referenced this pull request Jan 6, 2026
georgeharker pushed a commit to georgeharker/codecompanion.nvim that referenced this pull request Jan 8, 2026
georgeharker pushed a commit to georgeharker/codecompanion.nvim that referenced this pull request Jan 8, 2026
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