@skaleagents/swarm 0.6.3 → 0.6.4

package/dist/review.js

@@ -107,6 +107,30 @@ export function architectureFindings(content, focus) {
             detail: "Detected an explicit compute size. Compare the selected size with observed utilization and set a review point for scale-up and scale-down decisions.",
         });
     }
+    if (shouldInclude(focusValue, "reliability") &&
+        /(?:^|[^.\w])(?:timeout|read_timeout|connect_timeout|request_timeout|readTimeout|connectTimeout)\s*[:=]\s*(?:0|false|none)\b/i.test(content)) {
+        findings.push({
+            severity: "medium",
+            title: "Outbound call has no timeout",
+            detail: "A timeout is set to zero, false, or none. Set a finite timeout so a stalled dependency cannot hold the process open.",
+        });
+    }
+    if (shouldInclude(focusValue, "reliability") &&
+        /(?:pool_size|max_connections|connectionLimit|maxPoolSize|maximumPoolSize|max_pool_size)\s*[:=]\s*1\b/.test(content)) {
+        findings.push({
+            severity: "high",
+            title: "Connection pool is capped at one",
+            detail: "The pool allows one connection. Concurrent requests will queue behind that single connection.",
+        });
+    }
+    if (shouldInclude(focusValue, "reliability") &&
+        /(?:max_?retries|retry_?count|retries)\s*[:=]\s*(?:-1|Infinity)\b/i.test(content)) {
+        findings.push({
+            severity: "medium",
+            title: "Retry limit is unbounded",
+            detail: "Retries are set to unlimited. Cap retries and add backoff so a failing dependency cannot amplify load.",
+        });
+    }
     return findings.map((finding, index) => {
         if (index === 0)
             return finding;
@@ -122,6 +146,9 @@ export function architectureFindings(content, focus) {
             "Public data access pattern": /(?:public-read|publicRead|allUsers)/i,
             "Debug mode enabled": /(?:debug|app_debug)\s*[:=]\s*["']?(?:true|1|yes)/i,
             "Compute sizing needs review": /(?:instance_type|machine_type|vm_size)\s*[:=]/i,
+            "Outbound call has no timeout": /(?:^|[^.\w])(?:timeout|read_timeout|connect_timeout|request_timeout|readTimeout|connectTimeout)\s*[:=]\s*(?:0|false|none)\b/i,
+            "Connection pool is capped at one": /(?:pool_size|max_connections|connectionLimit|maxPoolSize|maximumPoolSize|max_pool_size)\s*[:=]\s*1\b/,
+            "Retry limit is unbounded": /(?:max_?retries|retry_?count|retries)\s*[:=]\s*(?:-1|Infinity)\b/i,
         };
         const pattern = matchers[finding.title];
         const matches = pattern
@@ -142,7 +169,13 @@ export function architectureFindings(content, focus) {
             ruleId: `APP${String(ruleIndex).padStart(3, "0")}`,
             category: (finding.title === "Compute sizing needs review"
                 ? "cost"
-                : ["Debug mode enabled", "Unpinned container image"].includes(finding.title)
+                : [
+                    "Debug mode enabled",
+                    "Unpinned container image",
+                    "Outbound call has no timeout",
+                    "Connection pool is capped at one",
+                    "Retry limit is unbounded",
+                ].includes(finding.title)
                     ? "reliability"
                     : "security"),
             remediation: finding.detail,

package/dist/server.js

@@ -56,7 +56,7 @@ export function createServer(remote = false) {
     registerConsultation(server, remote);
     server.registerTool("review_architecture", {
         title: "Review architecture",
-        description: "Review individual source snippets or parsed infrastructure for security, reliability, and cost risks. For a whole application or Next.js architecture consultation, use plan_architecture_review and review_application_architecture instead.",
+        description: "Review individual source snippets or parsed infrastructure for security, reliability, and cost risks. Application checks include infrastructure bottlenecks: a disabled timeout, a one-connection pool, and an unbounded retry limit. For a whole application or Next.js architecture consultation, use plan_architecture_review and review_application_architecture instead.",
         annotations: {
             readOnlyHint: true,
             destructiveHint: false,
@@ -108,6 +108,7 @@ export function createServer(remote = false) {
                 truncated: findings.length > maxFindings,
                 limitations: [
                     "Application checks are text patterns, not a language-aware or runtime analysis. Findings do not establish that code is safe.",
+                    "Infrastructure bottleneck checks cover a disabled timeout, a one-connection pool, and an unbounded retry limit.",
                 ],
                 botHints: await botHints,
             });

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const VERSION = "0.6.3";
+export declare const VERSION = "0.6.4";

package/dist/version.js

@@ -1 +1 @@
-export const VERSION = "0.6.3";
+export const VERSION = "0.6.4";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skaleagents/swarm",
-  "version": "0.6.3",
+  "version": "0.6.4",
   "private": false,
   "publishConfig": {
     "access": "public"