telecodex 0.1.4 → 0.1.5

package/dist/bot/handlers/messageHandlers.js

@@ -2,9 +2,10 @@ import { contextLogFields, requireScopedSession, } from "../commandSupport.js";
 import { handleUserInput, handleUserText } from "../inputService.js";
 import { telegramImageMessageToCodexInput } from "../../telegram/attachments.js";
 import { replyError, replyNotice } from "../../telegram/formatted.js";
+import { wrapUserFacingHandler } from "../userFacingErrors.js";
 export function registerMessageHandlers(deps) {
     const { bot, config, store, projects, codex, buffers, logger } = deps;
-    bot.on("message:text", async (ctx) => {
+    bot.on("message:text", wrapUserFacingHandler("message:text", logger, async (ctx) => {
         const text = ctx.message.text;
         logger?.info("received telegram text message", {
             ...contextLogFields(ctx),
@@ -30,8 +31,8 @@ export function registerMessageHandlers(deps) {
             bot,
             ...(logger ? { logger } : {}),
         });
-    });
-    bot.on(["message:photo", "message:document"], async (ctx) => {
+    }));
+    bot.on(["message:photo", "message:document"], wrapUserFacingHandler("message:attachment", logger, async (ctx) => {
         const session = await requireScopedSession(ctx, store, projects, config);
         if (!session) {
             logger?.warn("ignored telegram attachment because no scoped session was available", {
@@ -68,5 +69,5 @@ export function registerMessageHandlers(deps) {
             });
             await replyError(ctx, error instanceof Error ? error.message : String(error));
         }
-    });
+    }));
 }

package/dist/bot/handlers/operationalHandlers.js

@@ -5,12 +5,13 @@ import { refreshSessionIfActiveTurnIsStale } from "../inputService.js";
 import { contextLogFields, formatHelpText, formatPrivateStatus, formatProjectStatus, formatReasoningEffort, getProjectForContext, hasTopicContext, isPrivateChat, parseSubcommand, requireScopedSession, } from "../commandSupport.js";
 import { formatIsoTimestamp, sessionLogFields } from "../sessionFlow.js";
 import { codeField, replyDocument, replyError, replyNotice, replyUsage, textField } from "../../telegram/formatted.js";
+import { wrapUserFacingHandler } from "../userFacingErrors.js";
 export function registerOperationalHandlers(deps) {
     const { bot, config, store, projects, codex, logger } = deps;
-    bot.command(["start", "help"], async (ctx) => {
+    bot.command(["start", "help"], wrapUserFacingHandler("help", logger, async (ctx) => {
         await replyNotice(ctx, formatHelpText(ctx, projects));
-    });
-    bot.command("admin", async (ctx) => {
+    }));
+    bot.command("admin", wrapUserFacingHandler("admin", logger, async (ctx) => {
         if (!isPrivateChat(ctx)) {
             await replyNotice(ctx, "Use /admin in the bot private chat.");
             return;
@@ -62,8 +63,8 @@ export function registerOperationalHandlers(deps) {
             return;
         }
         await replyUsage(ctx, "/admin | /admin rebind | /admin cancel");
-    });
-    bot.command("status", async (ctx) => {
+    }));
+    bot.command("status", wrapUserFacingHandler("status", logger, async (ctx) => {
         if (isPrivateChat(ctx)) {
             await replyNotice(ctx, formatPrivateStatus(store, projects));
             return;
@@ -113,8 +114,8 @@ export function registerOperationalHandlers(deps) {
                 textField("effort", formatReasoningEffort(latestSession.reasoningEffort)),
             ],
         });
-    });
-    bot.command("queue", async (ctx) => {
+    }));
+    bot.command("queue", wrapUserFacingHandler("queue", logger, async (ctx) => {
         const session = await requireScopedSession(ctx, store, projects, config);
         if (!session)
             return;
@@ -155,8 +156,8 @@ export function registerOperationalHandlers(deps) {
             return;
         }
         await replyUsage(ctx, "/queue | /queue drop <id> | /queue clear");
-    });
-    bot.command("stop", async (ctx) => {
+    }));
+    bot.command("stop", wrapUserFacingHandler("stop", logger, async (ctx) => {
         const session = await requireScopedSession(ctx, store, projects, config);
         if (!session)
             return;
@@ -177,7 +178,7 @@ export function registerOperationalHandlers(deps) {
             });
             await replyError(ctx, `Interrupt failed: ${error instanceof Error ? error.message : String(error)}`);
         }
-    });
+    }));
 }
 function formatQueuedPreview(items) {
     if (items.length === 0)

package/dist/bot/handlers/projectHandlers.js

@@ -2,10 +2,11 @@ import path from "node:path";
 import { contextLogFields, ensureTopicSession, formatPrivateProjectList, formatProjectStatus, formatTopicName, getProjectForContext, hasTopicContext, isPrivateChat, isSupergroupChat, parseSubcommand, postTopicReadyMessage, requireScopedSession, resolveExistingDirectory, } from "../commandSupport.js";
 import { formatSessionRuntimeStatus } from "../../runtime/sessionRuntime.js";
 import { codeField, replyDocument, replyError, replyNotice, replyUsage, textField } from "../../telegram/formatted.js";
+import { wrapUserFacingHandler } from "../userFacingErrors.js";
 const PROJECT_REQUIRED_MESSAGE = "This supergroup has no project bound yet.\nRun /project bind <absolute-path> first.";
 export function registerProjectHandlers(deps) {
     const { bot, config, store, projects, logger } = deps;
-    bot.command("project", async (ctx) => {
+    bot.command("project", wrapUserFacingHandler("project", logger, async (ctx) => {
         const { command, args } = parseSubcommand(ctx.match.trim());
         if (isPrivateChat(ctx)) {
             if (!command || command === "list" || command === "status") {
@@ -67,8 +68,8 @@ export function registerProjectHandlers(deps) {
             return;
         }
         await replyUsage(ctx, ["/project", "/project bind <absolute-path>", "/project unbind"]);
-    });
-    bot.command("thread", async (ctx) => {
+    }));
+    bot.command("thread", wrapUserFacingHandler("thread", logger, async (ctx) => {
         if (isPrivateChat(ctx)) {
             await replyNotice(ctx, "The thread command is only available inside project supergroups.");
             return;
@@ -112,7 +113,7 @@ export function registerProjectHandlers(deps) {
             return;
         }
         await replyUsage(ctx, ["/thread list", "/thread resume <threadId>", "/thread new <topic-name>"]);
-    });
+    }));
     bot.on(["message:forum_topic_created", "message:forum_topic_edited"], async (ctx) => {
         const threadId = ctx.message.message_thread_id;
         if (threadId == null)

package/dist/bot/handlers/sessionConfigHandlers.js

@@ -2,6 +2,7 @@ import { APPROVAL_POLICIES, MODE_PRESETS, REASONING_EFFORTS, SANDBOX_MODES, WEB_
 import { parseCodexConfigOverrides } from "../../codex/configOverrides.js";
 import { assertProjectScopedPath, formatProfileReply, formatReasoningEffort, getProjectForContext, requireScopedSession, resolveExistingDirectory, } from "../commandSupport.js";
 import { codeField, replyDocument, replyError, replyNotice, replyUsage, textField } from "../../telegram/formatted.js";
+import { wrapUserFacingHandler } from "../userFacingErrors.js";
 export function registerSessionConfigHandlers(deps) {
     registerDirectoryHandlers(deps);
     registerProfileHandlers(deps);
@@ -9,8 +10,8 @@ export function registerSessionConfigHandlers(deps) {
     registerAdvancedHandlers(deps);
 }
 function registerDirectoryHandlers(deps) {
-    const { bot, config, store, projects } = deps;
-    bot.command("cwd", async (ctx) => {
+    const { bot, config, store, projects, logger } = deps;
+    bot.command("cwd", wrapUserFacingHandler("cwd", logger, async (ctx) => {
         const project = getProjectForContext(ctx, projects);
         const session = await requireScopedSession(ctx, store, projects, config);
         if (!project || !session)
@@ -34,11 +35,11 @@ function registerDirectoryHandlers(deps) {
         catch (error) {
             await replyError(ctx, error instanceof Error ? error.message : String(error));
         }
-    });
+    }));
 }
 function registerProfileHandlers(deps) {
-    const { bot, config, store, projects } = deps;
-    bot.command("mode", async (ctx) => {
+    const { bot, config, store, projects, logger } = deps;
+    bot.command("mode", wrapUserFacingHandler("mode", logger, async (ctx) => {
         const session = await requireScopedSession(ctx, store, projects, config);
         if (!session)
             return;
@@ -59,8 +60,8 @@ function registerProfileHandlers(deps) {
         store.setSandboxMode(session.sessionKey, profile.sandboxMode);
         store.setApprovalPolicy(session.sessionKey, profile.approvalPolicy);
         await replyNotice(ctx, formatProfileReply("Preset updated.", profile.sandboxMode, profile.approvalPolicy));
-    });
-    bot.command("sandbox", async (ctx) => {
+    }));
+    bot.command("sandbox", wrapUserFacingHandler("sandbox", logger, async (ctx) => {
         const session = await requireScopedSession(ctx, store, projects, config);
         if (!session)
             return;
@@ -75,8 +76,8 @@ function registerProfileHandlers(deps) {
         }
         store.setSandboxMode(session.sessionKey, sandboxMode);
         await replyNotice(ctx, formatProfileReply("Sandbox updated.", sandboxMode, session.approvalPolicy));
-    });
-    bot.command("approval", async (ctx) => {
+    }));
+    bot.command("approval", wrapUserFacingHandler("approval", logger, async (ctx) => {
         const session = await requireScopedSession(ctx, store, projects, config);
         if (!session)
             return;
@@ -91,8 +92,8 @@ function registerProfileHandlers(deps) {
         }
         store.setApprovalPolicy(session.sessionKey, approvalPolicy);
         await replyNotice(ctx, formatProfileReply("Approval policy updated.", session.sandboxMode, approvalPolicy));
-    });
-    bot.command("yolo", async (ctx) => {
+    }));
+    bot.command("yolo", wrapUserFacingHandler("yolo", logger, async (ctx) => {
         const session = await requireScopedSession(ctx, store, projects, config);
         if (!session)
             return;
@@ -110,11 +111,11 @@ function registerProfileHandlers(deps) {
         store.setSandboxMode(session.sessionKey, profile.sandboxMode);
         store.setApprovalPolicy(session.sessionKey, profile.approvalPolicy);
         await replyNotice(ctx, formatProfileReply(value === "on" ? "YOLO enabled." : "YOLO disabled. Restored the write preset.", profile.sandboxMode, profile.approvalPolicy));
-    });
+    }));
 }
 function registerExecutionHandlers(deps) {
-    const { bot, config, store, projects } = deps;
-    bot.command("model", async (ctx) => {
+    const { bot, config, store, projects, logger } = deps;
+    bot.command("model", wrapUserFacingHandler("model", logger, async (ctx) => {
         const session = await requireScopedSession(ctx, store, projects, config);
         if (!session)
             return;
@@ -125,8 +126,8 @@ function registerExecutionHandlers(deps) {
         }
         store.setModel(session.sessionKey, model);
         await replyNotice(ctx, `Set model: ${model}`);
-    });
-    bot.command("effort", async (ctx) => {
+    }));
+    bot.command("effort", wrapUserFacingHandler("effort", logger, async (ctx) => {
         const session = await requireScopedSession(ctx, store, projects, config);
         if (!session)
             return;
@@ -141,8 +142,8 @@ function registerExecutionHandlers(deps) {
         }
         store.setReasoningEffort(session.sessionKey, value === "default" ? null : value);
         await replyNotice(ctx, `Set reasoning effort: ${value === "default" ? "codex-default" : value}`);
-    });
-    bot.command("web", async (ctx) => {
+    }));
+    bot.command("web", wrapUserFacingHandler("web", logger, async (ctx) => {
         const session = await requireScopedSession(ctx, store, projects, config);
         if (!session)
             return;
@@ -157,8 +158,8 @@ function registerExecutionHandlers(deps) {
         }
         store.setWebSearchMode(session.sessionKey, value === "default" ? null : value);
         await replyNotice(ctx, `Set web search: ${value === "default" ? "codex-default" : value}`);
-    });
-    bot.command("network", async (ctx) => {
+    }));
+    bot.command("network", wrapUserFacingHandler("network", logger, async (ctx) => {
         const session = await requireScopedSession(ctx, store, projects, config);
         if (!session)
             return;
@@ -173,8 +174,8 @@ function registerExecutionHandlers(deps) {
         }
         store.setNetworkAccessEnabled(session.sessionKey, value === "on");
         await replyNotice(ctx, `Set network access: ${value}`);
-    });
-    bot.command("gitcheck", async (ctx) => {
+    }));
+    bot.command("gitcheck", wrapUserFacingHandler("gitcheck", logger, async (ctx) => {
         const session = await requireScopedSession(ctx, store, projects, config);
         if (!session)
             return;
@@ -189,11 +190,11 @@ function registerExecutionHandlers(deps) {
         }
         store.setSkipGitRepoCheck(session.sessionKey, value === "skip");
         await replyNotice(ctx, `Set git repo check: ${value}`);
-    });
+    }));
 }
 function registerAdvancedHandlers(deps) {
-    const { bot, config, store, projects, codex } = deps;
-    bot.command("adddir", async (ctx) => {
+    const { bot, config, store, projects, codex, logger } = deps;
+    bot.command("adddir", wrapUserFacingHandler("adddir", logger, async (ctx) => {
         const project = getProjectForContext(ctx, projects);
         const session = await requireScopedSession(ctx, store, projects, config);
         if (!project || !session)
@@ -275,8 +276,8 @@ function registerAdvancedHandlers(deps) {
             return;
         }
         await replyUsage(ctx, "/adddir list | /adddir add <path-inside-project> | /adddir add-external <absolute-path> | /adddir drop <index> | /adddir clear");
-    });
-    bot.command("schema", async (ctx) => {
+    }));
+    bot.command("schema", wrapUserFacingHandler("schema", logger, async (ctx) => {
         const session = await requireScopedSession(ctx, store, projects, config);
         if (!session)
             return;
@@ -316,8 +317,8 @@ function registerAdvancedHandlers(deps) {
         catch (error) {
             await replyError(ctx, `Failed to parse JSON: ${error instanceof Error ? error.message : String(error)}`);
         }
-    });
-    bot.command("codexconfig", async (ctx) => {
+    }));
+    bot.command("codexconfig", wrapUserFacingHandler("codexconfig", logger, async (ctx) => {
         const raw = ctx.match.trim();
         if (!raw || raw === "show") {
             const current = store.getAppState("codex_config_overrides");
@@ -353,7 +354,7 @@ function registerAdvancedHandlers(deps) {
         catch (error) {
             await replyError(ctx, error instanceof Error ? error.message : String(error));
         }
-    });
+    }));
 }
 function isPlainObject(value) {
     return typeof value === "object" && value !== null && !Array.isArray(value);

package/dist/bot/userFacingErrors.js

@@ -0,0 +1,59 @@
+import { GrammyError, HttpError } from "grammy";
+import { replyError } from "../telegram/formatted.js";
+import { contextLogFields } from "./commandSupport.js";
+export function wrapUserFacingHandler(handlerName, logger, handler) {
+    return async (ctx) => {
+        try {
+            await handler(ctx);
+        }
+        catch (error) {
+            logger?.error("telegram handler failed", {
+                handler: handlerName,
+                ...contextLogFields(ctx),
+                error,
+            });
+            await safeReplyError(ctx, describeUserFacingError(error), logger, handlerName, error);
+        }
+    };
+}
+async function safeReplyError(ctx, message, logger, handlerName, originalError) {
+    try {
+        await replyError(ctx, message);
+    }
+    catch (replyFailure) {
+        logger?.error("failed to send telegram handler error reply", {
+            handler: handlerName,
+            ...contextLogFields(ctx),
+            error: replyFailure,
+            originalError,
+        });
+    }
+}
+function describeUserFacingError(error) {
+    if (error instanceof GrammyError) {
+        return describeTelegramError(error);
+    }
+    if (error instanceof HttpError) {
+        return "Telegram request failed before it reached the API. Check the local network and try again.";
+    }
+    if (error instanceof Error) {
+        return error.message;
+    }
+    return String(error);
+}
+function describeTelegramError(error) {
+    const description = error.description || error.message;
+    const normalized = description.toLowerCase();
+    if (error.method === "createForumTopic") {
+        if (normalized.includes("not enough rights")) {
+            return "Telegram rejected topic creation because the bot lacks permission to create topics. Promote the bot to admin and grant topic management, then try again.";
+        }
+        if (normalized.includes("forum is disabled") ||
+            normalized.includes("chat is not a forum") ||
+            normalized.includes("topics are not enabled")) {
+            return "This supergroup does not currently allow forum topics. Enable topics for the group and try again.";
+        }
+        return `Failed to create the Telegram topic: ${description}`;
+    }
+    return description;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "telecodex",
-  "version": "0.1.4",
+  "version": "0.1.5",
   "description": "Telegram bridge for local Codex.",
   "license": "MIT",
   "type": "module",