Rename AI onboarding docs to Agent Toolkit#6510
Conversation
|
Proof screenshots from the local docs preview:
|
Merging this PR will improve performance by 4.28%
|
| Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|
| ⚡ | test_collect_imports[_complicated_page] |
1.6 ms | 1.5 ms | +4.28% |
Tip
Curious why this is faster? Comment @codspeedbot explain why this is faster on this PR, or directly use the CodSpeed MCP with your agent.
Comparing codex/agent-toolkit-docs (798242f) with main (fffe161)
Footnotes
-
8 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩
picklelo
left a comment
There was a problem hiding this comment.
agent toolkit changes look good.
For the event handler plugin - it was more the discoverability of the page rather than the contents. Let's maybe merge just the agent toolkit changes first and figure out the best UX / naming / search for the event handler plugin
af072f6 to
7ca8d91
Compare
|
Updated this to keep only the Agent Toolkit rename/searchability changes. I removed the Event Handler API Plugin discoverability/content changes from this PR so we can handle that UX/naming/search path separately.\n\nValidation: |
Greptile SummaryThis PR renames the "AI Onboarding" docs section to "Agent Toolkit" across the entire docs site, updating the markdown file, sidebar, nav, footer, llms.txt grouping, and tests.
Confidence Score: 5/5Safe to merge — all renamed paths have redirects and legacy aliases in place; no broken external links or route gaps. The changes are purely a documentation rename with comprehensive backward-compatibility coverage: explicit HTTP redirects for both the /ai/ and /ai-builder/ old slugs, a legacy markdown file alias serving identical content, breadcrumb overrides, and updated tests that now correctly target the lazy-imported get_config. No logic paths or runtime code are affected. No files require special attention. Important Files Changed
Reviews (2): Last reviewed commit: "Merge origin/main into codex/agent-toolk..." | Re-trigger Greptile |
| segment_labels = { | ||
| "ai": "AI", | ||
| "ai-onboarding": "Agent Toolkit", | ||
| "agent-toolkit": "Agent Toolkit", |
There was a problem hiding this comment.
Nit: this line isn't needed — to_title_case(to_snake_case("agent-toolkit")) already produces "Agent Toolkit", so the fallback handles it. Removing for clarity:
| "agent-toolkit": "Agent Toolkit", |
…ution # Conflicts: # docs/app/reflex_docs/templates/docpage/docpage.py
|
|
||
| def _llms_url_for_path(url_path: Path) -> str: | ||
| """Return the public URL for a generated markdown asset.""" | ||
| from reflex_base.config import get_config |
There was a problem hiding this comment.
We might want to move it to the top if not for a specific reason.
| segments = [segment for segment in path.split("/") if segment and segment != "docs"] | ||
| segment_labels = { | ||
| "ai": "AI", | ||
| "ai-onboarding": "Agent Toolkit", |
There was a problem hiding this comment.
The slug no longer exists, it redirects to agent-toolkit
| ## Reflex Docs for Agents | ||
|
|
||
| You can give your assistant current Reflex documentation in a few ways. | ||
| You can give your assistant current Reflex documentation in a few ways. These Agent Toolkit resources are designed for assistants that need focused Markdown pages, a broad docs index, structured MCP lookup, or repeatable local skills. |
There was a problem hiding this comment.
These Agent Toolkit resources are designed for assistants that need focused Markdown pages, a broad docs index, structured MCP lookup, or repeatable local skills.
maybe wording can be improved here. Feels too listy.
Resolve conflicts between the Agent Toolkit docs rename and main's docpage/sidebar refactor: - docpage.py: adopt main's module-level _BREADCRUMB_LABEL_OVERRIDES mechanism and add the "Ai Onboarding" -> "Agent Toolkit" entry so the legacy slug still renders the new label; drop the now-redundant local segment_labels dict. - sidebar.py: re-apply the ai_onboarding -> agent_toolkit rename (items, index, slug, category label "Agent Toolkit") on top of main's rewritten sidebar. - test_agent_files.py: keep patching reflex_base.config.get_config to match the PR's inline get_config imports in _plugin.py (main had switched the patch target to the module attribute, which no longer exists). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HCAERc8aspQq5TZr7WJ9fX
Summary
Validation
uv run --project docs/app pytest tests/test_agent_files.py tests/test_routes.pygit diff --check