feat: Add Context7 integration configuration #1450
Conversation
- Add context7.json to enable AI coding assistant integration - Configure documentation parsing for 40+ neurophysiology formats - Include best practices for NWB conversion workflows - Support for multiple recent versions (v0.6.7 - v0.7.5) - Exclude source code and focus on user-facing documentation This enables developers to get up-to-date NeuroConv documentation directly in AI coding assistants like Cursor and Claude.
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Another similar solution is this:
https://github.com/ref-tools/ref-tools-mcp
On one hand, this is something extra to maintain but the document is really small and not very costly. Maybe we should just have it for the sake of experimentation. I am OK with it but we should improve the rules a bit.
@bendichter has been thinking hard about AI tools and I think he will be interested on this. Maybe @luiztauffer will be interested as well.
| "Call get_metadata() and enrich the returned dict before running conversion.", | ||
| "Use run_conversion() with backend compression settings for large recordings.", | ||
| "Leverage chunking options to keep NWB files manageable.", | ||
| "Run check_read() on every interface before conversion to catch I/O issues early.", |
There was a problem hiding this comment.
The following rules from here to the end seem more dubious. I don't think we have a check_read for example.
Thanks, good to know. No doubt there will be others soon too.
Projects can be added directly to context7.com (by "community members") but the configuration is detected automatically and errors etc. are more likely to occur. Given there are competing projects in this area it does seem like there is a risk of over-committing on support. The downside of not doing the experimentation you propose would be that end users think it's supported but the efficiency/functionality of a well specified set of rules won't be part of the MCP server. I don't have enough experience with the project to specify the rules (clearly, given I didn't pick up on the hallucinations!). I hope we can get something along these lines merged though. If I can help just let me know.
Excellent. |
|
@leej3 very cool that you are thinking about agentic coding for NWB conversions! As @h-mayorquin we have been looking pretty deep into this. I agree that some of the rules here are great but some of them are not really right and I think we are going to need a lot more for making actually good conversions. While I am excited about embracing AI to lower the energy barrier of open data, I worry about an agent like this creating and publishing technically correct NWB files that do not have the appropriate data. For example having the neurophysiology signal, but missing key (or all) stimulus and behavioral information. This is a common problem already in the DANDI Archive and I think an agent like this might make it worse if we don't think very carefully about how we guide it. Would you be open to a meeting to discuss this? |
Add Context7 Configuration for NeuroConv
This pull request adds first‑class Context7 support for NeuroConv so that AI coding assistants (Cursor, Copilot Chat, Zed, etc.) can access version‑specific conversion guidance directly in prompts.
Benefits for users
*Interfaceand to validate withcheck_read().Details on the approach
I have tried to follow the best practices outlined in their document for guiding this. Some of the highlights are:
context7.json$schemakey enables auto‑completion in IDEs and guarantees the file adheres to the Context7 JSON‑Schema.