@exreve/exk 1.0.23 → 1.0.25

package/dist/agentSession.js

@@ -427,11 +427,15 @@ export class AgentSessionManager {
                         const mcpServer = this.buildMcpServer(sessionId);
                         if (mcpServer) {
                             const toolHint = getModuleToolHint(session.enabledModules || []);
+                            console.log(`[agentSession] MCP server created: name=${mcpServer.name}, hasHint=${!!toolHint}`);
+                            console.log(`[agentSession] MCP server keys:`, Object.keys(mcpServer));
+                            console.log(`[agentSession] MCP server type:`, mcpServer.type);
                             return {
                                 mcpServers: { [mcpServer.name]: mcpServer },
                                 ...(toolHint ? { systemPrompt: { type: 'preset', preset: 'claude_code', append: toolHint } } : {})
                             };
                         }
+                        console.log(`[agentSession] No MCP server created (enabledModules=${JSON.stringify(session.enabledModules)})`);
                         return {};
                     })(),
                     ...(pathToClaudeCodeExecutable ? { pathToClaudeCodeExecutable } : {}),

package/dist/index.js

@@ -16,7 +16,6 @@ import { startApp, stopApp, restartApp, getAppStatuses, getAppLogs } from './app
 import { spawn, execSync } from 'child_process';
 import readline from 'readline';
 import { fileURLToPath } from 'url';
-import { createHash } from 'crypto';
 // ============ Constants ============
 const CONFIG_DIR = path.join(os.homedir(), '.talk-to-code');
 const CONFIG_FILE = path.join(CONFIG_DIR, 'config.json');
@@ -200,97 +199,24 @@ function getCliVersion() {
         return '0.0.0';
     }
 }
