skyloom 1.4.0 → 1.4.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skyloom",
-  "version": "1.4.0",
+  "version": "1.4.2",
   "description": "天空织机 Skyloom — 6 weather-themed AI agents: Fog, Rain, Frost, Snow, Dew, Fair",
   "preferGlobal": true,
   "type": "commonjs",

package/src/cli/main.ts

@@ -377,22 +377,29 @@ function shouldAutoContinue(text: string): boolean {
 async function main(): Promise<void> {
   const args = process.argv.slice(2);
 
-  // If no args or just "chat", start interactive chat
-  if (args.length === 0 || args[0] === 'chat' || (args.length === 1 && !args[0].startsWith('-') && !['task', 'web', 'config', 'init', 'version'].includes(args[0]))) {
-    // Check if first arg is an agent name
-    const knownAgents = new Set(['fog', 'rain', 'frost', 'snow', 'dew', 'fair']);
-    let agent = 'fog';
-    let model: string | undefined;
+  // `sky` with no args = start chat directly (fastest path)
+  if (args.length === 0) {
+    await interactiveChat('fog');
+    return;
+  }
 
-    for (let i = 0; i < args.length; i++) {
-      if (knownAgents.has(args[i])) {
-        agent = args[i];
-      } else if (args[i] === '-m' || args[i] === '--model') {
+  // `sky <agent>` or `sky <agent> -m <model>` — chat with specific agent
+  const knownAgents = new Set(['fog', 'rain', 'frost', 'snow', 'dew', 'fair']);
+  if (knownAgents.has(args[0])) {
+    let model: string | undefined;
+    for (let i = 1; i < args.length; i++) {
+      if ((args[i] === '-m' || args[i] === '--model') && i + 1 < args.length) {
         model = args[++i];
       }
     }
+    await interactiveChat(args[0], model);
+    return;
+  }
 
-    await interactiveChat(agent, model);
+  // `sky <message>` — anything not a known subcommand → fast chat
+  const knownCommands = ['chat', 'task', 'web', 'config', 'init', 'version', 'mcp', 'help'];
+  if (!knownCommands.includes(args[0]) && !args[0].startsWith('-')) {
+    await interactiveChat('fog');
     return;
   }
 

package/src/core/agent.ts

@@ -729,13 +729,14 @@ export class BaseAgent {
     try {
       if (onStatus) onStatus('thinking...');
       const response = await this.llmLoop({ onStatus });
-      this.memory.addMessage('assistant', response.content, {
-        toolCalls: response.toolCalls,
-        reasoningContent: response.reasoningContent,
+      const content = response?.content || '(no response)';
+      this.memory.addMessage('assistant', content, {
+        toolCalls: response?.toolCalls || [],
+        reasoningContent: response?.reasoningContent,
       });
       await this.setState(AgentState.IDLE);
       this.maybeExtractFacts();
-      return response.content;
+      return content;
     } catch (e) {
       await this.setState(AgentState.ERROR);
       this.popLastUserMessage();

package/src/core/logger.ts

@@ -69,9 +69,8 @@ export class Logger {
       return value;
     });
 
-    // Output to stderr for logs, stdout for normal output
-    const output = level >= LogLevel.WARN ? console.error : console.log;
-    output(line);
+    // Write all logs to stderr to keep stdout clean for chat/TUI
+    process.stderr.write(line + "\n");
   }
 
   debug(msg: string, extra?: Record<string, unknown>) {
@@ -100,7 +99,7 @@ export class Logger {
  */
 const loggers = new Map<string, Logger>();
 let requestId: string | null = null;
-let defaultLogLevel = LogLevel.INFO;
+let defaultLogLevel = LogLevel.WARN; // Only warnings+errors by default
 
 /**
  * Get or create a logger for a component

package/src/core/memory.ts

@@ -276,7 +276,9 @@ export class Memory {
   private dbRun(sql: string, params?: any[]): void {
     if (!this.db) return;
     try {
-      this.db.run(sql, params);
+      // Sql.js rejects `undefined` in bind arrays; normalize to `null`
+      const safe = params ? params.map((v) => v === undefined ? null : v) : undefined;
+      this.db.run(sql, safe);
     } catch (err) {
       logger.warn('db_run_failed', { sql: sql.slice(0, 80), error: String(err) });
     }