Skip to content

ENG-1632 remove legacy markdown parsing for migrated routes#956

Merged
mdroidian merged 3 commits intomainfrom
eng-1632-remove-legacy-markdown-parsing-for-migrated-routes
Apr 12, 2026
Merged

ENG-1632 remove legacy markdown parsing for migrated routes#956
mdroidian merged 3 commits intomainfrom
eng-1632-remove-legacy-markdown-parsing-for-migrated-routes

Conversation

@mdroidian
Copy link
Copy Markdown
Member

@mdroidian mdroidian commented Apr 12, 2026


Open with Devin

Summary by CodeRabbit

  • Revert
    • Removed the documentation system and all related features from the website application. This includes documentation pages, page layout and navigation features, previous and next page link navigation, dynamic in-page table of contents rendering, and markdown content processing capabilities. Removed all related documentation dependencies and build tools. Package version updated to 0.2.0.

…dencies

- Deleted unused components related to documentation, including DocsHeader, DocsLayout, DocsPage, DocsRedirect, PrevNextLinks, Prose, TableOfContents, and associated utility functions.
- Removed outdated dependencies from package.json to streamline the project and improve maintainability.
@linear-code
Copy link
Copy Markdown

linear-code Bot commented Apr 12, 2026

@supabase
Copy link
Copy Markdown

supabase Bot commented Apr 12, 2026

This pull request has been ignored for the connected project zytfjzqyijgagqxrzbmz because there are no changes detected in packages/database/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@mdroidian
Copy link
Copy Markdown
Member Author

@coderabbitai full review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 12, 2026

✅ Actions performed

Full review triggered.

devin-ai-integration[bot]

This comment was marked as resolved.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 12, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 628f9d3f-5817-466b-b2fa-8bcaf78d731a

📥 Commits

Reviewing files that changed from the base of the PR and between fe6a8cd and 0cf32d6.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (14)
  • apps/website/app/components/DocsHeader.tsx
  • apps/website/app/components/DocsLayout.tsx
  • apps/website/app/components/DocsPage.tsx
  • apps/website/app/components/DocsRedirect.tsx
  • apps/website/app/components/PrevNextLinks.tsx
  • apps/website/app/components/Prose.tsx
  • apps/website/app/components/TableOfContents.tsx
  • apps/website/app/types/schema.tsx
  • apps/website/app/utils/getFileContent.ts
  • apps/website/app/utils/getFileMetadata.ts
  • apps/website/app/utils/getHtmlFromMarkdown.ts
  • apps/website/app/utils/getProcessedMarkdownFile.ts
  • apps/website/app/utils/getSections.ts
  • apps/website/package.json
💤 Files with no reviewable changes (13)
  • apps/website/app/components/DocsRedirect.tsx
  • apps/website/app/types/schema.tsx
  • apps/website/app/utils/getFileContent.ts
  • apps/website/app/utils/getFileMetadata.ts
  • apps/website/app/components/DocsHeader.tsx
  • apps/website/app/components/PrevNextLinks.tsx
  • apps/website/app/components/DocsLayout.tsx
  • apps/website/app/utils/getProcessedMarkdownFile.ts
  • apps/website/app/components/Prose.tsx
  • apps/website/app/utils/getSections.ts
  • apps/website/app/utils/getHtmlFromMarkdown.ts
  • apps/website/app/components/TableOfContents.tsx
  • apps/website/app/components/DocsPage.tsx

📝 Walkthrough

Walkthrough

This pull request removes the entire documentation infrastructure from the website, including components for rendering docs pages, navigation, table of contents, markdown processing utilities, content types, and related dependencies from the package manifest.

Changes

Cohort / File(s) Summary
Documentation Components
apps/website/app/components/DocsHeader.tsx, DocsLayout.tsx, DocsPage.tsx, DocsRedirect.tsx, PrevNextLinks.tsx, Prose.tsx, TableOfContents.tsx
Removed 7 React/Next.js components that collectively handled docs page rendering, header display, navigation, table of contents with scroll tracking, and prose styling.
Content Schema and Types
apps/website/app/types/schema.tsx
Removed Zod schema definitions and TypeScript types for page frontmatter (PageFrontmatter), heading nodes, and content organization structures (Section, Subsection).
Markdown and File Processing Utilities
apps/website/app/utils/getFileContent.ts, getFileMetadata.ts, getHtmlFromMarkdown.ts, getProcessedMarkdownFile.ts, getSections.ts
Removed 5 utility functions that collectively handled file reading, frontmatter parsing, markdown-to-HTML conversion using remark/rehype pipelines, and extraction of content sections from HTML.
Package Configuration
apps/website/package.json
Updated version from 0.1.0 to 0.2.0 and removed markdown/docs processing dependencies (remark-*, rehype-*, unified, @sindresorhus/slugify, mdast-util-to-string, and related TypeScript type definitions).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Suggested labels

codex

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title clearly and specifically describes the main change: removing legacy markdown parsing infrastructure for documentation routes that have been migrated elsewhere.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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 and usage tips.

@mdroidian mdroidian merged commit c316fe9 into main Apr 12, 2026
7 checks passed
@mdroidian mdroidian deleted the eng-1632-remove-legacy-markdown-parsing-for-migrated-routes branch April 12, 2026 04:49
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