palmier 0.8.3 → 0.8.4

package/palmier-server/pwa/src/components/SessionComposer.tsx

@@ -5,6 +5,7 @@ import type { AgentInfo } from "../types";
 
 interface SessionComposerProps {
   agents: AgentInfo[];
+  hostPlatform?: string;
   onStarted(taskId: string, runId?: string): void;
 }
 
@@ -15,7 +16,7 @@ function pickDefaultAgent(agents: AgentInfo[]): string {
   return agents[0]?.key ?? "";
 }
 
-export default function SessionComposer({ agents, onStarted }: SessionComposerProps) {
+export default function SessionComposer({ agents, hostPlatform, onStarted }: SessionComposerProps) {
   const { request } = useHostConnection();
   const [prompt, setPrompt] = useState("");
   const [agent, setAgent] = useState(() => pickDefaultAgent(agents));
@@ -62,7 +63,15 @@ export default function SessionComposer({ agents, onStarted }: SessionComposerPr
     try {
       const result = await request<{ task_id?: string; run_id?: string; error?: string }>(
         "task.run_oneoff",
-        { user_prompt: prompt, agent, yolo_mode: yoloMode },
+        {
+          user_prompt: prompt,
+          agent,
+          yolo_mode: yoloMode,
+          // Direct runs on Windows need a visible session so interactive tools
+          // (browsers, GUI apps) can attach; background task-scheduler runs
+          // would otherwise land in session 0 with no display.
+          ...(hostPlatform === "win32" ? { foreground_mode: true } : {}),
+        },
       );
       if (result.error) {
         setError(result.error);

package/palmier-server/pwa/src/components/SessionsView.tsx

@@ -14,13 +14,14 @@ interface SessionsViewProps {
   request<T = unknown>(method: string, params?: unknown, opts?: { timeout?: number }): Promise<T>;
   subscribeEvents(hostId: string, callback: (msg: { subject: string; data: Uint8Array }) => void): () => void;
   agents: AgentInfo[];
+  hostPlatform?: string;
   filterTaskId?: string | null;
   onClearFilter?: () => void;
 }
 
 const PAGE_SIZE = 10;
 
-export default function SessionsView({ connected, hostId, request, subscribeEvents, agents, filterTaskId, onClearFilter }: SessionsViewProps) {
+export default function SessionsView({ connected, hostId, request, subscribeEvents, agents, hostPlatform, filterTaskId, onClearFilter }: SessionsViewProps) {
   const [entries, setEntries] = useState<HistoryEntry[]>([]);
   const [total, setTotal] = useState(0);
   const [loading, setLoading] = useState(false);
@@ -175,6 +176,7 @@ export default function SessionsView({ connected, hostId, request, subscribeEven
   const composer = !filterTaskId && (
     <SessionComposer
       agents={agents}
+      hostPlatform={hostPlatform}
       onStarted={(taskId, runId) => {
         if (runId) navigate(`/runs/${encodeURIComponent(taskId)}/${encodeURIComponent(runId)}`);
         else navigate(`/runs/${encodeURIComponent(taskId)}`);

package/palmier-server/pwa/src/components/TaskCard.tsx

@@ -133,7 +133,7 @@ export default function TaskCard({ task, lastEvent, onEdit, onDelete, onViewRun
     values: string[] | undefined,
   ): string {
     if (!scheduleType) return "";
-    if (scheduleType === "on_new_notification") return "On new push notification";
+    if (scheduleType === "on_new_notification") return "On new notification";
     if (scheduleType === "on_new_sms") return "On new SMS";
     if (!values || values.length === 0) return "";
     if (values.length === 1) return formatSingleValue(scheduleType, values[0]);

package/palmier-server/pwa/src/components/TaskForm.tsx

@@ -96,12 +96,13 @@ interface TaskFormProps {
   initial?: Task;
   agents: AgentInfo[];
   hostPlatform?: string;
+  isNotificationListener: boolean;
   onSaved(task: Task): void;
   onRun(taskId: string, runId?: string): void;
   onCancel(): void;
 }
 
-export default function TaskForm({ initial, agents, hostPlatform, onSaved, onRun, onCancel }: TaskFormProps) {
+export default function TaskForm({ initial, agents, hostPlatform, isNotificationListener, onSaved, onRun, onCancel }: TaskFormProps) {
   const { request } = useHostConnection();
 
   const defaultAgent = () => {
@@ -154,46 +155,35 @@ export default function TaskForm({ initial, agents, hostPlatform, onSaved, onRun
     return "";
   })();
   const [notificationApp, setNotificationApp] = useState<string>(initialNotificationApp);
-  const [knownApps, setKnownApps] = useState<Array<{ packageName: string; appName: string; icon?: string }>>([]);
-  const [appDropdownOpen, setAppDropdownOpen] = useState(false);
-
-  // Merge native launcher enum (complete, native-only) with host registry
-  // (apps already seen — fills the gap on non-native clients). Dedup by
-  // packageName; free-form typing still works if neither knows the package.
+  const [knownApps, setKnownApps] = useState<Array<{ packageName: string; appName: string }>>([]);
+  const [knownAppsLoading, setKnownAppsLoading] = useState(false);
+  const [appFilterOpen, setAppFilterOpen] = useState(false);
+  const [appSearch, setAppSearch] = useState("");
+  const closeAppFilter = useCallback(() => setAppFilterOpen(false), []);
+  useBackClose(appFilterOpen, closeAppFilter);
+
+  // Only the notification-listening device can enumerate installed apps. On any
+  // other client we leave the list empty and fall back to a plain packageName
+  // input — the app registry we used to maintain on the host was inconsistent
+  // across devices, so we no longer cache it.
   useEffect(() => {
     if (scheduleMode !== "on_new_notification") return;
+    if (!isNotificationListener || !Capacitor.isNativePlatform() || !Device) return;
     let cancelled = false;
-
-    const merged = new Map<string, { packageName: string; appName: string; icon?: string }>();
-    const PALMIER_PACKAGE = "com.palmier.app";
-    const flush = () => { if (!cancelled) setKnownApps(Array.from(merged.values()).sort((a, b) => (a.appName || a.packageName).localeCompare(b.appName || b.packageName))); };
-
-    if (Capacitor.isNativePlatform() && Device) {
-      Device.getInstalledApps()
-        .then(({ apps }) => {
-          if (cancelled) return;
-          for (const a of apps) {
-            if (a.packageName === PALMIER_PACKAGE) continue;
-            merged.set(a.packageName, { packageName: a.packageName, appName: a.appName, icon: a.icon });
-          }
-          flush();
-        })
-        .catch(() => {});
-    }
-
-    request<{ apps?: Array<{ packageName: string; appName: string }> }>("device.notifications.apps")
-      .then((res) => {
-        if (cancelled || !res.apps) return;
-        for (const a of res.apps) {
-          if (a.packageName === PALMIER_PACKAGE) continue;
-          if (!merged.has(a.packageName)) merged.set(a.packageName, a);
-        }
-        flush();
+    setKnownAppsLoading(true);
+    Device.getInstalledApps()
+      .then(({ apps }) => {
+        if (cancelled) return;
+        const PALMIER_PACKAGE = "com.palmier.app";
+        const list = apps
+          .filter((a) => a.packageName !== PALMIER_PACKAGE)
+          .sort((a, b) => (a.appName || a.packageName).localeCompare(b.appName || b.packageName));
+        setKnownApps(list);
       })
-      .catch(() => {});
-
+      .catch(() => {})
+      .finally(() => { if (!cancelled) setKnownAppsLoading(false); });
     return () => { cancelled = true; };
-  }, [scheduleMode]);
+  }, [scheduleMode, isNotificationListener]);
 
   // Sender filter for on_new_sms tasks. Empty string = any sender; non-empty =
   // whitelist a single sender (stored as a single-entry schedule_values array
@@ -444,7 +434,7 @@ export default function TaskForm({ initial, agents, hostPlatform, onSaved, onRun
             <option value="daily">Daily</option>
             <option value="weekly">Weekly</option>
             <option value="monthly">Monthly</option>
-            <option value="on_new_notification">On New Push Notification</option>
+            <option value="on_new_notification">On New Notification</option>
             <option value="on_new_sms">On New SMS</option>
             <option value="command">Command-triggered</option>
           </select>
@@ -453,57 +443,53 @@ export default function TaskForm({ initial, agents, hostPlatform, onSaved, onRun
             <div className="schedule-reactive">
               <p className="command-help-text">
                 {scheduleMode === "on_new_notification"
-                  ? "Runs each time a new notification arrives on the paired Android device."
+                  ? "Runs each time a new push notification arrives on the paired Android device."
                   : "Runs each time a new SMS arrives on the paired Android device."}
                 {" "}The triggering payload is spliced into your task prompt — reference it as &ldquo;the new {scheduleMode === "on_new_notification" ? "notification" : "SMS"}&rdquo;.
               </p>
               {scheduleMode === "on_new_notification" && (() => {
-                const q = notificationApp.trim().toLowerCase();
-                const filtered = q
-                  ? knownApps.filter((a) => a.packageName.toLowerCase().includes(q) || a.appName.toLowerCase().includes(q))
-                  : knownApps;
+                const selected = knownApps.find((a) => a.packageName === notificationApp);
+                const selectedLabel = selected?.appName || notificationApp;
                 return (
                   <>
-                    <div className="app-combobox">
+                    {isNotificationListener ? (
+                      notificationApp.trim() ? (
+                        <div className="app-filter-selected">
+                          <span className="app-filter-selected-name">{selectedLabel}</span>
+                          {selected?.appName && <span className="app-filter-selected-pkg">{selected.packageName}</span>}
+                          <button
+                            type="button"
+                            className="app-filter-selected-clear"
+                            onClick={() => setNotificationApp("")}
+                            aria-label="Clear app filter"
+                            disabled={saving}
+                          >
+                            ✕
+                          </button>
+                        </div>
+                      ) : (
+                        <button
+                          type="button"
+                          className="btn btn-link app-filter-trigger"
+                          onClick={() => { setAppSearch(""); setAppFilterOpen(true); }}
+                          disabled={saving}
+                        >
+                          Select app
+                        </button>
+                      )
+                    ) : (
                       <input
                         className="form-input"
                         type="text"
                         value={notificationApp}
-                        onChange={(e) => { setNotificationApp(e.target.value); setAppDropdownOpen(true); }}
-                        onFocus={() => setAppDropdownOpen(true)}
-                        onBlur={() => setAppDropdownOpen(false)}
-                        placeholder={Capacitor.isNativePlatform() ? "App (optional)" : "App (optional, e.g. com.google.android.gm)"}
+                        onChange={(e) => setNotificationApp(e.target.value)}
+                        placeholder="App (optional), e.g. com.google.android.gm"
                         disabled={saving}
                       />
-                      {appDropdownOpen && filtered.length > 0 && (
-                        <ul className="app-combobox-list">
-                          {filtered.map((a) => (
-                            <li
-                              key={a.packageName}
-                              className="app-combobox-row"
-                              // onMouseDown instead of onClick so the input's blur doesn't close
-                              // the dropdown before the selection registers.
-                              onMouseDown={(e) => {
-                                e.preventDefault();
-                                setNotificationApp(a.packageName);
-                                setAppDropdownOpen(false);
-                              }}
-                            >
-                              {a.icon
-                                ? <img src={a.icon} alt="" className="app-combobox-icon" />
-                                : <div className="app-combobox-icon app-combobox-icon-placeholder" />}
-                              <div className="app-combobox-labels">
-                                <div className="app-combobox-name">{a.appName || a.packageName}</div>
-                                {a.appName && <div className="app-combobox-pkg">{a.packageName}</div>}
-                              </div>
-                            </li>
-                          ))}
-                        </ul>
-                      )}
-                    </div>
+                    )}
                     <p className="command-help-text app-filter-help">
                       {notificationApp.trim()
-                        ? `Only notifications from ${notificationApp.trim()} will trigger this task.`
+                        ? `Only notifications from ${selectedLabel} will trigger this task.`
                         : "Every notification from your device triggers this task."}
                     </p>
                   </>
@@ -699,6 +685,72 @@ export default function TaskForm({ initial, agents, hostPlatform, onSaved, onRun
 
         </>)}
       </div>
+      {appFilterOpen && (() => {
+        const q = appSearch.trim().toLowerCase();
+        const filtered = q
+          ? knownApps.filter((a) => a.packageName.toLowerCase().includes(q) || a.appName.toLowerCase().includes(q))
+          : knownApps;
+        return (
+          <div className="app-filter-overlay" onClick={closeAppFilter}>
+            <div className="app-filter-dialog" onClick={(e) => e.stopPropagation()}>
+              <div className="app-filter-header">
+                <h2>Select app</h2>
+                <button className="app-filter-close" onClick={closeAppFilter} aria-label="Close">
+                  <svg width="16" height="16" viewBox="0 0 16 16" fill="none">
+                    <path d="M4 4L12 12M12 4L4 12" stroke="currentColor" strokeWidth="1.5" strokeLinecap="round" />
+                  </svg>
+                </button>
+              </div>
+              <input
+                className="form-input app-filter-search"
+                type="text"
+                value={appSearch}
+                onChange={(e) => setAppSearch(e.target.value)}
+                onKeyDown={(e) => {
+                  if (e.key === "Enter" && appSearch.trim()) {
+                    setNotificationApp(appSearch.trim());
+                    closeAppFilter();
+                  }
+                }}
+                placeholder="Search or type a package name"
+                autoFocus
+              />
+              <ul className="app-filter-list">
+                {knownAppsLoading && knownApps.length === 0
+                  ? Array.from({ length: 6 }).map((_, i) => (
+                      <li key={`sk-${i}`} className="app-filter-row app-filter-skeleton">
+                        <div className="app-filter-skeleton-bar" />
+                      </li>
+                    ))
+                  : filtered.length === 0 && !appSearch.trim()
+                    ? <li className="app-filter-empty">No apps</li>
+                    : filtered.map((a) => (
+                        <li
+                          key={a.packageName}
+                          className="app-filter-row"
+                          onClick={() => { setNotificationApp(a.packageName); closeAppFilter(); }}
+                        >
+                          <div className="app-filter-row-labels">
+                            <div className="app-filter-row-name">{a.appName || a.packageName}</div>
+                            {a.appName && <div className="app-filter-row-pkg">{a.packageName}</div>}
+                          </div>
+                        </li>
+                      ))}
+                {appSearch.trim() && (
+                  <li
+                    className="app-filter-row"
+                    onClick={() => { setNotificationApp(appSearch.trim()); closeAppFilter(); }}
+                  >
+                    <div className="app-filter-row-labels">
+                      <div className="app-filter-row-name">{appSearch.trim()}</div>
+                    </div>
+                  </li>
+                )}
+              </ul>
+            </div>
+          </div>
+        );
+      })()}
     </div>
   );
 }

package/palmier-server/pwa/src/components/TasksView.tsx

@@ -12,10 +12,11 @@ interface TasksViewProps {
   subscribeEvents(hostId: string, callback: (msg: { subject: string; data: Uint8Array }) => void): () => void;
   agents: AgentInfo[];
   hostPlatform?: string;
+  isNotificationListener: boolean;
   onViewRun(taskId: string, runId?: string): void;
 }
 
-export default function TasksView({ connected, hostId, request, subscribeEvents, agents, hostPlatform, onViewRun }: TasksViewProps) {
+export default function TasksView({ connected, hostId, request, subscribeEvents, agents, hostPlatform, isNotificationListener, onViewRun }: TasksViewProps) {
   const [tasks, setTasks] = useState<Task[]>([]);
   const [loadingTasks, setLoadingTasks] = useState(false);
   const [taskError, setTaskError] = useState<string | null>(null);
@@ -167,6 +168,7 @@ export default function TasksView({ connected, hostId, request, subscribeEvents,
           initial={editingTask}
           agents={agents}
           hostPlatform={hostPlatform}
+          isNotificationListener={isNotificationListener}
           onSaved={handleTaskSaved}
           onRun={onViewRun}
           onCancel={closeForm}

package/palmier-server/pwa/src/native/Device.ts

@@ -28,8 +28,6 @@ export interface DeepLinkEvent {
 export interface InstalledApp {
   packageName: string;
   appName: string;
-  /** data:image/png;base64 URL, 96x96. May be absent if the icon couldn't be rendered. */
-  icon?: string;
 }
 
 interface DevicePlugin {

package/palmier-server/pwa/src/pages/Dashboard.tsx

@@ -348,6 +348,7 @@ export default function Dashboard() {
                 subscribeEvents={subscribeEvents}
                 agents={agents}
                 hostPlatform={hostPlatform}
+                isNotificationListener={!!activeClientToken && capabilityTokens["notifications"] === activeClientToken}
                 onViewRun={handleViewRun}
               />
             )}
@@ -367,6 +368,7 @@ export default function Dashboard() {
                 request={request}
                 subscribeEvents={subscribeEvents}
                 agents={agents}
+                hostPlatform={hostPlatform}
                 filterTaskId={runsFilterTaskId}
                 onClearFilter={() => { if (confirmLeaveDraft()) navigate("/"); }}
               />

package/palmier-server/pwa/src/pages/PairHost.tsx

@@ -12,6 +12,7 @@ interface PairResponse {
   hostId: string;
   clientToken: string;
   directUrl?: string;
+  hostName?: string;
 }
 
 /** LAN mode: PWA is served by palmier serve (marker injected into HTML). */
@@ -82,6 +83,7 @@ export default function PairHost() {
         hostId: response.hostId,
         clientToken: response.clientToken,
         directUrl: isLanMode ? window.location.origin : undefined,
+        ...(response.hostName ? { name: response.hostName } : {}),
       };
 
       addPairedHost(host);
@@ -106,7 +108,7 @@ export default function PairHost() {
         }
       }
 
-      navigate("/");
+      navigate(Capacitor.isNativePlatform() ? "/pair/setup" : "/");
     } catch (err) {
       const message = err instanceof Error ? err.message : String(err);
       if (message.includes("timeout") || message.includes("TIMEOUT") || message.includes("503") || message.toLowerCase().includes("no responders")) {

package/palmier-server/pwa/src/pages/PairSetup.tsx

@@ -0,0 +1,70 @@
+import { useEffect, useState } from "react";
+import { useNavigate } from "react-router-dom";
+import { useHostConnection } from "../contexts/HostConnectionContext";
+import { useHostStore } from "../contexts/HostStoreContext";
+import CapabilityToggles from "../components/CapabilityToggles";
+
+interface HostInfoResponse {
+  capability_tokens?: Record<string, string | null>;
+}
+
+export default function PairSetup() {
+  const navigate = useNavigate();
+  const { connected, request } = useHostConnection();
+  const { getActiveHost } = useHostStore();
+  const activeHost = getActiveHost();
+  const activeClientToken = activeHost?.clientToken ?? null;
+
+  const [capabilityTokens, setCapabilityTokens] = useState<Record<string, string | null>>({});
+  const [loaded, setLoaded] = useState(false);
+
+  // If the user lands here without an active host (direct URL, refresh), bounce
+  // back to the dashboard — setup only makes sense right after pairing.
+  useEffect(() => {
+    if (!activeHost) navigate("/", { replace: true });
+  }, [activeHost, navigate]);
+
+  useEffect(() => {
+    if (!connected || !activeHost) return;
+    let cancelled = false;
+    request<HostInfoResponse>("host.info")
+      .then((res) => {
+        if (cancelled) return;
+        setCapabilityTokens(res.capability_tokens ?? {});
+        setLoaded(true);
+      })
+      .catch(() => { if (!cancelled) setLoaded(true); });
+    return () => { cancelled = true; };
+  }, [connected, activeHost, request]);
+
+  return (
+    <div className="pair-setup">
+      <div className="pair-setup-inner">
+        <h1 className="pair-setup-title">What capabilities of this device do you want your host computer to have?</h1>
+        <p className="pair-setup-description">
+          You can change these later from the menu.
+        </p>
+
+        {!loaded ? (
+          <div className="pair-setup-loading">Connecting to host…</div>
+        ) : (
+          <CapabilityToggles
+            capabilityTokens={capabilityTokens}
+            activeClientToken={activeClientToken}
+            request={request}
+            onCapabilityTokensChange={setCapabilityTokens}
+          />
+        )}
+
+        <div className="pair-setup-actions">
+          <button
+            className="btn btn-primary btn-full"
+            onClick={() => navigate("/", { replace: true })}
+          >
+            Finish
+          </button>
+        </div>
+      </div>
+    </div>
+  );
+}

package/src/commands/pair.ts

@@ -1,4 +1,5 @@
 import * as http from "node:http";
+import * as os from "node:os";
 import { StringCodec } from "nats";
 import { loadConfig } from "../config.js";
 import { connectNats } from "../nats-client.js";
@@ -21,6 +22,7 @@ function buildPairResponse(config: HostConfig, label?: string) {
   return {
     hostId: config.hostId,
     clientToken: client.token,
+    hostName: os.hostname(),
   };
 }
 

package/src/commands/serve.ts

@@ -16,7 +16,6 @@ import { StringCodec, type NatsConnection } from "nats";
 import { addNotification } from "../notification-store.js";
 import { addSmsMessage } from "../sms-store.js";
 import { enqueueEvent } from "../event-queues.js";
-import { recordApp } from "../app-registry.js";
 
 const POLL_INTERVAL_MS = 30_000;
 const DAEMON_PID_FILE = path.join(CONFIG_DIR, "daemon.pid");
@@ -159,9 +158,7 @@ export async function serveCommand(): Promise<void> {
         let parsed: unknown;
         try {
           parsed = JSON.parse(raw);
-          const data = parsed as { packageName?: string; appName?: string };
           addNotification({ ...(parsed as object), receivedAt: Date.now() } as Parameters<typeof addNotification>[0]);
-          if (data.packageName && data.appName) recordApp(data.packageName, data.appName);
         } catch (err) {
           console.error("[nats] Failed to parse device notification:", err);
         }

package/src/platform/index.ts

@@ -1,6 +1,7 @@
 import type { PlatformService } from "./platform.js";
 import { LinuxPlatform } from "./linux.js";
 import { WindowsPlatform } from "./windows.js";
+import { MacOsPlatform } from "./macos.js";
 
 /** Windows needs an explicit shell for execSync to resolve .cmd shims. */
 export const SHELL: string | undefined = process.platform === "win32" ? "cmd.exe" : undefined;
@@ -9,9 +10,9 @@ let _instance: PlatformService | undefined;
 
 export function getPlatform(): PlatformService {
   if (!_instance) {
-    _instance = process.platform === "win32"
-      ? new WindowsPlatform()
-      : new LinuxPlatform();
+    if (process.platform === "win32") _instance = new WindowsPlatform();
+    else if (process.platform === "darwin") _instance = new MacOsPlatform();
+    else _instance = new LinuxPlatform();
   }
   return _instance;
 }