knoxis-helper 1.3.1 → 1.3.3

package/lib/knoxis-local-agent.js

@@ -657,6 +657,23 @@ async function handleRequest(req, res) {
 
   // Start pair programming session (opens terminal)
   if (pathname === '/pair/start' && method === 'POST') {
+    // Build CLAUDE.md content from task when backend doesn't provide claudeMdContent
+    function buildClaudeMdFromTask(taskText, workspacePath) {
+      const lines = [];
+      lines.push('# Project Instructions\n');
+      lines.push('## Current Task');
+      lines.push(taskText);
+      lines.push('');
+      lines.push('## Working Agreement');
+      lines.push('- Work autonomously. Do not ask clarifying questions - make your best engineering judgment and proceed.');
+      lines.push('- Read and understand existing code before making changes.');
+      lines.push('- Follow existing patterns in the codebase.');
+      lines.push('- Keep changes minimal and focused on the task.');
+      lines.push('- Verify your work compiles/runs where possible.');
+      lines.push('- Be direct and concise in any output.');
+      return lines.join('\n');
+    }
+
     try {
       const body = await parseBody(req);
       const { workspace, task, file, provider, headless, sessionId, claudeMdContent } = body;
@@ -675,18 +692,25 @@ async function handleRequest(req, res) {
         }
       }
 
-      // Write CLAUDE.md if provided — ensures fresh task context every session
-      if (claudeMdContent && fs.existsSync(workspaceDir)) {
+      // Write CLAUDE.md for supplementary context.
+      // If claudeMdContent was provided (from backend relay), use it directly.
+      // Otherwise, build it from the task field.
+      const effectiveClaudeMd = claudeMdContent || buildClaudeMdFromTask(task, workspace);
+      if (effectiveClaudeMd && fs.existsSync(workspaceDir)) {
         try {
-          fs.writeFileSync(path.join(workspaceDir, 'CLAUDE.md'), claudeMdContent, 'utf8');
-          console.log(`📄 Wrote CLAUDE.md to ${workspaceDir}`);
+          fs.writeFileSync(path.join(workspaceDir, 'CLAUDE.md'), effectiveClaudeMd, 'utf8');
+          console.log(`📄 Wrote CLAUDE.md to ${workspaceDir} (${effectiveClaudeMd.length} chars)`);
         } catch (e) {
           console.warn(`⚠️ Failed to write CLAUDE.md: ${e.message}`);
         }
       }
 
-      // Short command — Claude reads CLAUDE.md for full context
-      const command = `claude --dangerously-skip-permissions "Complete the task described in CLAUDE.md. Work autonomously."`;
+      // Write the actual task to a temp file and pipe it to Claude via stdin.
+      // This avoids shell escaping issues with quotes/backticks/etc in the task text.
+      const promptFile = path.join(os.tmpdir(), `knoxis-task-${sessionId || Date.now()}.txt`);
+      const promptText = file ? `Working on file: ${file}\n\nTask: ${task}` : task;
+      fs.writeFileSync(promptFile, promptText, 'utf8');
+      const command = `cat "${promptFile}" | claude --dangerously-skip-permissions`;
 
       if (headless) {
         const result = await runHeadlessProcess({
@@ -999,35 +1023,39 @@ function connectRelayWebSocket() {
       if (msg.type === 'execute_command') {
         console.log(`📥 Relay command received: ${msg.requestId}`);
         console.log(`   📂 Dir: ${msg.workingDir}`);
-        console.log(`   📝 Cmd: ${(msg.command || '').substring(0, 100)}...`);
+        console.log(`   📝 Prompt: ${msg.prompt ? msg.prompt.substring(0, 120) + '...' : '(none)'}`);
 
         const workspace = msg.workingDir || process.cwd();
-        const command = msg.command || '';
+        const wsDir = workspace.startsWith('~')
+          ? path.join(os.homedir(), workspace.slice(1))
+          : workspace;
 
-        // Write CLAUDE.md to workspace before executing — ensures Claude auto-reads
-        // fresh task context and prevents accumulation of old tasks
-        if (msg.claudeMdContent && workspace && workspace !== '~') {
+        // Write CLAUDE.md for supplementary context (developer brief, working agreement, etc.)
+        if (msg.claudeMdContent && fs.existsSync(wsDir)) {
           try {
-            const wsDir = workspace.startsWith('~')
-              ? path.join(os.homedir(), workspace.slice(1))
-              : workspace;
-            if (fs.existsSync(wsDir)) {
-              const claudeMdPath = path.join(wsDir, 'CLAUDE.md');
-              fs.writeFileSync(claudeMdPath, msg.claudeMdContent, 'utf8');
-              console.log(`   📄 Wrote CLAUDE.md to ${claudeMdPath} (${msg.claudeMdContent.length} chars)`);
-            } else {
-              console.warn(`   ⚠️ Workspace does not exist, skipping CLAUDE.md write: ${wsDir}`);
-            }
+            fs.writeFileSync(path.join(wsDir, 'CLAUDE.md'), msg.claudeMdContent, 'utf8');
+            console.log(`   📄 Wrote CLAUDE.md (${msg.claudeMdContent.length} chars)`);
           } catch (writeErr) {
             console.warn(`   ⚠️ Failed to write CLAUDE.md: ${writeErr.message}`);
           }
         }
 
+        // Build the actual command. If a prompt was sent, write it to a temp file
+        // and pipe it to Claude via stdin — avoids all shell escaping issues.
+        let command = msg.command || '';
+        let promptFile = null;
+        if (msg.prompt) {
+          promptFile = path.join(os.tmpdir(), `knoxis-task-${msg.requestId || Date.now()}.txt`);
+          fs.writeFileSync(promptFile, msg.prompt, 'utf8');
+          command = `cat "${promptFile}" | claude --dangerously-skip-permissions`;
+          console.log(`   📝 Task written to ${promptFile} (${msg.prompt.length} chars)`);
+        }
+
         let result;
         try {
           if (msg.headless) {
             result = await runHeadlessProcess({
-              workspace,
+              workspace: wsDir,
               command,
               prompt: msg.prompt,
               sessionLabel: msg.requestId || 'relay'
@@ -1035,11 +1063,11 @@ function connectRelayWebSocket() {
           } else {
             const platform = os.platform();
             if (platform === 'darwin') {
-              await openMacTerminal(workspace, command);
+              await openMacTerminal(wsDir, command);
             } else if (platform === 'win32') {
-              await openWindowsTerminal(workspace, command);
+              await openWindowsTerminal(wsDir, command);
             } else {
-              await openLinuxTerminal(workspace, command);
+              await openLinuxTerminal(wsDir, command);
             }
             result = { success: true, message: 'Terminal opened via relay' };
           }
@@ -1047,6 +1075,13 @@ function connectRelayWebSocket() {
           result = { success: false, error: err.message };
         }
 
+        // Clean up temp prompt file (Claude already read it)
+        if (promptFile) {
+          setTimeout(() => {
+            try { fs.unlinkSync(promptFile); } catch (e) {}
+          }, 5000); // Delay to ensure cat has finished piping
+        }
+
         // Send result back to backend
         const response = JSON.stringify({
           type: 'command_result',
@@ -1187,7 +1222,7 @@ server.listen(serverMeta.port, () => {
   const scheme = serverMeta.secure ? 'https' : 'http';
   console.log('');
   console.log('╔══════════════════════════════════════════════════════════════╗');
-  console.log('║       🚀 KNOXIS LOCAL AGENT v2.1.0 (HTTPS + CORS Fix)        ║');
+  console.log('║       🚀 KNOXIS LOCAL AGENT v2.3.0 (Prompt Pipe Fix)         ║');
   console.log('╚══════════════════════════════════════════════════════════════╝');
   console.log('');
   console.log(`🔒 Mode: ${serverMeta.secure ? 'HTTPS (Secure)' : 'HTTP (Insecure - see warning below)'}`);

package/package.json

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