knoxis-helper 1.8.5 → 1.8.6

package/lib/knoxis-interactive-pair.js

@@ -81,6 +81,48 @@ function loadTask() {
   return null;
 }
 
+/**
+ * When QIG sends a portal task-list or PHASES prompt, the local agent should
+ * treat only that text as scope — not a stale multi-goal handoff wrapped by
+ * the backend. Unwraps "# Pair Programming Task / ## Goal" when present.
+ */
+function extractScopedOperatorTask(raw) {
+  if (!raw) return raw;
+  let text = raw.trim();
+
+  const goalMatch = text.match(/## Goal\s*\n([\s\S]*?)(?=\n## |\n# |$)/i);
+  if (goalMatch && text.includes('# Pair Programming Task')) {
+    text = goalMatch[1].trim();
+  }
+
+  const selectedIdx = text.search(/Selected tasks:\s*\n/i);
+  if (selectedIdx >= 0) {
+    const header = text.includes('Execute the selected tasks')
+      ? 'Execute the selected tasks exactly as written below.\n'
+      : '';
+    return (header + text.slice(selectedIdx)).trim();
+  }
+
+  if (/execute the selected tasks/i.test(text)) {
+    return text;
+  }
+
+  if (/phases\.md/i.test(text) || /run all unchecked steps/i.test(text)) {
+    return text;
+  }
+
+  return raw.trim();
+}
+
+function isScopedOperatorTask(raw) {
+  if (!raw) return false;
+  const lower = raw.toLowerCase();
+  return lower.includes('selected tasks:')
+    || lower.includes('execute the selected tasks')
+    || lower.includes('phases.md')
+    || lower.includes('steps to complete:');
+}
+
 // === LOAD PROJECT CONTEXT (from CLAUDE.md) ===
 function loadProjectContext() {
   const claudeMd = path.join(process.cwd(), 'CLAUDE.md');
@@ -479,7 +521,9 @@ async function runKitMode(kitMode, task, identity, scaffoldResult) {
 // === MAIN ===
 async function main() {
   const kitMode = process.env.KNOXIS_KIT_MODE || null;
-  const task = loadTask();
+  const taskRaw = loadTask();
+  const scoped = isScopedOperatorTask(taskRaw);
+  const task = scoped ? extractScopedOperatorTask(taskRaw) : taskRaw;
   if (!task && !kitMode) {
     console.error('No task found. Set KNOXIS_TASK_FILE, pass as CLI argument, or include in CLAUDE.md.');
     process.exit(1);
@@ -527,6 +571,9 @@ async function main() {
   console.log('╚══════════════════════════════════════════════════════════════╝');
   console.log('');
   console.log('  Task:      ' + task.substring(0, 100) + (task.length > 100 ? '...' : ''));
+  if (scoped) {
+    console.log('  Scope:     portal task-list / PHASES (HANDOFF rounds ignored)');
+  }
   console.log('  Session:   ' + SESSION_ID);
   console.log('  Recorded:  ' + recorder.sessionId);
   persistActiveSession(recorder, identity, { mode: 'interactive', taskHint: task });
@@ -577,6 +624,9 @@ async function main() {
     '- NEVER ask Claude questions. You provide answers and direction only.',
     '- If Claude lists multiple options, pick the most practical one.',
     '- Push for minimal, focused changes. No scope creep.',
+    scoped
+      ? '- SCOPED SESSION: The operator task below is the only scope. Reject doc-only or HANDOFF re-affirmation plans that skip application source changes when the task requires implementation.'
+      : '',
   ].join('\n');
 
 
@@ -590,16 +640,21 @@ async function main() {
   appendLog('## Phase 1: Planning\n');
 
   const planPrompt = [
-    task,
+    scoped
+      ? 'SCOPED TASK (only implement this — ignore unrelated HANDOFF.md round work):\n' + task
+      : task,
     '',
     'Before implementing, I need you to:',
     '1. Read the relevant existing code in this workspace',
     '2. Understand the current patterns and conventions',
     '3. Create a brief implementation plan (which files to change, what approach)',
     '4. Note any key decisions or trade-offs you see',
+    scoped
+      ? '5. Plan changes under src/ (application code), not docs-only re-affirmation unless the scoped task explicitly requires docs'
+      : '',
     '',
     'Share your plan and then STOP. Do not implement yet. I will review it first.',
-  ].join('\n');
+  ].filter(Boolean).join('\n');
 
   const phase1Idx = recorder.startStep('understand-plan', 'Claude Code', planPrompt);
   recorder.setStepPrompt(phase1Idx, planPrompt);
@@ -635,7 +690,13 @@ async function main() {
 
   const planReviewPrompt = 'Claude produced the following plan:\n\n'
     + phase1.stdout.substring(0, 8000)
-    + '\n\nReview this plan. Answer any questions Claude asked. Approve or suggest specific changes. Then tell Claude to proceed with implementation.';
+    + '\n\nScoped operator task (authoritative):\n'
+    + task.substring(0, 4000)
+    + '\n\nReview this plan against the scoped task only. Answer any questions Claude asked. '
+    + (scoped
+      ? 'Reject doc-only or HANDOFF re-affirmation plans that do not implement the scoped task in source code. '
+      : '')
+    + 'Approve or suggest specific changes. Then tell Claude to proceed with implementation.';
   const planReviewIdx = recorder.startStep('knoxis-plan-review', 'Knoxis (Groq)', planReviewPrompt);
   recorder.setStepPrompt(planReviewIdx, planReviewPrompt);
   const planReview = await callGroq(groqSystem, planReviewPrompt);
@@ -656,9 +717,26 @@ async function main() {
   console.log('');
   appendLog('## Phase 2: Implementation\n');
 
-  const phase2Idx = recorder.startStep('implement', 'Claude Code', planReview);
-  recorder.setStepPrompt(phase2Idx, planReview);
-  const phase2 = await runClaudeTurn(planReview, true);
+  const phase2Prompt = scoped
+    ? [
+      'IMPLEMENTATION PHASE — implement the scoped task in application source code.',
+      '',
+      'Scoped task (authoritative):',
+      task,
+      '',
+      'Knoxis feedback on your plan:',
+      planReview,
+      '',
+      'Requirements:',
+      '- Change application source (e.g. under src/) to complete the scoped task.',
+      '- Do NOT spend this phase only updating docs/state/*.md or HANDOFF re-affirmations unless the scoped task explicitly requires documentation.',
+      '- Follow existing patterns. Run the most relevant build/test when done.',
+    ].join('\n')
+    : planReview;
+
+  const phase2Idx = recorder.startStep('implement', 'Claude Code', phase2Prompt);
+  recorder.setStepPrompt(phase2Idx, phase2Prompt);
+  const phase2 = await runClaudeTurn(phase2Prompt, true);
   appendLog(phase2.stdout.substring(0, 10000) + '\n');
   recorder.completeStep(phase2Idx, phase2.stdout, phase2.code !== 0 ? `exit ${phase2.code}: ${(phase2.stderr || '').slice(0, 200)}` : null);
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knoxis-helper",
-  "version": "1.8.5",
+  "version": "1.8.6",
   "description": "Local helper for Knoxis pair programming - connects your machine to Knoxis on qig.ai",
   "bin": {
     "knoxis-helper": "./bin/knoxis-helper.js"