chore: surface AI higher in IA - #1413
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Risk HIGH: Restructures the AI documentation section by adding a new overview page, reorganizing the sidebar navigation into sub-sections, and importing new icons into a shared UI component.
Reasons
components/ui/Icons.tsxis modified — this is a shared.tsxcomponent insidecomponents/, which triggers HIGH risk per classification rules.- A new MDX page (
content/ai/overview.mdx, 125 lines) is added, which independently triggers MEDIUM risk. - The sidebar navigation file (
data/sidebars/platformSidebar.ts) is restructured with nested sub-sections ("In the dashboard", "In workflows", "For developers"), which independently triggers MEDIUM risk. - 8 files are changed across content, components, sidebar config, and
next-env.d.ts. - The
next-env.d.tschange (.next/types/routes.d.ts→.next/dev/types/routes.d.ts) appears to be an incidental Next.js-generated file change.
Notes
- The
Icons.tsxchange is mechanical (adding 3 lucide-react icon imports:Sparkles,Workflow,Plug), but a reviewer should confirm these icons render correctly in the new overview page'sContentCardcomponents. - The sidebar restructuring introduces nested page groups with empty
slug: ""entries — verify that the sidebar component handles this pattern correctly and that all navigation links resolve. - The new overview page references images (
/images/ai/knock-agent.png,/images/designing-workflows/agent-function.png) — confirm these images exist and render properly. - The
next-env.d.tschange looks auto-generated and may be unintentional — consider whether it should be included in this PR.
Sent by Cursor Automation: Docs PR classifier
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit beb6bac. Configure here.
rachael-t
left a comment
There was a problem hiding this comment.
I really like this overview page - it's very helpful 🙌
| } | ||
| /> | ||
|
|
||
| ## Skills vs. MCP server |
samseely
left a comment
There was a problem hiding this comment.
This is looking great. My main feedback (which I started a Slack thread on here) is around calling out the Knock CLI more explicitly in this update.
IMO for devs approaching building with Knock AI, they are primarily choosing between MCP and CLI. That's the dialogue I see on X and this anthropic post does a good job of calling it out too.
Given that, we should make sure CLI is a recognizable top-level concept within the Knock AI section. To me that means it's one of main concepts in overview page and has its own H2, and also means it's a dedicated page in the sidebar under "Knock AI" section.
I'm not sure if that means skills is also its own page, or more combined with CLI, but will defer to you on that.
As a reference point, in resend build with AI page, first bullet is MCP server, second is CLI.
Let me know what you think!



Description
Adds an overview page to the 'Knock AI' section, small tweaks to nav items.
Latest preview: https://docs-gpfr8anoq-knocklabs.vercel.app/ai/overview