-const CURRENT_FILE = fileURLToPath(import.meta.url);
-const __dirname = path.dirname(CURRENT_FILE);
-function getCliHash() {
-    return createHash('sha256').update(fsSync.readFileSync(CURRENT_FILE)).digest('hex');
-}
-async function checkForUpdate() {
+// ============ Update Helpers ============
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+let isUpdating = false;
+/** Check if a newer version is available on npm */
+async function checkForNpmUpdate() {
     try {
-        const config = await readConfig();
-        const res = await fetch(`${config.apiUrl}/update/check`, {
-            method: 'POST',
-            headers: { 'Content-Type': 'application/json' },
-            body: JSON.stringify({ hash: getCliHash(), platform: os.platform() })
-        });
-        if (!res.ok)
-            return null;
-        return await res.json();
+        const currentVersion = getCliVersion();
+        const latestVersion = execSync('npm view @exreve/exk version', { encoding: 'utf-8', timeout: 10000 }).trim();
+        return {
+            updateAvailable: currentVersion !== latestVersion,
+            currentVersion,
+            latestVersion
+        };
     }
     catch {
         return null;
     }
 }
-async function replaceSelf(tarballBuffer) {
-    const extractDir = path.join(os.tmpdir(), `ttc-update-${Date.now()}`);
-    await fs.mkdir(extractDir, { recursive: true });
-    const tarPath = path.join(extractDir, 'ttc-cli.tar.gz');
-    await fs.writeFile(tarPath, tarballBuffer);
-    execSync(`tar -xzf "${tarPath}" -C "${extractDir}"`);
-    await fs.unlink(tarPath);
-    // Preserve user config/token files (never overwrite on update)
-    const preserveFiles = ['device-config.json', 'device-id.json', 'config.json'];
-    const preserved = {};
-    for (const f of preserveFiles) {
-        try {
-            preserved[f] = await fs.readFile(path.join(CONFIG_DIR, f));
-        }
-        catch {
-            /* file may not exist */
-        }
-    }
-    // Replace cli/ and shared/ in CONFIG_DIR
-    const cliDest = path.join(CONFIG_DIR, 'cli');
-    const sharedDest = path.join(CONFIG_DIR, 'shared');
-    await fs.rm(cliDest, { recursive: true, force: true });
-    await fs.rm(sharedDest, { recursive: true, force: true });
-    await fs.cp(path.join(extractDir, 'cli'), cliDest, { recursive: true });
-    await fs.cp(path.join(extractDir, 'shared'), sharedDest, { recursive: true });
-    // Update package.json and npm install
-    await fs.copyFile(path.join(extractDir, 'package.json'), path.join(CONFIG_DIR, 'package.json'));
-    await fs.rm(extractDir, { recursive: true, force: true });
-    // Restore preserved config
-    for (const f of preserveFiles) {
-        if (preserved[f])
-            await fs.writeFile(path.join(CONFIG_DIR, f), preserved[f]);
-    }
-    console.log('✓ CLI updated');
-    const npmPaths = [path.join(os.homedir(), '.nvm/versions/node/v22/bin/npm'), path.join(os.homedir(), '.nvm/versions/node/v20/bin/npm')];
-    const npmBin = npmPaths.find(p => fsSync.existsSync(p)) || 'npm';
-    try {
-        execSync(`"${npmBin}" install --omit=dev`, { cwd: CONFIG_DIR, stdio: 'inherit' });
-        console.log('✓ Dependencies updated');
-    }
-    catch {
-        console.warn('⚠ npm install failed');
-    }
-}
-async function selfUpdate(force = false) {
-    const info = await checkForUpdate();
-    if (!info || !info.updateAvailable) {
-        console.log('✓ Already up to date');
-        return;
-    }
-    console.log('📦 Update available!');
-    if (!force && process.stdin.isTTY) {
-        const rl = readline.createInterface({ input: process.stdin, output: process.stdout });
-        const answer = await new Promise(r => rl.question('Update now? [Y/n] ', a => { rl.close(); r(a.trim()); }));
-        if (answer.toLowerCase().startsWith('n'))
-            return;
-    }
-    if (!info.downloadUrl || !info.hash)
-        return;
-    console.log('Downloading...');
-    const res = await fetch(info.downloadUrl);
-    if (!res.ok)
-        throw new Error('Download failed');
-    const bundle = Buffer.from(await res.arrayBuffer());
-    if (createHash('sha256').update(bundle).digest('hex') !== info.hash) {
-        throw new Error('Hash mismatch');
-    }
-    await replaceSelf(bundle);
-    process.exit(0);
-}
 // ============ Commands ============
 async function registerDevice(name, email) {
     try {
@@ -590,9 +516,9 @@ async function runDaemon(foreground = false, email) {
     const ipAddress = getLocalIpAddress();
     const hostname = getHostname();
     // Silent update check on startup
-    checkForUpdate().then(info => {
+    checkForNpmUpdate().then(info => {
         if (info?.updateAvailable)
-            console.log('📦 Update available: run "ttc update"');
+            console.log(`📦 Update available: ${info.currentVersion} → ${info.latestVersion} (run "exk update")`);
     }).catch(() => { });
     if (foreground)
         console.log('=== TalkToCode CLI (Foreground) ===');
@@ -1716,10 +1642,13 @@ async function runDaemon(foreground = false, email) {
             // ========== Version & Update Handlers ==========
             // Respond with CLI version info
             socket.on('version:info', (_data, callback) => {
+                if (isUpdating) {
+                    callback({ success: false, error: 'Update in progress' });
+                    return;
+                }
                 callback({
                     success: true,
                     version: getCliVersion(),
-                    hash: getCliHash(),
                     date: new Date().toISOString(),
                     nodeVersion: process.version,
                     platform: os.platform(),
@@ -1728,6 +1657,10 @@ async function runDaemon(foreground = false, email) {
             });
             // Force update: npm update -g @exreve/exk then restart PM2
             socket.on('force-update', (_data, callback) => {
+                if (isUpdating) {
+                    callback?.({ success: false, error: 'Update already in progress' });
+                    return;
+                }
                 if (foreground) {
                     console.log('[force-update] Received force update command from server');
                 }
@@ -1768,33 +1701,6 @@ async function runDaemon(foreground = false, email) {
                     }
                 });
             });
-            // ========== update:start handler (legacy compatibility) ==========
-            socket.on('update:start', (_data, callback) => {
-                // Use npm-based self-update
-                if (foreground) {
-                    console.log('[update:start] Starting npm self-update...');
-                }
-                callback?.({ success: true, message: 'Update started via npm' });
-                const npmPaths = [
-                    path.join(os.homedir(), '.nvm/versions/node/v22/bin/npm'),
-                    path.join(os.homedir(), '.nvm/versions/node/v20/bin/npm')
-                ];
-                const npmBin = npmPaths.find(p => fsSync.existsSync(p)) || 'npm';
-                const updateProcess = spawn(npmBin, ['update', '-g', '@exreve/exk'], {
-                    stdio: 'pipe',
-                    detached: true
-                });
-                updateProcess.on('close', () => {
-                    setTimeout(() => {
-                        try {
-                            execSync('pm2 restart cli', { stdio: 'inherit' });
-                        }
-                        catch {
-                            process.exit(0);
-                        }
-                    }, 2000);
-                });
-            });
             // Cloudflared handlers
             socket.on('cloudflared:check:request', async () => {
                 try {
@@ -2305,7 +2211,7 @@ async function runDaemon(foreground = false, email) {
                     const { name, image, ports = [], env = {}, runAsRoot = false } = data;
                     // Get CLI directory path (where this script is running from)
                     // Use the same pattern as elsewhere in the file
-                    const cliDir = path.dirname(CURRENT_FILE);
+                    const cliDir = __dirname;
                     // Build docker run command
                     let cmd = `${runtime} run -d --name ${name}`;
                     // Security: Running as root INSIDE container is safe - it's still isolated from host

package/dist/moduleMcpServer.js

@@ -15,7 +15,7 @@ function createUserChoiceTool(onChoiceRequest) {
         options: z.array(z.object({ label: z.string(), value: z.string() })),
         timeout: z.number().optional()
     };
-    return tool('user_choice_request', 'Request user input when making decisions. Use this when you need the user to choose between options or provide input on a decision. This tool will present a modal to the user with your question and wait for their response.', schema, async (args, _extra) => {
+    return tool('user_choice_request', 'Present a choice modal to the user and wait for their selection. Use this whenever you need a decision, confirmation, or preference from the user. The AskUserQuestion tool is disabled — this is your only way to get interactive user input. Always use this before proceeding with ambiguous tasks or destructive actions.', schema, async (args, _extra) => {
         if (!onChoiceRequest) {
             return {
                 content: [
@@ -101,10 +101,15 @@ export function getModuleToolHint(enabledModules) {
         return null;
     const toolDescriptions = [];
     if (enabledModules.includes('user-choice')) {
-        toolDescriptions.push('- user_choice_request: Request user input when making decisions. Presents a modal to the user with options and waits for their response. Use this when you need the user to choose between options or confirm an action.');
+        toolDescriptions.push(`- user_choice_request(question: str, options: [{label: str, value: str}], timeout?: int): Present a modal to the remote user with a question and clickable options. Returns the user's selection.
+  IMPORTANT: The built-in AskUserQuestion tool is DISABLED. Whenever you need user input, decisions, or confirmations, you MUST use user_choice_request instead.
+  Always call this tool when:
+  - There are multiple valid approaches and you need the user to pick one
+  - You need explicit confirmation before a destructive or significant action
+  - The user's preference matters for how to proceed`);
     }
     // Add more module tool descriptions here as they are implemented
     if (toolDescriptions.length === 0)
         return null;
-    return `You have access to the following custom MCP tools:\n${toolDescriptions.join('\n')}\n\nUse these tools proactively when they are relevant to the task.`;
+    return `You have access to the following custom MCP tools:\n${toolDescriptions.join('\n')}\n\nThese tools are your ONLY way to interact with the user (AskUserQuestion is disabled). Call them directly — do NOT ask the user to type responses in chat.`;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exreve/exk",
-  "version": "1.0.23",
+  "version": "1.0.25",
   "description": "exk - Control Claude CLI with voice and programmable interfaces",
   "type": "module",
   "bin": {
@@ -14,7 +14,7 @@
     "container-entrypoint.sh"
   ],
   "scripts": {
-    "build": "tsc && node build-cli-tarball.js",
+    "build": "tsc",
     "build:tsc": "tsc",
     "typecheck": "tsc --noEmit",
     "prepublishOnly": "node -e \"require('fs').chmodSync('bin/exk', 0o755)\""