palmier 0.8.1 → 0.8.3

package/src/agents/cursor.ts

@@ -19,7 +19,7 @@ export class Cursor implements AgentTool {
     if (yolo) {
       args.push("--force");
     }
-    if (followupPrompt) {args.push("--continue");} // continue mode for followups
+    if (followupPrompt) {args.push("--continue");}
     args.push("-p", prompt);
 
     return { command: "cursor", args};

package/src/agents/deepagents.ts

@@ -19,7 +19,7 @@ export class DeepAgents implements AgentTool {
     if (yolo) {
       args.push("--auto-approve");
     }
-    if (followupPrompt) {args.push("--resume");} // continue mode for followups
+    if (followupPrompt) {args.push("--resume");}
     args.push("--non-interactive", prompt);
 
     return { command: "deepagents", args};

package/src/agents/gemini.ts

@@ -25,8 +25,9 @@ export class GeminiAgent implements AgentTool {
       args.push("--allowed-tools", tools.join(","));
     }
 
-    if (followupPrompt) {args.push("--resume");} // continue mode for followups
-    args.push("--prompt", "-"); // read prompt from stdin to avoid command line length limits
+    if (followupPrompt) {args.push("--resume");}
+    // Read prompt from stdin to avoid command-line length limits.
+    args.push("--prompt", "-");
     
     return { command: "gemini", args, stdin: prompt };
   }

package/src/agents/goose.ts

@@ -16,7 +16,7 @@ export class GooseAgent implements AgentTool {
     const prompt = followupPrompt ?? getAgentInstructions(task, yolo || !this.supportsPermissions);
     const args = ["run"];
 
-    if (followupPrompt) {args.push("--resume");} // continue mode for followups
+    if (followupPrompt) {args.push("--resume");}
     args.push("--text", prompt);
 
     return { command: "goose", args, ...(yolo ? { env: { GOOSE_MODE: "auto" } } : {}) };

package/src/agents/hermes.ts

@@ -19,7 +19,7 @@ export class Hermes implements AgentTool {
     if (yolo) {
       args.push("--trust-all-tools");
     }
-    if (followupPrompt) {args.push("--continue");} // continue mode for followups
+    if (followupPrompt) {args.push("--continue");}
     args.push("-q", prompt);
 
     return { command: "hermes", args};

package/src/agents/kiro.ts

@@ -19,7 +19,7 @@ export class Kiro implements AgentTool {
     if (yolo) {
       args.push("--trust-all-tools");
     }
-    if (followupPrompt) {args.push("--resume");} // continue mode for followups
+    if (followupPrompt) {args.push("--resume");}
     args.push("--no-interactive", prompt);
 
     return { command: "kiro-cli", args};

package/src/agents/opencode.ts

@@ -19,7 +19,7 @@ export class OpenCodeAgent implements AgentTool {
     if (yolo) {
       args.push("--dangerously-skip-permissions");
     }
-    if (followupPrompt) {args.push("--continue");} // continue mode for followups
+    if (followupPrompt) {args.push("--continue");}
     args.push(prompt);
 
     return { command: "opencode", args};

package/src/agents/qoder.ts

@@ -19,7 +19,7 @@ export class Qoder implements AgentTool {
     if (yolo) {
       args.push("--yolo");
     }
-    if (followupPrompt) {args.push("-c");} // continue mode for followups
+    if (followupPrompt) {args.push("-c");}
     args.push("-p", prompt);
 
     return { command: "qodercli", args};

package/src/agents/shared-prompt.ts

@@ -12,9 +12,6 @@ const AGENT_INSTRUCTIONS_TEMPLATE = fs.readFileSync(
   "utf-8",
 );
 
-/**
- * Build the full agent prompt: instructions + endpoint docs + task description.
- */
 export function getAgentInstructions(task: ParsedTask, skipPermissions?: boolean): string {
   const port = loadConfig().httpPort ?? 7256;
   const taskDescription = task.frontmatter.user_prompt;

package/src/app-registry.ts

@@ -0,0 +1,52 @@
+import * as fs from "fs";
+import * as path from "path";
+import { CONFIG_DIR } from "./config.js";
+
+const REGISTRY_FILE = path.join(CONFIG_DIR, "app-registry.json");
+
+/** Persistent cache of packageName → appName pairs seen via device notifications. */
+export interface AppInfo {
+  packageName: string;
+  appName: string;
+}
+
+let cache: Record<string, string> | null = null;
+
+function load(): Record<string, string> {
+  if (cache) return cache;
+  try {
+    if (fs.existsSync(REGISTRY_FILE)) {
+      cache = JSON.parse(fs.readFileSync(REGISTRY_FILE, "utf-8")) as Record<string, string>;
+      return cache;
+    }
+  } catch {
+    // Corrupt file — start fresh rather than fail notifications.
+  }
+  cache = {};
+  return cache;
+}
+
+function persist(map: Record<string, string>): void {
+  fs.mkdirSync(CONFIG_DIR, { recursive: true });
+  fs.writeFileSync(REGISTRY_FILE, JSON.stringify(map, null, 2), "utf-8");
+}
+
+/** Writes only on change so we track the latest label if an app renames itself. */
+export function recordApp(packageName: string, appName: string): void {
+  if (!packageName || !appName) return;
+  const map = load();
+  if (map[packageName] === appName) return;
+  map[packageName] = appName;
+  persist(map);
+}
+
+export function listApps(): AppInfo[] {
+  const map = load();
+  return Object.entries(map)
+    .map(([packageName, appName]) => ({ packageName, appName }))
+    .sort((a, b) => a.appName.localeCompare(b.appName));
+}
+
+export function getAppName(packageName: string): string | undefined {
+  return load()[packageName];
+}

package/src/commands/info.ts

@@ -1,9 +1,6 @@
 import { loadConfig } from "../config.js";
 import { loadClients } from "../client-store.js";
 
-/**
- * Print host connection info for setting up clients.
- */
 export async function infoCommand(): Promise<void> {
   const config = loadConfig();
   const clients = loadClients();
@@ -11,14 +8,12 @@ export async function infoCommand(): Promise<void> {
   console.log(`Host ID:      ${config.hostId}`);
   console.log(`Project root: ${config.projectRoot}`);
 
-  // Detected agents
   if (config.agents && config.agents.length > 0) {
     console.log(`Agents:       ${config.agents.map((a) => a.label).join(", ")}`);
   } else {
     console.log(`Agents:       (none detected — run \`palmier agents\`)`);
   }
 
-  // Clients
   console.log(`Clients:      ${clients.length} active`);
 
   if (clients.length === 0) {

package/src/commands/init.ts

@@ -15,9 +15,6 @@ const green = (s: string) => `\x1b[32m${s}\x1b[0m`;
 const cyan = (s: string) => `\x1b[36m${s}\x1b[0m`;
 const red = (s: string) => `\x1b[31m${s}\x1b[0m`;
 
-/**
- * Interactive wizard to provision this host.
- */
 export async function initCommand(): Promise<void> {
   const rl = readline.createInterface({ input: process.stdin, output: process.stdout });
   const ask: AskFn = (q) => new Promise<string>((resolve) => rl.question(q, resolve));
@@ -27,7 +24,6 @@ export async function initCommand(): Promise<void> {
     console.log(`By continuing, you agree to the ${cyan("Terms of Service")} (https://www.palmier.me/terms)`);
     console.log(`and ${cyan("Privacy Policy")} (https://www.palmier.me/privacy).\n`);
 
-    // Detect agents first — abort if none found
     console.log("Detecting installed agents...");
     const agents = await detectAgents();
 
@@ -41,7 +37,6 @@ export async function initCommand(): Promise<void> {
 
     console.log(`  Found: ${green(agents.map((a) => a.label).join(", "))}\n`);
 
-    // LAN mode
     const lanAnswer = await ask("Enable LAN access (direct HTTP from local network)? (y/N): ");
     const lanEnabled = lanAnswer.trim().toLowerCase() === "y";
 
@@ -51,7 +46,6 @@ export async function initCommand(): Promise<void> {
     const parsed = parseInt(portAnswer.trim(), 10);
     if (parsed > 0 && parsed < 65536) httpPort = parsed;
 
-    // Display summary and ask for confirmation before making any changes
     console.log(`\n${bold("Setup summary:")}\n`);
     console.log(`  ${dim("Task storage:")}   ${bold(process.cwd())}`);
     console.log(`                  All tasks and execution data will be stored here.\n`);
@@ -64,7 +58,6 @@ export async function initCommand(): Promise<void> {
     }
     console.log(`  ${dim("Agents:")}         ${agents.map((a) => a.label).join(", ")}\n`);
 
-    // Check for existing tasks to recover
     const existingTasks = listTasks(process.cwd());
     if (existingTasks.length > 0) {
       console.log(`  ${dim("Recover tasks:")}  ${existingTasks.length} existing task(s) found:`);
@@ -81,7 +74,6 @@ export async function initCommand(): Promise<void> {
       return;
     }
 
-    // Register with server
     let existingHostId: string | undefined;
     try { existingHostId = loadConfig().hostId; } catch { /* first init */ }
 
@@ -105,7 +97,6 @@ export async function initCommand(): Promise<void> {
       }
     }
 
-    // Build and save config
     const config: HostConfig = {
       hostId: registerResponse.hostId,
       projectRoot: process.cwd(),
@@ -123,8 +114,8 @@ export async function initCommand(): Promise<void> {
 
     getPlatform().installDaemon(config);
 
-    // Task recovery happens in the daemon (palmier serve) on startup,
-    // since the daemon runs elevated and can create S4U scheduled tasks.
+    // Task recovery runs in the daemon (palmier serve) because that process
+    // is elevated and can create S4U scheduled tasks.
 
     console.log("\nStarting pairing...");
     rl.close();

package/src/commands/pair.ts

@@ -24,9 +24,6 @@ function buildPairResponse(config: HostConfig, label?: string) {
   };
 }
 
-/**
- * POST to the running serve daemon and long-poll until paired or expired.
- */
 function httpPairRegister(port: number, code: string): Promise<boolean> {
   const body = JSON.stringify({ code, expiryMs: PAIRING_EXPIRY_MS });
 
@@ -60,10 +57,7 @@ function httpPairRegister(port: number, code: string): Promise<boolean> {
   });
 }
 
-/**
- * Generate a pairing code and wait for a PWA client to pair.
- * Listens on NATS (server mode) and HTTP (via serve daemon) in parallel.
- */
+/** Listens on NATS (server mode) and HTTP (via serve daemon) in parallel. */
 export async function pairCommand(): Promise<void> {
   const config = loadConfig();
   const code = generatePairingCode();
@@ -78,7 +72,6 @@ export async function pairCommand(): Promise<void> {
 
   const cleanups: Array<() => void | Promise<void>> = [];
 
-  // Display pairing info
   console.log("");
   console.log("Enter this code in your Palmier app:");
   console.log("");
@@ -86,7 +79,6 @@ export async function pairCommand(): Promise<void> {
   console.log("");
   console.log("Code expires in 1 minute.");
 
-  // NATS pairing (server mode)
   const nc = await connectNats(config);
   const sc = StringCodec();
   const subject = `pair.${code}`;
@@ -116,13 +108,11 @@ export async function pairCommand(): Promise<void> {
     }
   })();
 
-  // HTTP pairing — register with serve daemon's /pair-register endpoint
   (async () => {
     const result = await httpPairRegister(httpPort, code);
     if (result) onPaired();
   })();
 
-  // Wait for pairing or timeout
   const start = Date.now();
   await new Promise<void>((resolve) => {
     const interval = setInterval(() => {
@@ -133,7 +123,6 @@ export async function pairCommand(): Promise<void> {
     }, 500);
   });
 
-  // Cleanup
   for (const cleanup of cleanups) {
     await cleanup();
   }

package/src/commands/restart.ts

@@ -1,8 +1,5 @@
 import { getPlatform } from "../platform/index.js";
 
-/**
- * Restart the palmier serve daemon.
- */
 export async function restartCommand(): Promise<void> {
   const platform = getPlatform();
   await platform.restartDaemon();

package/src/commands/run.ts

@@ -13,10 +13,6 @@ import { publishHostEvent } from "../events.js";
 import type { HostConfig, ParsedTask, TaskRunningState, RequiredPermission } from "../types.js";
 import type { NatsConnection } from "nats";
 
-/**
- * Shared context for agent invocation retry loops.
- * Passed around to avoid threading many individual parameters.
- */
 interface InvocationContext {
   agent: AgentTool;
   task: ParsedTask;
@@ -35,11 +31,9 @@ interface InvocationResult {
 }
 
 /**
- * Invoke the agent CLI with a continuation loop for permissions and user input.
- *
- * Both standard and command-triggered execution use this.
- * The `invokeTask` is the ParsedTask whose prompt is passed to the agent
- * (for command-triggered mode this is the per-line augmented task).
+ * Invoke the agent CLI in a continuation loop to handle permission requests.
+ * `invokeTask` is the ParsedTask whose prompt is passed to the agent (in
+ * command-triggered mode this is the per-line augmented task).
  */
 async function invokeAgentWithRetries(
   ctx: InvocationContext,
@@ -47,7 +41,6 @@ async function invokeAgentWithRetries(
 ): Promise<InvocationResult> {
   // eslint-disable-next-line no-constant-condition
   while (true) {
-    // Stream agent output to TASKRUN.md in real-time, throttled to 500ms
     const writer = beginStreamingMessage(ctx.taskDir, ctx.runId, Date.now());
     let lineBuf = "";
     let notifyPending = false;
@@ -89,12 +82,10 @@ async function invokeAgentWithRetries(
     const reportFiles = parseReportFiles(result.output);
     const requiredPermissions = parsePermissions(result.output);
 
-    // Flush remaining buffered content
     if (lineBuf && !lineBuf.startsWith("[PALMIER")) {
       writer.write(lineBuf);
     }
 
-    // Include permission requests in the assistant message
     if (requiredPermissions.length > 0) {
       const permLines = requiredPermissions.map((p) => `- **${p.name}** ${p.description}`).join("\n");
       writer.write(`\n\n**Permissions requested:**\n${permLines}\n`);
@@ -112,7 +103,6 @@ async function invokeAgentWithRetries(
       });
     }
 
-    // Permission handling — agent requested permissions
     if (requiredPermissions.length > 0) {
       const response = await requestPermission(ctx.config, ctx.task, ctx.taskDir, requiredPermissions);
 
@@ -146,27 +136,20 @@ async function invokeAgentWithRetries(
         ctx.transientPermissions = [...ctx.transientPermissions, ...newPerms];
       }
 
-      // If the agent actually failed, retry with the new permissions
+      // Retry with the new permissions if the agent failed.
       if (outcome === "failed") {
         continue;
       }
     }
 
-    // Normal completion (success or terminal failure)
     return { outcome };
   }
 }
 
-/**
- * Strip [PALMIER_*] marker lines from agent output.
- */
 export function stripPalmierMarkers(output: string): string {
   return output.split("\n").filter((l) => !l.startsWith("[PALMIER")).join("\n").trim();
 }
 
-/**
- * Append a conversation message to the RESULT file and notify connected clients.
- */
 async function appendAndNotify(
   ctx: InvocationContext,
   msg: Parameters<typeof appendRunMessage>[2],
@@ -175,9 +158,7 @@ async function appendAndNotify(
   await publishHostEvent(ctx.nc, ctx.config.hostId, ctx.taskId, { event_type: "result-updated", run_id: ctx.runId });
 }
 
-/**
- * Find the latest run dir that has no status messages yet (just created by the RPC handler).
- */
+/** The latest run dir with no status messages yet — freshly created by the RPC handler. */
 function findLatestPendingRunId(taskDir: string): string | null {
   const dirs = fs.readdirSync(taskDir)
     .filter((f) => /^\d+$/.test(f) && fs.existsSync(path.join(taskDir, f, "TASKRUN.md")))
@@ -190,8 +171,8 @@ function findLatestPendingRunId(taskDir: string): string | null {
 }
 
 /**
- * If the RPC handler already wrote "aborted" to status.json (e.g. via task.abort),
- * respect that instead of overwriting with the process's own outcome.
+ * If the RPC handler already wrote "aborted" (via task.abort), respect that
+ * instead of overwriting with the process's own outcome.
  */
 function resolveOutcome(taskDir: string, outcome: TaskRunningState): TaskRunningState {
   const current = readTaskStatus(taskDir);
@@ -199,9 +180,6 @@ function resolveOutcome(taskDir: string, outcome: TaskRunningState): TaskRunning
   return outcome;
 }
 
-/**
- * Execute a task by ID.
- */
 export async function runCommand(taskId: string): Promise<void> {
   const config = loadConfig();
   const taskDir = getTaskDir(config.projectRoot, taskId);
@@ -211,7 +189,6 @@ export async function runCommand(taskId: string): Promise<void> {
   let nc: NatsConnection | undefined;
   const taskName = task.frontmatter.name;
 
-  // Use existing run dir if just created by RPC, otherwise create a new one
   const existingRunId = findLatestPendingRunId(taskDir);
   const runId = existingRunId ?? createRunDir(taskDir, taskName, Date.now(), task.frontmatter.agent);
   if (!existingRunId) {
@@ -231,7 +208,6 @@ export async function runCommand(taskId: string): Promise<void> {
     appendRunMessage(taskDir, runId, { role: "status", time: Date.now(), content: "", type: "started" });
     await publishHostEvent(nc, config.hostId, taskId, { event_type: "result-updated", run_id: runId });
 
-    // If requires_confirmation, notify clients and wait
     if (task.frontmatter.requires_confirmation) {
       const confirmed = await requestConfirmation(config, task, taskDir);
       const confirmPrompt = `**Task Confirmation**\n\nRun task "${taskName || task.frontmatter.user_prompt}"?`;
@@ -252,7 +228,6 @@ export async function runCommand(taskId: string): Promise<void> {
       await publishHostEvent(nc, config.hostId, taskId, { event_type: "result-updated", run_id: runId });
     }
 
-    // Shared invocation context
     const guiEnv = getPlatform().getGuiEnv();
     const agent = getAgent(task.frontmatter.agent);
     const ctx: InvocationContext = {
@@ -261,7 +236,6 @@ export async function runCommand(taskId: string): Promise<void> {
     };
 
     if (task.frontmatter.command) {
-      // Command-triggered mode
       const result = await runCommandTriggeredMode(ctx);
       const outcome = resolveOutcome(taskDir, result.outcome);
       appendRunMessage(taskDir, runId, { role: "status", time: Date.now(), content: "", type: outcome });
@@ -269,14 +243,12 @@ export async function runCommand(taskId: string): Promise<void> {
       console.log(`Task ${taskId} completed (command-triggered).`);
     } else if (task.frontmatter.schedule_type === "on_new_notification"
                || task.frontmatter.schedule_type === "on_new_sms") {
-      // Event-triggered mode (driven by NATS pub/sub of device notifications/SMS)
       const result = await runEventTriggeredMode(ctx);
       const outcome = resolveOutcome(taskDir, result.outcome);
       appendRunMessage(taskDir, runId, { role: "status", time: Date.now(), content: "", type: outcome });
       await publishTaskEvent(nc, config, taskDir, taskId, outcome, taskName, runId);
       console.log(`Task ${taskId} completed (event-triggered).`);
     } else {
-      // Standard execution — add user prompt as first message
       await appendAndNotify(ctx, {
         role: "user",
         time: Date.now(),
@@ -312,11 +284,9 @@ const MAX_LOG_ENTRIES = 1000;
 const MAX_LINE_LENGTH = 200_000;
 
 /**
- * Command-triggered execution mode.
- *
- * Spawns a long-running shell command and, for each line of stdout,
- * invokes the agent CLI with the user's prompt augmented by that line.
- * Processes lines sequentially with a bounded queue.
+ * Spawn a long-running shell command and invoke the agent CLI once per stdout
+ * line, with the user's prompt augmented by that line. Sequential with a
+ * bounded queue.
  */
 async function runCommandTriggeredMode(
   ctx: InvocationContext,
@@ -346,7 +316,6 @@ async function runCommandTriggeredMode(
     const entry = `[${new Date().toISOString()}] (${outcome}) input: ${line}\n${agentOutput}\n---\n`;
     fs.appendFileSync(logPath, entry, "utf-8");
 
-    // Trim log if too large (keep last MAX_LOG_ENTRIES entries)
     try {
       const content = fs.readFileSync(logPath, "utf-8");
       const entries = content.split("\n---\n").filter(Boolean);
@@ -380,7 +349,7 @@ async function runCommandTriggeredMode(
     }
     appendLog(line, "", result.outcome);
 
-    // Append monitoring status so the UI shows the task is waiting for more input
+    // Signal "waiting for more input" in the UI.
     appendRunMessage(ctx.taskDir, ctx.runId, { role: "status", time: Date.now(), content: "", type: "monitoring" });
     await publishHostEvent(ctx.nc, ctx.config.hostId, ctx.taskId, { event_type: "result-updated", run_id: ctx.runId });
   }
@@ -422,7 +391,6 @@ async function runCommandTriggeredMode(
     process.stderr.write(d);
   });
 
-  // Wait for command to exit
   const exitCode = await new Promise<number | null>((resolve) => {
     child.on("close", (code: number | null) => {
       commandExited = true;
@@ -438,7 +406,6 @@ async function runCommandTriggeredMode(
     });
   });
 
-  // Wait for any remaining queued lines to finish processing
   if (lineQueue.length > 0 || processing) {
     await new Promise<void>((resolve) => {
       resolveWhenDone = resolve;
@@ -464,13 +431,10 @@ async function runCommandTriggeredMode(
 }
 
 /**
- * Event-triggered execution mode.
- *
- * Drains the daemon-owned per-task event queue via the local /task-event/pop
- * HTTP endpoint, invoking the agent once per event with the payload spliced
- * into the user prompt. The run process itself holds no NATS subscription;
- * the daemon handles that and atomically clears the active flag when we see
- * an empty pop, so it can fire up a fresh run on the next incoming event.
+ * Drain the daemon-owned per-task event queue via /task-event/pop, invoking
+ * the agent once per event. The run process holds no NATS subscription — the
+ * daemon owns that and atomically clears the active flag on empty pop so it
+ * can fire a fresh run on the next incoming event.
  */
 async function runEventTriggeredMode(
   ctx: InvocationContext,
@@ -590,10 +554,6 @@ async function requestConfirmation(
   return confirmed;
 }
 
-/**
- * Extract report file names from agent output.
- * Looks for lines matching: [PALMIER_REPORT] <filename>
- */
 const ALLOWED_REPORT_EXT = [".md", ".txt", ".png", ".jpg", ".jpeg", ".gif", ".svg", ".webp"];
 
 export function parseReportFiles(output: string): string[] {
@@ -602,7 +562,7 @@ export function parseReportFiles(output: string): string[] {
   let match;
   while ((match = regex.exec(output)) !== null) {
     const name = match[1].trim();
-    // Skip placeholder examples echoed from the prompt (e.g. "<filename>")
+    // Skip placeholder examples echoed from the prompt (e.g. "<filename>").
     if (!name || name.startsWith("<")) continue;
     const ext = name.lastIndexOf(".") >= 0 ? name.slice(name.lastIndexOf(".")).toLowerCase() : "";
     if (!ALLOWED_REPORT_EXT.includes(ext)) continue;
@@ -611,17 +571,13 @@ export function parseReportFiles(output: string): string[] {
   return files;
 }
 
-/**
- * Extract required permissions from agent output.
- * Looks for lines matching: [PALMIER_PERMISSION] <tool> | <description>
- */
 export function parsePermissions(output: string): RequiredPermission[] {
   const regex = new RegExp(`^\\${TASK_PERMISSION_PREFIX}\\s+(.+)$`, "gm");
   const perms: RequiredPermission[] = [];
   let match;
   while ((match = regex.exec(output)) !== null) {
     const raw = match[1].trim();
-    // Skip placeholder examples echoed from the prompt (e.g. "<tool_name> | <description>")
+    // Skip placeholder examples echoed from the prompt (e.g. "<tool_name> | <description>").
     if (raw.startsWith("<")) continue;
     const sep = raw.indexOf("|");
     if (sep !== -1) {
@@ -633,10 +589,7 @@ export function parsePermissions(output: string): RequiredPermission[] {
   return perms;
 }
 
-/**
- * Parse the agent's output for success/failure markers.
- * Falls back to "finished" if no marker is found.
- */
+/** Falls back to "finished" if no success/failure marker is found. */
 export function parseTaskOutcome(output: string): TaskRunningState {
   const lastChunk = output.slice(-500);
   const regex = new RegExp(`^\\${TASK_FAILURE_MARKER}$|^\\${TASK_SUCCESS_MARKER}$`, "gm");