wisp-agents 0.11.0 → 0.13.0

package/server/runner.js

@@ -13,6 +13,7 @@ import { captureCheckpoint, freshCheckpoint, readGitFile } from './checkpoint.js
 import { teamNote, copies, SUBAGENT_ASK } from './subagents.js';
 
 const MAX_ACTIVITY = 400;
+const MODES = new Set(['plan', 'edits', 'auto']);
 const LINGER_MS = 3000; // how long a CLI may keep running after it reported its result
 const ASK_SERVER = path.join(path.dirname(fileURLToPath(import.meta.url)), 'mcp-ask.js');
 // The delegate tool lives on the same injected server, so harnesses name it like ask_user.
@@ -70,7 +71,7 @@ export function summarize(s, store) {
     toShip: s.toShip || null, // { dirty, ahead, unpushed } of the worktree, null until checked
     pr: s.pr ? { number: s.pr.number, url: s.pr.url, state: s.pr.state, draft: s.pr.draft, review: s.pr.review, checks: s.pr.checks } : null,
     groupId: s.groupId || null,
-    origin: s.origin || null, // { kind: 'recipe' | 'trigger', id, name }
+    origin: s.origin || null, // { kind: 'recipe' | 'trigger' | 'telegram', id, name }
     // The roster, for the stacked logos on the card: used = delegated to at least once, busy = working now.
     team: (s.subagents || []).flatMap((m) =>
       copies(m).map((name) => {
@@ -138,8 +139,11 @@ export class Runner extends EventEmitter {
     return session;
   }
 
-  reply(session, text, attachments = [], { queue = false } = {}) {
-    if (session.ask) {
+  // mode: switch the permission mode (plan / edits / auto) from this follow-up on. A queued
+  // follow-up keeps its own, so the run before it isn't affected.
+  reply(session, text, attachments = [], { queue = false, mode } = {}) {
+    if (!MODES.has(mode) || mode === session.mode) mode = undefined;
+    if (session.ask && !mode) {
       this.answer(session, text);
       return { answered: true };
     }
@@ -149,22 +153,23 @@ export class Runner extends EventEmitter {
       // The run already produced its result and the CLI is only lingering on its way out:
       // there's nothing to wait for, so let it go and start the follow-up now.
       if (turn?.status !== 'running') {
-        session.queue.push({ id: newId(), text, attachments });
+        session.queue.push({ id: newId(), text, attachments, ...(mode && { mode }) });
         handle.release();
         return { queued: session.queue.length > 0 };
       }
       // Interrupting needs an agent session to resume; before the agent reports one, queue.
       if (queue || !session.agentSessionId) {
-        session.queue.push({ id: newId(), text, attachments });
+        session.queue.push({ id: newId(), text, attachments, ...(mode && { mode }) });
         this.#touch(session);
         return { queued: true, ahead: session.queue.length - 1 };
       }
       // Ahead of anything queued, but after earlier interrupting follow-ups, so they keep their order.
       const at = session.queue.findIndex((q) => !q.now);
-      session.queue.splice(at < 0 ? session.queue.length : at, 0, { id: newId(), text, attachments, now: true });
+      session.queue.splice(at < 0 ? session.queue.length : at, 0, { id: newId(), text, attachments, now: true, ...(mode && { mode }) });
       this.#interrupt(session);
       return { interrupted: true };
     }
+    if (mode) session.mode = mode;
     this.#startTurn(session, text, attachments);
     return { queued: false };
   }
@@ -252,6 +257,7 @@ export class Runner extends EventEmitter {
       let child = this.store.get(lead.children?.[slot]);
       if (child) {
         child.cwd = workdir(lead); // follows the lead if its worktree moved
+        child.mode = lead.mode; // and its mode, if a follow-up switched it
         this.#startTurn(child, prompt);
       } else {
         child = this.create({ prompt, cwd: workdir(lead), agent: m.agent, mode: lead.mode, model: m.model, effort: m.effort, parentId: lead.id, member: slot });
@@ -317,6 +323,7 @@ export class Runner extends EventEmitter {
       agent: session.agent,
       model: session.model,
       effort: session.effort || '',
+      mode: session.mode,
       attachments: attachments.map(({ id, name, mime, size }) => ({ id, name, mime, size })),
       startedAt: Date.now(),
       endedAt: null,
@@ -391,6 +398,7 @@ export class Runner extends EventEmitter {
       this.interrupting.delete(session.id);
       this.stopping.delete(session.id); // a stop that landed after the result has nothing left to stop
       const next = session.queue.shift();
+      if (next?.mode) session.mode = next.mode;
       if (next && this.store.get(session.id)) this.#startTurn(session, next.text, next.attachments);
       else this.#touch(session);
     };

package/server/telemetry.js

@@ -5,6 +5,10 @@
 //
 // Off when any of: no API key, DO_NOT_TRACK=1, WISP_TELEMETRY=0, CI, or "telemetry": "off" in
 // settings. See the Telemetry section of the README.
+//
+// Feedback (thumbs on a result, /feedback, "Send feedback") is different: the user hands it to us
+// on purpose, so it goes out even with usage stats off. Only WISP_FEEDBACK=0 or no key stops it.
+// Offline, it waits in feedback-outbox.json and goes with the next feedback or the next start.
 
 import fs from 'node:fs';
 import os from 'node:os';
@@ -22,10 +26,16 @@ const TIMEOUT_MS = 3000;
 const VERSION = JSON.parse(fs.readFileSync(new URL('../package.json', import.meta.url), 'utf8')).version;
 const truthy = (v) => !!v && !/^(0|false|no|off)$/i.test(v);
 
+const falsy = (v) => /^(0|false|no|off)$/i.test(v || '');
+
 export function envDisabled(env = process.env) {
-  return !KEY || truthy(env.DO_NOT_TRACK) || /^(0|false|no|off)$/i.test(env.WISP_TELEMETRY || '') || truthy(env.CI);
+  return !KEY || truthy(env.DO_NOT_TRACK) || falsy(env.WISP_TELEMETRY) || truthy(env.CI);
 }
 
+export const feedbackAvailable = (env = process.env) => !!KEY && !falsy(env.WISP_FEEDBACK);
+const FEEDBACK_MAX = 4000;
+const OUTBOX_MAX = 100;
+
 const ROOT = path.dirname(path.dirname(fileURLToPath(import.meta.url)));
 
 // Paths in messages and stacks would leak usernames and project names: wisp's own files keep
@@ -50,10 +60,12 @@ function frames(stack = '') {
 }
 
 export class Telemetry {
-  constructor(store, { shell = process.versions.electron ? 'electron' : 'browser' } = {}) {
+  constructor(store, { shell = process.versions.electron ? 'electron' : 'browser', post } = {}) {
     this.store = store;
     this.queue = [];
     this.pendingFile = path.join(store.home, 'telemetry-pending.json');
+    this.outboxFile = path.join(store.home, 'feedback-outbox.json');
+    if (post) this.post = post;
     this.common = {
       $lib: 'wisp',
       $process_person_profile: false, // anonymous events: no person profiles
@@ -68,6 +80,7 @@ export class Telemetry {
     this.timer = setInterval(() => this.flush(), FLUSH_MS);
     this.timer.unref();
     this.#sendPending();
+    this.#sendOutbox();
   }
 
   get enabled() {
@@ -93,17 +106,49 @@ export class Telemetry {
     });
   }
 
-  async flush() {
-    if (!this.queue.length) return;
-    const batch = this.queue.splice(0);
+  // Thumbs on a turn ('turn_rated') or a written note ('feedback'). Resolves true once PostHog has
+  // it; false means it's kept in the outbox and retried.
+  async feedback(event, props = {}) {
+    if (!feedbackAvailable()) return false;
+    // With usage stats off, a fresh id per message so feedback can't be tied to anything else.
+    const id = (this.enabled && this.store.config.telemetryId) || randomUUID();
+    const text = props.text ? String(props.text).split(os.homedir()).join('~').slice(0, FEEDBACK_MAX) : undefined;
+    const ev = { event, timestamp: new Date().toISOString(), properties: { ...this.common, ...props, text, distinct_id: id } };
+    const batch = [...this.#outbox(), ev].slice(-OUTBOX_MAX);
+    const ok = await this.post(batch);
+    try {
+      if (ok) fs.rmSync(this.outboxFile, { force: true });
+      else fs.writeFileSync(this.outboxFile, JSON.stringify(batch));
+    } catch {}
+    return ok;
+  }
+
+  #outbox() {
+    try {
+      const list = JSON.parse(fs.readFileSync(this.outboxFile, 'utf8'));
+      return Array.isArray(list) ? list : [];
+    } catch {
+      return [];
+    }
+  }
+
+  async post(batch) {
     try {
-      await fetch(`${HOST}/batch/`, {
+      const res = await fetch(`${HOST}/batch/`, {
         method: 'POST',
         headers: { 'content-type': 'application/json' },
         body: JSON.stringify({ api_key: KEY, batch }),
         signal: AbortSignal.timeout(TIMEOUT_MS),
       });
-    } catch {} // offline or blocked: drop it, telemetry must never get in the way
+      return res.ok;
+    } catch {
+      return false;
+    }
+  }
+
+  async flush() {
+    if (!this.queue.length) return;
+    await this.post(this.queue.splice(0)); // offline or blocked: drop it, telemetry must never get in the way
   }
 
   // Shutdown and crashes leave no time for a request: write what's queued to disk (sync) and
@@ -122,6 +167,12 @@ export class Telemetry {
     this.persist();
   }
 
+  #sendOutbox() {
+    const batch = this.#outbox();
+    if (!batch.length || !feedbackAvailable()) return;
+    this.post(batch).then((ok) => ok && fs.rmSync(this.outboxFile, { force: true }));
+  }
+
   #sendPending() {
     try {
       const pending = JSON.parse(fs.readFileSync(this.pendingFile, 'utf8'));

package/web/css/app.css

@@ -305,7 +305,8 @@ body:has(.launcher.docked:is(:focus-within, :has(.open))) .to-top { display: non
 .l-bar { display: flex; align-items: center; gap: 8px; padding-top: 6px; }
 .l-bar > :not(.runners) { flex: none; }
 .l-bar .grow { flex: 1; }
-/* harness pill(s): "vs" rides on the end of it; on, the second harness joins the same pill */
+/* harness pill(s): "vs" rides on the end of it; each harness added joins the same pill behind a
+   "vs" of its own (hover: ×, removes it), and the end button turns into a + for the next one */
 .runners {
   display: inline-flex; align-items: stretch; min-width: 0; height: 30px; border-radius: 9px;
   border: 1px solid var(--line); background: var(--surface); transition: border-color 0.15s;
@@ -318,6 +319,12 @@ body:has(.launcher.docked:is(:focus-within, :has(.open))) .to-top { display: non
 .runners .rb-h { flex: 0 1 auto; min-width: 0; overflow: hidden; text-overflow: ellipsis; white-space: nowrap; }
 .runners .rb-m { flex: 0 100 auto; min-width: 0; }
 .runners.cmp .rb-e, .l-bar:has(.runners.cmp) #wtBtn span { display: none; }
+/* three or four harnesses: logos only (each button's tooltip names it), the bar's other labels fold too */
+.runners.many .runner-btn { flex: none; padding-inline: 6px; }
+.runners.many :is(.rb-h, .rb-m) { display: none; }
+.runners.many .rb-logo { margin-right: 3px; }
+.runners.many .runner-btn .chev { margin-left: 0; }
+.l-bar:has(.runners.many) :is(.mode-btn span, .team-btn .tb-label) { display: none; }
 /* two harnesses on a narrow bar: drop the model names, then the harness names (logos stay) */
 @container launcher (max-width: 640px) { .runners.cmp .rb-m { display: none; } }
 @container launcher (max-width: 520px) {
@@ -326,20 +333,29 @@ body:has(.launcher.docked:is(:focus-within, :has(.open))) .to-top { display: non
   .runners.cmp .runner-btn .chev { margin-left: 0; }
   .runners.cmp .rb-logo { margin-right: 4px; }
 }
+@container launcher (max-width: 600px) {
+  .runners.many .runner-btn .chev { display: none; }
+  .runners.many .vs-del { width: 18px; }
+}
 .vs-btn {
   position: relative; flex: none; display: grid; place-items: center; width: 30px;
   font: 600 11px var(--mono); letter-spacing: 0.02em; color: var(--faint); transition: color 0.15s, background 0.15s;
 }
-.vs-btn::before { content: ''; position: absolute; left: 0; top: 7px; bottom: 7px; width: 1px; background: var(--line); }
+.vs-btn[hidden] { display: none; }
 .vs-btn:hover { color: var(--accent-ink); }
 .vs-btn .i { display: none; width: 13px; height: 13px; }
 .vs-btn:focus-visible { outline: 0; border-radius: 6px; box-shadow: inset 0 0 0 1px var(--accent); }
+.vs-add::before { content: ''; position: absolute; left: 0; top: 7px; bottom: 7px; width: 1px; background: var(--line); }
 .runners.cmp { border-color: color-mix(in srgb, var(--accent) 35%, var(--line)); }
-.runners.cmp .vs-btn { width: 26px; color: var(--accent-ink); }
-.runners.cmp .vs-btn::before { display: none; }
-.runners.cmp .vs-btn:hover { color: var(--ink); }
-.runners.cmp .vs-btn:is(:hover, :focus-visible) span { display: none; }
-.runners.cmp .vs-btn:is(:hover, :focus-visible) .i { display: block; }
+.runners.cmp .vs-add { width: 26px; }
+.runners.cmp .vs-add span { display: none; }
+.runners.cmp .vs-add .i { display: block; width: 12px; height: 12px; }
+.vs-del { width: 24px; color: var(--accent-ink); }
+.runners.many .vs-del { width: 20px; }
+.vs-del:hover { color: var(--bad); }
+.vs-del:not(.fresh):is(:hover, :focus-visible) span { display: none; }
+.vs-del:not(.fresh):is(:hover, :focus-visible) .i { display: block; }
+.vs-del.fresh { cursor: default; }
 /* permission mode: icon-only on the default, labelled otherwise */
 .mode-btn {
   display: inline-flex; align-items: center; gap: 6px; height: 32px; min-width: 32px; padding: 0 9px; border-radius: 9px;
@@ -434,13 +450,13 @@ body:has(.launcher.docked:is(:focus-within, :has(.open))) .to-top { display: non
 }
 
 /* ---------- feed ---------- */
-.feed-head { display: flex; align-items: center; gap: 14px; min-height: 22px; margin: 26px 2px 10px; font-size: 13px; }
-.fh-status { display: flex; gap: 14px; }
+.feed-head { display: flex; align-items: center; gap: 14px; min-height: 22px; margin: 26px 0 10px; padding: 0 4px; font-size: 13px; }
 .feed-head[hidden] + .feed { margin-top: 26px; }
 .hs { display: inline-flex; align-items: center; gap: 7px; color: var(--ink-2); }
 .hs i { width: 7px; height: 7px; border-radius: 50%; }
 .hs.working i { background: var(--accent); animation: pulse 1.6s infinite; }
 .hs.needs i { background: var(--warn); }
+.hs.calm i { background: var(--good); }
 
 .f-group { margin-bottom: 22px; }
 .f-group h4 {
@@ -462,6 +478,11 @@ body:has(.launcher.docked:is(:focus-within, :has(.open))) .to-top { display: non
 .toggle-done:hover { color: var(--ink-2); }
 .toggle-done .chev { width: 12px; height: 12px; transition: transform 0.2s; }
 .f-group.done:not(.collapsed) .chev { transform: rotate(90deg); }
+/* Folded groups are just a row of headings: stack them tight instead of leaving a section gap each. */
+.f-group.collapsed { margin-bottom: 2px; }
+.f-group.collapsed h4 { margin: 0; }
+.f-group h4.toggle-done { width: fit-content; min-height: 24px; margin-left: -4px; padding: 0 8px; border-radius: 6px; transition: background 0.15s, color 0.15s; }
+.f-group h4.toggle-done:hover { background: var(--bg-2); }
 .f-list { display: flex; flex-direction: column; gap: 6px; }
 .feed-empty { padding: 48px 0; text-align: center; color: var(--faint); font-size: 13.5px; }
 
@@ -1065,6 +1086,28 @@ pre.code { margin: 0 0 12px; padding: 12px 14px; border-radius: 10px; background
 .trail-bar .link { font-size: 12px; }
 .trail-body { padding: 8px 0 0; animation: rise 0.4s var(--ease); }
 .trail-body .presentation { gap: 10px; }
+/* Thumbs: faint until the trail is hovered or the turn is rated, so a finished turn reads the same. */
+.rate { display: inline-flex; gap: 1px; margin-left: -6px; opacity: 0.5; transition: opacity 0.15s; }
+.turn:hover .rate, .rate.rated, .rate:focus-within { opacity: 1; }
+.rate-btn { display: inline-grid; place-items: center; width: 24px; height: 22px; border-radius: 6px; color: var(--faint); transition: color 0.15s, background 0.15s; }
+.rate-btn .i { width: 13px; height: 13px; }
+.rate-btn:hover { color: var(--ink); background: var(--bg-2); }
+.rate-btn[data-rate="up"].on { color: var(--good); }
+.rate-btn[data-rate="down"].on { color: var(--bad); }
+.rate-btn.on .i { fill: color-mix(in srgb, currentColor 18%, transparent); }
+.rate.thanked [data-rate="up"].on .i { animation: thumb-pop 0.45s var(--ease); }
+@keyframes thumb-pop { 40% { transform: scale(1.35) rotate(-12deg); } }
+.why { display: grid; gap: 6px; }
+.why .textarea { min-height: 0; resize: none; }
+.why-foot { display: flex; align-items: center; gap: 10px; }
+.why-foot .muted { margin-right: auto; }
+.why-foot .primary { height: 24px; padding: 0 10px; font-size: 12px; border-radius: 7px; }
+.fb-modal { width: min(520px, 100%); }
+.fb-rate { display: flex; align-items: center; gap: 4px; margin-bottom: 10px; font-size: 13px; color: var(--muted); }
+.fb-rate span { margin-right: 6px; }
+.fb-rate .rate-btn { width: 30px; height: 28px; }
+.fb-rate .rate-btn .i { width: 16px; height: 16px; }
+.fb-modal .ship-foot { margin-top: 12px; }
 
 /* ---------- per-item composer ---------- */
 .composer {
@@ -1082,6 +1125,12 @@ pre.code { margin: 0 0 12px; padding: 12px 14px; border-radius: 10px; background
 .cq .att-x:hover { color: var(--bad); }
 .cq-text { min-width: 0; overflow: hidden; text-overflow: ellipsis; white-space: nowrap; color: var(--muted); }
 .c-hint { flex: 1; font-size: 11.5px; color: var(--faint); }
+.c-bar .mode-btn { height: 30px; min-width: 30px; padding: 0 8px; }
+.c-bar .mode-btn.changed { background: var(--bg-2); }
+.t-mode { display: inline-flex; align-items: center; gap: 4px; padding: 1px 6px; border-radius: 5px; font: 500 11px var(--sans); color: var(--muted); border: 1px solid var(--line); }
+.t-mode .i { width: 12px; height: 12px; }
+.t-mode.m-plan { color: var(--accent-ink); }
+.t-mode.m-auto { color: var(--warn); }
 .send { display: grid; place-items: center; width: 30px; height: 30px; border-radius: 8px; background: var(--ink); color: var(--bg); transition: opacity 0.15s; }
 .send:hover { opacity: 0.86; }
 .send:disabled { opacity: 0.5; }
@@ -1200,6 +1249,8 @@ pre.code { margin: 0 0 12px; padding: 12px 14px; border-radius: 10px; background
 .sx-ctl .input { width: 300px; }
 .sx-state { display: inline-flex; align-items: center; gap: 6px; font-size: 12px; color: var(--muted); }
 .sx-state.bad { color: var(--bad); }
+#recheck.busy { cursor: progress; }
+#recheck.busy .i { animation: spin 0.8s linear infinite reverse; }
 .sx-usage .it-loading { grid-column: 1 / -1; }
 /* a view (Memory, Automations, Usage, Phone) opened inside Settings: no dialog chrome of its own */
 .modal.embedded { width: auto; max-height: none; overflow: visible; padding: 0; border: 0; border-radius: 0; box-shadow: none; background: none; transform: none; }
@@ -1476,6 +1527,12 @@ pre.code { margin: 0 0 12px; padding: 12px 14px; border-radius: 10px; background
 .mc-adv summary { cursor: pointer; width: max-content; font-size: 12.5px; color: var(--muted); }
 .mc-adv[open] summary { margin-bottom: 8px; }
 .mc-check { margin-top: 8px; }
+/* Hosted inside Settings: undo its row styling for labels (centred, padded, bordered, input flexed to 0 tall). */
+.mc-form label { padding: 0; border-bottom: 0; }
+.mc-form .mc-field { align-items: stretch; justify-content: flex-start; gap: 5px; }
+.mc-form .mc-field > span { text-align: left; }
+.mc-form .mc-field > .input { flex: none; max-width: none; }
+.mc-form .mc-check { display: inline-flex; justify-content: flex-start; gap: 8px; width: max-content; }
 .mc-foot { display: flex; align-items: center; gap: 10px; }
 .mc-foot .hint { flex: 1; margin: 0 !important; font-size: 12px !important; }
 .mc-foot .primary { flex: none; }
@@ -1680,6 +1737,8 @@ body.electron.mac .topbar { padding-left: 88px; }
 .ph-ok { color: var(--good); }
 .ph-bad { color: var(--bad); max-width: 260px; overflow: hidden; text-overflow: ellipsis; white-space: nowrap; }
 .ph-ok .i, .ph-bad .i { width: 12px; height: 12px; }
+.ph-start { margin-top: 10px; padding-top: 10px; border-top: 1px solid var(--line); }
+.ph-row { display: flex; align-items: center; gap: 8px; font-size: 13px; margin-bottom: 6px; }
 
 /* ---------- subagents ---------- */
 /* logo stack: the lead in front, its subagents tucked behind; they fan out on hover */

package/web/index.html

@@ -56,6 +56,8 @@
       <symbol id="i-clock" viewBox="0 0 24 24"><circle cx="12" cy="12" r="8.5" fill="none" stroke="currentColor" stroke-width="1.8"/><path d="M12 7.5V12l3 2" fill="none" stroke="currentColor" stroke-width="1.8" stroke-linecap="round" stroke-linejoin="round"/></symbol>
       <symbol id="i-laptop" viewBox="0 0 24 24"><rect x="4.5" y="5" width="15" height="10.5" rx="1.8" fill="none" stroke="currentColor" stroke-width="1.7"/><path d="M2.5 19h19" stroke="currentColor" stroke-width="1.7" stroke-linecap="round"/></symbol>
       <symbol id="i-server" viewBox="0 0 24 24"><rect x="3.5" y="4" width="17" height="7" rx="2" fill="none" stroke="currentColor" stroke-width="1.7"/><rect x="3.5" y="13" width="17" height="7" rx="2" fill="none" stroke="currentColor" stroke-width="1.7"/><circle cx="7.3" cy="7.5" r="1.1" fill="currentColor"/><circle cx="7.3" cy="16.5" r="1.1" fill="currentColor"/><path d="M11 7.5h5.5M11 16.5h5.5" stroke="currentColor" stroke-width="1.5" stroke-linecap="round"/></symbol>
+      <symbol id="i-thumb-up" viewBox="0 0 24 24"><path d="M7.5 10.5v9H4.8a.8.8 0 01-.8-.8v-7.4a.8.8 0 01.8-.8h2.7zm0 0l3.6-6.3a1.7 1.7 0 013.1 1.2l-.6 3.6h4.6a2 2 0 012 2.3l-1.1 6.4a2.4 2.4 0 01-2.4 2.1H7.5" fill="none" stroke="currentColor" stroke-width="1.7" stroke-linejoin="round"/></symbol>
+      <symbol id="i-thumb-down" viewBox="0 0 24 24"><path d="M7.5 13.5v-9H4.8a.8.8 0 00-.8.8v7.4a.8.8 0 00.8.8h2.7zm0 0l3.6 6.3a1.7 1.7 0 003.1-1.2l-.6-3.6h4.6a2 2 0 002-2.3l-1.1-6.4a2.4 2.4 0 00-2.4-2.1H7.5" fill="none" stroke="currentColor" stroke-width="1.7" stroke-linejoin="round"/></symbol>
       <symbol id="i-play" viewBox="0 0 24 24"><path d="M7 5.5v13l11-6.5-11-6.5z" fill="currentColor"/></symbol>
     </defs>
   </svg>