@llblab/pi-telegram 0.23.0 → 0.23.2

package/lib/thread-reconciler.ts

@@ -62,11 +62,6 @@ export interface TelegramReservedThreadMessageObservation {
   leaderEpoch?: number | string;
 }
 
-export interface ThreadReservationProbeResult {
-  target: ThreadTarget;
-  stale: boolean;
-}
-
 export interface ReplacedInstanceBindingInput {
   instanceId: string;
   replacementTarget: ThreadTarget;
@@ -111,14 +106,6 @@ export type ThreadReconciliationAction =
       instanceId?: string;
       leaderEpoch?: number | string;
     }
-  | {
-      kind: "close-delete-pruned-follower-topic";
-      target: TelegramTarget & { threadId: number };
-      reason: "pruned-follower";
-      instanceId?: string;
-      messageId?: number;
-      leaderEpoch?: number | string;
-    }
   | {
       kind: "close-delete-replaced-follower-topic";
       target: TelegramTarget & { threadId: number };
@@ -150,11 +137,6 @@ export type ThreadReconciliationAction =
       pendingProvisionId: string;
       instanceId?: string;
       leaderEpoch?: number | string;
-    }
-  | {
-      kind: "remove-reservation";
-      target: TelegramTarget & { threadId: number };
-      reason: "reservation-probe-stale";
     };
 
 export type ThreadReconciliationPhase =
@@ -203,7 +185,6 @@ export interface ThreadReconciliationApplyPorts {
     lastSyncError?: string,
   ) => boolean;
   persist?: () => Promise<void>;
