Skip to content

refactor(server): drop raw markdown route served by Content - #354

Merged
benjamincanac merged 3 commits into
mainfrom
refactor/nuxt-server
Oct 21, 2026
Merged

benjamincanac merged 3 commits into
mainfrom
refactor/nuxt-server

Conversation

@benjamincanac

@benjamincanac benjamincanac commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

This PR removes the template's /raw/[...slug].md route. @nuxt/content already registers the same route when nuxt-llms is installed, so it was shadowing it, and without it the production build serves /raw/getting-started/installation.md and /raw/essentials/markdown-syntax.md byte for byte like the live template. minimark goes with it, nothing else used it.

That was the template's last server handler on h3, so there is nothing left to move to nuxt/server. Content's own route gets Nuxt 5 support in nuxt/content#3891.

Summary by CodeRabbit

  • Removed Features

    • Removed the raw Markdown endpoint for serving documentation pages.
    • Requests to the former raw Markdown URLs are no longer processed through this endpoint.
  • Chores

    • Removed an unused package dependency from the project configuration.

@vercel

vercel Bot commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs-template Ready Ready Preview Oct 21, 2026 8:41am UTC

Request Review

@coderabbitai

coderabbitai Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 5459185a-cab7-45b3-b54f-443dc4860c56
📥 Commits

Reviewing files that changed from the base of the PR and between 393cc93 and 6e5a376.

⛔ Files ignored due to path filters: 1
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing: 2
  • package.json
  • server/routes/raw/[...slug].md.get.ts
💤 Files with no reviewable changes: 2
  • package.json
  • server/routes/raw/[...slug].md.get.ts

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.

📝 Walkthrough

The raw Markdown endpoint was deleted, and the minimark dependency was removed from package.json.

Changes: Raw Markdown endpoint removal

Layer / File(s) Summary
Remove raw Markdown serving
server/routes/raw/[...slug].md.get.ts, package.json
The raw Markdown route was deleted, including slug validation, documentation lookup, 404 handling, metadata injection, and Markdown response generation. The minimark dependency was removed.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 6e5a3

The template retains raw Markdown access through the configured content integration. No actionable issue remains before merge.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly identifies the main change: removing the server raw Markdown route because Content serves the route.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files.

✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@benjamincanac benjamincanac changed the title refactor(server): import from nuxt/server refactor(server): drop raw markdown route served by Content Oct 21, 2026
@benjamincanac
benjamincanac marked this pull request as ready for review October 21, 2026 08:45
@benjamincanac
benjamincanac merged commit bb8d41e into main Oct 21, 2026
4 checks passed
@benjamincanac
benjamincanac deleted the refactor/nuxt-server branch October 21, 2026 09:00

This branch was successfully deployed

1 active deployment
Preview — 6e5a3765 Deployed Oct 21, 2026 by vercel[bot]
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.

1 participant