Fix trailing slash redirects on internal links#17
Conversation
Google Search Console flagged two posts as "Page with redirect" because internal links pointed to /posts/slug instead of /posts/slug/, causing 301 redirects under our trailingSlash: true config. Fixed all internal /posts/ and /market/ links across 21 MDX files and 1 TSX component. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis PR standardizes internal link URLs by adding trailing slashes across 20+ MDX post files and a Next.js page, and substantially enriches the ChangesURL Normalization and Editorial Enrichment
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
app/market/page.tsx (1)
112-117:⚠️ Potential issue | 🟡 Minor | ⚡ Quick winIncomplete trailing-slash normalization in this page
Good fix here, but this file still links internally without trailing slashes at Line 132 and Line 231, which can reintroduce redirect hops under
trailingSlash: true.Suggested follow-up diff
- href={`/market/${p.slug}`} + href={`/market/${p.slug}/`} ... - href={`/market/startups/${s.slug}`} + href={`/market/startups/${s.slug}/`}🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@app/market/page.tsx` around lines 112 - 117, This page still contains internal <Link> hrefs that lack the required trailing slash (causing redirects with trailingSlash: true); update the internal Link components in app/market/page.tsx so all internal route href strings include a trailing slash (e.g. change "/posts/three-primitives" to "/posts/three-primitives/" and any other internal hrefs referenced in the file) ensuring every <Link> used in this module points to the trailing-slash form.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@content/posts/notion-ships-the-primitives.mdx`:
- Line 60: Replace the plain-text external references with proper Markdown
links: change the occurrences of "Orbit", "Remy", and "the Gemini app" in the
paragraph (the same sentence mentioning Pulse) to linked forms pointing to their
canonical external URLs, matching how "Pulse" is already linked; ensure link
text and punctuation are preserved and that links follow the project's
external-link style (e.g., [Orbit](https://...), [Remy](https://...), [Gemini
app](https://...)).
---
Outside diff comments:
In `@app/market/page.tsx`:
- Around line 112-117: This page still contains internal <Link> hrefs that lack
the required trailing slash (causing redirects with trailingSlash: true); update
the internal Link components in app/market/page.tsx so all internal route href
strings include a trailing slash (e.g. change "/posts/three-primitives" to
"/posts/three-primitives/" and any other internal hrefs referenced in the file)
ensuring every <Link> used in this module points to the trailing-slash form.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 4e6db6d0-5c97-4788-accd-2bec9284d654
📒 Files selected for processing (22)
app/market/page.tsxcontent/drafts/four-repos-one-filesystem.mdxcontent/market/proactive-agent-landscape.mdxcontent/posts/agent-moves-first.mdxcontent/posts/building-weekly-digest.mdxcontent/posts/chatgpt-pulse.mdxcontent/posts/every-tool-ships-an-agent.mdxcontent/posts/forty-two-percent.mdxcontent/posts/magical-agents.mdxcontent/posts/notion-ships-the-primitives.mdxcontent/posts/posthog-code.mdxcontent/posts/push-breaks-too.mdxcontent/posts/reactive-vs-proactive.mdxcontent/posts/review-agent-three-acts.mdxcontent/posts/the-genesis.mdxcontent/posts/the-prompt-cant-save-you.mdxcontent/posts/the-webhook-tax.mdxcontent/posts/the-wish-list.mdxcontent/posts/three-primitives.mdxcontent/posts/what-proactive-agents-cost.mdxcontent/posts/why-proactive-is-hard.mdxpublic/llms-full.txt
…mitives Addresses CodeRabbit review comment: Orbit, Remy, and Pulse were plain text in two paragraphs where the editorial guidelines require linked external references. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Summary
every-tool-ships-an-agentandnotion-ships-the-primitivesas "Page with redirect" — internal links pointed to/posts/slugwithout a trailing slash, causing 301 redirects under ourtrailingSlash: trueNext.js config./posts/and/market/links across 19 MDX content files, 1 draft, 1 market landscape page, and 1 TSX component (app/market/page.tsx).public/llms-full.txtandpublic/posts/*.mdto reflect the fixes.Test plan
npm run build)🤖 Generated with Claude Code