@desplega.ai/agent-swarm 1.131.1 → 1.134.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@desplega.ai/agent-swarm",
-  "version": "1.131.1",
+  "version": "1.134.0",
   "description": "agent-swarm.dev — an open-source operating system for AI work: a lead agent breaks goals into tasks, routes them to specialized workers such as Claude Code or Codex, runs each worker in an isolated container, and preserves shared memory, tools, schedules, and review gates so delegated work compounds across sessions.",
   "keywords": [
     "agent-swarm.dev",
@@ -169,14 +169,14 @@
     "@aws-sdk/client-bedrock": "3.1048.0",
     "@desplega.ai/business-use": "^0.4.2",
     "@desplega.ai/localtunnel": "^2.2.0",
-    "@earendil-works/pi-agent-core": "^0.84.1",
-    "@earendil-works/pi-ai": "^0.84.1",
-    "@earendil-works/pi-coding-agent": "^0.84.1",
+    "@earendil-works/pi-agent-core": "^0.84.2",
+    "@earendil-works/pi-ai": "^0.84.2",
+    "@earendil-works/pi-coding-agent": "^0.84.2",
     "@inkjs/ui": "^2.0.0",
     "@linear/sdk": "^77.0.0",
     "@modelcontextprotocol/sdk": "^1.25.1",
-    "@openai/codex-sdk": "^0.147.0",
-    "@opencode-ai/sdk": "^1.18.16",
+    "@openai/codex-sdk": "^0.148.0",
+    "@opencode-ai/sdk": "^1.18.18",
     "@openfort/openfort-node": "^0.9.1",
     "@opentelemetry/api": "^1.9.1",
     "@opentelemetry/exporter-metrics-otlp-http": "^0.218.0",

package/src/be/budget-refusal-notify.ts

@@ -14,6 +14,7 @@
 
 import { resolveTemplate } from "../prompts/resolver";
 import type { AgentTask } from "../types";
+import { scrubSecrets } from "../utils/secret-scrubber";
 import {
   createTaskExtended,
   getAgentById,
@@ -139,8 +140,12 @@ export function emitBudgetRefusalSideEffects(ctx: BudgetRefusalContext, inserted
   }
 
   // 2. Workflow event bus emit — every refusal, not just the first per day.
-  try {
-    import("../workflows/event-bus").then(({ workflowEventBus }) => {
+  // Mirrors the emit pattern in db.ts — any failure here (e.g. event bus module
+  // load error) must not break the refusal path. The rejection is caught on the
+  // promise rather than in a `try` block, because the callback runs after the
+  // surrounding frame has already returned.
+  import("../workflows/event-bus")
+    .then(({ workflowEventBus }) => {
       workflowEventBus.emit("task.budget_refused", {
         taskId: ctx.task.id,
         agentId: ctx.agentId,
@@ -153,9 +158,11 @@ export function emitBudgetRefusalSideEffects(ctx: BudgetRefusalContext, inserted
         userBudgetUsd: ctx.userBudgetUsd,
         resetAt: ctx.resetAt,
       });
-    });
-  } catch {
-    // Mirror the existing emit-pattern in db.ts:1561-1571 — any failure here
-    // (e.g. event bus module load error) must not break the refusal path.
-  }
+    })
+    .catch((err) =>
+      console.error(
+        "[budget-refusal-notify] task.budget_refused event not emitted:",
+        scrubSecrets(err instanceof Error ? err.message : String(err)),
+      ),
+    );
 }

package/src/be/db-queries/task-artifacts.ts

@@ -0,0 +1,182 @@
+import { extractGitHubPullRequestUrls } from "../../utils/github-pull-request";
+import { getDb } from "../db";
+
+export type TaskShippingEvidenceSource = "attachment" | "output-fallback" | "none";
+
+export interface TaskShippingEvidence {
+  hasArtifact: boolean;
+  hasPullRequest: boolean;
+  pullRequestUrls: string[];
+  pullRequestSource: TaskShippingEvidenceSource;
+}
+
+function taskAlias(alias: string): string {
+  if (!/^[A-Za-z_][A-Za-z0-9_]*$/.test(alias)) {
+    throw new Error(`Invalid task SQL alias: ${alias}`);
+  }
+  return alias;
+}
+
+// ECMAScript whitespace plus prose wrappers that terminate URL-like tokens.
+const githubTokenDelimiterCodePoints = [
+  9, 10, 11, 12, 13, 160, 5760, 8192, 8193, 8194, 8195, 8196, 8197, 8198, 8199, 8200, 8201, 8202,
+  8232, 8233, 8239, 8287, 12288, 65279, 40, 41, 91, 93, 123, 125, 60, 62, 34, 39, 96,
+];
+
+function githubTokenDelimitersSql(): string {
+  return githubTokenDelimiterCodePoints.map((code, index) => `(${index + 1}, ${code})`).join(", ");
+}
+
+/** Keep aggregate SQL matching aligned with extractGitHubPullRequestUrls(). */
+function githubPullRequestExistsSql(value: string): string {
+  return `EXISTS (
+    WITH RECURSIVE
+    token_delimiters(position, code_point) AS (
+      VALUES ${githubTokenDelimitersSql()}
+    ),
+    normalized(value, position) AS (
+      SELECT coalesce(${value}, ''), 0
+
+      UNION ALL
+
+      SELECT replace(value, char(code_point), ' '), normalized.position + 1
+      FROM normalized
+      JOIN token_delimiters ON token_delimiters.position = normalized.position + 1
+    ),
+    tokens(remaining, token) AS (
+      SELECT trim(value) || ' ', NULL
+      FROM normalized
+      WHERE position = (SELECT max(position) FROM token_delimiters)
+
+      UNION ALL
+
+      SELECT
+        ltrim(substr(remaining, instr(remaining, ' ') + 1)),
+        substr(remaining, 1, instr(remaining, ' ') - 1)
+      FROM tokens
+      WHERE remaining <> ''
+    ),
+    github_paths(path) AS (
+      SELECT CASE
+        WHEN lower(token) GLOB 'https://github.com/*'
+          THEN substr(token, length('https://github.com/') + 1)
+        WHEN lower(token) GLOB 'http://github.com/*'
+          THEN substr(token, length('http://github.com/') + 1)
+        ELSE substr(token, length('github.com/') + 1)
+      END
+      FROM tokens
+      WHERE lower(token) GLOB 'https://github.com/*'
+         OR lower(token) GLOB 'http://github.com/*'
+         OR lower(token) GLOB 'github.com/*'
+    ),
+    github_segments(owner, repo, remainder) AS (
+      SELECT
+        substr(path, 1, instr(path, '/') - 1),
+        substr(
+          substr(path, instr(path, '/') + 1),
+          1,
+          instr(substr(path, instr(path, '/') + 1), '/') - 1
+        ),
+        substr(
+          substr(path, instr(path, '/') + 1),
+          instr(substr(path, instr(path, '/') + 1), '/') + 1
+        )
+      FROM github_paths
+      WHERE instr(path, '/') > 1
+        AND instr(substr(path, instr(path, '/') + 1), '/') > 1
+    ),
+    github_pull_numbers(remainder, position) AS (
+      SELECT remainder, length('pull/') + 1
+      FROM github_segments
+      WHERE owner NOT GLOB '*[^A-Za-z0-9._-]*'
+        AND repo NOT GLOB '*[^A-Za-z0-9._-]*'
+        AND owner NOT IN ('.', '..')
+        AND repo NOT IN ('.', '..')
+        AND lower(remainder) GLOB 'pull/[0-9]*'
+
+      UNION ALL
+
+      SELECT remainder, position + 1
+      FROM github_pull_numbers
+      WHERE substr(remainder, position, 1) GLOB '[0-9]'
+    )
+    SELECT 1
+    FROM github_pull_numbers
+    WHERE substr(remainder, position, 1) NOT GLOB '[0-9]'
+      AND (
+        substr(remainder, position, 1) = ''
+        OR substr(remainder, position, 1) IN ('/', '?', '#')
+        OR trim(substr(remainder, position), ')]},.;:!') = ''
+      )
+    LIMIT 1
+  )`;
+}
+
+/**
+ * SQL expressions for aggregate task-reporting queries. Attachment rows are
+ * authoritative; output matching remains a compatibility fallback while old
+ * databases are upgraded/backfilled.
+ */
+export function taskShippingEvidenceSql(alias = "t"): {
+  hasArtifact: string;
+  hasPullRequest: string;
+} {
+  const t = taskAlias(alias);
+  const outputHasPullRequest = githubPullRequestExistsSql(`${t}.output`);
+  const hasAnyAttachment = `EXISTS (
+    SELECT 1 FROM task_attachments ta WHERE ta.task_id = ${t}.id
+  )`;
+  const hasPullRequestAttachment = `EXISTS (
+    SELECT 1 FROM task_attachments ta
+    WHERE ta.task_id = ${t}.id
+      AND ta.kind = 'url'
+      AND ${githubPullRequestExistsSql("ta.url")}
+  )`;
+
+  return {
+    hasArtifact: `CASE WHEN ${hasAnyAttachment} THEN 1 WHEN ${outputHasPullRequest} THEN 1 ELSE 0 END`,
+    hasPullRequest: `CASE WHEN ${hasPullRequestAttachment} THEN 1 WHEN ${outputHasPullRequest} THEN 1 ELSE 0 END`,
+  };
+}
+
+/** Look up one task's attachment-first shipping evidence. */
+export function getTaskShippingEvidence(taskId: string): TaskShippingEvidence | null {
+  const task = getDb()
+    .prepare<{ output: string | null }, [string]>("SELECT output FROM agent_tasks WHERE id = ?")
+    .get(taskId);
+  if (!task) return null;
+
+  const attachments = getDb()
+    .prepare<{ kind: string; url: string | null }, [string]>(
+      "SELECT kind, url FROM task_attachments WHERE task_id = ? ORDER BY created_at, rowid",
+    )
+    .all(taskId);
+  const attachmentPullRequests = Array.from(
+    new Set(
+      attachments.flatMap((attachment) =>
+        attachment.kind === "url"
+          ? extractGitHubPullRequestUrls(attachment.url).map((pullRequest) => pullRequest.url)
+          : [],
+      ),
+    ),
+  );
+
+  if (attachmentPullRequests.length > 0) {
+    return {
+      hasArtifact: true,
+      hasPullRequest: true,
+      pullRequestUrls: attachmentPullRequests,
+      pullRequestSource: "attachment",
+    };
+  }
+
+  const fallbackPullRequests = extractGitHubPullRequestUrls(task.output).map(
+    (pullRequest) => pullRequest.url,
+  );
+  return {
+    hasArtifact: attachments.length > 0 || fallbackPullRequests.length > 0,
+    hasPullRequest: fallbackPullRequests.length > 0,
+    pullRequestUrls: fallbackPullRequests,
+    pullRequestSource: fallbackPullRequests.length > 0 ? "output-fallback" : "none",
+  };
+}