Skip to content

Conversation

@Davidyz
Copy link
Owner

@Davidyz Davidyz commented Apr 29, 2025

olimorris/codecompanion.nvim#1290

example config:

opts = {
  extensions = {
    vectorcode = {
      opts = { add_tools = true, add_slash_command = true },
    },
  },
},

@Davidyz Davidyz added the enhancement New feature or request label Apr 29, 2025
@codecov
Copy link

codecov bot commented Apr 29, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.15%. Comparing base (397b6f6) to head (804845b).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #99   +/-   ##
=======================================
  Coverage   99.15%   99.15%           
=======================================
  Files          22       22           
  Lines        1423     1423           
=======================================
  Hits         1411     1411           
  Misses         12       12           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Davidyz Davidyz force-pushed the nvim/codecompanion_extension branch 2 times, most recently from f33ccaf to 7381e21 Compare April 29, 2025 17:23
@Davidyz Davidyz force-pushed the nvim/codecompanion_extension branch from 7381e21 to 36740d2 Compare April 29, 2025 18:12
@Davidyz Davidyz marked this pull request as ready for review April 30, 2025 02:01
@Davidyz
Copy link
Owner Author

Davidyz commented Apr 30, 2025

@ravitemer hi I think I'm done here with the extension. If you're available, could you take a look at my implementation? Just want to make sure I did everything correctly before I merge.

btw this really setting things up A LOT easier. Thanks for the extension API!

@ravitemer
Copy link

@Davidyz It's perfect. Your way of not using the callback to point to the extension is even better.

Looks great to me!

@Davidyz Davidyz merged commit 8050123 into main Apr 30, 2025
12 checks passed
@Davidyz
Copy link
Owner Author

Davidyz commented Apr 30, 2025

Something I came across when writing this PR:
It might be worth adding to the documentation that extension devs should check for duplicated tool names etc. because it looks like the setup function might overwrite things without the users noticing.

@ravitemer
Copy link

Yes! I am guilty of that 😅. Will update the docs and mcphub extension.

@Davidyz Davidyz deleted the nvim/codecompanion_extension branch May 14, 2025 02:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants