revert(web): remove Mastra documentation assistant - #215
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
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 configurationConfiguration used: Repository UI Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (32)
💤 Files with no reviewable changes (25)
Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review. 📝 WalkthroughWalkthroughThe 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. ChangesDocumentation assistant removal
Priority: ⬇️ Low Estimated code review effort: 4 (Complex) | ~45 minutes Change: Other Merge Risk: 🔵 Low · up to 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)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation 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.)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
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. A rabbit hops where chat once grew Comment |
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:
After this PR is merged and the Vercel production deployment completes, the Mastra deployment and the
MASTRA_CHAT_URL/MASTRA_CHAT_SECRETVercel 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
MASTRA_CHAT_URL/MASTRA_CHAT_SECRETVercel variables.Written for commit 4229d86. Summary will update on new commits.
Summary by CodeRabbit
Removed Features
Updates