@llblab/pi-telegram 0.46.0 → 0.47.0

package/lib/thread-display.ts

@@ -26,30 +26,88 @@ function boundedLabel(base: string, suffix = ""): string {
   return `${prefix}${suffix}`;
 }
 
-function directoryLabel(cwd: string, directories: readonly string[]): string {
-  const parts = cwd.split("/").filter(Boolean);
-  if (!parts.length) return "/";
+function directorySegments(cwd: string): string[] {
+  return cwd.split("/").filter(Boolean);
+}
+
+function distinguishingDirectorySegments(cwd: string, directories: readonly string[]): string[] {
+  const parts = directorySegments(cwd);
+  if (!parts.length) return [];
   for (let depth = 1; depth <= parts.length; depth++) {
     const candidate = labelText(parts.slice(-depth).join("/"));
     const collides = directories.some((other) => other !== cwd &&
-      labelText(other.split("/").filter(Boolean).slice(-depth).join("/")).toLowerCase() ===
+      labelText(directorySegments(other).slice(-depth).join("/")).toLowerCase() ===
         candidate.toLowerCase(),
     );
-    if (!collides) return candidate;
+    if (!collides) return parts.slice(-depth);
   }
-  return labelText(cwd);
+  return parts;
+}
+
+function directoryLabel(cwd: string, directories: readonly string[]): string {
+  const parts = distinguishingDirectorySegments(cwd, directories);
+  return parts.length ? labelText(parts.join("/")) : "/";
+}
+
+/** Pure directory tokenization shared by previews, initial titles, and reconciliation. */
+export function tokenizeTelegramDirectorySegment(segment: string): string[] {
+  return segment
+    .replace(/([\p{Ll}\p{N}])(\p{Lu})/gu, "$1\u0000$2")
+    .replace(/(\p{Lu})(\p{Lu}\p{Ll})/gu, "$1\u0000$2")
+    .split(/[^\p{L}\p{N}]+|\u0000/gu)
+    .filter(Boolean);
+}
+
+function formatDirectoryLabel(
+  cwd: string,
+  directories: readonly string[],
+  mode: "directory-snake" | "directory-title",
+): string {
+  const segments = distinguishingDirectorySegments(cwd, directories);
+  if (!segments.length) return "/";
+  const formatted = segments.map((segment) => {
+    const tokens = tokenizeTelegramDirectorySegment(segment);
+    if (!tokens.length) return "";
+    if (mode === "directory-snake") return tokens.map((token) => token.toLowerCase()).join("_");
+    return tokens.map((token) => /\p{L}/u.test(token) && token === token.toUpperCase()
+      ? token
+      : `${token.slice(0, 1).toUpperCase()}${token.slice(1).toLowerCase()}`).join(" ");
+  }).filter(Boolean);
+  if (!formatted.length) return directoryLabel(cwd, directories);
+  return formatted.join(mode === "directory-snake" ? "_" : " / ");
+}
+
+/** Maps one leader-captured authenticated owner roster onto retained binding identities. */
+export function resolveTelegramLiveWorkspaceBindingKeys(
+  bindings: readonly TelegramWorkspaceThreadBinding[],
+  leaderTarget: { chatId: number; threadId?: number } | undefined,
+  followers: readonly { target?: { chatId: number; threadId?: number } }[],
+): ReadonlySet<string> {
+  const targets = new Set<string>();
+  const add = (target: { chatId: number; threadId?: number } | undefined) => {
+    if (target && typeof target.threadId === "number") {
+      targets.add(`${target.chatId}:${target.threadId}`);
+    }
+  };
+  add(leaderTarget);
+  for (const follower of followers) add(follower.target);
+  return new Set(bindings.filter((binding) =>
+    targets.has(`${binding.target.chatId}:${binding.target.threadId}`),
+  ).map((binding) => binding.bindingKey));
 }
 
 /** Missing or ambiguous metadata yields no label rather than inventing identity. */
 export function resolveTelegramWorkspaceDisplayNames(
   bindings: readonly TelegramWorkspaceDisplayBinding[],
   mode: TelegramThreadDisplayMode,
+  liveBindingKeys: ReadonlySet<string> = new Set(),
 ): Map<string, string> {
   const labels = new Map<string, string>();
   const directories = Array.from(new Set(bindings.map((binding) => binding.cwd)));
-  const directoryCounts = new Map<string, number>();
+  const liveDirectoryCounts = new Map<string, number>();
   for (const binding of bindings) {
-    directoryCounts.set(binding.cwd, (directoryCounts.get(binding.cwd) ?? 0) + 1);
+    if (!liveBindingKeys.has(binding.bindingKey)) continue;
+    liveDirectoryCounts.set(binding.cwd, (liveDirectoryCounts.get(binding.cwd) ?? 0) + 1);
   }
   const bases = new Map<string, string>();
   for (const binding of bindings) {
@@ -65,18 +123,27 @@ export function resolveTelegramWorkspaceDisplayNames(
       const name = binding.threadName ? labelText(binding.threadName) : slot;
       if (name) labels.set(binding.bindingKey, boundedLabel(name));
     } else {
-      const base = directoryLabel(binding.cwd, directories);
+      const base = mode === "directories"
+        ? directoryLabel(binding.cwd, directories)
+        : formatDirectoryLabel(binding.cwd, directories, mode);
       bases.set(binding.bindingKey, base);
-      const showSuffix = binding.showSlotSuffix ||
-        (directoryCounts.get(binding.cwd) ?? 0) > 1;
+      const showSuffix = mode === "directories"
+        ? binding.showSlotSuffix || bindings.filter((candidate) => candidate.cwd === binding.cwd).length > 1
+        : (liveDirectoryCounts.get(binding.cwd) ?? 0) > 1;
       if (showSuffix && !slot) continue;
-      labels.set(binding.bindingKey, boundedLabel(base, showSuffix ? `_${slot!.toLowerCase()}` : ""));
+      const suffix = !showSuffix ? "" : mode === "directory-title"
+        ? ` · ${slot}`
+        : `_${slot!.toLowerCase()}`;
+      labels.set(binding.bindingKey, boundedLabel(base, suffix));
     }
   }
   // Long or whitespace-normalized paths can collide even after qualification.
-  if (mode === "directories") {
+  if (mode === "directories" || mode === "directory-snake" || mode === "directory-title") {
     const counts = new Map<string, number>();
-    for (const label of labels.values()) {
+    for (const binding of bindings) {
+      const label = labels.get(binding.bindingKey);
+      if (!label || ((mode === "directory-snake" || mode === "directory-title") &&
+          !liveBindingKeys.has(binding.bindingKey))) continue;
       const key = label.toLowerCase();
       counts.set(key, (counts.get(key) ?? 0) + 1);
     }
@@ -84,16 +151,24 @@ export function resolveTelegramWorkspaceDisplayNames(
       const label = labels.get(binding.bindingKey);
       if (binding.manualThreadName || !label ||
           (counts.get(label.toLowerCase()) ?? 0) < 2) continue;
+      if ((mode === "directory-snake" || mode === "directory-title") &&
+          (liveDirectoryCounts.get(binding.cwd) ?? 0) < 2) {
+        labels.delete(binding.bindingKey);
+        continue;
+      }
       if (!binding.slot || !/^[A-Z]$/u.test(binding.slot)) {
         labels.delete(binding.bindingKey);
         continue;
       }
       labels.set(binding.bindingKey, boundedLabel(bases.get(binding.bindingKey)!,
-        `_${binding.slot.toLowerCase()}`));
+        mode === "directory-title" ? ` · ${binding.slot}` : `_${binding.slot.toLowerCase()}`));
     }
   }
   const counts = new Map<string, number>();
-  for (const label of labels.values()) {
+  for (const binding of bindings) {
+    const label = labels.get(binding.bindingKey);
+    if (!label || ((mode === "directory-snake" || mode === "directory-title") &&
+        !liveBindingKeys.has(binding.bindingKey))) continue;
     const key = label.toLowerCase();
     counts.set(key, (counts.get(key) ?? 0) + 1);
   }
@@ -108,6 +183,7 @@ export function resolveTelegramInitialWorkspaceDisplayName(input: {
   binding: TelegramWorkspaceDisplayBinding;
   mode: TelegramThreadDisplayMode;
   preserveRetainedManualName?: boolean;
+  liveBindingKeys?: ReadonlySet<string>;
 }): string | undefined {
   const retained = input.bindings.find((binding) =>
     binding.bindingKey === input.binding.bindingKey,
@@ -126,7 +202,7 @@ export function resolveTelegramInitialWorkspaceDisplayName(input: {
       candidate.bindingKey !== binding.bindingKey,
     ),
     binding,
-  ], input.mode).get(binding.bindingKey);
+  ], input.mode, new Set([...(input.liveBindingKeys ?? []), binding.bindingKey])).get(binding.bindingKey);
 }
 
 export async function applyTelegramThreadDisplaySetting(
@@ -155,6 +231,7 @@ export interface TelegramThreadDisplayReconcilerDeps {
   getProfileKey(): string;
   getLeaderEpoch(): string | number | undefined;
   captureBindingAuthority(binding: TelegramWorkspaceThreadBinding): (() => boolean) | undefined;
+  captureLiveBindingKeys(bindings: readonly TelegramWorkspaceThreadBinding[]): ReadonlySet<string>;
   callApi<TResponse>(
     method: string,
     body: Record<string, unknown>,
@@ -179,7 +256,8 @@ export function createTelegramThreadDisplayReconciler(
     };
     assertAuthority();
     const bindings = deps.store.listWorkspaceBindings();
-    const titles = resolveTelegramWorkspaceDisplayNames(bindings, mode);
+    const liveBindingKeys = deps.captureLiveBindingKeys(bindings);
+    const titles = resolveTelegramWorkspaceDisplayNames(bindings, mode, liveBindingKeys);
     let changed = 0;
     for (const binding of bindings) {
       const isBindingCurrent = deps.captureBindingAuthority(binding);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@llblab/pi-telegram",
-  "version": "0.46.0",
+  "version": "0.47.0",
   "private": false,
   "publishConfig": {
     "access": "public"
@@ -30,15 +30,19 @@
     "test": "node --experimental-strip-types --test --test-concurrency=4 --test-reporter=dot tests/*.test.ts",
     "test:verbose": "node --experimental-strip-types --test --test-reporter=spec tests/*.test.ts",
     "typecheck": "tsc --noEmit",
+    "build": "node scripts/build-dist.mjs",
+    "prepack": "npm run build",
+    "check": "node -e \"await import('./dist/pi-telegram/index.js'); console.log('pi-telegram: extension import ok')\"",
     "audit": "npm audit --omit=peer",
     "audit:host": "npm audit",
     "pack:check": "npm pack --dry-run",
-    "validate": "npm run typecheck && npm test && npm run audit && npm run pack:check"
+    "validate": "npm run build && npm run typecheck && npm test && npm run audit && npm run check && npm run pack:check"
   },
   "files": [
     "index.ts",
     "api/",
     "lib/",
+    "dist/",
     "README.md",
     "AGENTS.md",
     "BACKLOG.md",
@@ -49,23 +53,62 @@
     "screenshot.png"
   ],
   "exports": {
-    ".": "./index.ts",
-    "./inbound": "./api/inbound.ts",
-    "./outbound": "./api/outbound.ts",
-    "./delivery": "./api/delivery.ts",
-    "./activity": "./api/activity.ts",
-    "./updates": "./api/updates.ts",
-    "./commands": "./api/commands.ts",
-    "./sections": "./api/sections.ts",
-    "./status": "./api/status.ts",
-    "./voice": "./api/voice.ts",
-    "./keyboard": "./api/keyboard.ts"
+    ".": {
+      "types": "./dist/index.d.ts",
+      "default": "./dist/index.js"
+    },
+    "./inbound": {
+      "types": "./dist/api/inbound.d.ts",
+      "default": "./dist/api/inbound.js"
+    },
+    "./outbound": {
+      "types": "./dist/api/outbound.d.ts",
+      "default": "./dist/api/outbound.js"
+    },
+    "./delivery": {
+      "types": "./dist/api/delivery.d.ts",
+      "default": "./dist/api/delivery.js"
+    },
+    "./activity": {
+      "types": "./dist/api/activity.d.ts",
+      "default": "./dist/api/activity.js"
+    },
+    "./updates": {
+      "types": "./dist/api/updates.d.ts",
+      "default": "./dist/api/updates.js"
+    },
+    "./commands": {
+      "types": "./dist/api/commands.d.ts",
+      "default": "./dist/api/commands.js"
+    },
+    "./sections": {
+      "types": "./dist/api/sections.d.ts",
+      "default": "./dist/api/sections.js"
+    },
+    "./status": {
+      "types": "./dist/api/status.d.ts",
+      "default": "./dist/api/status.js"
+    },
+    "./voice": {
+      "types": "./dist/api/voice.d.ts",
+      "default": "./dist/api/voice.js"
+    },
+    "./keyboard": {
+      "types": "./dist/api/keyboard.d.ts",
+      "default": "./dist/api/keyboard.js"
+    }
   },
   "pi": {
     "extensions": [
+      "./dist/pi-telegram/index.js"
+    ],
+    "sourceExtensions": [
       "./index.ts"
     ],
     "skills": [
+      "./dist/skills"
+    ],
+    "sourceSkills": [
       "./skills"
     ],
     "image": "https://raw.githubusercontent.com/llblab/pi-telegram/main/screenshot.png"

package/scripts/build-dist.mjs

@@ -0,0 +1,44 @@
+#!/usr/bin/env node
+
+/**
+ * Builds the distributive JavaScript, declarations, Pi entrypoint, and Skills.
+ * Usage: node scripts/build-dist.mjs
+ *
+ * The build replaces `dist`, compiles with `tsconfig.build.json`, then copies
+ * package-owned runtime assets. It does not publish, pack, or modify sources.
+ */
+
+import { spawnSync } from "node:child_process";
+import { cpSync, mkdirSync, rmSync, writeFileSync } from "node:fs";
+import { join } from "node:path";
+
+function run(command, args) {
+  const result = spawnSync(command, args, { stdio: "inherit" });
+  if (result.error) throw result.error;
+  if (result.status !== 0) process.exit(result.status ?? 1);
+}
+
+rmSync("dist", { recursive: true, force: true });
+mkdirSync("dist", { recursive: true });
+
+run(process.execPath, [
+  join("node_modules", "typescript", "bin", "tsc"),
+  "-p",
+  "tsconfig.build.json",
+]);
+
+mkdirSync(join("dist", "pi-telegram"), { recursive: true });
+writeFileSync(
+  join("dist", "pi-telegram", "index.js"),
+  'export { default } from "../index.js";\n',
+  "utf8",
+);
+
+cpSync("skills", join("dist", "skills"), { recursive: true });
+cpSync("package.json", join("dist", "package.json"));
+cpSync(
+  join("lib", "generative-app-worker.mjs"),
+  join("dist", "lib", "generative-app-worker.mjs"),
+);
+
+run(process.execPath, ["--check", join("dist", "pi-telegram", "index.js")]);

package/scripts/measure-bus.mjs

@@ -1,4 +1,11 @@
-// Counts isolated registry work without sockets, configured profiles, or Telegram calls.
+/**
+ * Counts isolated synchronous bus-registry work at representative sizes.
+ * Usage: node --experimental-strip-types scripts/measure-bus.mjs
+ *
+ * This assertion-backed fixture instruments Map operations only. It does not
+ * open sockets, read configured profiles, call Telegram, or measure latency.
+ */
+
 import assert from "node:assert/strict";
 import { createTelegramBusFollowerRegistry, createTelegramBusProtocolIdentity } from "../lib/bus.ts";
 

package/scripts/measure-workspace.mjs

@@ -1,4 +1,11 @@
-// Measures isolated Thread-store work; never reads configured profiles or calls Telegram.
+/**
+ * Measures isolated Thread-store filesystem work at representative sizes.
+ * Usage: node --experimental-strip-types scripts/measure-workspace.mjs
+ *
+ * This assertion-backed fixture uses and removes its own temporary directory.
+ * It never reads configured profiles, calls Telegram, or measures latency.
+ */
+
 import assert from "node:assert/strict";
 import fs from "node:fs/promises";
 import { syncBuiltinESMExports } from "node:module";

package/skills/generative-apps/SKILL.md

@@ -97,7 +97,7 @@ ordinary prompt          → model-mediated interpretation or explanation
 - Keep credentials and unrelated private state out of source, state, output, and diagnostics.
 - Route consequences requiring contextual judgment through an ordinary model prompt.
 - Fail closed on unavailable owners, stale actions, malformed state, absent methods, process failures, or uncertain effects.
-- Do not claim automatic refresh, removal, voice output, or other behavior still marked incomplete in the runtime document.
+- Treat `refreshAfterMs` validation, output-only `refresh`, scheduling after a successful hinted explicit `telegram_bind` delivery, same-handle rescheduling after a current-surface bound action, Delivery edit-failure classification, known unavailable-message invalidation with bounded classified diagnostics, session/runtime live-surface cancellation, and exact direct classic/leader/authenticated-follower target retention as implemented. The Bot API has no ordinary private-message deletion update; `deleted_business_messages` is limited to connected business accounts. Do not claim removal, voice output, proactive deletion-update invalidation, or other behavior still marked incomplete in the runtime document.
 
 ## Validation
 

package/skills/telegram-bridge/SKILL.md

@@ -60,7 +60,7 @@ Every enabled button has a self-contained prompt and an optional selection style
 - `{label|prompt}` separates visible label from queued prompt.
 - `{label|prompt|selected_style}` and `{|prompt|selected_style}` accept `primary`, `success`, or `danger`.
 - Fourth-position `1`/`true` disables, `0`/`false` enables; omission means enabled. JSON uses boolean `disabled`. `{|Next||1}` omits label/style; `{Next|||1}` omits prompt/style; `{|||1}` is blank (Telegram receives a non-breaking space). Prefer meaningful labels and retain a useful enabled action. Enabled CML requires a prompt. Disabled controls stay visible but have no callback, queued prompt, or bound-method invocation.
-- Top-level cells form vertical rows; one nested row groups horizontal peers.
+- Top-level cells form vertical rows; inside a fenced block, adjacent top-level JSON/CML objects may omit the outer array and commas. One nested row groups horizontal peers.
 - Prefer one matrix per related group. Fenced blocks stay in place in Rich mode; HTML compatibility moves them to the footer. Native rows allow eight buttons. Malformed/oversized/unclosed blocks activate nothing; drafts hide them. Outer code fences and quoted/indented examples remain literal.
 - Both placements share prompt/app routing. In-body clicks acknowledge without recoloring; selected-style highlighting remains footer-only.