lynkr 3.2.0 → 3.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "lynkr",
-  "version": "3.2.0",
+  "version": "3.2.1",
   "description": "Self-hosted Claude Code proxy with Databricks,Azure  adapters, openrouter, Ollama,llamacpp, workspace tooling, and MCP integration.",
   "main": "index.js",
   "bin": {

package/src/orchestrator/index.js

@@ -10,6 +10,7 @@ const tokens = require("../utils/tokens");
 const systemPrompt = require("../prompts/system");
 const historyCompression = require("../context/compression");
 const tokenBudget = require("../context/budget");
+const { classifyRequestType, selectToolsSmartly } = require("../tools/smart-selection");
 
 const DROP_KEYS = new Set([
   "provider",
@@ -1001,39 +1002,22 @@ function sanitizePayload(payload) {
 
   // Smart tool selection (universal, applies to all providers)
   if (config.smartToolSelection?.enabled && Array.isArray(clean.tools) && clean.tools.length > 0) {
-    const { classifyRequestType, selectToolsSmartly } = require('../tools/smart-selection');
-
     const classification = classifyRequestType(clean);
-
-    logger.debug({
-      originalToolCount: clean.tools.length,
-      classificationType: classification.type,
-      confidence: classification.confidence,
-      keywords: classification.keywords
-    }, "Smart tool selection: classified request");
-
     const selectedTools = selectToolsSmartly(clean.tools, classification, {
       provider: providerType,
       tokenBudget: config.smartToolSelection.tokenBudget,
       config: config.smartToolSelection
     });
 
-    const removedTools = clean.tools
-      .filter(t => !selectedTools.find(s => s.name === t.name))
-      .map(t => t.name);
-
-    const keptTools = selectedTools.map(t => t.name);
-
-    logger.info({
-      feature: 'smart_tool_selection',
-      requestType: classification.type,
-      confidence: classification.confidence,
-      originalToolCount: clean.tools.length,
-      selectedToolCount: selectedTools.length,
-      removedTools,
-      keptTools,
-      provider: providerType
-    }, "Smart tool selection applied");
+    // Only log if tools were actually filtered (avoid logging overhead)
+    if (selectedTools.length !== clean.tools.length) {
+      logger.info({
+        requestType: classification.type,
+        originalCount: clean.tools.length,
+        selectedCount: selectedTools.length,
+        provider: providerType
+      }, "Smart tool selection applied");
+    }
 
     clean.tools = selectedTools.length > 0 ? selectedTools : undefined;
   }

package/src/tools/smart-selection.js

@@ -9,6 +9,18 @@
 
 const logger = require('../logger');
 
+// Pre-compiled regex patterns for performance (avoid recompiling on every request)
+const GREETING_PATTERN = /^(hi|hello|hey|good morning|good afternoon|good evening|howdy|greetings|sup|yo)[\s\.\!\?]*$/i;
+const QUESTION_PATTERN = /^(what is|what's|how does|when|where|why|explain|define|tell me about|can you explain)/i;
+const TECHNICAL_KEYWORDS = /code|function|class|file|module|import|export|async|await|promise|callback|api|database|server|client|component|method|variable|array|object|string|number/i;
+const EXPLANATION_PATTERN = /explain|describe|summarize|what does|how does|tell me about|give me an overview|clarify|elaborate/i;
+const WEB_PATTERN = /search|lookup|find info|google|documentation|docs|website|url|link|online|internet|browse/i;
+const READ_PATTERN = /read|show|display|view|cat|check|inspect|look at|see|examine|review|print|output/i;
+const WRITE_PATTERN = /write|create|add|update|modify|change|fix|delete|remove|insert|append|replace|save/i;
+const EDIT_PATTERN = /edit|refactor|rename|move|reorganize|restructure|rewrite/i;
+const EXECUTION_PATTERN = /run|execute|test|compile|build|deploy|start|install|launch|boot|fire up|npm|git|python|node|docker|bash|sh|cmd/i;
+const COMPLEX_PATTERN = /implement|build|create|develop|design|architect|plan|strategy|approach|help with|work on|improve|optimize|enhance|refactor|migrate/i;
+
 /**
  * Tool selection map: request type → relevant tools
  */
@@ -91,8 +103,7 @@ function extractContent(message) {
  * Check if content matches greeting patterns
  */
 function isGreeting(content) {
-  const greetingPattern = /^(hi|hello|hey|good morning|good afternoon|good evening|howdy|greetings|sup|yo)[\s\.\!\?]*$/i;
-  return greetingPattern.test(content.trim());
+  return GREETING_PATTERN.test(content.trim());
 }
 
 /**
@@ -100,85 +111,70 @@ function isGreeting(content) {
  */
 function isShortNonTechnical(content) {
   const trimmed = content.trim();
-
-  // Very short messages (< 20 chars)
-  if (trimmed.length >= 20) return false;
-
-  // Check for technical keywords
-  const technicalKeywords = /code|file|function|error|bug|fix|write|read|create|implement|class|module|import|export|async|await/i;
-  return !technicalKeywords.test(trimmed);
+  return trimmed.length < 20 && !TECHNICAL_KEYWORDS.test(trimmed);
 }
 
 /**
  * Check if content is a simple question
  */
 function isSimpleQuestion(content) {
-  const questionPattern = /^(what is|what's|how does|when|where|why|explain|define|tell me about|can you explain)/i;
-  return questionPattern.test(content.trim());
+  return QUESTION_PATTERN.test(content.trim());
 }
 
 /**
  * Check for technical keywords
  */
 function hasTechnicalKeywords(content) {
-  const technicalPattern = /code|function|class|file|module|import|export|async|await|promise|callback|api|database|server|client|component|method|variable|array|object|string|number/i;
-  return technicalPattern.test(content);
+  return TECHNICAL_KEYWORDS.test(content);
 }
 
 /**
  * Check for explanation/research keywords
  */
 function hasExplanationKeywords(content) {
-  const explanationPattern = /explain|describe|summarize|what does|how does|tell me about|give me an overview|clarify|elaborate/i;
-  return explanationPattern.test(content);
+  return EXPLANATION_PATTERN.test(content);
 }
 
 /**
  * Check for web/search keywords
  */
 function hasWebKeywords(content) {
-  const webPattern = /search|lookup|find info|google|documentation|docs|website|url|link|online|internet|browse/i;
-  return webPattern.test(content);
+  return WEB_PATTERN.test(content);
 }
 
 /**
  * Check for file reading keywords
  */
 function hasReadKeywords(content) {
-  const readPattern = /read|show|display|view|cat|check|inspect|look at|see|examine|review|print|output/i;
-  return readPattern.test(content);
+  return READ_PATTERN.test(content);
 }
 
 /**
  * Check for file writing/modification keywords
  */
 function hasWriteKeywords(content) {
-  const writePattern = /write|create|add|update|modify|change|fix|delete|remove|insert|append|replace|save/i;
-  return writePattern.test(content);
+  return WRITE_PATTERN.test(content);
 }
 
 /**
  * Check for edit/refactor keywords
  */
 function hasEditKeywords(content) {
-  const editPattern = /edit|refactor|rename|move|reorganize|restructure|rewrite/i;
-  return editPattern.test(content);
+  return EDIT_PATTERN.test(content);
 }
 
 /**
  * Check for execution/testing keywords
  */
 function hasExecutionKeywords(content) {
-  const executionPattern = /run|execute|test|compile|build|deploy|start|install|launch|boot|fire up|npm|git|python|node|docker|bash|sh|cmd/i;
-  return executionPattern.test(content);
+  return EXECUTION_PATTERN.test(content);
 }
 
 /**
  * Check for complex task keywords
  */
 function hasComplexKeywords(content) {
-  const complexPattern = /implement|build|create|develop|design|architect|plan|strategy|approach|help with|work on|improve|optimize|enhance|refactor|migrate/i;
-  return complexPattern.test(content);
+  return COMPLEX_PATTERN.test(content);
 }
 
 /**
@@ -191,7 +187,6 @@ function classifyRequestType(payload) {
   const lastMessage = getLastUserMessage(payload);
 
   if (!lastMessage) {
-    logger.debug('No user message found for classification');
     return { type: 'coding', confidence: 0.5, keywords: [] };
   }
 
@@ -199,12 +194,6 @@ function classifyRequestType(payload) {
   const contentLower = content.toLowerCase();
   const messageCount = payload.messages?.length ?? 0;
 
-  logger.debug({
-    contentLength: content.length,
-    messageCount,
-    contentPreview: content.substring(0, 100)
-  }, 'Classifying request type');
-
   // 1. Conversational (no tools)
   if (isGreeting(contentLower)) {
     return { type: 'conversational', confidence: 1.0, keywords: ['greeting'] };
@@ -286,13 +275,6 @@ function selectToolsSmartly(tools, classification, options = {}) {
   // Get relevant tool names for this request type
   const relevantToolNames = TOOL_SELECTION_MAP[requestType] || TOOL_SELECTION_MAP.coding;
 
-  logger.debug({
-    requestType,
-    confidence: classification.confidence,
-    relevantToolNames,
-    mode: config.mode
-  }, 'Selecting tools for request type');
-
   // Filter to relevant tools only
   let selectedTools = tools.filter(tool => relevantToolNames.includes(tool.name));
 
@@ -312,10 +294,6 @@ function selectToolsSmartly(tools, classification, options = {}) {
 
   // Provider-specific limits
   if (provider === 'ollama' && selectedTools.length > 8) {
-    logger.debug({
-      originalCount: selectedTools.length,
-      limit: 8
-    }, 'Limiting tools for Ollama provider');
     selectedTools = selectedTools.slice(0, 8);
   }
 
@@ -323,11 +301,6 @@ function selectToolsSmartly(tools, classification, options = {}) {
   const estimatedTokens = estimateToolTokens(selectedTools);
   if (estimatedTokens > tokenBudget) {
     const targetCount = Math.floor(tokenBudget / 175);
-    logger.debug({
-      estimatedTokens,
-      tokenBudget,
-      targetCount
-    }, 'Enforcing token budget on tools');
     selectedTools = selectedTools.slice(0, Math.max(targetCount, 0));
   }
 
@@ -337,14 +310,6 @@ function selectToolsSmartly(tools, classification, options = {}) {
     selectedTools = selectedTools.filter(t => minimalTools.includes(t.name));
   }
 
-  logger.info({
-    requestType,
-    originalToolCount: tools.length,
-    selectedToolCount: selectedTools.length,
-    removedToolCount: tools.length - selectedTools.length,
-    estimatedTokenSavings: estimateToolTokens(tools) - estimateToolTokens(selectedTools)
-  }, 'Smart tool selection completed');
-
   return selectedTools;
 }