Skip to content

fix: exclude system prompt from routing classification #14

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
evansantos merged 1 commit into from
Feb 19, 2026
Merged

fix: exclude system prompt from routing classification #14

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 2 additions & 4 deletions src/index.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -344,12 +344,10 @@ const slimclawPlugin = {
// Only classify the CURRENT request intent, not the full history.
// Full history causes everything to be "reasoning/lengthy-content".
// Include: system prompt (for context) + last 3 messages (conversational flow) + current prompt.
// Classify based on user intent only — exclude system prompt
// (system prompt is static context, not indicative of request complexity)
const classificationMessages: Message[] = [];

Comment on lines 349 to 350
Copy link
Copy Markdown

@gitsniff gitsniff bot Feb 19, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ℹ️ Info

The blank line at L350 is unnecessary and can be removed to improve code readability and adhere to typical code formatting standards.

Fix: Remove the blank line at L350.

Suggested change
const classificationMessages: Message[] = [];
const classificationMessages: Message[] = [];

if (systemPrompt) {
classificationMessages.push({ role: 'system', content: systemPrompt });
}

// Last few messages for conversational context
const history = (historyMessages as any[]) || [];
const recentHistory = history.slice(-3);
Expand Down