Skip to content

Generate static network variable tabs#663

Merged
hrischuk-da merged 2 commits into
mainfrom
mintlify-tabbed-snippet-search
May 29, 2026
Merged

Generate static network variable tabs#663
hrischuk-da merged 2 commits into
mainfrom
mintlify-tabbed-snippet-search

Conversation

@danielporterda
Copy link
Copy Markdown
Contributor

@danielporterda danielporterda commented May 29, 2026

Summary

Fixes copy paste issue #654. May fix search issue #657.

Technical summary

Moves searchable network-variable tab content out of runtime-only NetworkVariables expansion and into generated static Mintlify <Tabs>/<Tab> blocks.

  • adds scripts/generate_network_variable_tabs.py and npm run generate:network-variable-tabs
  • adds scripts/validate_network_variable_tabs.py and npm run validate:network-variable-tabs for merge protection against stale generated MDX
  • adds a GitHub Actions PR check, Validate network variable tabs, that runs the merge-protection validator
  • bootstraps editable source snippets under docs-main/snippets/networkvars
  • rewrites 16 authored pages with NETWORKVARS_START/END markers around generated native tabs
  • removes the now-unused runtime NetworkVariables component export
  • adds focused tests for token substitution, imported snippet inlining, indentation preservation, generated-page shape, and stale-output validation

@mintlify
Copy link
Copy Markdown
Contributor

mintlify Bot commented May 29, 2026

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

Project Status Preview Updated (UTC)
cantonfoundation 🟢 Ready View Preview May 29, 2026, 2:52 PM

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

@danielporterda danielporterda force-pushed the mintlify-tabbed-snippet-search branch from 2012d9f to 41ea354 Compare May 29, 2026 15:26
@danielporterda danielporterda marked this pull request as draft May 29, 2026 15:27
@danielporterda danielporterda marked this pull request as ready for review May 29, 2026 15:30
@hrischuk-da
Copy link
Copy Markdown
Contributor

@danielporterda this seems to work. What exactly was the fix? Is there a need to rebuild stuff based on some Git action?

@hrischuk-da hrischuk-da merged commit f9e7640 into main May 29, 2026
2 checks passed
@hrischuk-da hrischuk-da deleted the mintlify-tabbed-snippet-search branch May 29, 2026 16:08
@da-tanabe da-tanabe mentioned this pull request May 29, 2026
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.

4 participants