palmier 0.9.8 → 0.9.10

package/dist/agents/agent-instructions.md

@@ -1,19 +1,21 @@
 You are an AI agent executing a task on behalf of the user. Follow these instructions carefully.
 
+All `[PALMIER_*]` markers below are control signals parsed by the host. They MUST be written to **stdout** (not stderr). Markers on stderr are ignored.
+
 ## Reporting Output
 
-If you generate report or output files, print each file path on its own line using this exact format:
+If you generate report or output files, print each file path on its own line to stdout using this exact format:
 [PALMIER_REPORT] <filename>
 
 ## Completion
 
-When you are done, output exactly one of these markers as the very last line (no other text on the same line):
+When you are done, output exactly one of these markers as the very last line on stdout (no other text on the same line):
 [PALMIER_TASK_SUCCESS]
 [PALMIER_TASK_FAILURE]
 
 ## Permissions
 
-Whenever a tool you are trying to use is denied or you lack the required permissions, print each required permission on its own line using this exact format:
+Whenever a tool you are trying to use is denied or you lack the required permissions, print each required permission on its own line to stdout using this exact format:
 [PALMIER_PERMISSION] <tool_name> | <description>
 
 ## HTTP Endpoints

package/dist/commands/run.js

@@ -30,8 +30,9 @@ async function sendPushNotification(nc, hostId, title, body) {
 async function invokeAgentWithRetries(ctx, invokeTask) {
     // eslint-disable-next-line no-constant-condition
     while (true) {
-        const writer = beginStreamingMessage(ctx.taskDir, ctx.runId, Date.now());
-        let lineBuf = "";
+        let writer = beginStreamingMessage(ctx.taskDir, ctx.runId, Date.now(), "stdout");
+        let activeStream = "stdout";
+        const lineBufs = { stdout: "", stderr: "" };
         let notifyPending = false;
         let notifyTimer;
         function throttledNotify() {
@@ -43,6 +44,21 @@ async function invokeAgentWithRetries(ctx, invokeTask) {
                 publishHostEvent(ctx.nc, ctx.config.hostId, ctx.taskId, { event_type: "result-updated", run_id: ctx.runId });
             }, 500);
         }
+        function emit(stream, chunk) {
+            lineBufs[stream] += chunk;
+            const lines = lineBufs[stream].split("\n");
+            lineBufs[stream] = lines.pop() ?? "";
+            const filtered = lines.filter((l) => !l.startsWith("[PALMIER"));
+            if (filtered.length === 0)
+                return;
+            if (stream !== activeStream) {
+                writer.end();
+                writer = beginStreamingMessage(ctx.taskDir, ctx.runId, Date.now(), stream);
+                activeStream = stream;
+            }
+            writer.write(filtered.join("\n") + "\n");
+            throttledNotify();
+        }
         const { command, args, stdin, env: agentEnv } = ctx.agent.getTaskRunCommandLine(invokeTask, undefined, ctx.task.frontmatter.yolo_mode ? "yolo" : ctx.transientPermissions);
         const result = await spawnCommand(command, args, {
             cwd: getRunDir(ctx.taskDir, ctx.runId),
@@ -50,29 +66,39 @@ async function invokeAgentWithRetries(ctx, invokeTask) {
             echoStdout: true,
             resolveOnFailure: true,
             stdin,
-            onData: (chunk) => {
-                lineBuf += chunk;
-                const lines = lineBuf.split("\n");
-                lineBuf = lines.pop() ?? "";
-                const filtered = lines.filter((l) => !l.startsWith("[PALMIER"));
-                if (filtered.length > 0) {
-                    writer.write(filtered.join("\n") + "\n");
-                    throttledNotify();
-                }
-            },
+            onStdout: (chunk) => emit("stdout", chunk),
+            onStderr: (chunk) => emit("stderr", chunk),
         });
         if (notifyTimer)
             clearTimeout(notifyTimer);
         const outcome = result.exitCode !== 0 ? "failed" : parseTaskOutcome(result.output);
         const reportFiles = parseReportFiles(result.output);
         const requiredPermissions = parsePermissions(result.output);
-        if (lineBuf && !lineBuf.startsWith("[PALMIER")) {
-            writer.write(lineBuf);
+        for (const stream of ["stdout", "stderr"]) {
+            const trailing = lineBufs[stream];
+            if (trailing && !trailing.startsWith("[PALMIER")) {
+                if (stream !== activeStream) {
+                    writer.end();
+                    writer = beginStreamingMessage(ctx.taskDir, ctx.runId, Date.now(), stream);
+                    activeStream = stream;
+                }
+                writer.write(trailing);
+            }
         }
         if (requiredPermissions.length > 0) {
+            if (activeStream !== "stdout") {
+                writer.end();
+                writer = beginStreamingMessage(ctx.taskDir, ctx.runId, Date.now(), "stdout");
+                activeStream = "stdout";
+            }
             const permLines = requiredPermissions.map((p) => `- **${p.name}** ${p.description}`).join("\n");
             writer.write(`\n\n**Permissions requested:**\n${permLines}\n`);
         }
+        if (reportFiles.length > 0 && activeStream !== "stdout") {
+            writer.end();
+            writer = beginStreamingMessage(ctx.taskDir, ctx.runId, Date.now(), "stdout");
+            activeStream = "stdout";
+        }
         writer.end(reportFiles.length > 0 ? reportFiles : undefined);
         await publishHostEvent(ctx.nc, ctx.config.hostId, ctx.taskId, { event_type: "result-updated", run_id: ctx.runId });
         if (reportFiles.length > 0) {

package/dist/platform/windows.js

@@ -2,7 +2,7 @@ import * as fs from "fs";
 import * as path from "path";
 import { execFileSync } from "child_process";
 import { CONFIG_DIR, loadConfig } from "../config.js";
-import { getTaskDir, readTaskStatus } from "../task.js";
+import { getTaskDir, readTaskStatus, parseTaskFile } from "../task.js";
 const TASK_PREFIX = "\\Palmier\\PalmierTask-";
 const DAEMON_TASK_NAME = "PalmierDaemon";
 const DOW_NAMES = ["Sunday", "Monday", "Tuesday", "Wednesday", "Thursday", "Friday", "Saturday", "Sunday"];
@@ -72,6 +72,52 @@ export function buildTaskXml(tr, triggers, foreground) {
 function schtasksTaskName(taskId) {
     return `${TASK_PREFIX}${taskId}`;
 }
+function querySchtasksStatus(tn) {
+    try {
+        const out = execFileSync("schtasks", ["/query", "/tn", tn, "/v", "/fo", "LIST"], {
+            encoding: "utf-8", windowsHide: true, stdio: ["ignore", "pipe", "pipe"],
+        });
+        const status = out.match(/^\s*Status:\s*(.+?)\s*$/im)?.[1] ?? "";
+        const lastResult = out.match(/^\s*Last Result:\s*(.+?)\s*$/im)?.[1] ?? "";
+        return { status, lastResult };
+    }
+    catch {
+        return undefined;
+    }
+}
+/** Map common Last Result HRESULTs to a human-readable cause. */
+function explainLastResult(lastResult, foreground) {
+    const code = lastResult.trim();
+    // Decimal forms emitted by schtasks: 267011 = 0x41303 SCHED_S_TASK_HAS_NOT_RUN.
+    if (code === "267011" || /0x0*41303/i.test(code)) {
+        return foreground
+            ? "Foreground mode requires an active Windows session, but no user is logged in. Sign in to Windows and try again, or disable foreground mode for this task."
+            : "Task Scheduler reported the task did not run. Check that the daemon has permission to launch it.";
+    }
+    if (code === "0" || code === "0x0")
+        return undefined;
+    return `Task Scheduler reported Last Result=${code}.`;
+}
+/**
+ * 2s after `schtasks /run`, confirm the action actually launched. Some failure
+ * modes — most notably foreground tasks with no interactive session — make
+ * /run return success while the action is silently skipped (Status stays
+ * "Ready", Last Result stays at 0x41303). If the run process has already
+ * written status.json by then, it clearly launched; skip the Scheduler query.
+ */
+async function verifyTaskLaunched(tn, taskDir, startTime, foreground) {
+    await new Promise((r) => setTimeout(r, 2000));
+    const status = readTaskStatus(taskDir);
+    if (status && status.time_stamp >= startTime)
+        return;
+    const last = querySchtasksStatus(tn);
+    if (last && /running/i.test(last.status))
+        return;
+    const explained = explainLastResult(last?.lastResult ?? "", foreground);
+    if (explained)
+        throw new Error(explained);
+    throw new Error(`Task Scheduler did not launch the task within 2s (status=${last?.status || "unknown"}, last_result=${last?.lastResult || "unknown"}).`);
+}
 export class WindowsPlatform {
     installDaemon(config) {
         const script = process.argv[1] || "palmier";
@@ -218,6 +264,13 @@ export class WindowsPlatform {
     }
     async startTask(taskId) {
         const tn = schtasksTaskName(taskId);
+        const taskDir = getTaskDir(loadConfig().projectRoot, taskId);
+        let foreground = false;
+        try {
+            foreground = !!parseTaskFile(taskDir).frontmatter.foreground_mode;
+        }
+        catch { /* fall through; verifyTaskLaunched still detects most failures */ }
+        const startTime = Date.now();
         try {
             execFileSync("schtasks", ["/run", "/tn", tn], { encoding: "utf-8", windowsHide: true });
         }
@@ -225,6 +278,7 @@ export class WindowsPlatform {
             const e = err;
             throw new Error(`Failed to start task via schtasks: ${e.stderr || e.message}`);
         }
+        await verifyTaskLaunched(tn, taskDir, startTime, foreground);
     }
     async stopTask(taskId) {
         // schtasks /end leaves agent children orphaned, so kill the process tree