Rename AI onboarding docs to Agent Toolkit#6510
Conversation
|
Proof screenshots from the local docs preview:
|
picklelo
left a comment
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
Alek99
force-pushed
the
codex/agent-toolkit-docs
branch
from
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 page to "Agent Toolkit" across all relevant files, including the sidebar, nav, footer, marketing links, llms.txt groupings, and tests. Backward compatibility is preserved via HTTP redirects for the old slugs and a legacy
Confidence Score: 5/5Safe to merge — all old URLs are covered by redirects, the legacy markdown alias preserves agent-facing content at the old path, and tests confirm the new routes and content parity. The rename is consistent across every touch point: route, sidebar, breadcrumb, footer, nav, llms.txt grouping, and tests. The markdown generator materializes results as a tuple before building the lookup dict so the iterable can be expanded again. The _ordered_sections logic produces the correct AI Builder → Agent Toolkit → MCP → Skills sequence, verified by updated test assertions. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A["/ai/integrations/ai-onboarding/"] -->|redirect| C["/ai/integrations/agent-toolkit/"]
B["/ai-builder/integrations/ai-onboarding/"] -->|redirect| C
D["/ai-builder/integrations/agent-toolkit/"] -->|redirect - general rule| C
C --> E["Agent Toolkit doc page"]
F["ai/integrations/ai-onboarding.md"] -->|legacy alias - same content| G["ai/integrations/agent-toolkit.md"]
G --> H["llms.txt: Agent Toolkit section"]
Reviews (1): Last reviewed commit: "Rename AI onboarding docs to 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
Alek99
changed the title
|
|
||
| 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.
3 participants
Summary
Validation
uv run --project docs/app pytest tests/test_agent_files.py tests/test_routes.pygit diff --check