@multiclawcli/core 0.10.1 → 0.12.0

package/dist/index.js

@@ -1175,7 +1175,7 @@ async function validateToolsPolicy(config, options = {}) {
   const descriptions = /* @__PURE__ */ new Map();
   for (const agent of agents2) {
     checkAbort();
-    if (!hasExplicitTools(agent)) continue;
+    if (!hasExplicitTools(agent) && (agent.tools !== void 0 || !config.defaults?.tools)) continue;
     const runtime = runtimeRegistry.get(agent.runtime ?? "acp");
     const command = resolveCommand(agent, config);
     let commands = descriptions.get(runtime);
@@ -2479,7 +2479,7 @@ var AgentRunner = class {
       const runtime = runtimeRegistry.get(agent.runtime ?? "acp");
       const retries = agent.retries ?? 0;
       const timeout = agent.timeout ?? 6e5;
-      const tools = agent.tools ?? ["Read", "Write", "Bash"];
+      const tools = agent.tools ?? this.config.defaults?.tools ?? ["Read", "Write", "Bash"];
       const completed = await withTimeout(
         (innerSignal) => withRetry(
           async (attempt) => {
@@ -3252,13 +3252,38 @@ Address these findings within your assigned scope, preserve the complete task re
         payload: { agentId: agent.id, agentName: agent.name, icon: agent.icon }
       });
     }
+    const agentMap = new Map(this.config.agents.map((a) => [a.id, a]));
     return this.config.agents.filter((a) => runSet.has(a.id) && a.id !== this.config.review?.reviewer).map((a) => ({
       ...a,
-      dependsOn: a.dependsOn?.filter(
-        (dep) => !skippedIds.has(dep) && dep !== this.config.review?.reviewer
-      )
+      dependsOn: a.dependsOn !== void 0 ? this.resolveEffectiveDeps(a.dependsOn, skippedIds, agentMap) : void 0
     }));
   }
+  // When an intermediate agent is skipped, replace it in dependsOn with its own
+  // dependencies (recursively). This preserves topological ordering across gaps:
+  // code-reviewer → tester (skip) → backend-developer becomes
+  // code-reviewer → backend-developer, so the reviewer never runs before the code is written.
+  resolveEffectiveDeps(deps, skippedIds, agentMap, visited = /* @__PURE__ */ new Set()) {
+    const reviewerId = this.config.review?.reviewer;
+    const result = /* @__PURE__ */ new Set();
+    for (const id of deps) {
+      if (id === reviewerId) continue;
+      if (visited.has(id)) continue;
+      if (!skippedIds.has(id)) {
+        result.add(id);
+      } else {
+        visited.add(id);
+        for (const transitive of this.resolveEffectiveDeps(
+          agentMap.get(id)?.dependsOn ?? [],
+          skippedIds,
+          agentMap,
+          visited
+        )) {
+          result.add(transitive);
+        }
+      }
+    }
+    return [...result];
+  }
   async runStage(agents2, signal) {
     const { maxConcurrency } = this.config;
     const results = /* @__PURE__ */ new Map();

package/dist/orchestrator/Orchestrator.d.ts

@@ -75,6 +75,7 @@ export declare class Orchestrator {
      * reader looking in the wrong place.
      */
     private selectAgents;
+    private resolveEffectiveDeps;
     private runStage;
     /**
      * Whether `agent` owns task decomposition.

package/dist/types/config.d.ts

@@ -1,4 +1,4 @@
-import type { AgentDefinition } from "./agent";
+import type { AgentDefinition, ToolName } from "./agent";
 import type { LeaderMessage } from "./leader";
 import type { RuntimePermissionHandler, RuntimeQuestionHandler } from "./runtime";
 /**
@@ -146,6 +146,7 @@ export interface MultiClawConfig {
         provider?: string;
         model?: string;
         command?: string;
+        tools?: ToolName[];
     };
 }
 export interface OrchestrationPreview {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@multiclawcli/core",
-  "version": "0.10.1",
+  "version": "0.12.0",
   "type": "module",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",