Conversation
- Replace Vexor semantic search with Probe (MCP agent + CLI) across entire codebase - MCP (search_code/extract_code) is primary search tool, CLI is secondary fallback - Remove ~300 lines of Vexor install code (MLX/CUDA/model management), add simple npm install - Replace VexorStatus dashboard widget with UsageSummary (daily/monthly costs) - Remove VexorRoutes API endpoints from Console worker service - Update all rules, commands, hooks, installer, docs site, README, and uninstall script - Rebuild generated bundles (worker-service.cjs, viewer-bundle.js) - Install via: npm install -g @probelabs/probe - All 1128 Python tests and 1049 Console tests pass
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
🚧 Files skipped from review as they are similar to previous changes (1)
WalkthroughThe PR replaces Vexor with Probe across docs and installer, swaps the Vexor MLX installation flow for a simplified Probe npm-based install, removes several interactive Console methods, updates uninstall/finalize messaging, and adjusts tests to the Probe-focused flows. Changes
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Possibly related PRs
Suggested labels
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 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. Comment |
|
Claude finished @maxritter's task in 2m 5s —— View job Code Review: Replace Vexor with Probe
Overall this is a clean, well-scoped migration. The simplification from ~300 lines of Vexor MLX/CUDA/model-management code to ~20 lines of Issues1. README.md —
-Start a task, grab a coffee, come back to production-grade code.</br>
+Start a task, grab a coffee, come back to production-grade code.`</br>`The 2. README.md — two more whitespace regressions
-| Same as above, with optional flags (e.g., `--skip-update-check`) |
+| Same as above, with optional flags (e.g.,`--skip-update-check`) | # missing space before backtick-community support via [GitHub Issues][gh-issues]
+community support via[GitHub Issues][gh-issues] # missing space before linkNeither is critical but both look like artifacts from a formatting pass. 3.
result = subprocess.run(
["npm", "list", "-g", "@probelabs/probe", "--depth=0"],
capture_output=True,
text=True,
# no timeout!
)All other 4.
5. -steps.append(("👨✈️ Launch Pilot Shell", "Run: pilot (in your project folder)"))
+steps.append(("Launch Pilot Shell", "Run: pilot (in your project folder)"))This is a cosmetic change unrelated to the Vexor→Probe swap, and not mentioned in the PR description. Not a blocker, but it changes the installer's post-install UX appearance. Positives
|
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/site/src/pages/docs/SyncSection.tsx`:
- Around line 12-14: Update the explanatory paragraph in the SyncSection
component to match the Phase 2 entry that states "no indexing required": locate
the SyncSection.tsx component (where the phases array and the explanatory text
live) and change the sentence that says "build a semantic search index" (around
the paragraph referenced) to instead reflect Probe's no-indexing workflow—for
example, remove the phrase or replace it with wording like "use Probe without
building a semantic search index" so the paragraph aligns with the action "Check
Probe availability (no indexing required)" in the phases list.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 68516205-02f2-4364-8fb7-9d8f59499ea7
⛔ Files ignored due to path filters (49)
console/src/services/worker-service.tsis excluded by!console/**console/src/services/worker/http/routes/TeamsRoutes.tsis excluded by!console/**console/src/services/worker/http/routes/VexorRoutes.tsis excluded by!console/**console/src/ui/viewer/hooks/useStats.tsis excluded by!console/**console/src/ui/viewer/hooks/useTeams.tsis excluded by!console/**console/src/ui/viewer/views/Dashboard/UsageSummary.tsxis excluded by!console/**console/src/ui/viewer/views/Dashboard/VexorStatus.tsxis excluded by!console/**console/src/ui/viewer/views/Dashboard/index.tsxis excluded by!console/**console/src/ui/viewer/views/Teams/index.tsxis excluded by!console/**console/tests/ui/search-removal.test.tsis excluded by!console/**console/tests/ui/vexor-status-widget.test.tsis excluded by!console/**console/tests/vexor/vexor-routes.test.tsis excluded by!console/**console/tests/worker/teams-routes.test.tsis excluded by!console/**console/tests/worker/vexor-removal.test.tsis excluded by!console/**launcher/config.pyis excluded by!launcher/**launcher/helper.pyis excluded by!launcher/**launcher/model_config.pyis excluded by!launcher/**launcher/notify.pyis excluded by!launcher/**launcher/statusline/formatter.pyis excluded by!launcher/**launcher/statusline/providers.pyis excluded by!launcher/**launcher/statusline/widgets.pyis excluded by!launcher/**launcher/tests/unit/statusline/test_formatter.pyis excluded by!launcher/**launcher/tests/unit/statusline/test_plan_provider.pyis excluded by!launcher/**launcher/tests/unit/statusline/test_widgets.pyis excluded by!launcher/**launcher/tests/unit/test_helper.pyis excluded by!launcher/**launcher/tests/unit/test_model_config.pyis excluded by!launcher/**launcher/tests/unit/test_notify.pyis excluded by!launcher/**launcher/tests/unit/test_tool_redirect.pyis excluded by!launcher/**launcher/tests/unit/test_updater.pyis excluded by!launcher/**launcher/updater.pyis excluded by!launcher/**launcher/wrapper.pyis excluded by!launcher/**pilot/.mcp.jsonis excluded by!pilot/**pilot/commands/spec-bugfix-plan.mdis excluded by!pilot/**pilot/commands/spec-implement.mdis excluded by!pilot/**pilot/commands/spec-plan.mdis excluded by!pilot/**pilot/commands/sync.mdis excluded by!pilot/**pilot/hooks/tests/test__util.pyis excluded by!pilot/**pilot/hooks/tool_redirect.pyis excluded by!pilot/**pilot/rules/cli-tools.mdis excluded by!pilot/**pilot/rules/code-review-reception.mdis excluded by!pilot/**pilot/rules/development-practices.mdis excluded by!pilot/**pilot/rules/mcp-servers.mdis excluded by!pilot/**pilot/rules/research-tools.mdis excluded by!pilot/**pilot/rules/task-and-workflow.mdis excluded by!pilot/**pilot/rules/testing.mdis excluded by!pilot/**pilot/scripts/worker-service.cjsis excluded by!pilot/**pilot/settings.jsonis excluded by!pilot/**pilot/ui/viewer-bundle.jsis excluded by!pilot/**pilot/ui/viewer.cssis excluded by!pilot/**
📒 Files selected for processing (19)
README.mddocs/site/index.htmldocs/site/src/components/DeepDiveSection.tsxdocs/site/src/components/FAQSection.tsxdocs/site/src/components/TechStack.tsxdocs/site/src/components/WorkflowSteps.tsxdocs/site/src/pages/docs/HooksSection.tsxdocs/site/src/pages/docs/InstallationSection.tsxdocs/site/src/pages/docs/McpServersSection.tsxdocs/site/src/pages/docs/QuickModeSection.tsxdocs/site/src/pages/docs/RulesSection.tsxdocs/site/src/pages/docs/SpecSection.tsxdocs/site/src/pages/docs/SyncSection.tsxinstaller/steps/dependencies.pyinstaller/steps/finalize.pyinstaller/tests/unit/steps/test_dependencies.pyinstaller/tests/unit/test_ui.pyinstaller/ui.pyuninstall.sh
💤 Files with no reviewable changes (2)
- installer/tests/unit/test_ui.py
- installer/ui.py
- Fix README formatting regressions (backtick-wrapped </br>, missing spaces) - Add timeout=15 to _is_probe_installed() subprocess call - Remove stale 'build semantic search index' text from SyncSection
|
Claude finished @maxritter's task in 1m 38s —— View job Incremental Code ReviewReviewing changes pushed to: ✅ Resolved from prior review
🔴 New issueREADME.md:162 — "builds a search index" is a leftover Vexor reference | `/sync` | Explores your codebase, discovers conventions, builds a search index, updates project rules. ... |The Still open (from prior review, no changes in this push)
|
# [7.3.0](v7.2.2...v7.3.0) (2026-03-08) ### Features * replace Vexor with Probe for code search ([#91](#91)) ([7ccf96d](7ccf96d))
|
🎉 This PR is included in version 7.3.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Summary
Replace Vexor (semantic code search) with Probe across the entire codebase.
What changed
search_code/extract_code) is the primary search tool, CLI is secondary fallbacknpm install -g @probelabs/probeworker-service.cjsandviewer-bundle.jsStats
Install
MCP config (already in
pilot/.mcp.json):{"probe": {"command": "npx", "args": ["-y", "@probelabs/probe@latest", "agent", "--mcp"]}}Summary by CodeRabbit
Documentation
Refactor