palmier 0.5.0 → 0.5.1

package/dist/commands/run.js

@@ -372,7 +372,11 @@ async function requestPermission(config, task, taskDir, requiredPermissions) {
             permissions: requiredPermissions,
         }),
     });
-    const { response } = await res.json();
+    const body = await res.json();
+    const response = body.response;
+    if (!response || !["granted", "granted_all", "aborted"].includes(response)) {
+        throw new Error(`Permission request failed: ${body.error ?? `unexpected response: ${JSON.stringify(body)}`}`);
+    }
     writeTaskStatus(taskDir, {
         running_state: response === "aborted" ? "aborted" : "started",
         time_stamp: Date.now(),
@@ -386,7 +390,11 @@ async function requestConfirmation(config, task, taskDir) {
         headers: { "Content-Type": "application/json" },
         body: JSON.stringify({ taskId: task.frontmatter.id, taskName: task.frontmatter.name }),
     });
-    const { confirmed } = await res.json();
+    const body = await res.json();
+    if (typeof body.confirmed !== "boolean") {
+        throw new Error(`Confirmation request failed: ${body.error ?? `unexpected response: ${JSON.stringify(body)}`}`);
+    }
+    const { confirmed } = body;
     writeTaskStatus(taskDir, {
         running_state: confirmed ? "started" : "aborted",
         time_stamp: Date.now(),

package/dist/platform/windows.js

@@ -106,7 +106,7 @@ export class WindowsPlatform {
             this.startDaemonTask();
             process.exit(0);
         }
-        // Kill old daemon first, then spawn new one.
+        // Kill old daemon by PID
         if (oldPid) {
             try {
                 execFileSync("taskkill", ["/pid", oldPid, "/f", "/t"], { windowsHide: true, stdio: "pipe" });
@@ -115,6 +115,22 @@ export class WindowsPlatform {
                 // Process may have already exited
             }
         }
+        // Also kill any stale palmier serve processes (e.g. leftover from a previous daemon)
+        try {
+            const out = execFileSync("wmic", ["process", "where", `CommandLine like '%palmier%serve%' and ProcessId != '${process.pid}'`, "get", "ProcessId"], { encoding: "utf-8", windowsHide: true, stdio: "pipe" });
+            for (const line of out.split("\n")) {
+                const pid = line.trim();
+                if (pid && /^\d+$/.test(pid)) {
+                    try {
+                        execFileSync("taskkill", ["/pid", pid, "/f", "/t"], { windowsHide: true, stdio: "pipe" });
+                    }
+                    catch { }
+                }
+            }
+        }
+        catch {
+            // wmic may not be available on all Windows versions
+        }
         this.startDaemonTask();
     }
     /** Create or update the Task Scheduler entry for the daemon. */

package/dist/rpc-handler.js

@@ -126,10 +126,17 @@ const activeFollowups = new Map();
 export function createRpcHandler(config, nc) {
     function flattenTask(task) {
         const taskDir = getTaskDir(config.projectRoot, task.frontmatter.id);
+        const status = readTaskStatus(taskDir);
+        const pending = getPending(task.frontmatter.id);
         return {
             ...task.frontmatter,
             body: task.body,
-            status: readTaskStatus(taskDir),
+            status: status ? {
+                ...status,
+                ...(pending?.type === "confirmation" ? { pending_confirmation: true } : {}),
+                ...(pending?.type === "permission" ? { pending_permission: pending.params } : {}),
+                ...(pending?.type === "input" ? { pending_input: pending.params } : {}),
+            } : undefined,
         };
     }
     async function handleRpc(request) {

package/dist/transports/http-transport.js

@@ -242,13 +242,14 @@ export async function startHttpTransport(config, handleRpc, port, nc, pairingCod
                 }
                 const taskDir = getTaskDir(config.projectRoot, taskId);
                 const task = parseTaskFile(taskDir);
+                const pendingPromise = registerPending(taskId, "input", descriptions);
                 await publishEvent(taskId, {
                     event_type: "input-request",
                     host_id: config.hostId,
                     input_descriptions: descriptions,
                     name: task.frontmatter.name,
                 });
-                const response = await registerPending(taskId, "input", descriptions);
+                const response = await pendingPromise;
                 if (response.length === 1 && response[0] === "aborted") {
                     await publishEvent(taskId, { event_type: "input-resolved", host_id: config.hostId, status: "aborted" });
                     if (runId) {
@@ -283,11 +284,12 @@ export async function startHttpTransport(config, handleRpc, port, nc, pairingCod
                     sendJson(res, 400, { error: "taskId is required" });
                     return;
                 }
+                const pendingPromise = registerPending(taskId, "confirmation");
                 await publishEvent(taskId, {
                     event_type: "confirm-request",
                     host_id: config.hostId,
                 });
-                const response = await registerPending(taskId, "confirmation");
+                const response = await pendingPromise;
                 const confirmed = response[0] === "confirmed";
                 await publishEvent(taskId, {
                     event_type: "confirm-resolved",
@@ -314,13 +316,14 @@ export async function startHttpTransport(config, handleRpc, port, nc, pairingCod
                     sendJson(res, 400, { error: "taskId and permissions are required" });
                     return;
                 }
+                const pendingPromise = registerPending(taskId, "permission", permissions);
                 await publishEvent(taskId, {
                     event_type: "permission-request",
                     host_id: config.hostId,
                     required_permissions: permissions,
                     name: taskName,
                 });
-                const response = await registerPending(taskId, "permission", permissions);
+                const response = await pendingPromise;
                 const status = response[0];
                 await publishEvent(taskId, {
                     event_type: "permission-resolved",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "palmier",
-  "version": "0.5.0",
+  "version": "0.5.1",
   "description": "Palmier host CLI - provisions, executes tasks, and serves NATS RPC",
   "license": "Apache-2.0",
   "author": "Hongxu Cai",

package/src/commands/run.ts

@@ -454,7 +454,11 @@ async function requestPermission(
       permissions: requiredPermissions,
     }),
   });
-  const { response } = await res.json() as { response: "granted" | "granted_all" | "aborted" };
+  const body = await res.json() as { response?: string; error?: string };
+  const response = body.response as "granted" | "granted_all" | "aborted" | undefined;
+  if (!response || !["granted", "granted_all", "aborted"].includes(response)) {
+    throw new Error(`Permission request failed: ${body.error ?? `unexpected response: ${JSON.stringify(body)}`}`);
+  }
   writeTaskStatus(taskDir, {
     running_state: response === "aborted" ? "aborted" : "started",
     time_stamp: Date.now(),
@@ -474,7 +478,11 @@ async function requestConfirmation(
     headers: { "Content-Type": "application/json" },
     body: JSON.stringify({ taskId: task.frontmatter.id, taskName: task.frontmatter.name }),
   });
-  const { confirmed } = await res.json() as { confirmed: boolean };
+  const body = await res.json() as { confirmed?: boolean; error?: string };
+  if (typeof body.confirmed !== "boolean") {
+    throw new Error(`Confirmation request failed: ${body.error ?? `unexpected response: ${JSON.stringify(body)}`}`);
+  }
+  const { confirmed } = body;
   writeTaskStatus(taskDir, {
     running_state: confirmed ? "started" : "aborted",
     time_stamp: Date.now(),

package/src/platform/windows.ts

@@ -127,7 +127,7 @@ export class WindowsPlatform implements PlatformService {
       process.exit(0);
     }
 
-    // Kill old daemon first, then spawn new one.
+    // Kill old daemon by PID
     if (oldPid) {
       try {
         execFileSync("taskkill", ["/pid", oldPid, "/f", "/t"], { windowsHide: true, stdio: "pipe" });
@@ -136,6 +136,19 @@ export class WindowsPlatform implements PlatformService {
       }
     }
 
+    // Also kill any stale palmier serve processes (e.g. leftover from a previous daemon)
+    try {
+      const out = execFileSync("wmic", ["process", "where", `CommandLine like '%palmier%serve%' and ProcessId != '${process.pid}'`, "get", "ProcessId"], { encoding: "utf-8", windowsHide: true, stdio: "pipe" });
+      for (const line of out.split("\n")) {
+        const pid = line.trim();
+        if (pid && /^\d+$/.test(pid)) {
+          try { execFileSync("taskkill", ["/pid", pid, "/f", "/t"], { windowsHide: true, stdio: "pipe" }); } catch {}
+        }
+      }
+    } catch {
+      // wmic may not be available on all Windows versions
+    }
+
     this.startDaemonTask();
   }
 

package/src/rpc-handler.ts

@@ -151,10 +151,17 @@ const activeFollowups = new Map<string, ChildProcess>();
 export function createRpcHandler(config: HostConfig, nc?: NatsConnection) {
   function flattenTask(task: ParsedTask) {
     const taskDir = getTaskDir(config.projectRoot, task.frontmatter.id);
+    const status = readTaskStatus(taskDir);
+    const pending = getPending(task.frontmatter.id);
     return {
       ...task.frontmatter,
       body: task.body,
-      status: readTaskStatus(taskDir),
+      status: status ? {
+        ...status,
+        ...(pending?.type === "confirmation" ? { pending_confirmation: true } : {}),
+        ...(pending?.type === "permission" ? { pending_permission: pending.params } : {}),
+        ...(pending?.type === "input" ? { pending_input: pending.params } : {}),
+      } : undefined,
     };
   }
 

package/src/transports/http-transport.ts

@@ -262,6 +262,8 @@ export async function startHttpTransport(
         const taskDir = getTaskDir(config.projectRoot, taskId);
         const task = parseTaskFile(taskDir);
 
+        const pendingPromise = registerPending(taskId, "input", descriptions);
+
         await publishEvent(taskId, {
           event_type: "input-request",
           host_id: config.hostId,
@@ -269,7 +271,7 @@ export async function startHttpTransport(
           name: task.frontmatter.name,
         });
 
-        const response = await registerPending(taskId, "input", descriptions);
+        const response = await pendingPromise;
 
         if (response.length === 1 && response[0] === "aborted") {
           await publishEvent(taskId, { event_type: "input-resolved", host_id: config.hostId, status: "aborted" });
@@ -300,12 +302,14 @@ export async function startHttpTransport(
         const { taskId } = JSON.parse(body) as { taskId: string };
         if (!taskId) { sendJson(res, 400, { error: "taskId is required" }); return; }
 
+        const pendingPromise = registerPending(taskId, "confirmation");
+
         await publishEvent(taskId, {
           event_type: "confirm-request",
           host_id: config.hostId,
         });
 
-        const response = await registerPending(taskId, "confirmation");
+        const response = await pendingPromise;
         const confirmed = response[0] === "confirmed";
 
         await publishEvent(taskId, {
@@ -335,6 +339,8 @@ export async function startHttpTransport(
           return;
         }
 
+        const pendingPromise = registerPending(taskId, "permission", permissions);
+
         await publishEvent(taskId, {
           event_type: "permission-request",
           host_id: config.hostId,
@@ -342,7 +348,7 @@ export async function startHttpTransport(
           name: taskName,
         });
 
-        const response = await registerPending(taskId, "permission", permissions);
+        const response = await pendingPromise;
         const status = response[0] as "granted" | "granted_all" | "aborted";
 
         await publishEvent(taskId, {