chore: fix broken links on the home page and the OpenAPI specification page - #1941
Merged
dinwwwh merged 1 commit intoAug 24, 2026
Merged
Conversation
…n page The home page "Runs on" strip still linked to /docs/adapters/react-native, removed when the adapter page became expo.mdx, and the OpenAPI specification page linked its source code at github.com/orpc/orpc instead of middleapi/orpc. Both returned 404.
More templates
@orpc/ai-sdk
@orpc/arktype
@orpc/bun
@orpc/client
@orpc/cloudflare
@orpc/contract
@orpc/experimental-effect
@orpc/evlog
@orpc/hibernation
@orpc/json-schema
@orpc/experimental-msw
@orpc/nest
@orpc/next
@orpc/node
@orpc/openapi
@orpc/opentelemetry
@orpc/pinia-colada
@orpc/pino
@orpc/publisher
@orpc/ratelimit
@orpc/server
@orpc/shared
@orpc/swr
@orpc/tanstack-query
@orpc/trpc
@orpc/valibot
@orpc/zod
commit: |
![pullfrog[bot]](https://avatars.githubusercontent.com/in/1768019?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes — reviewed both changed files in full against main.
pages/_home/Runtimes.astro— the "Runs on" strip now links/docs/adapters/expounder the "Expo" label, replacing the dead/docs/adapters/react-nativelink. Verified the Expo page exists and that the renamed page left no dangling references elsewhere inapps/content.docs/openapi/specification.mdx— the "source code" link is re-pointed fromgithub.com/orpc/orpctogithub.com/middleapi/orpc, matching the repo's actual org (all other doc links already use it) and the resolved targetpackages/openapi/src/meta.ts.
Both links verified against the working tree. Two minor observations, neither requiring action:
- The author's note about leaving the dead sponsor link at
blog/v1-announcement.mdx:133(deleted GitHub account) is a reasonable call — keeping the attribution for a real person beats guessing a handle. Worth a quick second opinion from the maintainers, but not a blocker. - One thing to watch: the sidebar label for the Expo page (
label: "Expo"inexpo.mdx) matches the new home-page entry, so the wording is consistent.
DeepSeek Flash (free via Pullfrog for OSS) ｜ 𝕏
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Contributor
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
1 participant
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.
Two links in the docs site returned 404. The home page "Runs on" strip still pointed at the React Native adapter page, which was removed in 6a7f785 when it became the Expo adapter page, and the OpenAPI specification page linked its source code under
github.com/orpc/orpcinstead ofmiddleapi/orpc.Fixes
/docs/adapters/expo, matching the doc's sidebar label.Note that the live orpc.dev deploy predates the Expo rename and still serves the dead link, so this takes effect on the next deploy.
Testing
blume validate --strictreports no broken links.apps/content(943 internal, 419 external) found no other broken page, anchor, or asset. All 45 config redirect targets resolve and none shadow a real page. The checker was itself validated against injected failures of each kind.registry.npmjs.org.For the reviewer
One dead link is deliberately left untouched:
blog/v1-announcement.mdx:133linksgithub.com/Stijn-Timmer, whose account no longer exists (404 on both the page and the API, with no rename redirect). It's a sponsor thank-you credit for a real person, so it seemed better to leave the attribution than to guess a new handle. Happy to keep the name and drop the link if preferred.