Skip to content

Conversation

@olimorris
Copy link
Owner

Description

Update the plugin's commands, allowing for:

  • :CodeCompanionChat adapter=claude_code are you working?

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

@cloudflare-workers-and-pages
Copy link

Deploying codecompanion with  Cloudflare Pages  Cloudflare Pages

Latest commit: a403a17
Status: ✅  Deploy successful!
Preview URL: https://ce8582c2.codecompanion.pages.dev
Branch Preview URL: https://feat-improved-cmds.codecompanion.pages.dev

View logs

@bassamsdata
Copy link
Contributor

This is a great addition, it’s definitely going to make my Neovim workflow smoother :)

Would you be open to something like this?

:CodeCompanionChat adapter=copilot/gpt-5-mini are you working?

Basically, if we detect a / separator, we could treat the part before it as the adapter and the part after as the model name.

If that sounds good to you, I’d be happy to take this on and send an update to the PR!

@olimorris
Copy link
Owner Author

That's a great idea. I've built the commands pipeline such that we could do something like:

:CodeCompanionChat adapter=copilot model=gpt-5-mini are you working?

That would be my preferred way of doing it as it behaves like you're in the CLI. I'll need to do some resolving of the models in the background for completion purposes but I'll put this on the list.

@bassamsdata
Copy link
Contributor

I've built the commands pipeline such that we could do something like:

:CodeCompanionChat adapter=copilot model=gpt-5-mini are you working?

Ah this is cleaner, I didn't know that. Thanks

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.

3 participants