onkol 0.5.0 → 0.5.2

package/dist/cli/index.js

@@ -570,32 +570,45 @@ program
         console.log(chalk.cyan('[1/3] Updating files from npm package...'));
         try {
             // Find where this CLI is running from — that's the latest package
+            // __dirname is dist/cli/, so pkgRoot is the npm package root
             const pkgRoot = resolve(__dirname, '..');
-            const pluginSrc = existsSync(resolve(pkgRoot, 'src/plugin'))
-                ? resolve(pkgRoot, 'src/plugin')
-                : resolve(pkgRoot, 'dist/plugin');
-            const scriptsSrc = resolve(pkgRoot, 'scripts');
-            // Copy plugin files
-            if (existsSync(pluginSrc)) {
-                const pluginDest = resolve(dir, 'plugins/discord-filtered');
-                const { readdirSync } = await import('fs');
-                for (const f of readdirSync(pluginSrc)) {
-                    if (f.endsWith('.ts') || f.endsWith('.js')) {
-                        copyFileSync(resolve(pluginSrc, f), resolve(pluginDest, f));
+            const { readdirSync, chmodSync } = await import('fs');
+            // Try src/plugin first (has .ts files), then dist/plugin (.js files)
+            let pluginUpdated = false;
+            for (const candidate of ['src/plugin', 'dist/plugin']) {
+                const pluginSrc = resolve(pkgRoot, candidate);
+                if (existsSync(pluginSrc)) {
+                    const pluginDest = resolve(dir, 'plugins/discord-filtered');
+                    mkdirSync(pluginDest, { recursive: true });
+                    for (const f of readdirSync(pluginSrc)) {
+                        if (f.endsWith('.ts') || f.endsWith('.js')) {
+                            copyFileSync(resolve(pluginSrc, f), resolve(pluginDest, f));
+                        }
                     }
+                    console.log(chalk.green(`  ✓ Plugin files updated (from ${candidate})`));
+                    pluginUpdated = true;
+                    break;
                 }
-                console.log(chalk.green('  ✓ Plugin files updated'));
+            }
+            if (!pluginUpdated) {
+                console.log(chalk.yellow(`  ⚠ No plugin source found in package (looked in ${pkgRoot})`));
             }
             // Copy scripts
+            const scriptsSrc = resolve(pkgRoot, 'scripts');
             if (existsSync(scriptsSrc)) {
-                const { readdirSync, chmodSync } = await import('fs');
+                mkdirSync(resolve(dir, 'scripts'), { recursive: true });
+                let count = 0;
                 for (const f of readdirSync(scriptsSrc)) {
                     if (f.endsWith('.sh')) {
                         copyFileSync(resolve(scriptsSrc, f), resolve(dir, 'scripts', f));
                         chmodSync(resolve(dir, 'scripts', f), 0o755);
+                        count++;
                     }
                 }
-                console.log(chalk.green('  ✓ Scripts updated'));
+                console.log(chalk.green(`  ✓ ${count} scripts updated`));
+            }
+            else {
+                console.log(chalk.yellow(`  ⚠ No scripts dir found at ${scriptsSrc}`));
             }
         }
         catch (err) {
@@ -665,11 +678,17 @@ program
         --intent "${w.intent}" \
         ${resumeArg}`;
         try {
-            execSync(cmd, { stdio: 'pipe' });
+            const output = execSync(cmd, { encoding: 'utf-8', stdio: ['pipe', 'pipe', 'pipe'] });
             console.log(chalk.green(`  ✓ ${w.name} respawned${w.sessionId ? ' (resumed)' : ''}`));
+            if (output.trim())
+                console.log(chalk.gray(`    ${output.trim()}`));
         }
         catch (err) {
-            console.log(chalk.red(`  ✗ Failed to spawn ${w.name}: ${err instanceof Error ? err.message : err}`));
+            console.log(chalk.red(`  ✗ Failed to spawn ${w.name}`));
+            if (err.stderr)
+                console.log(chalk.red(`    stderr: ${err.stderr.toString().trim()}`));
+            if (err.stdout)
+                console.log(chalk.gray(`    stdout: ${err.stdout.toString().trim()}`));
         }
         // Small delay to avoid Discord rate limits
         await new Promise(r => setTimeout(r, 2000));

package/dist/plugin/discord-client.js

@@ -1,4 +1,10 @@
 import { Client, GatewayIntentBits } from 'discord.js';
+import { mkdirSync, writeFileSync } from 'fs';
+import { join } from 'path';
+import { tmpdir } from 'os';
+const IMAGE_EXTENSIONS = new Set(['.png', '.jpg', '.jpeg', '.gif', '.webp', '.bmp', '.svg']);
+const ATTACHMENT_DIR = join(tmpdir(), 'onkol-attachments');
+mkdirSync(ATTACHMENT_DIR, { recursive: true });
 export function shouldForwardMessage(messageChannelId, authorId, isBot, targetChannelId, allowedUsers) {
     if (isBot)
         return false;
@@ -8,17 +14,38 @@ export function shouldForwardMessage(messageChannelId, authorId, isBot, targetCh
         return false;
     return true;
 }
-// When a message is too long, Discord auto-converts it to a .txt file attachment
-// with empty message content. This fetches the text from those attachments.
-async function resolveTextAttachments(message) {
+function isTextAttachment(a) {
+    return a.contentType?.startsWith('text/') === true || a.name?.endsWith('.txt') === true;
+}
+function isImageAttachment(a) {
+    if (a.contentType?.startsWith('image/'))
+        return true;
+    const name = (a.name || '').toLowerCase();
+    return IMAGE_EXTENSIONS.has(name.slice(name.lastIndexOf('.')));
+}
+// Resolve all attachments: text gets inlined, images get downloaded to temp files
+async function resolveAttachments(message) {
     let content = message.content;
-    const textAttachments = message.attachments.filter((a) => a.contentType?.startsWith('text/') || a.name?.endsWith('.txt'));
-    for (const attachment of textAttachments.values()) {
+    for (const attachment of message.attachments.values()) {
         try {
-            const res = await fetch(attachment.url);
-            if (res.ok) {
-                const text = await res.text();
-                content = content ? `${content}\n\n${text}` : text;
+            if (isTextAttachment(attachment)) {
+                const res = await fetch(attachment.url);
+                if (res.ok) {
+                    const text = await res.text();
+                    content = content ? `${content}\n\n${text}` : text;
+                }
+            }
+            else if (isImageAttachment(attachment)) {
+                const res = await fetch(attachment.url);
+                if (res.ok) {
+                    const buffer = Buffer.from(await res.arrayBuffer());
+                    const filename = `${message.id}-${attachment.name || 'image.png'}`;
+                    const filepath = join(ATTACHMENT_DIR, filename);
+                    writeFileSync(filepath, buffer);
+                    const note = `[User sent an image: ${attachment.name || 'image'}. Saved to ${filepath} — use the Read tool to view it.]`;
+                    content = content ? `${content}\n\n${note}` : note;
+                    console.error(`[discord-filtered] Downloaded image: ${filepath} (${buffer.length} bytes)`);
+                }
             }
         }
         catch (err) {
@@ -37,7 +64,7 @@ export function createDiscordClient(config, onMessage) {
     });
     client.on('messageCreate', async (message) => {
         if (shouldForwardMessage(message.channel.id, message.author.id, message.author.bot, config.channelId, config.allowedUsers)) {
-            const content = await resolveTextAttachments(message);
+            const content = await resolveAttachments(message);
             if (content) {
                 onMessage(content, message);
             }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "onkol",
-  "version": "0.5.0",
+  "version": "0.5.2",
   "description": "Decentralized on-call agent system powered by Claude Code",
   "type": "module",
   "bin": {

package/src/plugin/discord-client.ts

@@ -1,4 +1,11 @@
 import { Client, GatewayIntentBits, type Message, type Attachment } from 'discord.js'
+import { mkdirSync, writeFileSync } from 'fs'
+import { join } from 'path'
+import { tmpdir } from 'os'
+
+const IMAGE_EXTENSIONS = new Set(['.png', '.jpg', '.jpeg', '.gif', '.webp', '.bmp', '.svg'])
+const ATTACHMENT_DIR = join(tmpdir(), 'onkol-attachments')
+mkdirSync(ATTACHMENT_DIR, { recursive: true })
 
 export interface DiscordClientConfig {
   botToken: string
@@ -19,24 +26,45 @@ export function shouldForwardMessage(
   return true
 }
 
-// When a message is too long, Discord auto-converts it to a .txt file attachment
-// with empty message content. This fetches the text from those attachments.
-async function resolveTextAttachments(message: Message): Promise<string> {
+function isTextAttachment(a: Attachment): boolean {
+  return a.contentType?.startsWith('text/') === true || a.name?.endsWith('.txt') === true
+}
+
+function isImageAttachment(a: Attachment): boolean {
+  if (a.contentType?.startsWith('image/')) return true
+  const name = (a.name || '').toLowerCase()
+  return IMAGE_EXTENSIONS.has(name.slice(name.lastIndexOf('.')))
+}
+
+// Resolve all attachments: text gets inlined, images get downloaded to temp files
+async function resolveAttachments(message: Message): Promise<string> {
   let content = message.content
-  const textAttachments = message.attachments.filter(
-    (a: Attachment) => a.contentType?.startsWith('text/') || a.name?.endsWith('.txt')
-  )
-  for (const attachment of textAttachments.values()) {
+
+  for (const attachment of message.attachments.values()) {
     try {
-      const res = await fetch(attachment.url)
-      if (res.ok) {
-        const text = await res.text()
-        content = content ? `${content}\n\n${text}` : text
+      if (isTextAttachment(attachment)) {
+        const res = await fetch(attachment.url)
+        if (res.ok) {
+          const text = await res.text()
+          content = content ? `${content}\n\n${text}` : text
+        }
+      } else if (isImageAttachment(attachment)) {
+        const res = await fetch(attachment.url)
+        if (res.ok) {
+          const buffer = Buffer.from(await res.arrayBuffer())
+          const filename = `${message.id}-${attachment.name || 'image.png'}`
+          const filepath = join(ATTACHMENT_DIR, filename)
+          writeFileSync(filepath, buffer)
+          const note = `[User sent an image: ${attachment.name || 'image'}. Saved to ${filepath} — use the Read tool to view it.]`
+          content = content ? `${content}\n\n${note}` : note
+          console.error(`[discord-filtered] Downloaded image: ${filepath} (${buffer.length} bytes)`)
+        }
       }
     } catch (err) {
       console.error(`[discord-filtered] Failed to fetch attachment ${attachment.name}: ${err}`)
     }
   }
+
   return content
 }
 
@@ -62,7 +90,7 @@ export function createDiscordClient(
         config.allowedUsers
       )
     ) {
-      const content = await resolveTextAttachments(message)
+      const content = await resolveAttachments(message)
       if (content) {
         onMessage(content, message)
       }