opencode-router 0.11.87 → 0.11.88

package/README.md

@@ -50,13 +50,18 @@ opencode-router
 
 Telegram support is configured via identities. You can either:
 - Use env vars for a single bot: `TELEGRAM_BOT_TOKEN=...`
- - Or add multiple bots to the config file (`opencode-router.json`) using the CLI:
+  - Or add multiple bots to the config file (`opencode-router.json`) using the CLI:
 
 ```bash
 opencode-router telegram add <token> --id default
 opencode-router telegram list
 ```
 
+Important for direct sends and bindings:
+- Telegram targets must use numeric `chat_id` values.
+- `@username` values are not valid direct `peerId` targets for router sends.
+- If a user has not started a chat with the bot yet, Telegram may return `chat not found`.
+
 ## Slack (Socket Mode)
 
 Slack support uses Socket Mode and replies in threads when @mentioned in channels.

package/dist/bridge.js

@@ -8,7 +8,7 @@ import { startHealthServer } from "./health.js";
 import { buildPermissionRules, createClient } from "./opencode.js";
 import { chunkText, formatInputSummary, truncateText } from "./text.js";
 import { createSlackAdapter } from "./slack.js";
-import { createTelegramAdapter } from "./telegram.js";
+import { createTelegramAdapter, isTelegramPeerId } from "./telegram.js";
 async function startAdapterBounded(adapter, options) {
     const outcome = adapter
         .start()
@@ -50,6 +50,14 @@ const CHANNEL_LABELS = {
 const TYPING_INTERVAL_MS = 6000;
 const OPENCODE_ROUTER_AGENT_FILE_RELATIVE_PATH = ".opencode/agents/opencode-router.md";
 const OPENCODE_ROUTER_AGENT_MAX_CHARS = 16_000;
+const DEFAULT_MESSAGING_AGENT_INSTRUCTIONS = [
+    "Respond for non-technical users first.",
+    "Do not tell users to run router commands; use tools on their behalf.",
+    "Never expose raw peer IDs or Telegram chat IDs unless the user explicitly asks for debug details.",
+    "For Telegram send requests, try delivery immediately using existing bindings or direct tool calls.",
+    "If Telegram returns 'chat not found', explain that the recipient must message the bot first (for example with /start), then ask the user to retry.",
+    "Keep status updates concise and action-oriented.",
+].join("\n");
 // Model presets for quick switching
 const MODEL_PRESETS = {
     opus: { providerID: "anthropic", modelID: "claude-opus-4-5-20251101" },
@@ -76,6 +84,11 @@ function setUserModel(channel, identityId, peerId, model) {
 function adapterKey(channel, identityId) {
     return `${channel}:${identityId}`;
 }
+function invalidTelegramPeerIdError() {
+    const error = new Error("Telegram requires a numeric chat_id for direct targets. Usernames like @name cannot be used as peerId.");
+    error.status = 400;
+    return error;
+}
 function normalizeIdentityId(value) {
     const trimmed = (value ?? "").trim();
     if (!trimmed)
@@ -792,6 +805,9 @@ export async function startBridge(config, logger, reporter, deps = {}) {
                 if (!peerKey || !directory) {
                     throw new Error("peerId and directory are required");
                 }
+                if (channel === "telegram" && !isTelegramPeerId(peerKey)) {
+                    throw invalidTelegramPeerIdError();
+                }
                 const scoped = resolveScopedDirectory(directory);
                 if (!scoped.ok) {
                     const error = new Error(scoped.error);
@@ -833,6 +849,9 @@ export async function startBridge(config, logger, reporter, deps = {}) {
                 if (!directoryInput && !peerId) {
                     throw new Error("directory or peerId is required");
                 }
+                if (channel === "telegram" && peerId && !isTelegramPeerId(peerId)) {
+                    throw invalidTelegramPeerIdError();
+                }
                 const normalizedDir = directoryInput ? (() => {
                     const scoped = resolveScopedDirectory(directoryInput);
                     if (!scoped.ok) {
@@ -935,6 +954,16 @@ export async function startBridge(config, logger, reporter, deps = {}) {
                 let sent = 0;
                 for (const binding of bindings) {
                     attempted += 1;
+                    if (channel === "telegram" && !isTelegramPeerId(binding.peer_id)) {
+                        store.deleteBinding(channel, binding.identity_id, binding.peer_id);
+                        store.deleteSession(channel, binding.identity_id, binding.peer_id);
+                        failures.push({
+                            identityId: binding.identity_id,
+                            peerId: binding.peer_id,
+                            error: "Invalid Telegram peerId binding removed (expected numeric chat_id)",
+                        });
+                        continue;
+                    }
                     const adapter = adapters.get(adapterKey(channel, binding.identity_id));
                     if (!adapter) {
                         failures.push({
@@ -1189,18 +1218,20 @@ export async function startBridge(config, logger, reporter, deps = {}) {
             try {
                 const effectiveModel = getUserModel(inbound.channel, inbound.identityId, peerKey, config.model);
                 const messagingAgent = await loadMessagingAgentConfig();
-                const promptText = messagingAgent.instructions
-                    ? [
-                        "You are handling a Slack/Telegram message via OpenWork.",
-                        `Workspace agent file: ${messagingAgent.filePath}`,
-                        ...(messagingAgent.selectedAgent ? [`Selected OpenCode agent: ${messagingAgent.selectedAgent}`] : []),
-                        "Follow these workspace messaging instructions:",
-                        messagingAgent.instructions,
-                        "",
-                        "Incoming user message:",
-                        inbound.text,
-                    ].join("\n")
-                    : inbound.text;
+                const effectiveInstructions = [DEFAULT_MESSAGING_AGENT_INSTRUCTIONS, messagingAgent.instructions]
+                    .map((value) => value.trim())
+                    .filter(Boolean)
+                    .join("\n\n");
+                const promptText = [
+                    "You are handling a Slack/Telegram message via OpenWork.",
+                    `Workspace agent file: ${messagingAgent.filePath}`,
+                    ...(messagingAgent.selectedAgent ? [`Selected OpenCode agent: ${messagingAgent.selectedAgent}`] : []),
+                    "Follow these workspace messaging instructions:",
+                    effectiveInstructions,
+                    "",
+                    "Incoming user message:",
+                    inbound.text,
+                ].join("\n");
                 logger.debug({
                     sessionID,
                     length: inbound.text.length,

package/dist/telegram.js

@@ -1,5 +1,18 @@
 import { Bot } from "grammy";
 const MAX_TEXT_LENGTH = 4096;
+const TELEGRAM_CHAT_ID_PATTERN = /^-?\d+$/;
+export function isTelegramPeerId(peerId) {
+    return TELEGRAM_CHAT_ID_PATTERN.test(peerId.trim());
+}
+export function parseTelegramPeerId(peerId) {
+    const trimmed = peerId.trim();
+    if (!isTelegramPeerId(trimmed))
+        return null;
+    const parsed = Number(trimmed);
+    if (!Number.isFinite(parsed))
+        return null;
+    return parsed;
+}
 export function createTelegramAdapter(identity, config, logger, onMessage) {
     const token = identity.token?.trim() ?? "";
     if (!token) {
@@ -72,7 +85,13 @@ export function createTelegramAdapter(identity, config, logger, onMessage) {
             log.info("telegram adapter stopped");
         },
         async sendText(peerId, text) {
-            await bot.api.sendMessage(Number(peerId), text);
+            const chatId = parseTelegramPeerId(peerId);
+            if (chatId === null) {
+                const error = new Error("Telegram peerId must be a numeric chat_id. Usernames like @name are not valid direct targets.");
+                error.status = 400;
+                throw error;
+            }
+            await bot.api.sendMessage(chatId, text);
         },
     };
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-router",
-  "version": "0.11.87",
+  "version": "0.11.88",
   "description": "opencode-router: Slack + Telegram bridge + directory routing for a running opencode server",
   "private": false,
   "type": "module",