@claude-flow/cli 3.56.8 → 3.56.10

package/dist/src/mods/command-registry/schema.d.ts

@@ -63,19 +63,19 @@ export declare const commandDefinitionSchema: z.ZodEffects<z.ZodObject<{
         path: z.ZodString;
     }, "strict", z.ZodTypeAny, {
         path: string;
-        commit: string;
         repository: string;
+        commit: string;
     }, {
         path: string;
-        commit: string;
         repository: string;
+        commit: string;
     }>;
 }, "strict", z.ZodTypeAny, {
     id: string;
     source: {
         path: string;
-        commit: string;
         repository: string;
+        commit: string;
     };
     schemaVersion: 1;
     kind: "help" | "query" | "prompt" | "workflow" | "mutation";
@@ -100,8 +100,8 @@ export declare const commandDefinitionSchema: z.ZodEffects<z.ZodObject<{
     id: string;
     source: {
         path: string;
-        commit: string;
         repository: string;
+        commit: string;
     };
     schemaVersion: 1;
     kind: "help" | "query" | "prompt" | "workflow" | "mutation";
@@ -126,8 +126,8 @@ export declare const commandDefinitionSchema: z.ZodEffects<z.ZodObject<{
     id: string;
     source: {
         path: string;
-        commit: string;
         repository: string;
+        commit: string;
     };
     schemaVersion: 1;
     kind: "help" | "query" | "prompt" | "workflow" | "mutation";
@@ -152,8 +152,8 @@ export declare const commandDefinitionSchema: z.ZodEffects<z.ZodObject<{
     id: string;
     source: {
         path: string;
-        commit: string;
         repository: string;
+        commit: string;
     };
     schemaVersion: 1;
     kind: "help" | "query" | "prompt" | "workflow" | "mutation";
@@ -244,19 +244,19 @@ export declare const catalogEntrySchema: z.ZodObject<{
             path: z.ZodString;
         }, "strict", z.ZodTypeAny, {
             path: string;
-            commit: string;
             repository: string;
+            commit: string;
         }, {
             path: string;
-            commit: string;
             repository: string;
+            commit: string;
         }>;
     }, "strict", z.ZodTypeAny, {
         id: string;
         source: {
             path: string;
-            commit: string;
             repository: string;
+            commit: string;
         };
         schemaVersion: 1;
         kind: "help" | "query" | "prompt" | "workflow" | "mutation";
@@ -281,8 +281,8 @@ export declare const catalogEntrySchema: z.ZodObject<{
         id: string;
         source: {
             path: string;
-            commit: string;
             repository: string;
+            commit: string;
         };
         schemaVersion: 1;
         kind: "help" | "query" | "prompt" | "workflow" | "mutation";
@@ -307,8 +307,8 @@ export declare const catalogEntrySchema: z.ZodObject<{
         id: string;
         source: {
             path: string;
-            commit: string;
             repository: string;
+            commit: string;
         };
         schemaVersion: 1;
         kind: "help" | "query" | "prompt" | "workflow" | "mutation";
@@ -333,8 +333,8 @@ export declare const catalogEntrySchema: z.ZodObject<{
         id: string;
         source: {
             path: string;
-            commit: string;
             repository: string;
+            commit: string;
         };
         schemaVersion: 1;
         kind: "help" | "query" | "prompt" | "workflow" | "mutation";
@@ -404,8 +404,8 @@ export declare const catalogEntrySchema: z.ZodObject<{
         id: string;
         source: {
             path: string;
-            commit: string;
             repository: string;
+            commit: string;
         };
         schemaVersion: 1;
         kind: "help" | "query" | "prompt" | "workflow" | "mutation";
@@ -445,8 +445,8 @@ export declare const catalogEntrySchema: z.ZodObject<{
         id: string;
         source: {
             path: string;
-            commit: string;
             repository: string;
+            commit: string;
         };
         schemaVersion: 1;
         kind: "help" | "query" | "prompt" | "workflow" | "mutation";
@@ -510,19 +510,19 @@ export declare const commandCatalogSchema: z.ZodObject<{
                 path: z.ZodString;
             }, "strict", z.ZodTypeAny, {
                 path: string;
-                commit: string;
                 repository: string;
+                commit: string;
             }, {
                 path: string;
-                commit: string;
                 repository: string;
+                commit: string;
             }>;
         }, "strict", z.ZodTypeAny, {
             id: string;
             source: {
                 path: string;
-                commit: string;
                 repository: string;
+                commit: string;
             };
             schemaVersion: 1;
             kind: "help" | "query" | "prompt" | "workflow" | "mutation";
@@ -547,8 +547,8 @@ export declare const commandCatalogSchema: z.ZodObject<{
             id: string;
             source: {
                 path: string;
-                commit: string;
                 repository: string;
+                commit: string;
             };
             schemaVersion: 1;
             kind: "help" | "query" | "prompt" | "workflow" | "mutation";
@@ -573,8 +573,8 @@ export declare const commandCatalogSchema: z.ZodObject<{
             id: string;
             source: {
                 path: string;
-                commit: string;
                 repository: string;
+                commit: string;
             };
             schemaVersion: 1;
             kind: "help" | "query" | "prompt" | "workflow" | "mutation";
@@ -599,8 +599,8 @@ export declare const commandCatalogSchema: z.ZodObject<{
             id: string;
             source: {
                 path: string;
-                commit: string;
                 repository: string;
+                commit: string;
             };
             schemaVersion: 1;
             kind: "help" | "query" | "prompt" | "workflow" | "mutation";
@@ -670,8 +670,8 @@ export declare const commandCatalogSchema: z.ZodObject<{
             id: string;
             source: {
                 path: string;
-                commit: string;
                 repository: string;
+                commit: string;
             };
             schemaVersion: 1;
             kind: "help" | "query" | "prompt" | "workflow" | "mutation";
@@ -711,8 +711,8 @@ export declare const commandCatalogSchema: z.ZodObject<{
             id: string;
             source: {
                 path: string;
-                commit: string;
                 repository: string;
+                commit: string;
             };
             schemaVersion: 1;
             kind: "help" | "query" | "prompt" | "workflow" | "mutation";
@@ -754,8 +754,8 @@ export declare const commandCatalogSchema: z.ZodObject<{
             id: string;
             source: {
                 path: string;
-                commit: string;
                 repository: string;
+                commit: string;
             };
             schemaVersion: 1;
             kind: "help" | "query" | "prompt" | "workflow" | "mutation";
@@ -800,8 +800,8 @@ export declare const commandCatalogSchema: z.ZodObject<{
             id: string;
             source: {
                 path: string;
-                commit: string;
                 repository: string;
+                commit: string;
             };
             schemaVersion: 1;
             kind: "help" | "query" | "prompt" | "workflow" | "mutation";

package/dist/src/parser.js

@@ -134,7 +134,6 @@ export class CommandParser {
         // command, because that's what caused `daemon start` to resolve as `start`
         // with `daemon` left as a positional.
         let resolvedCmd;
-        let resolvedSub;
         const resolvedCommands = [];
         let sawFirstPositional = false;
         for (let index = 0; index < args.length; index++) {
@@ -173,16 +172,16 @@ export class CommandParser {
                 const child = parent.subcommands.find(sc => sc.name === arg || sc.aliases?.includes(arg));
                 if (child) {
                     resolvedCommands.push(child);
-                    resolvedSub ??= child;
                     continue;
                 }
             }
             // A positional that is not the next subcommand ends the command path.
             break;
         }
-        const { aliases, booleanFlags, stringFlags } = this.getFlagScope(resolvedCommands);
+        const { aliases, booleanFlags, stringFlags, arrayFlags } = this.getFlagScope(resolvedCommands);
         let i = 0;
         let parsingFlags = true;
+        let commandSlotAvailable = true;
         while (i < args.length) {
             const arg = args[i];
             // Check for end of flags marker
@@ -194,8 +193,17 @@ export class CommandParser {
             // Handle flags
             if (parsingFlags && arg.startsWith('-')) {
                 const parseResult = this.parseFlag(args, i, aliases, booleanFlags, stringFlags);
+                // Array declarations retain text and accumulate repeated occurrences.
+                for (const key of arrayFlags) {
+                    const value = parseResult.flags[key];
+                    if (value !== undefined && value !== true && value !== false) {
+                        const previous = result.flags[key];
+                        parseResult.flags[key] = [...(Array.isArray(previous) ? previous : []), String(value)];
+                    }
+                }
                 // Apply to result flags
-                Object.assign(result.flags, parseResult.flags);
+                const { _: ignoredPositionals, ...parsedFlags } = parseResult.flags;
+                Object.assign(result.flags, parsedFlags);
                 i = parseResult.nextIndex;
                 continue;
             }
@@ -203,7 +211,9 @@ export class CommandParser {
             // Fix for #1596: treat lazy command names as commands here too so that
             // downstream dispatch sees `commandPath = ['daemon', 'start']` instead of
             // `commandPath = ['start'], positional = ['daemon']`.
-            if (result.command.length === 0 && this.isKnownCommandName(arg)) {
+            const isCommandSlot = commandSlotAvailable && parsingFlags;
+            commandSlotAvailable = false;
+            if (isCommandSlot && this.isKnownCommandName(arg)) {
                 // This is a command
                 result.command.push(arg);
                 // Check for subcommand (level 1) — only possible for sync commands
@@ -248,7 +258,7 @@ export class CommandParser {
         // `default: true` silently dropped and reached the action handler as
         // `undefined`. That trapped `memory store --upsert` and every other
         // subcommand that leaned on a per-flag default.
-        this.applyDefaults(result.flags, resolvedCmd, resolvedSub);
+        this.applyDefaults(result.flags, resolvedCommands);
         return result;
     }
     parseFlag(args, index, aliases, booleanFlags, stringFlags) {
@@ -472,11 +482,18 @@ export class CommandParser {
                     aliases[option.short] = option.name;
             }
         }
+        const arrayFlags = new Set((this.options.arrayFlags ?? []).map(flag => this.normalizeKey(flag)));
+        for (const flag of arrayFlags)
+            stringFlags.add(flag);
         const layers = [this.globalOptions, ...commands.map(command => command.options ?? [])];
         for (const options of layers) {
             for (const option of options) {
                 const key = this.normalizeKey(option.name);
-                if (option.type === 'string')
+                if (option.type === 'array')
+                    arrayFlags.add(key);
+                else if (option.type)
+                    arrayFlags.delete(key);
+                if (option.type === 'string' || option.type === 'array')
                     stringFlags.add(key);
                 else if (option.type)
                     stringFlags.delete(key);
@@ -488,7 +505,7 @@ export class CommandParser {
         }
         for (const flag of stringFlags)
             booleanFlags.delete(flag);
-        return { aliases, booleanFlags, stringFlags };
+        return { aliases, booleanFlags, stringFlags, arrayFlags };
     }
     getBooleanFlags() {
         const flags = new Set();
@@ -526,23 +543,16 @@ export class CommandParser {
         }
         return flags;
     }
-    applyDefaults(flags, command, subcommand) {
-        // #2775: apply defaults from globals AND the resolved command/subcommand.
-        // Subcommand > command > global (later writes lose to earlier — because
-        // we only set when `undefined`, so the FIRST option definition that
-        // supplies a default wins; walk narrow-to-broad so subcommand options
-        // apply before broader ones do).
-        const layers = [];
-        if (subcommand?.options)
-            layers.push(subcommand.options);
-        if (command?.options)
-            layers.push(command.options);
+    applyDefaults(flags, commands) {
+        // Walk every resolved command from narrow to broad so nested defaults
+        // have the same precedence as their declared flag types.
+        const layers = [...commands].reverse().map(command => command.options ?? []);
         layers.push(this.globalOptions);
         for (const layer of layers) {
             for (const opt of layer) {
                 const key = this.normalizeKey(opt.name);
                 if (flags[key] === undefined && opt.default !== undefined) {
-                    flags[key] = opt.default;
+                    flags[key] = (Array.isArray(opt.default) ? [...opt.default] : opt.default);
                 }
             }
         }
@@ -551,7 +561,7 @@ export class CommandParser {
             for (const [key, value] of Object.entries(this.options.defaults)) {
                 const normalizedKey = this.normalizeKey(key);
                 if (flags[normalizedKey] === undefined) {
-                    flags[normalizedKey] = value;
+                    flags[normalizedKey] = (Array.isArray(value) ? [...value] : value);
                 }
             }
         }
@@ -572,7 +582,7 @@ export class CommandParser {
             // followed by another flag) is parsed as boolean `true`. Commands then treat `true` as
             // the value: `hooks route --task` crashed with "task.trim is not a function" and
             // `memory retrieve --key` looked up the key "true". Report it instead.
-            if ((opt.type === 'string' || opt.type === 'number') && flags[key] === true) {
+            if ((opt.type === 'string' || opt.type === 'number' || opt.type === 'array') && flags[key] === true) {
                 errors.push(`Option --${opt.name} needs a value`);
             }
             // Check choices

package/dist/src/production/monitoring.d.ts

@@ -71,6 +71,7 @@ export declare class MonitoringHooks {
     private healthStatus;
     private startTime;
     private activeRequests;
+    private requestGeneration;
     private requestCount;
     private errorCount;
     private healthChecks;

package/dist/src/production/monitoring.js

@@ -37,6 +37,7 @@ export class MonitoringHooks {
     healthStatus;
     startTime = Date.now();
     activeRequests = 0;
+    requestGeneration = 0;
     requestCount = 0;
     errorCount = 0;
     healthChecks = new Map();
@@ -101,9 +102,14 @@ export class MonitoringHooks {
         this.activeRequests++;
         this.requestCount++;
         const startTime = Date.now();
+        const generation = this.requestGeneration;
+        let completed = false;
         this.counter('request_started', 1, { requestId });
         // Return end function
         return () => {
+            if (completed || generation !== this.requestGeneration)
+                return;
+            completed = true;
             this.activeRequests--;
             const duration = Date.now() - startTime;
             this.responseTimes.push(duration);
@@ -156,7 +162,7 @@ export class MonitoringHooks {
             catch (error) {
                 checks[name] = {
                     status: 'unhealthy',
-                    message: error.message,
+                    message: error instanceof Error ? error.message : String(error),
                     lastCheck: Date.now(),
                     responseTimeMs: Date.now() - startTime,
                 };
@@ -168,13 +174,13 @@ export class MonitoringHooks {
             checks,
             timestamp: Date.now(),
         };
-        return this.healthStatus;
+        return structuredClone(this.healthStatus);
     }
     /**
      * Get current health status
      */
     getHealthStatus() {
-        return this.healthStatus;
+        return structuredClone(this.healthStatus);
     }
     // ============================================================================
     // Performance Metrics
@@ -217,7 +223,7 @@ export class MonitoringHooks {
         if (level) {
             filtered = filtered.filter(a => a.level === level);
         }
-        return filtered;
+        return filtered.map(alert => ({ ...alert }));
     }
     /**
      * Acknowledge an alert
@@ -248,7 +254,7 @@ export class MonitoringHooks {
         if (since) {
             filtered = filtered.filter(m => m.timestamp >= since);
         }
-        return filtered;
+        return filtered.map(metric => ({ ...metric, labels: { ...metric.labels } }));
     }
     /**
      * Get all metrics summary
@@ -278,6 +284,7 @@ export class MonitoringHooks {
      * Reset all metrics
      */
     reset() {
+        this.requestGeneration++;
         this.metrics = [];
         this.responseTimes = [];
         this.alerts = [];
@@ -308,7 +315,13 @@ export class MonitoringHooks {
             const recentAlert = this.alerts.find(a => a.metric === name &&
                 !a.acknowledged &&
                 Date.now() - a.timestamp < 60000);
-            if (!recentAlert) {
+            if (recentAlert && level === 'critical' && recentAlert.level !== 'critical') {
+                Object.assign(recentAlert, {
+                    level, value, threshold, timestamp: Date.now(),
+                    message: `${name} exceeded ${level} threshold: ${value} >= ${threshold}`,
+                });
+            }
+            else if (!recentAlert) {
                 this.alerts.push({
                     id: `alert_${Date.now()}_${Math.random().toString(36).substring(7)}`,
                     level,

package/dist/src/production/rate-limiter.js

@@ -90,6 +90,7 @@ export class RateLimiter {
      */
     getStatus(operation, userId) {
         const limits = this.getLimits(operation);
+        const capacity = Math.floor(limits.maxRequests * this.config.burstMultiplier);
         const bucketKey = userId && this.config.perUserLimits
             ? `${operation}:${userId}`
             : `global:${operation}`;
@@ -98,8 +99,8 @@ export class RateLimiter {
         if (!bucket) {
             return {
                 current: 0,
-                limit: limits.maxRequests,
-                remaining: limits.maxRequests,
+                limit: capacity,
+                remaining: capacity,
                 resetAt: now + limits.windowMs,
             };
         }
@@ -107,8 +108,8 @@ export class RateLimiter {
         const validRequests = bucket.requests.filter(t => t > now - limits.windowMs);
         return {
             current: validRequests.length,
-            limit: limits.maxRequests,
-            remaining: Math.max(0, limits.maxRequests - validRequests.length),
+            limit: capacity,
+            remaining: Math.max(0, capacity - validRequests.length),
             resetAt: validRequests.length > 0
                 ? validRequests[0] + limits.windowMs
                 : now + limits.windowMs,

package/dist/src/production/retry.js

@@ -74,6 +74,9 @@ function fibonacci(n) {
  * Check if an error should be retried
  */
 function shouldRetryError(error, attempt, config) {
+    // Exhaustion always takes precedence over custom retry policies.
+    if (attempt >= config.maxAttempts)
+        return false;
     // Check custom retry function
     if (config.shouldRetry) {
         return config.shouldRetry(error, attempt);

package/dist/src/ruvector/agent-wasm.d.ts

@@ -70,6 +70,16 @@ export interface ToolResult {
     success: boolean;
     output: string;
 }
+/** Shown wherever the optional @ruvector/rvagent-wasm package cannot be loaded. */
+export declare const AGENT_WASM_NOT_INSTALLED_MESSAGE: string;
+/**
+ * True when err is the module-resolution failure for @ruvector/rvagent-wasm itself.
+ * Decided by the error code OR its text, because wrappers (loaders, test harnesses)
+ * can drop `code` while keeping the message.
+ */
+export declare function isMissingAgentWasm(err: unknown): boolean;
+/** The error initAgentWasm() throws for a load failure: "not installed" with the fix, or a genuine init failure. */
+export declare function agentWasmLoadError(err: unknown): Error;
 /**
  * Check if @ruvector/rvagent-wasm is installed and loadable.
  */

package/dist/src/ruvector/agent-wasm.js

@@ -14,6 +14,30 @@ import { readFileSync } from 'node:fs';
 import { createRequire } from 'node:module';
 // ── WASM Module Detection & Init ─────────────────────────────
 let _wasmReady = false;
+/** Shown wherever the optional @ruvector/rvagent-wasm package cannot be loaded. */
+export const AGENT_WASM_NOT_INSTALLED_MESSAGE = '@ruvector/rvagent-wasm is not installed.\n' +
+    'Install it with: npm install @ruvector/rvagent-wasm (or npm install -g @ruvector/rvagent-wasm for a global ruflo)';
+/**
+ * True when err is the module-resolution failure for @ruvector/rvagent-wasm itself.
+ * Decided by the error code OR its text, because wrappers (loaders, test harnesses)
+ * can drop `code` while keeping the message.
+ */
+export function isMissingAgentWasm(err) {
+    const e = err;
+    const code = typeof e?.code === 'string' ? e.code : '';
+    const message = typeof e?.message === 'string' ? e.message : String(err);
+    if (!message.includes('@ruvector/rvagent-wasm'))
+        return false;
+    return (code === 'ERR_MODULE_NOT_FOUND' ||
+        code === 'MODULE_NOT_FOUND' ||
+        /ERR_MODULE_NOT_FOUND|MODULE_NOT_FOUND|Cannot find (package|module)/.test(message));
+}
+/** The error initAgentWasm() throws for a load failure: "not installed" with the fix, or a genuine init failure. */
+export function agentWasmLoadError(err) {
+    return isMissingAgentWasm(err)
+        ? new Error(AGENT_WASM_NOT_INSTALLED_MESSAGE)
+        : new Error(`Failed to initialize @ruvector/rvagent-wasm: ${err}`);
+}
 /**
  * Check if @ruvector/rvagent-wasm is installed and loadable.
  */
@@ -43,7 +67,9 @@ export async function initAgentWasm() {
         _wasmReady = true;
     }
     catch (err) {
-        throw new Error(`Failed to initialize @ruvector/rvagent-wasm: ${err}`);
+        // #3979 item 1: a missing optional package is "not installed", with the fix,
+        // not a raw ERR_MODULE_NOT_FOUND trace dressed up as an initialization failure.
+        throw agentWasmLoadError(err);
     }
 }
 // ── Agent Registry ───────────────────────────────────────────

package/dist/src/ruvector/diff-classifier.d.ts

@@ -145,10 +145,8 @@ export declare function assessFileRisk(file: DiffFile): FileRisk;
  * Assess overall risk from files and file risks
  */
 export declare function assessOverallRisk(files: DiffFile[], fileRisks: FileRisk[]): OverallRisk;
-/**
- * Classify a diff based on files (uses singleton classifier)
- */
-export declare function classifyDiff(files: DiffFile[]): DiffClassification;
+/** Classify file metadata with the actual patch and, for one-commit ranges, its subject. */
+export declare function classifyDiff(files: DiffFile[], patch?: string, commitSubject?: string): DiffClassification;
 /**
  * Suggest reviewers based on files and risks
  */