docs(integrations): add copy-paste recipes for LlamaIndex/CrewAI/Auto… - #65
docs(integrations): add copy-paste recipes for LlamaIndex/CrewAI/Auto…#65popwatt wants to merge 2 commits into
Conversation
|
Warning Review limit reached
Next review available in: 45 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughAdded a bilingual Python Framework Integrations guide. It documents MCP and REST integration with LlamaIndex, CrewAI, AutoGen 0.4+, and LangGraph. Repository README files and the VitePress sidebar now link to the guide. ChangesFramework integrations
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related issues
Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 6
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@repo_pages/guide/framework-integrations.md`:
- Around line 36-37: Update the “The agent needs an LLM” tip in the framework
integrations guide to exclude the LlamaIndex snippet, since it directly calls
search.acall without creating an LLM client. State that only the snippets using
an LLM require one, while preserving the existing OpenAI-compatible and
local-model guidance and the LlamaIndex no-key clarification.
- Around line 26-28: Update the curl readiness-check command for the POST search
request to include --fail-with-body (or -f), so HTTP 4xx/5xx responses produce a
failure while preserving the existing request and response body behavior.
In `@repo_pages/zh/guide/framework-integrations.md`:
- Around line 26-28: Update the curl readiness-check command in the framework
integrations guide to include -f or --fail-with-body, ensuring HTTP 4xx and 5xx
responses produce a failure while preserving the existing request options.
- Line 65: Translate all natural-language content in the Chinese document,
including the search query near search.acall and the role, goal, backstory, task
descriptions, expected output, and user prompts at the referenced sections. Keep
framework names, package names, API identifiers, and code syntax unchanged, and
ensure no English prose remains mixed into the Chinese page.
- Around line 36-37: Update the prerequisite text in the 智能体需要一个 LLM tip to
exclude the LlamaIndex snippet, stating that only the snippets that invoke an
LLM require one. Preserve the existing provider examples and framework-specific
documentation reference.
- Around line 183-186: Update the REST API example near the httpx import to
explicitly instruct readers to install the HTTP client in the environment
running the framework code, adding the command pip install httpx before the
example.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: 66be59ca-93d7-4d51-a566-78bd41130420
📒 Files selected for processing (5)
README.mdexamples/README.mdrepo_pages/.vitepress/config.mtsrepo_pages/guide/framework-integrations.mdrepo_pages/zh/guide/framework-integrations.md
…ns guide - make the curl readiness check fail on HTTP 4xx/5xx (-f) - scope the LLM prerequisite tip to the agent snippets; LlamaIndex direct-call example needs no LLM key - translate natural-language strings in the zh page code snippets (queries, agent metadata, prompts); keep API identifiers unchanged - add explicit pip install httpx before the REST example (EN + zh parity) Closes afx-team#29
38c3184 to
ab10dc4
Compare
1 participant
…Gen/LangGraph
Summary
Motivation
Checklist
pytest tests/ -v)repo_pages/, docstrings, or README) if user-facingCHANGELOG.mdupdated under## [Unreleased]ruff check src/passesmypy src/hebb/passes (project usesstrict = true)Notes for reviewers
Summary by CodeRabbit