@timqi/pier 0.1.3 → 0.1.5

package/dist/channels/routes.js

@@ -1,5 +1,7 @@
-// Settings → Channels HTTP surface: one document per platform.
+// Settings → Channels HTTP surface: one document per platform, and the
+// web → IM handoff beside it.
 import { isThinkingLevel } from "../core/types.js";
+import { HandoffError } from "./handoff.js";
 import { defaultChannelConfig, isChannelPlatform, } from "./types.js";
 /** Stable mask: recomputable at write time, so "unchanged" is detectable. */
 const maskToken = (token) => token ? `${"•".repeat(8)}${token.slice(-4)}` : "";
@@ -35,14 +37,13 @@ function parseChats(raw, known) {
             enabled: asBool(edit.enabled),
             requireMention: asBool(edit.requireMention),
             requireBind: asBool(edit.requireBind),
-            topicMode: asBool(edit.topicMode),
             cwd: asString(edit.cwd),
             model: asModel(edit.model),
             thinking: asThinking(edit.thinking),
         };
     });
 }
-export function registerChannelRoutes(app, store, runtime) {
+export function registerChannelRoutes(app, store, runtime, handoff) {
     app.get("/api/channels/:platform", (c) => {
         const platform = c.req.param("platform");
         if (!isChannelPlatform(platform))
@@ -70,12 +71,8 @@ export function registerChannelRoutes(app, store, runtime) {
             enabled: asBool(body.enabled),
             token: kept(asString(body.token), current.token),
             appToken: kept(asString(body.appToken), current.appToken),
-            // Absent means "on": a client that predates the field must not silently
-            // switch off a capability the operator never touched.
-            agentTool: body.agentTool === undefined ? current.agentTool : asBool(body.agentTool),
             requireMention: asBool(body.requireMention),
             requireBind: asBool(body.requireBind),
-            topicMode: asBool(body.topicMode),
             cwd: asString(body.cwd),
             model: asModel(body.model),
             thinking: asThinking(body.thinking),
@@ -100,4 +97,21 @@ export function registerChannelRoutes(app, store, runtime) {
         store.unbind(platform, c.req.param("id"));
         return c.json({ ok: true });
     });
+    app.get("/api/handoff/targets", (c) => c.json({ targets: handoff.targets() }));
+    app.post("/api/handoff", async (c) => {
+        const body = (await c.req.json().catch(() => null));
+        const sessionId = asString(body?.sessionId);
+        const chatId = asString(body?.chatId);
+        const platform = body?.platform;
+        if (!sessionId || !chatId || !isChannelPlatform(platform))
+            return c.json({ error: "invalid body" }, 400);
+        try {
+            return c.json(await handoff.continueIn({ sessionId, platform, chatId }), 201);
+        }
+        catch (err) {
+            if (err instanceof HandoffError)
+                return c.json({ error: err.message }, err.status);
+            throw err;
+        }
+    });
 }

package/dist/channels/runtime.js

@@ -3,12 +3,10 @@
 import { logger } from "../log.js";
 import { LarkChannel } from "./lark.js";
 import { SlackChannel } from "./slack.js";
-import { TelegramChannel } from "./telegram.js";
 const ADAPTERS = [
-    { platform: "telegram", needsAppToken: false, build: (deps) => new TelegramChannel(deps) },
-    { platform: "slack", needsAppToken: true, build: (deps) => new SlackChannel(deps) },
+    { platform: "slack", build: (deps) => new SlackChannel(deps) },
     // Lark's "token" is the App ID and "appToken" the App Secret.
-    { platform: "lark", needsAppToken: true, build: (deps) => new LarkChannel(deps) },
+    { platform: "lark", build: (deps) => new LarkChannel(deps) },
 ];
 // The injected sink is for warnings; "slack started" is not one.
 const log = logger("channels");
@@ -18,14 +16,18 @@ export class ChannelRuntime {
     store;
     router;
     control;
+    handoff;
     log;
     live = new Map();
     reloading = Promise.resolve();
     stopped = false;
-    constructor(store, router, control, log = warn) {
+    constructor(store, router, control, 
+    /** The pull half only; the push half needs this runtime, so main.ts closes the loop. */
+    handoff, log = warn) {
         this.store = store;
         this.router = router;
         this.control = control;
+        this.handoff = handoff;
         this.log = log;
     }
     /** Serialized: two concurrent Console saves would race into duplicate live
@@ -45,7 +47,7 @@ export class ChannelRuntime {
         return run;
     }
     async restart(adapter) {
-        const { platform, needsAppToken, build } = adapter;
+        const { platform, build } = adapter;
         const existing = this.live.get(platform);
         if (existing) {
             this.live.delete(platform);
@@ -55,7 +57,7 @@ export class ChannelRuntime {
         const config = this.store.get(platform);
         if (!config.enabled || !config.token)
             return;
-        if (needsAppToken && !config.appToken) {
+        if (!config.appToken) {
             // "Enabled but nothing happens" is indistinguishable from a broken adapter.
             this.log(`${platform}: enabled but no app token, not starting`);
             return;
@@ -64,6 +66,7 @@ export class ChannelRuntime {
             store: this.store,
             log: (m) => this.log(`${platform}: ${m}`),
             control: this.control,
+            handoff: this.handoff,
         });
         try {
             await channel.start((msg) => {
@@ -87,6 +90,17 @@ export class ChannelRuntime {
         await channel.notify(conversationId, { text, origin: { kind: "error" } });
         return true;
     }
+    running() {
+        return [...this.live.keys()];
+    }
+    /** Throws by name when the platform is not running: the caller's answer is
+     *  "enable it in Settings", not a silent no-op. */
+    async openThread(platform, chatId, note) {
+        const channel = this.live.get(platform);
+        if (!channel)
+            throw new Error(`${platform} is not running`);
+        return channel.openThread(chatId, note);
+    }
     async stop() {
         // An in-flight restart could otherwise register an adapter after `live`
         // was cleared — running, unstoppable.

package/dist/channels/slack-api.js

@@ -21,18 +21,24 @@ export class SlackApi {
     appToken;
     log;
     openSocket;
+    retries;
     socketRunning = false;
     constructor(token, appToken, log = () => { }, 
     /** Injected in tests. */
-    openSocket = (url) => new WebSocket(url)) {
+    openSocket = (url) => new WebSocket(url), 
+    /** How many rate-limit waits one call sits through. The adapter takes one:
+     *  a turn must not hang for minutes. A batch process (`pier slack`) takes more. */
+    retries = 1) {
         this.token = token;
         this.appToken = appToken;
         this.log = log;
         this.openSocket = openSocket;
+        this.retries = retries;
     }
     /** Slack accepts a JSON body only on write methods; a read method silently
      *  ignores it and reports the parameter missing (`users.info` answers
-     *  `user_not_found`). So reads go form-encoded. */
+     *  `user_not_found`). So reads go form-encoded. Public for the long tail of
+     *  read methods `pier slack` needs and the adapter never calls. */
     async read(method, params) {
         const form = new URLSearchParams();
         for (const [key, value] of Object.entries(params)) {
@@ -41,7 +47,7 @@ export class SlackApi {
         }
         return this.call(method, form);
     }
-    async call(method, payload, token = this.token, retry = true) {
+    async call(method, payload, token = this.token, tries = this.retries) {
         const form = payload instanceof URLSearchParams;
         const res = await fetch(`${BASE}/${method}`, {
             method: "POST",
@@ -54,16 +60,17 @@ export class SlackApi {
             body: form ? payload.toString() : JSON.stringify(payload),
             signal: AbortSignal.timeout(30_000),
         });
+        const body = (await res.json().catch(() => null));
         // A long turn split into chunks hits ~1 msg/s per channel; the header
-        // carries the exact wait. Obeyed once; a second 429 throws.
-        if (res.status === 429 && retry) {
+        // carries the exact wait. Some methods say `ratelimited` in a 200 body
+        // instead. Obeyed `retries` times; the next one throws.
+        if ((res.status === 429 || body?.error === "ratelimited") && tries > 0) {
             const after = Number(res.headers.get("retry-after") ?? "1");
             if (Number.isFinite(after) && after <= 60) {
                 await new Promise((r) => setTimeout(r, (after + 1) * 1000));
-                return this.call(method, payload, token, false);
+                return this.call(method, payload, token, tries - 1);
             }
         }
-        const body = (await res.json().catch(() => null));
         if (!body)
             throw new Error(`slack ${method}: ${res.status} with no JSON body`);
         if (!body.ok)
@@ -259,7 +266,7 @@ export class SlackApi {
     }
     /** Three calls is Slack's current upload (`files.upload` is retired); the
      *  upload host is not the Web API and answers plain text. */
-    async uploadFile(channel, threadTs, file) {
+    async uploadFile(channel, threadTs, file, comment) {
         const slot = await this.read("files.getUploadURLExternal", { filename: file.name, length: file.bytes.length }).catch((err) => {
             // The manifest is only applied when an app is created; name the fix.
             if (!/missing_scope/.test(String(err)))
@@ -279,10 +286,12 @@ export class SlackApi {
         });
         if (!put.ok)
             throw new Error(`slack file upload: ${put.status}`);
-        await this.call("files.completeUploadExternal", {
+        const done = await this.call("files.completeUploadExternal", {
             files: [{ id: slot.file_id, title: file.name }],
             channel_id: channel,
-            thread_ts: threadTs,
+            thread_ts: threadTs || undefined,
+            initial_comment: comment,
         });
+        return { id: done.files?.[0]?.id ?? slot.file_id };
     }
 }