Skip to content

fix(docs): revert homepage href prefixes causing doubled /docs/docs/ 404s - #1341

Merged
Dhravya merged 1 commit into
mainfrom
fix-docs-doubled-href-prefix
Jul 22, 2026
Merged

fix(docs): revert homepage href prefixes causing doubled /docs/docs/ 404s#1341
Dhravya merged 1 commit into
mainfrom
fix-docs-doubled-href-prefix

Conversation

@Dhravya

@Dhravya Dhravya commented Jul 22, 2026

Copy link
Copy Markdown
Member

Mintlify's client-side router intercepts same-origin anchor clicks
(including raw tags, not just its own components) and automatically
prepends /docs on navigation — that's how the unprefixed top-nav links
correctly land on /docs/company-brain/overview etc. Hardcoding /docs
into these homepage hrefs made the router double it up to
/docs/docs/company-brain/overview, a real 404 confirmed by clicking
through in a live browser. Reverted to unprefixed hrefs, matching every
other internal link in the docs and what mintlify broken-links expects.
Image src/icon/imageUrl paths are untouched — those are literal
resource fetches never touched by the router, so they still need the
explicit /docs prefix.

Co-Authored-By: Claude Sonnet 5 noreply@anthropic.com


Note

Low Risk
Docs-only link path fix with no runtime, auth, or data impact.

Overview
Fixes broken homepage navigation by changing internal hrefs on the docs landing page from /docs/... back to site-root paths (e.g. /quickstart, /company-brain/overview).

Mintlify’s client-side router already prepends /docs on same-origin link clicks, so explicit /docs prefixes were doubled to /docs/docs/... and 404’d. The update aligns the hero buttons and HeroCard links with other internal doc links and mintlify broken-links expectations.

Image imageUrl paths are unchanged — they still use /docs/images/... because those are static asset URLs, not router-handled navigation.

Reviewed by Cursor Bugbot for commit 8d05a65. Bugbot is set up for automated code reviews on this repo. Configure here.

…404s

Mintlify's client-side router intercepts same-origin anchor clicks
(including raw <a> tags, not just its own components) and automatically
prepends /docs on navigation — that's how the unprefixed top-nav links
correctly land on /docs/company-brain/overview etc. Hardcoding /docs
into these homepage hrefs made the router double it up to
/docs/docs/company-brain/overview, a real 404 confirmed by clicking
through in a live browser. Reverted to unprefixed hrefs, matching every
other internal link in the docs and what `mintlify broken-links` expects.
Image src/icon/imageUrl paths are untouched — those are literal
resource fetches never touched by the router, so they still need the
explicit /docs prefix.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 22, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
supermemory-app 8d05a65 Commit Preview URL

Branch Preview URL
Jul 22 2026, 06:05 PM

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 22, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
supermemory-mcp 8d05a65 Jul 22 2026, 06:03 PM

Dhravya commented Jul 22, 2026

Copy link
Copy Markdown
Member Author

How to use the Graphite Merge Queue

Add the label Main to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@mintlify

mintlify Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

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

Project Status Preview Updated (UTC)
supermemory 🟢 Ready View Preview Jul 22, 2026, 6:03 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@mintlify

mintlify Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

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

Project Status Preview Updated (UTC)
supermemory 🟡 Building Jul 22, 2026, 6:02 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@Dhravya
Dhravya marked this pull request as ready for review July 22, 2026 20:24
@Dhravya
Dhravya merged commit 72431e9 into main Jul 22, 2026
10 of 11 checks passed
@claude

claude Bot commented Jul 22, 2026

Copy link
Copy Markdown

Claude finished @Dhravya's task in 39s —— View job


Reviewing PR #1341

  • Get PR diff and review changes
  • Check for any bugs, security issues, or logic errors
  • Post review summary

Review Summary

Overview: Removes hardcoded /docs prefix from navigation hrefs in the docs homepage to prevent Mintlify's router from doubling the path to /docs/docs/....

Issues found: None — this looks good to ship.

The change is correct and well-scoped:

  • 7 href attributes updated from /docs/X to /X to work with Mintlify's automatic /docs prefixing
  • Image src/imageUrl paths correctly left unchanged since those are literal resource fetches that bypass the router
  • Fix is consistent with how other internal links work throughout the docs

Score: 10/10

Clean, targeted fix that resolves a real 404 bug. No bugs, no security issues, no logic errors.

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