Skip to content

docs: add interactive demo visibility rule to style guide - #2145

Open
youssefea wants to merge 4 commits into
masterfrom
docs/demo-visibility-style-rule
Open

youssefea wants to merge 4 commits into
masterfrom
docs/demo-visibility-style-rule

Conversation

@youssefea

Copy link
Copy Markdown
Contributor

What changed? Why?

Follow-up to #2142, addressing this review: the rule for hiding interactive demos from agents should live in the docs style guide, not only in claude.md.

  • docs/content-guidelines.md: new Interactive Demos section under Writing Rules. It explains why (Mintlify inlines each imported JSX component's full source into the page's Markdown export) and sets the pattern: wrap the demo, its ## Demo heading, and any demo-only callout in <Visibility for="humans">; keep version notes, addresses, warnings, and Steps outside; wrap every use of the component on the page. Includes a copyable example.
  • .claude/skills/doc-feedback.md: adds a checklist item so the doc review skill checks for it.

Notes to reviewers

How has it been tested?

  • npm test: all pass
  • node scripts/check-terminology.js: pass
  • Docs-only change to contributor guidance; no pages or docs.json changed.

Screenshots

N/A (no user-facing changes)

Generated with Toshi

@mintlify

mintlify Bot commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated
base 🟢 Ready View Preview Oct 15, 2026, 3:04 PM

💡 Tip: Enable Automations to automatically generate PRs for you.

@cb-heimdall

Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

This branch was successfully deployed

1 active deployment
staging - docs — 2aa4d80d Deployed Oct 15, 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.

2 participants