Function needsClarification

src/session/clarify.js:23–47

* Check if a user message is too vague to act on. * Returns true if clarification should be requested.

(message)

 * Returns true if clarification should be requested.
 */
function needsClarification(message) {
  const msg = message.trim();

  // Never trigger on empty (already handled) or file references
  if (!msg || msg.startsWith('@') || msg.startsWith('/')) return false;

  // Never trigger on confirmations (these are answers to prior model questions)
  if (/^(yes|no|ok|sure|go|do it|y|n|yep|nope|yeah|nah)$/i.test(msg)) return false;

  // Never trigger on multi-word continuations and follow-ups
  if (/^(go ahead|go for it|just do it|do that|do both|read it|show me|that one|sounds good|let's do it|let's go|that works)\b/i.test(msg)) return false;

  // Never trigger on multi-number selections ("1 and 2", "1, 2", "both 1 and 2")
  if (/^(both\s+)?\d+(\s*,\s*|\s+and\s+)\d+$/i.test(msg)) return false;

  // Vague patterns that genuinely lack specifics and need clarification
  const vaguePatterns = [
    /^(fix|do|make|change|update|improve)\s+(it|this|that|things?)$/i,
    /^(help|please|can you|could you)$/i,
    /^(make it|do the|fix the)\s+(better|work|thing|stuff)$/i,
    /^(same|again|more|another)$/i,
  ];

  return vaguePatterns.some(p => p.test(msg));
}

/**
 * Generate a clarification prompt to inject into the system message.