-
Notifications
You must be signed in to change notification settings - Fork 8
Reduce redirects from 985 → 664 (~33% reduction) #902
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR removes 321 outdated redirects (33% reduction from 985 to 664) by eliminating stale GitBook-era redirects and non-functional 404 redirects that no longer serve any purpose.
Key Changes:
- Removed 202 GitBook-era redirects from paths containing
/alchemy/,/enhanced-apis/, and/documentation/ - Removed 119 redirects that return 404 errors when tested against the live site
- Retained dynamic redirects with
:path*and:method*patterns as they cannot be statically tested
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
🌿 Documentation Preview
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Redirect Cleanup: Remove stale GitBook & 404 redirects
Summary
Total removed: 321 redirects (33% reduction)
What's in this PR
1. Removed GitBook-era redirects (202 removed)
These are redirects with
/alchemy/...,/enhanced-apis/..., and/documentation/...source paths, all from our GitBook days.GitBook was 3+ years ago and we've done 2 full migrations since (GitBook → ReadMe → Fern). Safe to assume any external links have either updated by now or are dead anyway.
2. Removed 404 redirects (119 removed)
Wrote a custom script to test each redirect source URL against our live site. If hitting the source returned a 404, the redirect isn't doing anything, so I removed them.
Skipped dynamic redirects (
:path*,:method*patterns) since those can't be tested statically.