Skip to content

Fix noIndex being enabled on production builds - #825

Merged
MattiasBuelens merged 1 commit into
mainfrom
devin/1787310440-fix-no-index
Aug 21, 2026
Merged

Fix noIndex being enabled on production builds#825
MattiasBuelens merged 1 commit into
mainfrom
devin/1787310440-fix-no-index

Conversation

@MattiasBuelens

Copy link
Copy Markdown
Collaborator

Summary

Production builds were silently marked noIndex, which makes @docusaurus/plugin-sitemap skip writing sitemap.xml entirely (and adds <meta name="robots" content="noindex, nofollow"> to every page — currently live on https://optiview.dolby.com/docs/).

Cause: the workflow passes docusaurus_no_index: 0 for production, which reaches the config as the string "0", and !!'0' === true.

-  noIndex: !!process.env.DOCUSAURUS_NO_INDEX,
+  noIndex: NO_INDEX,   // ['1', 'true'].includes(DOCUSAURUS_NO_INDEX.trim().toLowerCase())

PR previews (docusaurus_no_index: 1) keep noIndex: true.

Verified with a local DOCUSAURUS_NO_INDEX=0 npm run build: build/sitemap.xml is now generated and no page contains a noindex robots tag.

Link to Devin session: https://dolby.devinenterprise.com/sessions/b3059def9f1642a3bc87a647c751cdce
Requested by: @MattiasBuelens

Co-Authored-By: Mattias Buelens <mattias.buelens@dolby.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@theoplayer-bot

This comment has been minimized.

@MattiasBuelens
MattiasBuelens merged commit 88067e6 into main Aug 21, 2026
4 checks passed
@MattiasBuelens
MattiasBuelens deleted the devin/1787310440-fix-no-index branch August 21, 2026 11:26
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.

1 participant