Skip to content

revert(web): remove Mastra documentation assistant - #215

Merged
maxktz merged 1 commit into
mainfrom
revert/docs-agent
Sep 22, 2026
Merged

maxktz merged 1 commit into
mainfrom
revert/docs-agent

Conversation

@maxktz

@maxktz maxktz commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

Reverts #213 and restores the repository to its pre-docs-agent state.

This removes the documentation assistant UI, chat and MCP routes, Mastra workspace, AI dependencies, environment requirements, scripts, and CI integration.

Validation:

  • 143 unit tests pass
  • typecheck passes
  • lint passes
  • format check passes
  • full build passes

After this PR is merged and the Vercel production deployment completes, the Mastra deployment and the MASTRA_CHAT_URL / MASTRA_CHAT_SECRET Vercel variables can be removed.


Summary by cubic

Reverts #213 and removes the Mastra documentation assistant, restoring the repository to its pre-docs-agent state. The assistant UI, chat and MCP routes, Mastra workspace, AI dependencies, environment requirements, scripts, and CI integration are all removed.

Validation: 143 unit tests pass, and typecheck, lint, format, and the full build all pass.

Migration

  • After this merges and the Vercel production deployment completes, remove the Mastra deployment and the MASTRA_CHAT_URL / MASTRA_CHAT_SECRET Vercel variables.

Written for commit 4229d86. Summary will update on new commits.

Review in cubic

Summary by CodeRabbit

  • Removed Features

    • Removed the documentation assistant from the documentation site, including chat, suggestions, and related controls.
    • Removed the documentation chat and MCP API endpoints; requests to these endpoints are no longer supported.
  • Updates

    • Documentation search and LLM text index generation continue to be available through updated integrations.
    • Web app commands now run directly through the framework’s standard tooling.

This reverts commit d9e5155, reversing
changes made to 1571540.
@vercel

vercel Bot commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated
paykit Ready Ready Preview Sep 22, 2026 4:12pm UTC

@coderabbitai

coderabbitai Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Advanced

Run ID: 5ef2e3e7-f468-4098-a3a9-b4e86e70eafa

📥 Commits

Reviewing files that changed from the base of the PR and between d9e5155 and 4229d86.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (32)
  • .env.example
  • .github/workflows/ci.yml
  • apps/docs-agent/.gitignore
  • apps/docs-agent/.mastra-project.json
  • apps/docs-agent/package.json
  • apps/docs-agent/src/env.ts
  • apps/docs-agent/src/evals/cases.ts
  • apps/docs-agent/src/evals/seed.ts
  • apps/docs-agent/src/mastra/agents/__tests__/instructions.test.ts
  • apps/docs-agent/src/mastra/agents/docs-agent.ts
  • apps/docs-agent/src/mastra/agents/instructions.ts
  • apps/docs-agent/src/mastra/index.ts
  • apps/docs-agent/src/mastra/scorers/docs-scorers.ts
  • apps/docs-agent/src/mastra/storage.ts
  • apps/docs-agent/tsconfig.json
  • apps/docs-agent/turbo.json
  • apps/web/package.json
  • apps/web/src/app/api/chat/route.ts
  • apps/web/src/app/api/mcp/route.ts
  • apps/web/src/app/api/search/route.ts
  • apps/web/src/app/llms.txt/route.ts
  • apps/web/src/components/docs/docs-assistant.tsx
  • apps/web/src/components/docs/docs-layout.tsx
  • apps/web/src/env.js
  • apps/web/src/lib/__tests__/docs-chat-request.test.ts
  • apps/web/src/lib/docs-chat-request.ts
  • apps/web/src/lib/source.ts
  • apps/web/turbo.json
  • package.json
  • pnpm-workspace.yaml
  • scripts/run-with-env.mjs
  • vitest.unit.config.ts
💤 Files with no reviewable changes (25)
  • .github/workflows/ci.yml
  • apps/docs-agent/.mastra-project.json
  • pnpm-workspace.yaml
  • apps/web/turbo.json
  • apps/docs-agent/src/mastra/index.ts
  • apps/docs-agent/src/mastra/agents/docs-agent.ts
  • apps/docs-agent/package.json
  • .env.example
  • apps/web/src/app/api/mcp/route.ts
  • apps/docs-agent/src/mastra/agents/instructions.ts
  • apps/docs-agent/src/mastra/storage.ts
  • apps/web/src/lib/tests/docs-chat-request.test.ts
  • apps/docs-agent/turbo.json
  • apps/docs-agent/.gitignore
  • apps/docs-agent/src/evals/cases.ts
  • apps/web/src/lib/docs-chat-request.ts
  • apps/web/src/env.js
  • apps/docs-agent/src/mastra/scorers/docs-scorers.ts
  • apps/web/src/components/docs/docs-assistant.tsx
  • apps/docs-agent/src/evals/seed.ts
  • apps/docs-agent/src/env.ts
  • scripts/run-with-env.mjs
  • apps/web/src/app/api/chat/route.ts
  • apps/docs-agent/src/mastra/agents/tests/instructions.test.ts
  • apps/docs-agent/tsconfig.json

Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review.


📝 Walkthrough

Walkthrough

The pull request removes the docs-agent workspace and the web documentation assistant. It deletes chat and MCP endpoints, updates search and LLM routes, and cleans related environment, build, workspace, dependency, and test configuration.

Changes

Documentation assistant removal

Layer / File(s) Summary
Remove docs-agent service and evaluation stack
apps/docs-agent/...
The docs-agent workspace, Mastra agent, storage, scorers, evaluation cases, seed script, tests, and project configuration are deleted.
Remove web assistant and API endpoints
apps/web/src/app/api/..., apps/web/src/components/docs/..., apps/web/src/lib/..., apps/web/src/env.js, apps/web/turbo.json
The chat and MCP routes, request validation, assistant UI, assistant layout state, related tests, and Mastra environment variables are removed.
Update documentation routes
apps/web/src/app/api/search/route.ts, apps/web/src/app/llms.txt/route.ts, apps/web/src/lib/source.ts
Search and LLM text handlers are created directly from the documentation source.
Update scripts and repository configuration
.env.example, .github/workflows/ci.yml, apps/web/package.json, package.json, pnpm-workspace.yaml, scripts/run-with-env.mjs, vitest.unit.config.ts
Removed environment entries, scripts, dependency wrappers, and release-age exceptions. Web dependencies and test discovery settings are also changed.

Priority: ⬇️ Low

Estimated code review effort: 4 (Complex) | ~45 minutes

Change: Other

Merge Risk: 🔵 Low · up to 4229d

The unit-test configuration now skips tests outside apps/web, which can hide regressions; merge is reasonable with owner awareness and follow-up to confirm the narrowed scope is intentional.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 4 functions across 5 files. (2 skipped: 2 … Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
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: reverting the web documentation assistant and its related Mastra integration.
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

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 4 functions across 5 files. (2 skipped: 2 unsupported.)

  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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

A rabbit hops where chat once grew
The docs-agent leaves the burrow too
Search still runs through pages bright
Old routes fade into the night
Clean scripts guide the morning light

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

@maxktz
maxktz merged commit 8db4b94 into main Sep 22, 2026
11 checks passed

This branch was successfully deployed

1 active deployment
Preview — 4229d86d Deployed Sep 22, 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