@defai.digital/automatosx 11.3.3 → 11.3.4

package/dist/mcp/index.js

@@ -2012,251 +2012,148 @@ function isRateLimitError(error, providerName) {
 function isLimitError(error, providerName) {
   return isQuotaError(error, providerName) || isRateLimitError(error, providerName);
 }
-var PROVIDER_ERROR_PATTERNS, GENERIC_ERROR_PATTERNS;
+var GEMINI_PATTERNS, CLAUDE_PATTERNS, OPENAI_PATTERNS, AX_CLI_PATTERNS, PROVIDER_ERROR_PATTERNS, GENERIC_ERROR_PATTERNS;
 var init_error_patterns = __esm({
   "src/providers/error-patterns.ts"() {
     init_esm_shims();
+    GEMINI_PATTERNS = {
+      quota: [
+        "RESOURCE_EXHAUSTED",
+        "resource_exhausted",
+        "quotaExceeded",
+        "quota exceeded",
+        "quota limit reached",
+        "daily quota exceeded",
+        "monthly quota exceeded",
+        "insufficient quota"
+      ],
+      rateLimit: [
+        "RATE_LIMIT_EXCEEDED",
+        "rate_limit_exceeded",
+        "rate limit exceeded",
+        "too many requests",
+        "requests per minute exceeded",
+        "requests per day exceeded"
+      ],
+      statusCodes: [429],
+      // Too Many Requests
+      errorCodes: [
+        "RESOURCE_EXHAUSTED",
+        "RATE_LIMIT_EXCEEDED",
+        "QUOTA_EXCEEDED"
+      ]
+    };
+    CLAUDE_PATTERNS = {
+      quota: [
+        "insufficient_quota",
+        "quota_exceeded",
+        "quota exceeded",
+        "credit limit reached",
+        "usage limit exceeded",
+        "monthly quota exceeded"
+      ],
+      rateLimit: [
+        "rate_limit_error",
+        "rate limit exceeded",
+        "overloaded_error",
+        "overloaded",
+        "too many requests",
+        "requests per minute exceeded"
+      ],
+      statusCodes: [429, 529],
+      // Too Many Requests, Overloaded
+      errorCodes: [
+        "rate_limit_error",
+        "overloaded_error",
+        "insufficient_quota"
+      ]
+    };
+    OPENAI_PATTERNS = {
+      quota: [
+        "insufficient_quota",
+        "quota_exceeded",
+        "quota exceeded",
+        "billing hard limit reached",
+        "usage limit exceeded",
+        "monthly quota exceeded",
+        "credit limit reached"
+      ],
+      rateLimit: [
+        "rate_limit_exceeded",
+        "rate limit exceeded",
+        "too_many_requests",
+        "too many requests",
+        "requests per minute exceeded",
+        "tokens per minute exceeded",
+        "rate limit reached"
+      ],
+      statusCodes: [429],
+      // Too Many Requests
+      errorCodes: [
+        "insufficient_quota",
+        "rate_limit_exceeded",
+        "quota_exceeded",
+        "billing_hard_limit_reached"
+      ]
+    };
+    AX_CLI_PATTERNS = {
+      quota: [
+        // GLM patterns
+        "quota exceeded",
+        "quota limit reached",
+        "insufficient quota",
+        // OpenAI patterns (via ax-cli)
+        "insufficient_quota",
+        "quota_exceeded",
+        "billing hard limit reached",
+        "usage limit exceeded",
+        "monthly quota exceeded",
+        "credit limit reached",
+        // Generic patterns
+        "daily quota exceeded",
+        "api quota exceeded"
+      ],
+      rateLimit: [
+        // Common patterns
+        "rate_limit_exceeded",
+        "rate limit exceeded",
+        "too_many_requests",
+        "too many requests",
+        "requests per minute exceeded",
+        "tokens per minute exceeded",
+        "rate limit reached",
+        // Anthropic patterns (via ax-cli)
+        "rate_limit_error",
+        "overloaded_error",
+        "overloaded",
+        // xAI/Grok patterns
+        "throttled",
+        "request throttled"
+      ],
+      statusCodes: [429, 529],
+      errorCodes: [
+        "insufficient_quota",
+        "rate_limit_exceeded",
+        "quota_exceeded",
+        "rate_limit_error",
+        "overloaded_error",
+        "RATE_LIMIT_EXCEEDED",
+        "QUOTA_EXCEEDED"
+      ]
+    };
     PROVIDER_ERROR_PATTERNS = {
-      /**
-       * Google Gemini Error Patterns
-       *
-       * Gemini uses gRPC-style error codes and returns structured errors.
-       * Common quota errors include RESOURCE_EXHAUSTED and quotaExceeded.
-       */
-      gemini: {
-        quota: [
-          "RESOURCE_EXHAUSTED",
-          "resource_exhausted",
-          "quotaExceeded",
-          "quota exceeded",
-          "quota limit reached",
-          "daily quota exceeded",
-          "monthly quota exceeded",
-          "insufficient quota"
-        ],
-        rateLimit: [
-          "RATE_LIMIT_EXCEEDED",
-          "rate_limit_exceeded",
-          "rate limit exceeded",
-          "too many requests",
-          "requests per minute exceeded",
-          "requests per day exceeded"
-        ],
-        statusCodes: [429],
-        // Too Many Requests
-        errorCodes: [
-          "RESOURCE_EXHAUSTED",
-          "RATE_LIMIT_EXCEEDED",
-          "QUOTA_EXCEEDED"
-        ]
-      },
-      /**
-       * Anthropic Claude Error Patterns
-       *
-       * Claude returns different error types for rate limiting vs overload.
-       * Status 529 indicates temporary overload, 429 indicates rate limiting.
-       */
-      claude: {
-        quota: [
-          "insufficient_quota",
-          "quota_exceeded",
-          "quota exceeded",
-          "credit limit reached",
-          "usage limit exceeded",
-          "monthly quota exceeded"
-        ],
-        rateLimit: [
-          "rate_limit_error",
-          "rate limit exceeded",
-          "overloaded_error",
-          "overloaded",
-          "too many requests",
-          "requests per minute exceeded"
-        ],
-        statusCodes: [429, 529],
-        // Too Many Requests, Overloaded
-        errorCodes: [
-          "rate_limit_error",
-          "overloaded_error",
-          "insufficient_quota"
-        ]
-      },
-      /**
-       * OpenAI Error Patterns
-       *
-       * OpenAI provides detailed error messages for different types of limits.
-       * Distinguishes between quota exhaustion and rate limiting.
-       */
-      openai: {
-        quota: [
-          "insufficient_quota",
-          "quota_exceeded",
-          "quota exceeded",
-          "billing hard limit reached",
-          "usage limit exceeded",
-          "monthly quota exceeded",
-          "credit limit reached"
-        ],
-        rateLimit: [
-          "rate_limit_exceeded",
-          "rate limit exceeded",
-          "too_many_requests",
-          "too many requests",
-          "requests per minute exceeded",
-          "tokens per minute exceeded",
-          "rate limit reached"
-        ],
-        statusCodes: [429],
-        // Too Many Requests
-        errorCodes: [
-          "insufficient_quota",
-          "rate_limit_exceeded",
-          "quota_exceeded",
-          "billing_hard_limit_reached"
-        ]
-      },
-      /**
-       * Claude Code Provider (extension of Claude patterns)
-       */
-      "claude-code": {
-        quota: [
-          "insufficient_quota",
-          "quota_exceeded",
-          "quota exceeded",
-          "credit limit reached",
-          "usage limit exceeded",
-          "monthly quota exceeded"
-        ],
-        rateLimit: [
-          "rate_limit_error",
-          "rate limit exceeded",
-          "overloaded_error",
-          "overloaded",
-          "too many requests",
-          "requests per minute exceeded"
-        ],
-        statusCodes: [429, 529],
-        errorCodes: [
-          "rate_limit_error",
-          "overloaded_error",
-          "insufficient_quota"
-        ]
-      },
-      /**
-       * Gemini CLI Provider (same patterns as Gemini)
-       */
-      "gemini-cli": {
-        quota: [
-          "RESOURCE_EXHAUSTED",
-          "resource_exhausted",
-          "quotaExceeded",
-          "quota exceeded",
-          "quota limit reached",
-          "daily quota exceeded",
-          "monthly quota exceeded",
-          "insufficient quota"
-        ],
-        rateLimit: [
-          "RATE_LIMIT_EXCEEDED",
-          "rate_limit_exceeded",
-          "rate limit exceeded",
-          "too many requests",
-          "requests per minute exceeded",
-          "requests per day exceeded"
-        ],
-        statusCodes: [429],
-        errorCodes: [
-          "RESOURCE_EXHAUSTED",
-          "RATE_LIMIT_EXCEEDED",
-          "QUOTA_EXCEEDED"
-        ]
-      },
-      /**
-       * Codex CLI Provider (same patterns as OpenAI)
-       */
-      codex: {
-        quota: [
-          "insufficient_quota",
-          "quota_exceeded",
-          "quota exceeded",
-          "billing hard limit reached",
-          "usage limit exceeded",
-          "monthly quota exceeded",
-          "credit limit reached"
-        ],
-        rateLimit: [
-          "rate_limit_exceeded",
-          "rate limit exceeded",
-          "too_many_requests",
-          "too many requests",
-          "requests per minute exceeded",
-          "tokens per minute exceeded",
-          "rate limit reached"
-        ],
-        statusCodes: [429],
-        errorCodes: [
-          "insufficient_quota",
-          "rate_limit_exceeded",
-          "quota_exceeded",
-          "billing_hard_limit_reached"
-        ]
-      },
-      /**
-       * ax-cli Provider Error Patterns (v9.2.0)
-       *
-       * ax-cli is a multi-model provider supporting GLM, xAI, OpenAI, Anthropic, Ollama, etc.
-       * BUG FIX: Added missing patterns - previously fell back to generic patterns which
-       * may miss provider-specific errors from the underlying model providers.
-       *
-       * Combines patterns from all supported backends since ax-cli proxies to them.
-       */
-      "ax-cli": {
-        quota: [
-          // GLM patterns
-          "quota exceeded",
-          "quota limit reached",
-          "insufficient quota",
-          // OpenAI patterns (via ax-cli)
-          "insufficient_quota",
-          "quota_exceeded",
-          "billing hard limit reached",
-          "usage limit exceeded",
-          "monthly quota exceeded",
-          "credit limit reached",
-          // Anthropic patterns (via ax-cli)
-          "credit limit reached",
-          // Generic patterns
-          "daily quota exceeded",
-          "api quota exceeded"
-        ],
-        rateLimit: [
-          // Common patterns
-          "rate_limit_exceeded",
-          "rate limit exceeded",
-          "too_many_requests",
-          "too many requests",
-          "requests per minute exceeded",
-          "tokens per minute exceeded",
-          "rate limit reached",
-          // Anthropic patterns (via ax-cli)
-          "rate_limit_error",
-          "overloaded_error",
-          "overloaded",
-          // xAI/Grok patterns
-          "throttled",
-          "request throttled"
-        ],
-        statusCodes: [429, 529],
-        errorCodes: [
-          "insufficient_quota",
-          "rate_limit_exceeded",
-          "quota_exceeded",
-          "rate_limit_error",
-          "overloaded_error",
-          "RATE_LIMIT_EXCEEDED",
-          "QUOTA_EXCEEDED"
-        ]
-      }
+      // Primary providers
+      gemini: GEMINI_PATTERNS,
+      claude: CLAUDE_PATTERNS,
+      codex: OPENAI_PATTERNS,
+      "ax-cli": AX_CLI_PATTERNS,
+      // Aliases - reference same patterns to avoid duplication
+      "gemini-cli": GEMINI_PATTERNS,
+      // Alias for gemini
+      "claude-code": CLAUDE_PATTERNS,
+      // Alias for claude
+      "openai": OPENAI_PATTERNS
+      // Alias for codex (same underlying API)
     };
     GENERIC_ERROR_PATTERNS = {
       quota: [
@@ -2639,9 +2536,14 @@ var init_base_provider = __esm({
               });
               child.kill("SIGTERM");
               forceKillTimer = setTimeout(() => {
-                if (child.pid && !child.killed) {
-                  logger.warn("Force killing child process", { pid: child.pid });
-                  child.kill("SIGKILL");
+                if (child.pid) {
+                  try {
+                    process.kill(child.pid, 0);
+                    logger.warn("Force killing child process", { pid: child.pid });
+                    child.kill("SIGKILL");
+                  } catch {
+                    logger.debug("Process already exited before SIGKILL", { pid: child.pid });
+                  }
                 }
               }, _BaseProvider.SIGKILL_ESCALATION_MS);
             }
@@ -2822,9 +2724,14 @@ var init_base_provider = __esm({
               });
               child.kill("SIGTERM");
               forceKillTimer = setTimeout(() => {
-                if (child.pid && !child.killed) {
-                  logger.warn("Force killing child process", { pid: child.pid });
-                  child.kill("SIGKILL");
+                if (child.pid) {
+                  try {
+                    process.kill(child.pid, 0);
+                    logger.warn("Force killing child process", { pid: child.pid });
+                    child.kill("SIGKILL");
+                  } catch {
+                    logger.debug("Process already exited before SIGKILL", { pid: child.pid });
+                  }
                 }
               }, _BaseProvider.SIGKILL_ESCALATION_MS);
             }
@@ -4103,12 +4010,20 @@ var init_openai_provider = __esm({
     OpenAIProvider = class extends BaseProvider {
       hybridAdapter = null;
       providerConfig;
+      /** BUG FIX: Track pending initialization to prevent race condition where
+       * concurrent execute() calls could each trigger initializeHybridAdapter() */
+      initializationPromise = null;
+      /** BUG FIX: Track destroyed state to prevent use-after-destroy race condition */
+      isDestroyed = false;
       constructor(config) {
         super(config);
         this.providerConfig = config;
         logger.debug("[OpenAI/Codex] Initialized", { mode: config.mode || "auto" });
       }
       async execute(request) {
+        if (this.isDestroyed) {
+          throw new Error("OpenAIProvider has been destroyed and cannot execute requests");
+        }
         if (process.env.AX_MOCK_PROVIDERS === "true") {
           return {
             content: this.getMockResponse(),
@@ -4119,9 +4034,7 @@ var init_openai_provider = __esm({
           };
         }
         const startTime = Date.now();
-        if (!this.hybridAdapter) {
-          this.initializeHybridAdapter();
-        }
+        await this.ensureInitialized();
         try {
           const result = await this.hybridAdapter.execute(
             request.prompt,
@@ -4146,7 +4059,30 @@ var init_openai_provider = __esm({
           throw this.handleError(error);
         }
       }
-      initializeHybridAdapter() {
+      /**
+       * BUG FIX: Ensure adapter is initialized exactly once, even with concurrent calls.
+       * Uses a promise-based lock pattern to prevent race conditions where multiple
+       * execute() calls could each trigger initializeHybridAdapter().
+       */
+      async ensureInitialized() {
+        if (this.hybridAdapter) {
+          return;
+        }
+        if (this.initializationPromise) {
+          await this.initializationPromise;
+          return;
+        }
+        this.initializationPromise = this.initializeHybridAdapter();
+        try {
+          await this.initializationPromise;
+        } finally {
+          this.initializationPromise = null;
+        }
+      }
+      async initializeHybridAdapter() {
+        if (this.hybridAdapter) {
+          return;
+        }
         const options = {
           mode: this.providerConfig.mode || "auto",
           cli: {
@@ -4184,6 +4120,7 @@ var init_openai_provider = __esm({
         this.hybridAdapter?.switchToCliMode();
       }
       async destroy() {
+        this.isDestroyed = true;
         if (this.hybridAdapter) {
           await this.hybridAdapter.destroy();
           this.hybridAdapter = null;
@@ -18203,13 +18140,17 @@ init_logger();
 // src/providers/mcp/mcp-client.ts
 init_esm_shims();
 init_logger();
-var McpClient = class extends EventEmitter {
+var McpClient = class _McpClient extends EventEmitter {
   config;
   process = null;
   readline = null;
   state;
   pendingRequests = /* @__PURE__ */ new Map();
   nextRequestId = 1;
+  /** BUG FIX: Maximum request ID before wraparound to prevent Number precision loss.
+   * JavaScript loses integer precision after Number.MAX_SAFE_INTEGER (2^53 - 1).
+   * Using 1 billion as a safe wraparound point - still unique within any realistic session. */
+  static MAX_REQUEST_ID = 1e9;
   serverInfo = null;
   constructor(config) {
     super();
@@ -18399,6 +18340,9 @@ var McpClient = class extends EventEmitter {
       throw new Error("MCP client not connected");
     }
     const id = this.nextRequestId++;
+    if (this.nextRequestId > _McpClient.MAX_REQUEST_ID) {
+      this.nextRequestId = 1;
+    }
     const request = {
       jsonrpc: "2.0",
       id,
@@ -18729,10 +18673,13 @@ var McpClientPool = class extends EventEmitter {
       } else {
         this.removeFromPool(pool, pooledClient);
         this.emitEvent("connection_closed", provider, { reason: "connection_died" });
+        pool.pendingConnections++;
         this.createConnection(provider).then((newClient) => {
           pool.clients.push(this.createPooledClient(newClient));
           waiter.resolve(newClient);
-        }).catch(waiter.reject);
+        }).catch(waiter.reject).finally(() => {
+          pool.pendingConnections--;
+        });
       }
     }
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@defai.digital/automatosx",
-  "version": "11.3.3",
+  "version": "11.3.4",
   "description": "Provider-agnostic AI orchestration platform with 20+ specialized agents, persistent memory, and multi-provider routing for Claude Code, Gemini CLI, Codex CLI, and ax-cli",
   "type": "module",
   "publishConfig": {