Skip to content

fix(docs): prefix raw internal href/src literals with /docs - #1333

Merged
Dhravya merged 1 commit into
mainfrom
fix-docs-internal-hrefs
Jul 22, 2026
Merged

fix(docs): prefix raw internal href/src literals with /docs#1333
Dhravya merged 1 commit into
mainfrom
fix-docs-internal-hrefs

Conversation

@Dhravya

@Dhravya Dhravya commented Jul 22, 2026

Copy link
Copy Markdown
Member

index.mdx uses mode: "custom", which renders raw HTML instead of
Mintlify's own components — Mintlify only auto-prefixes internal links
it generates itself (nav, Card href), not literal href/src strings in
raw / tags or JS string literals, so those still 404 under the
/docs mount. Fixed the homepage's raw links and the hardcoded
avatar path literals in the shared Slack-message snippet component.

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


Note

Low Risk
Documentation-only URL path fixes with no runtime or security impact.

Overview
Fixes broken navigation and images on the docs homepage and in Slack UI snippets when the site is served under the /docs mount.

Because index.mdx uses mode: "custom", Mintlify does not rewrite literal href values on raw <a> tags or HeroCard props—those links now use paths like /docs/quickstart and /docs/overview/what-is-supermemory instead of root-relative paths that 404.

The shared slack-message snippet updates hardcoded avatar src strings (including DM fallbacks and SlackMessage defaults) from /images/... to /docs/images/... so assets resolve correctly.

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

index.mdx uses `mode: "custom"`, which renders raw HTML instead of
Mintlify's own components — Mintlify only auto-prefixes internal links
it generates itself (nav, Card href), not literal href/src strings in
raw <a>/<img> tags or JS string literals, so those still 404 under the
/docs mount. Fixed the homepage's raw <a href> links and the hardcoded
avatar path literals in the shared Slack-message snippet component.

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 ec1817e Commit Preview URL

Branch Preview URL
Jul 22 2026, 03:50 AM

@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 ec1817e Jul 22 2026, 03:48 AM

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, 3:48 AM

💡 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, 3:46 AM

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

@Dhravya
Dhravya marked this pull request as ready for review July 22, 2026 07:40
@Dhravya
Dhravya merged commit 90f709b 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 54s —— View job


PR Review Complete

  • Load review tools
  • Fetch PR diff and existing review comments
  • Analyze changes for bugs, security issues, and logic errors
  • Submit review summary

Overview: This PR fixes 404 errors on the docs homepage by adding the /docs prefix to raw internal href/src literals that Mintlify doesn't auto-prefix (since index.mdx uses mode: "custom" for raw HTML rendering).

Issues found: None — this looks good to ship.

The changes are straightforward and correct:

  • apps/docs/index.mdx: 7 internal hrefs prefixed with /docs
  • apps/docs/snippets/slack-message.mdx: 5 image paths prefixed with /docs

All paths are consistently updated, and the Mintlify preview deployment confirms the fix works.

Score: 10/10

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