Add context7 repository indexing#100
Conversation
Signed-off-by: EfeDurmaz16 <efebarandurmaz05@gmail.com>
|
Caution Review failedPull request was closed or merged during review No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughA new ChangesContext7 Indexing Configuration
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 351a79d89c
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| library identity is: | ||
|
|
||
| ```text | ||
| agentcommercekit/ack |
There was a problem hiding this comment.
Include the leading slash in the Context7 ID
When readers copy this as the exact Context7 library identity for MCP/API usage, it does not match Context7's documented ID format: their API guide defines a library ID as the URL path and says to use /owner/repo for GitHub repositories (for example /vercel/next.js). The guide should show /agentcommercekit/ack so users can paste it directly into use library ..., refresh, or docs queries without first re-resolving the library.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Addressed in 3ef8133 by using the canonical /agentcommercekit/ack Context7 library ID.
Signed-off-by: EfeDurmaz16 <efebarandurmaz05@gmail.com>
Remove the agent-framework-integrations doc page and its nav entry; keep only the context7.json repository indexing config. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
Thanks for the contribution, @EfeDurmaz16 — and congrats on your first one merged into ACK! 🎉 I trimmed this PR down a bit before merging: I dropped the Appreciate you getting the Context7 indexing set up for us! |
venables
left a comment
There was a problem hiding this comment.
LGTM — clean, self-contained Context7 indexing config. Thanks again!
Summary
Closes #98
Verification
Notes
context7.json; this PR keeps runtime framework adapters as follow-up work.AI Usage Disclosure
This contribution was AI-assisted using Codex CLI and the Codex app. AI assistance was used for repository/docs navigation, understanding ACK/Catena context, identifying relevant issues or contribution areas, and assisting with small edits/validation. I reviewed the final diff and take responsibility for the submitted changes.
Summary by CodeRabbit