Skip to content

chore: surface AI higher in IA - #1413

Merged
JEverhart383 merged 12 commits into
mainfrom
je-update-ai-section
Apr 30, 2026
Merged

chore: surface AI higher in IA#1413
JEverhart383 merged 12 commits into
mainfrom
je-update-ai-section

Conversation

@JEverhart383

@JEverhart383 JEverhart383 commented Apr 27, 2026

Copy link
Copy Markdown
Contributor

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
Screenshot 2026-04-27 at 4 18 26 PM

@vercel

vercel Bot commented Apr 27, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, ✅ 5 resolved Apr 30, 2026 1:50pm

Request Review

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.tsx is modified — this is a shared .tsx component inside components/, 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.ts change (.next/types/routes.d.ts.next/dev/types/routes.d.ts) appears to be an incidental Next.js-generated file change.

Notes

  • The Icons.tsx change 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's ContentCard components.
  • 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.ts change looks auto-generated and may be unintentional — consider whether it should be included in this PR.
Open in Web View Automation 

Sent by Cursor Automation: Docs PR classifier

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ 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.

Comment thread next-env.d.ts

@rachael-t rachael-t left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I really like this overview page - it's very helpful 🙌

Comment thread content/ai/overview.mdx Outdated
Comment thread content/ai/overview.mdx Outdated
}
/>

## Skills vs. MCP server

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I love this section 👏

Comment thread content/ai/overview.mdx

@samseely samseely left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Image

Let me know what you think!

@JEverhart383
JEverhart383 merged commit ddb2700 into main Apr 30, 2026
5 checks passed
@JEverhart383
JEverhart383 deleted the je-update-ai-section branch April 30, 2026 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants