@gonzih/cc-discord 0.1.3 → 0.1.5

package/dist/bot.d.ts

@@ -36,7 +36,7 @@ export declare class CcDiscordBot {
     /** Channels created by the bot for a meta-agent namespace → skip local Claude session */
     private channelNamespaceMap;
     private storeSnowflake;
-    private reverseSnowflakeLookup;
+    reverseSnowflakeLookup(n: number): string | undefined;
     /** Session key: "channelId" or "channelId:threadId" for threads */
     private sessionKey;
     /** Get the channel/thread for sending messages */

package/dist/bot.js

@@ -82,6 +82,16 @@ async function fetchAsBase64(url) {
         }).on("error", reject);
     });
 }
+/**
+ * Resolve the Discord category ID to use as `parent` when creating new text channels.
+ * Priority: DISCORD_DEFAULT_CATEGORY_ID env var → first category named "Text Channels" → undefined.
+ */
+function resolveCategoryId(guild) {
+    if (process.env.DISCORD_DEFAULT_CATEGORY_ID)
+        return process.env.DISCORD_DEFAULT_CATEGORY_ID;
+    const category = guild.channels.cache.find((ch) => ch.type === ChannelType.GuildCategory && /text channels/i.test(ch.name));
+    return category?.id;
+}
 export class CcDiscordBot {
     client;
     sessions = new Map();
@@ -120,6 +130,12 @@ export class CcDiscordBot {
             this.registerSlashCommands().catch((err) => {
                 console.error("[discord] slash command registration failed:", err.message);
             });
+            // Pre-populate snowflakeMap so reverse-lookup works for all channels visible at login
+            for (const [, guild] of readyClient.guilds.cache) {
+                for (const [, channel] of guild.channels.cache) {
+                    this.storeSnowflake(channel.id);
+                }
+            }
         });
         this.client.on(Events.MessageCreate, (msg) => {
             void this.handleMessage(msg);
@@ -618,7 +634,7 @@ export class CcDiscordBot {
                 }
                 await interaction.deferReply();
                 try {
-                    const newChannel = await guild.channels.create({ name: namespace, type: ChannelType.GuildText });
+                    const newChannel = await guild.channels.create({ name: namespace, type: ChannelType.GuildText, parent: resolveCategoryId(guild) });
                     this.channelNamespaceMap.set(newChannel.id, { namespace, repoUrl });
                     this.opts.registerRoutedChannelId?.(namespace, newChannel.id);
                     await interaction.editReply(`Created <#${newChannel.id}> — messages there route to the ${repoUrl} meta-agent`);
@@ -648,7 +664,11 @@ export class CcDiscordBot {
                     await interaction.reply("No cron jobs for this channel.");
                 }
                 else {
-                    const lines = jobs.map((j) => `• **${j.id}** ${j.schedule}: \`${j.prompt}\``);
+                    const lines = jobs.map((j) => {
+                        const chanId = this.reverseSnowflakeLookup(j.chatId);
+                        const chanMention = chanId ? ` <#${chanId}>` : "";
+                        return `• **${j.id}**${chanMention} ${j.schedule}: \`${j.prompt}\``;
+                    });
                     await interaction.reply(lines.join("\n"));
                 }
                 break;
@@ -752,7 +772,7 @@ export class CcDiscordBot {
         }
         let newChannel;
         try {
-            newChannel = await guild.channels.create({ name: namespace, type: ChannelType.GuildText });
+            newChannel = await guild.channels.create({ name: namespace, type: ChannelType.GuildText, parent: resolveCategoryId(guild) });
         }
         catch (err) {
             await channel.send(`Failed to create channel: ${err.message}`).catch(() => { });

package/dist/index.js

@@ -90,7 +90,7 @@ const bot = new CcDiscordBot({
     namespace,
     registerRoutedChannelId: (ns, channelId) => notifier.registerRoutedChannelId(ns, channelId),
 });
-const notifier = startNotifier(bot, notifyChannelId, namespace, sharedRedis, (channelId, text) => handleUserMessageFn?.(channelId, text), (channelId, text) => forwardNotificationFn?.(channelId, text), () => getLastActiveChannelIdFn());
+const notifier = startNotifier(bot, notifyChannelId, namespace, sharedRedis, (channelId, text) => handleUserMessageFn?.(channelId, text), (channelId, text) => forwardNotificationFn?.(channelId, text), () => getLastActiveChannelIdFn(), (n) => bot.reverseSnowflakeLookup(n));
 console.log(`[notifier] started for namespace=${namespace} notifyChannelId=${notifyChannelId ?? "dynamic"}`);
 // Wire closures now that bot is constructed
 getLastActiveChannelIdFn = () => bot.getLastActiveChannelId();

package/dist/notifier.d.ts

@@ -20,12 +20,17 @@ export interface ChatMessage {
     timestamp: string;
     chatId: number;
 }
+export interface ParsedNotification {
+    text: string;
+    chatId?: number;
+}
 /**
- * Parse a notification payload and return the display text.
- * Appends a [driver] or [driver:model] badge whenever the driver field is present.
+ * Parse a notification payload.
+ * Returns the display text plus an optional chatId for per-channel routing.
+ * Appends a [driver] or [driver:model] badge when present.
  * Appends " cost: $X.XXX" if a numeric cost field is present.
  */
-export declare function parseNotification(raw: string): string;
+export declare function parseNotification(raw: string): ParsedNotification;
 /**
  * Write a message to the chat log in Redis.
  * Fire-and-forget — errors are logged but not thrown.
@@ -42,12 +47,13 @@ export interface NotifierHandle {
 /**
  * Start the Discord notifier.
  *
- * @param bot               - CcDiscordBot instance (for sending messages)
- * @param notifyChannelId   - Discord channel ID to forward notifications to. Pass null to use getActiveChannelId.
- * @param namespace         - cc-agent namespace (used to build Redis channel names)
- * @param redis             - ioredis client in normal mode (will be duplicated for pub/sub)
- * @param handleUserMessage - Optional callback to feed UI messages into the active Claude session
- * @param forwardNotification - Optional callback to forward job notifications
- * @param getActiveChannelId  - Optional callback to resolve channelId dynamically
+ * @param bot                     - CcDiscordBot instance (for sending messages)
+ * @param notifyChannelId         - Discord channel ID to forward notifications to. Pass null to use getActiveChannelId.
+ * @param namespace               - cc-agent namespace (used to build Redis channel names)
+ * @param redis                   - ioredis client in normal mode (will be duplicated for pub/sub)
+ * @param handleUserMessage       - Optional callback to feed UI messages into the active Claude session
+ * @param forwardNotification     - Optional callback to forward job notifications
+ * @param getActiveChannelId      - Optional callback to resolve channelId dynamically
+ * @param reverseSnowflakeLookup  - Optional callback to resolve a chatId integer to a Discord channelId
  */
-export declare function startNotifier(bot: CcDiscordBot, notifyChannelId: string | null, namespace: string, redis: Redis, handleUserMessage?: (channelId: string, text: string) => void, forwardNotification?: (channelId: string, text: string) => void, getActiveChannelId?: () => string | undefined): NotifierHandle;
+export declare function startNotifier(bot: CcDiscordBot, notifyChannelId: string | null, namespace: string, redis: Redis, handleUserMessage?: (channelId: string, text: string) => void, forwardNotification?: (channelId: string, text: string) => void, getActiveChannelId?: () => string | undefined, reverseSnowflakeLookup?: (n: number) => string | undefined): NotifierHandle;

package/dist/notifier.js

@@ -31,8 +31,9 @@ function shortenModelName(model, driver) {
     return model;
 }
 /**
- * Parse a notification payload and return the display text.
- * Appends a [driver] or [driver:model] badge whenever the driver field is present.
+ * Parse a notification payload.
+ * Returns the display text plus an optional chatId for per-channel routing.
+ * Appends a [driver] or [driver:model] badge when present.
  * Appends " cost: $X.XXX" if a numeric cost field is present.
  */
 export function parseNotification(raw) {
@@ -40,6 +41,7 @@ export function parseNotification(raw) {
     let driver;
     let model;
     let cost;
+    let chatId;
     try {
         const parsed = JSON.parse(raw);
         if (parsed.text)
@@ -48,16 +50,18 @@ export function parseNotification(raw) {
         model = parsed.model;
         if (typeof parsed.cost === "number")
             cost = parsed.cost;
+        if (typeof parsed.chat_id === "number" && parsed.chat_id !== 0)
+            chatId = parsed.chat_id;
     }
     catch {
-        return text;
+        return { text };
     }
     if (!driver)
-        return text;
+        return { text, chatId };
     const shortModel = shortenModelName(model ?? "", driver);
     const badge = shortModel ? `${driver}:${shortModel}` : driver;
     const costStr = cost != null ? ` cost: $${cost.toFixed(3)}` : "";
-    return `${text}\n[${badge}]${costStr}`;
+    return { text: `${text}\n[${badge}]${costStr}`, chatId };
 }
 /**
  * Write a message to the chat log in Redis.
@@ -77,18 +81,32 @@ export function writeChatLog(redis, namespace, msg) {
         log("warn", "writeChatLog publish failed:", err.message);
     });
 }
+/**
+ * Resolve the target Discord channelId for a notification.
+ * When chatId is set and a reverse-lookup function is available, prefer the originating channel.
+ * Falls back to notifyChannelId, then getActiveChannelId.
+ */
+function resolveNotifyChannel(chatId, notifyChannelId, getActiveChannelId, reverseSnowflakeLookup) {
+    if (chatId != null && reverseSnowflakeLookup) {
+        const resolved = reverseSnowflakeLookup(chatId);
+        if (resolved)
+            return resolved;
+    }
+    return notifyChannelId ?? getActiveChannelId?.();
+}
 /**
  * Start the Discord notifier.
  *
- * @param bot               - CcDiscordBot instance (for sending messages)
- * @param notifyChannelId   - Discord channel ID to forward notifications to. Pass null to use getActiveChannelId.
- * @param namespace         - cc-agent namespace (used to build Redis channel names)
- * @param redis             - ioredis client in normal mode (will be duplicated for pub/sub)
- * @param handleUserMessage - Optional callback to feed UI messages into the active Claude session
- * @param forwardNotification - Optional callback to forward job notifications
- * @param getActiveChannelId  - Optional callback to resolve channelId dynamically
+ * @param bot                     - CcDiscordBot instance (for sending messages)
+ * @param notifyChannelId         - Discord channel ID to forward notifications to. Pass null to use getActiveChannelId.
+ * @param namespace               - cc-agent namespace (used to build Redis channel names)
+ * @param redis                   - ioredis client in normal mode (will be duplicated for pub/sub)
+ * @param handleUserMessage       - Optional callback to feed UI messages into the active Claude session
+ * @param forwardNotification     - Optional callback to forward job notifications
+ * @param getActiveChannelId      - Optional callback to resolve channelId dynamically
+ * @param reverseSnowflakeLookup  - Optional callback to resolve a chatId integer to a Discord channelId
  */
-export function startNotifier(bot, notifyChannelId, namespace, redis, handleUserMessage, forwardNotification, getActiveChannelId) {
+export function startNotifier(bot, notifyChannelId, namespace, redis, handleUserMessage, forwardNotification, getActiveChannelId, reverseSnowflakeLookup) {
     // Per-namespace channelId registry
     const routedChannelIds = new Map();
     const sub = redis.duplicate({
@@ -210,12 +228,13 @@ export function startNotifier(bot, notifyChannelId, namespace, redis, handleUser
             }
         }
         for (const raw of items) {
-            const text = parseNotification(raw);
-            bot.sendToChannelById(targetId, text).catch((err) => {
+            const notification = parseNotification(raw);
+            const destChannelId = resolveNotifyChannel(notification.chatId, notifyChannelId, getActiveChannelId, reverseSnowflakeLookup) ?? targetId;
+            bot.sendToChannelById(destChannelId, notification.text).catch((err) => {
                 log("warn", "notify list send failed:", err.message);
             });
             if (forwardNotification) {
-                forwardNotification(targetId, text);
+                forwardNotification(destChannelId, notification.text);
             }
         }
         if (remaining > 0) {
@@ -231,14 +250,14 @@ export function startNotifier(bot, notifyChannelId, namespace, redis, handleUser
         const notifyCh = notifyChannel(namespace);
         const incomingCh = chatIncomingChannel(namespace);
         if (channel === notifyCh) {
-            const targetId = notifyChannelId ?? getActiveChannelId?.();
+            const notification = parseNotification(message);
+            const targetId = resolveNotifyChannel(notification.chatId, notifyChannelId, getActiveChannelId, reverseSnowflakeLookup);
             if (targetId != null) {
-                const text = parseNotification(message);
-                bot.sendToChannelById(targetId, text).catch((err) => {
+                bot.sendToChannelById(targetId, notification.text).catch((err) => {
                     log("warn", "notify send failed:", err.message);
                 });
                 if (forwardNotification) {
-                    forwardNotification(targetId, text);
+                    forwardNotification(targetId, notification.text);
                 }
             }
             else {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gonzih/cc-discord",
-  "version": "0.1.3",
+  "version": "0.1.5",
   "description": "Claude Code Discord bot — chat with Claude Code via Discord",
   "type": "module",
   "bin": {