-  removeReservationByTarget?: (target: ThreadTarget) => boolean;
   removePendingProvisionById?: (id: string) => boolean;
   getCurrentLeaderEpoch?: () => number | string | undefined;
   recordRuntimeEvent?: (
@@ -223,8 +204,6 @@ export interface ThreadReconciliationInput {
   unboundMessages?: readonly TelegramUnboundThreadMessageObservation[];
   reservedMessages?: readonly TelegramReservedThreadMessageObservation[];
   proactiveReservationCleanup?: boolean;
-  reservationProbeResults?: readonly ThreadReservationProbeResult[];
-  prunedFollowerInstanceIds?: readonly string[];
   replacedBindings?: readonly ReplacedInstanceBindingInput[];
   previousLeaderCleanup?: PreviousLeaderCleanupInput;
   previousState?: ThreadReconciliationMachineState;
@@ -309,7 +288,6 @@ function isCleanupAction(action: ThreadReconciliationAction): boolean {
     action.kind === "close-delete-unbound-topic" ||
     action.kind === "close-delete-reserved-topic" ||
     action.kind === "close-stale-replaced-topic" ||
-    action.kind === "close-delete-pruned-follower-topic" ||
     action.kind === "close-delete-replaced-follower-topic" ||
     action.kind === "close-delete-previous-leader-topic" ||
     action.kind === "close-delete-disconnected-instance-topic" ||
@@ -318,11 +296,7 @@ function isCleanupAction(action: ThreadReconciliationAction): boolean {
 }
 
 function isSyncAction(action: ThreadReconciliationAction): boolean {
-  return (
-    action.kind === "mark-topic-active" ||
-    action.kind === "mark-topic-stale" ||
-    action.kind === "remove-reservation"
-  );
+  return action.kind === "mark-topic-active" || action.kind === "mark-topic-stale";
 }
 
 function createThreadReconciliationMachineState(
@@ -520,11 +494,6 @@ export async function applyThreadReconciliationPlan(
   const persistFences: ThreadReconciliationAction[] = [];
   const incompleteActions: ThreadReconciliationAction[] = [];
   for (const action of plan.actions) {
-    if (action.kind === "remove-reservation") {
-      shouldPersist =
-        ports.removeReservationByTarget?.(action.target) || shouldPersist;
-      continue;
-    }
     if (action.kind === "mark-topic-active") {
       shouldPersist =
         ports.markActiveByTarget?.(action.target) || shouldPersist;
@@ -586,7 +555,6 @@ export async function applyThreadReconciliationPlan(
     if (
       action.kind === "close-delete-unbound-topic" ||
       action.kind === "close-delete-reserved-topic" ||
-      action.kind === "close-delete-pruned-follower-topic" ||
       action.kind === "close-delete-replaced-follower-topic" ||
       action.kind === "close-delete-previous-leader-topic" ||
       action.kind === "close-delete-disconnected-instance-topic" ||
@@ -658,7 +626,6 @@ export async function applyThreadReconciliationPlan(
       if (shouldSkipForStaleLeaderEpoch(action, ports)) continue;
       if (
         action.kind !== "close-delete-previous-leader-topic" &&
-        action.kind !== "close-delete-pruned-follower-topic" &&
         action.kind !== "close-delete-replaced-follower-topic" &&
         action.kind !== "close-delete-disconnected-instance-topic" &&
         action.kind !== "close-delete-expired-pending-provision-topic"
@@ -674,9 +641,7 @@ export async function applyThreadReconciliationPlan(
           ? "Unbound Telegram topic deleted"
           : action.kind === "close-delete-reserved-topic"
             ? "Reserved Telegram topic deleted"
-            : action.kind === "close-delete-pruned-follower-topic"
-              ? "Pruned follower Telegram topic deleted"
-              : action.kind === "close-delete-replaced-follower-topic"
+            : action.kind === "close-delete-replaced-follower-topic"
                 ? "Replaced follower Telegram topic deleted"
                 : action.kind === "close-delete-previous-leader-topic"
                   ? "Previous leader Telegram topic deleted"
@@ -689,9 +654,7 @@ export async function applyThreadReconciliationPlan(
               ? "thread-reconciler-unbound-topic-delete"
               : action.kind === "close-delete-reserved-topic"
                 ? "thread-reconciler-reserved-topic-delete"
-                : action.kind === "close-delete-pruned-follower-topic"
-                  ? "thread-reconciler-pruned-follower-topic-delete"
-                  : action.kind === "close-delete-replaced-follower-topic"
+                : action.kind === "close-delete-replaced-follower-topic"
                     ? "thread-reconciler-replaced-follower-topic-delete"
                     : action.kind === "close-delete-previous-leader-topic"
                       ? "thread-reconciler-previous-leader-topic-delete"
@@ -746,11 +709,6 @@ export function planThreadReconciliation(
       .filter(isCurrentRecord)
       .map((record) => targetKey(record.target)),
   );
-  const allReservationTargets = new Set(
-    (input.reservations ?? []).map((reservation) =>
-      targetKey(reservation.target),
-    ),
-  );
   const reservedTargets = new Set(
     (input.reservations ?? [])
       .filter((reservation) => isReservationAlive(reservation, input.nowMs))
@@ -826,22 +784,6 @@ export function planThreadReconciliation(
     }
   }
 
-  for (const prunedInstanceId of input.prunedFollowerInstanceIds ?? []) {
-    for (const record of input.records) {
-      if (record.instanceId !== prunedInstanceId) continue;
-      if (!isActiveOrStartingRecord(record)) continue;
-      actions.push({
-        kind: "close-delete-pruned-follower-topic",
-        target: record.target,
-        reason: "pruned-follower",
-        instanceId: prunedInstanceId,
-        ...(input.currentLeaderEpoch !== undefined
-          ? { leaderEpoch: input.currentLeaderEpoch }
-          : {}),
-      });
-    }
-  }
-
   for (const replacement of input.replacedBindings ?? []) {
     for (const record of input.records) {
       if (record.instanceId !== replacement.instanceId) continue;
@@ -900,16 +842,6 @@ export function planThreadReconciliation(
     }
   }
 
-  for (const probe of input.reservationProbeResults ?? []) {
-    if (!probe.stale) continue;
-    if (!allReservationTargets.has(targetKey(probe.target))) continue;
-    actions.push({
-      kind: "remove-reservation",
-      target: probe.target,
-      reason: "reservation-probe-stale",
-    });
-  }
-
   for (const message of input.reservedMessages ?? []) {
     const key = targetKey(message.target);
     if (!reservedTargets.has(key)) continue;

package/lib/threads.ts

@@ -237,7 +237,6 @@ export interface TelegramTopicTargetStore {
     target: TelegramTarget & { threadId: number },
   ) => Promise<boolean>;
   removePendingProvision: (id: string) => boolean;
-  removeReservationByTarget: (target: TelegramTarget) => boolean;
   getBotState: () => TelegramBotStateSnapshot;
   setBotState: (state: Partial<TelegramBotStateSnapshot>) => void;
   setStatusSnapshot: (snapshot: {
@@ -1396,15 +1395,6 @@ export function createTelegramTopicTargetStore(
       if (changed) markDirty();
       return changed;
     },
-    removeReservationByTarget(target) {
-      const before = reservations.length;
-      reservations = reservations.filter(
-        (reservation) => !targetMatches(reservation.target, target),
-      );
-      const changed = reservations.length !== before;
-      if (changed) markDirty();
-      return changed;
-    },
     getBotState() {
       return Object.fromEntries(
         Object.entries(botState).filter(([, value]) => value !== undefined),
@@ -1983,9 +1973,6 @@ export async function provisionOwnBusTopic(
       syncStatus?: "closed" | "deleted",
       lastSyncError?: string,
     ) => deps.store.markStaleByTarget(target, syncStatus, lastSyncError),
-    removeReservationByTarget: (
-      target: TelegramTarget & { threadId: number },
-    ) => deps.store.removeReservationByTarget(target),
     removePendingProvisionById: (id: string) =>
       deps.store.removePendingProvision(id),
     persist: () => deps.store.persist(),
@@ -2024,36 +2011,6 @@ export async function provisionOwnBusTopic(
     durationMs: Date.now() - reservationCleanupApplyStartedAtMs,
     actions: reservationCleanupPlan.actions.length,
   });
-  const reservationProbeResults: ThreadReconciler.ThreadReservationProbeResult[] =
-    [];
-  deps.recordEvent("bus", "Bus leader reservation probes skipped", {
-    phase: "leader-topic-reservation-probe-skipped",
-    reservations: reservationsBeforeCleanup.length,
-  });
-  const reservationProbePlan = ThreadReconciler.planThreadReconciliation({
-    nowMs: Date.now(),
-    currentLeaderEpoch: deps.getCurrentLeaderEpoch?.(),
-    previousState: deps.getThreadReconciliationMachineState?.(),
-    records: deps.store.list(),
-    reservations: deps.store.listReservations(),
-    pendingProvisions: deps.store.listPendingProvisions(),
-    reservationProbeResults,
-  });
-  deps.recordThreadReconciliationPlan?.(reservationProbePlan);
-  const reservationProbeApplyStartedAtMs = Date.now();
-  await ThreadReconciler.applyThreadReconciliationPlan(
-    reservationProbePlan,
-    reservationCleanupPorts,
-  );
-  deps.recordEvent(
-    "bus",
-    "Bus leader reservation probe reconciliation applied",
-    {
-      phase: "leader-topic-reservation-probe-apply-duration",
-      durationMs: Date.now() - reservationProbeApplyStartedAtMs,
-      actions: reservationProbePlan.actions.length,
-    },
-  );
   const nowMs = Date.now();
   const currentLeaderOwner: TelegramThreadOwner = {
     kind: "leader",

package/lib/turns.ts

@@ -13,9 +13,9 @@ import {
   collectTelegramMessageIds,
   downloadTelegramMessageFiles,
   extractTelegramForwardContextText,
-  extractTelegramMessageText,
   extractTelegramMessagesPromptText,
   extractTelegramMessagesText,
+  extractTelegramMessageText,
   formatTelegramHistoryText,
   guessMediaType,
   type DownloadedTelegramMessageFile,
@@ -59,14 +59,19 @@ export interface TelegramTurnMessage {
 
 export type DownloadedTelegramTurnFile = DownloadedTelegramMessageFile;
 
-function getTelegramTurnTarget(message: TelegramTurnMessage): TelegramTurnTarget {
+function getTelegramTurnTarget(
+  message: TelegramTurnMessage,
+): TelegramTurnTarget {
   return Number.isInteger(message.message_thread_id)
     ? { chatId: message.chat.id, threadId: message.message_thread_id }
     : { chatId: message.chat.id };
 }
 
 function formatTelegramPrefixAttributeValue(value: string): string {
-  return value.replace(/[\]\n\r|]+/g, " ").replace(/\s+/g, " ").trim();
+  return value
+    .replace(/[\]\n\r|]+/g, " ")
+    .replace(/\s+/g, " ")
+    .trim();
 }
 
 export function createTelegramTurnPrefix(
@@ -142,7 +147,6 @@ function appendTelegramSourceContext(
   return text ? `${text}\n\n${sourceContext}` : sourceContext;
 }
 
-
 function buildTelegramForwardContextBlock(options: {
   context: string;
   text: string;
@@ -411,7 +415,6 @@ export interface BuildTelegramPromptTurnOptions {
   readBinaryFile: (path: string) => Promise<Uint8Array>;
   inferImageMimeType: (path: string) => string | undefined;
   voiceReplyMode?: TelegramVoiceReplyMode;
-  voiceReplyModeConfigured?: boolean;
   voicePromptContribution?: string;
 }
 
@@ -435,9 +438,11 @@ export interface TelegramPromptTurnRuntimeBuilderDeps<
   }>;
   resolveTimeLine?: (chatId: number) => string | null;
   getVoiceReplyMode?: () => TelegramVoiceReplyMode;
-  isVoiceReplyModeConfigured?: () => boolean;
   /** Returns the visible thread label for a message target, used to add thread context to the prompt prefix. */
-  getTelegramThreadLabel?: (message: { chat: { id: number }; message_thread_id?: number }) => string | undefined;
+  getTelegramThreadLabel?: (message: {
+    chat: { id: number };
+    message_thread_id?: number;
+  }) => string | undefined;
   getAllowedUserId?: () => number | undefined;
 }
 
@@ -475,9 +480,7 @@ export function createTelegramPromptTurnRuntimeBuilder<
               text: extractTelegramMessageText(message),
               message,
               fileNames: new Set(
-                collectTelegramFileInfos(message.rich_message ? [message] : []).map(
-                  (file) => file.fileName,
-                ),
+                collectTelegramFileInfos([message]).map((file) => file.fileName),
               ),
             },
           ]
@@ -567,7 +570,6 @@ export function createTelegramPromptTurnRuntimeBuilder<
       timeLine,
       inferImageMimeType: guessMediaType,
       voiceReplyMode,
-      voiceReplyModeConfigured: deps.isVoiceReplyModeConfigured?.(),
       voicePromptContribution: computeVoicePromptContribution(
         voiceReplyMode,
         files,
@@ -581,9 +583,13 @@ function getTelegramVoicePromptContext(
   voiceReplyMode: TelegramVoiceReplyMode,
   hasVoiceFile: boolean,
 ): Record<string, string> | undefined {
-  if (voiceReplyMode === "always") return { "reply mode": "always" };
-  if (!hasVoiceFile) return undefined;
-  return { "reply mode": voiceReplyMode };
+  if (
+    voiceReplyMode !== "always" &&
+    !(voiceReplyMode === "mirror" && hasVoiceFile)
+  ) {
+    return undefined;
+  }
+  return { delivery: "automatic voice" };
 }
 
 export async function buildTelegramPromptTurn(
@@ -597,8 +603,6 @@ export async function buildTelegramPromptTurn(
     (f) => f.kind === "voice" || f.kind === "audio",
   );
   const voiceReplyMode = options.voiceReplyMode ?? getTelegramVoiceReplyMode();
-  const showVoiceContext =
-    options.voiceReplyModeConfigured ?? options.voiceReplyMode !== undefined;
   const content: TelegramPromptContent[] = [
     {
       type: "text",
@@ -615,9 +619,10 @@ export async function buildTelegramPromptTurn(
         sourceContext: options.sourceContext,
         historyTurns: options.historyTurns,
         timeLine: options.timeLine,
-        voiceContext: showVoiceContext
-          ? getTelegramVoicePromptContext(voiceReplyMode, hasVoiceFile)
-          : undefined,
+        voiceContext: getTelegramVoicePromptContext(
+          voiceReplyMode,
+          hasVoiceFile,
+        ),
       }),
     },
   ];
@@ -634,8 +639,7 @@ export async function buildTelegramPromptTurn(
   }
   if (options.voicePromptContribution?.trim()) {
     const textItem = content.find((c) => c.type === "text") as
-      | { type: "text"; text: string }
-      | undefined;
+      { type: "text"; text: string } | undefined;
     if (textItem) {
       textItem.text = `${textItem.text}\n\n${options.voicePromptContribution.trim()}`;
     }

package/lib/voice.ts

@@ -38,7 +38,7 @@ function getNextAvailableProviderId<T>(
   return id;
 }
 
-export type TelegramVoiceReplyMode = "mirror" | "always" | "manual";
+export type TelegramVoiceReplyMode = "hidden" | "mirror" | "always";
 
 export type TelegramVoiceSynthesisProviderResult =
   | string
@@ -67,9 +67,7 @@ export interface TelegramVoiceSynthesisProvider {
 }
 
 export type TelegramVoiceTranscriptionProviderResult =
-  | string
-  | { text: string; language?: string }
-  | undefined;
+  string | { text: string; language?: string } | undefined;
 
 export interface TelegramVoiceTranscriptionFile {
   path: string;
@@ -217,16 +215,17 @@ export function clearTelegramVoiceTranscriptionProviders(): void {
 // --- Voice Reply Modes ---
 
 export const TELEGRAM_VOICE_REPLY_MODES = [
+  "hidden",
   "mirror",
   "always",
-  "manual",
 ] as const;
 
 /**
  * Returns the active voice reply mode for the current session.
  *
  * Pi-telegram owns reply-mode policy through telegram.json. If
- * config.voice.replyMode is missing or invalid, the safe default is manual.
+ * config.voice.replyMode is missing, invalid, or legacy `manual`, the effective
+ * mode is hidden.
  */
 export function getTelegramVoiceReplyMode(config?: {
   voice?: { replyMode?: string };
@@ -238,7 +237,7 @@ export function getTelegramVoiceReplyMode(config?: {
   ) {
     return configMode as TelegramVoiceReplyMode;
   }
-  return "manual";
+  return "hidden";
 }
 
 /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@llblab/pi-telegram",
-  "version": "0.23.0",
+  "version": "0.23.2",
   "private": false,
   "publishConfig": {
     "access": "public"
@@ -27,7 +27,8 @@
     "node": ">=22.19.0"
   },
   "scripts": {
-    "test": "node --experimental-strip-types --test tests/*.test.ts",
+    "test": "node --experimental-strip-types --test --test-reporter=dot tests/*.test.ts",
+    "test:verbose": "node --experimental-strip-types --test --test-reporter=spec tests/*.test.ts",
     "typecheck": "tsc --noEmit",
     "audit": "npm audit",
     "pack:check": "npm pack --dry-run",