@indiekitai/pg-dash 0.4.5 → 0.4.6

package/dist/mcp.js

@@ -522,30 +522,44 @@ SHOW shared_buffers;`,
     }
     try {
       const r = await client.query(`
-        SELECT pid, state, now() - state_change AS idle_duration,
+        SELECT pid,
           client_addr::text, application_name,
           extract(epoch from now() - state_change)::int AS idle_seconds
         FROM pg_stat_activity
-        WHERE state IN ('idle', 'idle in transaction')
+        WHERE state = 'idle in transaction'
           AND now() - state_change > $1 * interval '1 minute'
           AND pid != pg_backend_pid()
+        ORDER BY idle_seconds DESC
       `, [longQueryThreshold2]);
-      for (const row of r.rows) {
-        const isIdleTx = row.state === "idle in transaction";
+      if (r.rows.length === 1) {
+        const row = r.rows[0];
         issues.push({
-          id: `maint-idle-${row.pid}`,
-          severity: isIdleTx ? "warning" : "info",
+          id: `maint-idle-tx-${row.pid}`,
+          severity: "warning",
           category: "maintenance",
-          title: `${isIdleTx ? "Idle in transaction" : "Idle connection"} (PID ${row.pid})`,
-          description: `PID ${row.pid} from ${row.client_addr || "local"} (${row.application_name || "unknown"}) has been ${row.state} for ${Math.round(row.idle_seconds / 60)} minutes.`,
+          title: `Idle-in-transaction connection (PID ${row.pid})`,
+          description: `PID ${row.pid} from ${row.client_addr || "local"} (${row.application_name || "unknown"}) has been idle in transaction for ${Math.round(row.idle_seconds / 60)} minutes. This holds locks and blocks VACUUM.`,
           fix: `SELECT pg_terminate_backend(${row.pid});`,
-          impact: isIdleTx ? "Idle-in-transaction connections hold locks and prevent VACUUM." : "Idle connections consume connection slots.",
+          impact: "Idle-in-transaction connections hold locks and prevent VACUUM.",
+          effort: "quick"
+        });
+      } else if (r.rows.length > 1) {
+        const pids = r.rows.map((row) => row.pid);
+        const maxMin = Math.round(r.rows[0].idle_seconds / 60);
+        issues.push({
+          id: `maint-idle-tx-multi`,
+          severity: "warning",
+          category: "maintenance",
+          title: `${r.rows.length} idle-in-transaction connections (longest: ${maxMin}m)`,
+          description: `${r.rows.length} connections have been idle in transaction for over ${longQueryThreshold2} minutes. These hold locks and prevent VACUUM from running.`,
+          fix: `SELECT pg_terminate_backend(pid) FROM pg_stat_activity WHERE state = 'idle in transaction' AND now() - state_change > interval '${longQueryThreshold2} minutes';`,
+          impact: "Idle-in-transaction connections hold locks and prevent VACUUM.",
           effort: "quick"
         });
       }
     } catch (err) {
-      console.error("[advisor] Error checking idle connections:", err.message);
-      skipped.push("idle connections: " + err.message);
+      console.error("[advisor] Error checking idle-in-transaction connections:", err.message);
+      skipped.push("idle-in-transaction: " + err.message);
     }
     try {
       const r = await client.query(`