Skip to content

Update mcp.md#2492

Merged
jonweng1 merged 1 commit intomainfrom
jonweng1-patch-2
Feb 12, 2026
Merged

Update mcp.md#2492
jonweng1 merged 1 commit intomainfrom
jonweng1-patch-2

Conversation

@jonweng1
Copy link
Contributor

Hook configuration is loaded at startup, not dynamically after updating in the UI. Apparently this is an annoying quirk with cursor. We should add this step to ensure the hook configs are applied properly. A user got confused because they followed the steps but got no response from the hooks.

Thanks for improving Semgrep Docs 😀

Please ensure

  • A subject matter expert (SME) reviews the content
  • A technical writer reviews the content or PR
  • This change has no security implications or else you have pinged the security team
  • Redirects are added if the PR changes page URLs
  • If you have changed any header tag links (doc/#this-kind-of-anchor), update all instances of that link

Adding a new documentation page? Click to expand the checklist
  • Create .md or .mdx file in /docs/[section]/ with frontmatter: slug, title, description, displayed_sidebar, tags
  • Add page to appropriate sidebar in /sidebars.js (shows in side nav)
  • If adding the doc in a new directory: Update /src/theme/Navbar/Content/index.tsx → add path to getCurrentSection() (highlights top nav)

Sidebars fields for displayed_sidebar:
scanSidebar | rulewritingSidebar | devSidebar | learnSidebar | aboutSidebar | kbSidebar | whatsSemgrepSidebar

Top nav fields for getCurrentSection():
'scan' | 'write-rules' | 'learning-guides' | 'help' | 'explore'

Hook configuration is loaded at startup, not dynamically after updating in the UI. Apparently this is an annoying quirk with cursor. We should add this step to ensure the hook configs are applied properly. A user got confused because they followed the steps but got no response from the hooks.
@netlify
Copy link

netlify bot commented Feb 11, 2026

Deploy Preview for semgrep-docs-prod ready!

Name Link
🔨 Latest commit 1cecebc
🔍 Latest deploy log https://app.netlify.com/projects/semgrep-docs-prod/deploys/698cb2d168256000081afdc8
😎 Deploy Preview https://deploy-preview-2492--semgrep-docs-prod.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

@abhijna-docs-test abhijna-docs-test left a comment

Choose a reason for hiding this comment

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

lgtm. Ty!!

@armchairlinguist
Copy link
Collaborator

@jonweng1 I recommend getting someone from MCP team to review - I'm not really an SME in this area!

@armchairlinguist armchairlinguist removed their request for review February 11, 2026 19:44
@jonweng1 jonweng1 merged commit ae2a665 into main Feb 12, 2026
8 checks passed
@jonweng1 jonweng1 deleted the jonweng1-patch-2 branch February 12, 2026 17:27
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.

5 participants