toolroll 0.9.16 → 0.9.17

package/dist/telegram-mate.js

@@ -23,11 +23,13 @@ export { channelRepos as telegramConversationRepos, resolveChannelMate as resolv
  * the console and CLI state), and the door re-proves everything again.
  */
 import { createHash, randomBytes } from "node:crypto";
-import { MATE_MESSAGE_MAX_CHARS, runMateTurn } from "./mate.js";
+import { MATE_MESSAGE_MAX_CHARS, mateFailureText, runMateTurn } from "./mate.js";
 import { confirmMateProposal, dismissMateProposal } from "./mate-doors.js";
 import { ceilingDigestOf, verifyApproverStanding } from "./principal.js";
 import { MATE_ASK_OTHER } from "./store.js";
 import { phoneText } from "./telegram-status.js";
+import { shapeReplyParts, telegramReply } from "./reply-shape.js";
+import { WARM_EMOJI, warmTurn } from "./chat-warmth.js";
 import { TeamLeads } from "./team-leads.js";
 import { chatResultHref } from "./chat-controls.js";
 import { resultImageFileName, safeResultImageCaption, verifyResultImage } from "./chat-evidence.js";
@@ -40,7 +42,7 @@ export const CONVERSATION_MAX_AGE_MS = 10 * 60_000;
 export const CARD_TTL_MS = 24 * 3_600_000;
 export const CHALLENGE_TTL_MS = 10 * 60_000;
 /** Telegram's own message ceiling, with room for our part headers (the same bound telegram.ts splits at). */
-const PART_CAP = 3_900;
+export const PART_CAP = 3_900;
 // ---- the ceiling and the principal --------------------------------------------
 /**
  * The phone's project ceiling: the enrolled registry, canonicalized and
@@ -405,7 +407,7 @@ async function runTelegramConversation(row, args) {
         const bound = store.getTelegramConversation(row.id)?.session ?? null;
         const session = bound === null ? null : store.getMateSession(bound);
         if (session === null) {
-            await notify(couldNotAnswerText("the chat session for this reply is gone"));
+            await notify("This conversation was restarted before my reply went out, so it wasn't sent. Nothing was changed. Send your message again.");
             finish({ state: "failed", outcome: "unsent:no-session" });
             report.refused++;
             return;
@@ -525,11 +527,14 @@ async function runTelegramConversation(row, args) {
                 const origin = proposal === null ? null : options.phoneOrigin?.() ?? null;
                 const wanted = proposal === null ? null : proposalLink(store, proposal, repos);
                 const keyboard = keyboardWith(part.keyboard, phoneLinkButton(origin, wanted));
-                const text = proposal !== null && part.keyboard === null ? handoffCardText(part.text, linkNote(origin, wanted)) : part.text;
+                // The lead's own words carry their bold anchors and labelled links as entities, never as Markdown characters.
+                const voiced = part.kind === "reply" ? telegramReply(part.text) : null;
+                const text = proposal !== null && part.keyboard === null ? handoffCardText(part.text, linkNote(origin, wanted)) : voiced?.text ?? part.text;
                 method = "sendMessage";
                 send = () => transport("sendMessage", {
                     chat_id: chatId,
                     text,
+                    ...(voiced === null || voiced.entities.length === 0 ? {} : { entities: voiced.entities }),
                     link_preview_options: { is_disabled: true },
                     ...(part.replyTo === null ? {} : { reply_parameters: { message_id: Number(part.replyTo) } }),
                     ...(keyboard === undefined ? {} : { reply_markup: { inline_keyboard: keyboard } }),
@@ -590,7 +595,7 @@ async function runTelegramConversation(row, args) {
         try {
             return store.transact(() => {
                 const now = clock();
-                const parts = splitParts(reply).map((text, index) => ({ kind: "reply", text, replyTo: index === 0 ? row.messageId : null }));
+                const parts = shapeReplyParts(reply, PART_CAP, { asked: row.text, appOrigin: options.phoneOrigin?.() ?? null }, shaped => telegramReply(shaped).text.length).map((text, index) => ({ kind: "reply", text, replyTo: index === 0 ? row.messageId : null }));
                 // The lead's question to its owner: one tap per option, then "Something else".
                 const ask = store.mateAsk(turn);
                 if (ask !== null)
@@ -633,7 +638,7 @@ async function runTelegramConversation(row, args) {
             return;
         }
         if (turn.state !== "answered") {
-            await notify(couldNotAnswerText(`the assistant's reply did not complete (${phoneText(turn.failureReason ?? "unknown", 40)})`));
+            await notify(mateFailureText(turn.failureReason));
             finish({ state: "failed", outcome: `replayed:${turn.failureReason ?? "failed"}` });
             report.refused++;
             return;
@@ -693,7 +698,13 @@ async function runTelegramConversation(row, args) {
     const { who, session, thread, config } = resolved;
     const revalidate = async () => {
         const problem = await telegramChannelProblem(store, { botId, bindingId: binding.id, approverGeneration: binding.approverGeneration, repos: who.repos }, readProjects);
-        return problem === null ? { ok: true } : { ok: false, reason: problem };
+        if (problem === null)
+            return { ok: true };
+        const reason = problem === UNREADABLE_REGISTRY ? "projects-unreadable"
+            : problem === "this chat is no longer paired" ? "unpaired"
+                : problem === "the paired account is no longer an approver" ? "not-approver"
+                    : problem === "the connected projects changed" ? "projects-changed" : "access-changed";
+        return { ok: false, reason };
     };
     // Bound BEFORE the dispatch, under the claim: a crash from here on finds
     // the receipt in this session, not in whichever session is live later.
@@ -704,17 +715,23 @@ async function runTelegramConversation(row, args) {
     // crash gets the original turn back, never a second dispatch.
     const heartbeat = setInterval(() => { held(); }, 30_000);
     heartbeat.unref?.();
+    // Once the turn reaches a tool step: a 👍 on the owner's message, and "typing…" while the lead works.
+    const warm = warmTurn({
+        react: () => transport("setMessageReaction", { chat_id: chatId, message_id: Number(row.messageId), reaction: [{ type: "emoji", emoji: WARM_EMOJI }] }),
+        typing: () => transport("sendChatAction", { chat_id: chatId, action: "typing" }),
+    });
     let outcome;
     try {
         outcome = await runMateTurn({
             store, who, session, thread, config, key: null, message: row.text, requestId: row.request,
             ...(row.context === null ? {} : { context: row.context }),
             ...(options.subscriptionRunner === undefined ? {} : { subscriptionRunner: options.subscriptionRunner }),
-            clock, evidenceRoot: options.evidenceRoot, revalidate, mediaDelivery: "documents", channel: "telegram",
+            clock, evidenceRoot: options.evidenceRoot, revalidate, mediaDelivery: "documents", channel: "telegram", onProgress: warm.onProgress,
         });
     }
     finally {
         clearInterval(heartbeat);
+        warm.stop();
     }
     const receipt = store.mateRequestReceipt(session.id, row.request);
     if (receipt !== null)
@@ -724,7 +741,7 @@ async function runTelegramConversation(row, args) {
             await requeue("busy");
             return;
         }
-        await notify(phoneText(couldNotAnswerText(outcome.message), 1_000));
+        await notify(phoneText(outcome.message, 1_000));
         finish({ state: "failed", outcome: `refused:${outcome.refused}` });
         report.refused++;
         return;
@@ -732,8 +749,7 @@ async function runTelegramConversation(row, args) {
     if (!outcome.ok) {
         // Truthful: what failed, that nothing was kept, and that a new message
         // is a new turn. A failed turn's drafts are already gone.
-        const words = outcome.failed === "revoked" ? `${outcome.message.charAt(0).toUpperCase()}${outcome.message.slice(1)}.` : couldNotAnswerText(`the assistant's reply did not complete (${outcome.message})`);
-        await notify(phoneText(words, 1_000));
+        await notify(phoneText(outcome.message, 1_000));
         finish({ state: "failed", outcome: `failed:${outcome.failed}` });
         report.refused++;
         return;

package/dist/telegram-team.js

@@ -1,6 +1,7 @@
+import { renderReply, shapeReplyParts } from "./reply-shape.js";
 import { TeamLeads } from "./team-leads.js";
 import { proposalPreview, tooLongText } from "./chat-channel.js";
-import { mintCardTokens, phoneLinkButton, splitParts } from "./telegram-mate.js";
+import { PART_CAP, mintCardTokens, phoneLinkButton, splitParts } from "./telegram-mate.js";
 import { MATE_MESSAGE_MAX_CHARS } from "./mate.js";
 import { phoneText } from "./telegram-status.js";
 /** `/team` lists, `/team <n>` chooses, `/team off` (or `private`) leaves. Any other slash text is not ours. */
