pikiclaw 0.3.74 → 0.3.75

package/dist/agent/drivers/claude-tui.js

@@ -45,7 +45,7 @@ import { encodePathAsDirName, getHome, whichSync } from '../../core/platform.js'
 import { createRetainedLogSink } from '../../core/logging.js';
 import { stripAnsiEscapes } from '../../core/utils.js';
 import { AGENT_STREAM_HARD_KILL_GRACE_MS, CLAUDE_TUI_STALL_QUIET_MS, CLAUDE_TUI_STALL_PENDING_TOOL_MS, CLAUDE_TUI_STALL_PTY_DEAD_MS, CLAUDE_TUI_STOP_HOLD_QUIET_TTL_MS, } from '../../core/constants.js';
-import { claudeParse, createClaudeStreamState, claudeContextWindowFromModel, claudeEffectiveContextWindow, registerClaudeBackgroundAgentLaunch, pendingClaudeBackgroundAgentCount, registerClaudeBackgroundBashLaunch, pendingClaudeBackgroundBashCount, extractClaudeBackgroundTaskId, extractClaudeWorkflowRunId, } from './claude.js';
+import { claudeParse, createClaudeStreamState, claudeContextWindowFromModel, claudeEffectiveContextWindow, registerClaudeBackgroundAgentLaunch, pendingClaudeBackgroundAgentCount, registerClaudeBackgroundBashLaunch, pendingClaudeBackgroundBashCount, extractClaudeBackgroundTaskId, extractClaudeWorkflowRunId, claudeEffortAndWorkflowArgs, } from './claude.js';
 // ---------------------------------------------------------------------------
 // Stall diagnostics (capture-only)
 // ---------------------------------------------------------------------------
@@ -1002,8 +1002,9 @@ export async function doClaudeTuiStream(opts) {
         claudeArgs.push('--model', model);
     if (opts.claudePermissionMode)
         claudeArgs.push('--permission-mode', opts.claudePermissionMode);
-    if (opts.thinkingEffort)
-        claudeArgs.push('--effort', opts.thinkingEffort);
+    // Effort + Workflow gate — same source of truth as the `claude -p` driver, so
+    // the TUI path drops the Workflow tool unless orchestration was opted in.
+    claudeArgs.push(...claudeEffortAndWorkflowArgs(opts));
     if (opts.claudeAppendSystemPrompt)
         claudeArgs.push('--append-system-prompt', opts.claudeAppendSystemPrompt);
     if (opts.mcpConfigPath)

package/dist/agent/drivers/claude.js

@@ -42,6 +42,29 @@ function claudeUsesStreamJsonInput(o) {
     return !!o.attachments?.length || !!o.onSteerReady;
 }
 const CLAUDE_STEER_IDLE_CLOSE_MS = 1200;
+/**
+ * Effort + multi-agent-Workflow gate args, shared by BOTH Claude spawn paths
+ * (`claude -p` in claudeCmd below and the PTY/TUI driver in claude-tui.ts).
+ * Kept in one place so the gate can never drift between them — the omission
+ * that once left the Workflow tool always-on under the TUI driver.
+ *
+ * "ultra" is a synthetic picker rung (max depth + Workflow orchestration), never
+ * a real --effort value — translate it to `max` so a stray "ultra" can't reach
+ * and break the CLI, and treat it as an implicit workflow opt-in. The Workflow
+ * tool ships in the default toolset and triggers on a bare "workflow" keyword;
+ * under the bypassPermissions mode pikiclaw runs by default that could auto-spawn
+ * a fleet of sub-agents, so drop it entirely unless orchestration was explicitly
+ * enabled (the workflow flag or the "ultra" rung).
+ */
+export function claudeEffortAndWorkflowArgs(o) {
+    const args = [];
+    const ultraEffort = o.thinkingEffort === 'ultra';
+    if (o.thinkingEffort)
+        args.push('--effort', ultraEffort ? 'max' : o.thinkingEffort);
+    if (!o.claudeWorkflowEnabled && !ultraEffort)
+        args.push('--disallowed-tools', 'Workflow');
+    return args;
+}
 // ---------------------------------------------------------------------------
 // Command & parser
 // ---------------------------------------------------------------------------
@@ -70,21 +93,9 @@ function claudeCmd(o) {
         if (o.attachments?.length)
             o._stdinOverride = buildClaudeUserMessage(o.prompt, o.attachments);
     }
-    // "ultra" is a synthetic picker rung (max depth + Workflow orchestration),
-    // never a real --effort value. The effort-write paths decompose it upstream;
-    // translate defensively here too so a stray "ultra" can never reach — and
-    // break — the CLI, and so it never suppresses the Workflow tool below.
-    const ultraEffort = o.thinkingEffort === 'ultra';
-    if (o.thinkingEffort)
-        args.push('--effort', ultraEffort ? 'max' : o.thinkingEffort);
-    // Multi-agent Workflow gate. The Workflow tool is always present in the
-    // toolset and triggers on a bare "workflow" keyword — combined with the
-    // bypassPermissions mode pikiclaw runs by default, that means an offhand
-    // mention could auto-spawn a fleet of sub-agents. Unless orchestration is
-    // explicitly enabled, drop the tool entirely so it can't fire at all. When
-    // enabled, the bot injects a standing opt-in directive via the system prompt.
-    if (!o.claudeWorkflowEnabled && !ultraEffort)
-        args.push('--disallowed-tools', 'Workflow');
+    // Effort + Workflow gate — shared with the TUI driver (claude-tui.ts) so the
+    // two spawn paths can never drift. See claudeEffortAndWorkflowArgs.
+    args.push(...claudeEffortAndWorkflowArgs(o));
     if (o.claudeAppendSystemPrompt)
         args.push('--append-system-prompt', o.claudeAppendSystemPrompt);
     if (o.mcpConfigPath)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pikiclaw",
-  "version": "0.3.74",
+  "version": "0.3.75",
   "description": "Put the world's smartest AI agents in your pocket. Command local Claude & Gemini via IM. | 让最好用的 IM 变成你电脑上的顶级 Agent 控制台",
   "type": "module",
   "bin": {