pikiloom 0.4.26 → 0.4.27

package/dist/channels/weixin/bot.js

@@ -1,6 +1,3 @@
-/**
- * WeChat bot orchestration.
- */
 import fs from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
@@ -21,11 +18,6 @@ const SHUTDOWN_EXIT_CODE = {
 function describeError(error) {
     return error instanceof Error ? error.message : String(error ?? 'unknown error');
 }
-/**
- * Plain-text echo of a resolved human-loop prompt — WeChat doesn't render
- * markdown or cards, so we keep the format minimal and obviously distinct
- * from a regular agent reply.
- */
 function buildInteractionEchoPlain(summary) {
     if (summary.status === 'cancelled')
         return '⊘ Prompt cancelled.';
@@ -143,9 +135,6 @@ export class WeixinBot extends Bot {
         const [rawCommand, ...rest] = text.trim().slice(1).split(/\s+/);
         const command = rawCommand?.toLowerCase() || '';
         const args = rest.join(' ').trim();
-        // `/cancel` aborts any pending command-UI / im_ask_user prompt for this
-        // chat. Special-cased before the auto-cancel below so a bare `/cancel`
-        // doesn't get treated as the start of a new interactive command.
         if (command === 'cancel' || command === 'quit') {
             const pending = this.pendingHumanLoopPrompt(ctx.chatId);
             if (pending) {
@@ -156,8 +145,6 @@ export class WeixinBot extends Bot {
             }
             return true;
         }
-        // Auto-clear any stale picker before starting a new interactive command
-        // so the user isn't trapped behind a forgotten `/agent` / `/models` etc.
         const pendingPrompt = this.pendingHumanLoopPrompt(ctx.chatId);
         if (pendingPrompt) {
             this.humanLoopCancel(pendingPrompt.promptId, 'Cancelled — new command issued.');
@@ -292,9 +279,6 @@ export class WeixinBot extends Bot {
     }
     async cmdAgent(ctx, args) {
         if (args) {
-            // Power-user shortcut: `/agent claude` switches directly without the
-            // interactive picker — same behaviour as before, preserved for muscle
-            // memory and scripted flows.
             try {
                 const agent = normalizeAgent(args);
                 this.switchAgentForChat(ctx.chatId, agent);
@@ -309,9 +293,6 @@ export class WeixinBot extends Bot {
     }
     async cmdModels(ctx, args) {
         if (args) {
-            // Power-user shortcut: `/models <number|name>` switches directly. Skips
-            // the multi-step picker (model + effort + Apply) used by the interactive
-            // path; effort stays unchanged.
             const d = await getModelsListData(this, ctx.chatId);
             const idx = parseInt(args, 10);
             let modelId = null;
@@ -378,7 +359,6 @@ export class WeixinBot extends Bot {
         }
         const trimmed = args.trim();
         if (trimmed) {
-            // Power-user shortcut: `/workspaces <#>` switches directly.
             const idx = parseInt(trimmed, 10);
             if (Number.isNaN(idx) || idx < 1 || idx > data.workspaces.length) {
                 await ctx.reply(`Workspace #${trimmed} not found. Use /workspaces to list.`);
@@ -393,10 +373,6 @@ export class WeixinBot extends Bot {
             await ctx.reply(`Workdir switched:\n${oldPath}\n→ ${ws.path}`);
             return;
         }
-        // Interactive picker — there is no `command-ui` builder for workspaces
-        // (Telegram/Feishu render this via channel-local helpers), so we open a
-        // tailored human-loop prompt here. Disabled rows for missing paths are
-        // marked so users see them but can't pick.
         const taskId = `wxcmd-ws-${Date.now().toString(36)}`;
         const promptLines = [
             '【Workspaces】',
@@ -480,8 +456,6 @@ export class WeixinBot extends Bot {
         }
         const idx = parseInt(arg, 10);
         if (!isNaN(idx) && idx >= 1) {
-            // Power-user shortcut: `/sessions <#>` jumps directly using the same
-            // first-100 lookup the old text flow used.
             const d = await getSessionsPageData(this, ctx.chatId, 0, 100);
             const target = d.sessions[idx - 1];
             if (target) {
@@ -499,8 +473,6 @@ export class WeixinBot extends Bot {
             await ctx.reply(`Session #${idx} not found.`);
             return;
         }
-        // Interactive list with pagination — `executeCommandAction` returns the
-        // next page as a fresh view, so the loop re-renders without leaving.
         await this.runCommandUiLoop(ctx, () => buildSessionsCommandView(this, ctx.chatId, 0));
     }
     async cmdSkills(ctx) {
@@ -553,12 +525,6 @@ export class WeixinBot extends Bot {
             : ['Task failed.', result.error || result.message || 'Unknown error.'].filter(Boolean).join('\n');
         await this.channel.send(chatId, text);
     }
-    /**
-     * Format a human-in-the-loop prompt as plain text for WeChat (no card / button
-     * support on personal accounts). Options are numbered so the user can either
-     * reply with the number or freeform text — both are routed back through the
-     * same handler in `handleMessage`.
-     */
     formatHumanLoopPromptText(prompt) {
         const question = currentHumanLoopQuestion(prompt);
         const lines = [];
@@ -586,17 +552,10 @@ export class WeixinBot extends Bot {
         }
         return lines.join('\n');
     }
-    /**
-     * IM presenter for programmatic submissions (e.g. /goal-driven turns).
-     * WeChat has no edit / card primitive, so streaming preview isn't possible
-     * — the result is just posted once when the turn completes, mirroring
-     * handleMessage's plain-text path.
-     */
     async createImTaskPresenter(opts) {
         const chatId = String(opts.chatId);
         return {
             onText: () => {
-                // No streaming render in WeChat — text accumulates and is sent once.
             },
             onSuccess: async (result) => {
                 await this.sendResult(chatId, result);
@@ -621,11 +580,6 @@ export class WeixinBot extends Bot {
             this.log(`weixin renderInteractionPrompt failed: ${describeError(error)}`);
         }
     }
-    /**
-     * WeChat has no message-edit primitive, so the original prompt text stays
-     * frozen in chat history. The echo message is the only way to record the
-     * decision — keep it short and prefix-marked so it's easy to skim.
-     */
     async onInteractionAnswered(prompt, summary) {
         const text = buildInteractionEchoPlain(summary);
         if (!text)
@@ -637,11 +591,6 @@ export class WeixinBot extends Bot {
             this.log(`weixin onInteractionAnswered echo failed: ${describeError(error)}`);
         }
     }
-    /**
-     * If the incoming text looks like an option number (e.g. "1", "2.", "③"),
-     * resolve it against the current question's options. Otherwise return null
-     * and let the freeform path take over.
-     */
     parseHumanLoopOptionPick(text, prompt) {
         const question = currentHumanLoopQuestion(prompt);
         if (!question?.options?.length)
@@ -656,14 +605,6 @@ export class WeixinBot extends Bot {
         const opt = question.options[idx - 1];
         return opt?.value || opt?.label || null;
     }
-    // ---- Command-UI adapter (numbered-text fallback for non-card IMs) -------
-    //
-    // Telegram / Feishu render `CommandSelectionView` as tap-able button cards.
-    // WeChat (personal accounts) can't render cards, so we flatten each view's
-    // button rows into a numbered text prompt and let the user reply with a
-    // digit. Re-uses the SAME `executeCommandAction` data layer — no parallel
-    // logic — so `/agents` / `/models` / `/sessions` / `/skills` / `/mode`
-    // behave identically across channels, just rendered differently.
     decorateCommandButtonLabel(button) {
         let label = button.label.trim();
         if (button.state === 'current' || button.primary)
@@ -674,11 +615,6 @@ export class WeixinBot extends Bot {
             label += ' [n/a]';
         return label;
     }
-    /**
-     * Render a `CommandSelectionView` as plain WeChat text: title + meta lines +
-     * detailed item list (when present) + numbered button list. Always ends with
-     * "回复编号选择" so the user knows what to do next.
-     */
     formatCommandViewText(view, buttons) {
         const lines = [];
         lines.push(`【${view.title}】`);
@@ -717,12 +653,6 @@ export class WeixinBot extends Bot {
             parts.push(notice.detail);
         return parts.join('\n');
     }
-    /**
-     * Open one prompt for a `CommandSelectionView` and resolve once the user
-     * picks an option (or cancels). The picked CommandAction is executed via the
-     * shared `executeCommandAction` so the side effects (state mutations,
-     * notices) match every other channel exactly.
-     */
     async promptCommandView(ctx, view) {
         const buttons = view.rows.flat();
         if (!buttons.length) {
@@ -779,13 +709,6 @@ export class WeixinBot extends Bot {
                 .catch(() => resolve(null));
         });
     }
-    /**
-     * Multi-step command flow driver. Some views (notably `/models`'s "select
-     * model → set effort → Apply") return another `CommandSelectionView` from
-     * `executeCommandAction` rather than a terminal notice; this loop keeps
-     * re-prompting until we hit a notice / skill / noop / cancellation. Capped
-     * at 12 iterations to guard against pathological loops.
-     */
     async runCommandUiLoop(ctx, viewBuilder) {
         let view = await Promise.resolve(viewBuilder());
         let safety = 12;
@@ -801,9 +724,6 @@ export class WeixinBot extends Bot {
                     await ctx.reply(this.formatCommandNotice(result.notice));
                     return;
                 case 'skill':
-                    // Dispatch the resolved skill prompt back through the regular task
-                    // pipeline (same machinery as a typed message), so the agent picks
-                    // it up with the right session + workspace.
                     await ctx.reply(`Running /${result.skillName}…`);
                     await this.dispatchUserPrompt(ctx, result.prompt, []);
                     return;
@@ -815,11 +735,6 @@ export class WeixinBot extends Bot {
         if (safety <= 0)
             await ctx.reply('Command UI loop terminated (too many steps).');
     }
-    /**
-     * Submit a user-authored prompt through the same path `handleMessage` uses
-     * for natural-language messages. Lets the command-UI loop forward a resolved
-     * `/skill` expansion as if the user had typed it.
-     */
     async dispatchUserPrompt(ctx, text, files) {
         const session = this.resolveSession(ctx.chatId, text, files);
         const prompt = buildPrompt(text, files);
@@ -879,8 +794,6 @@ export class WeixinBot extends Bot {
             await ctx.reply('This Weixin channel currently supports text input only.');
             return;
         }
-        // Active human-in-the-loop prompt takes priority — text-only replies are
-        // routed back to the pending question (option-number pick OR freeform).
         const pendingPrompt = this.pendingHumanLoopPrompt(ctx.chatId);
         if (pendingPrompt && text && !msg.files.length) {
             const optionValue = this.parseHumanLoopOptionPick(text, pendingPrompt);
@@ -892,7 +805,6 @@ export class WeixinBot extends Bot {
                 return;
             }
             if (result.completed) {
-                // Closing message comes from onInteractionAnswered — no extra reply.
             }
             else if (result.advanced) {
                 const next = this.humanLoopPrompt(pendingPrompt.promptId);

package/dist/channels/weixin/channel.js

@@ -1,6 +1,3 @@
-/**
- * WeChat channel transport.
- */
 import { Channel, DEFAULT_CHANNEL_CAPABILITIES, splitText, sleep, } from '../base.js';
 import { WEIXIN_LIMITS } from '../../core/constants.js';
 import { ChannelHealth } from '../health.js';
@@ -82,9 +79,6 @@ export class WeixinChannel extends Channel {
                     });
                     if (response.get_updates_buf !== undefined)
                         this.updateBuf = response.get_updates_buf || '';
-                    // Errcode check BEFORE recordSuccess(): an HTTP 200 carrying a
-                    // business error code (e.g. "session timeout") must not silently
-                    // reset the exponential backoff.
                     if ((response.ret ?? 0) !== 0 || (response.errcode ?? 0) !== 0) {
                         throw new Error(`Weixin getupdates failed: ${response.errmsg || response.errcode || response.ret}`);
                     }

package/dist/cli/autostart.js

@@ -1,26 +1,3 @@
-/**
- * macOS LaunchAgent integration for `pikiloom --daemon`.
- *
- * Every time the user runs pikiloom with an explicit `--daemon` flag on
- * macOS *without* a LaunchAgent already installed, an osascript dialog asks
- * whether to enable login auto-start. Choosing Enable writes
- * `~/Library/LaunchAgents/ai.pikiloom.gateway.plist` and loads it via
- * `launchctl bootstrap`. There is intentionally no CLI to disable: the user
- * toggles it off under System Settings → General → Login Items, which is
- * where macOS surfaces every LaunchAgent installed in
- * `~/Library/LaunchAgents`.
- *
- * Decision flow (`maybePromptAutostart`):
- *   1. Non-darwin → no-op.
- *   2. Already running under launchd (PIKILOOM_FROM_LAUNCHD set) → no-op.
- *   3. plist exists but its ProgramArguments no longer point to a valid
- *      binary (e.g. Homebrew migration moved node) → silently rewrite.
- *   4. plist already valid → no-op.
- *   5. Non-interactive (no TTY, CI=1) → no-op.
- *   6. Otherwise → show dialog after a short delay so the bot is already
- *      live when the user sees it. "Not now" silently dismisses; the next
- *      `--daemon` run will ask again.
- */
 import { exec, execFile } from 'node:child_process';
 import fs from 'node:fs';
 import os from 'node:os';
@@ -33,16 +10,9 @@ const PLIST_LABEL = 'ai.pikiloom.gateway';
 const PLIST_DIR = path.join(os.homedir(), 'Library', 'LaunchAgents');
 const PLIST_PATH = path.join(PLIST_DIR, `${PLIST_LABEL}.plist`);
 const PIKILOOM_HOME = path.join(os.homedir(), STATE_DIR_NAME);
-// Pre-rename LaunchAgent (pikiclaw) — removed on (re)install so an upgraded
-// machine never runs two daemons. Delete a couple releases post-rename.
 const LEGACY_PLIST_LABELS = ['ai.pikiclaw.gateway'];
 const PROMPT_DELAY_MS = 3000;
 export const FROM_LAUNCHD_ENV = 'PIKILOOM_FROM_LAUNCHD';
-/**
- * Resolve the command used to launch pikiloom, so the plist can re-launch it
- * the same way. We distinguish npx (`.../_npx/<hash>/.../main.js`) from a
- * globally installed binary (`pikiloom` on PATH).
- */
 function detectInvocation() {
     const entry = process.argv[1] || '';
     const userArgs = process.argv.slice(2);
@@ -146,15 +116,12 @@ async function showEnableDialog() {
         return 'closed';
     }
     catch {
-        // User pressed Cmd-. or closed the dialog — osascript exits non-zero.
         return 'closed';
     }
 }
 async function bootstrapLaunchAgent(log) {
     const uid = process.getuid?.() ?? 0;
     const domain = `gui/${uid}`;
-    // bootout first so a reinstall replaces any previously-loaded copy. We
-    // swallow errors here since bootout fails harmlessly when nothing is loaded.
     await new Promise(resolve => {
         exec(`launchctl bootout ${domain}/${PLIST_LABEL}`, () => resolve());
     });
@@ -167,11 +134,6 @@ async function bootstrapLaunchAgent(log) {
         return false;
     }
 }
-/**
- * Remove the pre-rename LaunchAgent (`ai.pikiclaw.gateway`) if present, so an
- * upgraded install never runs two daemons. Best-effort; safe when nothing is
- * loaded. Delete a couple releases post-rename.
- */
 async function cleanupLegacyAutostart(log) {
     const uid = process.getuid?.() ?? 0;
     for (const label of LEGACY_PLIST_LABELS) {
@@ -193,7 +155,7 @@ async function cleanupLegacyAutostart(log) {
                 log(`autostart: removed legacy LaunchAgent ${label}`);
             }
         }
-        catch { /* best-effort */ }
+        catch { }
     }
 }
 async function installAutostart(log, invocation) {
@@ -213,10 +175,6 @@ async function installAutostart(log, invocation) {
     }
     return loaded;
 }
-/**
- * Top-level entry called once from the watchdog process when the user passed
- * an explicit `--daemon` flag. Fires and forgets; never throws.
- */
 export function maybePromptAutostart(log) {
     if (process.platform !== 'darwin')
         return;
@@ -250,7 +208,6 @@ export function maybePromptAutostart(log) {
         })();
     }, PROMPT_DELAY_MS).unref?.();
 }
-// ─── exports for tests ─────────────────────────────────────────────────
 export const __test = {
     PLIST_LABEL,
     PLIST_PATH,

package/dist/cli/channel-supervisor.js

@@ -1,20 +1,3 @@
-/**
- * channel-supervisor.ts — Owns the set of running IM channel bots and
- * reconciles it against the current user config without restarting the
- * pikiloom process.
- *
- * Subscribes to onUserConfigChange. Whenever the channels array or any
- * channel's credentials change, the supervisor stops the affected bot,
- * disposes its transport, and (if still configured) launches a fresh
- * instance with the new config — all in-process, leaving the dashboard,
- * other channels, and active coding sessions on unaffected bots running.
- *
- * Dashboard-as-terminal: the Web Dashboard is a first-class terminal, equal to
- * the IM channels. When no IM channel is configured, the supervisor keeps a
- * headless bot attached to the dashboard so it's a fully usable terminal on its
- * own; when an IM channel is added, that channel's bot takes over the dashboard
- * attachment and the headless bot is torn down.
- */
 import { loadUserConfig, onUserConfigChange } from '../core/config/user-config.js';
 import { resolveConfiguredChannels } from './channels.js';
 const CHANNEL_AFFECTING_KEYS = new Set([
@@ -35,26 +18,10 @@ const CHANNEL_AFFECTING_KEYS = new Set([
     'wecomBotSecret',
     'wecomEndpoint',
 ]);
-/**
- * How long to wait between stopping a channel and starting a replacement
- * with new credentials. Some IM providers hold the old connection slot
- * server-side for a moment after we close the socket; if a new client
- * with the same credentials connects too quickly the server rejects with
- * "system busy" and the SDK ends up in a broken state.
- *
- * Feishu's lark SDK is the known offender — its WSClient handshake races
- * with a "PingInterval" setup that crashes when the server rejects the
- * second connect. Telegram (long-poll over HTTP) and Weixin (HTTP long-
- * poll) tolerate immediate replacement.
- */
 const CHANNEL_REPLACE_SETTLE_MS = {
     telegram: 0,
     weixin: 0,
     feishu: 5_000,
-    // The Slack / Discord / DingTalk / WeChat-Work providers all hold the
-    // single-bot socket slot for a moment after disconnect; give them a
-    // small grace window so token rotation doesn't trip "already connected"
-    // style errors on the immediate replacement connection.
     slack: 3_000,
     discord: 3_000,
     dingtalk: 3_000,
@@ -140,7 +107,6 @@ function describeError(error) {
 }
 export class ChannelSupervisor {
     running = new Map();
-    /** Headless dashboard-terminal bot, present only while no IM channel runs. */
     headless = null;
     dashboard;
     log;
@@ -152,10 +118,6 @@ export class ChannelSupervisor {
         this.dashboard = opts.dashboard;
         this.log = opts.log;
     }
-    /**
-     * Initial launch + subscribe to config changes. Idempotent: calling
-     * twice is a no-op for the second call.
-     */
     async start() {
         if (this.started)
             return;
@@ -167,7 +129,6 @@ export class ChannelSupervisor {
             void this.reconcile(config);
         });
     }
-    /** Stop every running channel and unsubscribe from config changes. */
     async stop() {
         this.unsubscribe?.();
         this.unsubscribe = null;
@@ -175,7 +136,6 @@ export class ChannelSupervisor {
         await Promise.all(channels.map(channel => this.stopChannel(channel)));
         await this.stopHeadless();
     }
-    /** Reconcile running bots against the desired channel set + credentials. */
     async reconcile(config) {
         if (this.reconcileInFlight) {
             this.reconcilePending = true;
@@ -189,8 +149,6 @@ export class ChannelSupervisor {
                 await this.doReconcile(current);
                 if (!this.reconcilePending)
                     break;
-                // Newer config arrived during reconcile — re-read from disk so the
-                // next pass acts on the latest state.
                 current = loadUserConfig();
             }
         }
@@ -201,7 +159,6 @@ export class ChannelSupervisor {
     async doReconcile(config) {
         const desired = resolveConfiguredChannels({ config });
         const desiredSet = new Set(desired);
-        // Stop channels that are no longer desired OR whose credentials changed.
         const toStop = [];
         for (const [channel, entry] of this.running) {
             if (!desiredSet.has(channel)) {
@@ -215,18 +172,12 @@ export class ChannelSupervisor {
         if (toStop.length) {
             await Promise.all(toStop.map(channel => this.stopChannel(channel)));
         }
-        // For channels being replaced (still desired, just stopped above for
-        // credential rotation), wait the per-channel settle delay so the
-        // remote server has time to release the old connection slot before
-        // we open a new one.
         const replacing = toStop.filter(channel => desiredSet.has(channel));
         const settleMs = replacing.reduce((max, channel) => Math.max(max, CHANNEL_REPLACE_SETTLE_MS[channel] ?? 0), 0);
         if (settleMs > 0) {
             this.log(`waiting ${settleMs}ms for ${replacing.join(', ')} to settle before relaunch`);
             await sleep(settleMs);
         }
-        // Start channels that should be running but aren't (either added or
-        // just stopped above due to credential rotation).
         for (const channel of desired) {
             if (this.running.has(channel))
                 continue;
@@ -237,11 +188,6 @@ export class ChannelSupervisor {
                 this.log(`channel ${channel}: failed to start — ${describeError(err)}`);
             }
         }
-        // Dashboard-as-terminal: keep a headless bot attached whenever no channel
-        // bot is actually running (none configured, or all failed to start), so the
-        // dashboard stays a usable terminal on its own. Gate on `running.size`
-        // rather than `desired.length` so a channel that fails to come up still
-        // leaves the dashboard with a working bot.
         if (this.running.size > 0) {
             await this.stopHeadless();
         }
@@ -249,7 +195,6 @@ export class ChannelSupervisor {
             await this.startHeadless();
         }
     }
-    /** Launch the headless dashboard-terminal bot and attach it to the dashboard. */
     async startHeadless() {
         if (this.headless)
             return;
@@ -263,7 +208,6 @@ export class ChannelSupervisor {
         });
         this.headless = { bot, runPromise };
     }
-    /** Tear down the headless bot once a real channel bot owns the dashboard. */
     async stopHeadless() {
         if (!this.headless)
             return;
@@ -274,7 +218,7 @@ export class ChannelSupervisor {
         try {
             await runPromise;
         }
-        catch { /* already logged */ }
+        catch { }
     }
     async startChannel(channel, config) {
         this.log(`channel ${channel}: starting`);
@@ -295,8 +239,6 @@ export class ChannelSupervisor {
         if (!entry)
             return;
         this.log(`channel ${channel}: stopping`);
-        // Remove from the map up-front so a concurrent reconcile pass doesn't
-        // try to stop the same instance twice.
         this.running.delete(channel);
         try {
             entry.bot.requestStop();

package/dist/cli/channels.js

@@ -1,11 +1,5 @@
-/**
- * Channel resolution helpers for CLI startup.
- */
 import { applyChannelEnvFallback } from '../core/config/user-config.js';
 export function hasConfiguredChannelToken(rawConfig, channel, tokenOverride) {
-    // Channel transports already fall back to env vars at runtime
-    // (e.g. TELEGRAM_BOT_TOKEN). Mirror that here so a docker-only env setup
-    // doesn't make startup think there's nothing configured.
     const config = applyChannelEnvFallback(rawConfig);
     switch (channel) {
         case 'telegram':