update copy prompt to match landing page by juecd · Pull Request #637 · kernel/docs · GitHub
Skip to content

update copy prompt to match landing page - #637

Open
juecd wants to merge 1 commit into
mainfrom
cj/update-copy-prompt
Open

juecd wants to merge 1 commit into
mainfrom
cj/update-copy-prompt

Conversation

@juecd

@juecd juecd commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Note

Low Risk
Copy-only change to the default prompt string; no runtime logic or security-sensitive code paths were modified.

Overview
Replaces the default clipboard prompt in CopyPromptButton so it matches the landing-page messaging instead of a CLI-only “setup Kernel” script.

The new default text asks users to describe a task (with a recommended Amazon vs Walmart price-comparison + replay example), points agents at kernel.sh/auth.md and npx skills add, and adds broader guidance on picking the right Kernel surface, handling credentials safely, cleanup, and reporting outcomes. Copy behavior and custom props.prompt overrides are unchanged.

Reviewed by Cursor Bugbot for commit e0960e3. Bugbot is set up for automated code reviews on this repo. Configure here.

@mintlify

mintlify Bot commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

@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: Medium. Not approving.

snippets/copy-prompt-button.jsx is the only file in the diff, and the component’s copy logic is unchanged. The default clipboard string is not a cosmetic copy tweak. It is the prompt users paste into a coding agent from the docs homepage (index.mdx renders <CopyPromptButton /> with no override). The replacement changes agent behavior: authentication moves to the verified-email flow in auth.md and tells the agent to skip dashboard signup, skill install switches to npx skills add, and the text adds interface selection plus credential-handling rules.

That string also fails to parse. The default value is a template literal, and the new text includes an unescaped backtick pair around kernel <command> --help (line 16). The previous prompt escaped inner backticks. Acorn reports Unexpected token at that backtick, so the module does not load. integrations/vercel/foreman.mdx imports the same snippet and passes its own prompt, so a parse failure breaks that page as well as the homepage button.

No infra, auth, or permission code paths changed. The prompt rewrite plus a shared-snippet parse error is enough to keep this at Medium. Escape the inner backticks (or switch the string delimiter) before this is safe to merge.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

These backticks are unescaped inside the surrounding template literal, so the snippet does not parse (Unexpected token at `kernel <command> --help`). The previous default prompt escaped inner backticks as \`. Because index.mdx and integrations/vercel/foreman.mdx both import this module, a parse error takes down both pages, including the Foreman button that passes its own prompt.

This branch was successfully deployed

1 active deployment
staging — e0960e32 Deployed Sep 25, 2026 by mintlify[bot]
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.

1 participant