aglio-worker 0.14.0 → 0.22.1

package/agent.js

@@ -27378,6 +27378,20 @@ var autoClaimScheduleSchema = external_exports.object({
   start: autoClaimTimeSchema.default("00:00"),
   end: autoClaimTimeSchema.default("00:00")
 }).default({ enabled: false, timezone: "UTC", days: [0, 1, 2, 3, 4, 5, 6], start: "00:00", end: "00:00" });
+var routingLabelsSchema = external_exports.array(external_exports.string()).max(16).superRefine((labels, ctx) => {
+  try {
+    validateRoutingLabels(labels);
+  } catch (error62) {
+    ctx.addIssue({
+      code: "custom",
+      message: error62 instanceof Error ? error62.message : "Invalid worker labels."
+    });
+  }
+});
+var cardRoutingOverrideSchema = external_exports.object({
+  requiredWorkerLabels: routingLabelsSchema.nullable().optional(),
+  planningRequiredWorkerLabels: routingLabelsSchema.nullable().optional()
+});
 var projectSettingsSchema = external_exports.object({
   /** Demo projects are sandbox rows: the worker never claims them and no model runs. */
   demoMode: external_exports.boolean().default(false),
@@ -27402,23 +27416,20 @@ var projectSettingsSchema = external_exports.object({
   runRetryBackoffSeconds: external_exports.number().int().min(10).max(600).default(60),
   runRetryMaxBackoffSeconds: external_exports.number().int().min(60).max(3600).default(600),
   notificationSound: external_exports.boolean().default(true),
+  /** Null inherits the workspace default. True or false overrides it for this project. */
+  requireVerificationEvidence: external_exports.boolean().nullable().default(null),
   /** Slack incoming webhook for question, failure and PR-ready alerts. Null disables Slack. */
   slackWebhookUrl: external_exports.string().trim().max(500).refine(isSlackWebhookUrl, "Slack webhook must be an https URL").nullable().default(null),
   /** Rendered column strip: built-in columns and custom list ids. */
   boardColumnOrder: external_exports.array(external_exports.string()).optional(),
   /** Last repository connectivity check from Project settings or the worker gate. */
   repoConnection: repoConnectionReportSchema.nullish(),
-  /** Worker labels every job in this project needs. Empty means any worker. */
-  requiredWorkerLabels: external_exports.array(external_exports.string()).max(16).default([]).superRefine((labels, ctx) => {
-    try {
-      validateRoutingLabels(labels);
-    } catch (error62) {
-      ctx.addIssue({
-        code: "custom",
-        message: error62 instanceof Error ? error62.message : "Invalid worker labels."
-      });
-    }
-  })
+  /** Build, consult and review job labels. Empty means any worker. */
+  requiredWorkerLabels: routingLabelsSchema.default([]),
+  /** Planning job labels. Empty means any online worker, so planning never waits on a build machine. */
+  planningRequiredWorkerLabels: routingLabelsSchema.default([]),
+  /** Undefined inherits the workspace switch. True or false overrides it. */
+  postBuildReply: external_exports.boolean().optional()
 });
 var repoConfigSchema = external_exports.object({
   id: external_exports.string(),
@@ -27671,6 +27682,18 @@ function artifactFileName(id, label) {
   return `${id}-${sanitizeFileName(label)}`;
 }
 
+// ../../packages/shared/src/feedback.ts
+var FEEDBACK_TYPES = ["bug", "feature"];
+var FEEDBACK_MAX_ATTACHMENT_BYTES = 10 * 1024 * 1024;
+var FEEDBACK_RATE_WINDOW_MS = 60 * 60 * 1e3;
+var feedbackTypeSchema = external_exports.enum(FEEDBACK_TYPES);
+var feedbackInputSchema = external_exports.object({
+  type: feedbackTypeSchema,
+  title: external_exports.string().trim().min(5, "Title must be at least 5 characters").max(140, "Title must be 140 characters or fewer"),
+  description: external_exports.string().trim().min(10, "Description must be at least 10 characters").max(5e3, "Description must be 5000 characters or fewer"),
+  pageUrl: external_exports.string().trim().max(500, "Page URL must be 500 characters or fewer").refine((value) => value === "" || URL.canParse(value), "Page URL must be a valid URL").optional().default("")
+});
+
 // ../../packages/shared/src/providers.ts
 var apiKey = (env, drivers, label = "API key") => ({
   type: "api_key",
@@ -28161,7 +28184,9 @@ var WORKSPACE_SETTINGS_DEFAULTS = {
   artifactRetentionDays: 0,
   defaultGitProvider: "github",
   /** The shared workspace consultant is opt-out. Disabled hides every entry point. */
-  consultantEnabled: true
+  consultantEnabled: true,
+  /** Post the agent's exact final build message as a card comment. */
+  postBuildReply: true
 };
 var workspaceSettingsSchema = external_exports.object({
   allowMemberInvites: external_exports.boolean().catch(WORKSPACE_SETTINGS_DEFAULTS.allowMemberInvites),
@@ -28169,7 +28194,8 @@ var workspaceSettingsSchema = external_exports.object({
   requireVerificationEvidence: external_exports.boolean().catch(WORKSPACE_SETTINGS_DEFAULTS.requireVerificationEvidence),
   artifactRetentionDays: external_exports.number().int().min(0).max(3650).catch(WORKSPACE_SETTINGS_DEFAULTS.artifactRetentionDays),
   defaultGitProvider: external_exports.string().optional().transform((value) => normalizeGitProvider(value)).catch(WORKSPACE_SETTINGS_DEFAULTS.defaultGitProvider),
-  consultantEnabled: external_exports.boolean().catch(WORKSPACE_SETTINGS_DEFAULTS.consultantEnabled)
+  consultantEnabled: external_exports.boolean().catch(WORKSPACE_SETTINGS_DEFAULTS.consultantEnabled),
+  postBuildReply: external_exports.boolean().catch(WORKSPACE_SETTINGS_DEFAULTS.postBuildReply)
 }).passthrough();
 
 // ../../packages/shared/src/theme.ts
@@ -28252,7 +28278,7 @@ var CARD_OPEN_SIZES = ["default", "large", "fullscreen"];
 var cardOpenSizeSchema = external_exports.enum(CARD_OPEN_SIZES);
 
 // src/remote-docker.ts
-import { existsSync } from "node:fs";
+import { existsSync, readFileSync as readFileSync2 } from "node:fs";
 import { PassThrough } from "node:stream";
 import { join as join2, posix, relative, resolve } from "node:path";
 import Dockerode from "dockerode";
@@ -28477,6 +28503,17 @@ var CONTAINER_WORKSPACE = "/workspace";
 function isRunningInContainer() {
   return existsSync("/.dockerenv") || existsSync("/run/.containerenv");
 }
+function parseAgentContainerId(mountInfo) {
+  const match = mountInfo.match(/containers\/([0-9a-f]{64})\/(?:hostname|hosts|resolv\.conf)/);
+  return match?.[1] ?? null;
+}
+function readAgentContainerId() {
+  try {
+    return parseAgentContainerId(readFileSync2("/proc/self/mountinfo", "utf8"));
+  } catch {
+    return null;
+  }
+}
 function resolveRunnerNetworkMode(input2) {
   const override = input2.override?.trim();
   if (override) return override;
@@ -29850,10 +29887,13 @@ function parsePiLine(raw) {
         cost: message.usage.cost?.total
       } : void 0;
       const error62 = message.stopReason === "error" ? message.errorMessage ?? "Agent provider error" : void 0;
+      const text = Array.isArray(message.content) ? message.content.map(
+        (block) => block && typeof block === "object" && block.type === "text" ? String(block.text ?? "") : ""
+      ).filter(Boolean).join("\n\n") : typeof message.content === "string" ? message.content : void 0;
       return {
         type,
         error: error62,
-        assistantText: typeof message.content === "string" ? message.content : void 0,
+        assistantText: text || void 0,
         messageEnd: true,
         final: message.stopReason === "stop",
         at: timestampOf(message.timestamp) ?? timestampOf(raw.timestamp),
@@ -30470,6 +30510,7 @@ async function runRemoteCardJob(deps) {
     });
     const exit = await runAgent(command, env);
     if (deps.cancelled()) return { state: "cancelled", error: "Cancelled from the board" };
+    await status({ finalMessage: exit.finalMessage });
     if (exit.error) return { state: "failed", error: exit.error };
     if (exit.sessionId) await status({ sessionId: exit.sessionId, usage: exit.usage });
     if (context.card && (context.run.kind === "build" || context.run.kind === "consult")) {
@@ -30780,6 +30821,7 @@ async function runRemoteCardJob(deps) {
     let messageSeq = 0;
     let lastFlushAt = 0;
     let chatMessageId = null;
+    let finalMessage = "";
     let toolCalls = 0;
     let consumeQueue = Promise.resolve();
     const isPlanning = context.run.kind === "planning";
@@ -30865,10 +30907,12 @@ async function runRemoteCardJob(deps) {
           }
           if (parsed.messageEnd) {
             await flushAssistant(true);
+            finalMessage = text || messageText;
             messageText = "";
             messageSeq += 1;
             chatMessageId = null;
           } else if (!parsed.assistantDelta) {
+            finalMessage = text;
             messageSeq += 1;
             chatMessageId = null;
           }
@@ -30923,18 +30967,20 @@ async function runRemoteCardJob(deps) {
     });
     await consumeQueue;
     await flushAssistant(true);
+    if (!finalMessage && messageText) finalMessage = messageText;
     clearTimeout(timeout);
     deps.signal.removeEventListener("abort", forwardAbort);
-    if (deps.cancelled()) return { error: null, sessionId, usage: usagePatch };
-    if (error62) return { error: error62, sessionId, usage: usagePatch };
+    if (deps.cancelled()) return { error: null, sessionId, finalMessage, usage: usagePatch };
+    if (error62) return { error: error62, sessionId, finalMessage, usage: usagePatch };
     if (code !== 0 && !final && !timedOut) {
       return {
         error: stderr.trim().slice(-2e3) || `Agent exited with code ${code}`,
         sessionId,
+        finalMessage,
         usage: usagePatch
       };
     }
-    return { error: null, sessionId, usage: usagePatch };
+    return { error: null, sessionId, finalMessage, usage: usagePatch };
   }
 }
 
@@ -43699,7 +43745,7 @@ function createServerForRun(run) {
   server.registerTool(
     "verification_record",
     {
-      description: "Record mandatory verification evidence before opening a PR. Non-N/A kinds require an artifactId. N/A requires a reason.",
+      description: "Record verification evidence before opening a PR. Non-N/A kinds require an artifactId unless this project or its workspace made evidence optional. N/A requires a reason.",
       inputSchema: {
         kind: external_exports.enum(VERIFICATION_KINDS),
         summary: external_exports.string().min(1).max(400),
@@ -43776,7 +43822,7 @@ function createServerForRun(run) {
   server.registerTool(
     "pr_open",
     {
-      description: "Push the branch and open a pull request for one repo. Requires verification evidence unless the workspace relaxed the rule. Moves the card to review.",
+      description: "Push the branch and open a pull request for one repo. Requires verification evidence unless this project or its workspace made evidence optional. Moves the card to review.",
       inputSchema: {
         repoId: external_exports.string().min(1),
         branch: external_exports.string().min(1),
@@ -43902,10 +43948,11 @@ async function main() {
   const mcpPort = Number(process.env.AGLIO_WORKER_MCP_PORT ?? 4310);
   const claimTimeoutRaw = Number(process.env.AGLIO_WORKER_CLAIM_TIMEOUT_SECONDS);
   const claimTimeoutSeconds = Number.isFinite(claimTimeoutRaw) && claimTimeoutRaw > 0 ? claimTimeoutRaw : void 0;
-  const agentContainerId = process.env.HOSTNAME?.trim() || hostname3();
+  const containerized = isRunningInContainer();
+  const agentContainerId = (containerized ? readAgentContainerId() : null) || process.env.HOSTNAME?.trim() || hostname3();
   const runnerNetwork = mode === "docker" ? resolveRunnerNetworkMode({
     override: process.env.AGLIO_WORKER_RUNNER_NETWORK,
-    containerized: isRunningInContainer(),
+    containerized,
     hostname: agentContainerId
   }) : "host";
   const containerMcpUrl = runnerMcpUrl({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "aglio-worker",
-  "version": "0.14.0",
+  "version": "0.22.1",
   "description": "Host machine agent for Aglio: claims jobs over the API and starts aglio-runner containers.",
   "type": "module",
   "bin": {