Skip to content

fix: docs sitemap fix - #525

Open
victormeirelests wants to merge 8 commits into
mainfrom
chore/sitemap-fix
Open

victormeirelests wants to merge 8 commits into
mainfrom
chore/sitemap-fix

Conversation

@victormeirelests

@victormeirelests victormeirelests commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Describe your changes

Adding sitemap to docs website to fix Docs sitemap: 0 of 729 have <lastmod> either — and docs are your stated H2 bet (make docs the source AI learns from), so freshness signals matter most there.

Affected pages

No content pages affected by this PR. Open the preview to confirm the build deployed successfully.

Related Issues

If there is a related issue, please add it below (just put the number after the # below, and GitHub will automatically create a link):

Issue: #number

Checklist before requesting a review

  • - This is ready for review. If not, raise as a draft PR
  • - I have reviewed my changes.
  • - I have confirmed the content is technically accurate.
  • - I have tested any code that is added or updated on the latest available version.
  • - I have confirmed the content is free of typos or grammar errors.
  • - I have verified all images and videos are clear and match production (or dev for unreleased features).
  • - I have checked the component kitchen sink for any visual breakage (only needed if you changed components, styles, or dependencies). A link to this PR's own preview appears under Affected pages once the build deploys.
  • - This references a feature that is public. If not, add a note and we can schedule the merge for after the feature release.

@victormeirelests
victormeirelests requested a review from a team September 11, 2026 15:01
@vercel

vercel Bot commented Sep 11, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
tiger-data-docs Ready Ready Preview Sep 17, 2026 1:12pm UTC

@github-actions

Copy link
Copy Markdown

✅ Checklist Complete

Thank you for taking the time to properly review your PR! All checklist items are complete.

@CLAassistant

CLAassistant commented Sep 11, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@erinmikailstaples

Copy link
Copy Markdown
Collaborator

Hey @victormeirelests can you please accept the CLA please?

@erinmikailstaples erinmikailstaples left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@erinmikailstaples erinmikailstaples left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Accept CLA, make sure this deploys properly.

The build was failing because starlight-llms-txt couldn't render React
components like Card from @stainless-api/docs/mintlify-compat. The issue
was that the astroContainer in starlight-llms-txt only had the MDX renderer,
not the React renderer.

The fix patches starlight-llms-txt to:
1. Import and register the React server-side renderer
2. Provide a stub starlightRoute in the rendering context so Starlight
   components can access the required context without throwing

This allows pages with Card components to render successfully for LLM
text extraction.
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
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.

3 participants