palmier 0.5.1 → 0.5.2

package/src/transports/http-transport.ts

@@ -3,7 +3,8 @@ import * as os from "os";
 import { StringCodec, type NatsConnection } from "nats";
 import { validateSession, addSession } from "../session-store.js";
 import { registerPending } from "../pending-requests.js";
-import { getTaskDir, parseTaskFile, appendRunMessage } from "../task.js";
+import * as fs from "node:fs";
+import { getTaskDir, parseTaskFile, spliceUserMessage } from "../task.js";
 import type { HostConfig, RpcMessage, RequiredPermission } from "../types.js";
 
 const PWA_ORIGIN = "https://app.palmier.me";
@@ -99,6 +100,18 @@ export function detectLanIp(): string {
   return "127.0.0.1";
 }
 
+/** Find the latest (highest-numbered) run directory for a task. */
+function findLatestRunId(taskDir: string): string | null {
+  try {
+    const dirs = fs.readdirSync(taskDir)
+      .filter((f) => /^\d+$/.test(f) && fs.statSync(`${taskDir}/${f}`).isDirectory())
+      .sort();
+    return dirs.length > 0 ? dirs[dirs.length - 1] : null;
+  } catch {
+    return null;
+  }
+}
+
 /**
  * Start the HTTP transport: server with RPC, SSE, PWA proxy, pairing, and
  * localhost-only agent endpoints (notify, request-input, confirmation, permission).
@@ -262,6 +275,9 @@ export async function startHttpTransport(
         const taskDir = getTaskDir(config.projectRoot, taskId);
         const task = parseTaskFile(taskDir);
 
+        // Resolve runId: use provided value, otherwise find the latest run directory
+        const effectiveRunId = runId ?? findLatestRunId(taskDir);
+
         const pendingPromise = registerPending(taskId, "input", descriptions);
 
         await publishEvent(taskId, {
@@ -273,17 +289,20 @@ export async function startHttpTransport(
 
         const response = await pendingPromise;
 
+        const questionsBlock = "\n\n" + descriptions.map((d) => `**${d}**`).join("\n");
+
         if (response.length === 1 && response[0] === "aborted") {
           await publishEvent(taskId, { event_type: "input-resolved", host_id: config.hostId, status: "aborted" });
-          if (runId) {
-            appendRunMessage(taskDir, runId, { role: "user", time: Date.now(), content: "Input request aborted.", type: "input" });
+          if (effectiveRunId) {
+            spliceUserMessage(taskDir, effectiveRunId, { role: "user", time: Date.now(), content: "Aborted", type: "input" }, questionsBlock);
+            await publishEvent(taskId, { event_type: "result-updated", run_id: effectiveRunId });
           }
           sendJson(res, 200, { aborted: true });
         } else {
           await publishEvent(taskId, { event_type: "input-resolved", host_id: config.hostId, status: "provided" });
-          if (runId) {
-            const lines = descriptions.map((desc, i) => `**${desc}** ${response[i]}`);
-            appendRunMessage(taskDir, runId, { role: "user", time: Date.now(), content: lines.join("\n"), type: "input" });
+          if (effectiveRunId) {
+            spliceUserMessage(taskDir, effectiveRunId, { role: "user", time: Date.now(), content: response.join("\n"), type: "input" }, questionsBlock);
+            await publishEvent(taskId, { event_type: "result-updated", run_id: effectiveRunId });
           }
           sendJson(res, 200, { values: response });
         }

package/src/types.ts

@@ -23,6 +23,7 @@ export interface TaskFrontmatter {
   triggers: Trigger[];
   triggers_enabled: boolean;
   requires_confirmation: boolean;
+  yolo_mode?: boolean;
   permissions?: RequiredPermission[];
   command?: string;
 }

package/test/agent-instructions.test.ts

@@ -0,0 +1,31 @@
+import { describe, it } from "node:test";
+import assert from "node:assert/strict";
+import { getAgentInstructions } from "../src/agents/shared-prompt.js";
+
+describe("getAgentInstructions", () => {
+  it("includes Permissions section by default", () => {
+    const result = getAgentInstructions("test-task-id");
+    assert.match(result, /## Permissions/);
+    assert.match(result, /PALMIER_PERMISSION/);
+  });
+
+  it("strips Permissions section when skipPermissions is true", () => {
+    const result = getAgentInstructions("test-task-id", true);
+    assert.doesNotMatch(result, /## Permissions/);
+    assert.doesNotMatch(result, /PALMIER_PERMISSION/);
+  });
+
+  it("preserves other sections when Permissions is stripped", () => {
+    const result = getAgentInstructions("test-task-id", true);
+    assert.match(result, /## Reporting Output/);
+    assert.match(result, /## Completion/);
+    assert.match(result, /## HTTP Endpoints/);
+  });
+
+  it("replaces template variables", () => {
+    const result = getAgentInstructions("my-task-123");
+    assert.match(result, /my-task-123/);
+    assert.doesNotMatch(result, /\{\{TASK_ID\}\}/);
+    assert.doesNotMatch(result, /\{\{PORT\}\}/);
+  });
+});