Make headings start at h2 and sync the AI Skills page - #65
Merged
Merged
Conversation
48 pages started at h3 under the page title and three skipped a level; body headings now start at h2 with siblings kept level. Bold-only lines that worked as section titles are headings, bold prose links are plain, and a new check-markup script runs in npm run check so this doesn't drift back. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01H5QjNaRFvguF2WDVyWwwtw
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
developer-docs | 7bac6fe | Commit Preview URL Branch Preview URL |
Sep 30 2026, 09:05 AM |
Also renames manifest.mdx's first section, which repeated the page title. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01H5QjNaRFvguF2WDVyWwwtw
Adds next-theme-design, refreshes every summary from skills.json, and brings the install, update and prerequisite notes in line with the repo's README (guided ./skills.sh installer, npx -g/--skill flags, Admin API token in .env). The capability map lists next-theme-design alongside the other theme skills. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01H5QjNaRFvguF2WDVyWwwtw
next-devin
changed the title
![kilo-code-bot[bot]](https://avatars.githubusercontent.com/in/2193792?s=60&v=4){kind=small}
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Overview
Both findings from the previous review (commit
No new issues found in the incremental diff. Files Reviewed (2 files changed since last review)
Previous Review Summary (commit 6ba2b78)Current summary above is authoritative. Previous snapshots are kept for context only. Previous review (commit 6ba2b78)Status: 2 Issues Found | Recommendation: Address before merge Overview
Issue Details (click to expand)WARNING
SUGGESTION
Files Reviewed (62 files)
Reviewed by minimax-m3 · Input: 0 · Output: 0 · Cached: 0 |
…t heading for a title repeat Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01H5QjNaRFvguF2WDVyWwwtw
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Requested by Devin · project thread
Before: 48 pages (most Admin API guides, the payment-method guides, themes and apps) began at h3 right under the page title, so their first section looked smaller than on other pages and the TOC was indented one step. App Kit and OAuth skipped levels (h2 to h4, h3 to h5), and about 20 section titles were bold paragraphs that never reached the TOC (subscription management, custom page/product templates, API versions,
session.price). The AI Skills page was missingnext-theme-design, and several summaries and the install notes had drifted from NextCommerceCo/skills.After: every authored page's body starts at h2 with no skipped levels, sibling headings stay at the same level, those bold labels are real headings, and bold prose links are plain like the rest of the site. The AI Skills page lists all nine skills with summaries taken from
skills.json. Its install, update and prerequisite notes now match the skills repo README: the guided./skills.shinstaller, thenpx skills add -g --skillflags, and the Admin API token kept in.env. The capability map listsnext-theme-designwith the other theme skills.How: same normaliser and
scripts/check-markup.mjsas docs#50, reading git-tracked pages only (generated reference trees and the committed GraphQL output are skipped). It runs innpm run checkand fails on bold or empty headings, skipped levels, a first heading that repeats the title, and unbalanced**in link text. The fulfillment-order snippets move from h4 to h3 to match the page they're imported into. The AI Skills table keeps its bold names becausegenerate-capability-map.mjsreads them. Anchors follow heading text, so no links move, exceptmanifest.mdx, whose first section repeated the page title and is now "Example manifest.json".Checked locally:
check-markup,check-agent-surfaces,check-frontmatter,generate-capability-map.validate-linksreports 19 missing/docs/storefront/graphql/...pages here because this sandbox can't reach the GraphQL schema to generate them;mainshows the identical 19, and CI generates them.🤖 Generated with Claude Code
https://claude.ai/code/session_01H5QjNaRFvguF2WDVyWwwtw