Skip to content

Redirect /docs/index to /docs - #65

Closed
devin-ai-integration[bot] wants to merge 4 commits into
mainfrom
devin/1787260274-docs-index-redirect
Closed

Redirect /docs/index to /docs#65
devin-ai-integration[bot] wants to merge 4 commits into
mainfrom
devin/1787260274-docs-index-redirect

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Summary

index.mdx is served at both /docs and /docs/index, so https://www.ravion.com/docs/index returns 200 with the same content as /docs — a duplicate URL that the Lightsage site audit picked up as a crawled page absent from every sitemap (/docs/sitemap.xml only lists /docs).

Adds a permanent redirect to the existing redirects array so the duplicate collapses onto the canonical URL:

{"source": "/index", "destination": "/", "permanent": true}

No local build verifies this one (Mintlify builds on push); docs.json parses.

The other half of the audit finding is in ravionhq/flightcontrol#7054, which makes /sitemap.xml a sitemap index pointing at /docs/sitemap.xml so the ~330 docs URLs are discoverable without relying on the Sitemap: lines in robots.txt.

Link to Devin session: https://app.devin.ai/sessions/6e3743a68c8e47128937c6e91f46f8bb
Requested by: @flybayer

Greptile Summary

Adds a permanent Mintlify redirect that canonicalizes /docs/index to /docs.

  • Adds /index as the docs-root-relative redirect source.
  • Uses / as the docs-root-relative destination.
  • Marks the redirect as permanent.

Confidence Score: 5/5

The PR appears safe to merge because the redirect correctly maps the duplicate docs index route to the canonical documentation root.

The source and destination are interpreted relative to the Mintlify docs root, so /docs/index redirects to /docs without changing unrelated routes.

Important Files Changed

Filename Overview
docs.json Adds a correctly scoped permanent redirect from the duplicate index route to the documentation root.

Reviews (1): Last reviewed commit: "Redirect docs index to root" | Re-trigger Greptile

Co-Authored-By: brandon <brandon@flightcontrol.dev>
@mintlify

mintlify Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
Ravion 🟢 Ready View Preview Aug 20, 2026, 9:19 PM

@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

🤖 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

Co-Authored-By: brandon <brandon@flightcontrol.dev>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

Closing: the redirect breaks the docs root. On the Mintlify preview, /index redirects to / but / also 308s to / — an infinite loop, since Mintlify normalizes / to /index and the redirect matches its own destination. No fix is needed anyway: Mintlify already emits <link rel="canonical" href="https://www.ravion.com/docs"> on /docs/index, so the duplicate is correctly canonicalized and only the audit crawler treated it as a separate page. The repo's SEO invariant check (redirect source must not be a live MDX page) was right to reject this.

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