fix(docs): add summarizing storage page to the Swift sidebar - #646
Merged
Conversation
The page shipped ~July 14 without a sidebar entry; the starlight-sidebar-topics middleware throws for any page not attached to a topic, which has failed every docs deploy since — the live site is two weeks stale. Verified locally: full astro build now completes (111 pages). Closes #645 Code written by Claude (Fable 5), architected and approved by @cornelcroi.
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
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.
Issue Link
Closes #645
Summary
Every docs deploy on main has failed since ~July 14 with
Caught error rendering /swift/storage/built-in/summarizing. The page was added without a sidebar entry, and thestarlight-sidebar-topicsmiddleware throws for pages not attached to any topic. The live site is two weeks stale as a result — including all the MCP docs from #632–#634.Changes
docs/astro.config.mjs:Summarizingentry next to its storage siblings in the Swift topic.Verification
Full
astro buildrun locally: 111 pages, completes cleanly (was failing on this exact page before the change).Checklist
Code written by Claude (Fable 5), architected and approved by @cornelcroi.