docs: fix dead link and anchors, tidy TikTok image filenames - #183
Open
CodeFire98 wants to merge 1 commit into
Open
docs: fix dead link and anchors, tidy TikTok image filenames#183CodeFire98 wants to merge 1 commit into
CodeFire98 wants to merge 1 commit into
Conversation
One genuinely broken link, two dead anchors, and some link/filename hygiene. Real breakage: - `features/mcp.mdx`: `/api-reference/introduction` 404s (no such page, no redirect). Now points at `/api-reference/data-apis`, matching the link text. - `sdk/expo.mdx`, `sdk/flutter.mdx`: `#backup-configuration` does not exist in `sdk/android.mdx`, so the link landed at the top of the page. The heading is `### Step 3: Backup Configuration`, so the anchor is `#step-3%3A-backup-configuration` (matching how `features/remarketing.mdx` and `testing/integration-testing.mdx` already write it). Hygiene, these already resolved via redirects: - `features/deferred-deep-linking.mdx`: `/sdk/<platform>/usage` -> `/sdk/<platform>`, skipping the redirect hop. - `features/skadnetwork-conversion-values.mdx`: `/sdk/skadnetwork/integration` -> `/features/skadnetwork-integration`. Hygiene, these were never broken: - TikTok screenshots were named `image NN.png` with a literal space and referenced as `%20`, which resolves correctly. Renamed to `image-NN.png` per the lowercase-hyphens convention in CLAUDE.md. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
CodeFire98
force-pushed
the
fix/broken-links
branch
from
August 13, 2026 13:26
b46581a to
8282210
Compare
CodeFire98
marked this pull request as ready for review
August 13, 2026 13:27
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Rebased onto
main(ad5f9cb). Fixes one genuinely broken link and two dead anchors, plus some link/filename hygiene.Correction to the original framing
I opened this claiming it fixed "29 broken links." That count was wrong, and the reason is worth knowing.
This repo's
node_moduleshas mintlify 4.0.592 pinned, whilenpx mintlifypulls 4.2.800. They disagree badly:main4.0.592(repo'snode_modules)4.2.800(npx, current)4.0.592 false-positives on percent-encoded image paths and on any path that resolves through a
docs.jsonredirect. I verified each case by hand againstdocs.jsonand the target headings rather than trusting either tool. Only 3 links were actually doing the wrong thing.package.jsondeclares"mintlify": "latest", so whoever installed last froze an old version into the lockfile. Worth a separate bump.Actually broken
features/mcp.mdx—/api-reference/introduction404s. No such page, no redirect covering it. Now points at/api-reference/data-apis, which matches the link text ("Linkrunner data API").sdk/expo.mdx,sdk/flutter.mdx—/sdk/android/installation#backup-configuration. The path redirects fine, but#backup-configurationdoes not exist insdk/android.mdx; the heading is### Step 3: Backup Configuration. So the link silently dumped you at the top of the Android page. Now/sdk/android#step-3%3A-backup-configuration, matching howfeatures/remarketing.mdx,features/remarketing-guide.mdx, andtesting/integration-testing.mdxalready write that same anchor.Not broken, cleaned up anyway
features/deferred-deep-linking.mdx— the five/sdk/<platform>/usage#getting-attribution-datalinks resolve via redirects indocs.json, and the fragment survives the hop. Pointed directly at/sdk/<platform>#getting-attribution-datato drop the redirect.features/skadnetwork-conversion-values.mdx—/sdk/skadnetwork/integrationlikewise redirects to/features/skadnetwork-integration. Now direct.image NN.pngwith a literal space, referenced as%20, which percent-decodes correctly and works in production. Renamed toimage-NN.pngper the lowercase-hyphens rule inCLAUDE.md, since spaces in asset filenames are a trap waiting to spring. No screenshots need recapturing.Verification
npx mintlify broken-links(4.2.800) →no broken links foundnpm run check:routes→86 pages, no unredirected removalsCaveat worth knowing:
mintlify broken-linksvalidates paths but not anchors, in either version. I confirmed this by pointing a link at a deliberately bogus anchor and the check still passed. That is exactly why the#backup-configurationbug survived this long. So every anchor here was checked by hand against the heading in the target file:## Getting Attribution Dataexists insdk/android.mdx,sdk/ios.mdx,sdk/react-native.mdx,sdk/flutter.mdx### Step 3: Backup Configurationexists insdk/android.mdx:59No overlap with #185 — that PR touches
docs.jsonandfeatures/{users,events}-tab.mdx, none of which this one changes. They can merge in either order.🤖 Generated with Claude Code