Skip to content

Fix ProcessType routing for Cortex and Compactor#290

Open
artlu99 wants to merge 4 commits intospacedriveapp:mainfrom
artlu99:fix/processtype-routing
Open

Fix ProcessType routing for Cortex and Compactor#290
artlu99 wants to merge 4 commits intospacedriveapp:mainfrom
artlu99:fix/processtype-routing

Conversation

@artlu99
Copy link

@artlu99 artlu99 commented Mar 2, 2026

This PR addresses cortex and compactor pulling from the wrong routing config. (likely due to copy-paste type errors)

Diagnosed in public #spacebot-debug chat with Spacebot, fixes prepared by Spacebot. LGTM!

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 2, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7ebd1e1 and 73b1cc3.

📒 Files selected for processing (3)
  • src/agent/compactor.rs
  • src/agent/cortex.rs
  • src/agent/cortex_chat.rs

Walkthrough

This PR updates process type selections in agent model resolution across three files. Changes involve selecting Compactor instead of Worker in compaction operations and Cortex instead of Branch in Cortex-related operations for LLM model routing.

Changes

Cohort / File(s) Summary
ProcessType Model Resolution Updates
src/agent/compactor.rs, src/agent/cortex.rs, src/agent/cortex_chat.rs
Updated ProcessType selection for LLM model routing: switched from Worker to Compactor in compaction operations, and from Branch to Cortex in bulletin generation, profile generation, and chat operations.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: fixing ProcessType routing for Cortex and Compactor modules.
Description check ✅ Passed The description is directly related to the changeset, explaining the routing configuration fixes and their origin.
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.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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.

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