Skip to content

Split cookbooks page into common patterns and harnesses & models sections - #626

Merged
dprevoznik merged 3 commits into
mainfrom
hypeship/cookbooks-mobile-common-patterns
Sep 25, 2026
Merged

dprevoznik merged 3 commits into
mainfrom
hypeship/cookbooks-mobile-common-patterns

Conversation

@dprevoznik

@dprevoznik dprevoznik commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Summary

The cookbooks page rendered every card in one undifferentiated grid. The "common patterns" grouping only existed in the sidebar nav, which on mobile is hidden behind the navbar drawer — so mobile visitors couldn't see it was there without opening the menu.

This splits the grid into two sections, each under its own heading, matching the existing sidebar grouping in docs.json:

  • Common patterns — the 4 pattern cookbooks
  • Harnesses & models — the remaining 16 integration/model cookbooks

Card content, tags, and links are unchanged; only the grouping and headings are new.

Testing

  • mintlify dev and loaded /cookbooks at a 390px mobile viewport and desktop width — both sections render, cards keep their tag/link/image, and the search box still filters across both sections (it walks the DOM globally, not section-scoped).
  • One row in the harnesses & models grid ends up as three same-background cards, since only 4 of those 16 cards use the rarer background color — not fully avoidable without reshuffling cards for looks alone, and outside the scope of this page-visibility fix.

🤖 Generated with Claude Code


Note

Low Risk
Documentation-only layout change on the cookbooks index with no runtime or API impact.

Overview
The cookbooks landing page no longer uses a single card grid. It now has two on-page sections—Common patterns and Harnesses & models—with separate Columns blocks under each heading, mirroring the sidebar grouping in docs.json.

Common patterns holds the four pattern cookbooks (credentials, payments, Playwright fallback, captcha pausing). Harnesses & models holds the integration/model cards, including Jev Browser Use, which previously led the mixed grid. Card copy, tags, images, and links are unchanged; only layout and grouping moved.

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

…ions

The common patterns cards were only discoverable via the mobile nav
drawer sidebar grouping; the page itself showed one undifferentiated
grid. Group cards under matching headings so common patterns are
visible directly on the page on mobile without opening the navbar.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@mintlify

mintlify Bot commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

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

Project Status Preview Updated
Kernel 🟢 Ready View Preview Sep 25, 2026, 1:49 PM

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

@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: Very Low. Approved.

This change only rearranges the cookbooks landing page in cookbooks.mdx. It adds Common patterns and Harnesses & models headings and splits the existing card grid into two groups that match those headings. Card titles, copy, tags, images, and links are unchanged.

  • Blast radius is the docs page layout only.
  • No application logic, API, auth, or infrastructure changes.
  • No code owner review is required on main.

Safe to merge as a documentation layout update.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

@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: Very Low. Existing approval stands.

Re-checked this pull request after the merge from main. The pull request still changes only cookbooks.mdx: it adds Common patterns and Harnesses & models headings and splits the existing card grid into those two groups. Card titles, copy, tags, images, and links are unchanged.

  • Blast radius is the docs page layout only.
  • No application logic, API, auth, or infrastructure changes.
  • The merge brings in documentation already on main; it does not add new pull-request changes.
  • Code owner review is not required (require_code_owner_review is off, and this repo has no CODEOWNERS file).

Risk is unchanged from the earlier Very Low assessment, so the existing approval is left in place.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

@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: Very Low. Existing approval stands.

Re-checked this pull request after the latest merge from main (d6e2372). The only file that still differs from main is cookbooks.mdx. The change adds Common patterns and Harnesses & models headings and splits the existing card grid into those two groups. Card titles, copy, tags, images, and links are unchanged.

  • Blast radius is the docs page layout only.
  • No application logic, API, auth, or infrastructure changes.
  • The merge brings in changelog.mdx and .github/workflows/broken-links.yml that are already on main; those files match main and are not part of this pull request's diff.
  • Code owner review is not required (require_code_owner_review is off, and this repo has no CODEOWNERS file).

Risk is unchanged from the earlier Very Low assessment, so the existing approval is left in place.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

@dprevoznik
dprevoznik merged commit ffb06cd into main Sep 25, 2026
4 checks passed
@dprevoznik
dprevoznik deleted the hypeship/cookbooks-mobile-common-patterns branch September 25, 2026 14:17

This branch was successfully deployed

1 active deployment
staging — d6e23720 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.

2 participants