Repository navigation
Render pricing from the API and fix SEO/AEO findings - #101
Conversation
The Access page option now reads "Anyone with the share link can open it, no account needed." Audience copy says share link everywhere.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
All clear! No issues remaining. 🎉 6 issues already resolved
All required CI checks passed on
Select any unchecked box below to run or rerun that agent. Full resultsIndent Review Agent
|
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d3cd4358cb
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
The rewrite pattern could start inside external URLs such as https://docs.github.com and turn them into https:/docs/.github.com. It now matches only the Spacefast origin or a root-relative /docs path at a path boundary. Pricing tables use explicit row labels, format bytes, durations and plurals, and show Custom for negotiated plans, whose credit the API now reports as null.
The Access page option now reads "Anyone with the share link can open it, no account needed." Audience copy says share link everywhere.
General access has two options, Private and Public. Share links have their own card, and the role picker only appears for public Spaces.
71e1658 to
4186ce5
Compare
Section-alias collapse from main now runs before operation-alias compression, and the budget counts named-segment rules as dynamic, as the platform does. Retired-page Markdown twins point at slashless .md files. The redirect manifests for other hosts are no longer published.
Drop has its own address. The quickstart, overview, glossary, publish page and artifact guide now send people there instead of to the dashboard.
Blume publishes .md and .mdx mirrors, so a retired page's .mdx URL also gets an HTTP redirect. The Claude artifact guide no longer says Public means anyone with the share link.
An .mdx twin rule per retired page pushed the composed Website and Docs artifact past 2,000 static redirects. Retired pages keep their .md HTTP redirect, and the routing budget now reserves the Website's exact rules.
They have no HTTP rule under the static redirect cap, so the noindex stub still forwards old links. A stub is not a page, so it carries no canonical and the SEO check skips it.
Stacks on #99; the base is
private-beta-labels. Do not merge before that label change.Docs now render prices, seats, allowances, usage rates, and worked bills from one validated build-time pricing document. Deploy the public
GET /v1/pricingendpoint before deploying these docs. CI explicitly uses the captured pricing fixture; normal builds fail on fetch or validation errors and never fall back to stored prices./docs/setup/and final trailing-slash links. The setup-index reader accepts those final links when grouping clients in navigation.HTML bytes, before → after (uncompressed; the refreshed generated snapshot also changes page content):
/docs/quickstart//docs/billing//docs/api/reference/spaces/updatespace/The serialized client payload on these pages drops from about 387 KB to 13 KB. The built-output checks cover every content page, rather than sampling these three.
Validation with Bun 1.3.11 and Node 26, with the explicit fixture:
The refreshed producer snapshot uses OpenAPI 3.1 null unions and corrects the live-capability lookup description. Strict validation and Vale now pass. The snapshot is copied wholesale, digest-verified, and consumed through the existing overlays; the Private Beta and share-link wording is preserved.
Remaining findings and limits:
Need help on this PR? Tag
@codesmith-botwith what you need.