@@ -265,7 +266,9 @@ export async function deliverTeamChats(store, botId, transport, clock, report, p
             const skip = role === "operator" && (ownMessage || status === "cancelled" || String(message["author"] ?? "").length === 0);
             const prefix = `telegram-team:${chat.id}:`;
             if (!skip) {
-                const text = role === "assistant" ? String(message["text"]) : `${phoneText(String(message["author"]), 40)}: ${String(message["text"])}`;
+                // The lead's reply is split as written, then each part shaped and measured as Telegram shows it, so no cut lands
+                // inside a link and no rendered part passes Telegram's ceiling.
+                const text = role === "assistant" ? shapeReplyParts(String(message["text"]), PART_CAP, { appOrigin: phoneOrigin?.() ?? null }, shaped => renderReply(shaped, "telegram").length).map(part => renderReply(part, "telegram")) : `${phoneText(String(message["author"]), 40)}: ${String(message["text"])}`;
                 const sent = await sendParts(store, transport, chat.chatId, text, `${prefix}message:${id}`, clock, access, report);
                 if (sent !== null) {
                     report.problems.push(`team chat ${chat.chatId}: ${sent}`);
@@ -329,7 +332,7 @@ function cardBindingFor(store, botId, chat, turn) {
     return bindings.find(one => members.has(one.approver)) ?? null;
 }
 async function sendParts(store, transport, chatId, text, receipt, clock, access, report, keyboard, tokens = []) {
-    const parts = splitParts(text);
+    const parts = typeof text === "string" ? splitParts(text) : text;
     const delivered = store.serviceCursor(receipt);
     for (const [index, part] of parts.entries()) {
         if (index < delivered)

package/dist/telegram.js

@@ -920,7 +920,7 @@ async function deliverOutboxTo(store, botId, binding, transport, owner, clock, r
         };
         /** The lead's words (lead-voice.ts): one message; a later say within two minutes edits it in place. */
         const leadSayOutcome = async (fact) => {
-            const words = leadSayText(store, fact);
+            const words = leadSayText(store, fact, binding.approver);
             const button = fact.link === null ? null : factButton(phoneOrigin, fact.link);
             const keyboard = button === null ? [] : [button];
             const earlier = leadSayEarlier(store, fact).map(id => store.telegramMessageOf(id, fact.destination)).find(one => one !== null) ?? null;

package/dist/test-temp.js

@@ -18,7 +18,7 @@ export const TEST_TEMP_PREFIXES = [
     "flow-goal-", "flow-scratch", "flows-chat-", "invoke-attest-", "knowledge-test-", "learning-", "peek-", "phone-origin-",
     "proposal-repo-", "race-", "refresh-migration-", "refresh-refusal-", "release-check-", "shared-chat-", "skills-test-",
     "task-status-surfaces-", "template-repo-", "template-test-", "v14-migrate-", "v17-migrate-", "watch-anchor-",
-    "needs-you-", "deps-lock-", "deps-state-", "upgrade-fresh-", "upgrade-path-", "deploy-restore-", "result-review-", "one-result-page-", "deploy-before-swap-", "one-vocabulary-", "accept-finishes-", "lead-promises-",
+    "needs-you-", "deps-lock-", "deps-state-", "upgrade-fresh-", "upgrade-path-", "deploy-restore-", "result-review-", "one-result-page-", "deploy-before-swap-", "one-vocabulary-", "accept-finishes-", "lead-promises-", "lead-people-",
     // Playwright's browser profiles and artifacts.
     "playwright_chromiumdev_profile-", "playwright_firefoxdev_profile-", "playwright-artifacts-",
 ];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "toolroll",
-  "version": "0.9.16",
+  "version": "0.9.17",
   "description": "A control plane for unattended coding agents — queue tasks, walk away, come back to pull requests. Agents build in leased worktrees, never touch your default branch, and interrupt you only for decisions that need a human.",
   "directories": {
     "doc": "docs"