Skip to content

docs(agent-tars): fix showcase not found#1708

Merged
ulivz merged 12 commits intomainfrom
fix/restore-showcase-redirects
Oct 10, 2025
Merged

docs(agent-tars): fix showcase not found#1708
ulivz merged 12 commits intomainfrom
fix/restore-showcase-redirects

Conversation

@ulivz
Copy link
Member

@ulivz ulivz commented Oct 10, 2025

Summary

Fix 404 error for /showcase/analyze-google-network-request-ea86c5 caused by upgrading rspress in PR #1495

Before

image

After

image

Checklist

  • Added or updated necessary tests (Optional).
  • Updated documentation to align with changes (Optional).
  • Verified no breaking changes, or prepared solutions for any occurring breaking changes (Optional).
  • My change does not involve the above items.

@netlify
Copy link

netlify bot commented Oct 10, 2025

Deploy Preview for tarko ready!

Name Link
🔨 Latest commit e18e1f2
🔍 Latest deploy log https://app.netlify.com/projects/tarko/deploys/68e877fe07fcda0008ad87c7
😎 Deploy Preview https://deploy-preview-1708--tarko.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Oct 10, 2025

Deploy Preview for agent-tars-docs ready!

Name Link
🔨 Latest commit e18e1f2
🔍 Latest deploy log https://app.netlify.com/projects/agent-tars-docs/deploys/68e877fe07fcda0008ad87c5
😎 Deploy Preview https://deploy-preview-1708--agent-tars-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

ulivz added 2 commits October 10, 2025 09:11
- Remove @rspress/plugin-client-redirects dependency and usage
- Rely on netlify.toml for redirects instead
- Fixes issue where empty redirects[] array was overriding netlify.toml
- Simplifies configuration by using single source of truth for redirects
@ulivz ulivz changed the title fix(docs): restore showcase redirects in rspress config docs: fix showcase not found Oct 10, 2025
ulivz added 7 commits October 10, 2025 09:24
- Add status = 200 to /showcase/* and /replay/* redirects
- This ensures proper SPA routing behavior on Netlify
- Without status 200, Netlify treats as 301/302 redirect instead of rewrite
- Add debug logs to identify why showcase routes not rendering
- Check pathname and routing logic in theme/index.tsx
- Add more detailed path logging to identify routing issues
- Clarify that showcase/replay routes should override rspress routing
- This will help diagnose why dynamic URLs show 404
- Set ssg: false in rspress.config.ts
- This allows our theme Layout to handle dynamic showcase/replay routes
- Previously SSG was pre-rendering 404 pages for non-existent paths
- Remove pageType: custom from showcase and replay pages
- This was causing issues with rspress v2.0.0-beta.31
- Now relying on theme Layout to handle routing
@ulivz ulivz force-pushed the fix/restore-showcase-redirects branch from 2d00cae to 1d34551 Compare October 10, 2025 02:37
@ulivz ulivz merged commit 06a3df7 into main Oct 10, 2025
10 checks passed
@ulivz ulivz deleted the fix/restore-showcase-redirects branch October 10, 2025 03:17
@ulivz ulivz changed the title docs: fix showcase not found docs(agent-tars): fix showcase not found Oct 10, 2025
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.

2 participants

Comments