onkol 0.5.1 → 0.5.4

package/dist/cli/index.js

@@ -570,8 +570,8 @@ 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, '..');
+            // __dirname is dist/cli/, so pkgRoot is two levels up (dist/cli -> dist -> package root)
+            const pkgRoot = resolve(__dirname, '../..');
             const { readdirSync, chmodSync } = await import('fs');
             // Try src/plugin first (has .ts files), then dist/plugin (.js files)
             let pluginUpdated = false;

package/dist/plugin/discord-client.js

@@ -1,4 +1,12 @@
 import { Client, GatewayIntentBits } from 'discord.js';
+import { mkdirSync, writeFileSync } from 'fs';
+import { join } from 'path';
+import { tmpdir } from 'os';
+const ATTACHMENT_DIR = join(tmpdir(), 'onkol-attachments');
+mkdirSync(ATTACHMENT_DIR, { recursive: true });
+// Small text files get inlined into the message content
+const INLINE_MAX_BYTES = 10_000;
+const INLINE_CONTENT_TYPES = ['text/', 'application/json', 'application/xml', 'application/csv'];
 export function shouldForwardMessage(messageChannelId, authorId, isBot, targetChannelId, allowedUsers) {
     if (isBot)
         return false;
@@ -8,17 +16,37 @@ 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 shouldInline(a) {
+    if (a.name?.endsWith('.txt') || a.name?.endsWith('.csv') || a.name?.endsWith('.json') || a.name?.endsWith('.md')) {
+        return (a.size || 0) <= INLINE_MAX_BYTES;
+    }
+    const ct = a.contentType || '';
+    return INLINE_CONTENT_TYPES.some(t => ct.startsWith(t)) && (a.size || 0) <= INLINE_MAX_BYTES;
+}
+// Download all attachments: small text gets inlined, everything else saved to disk.
+// Claude Code's Read tool handles images, PDFs, CSVs, notebooks, etc. natively.
+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) {
+            if (!res.ok)
+                continue;
+            if (shouldInline(attachment)) {
+                // Inline small text files directly into the message
                 const text = await res.text();
-                content = content ? `${content}\n\n${text}` : text;
+                const label = attachment.name ? `[${attachment.name}]` : '';
+                content = content ? `${content}\n\n${label}\n${text}` : `${label}\n${text}`;
+            }
+            else {
+                // Save to disk — Claude Code can Read images, PDFs, CSVs, etc.
+                const buffer = Buffer.from(await res.arrayBuffer());
+                const filename = `${message.id}-${attachment.name || 'file'}`;
+                const filepath = join(ATTACHMENT_DIR, filename);
+                writeFileSync(filepath, buffer);
+                const note = `[User sent a file: ${attachment.name || 'file'} (${formatSize(buffer.length)}). Saved to ${filepath} — use the Read tool to view it.]`;
+                content = content ? `${content}\n\n${note}` : note;
+                console.error(`[discord-filtered] Downloaded: ${filepath} (${formatSize(buffer.length)})`);
             }
         }
         catch (err) {
@@ -27,6 +55,13 @@ async function resolveTextAttachments(message) {
     }
     return content;
 }
+function formatSize(bytes) {
+    if (bytes < 1024)
+        return `${bytes}B`;
+    if (bytes < 1024 * 1024)
+        return `${(bytes / 1024).toFixed(1)}KB`;
+    return `${(bytes / (1024 * 1024)).toFixed(1)}MB`;
+}
 export function createDiscordClient(config, onMessage) {
     const client = new Client({
         intents: [
@@ -37,7 +72,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.1",
+  "version": "0.5.4",
   "description": "Decentralized on-call agent system powered by Claude Code",
   "type": "module",
   "bin": {

package/scripts/dissolve-worker.sh

@@ -64,8 +64,14 @@ if [ -n "$CHANNEL_ID" ] && [ "$CHANNEL_ID" != "null" ]; then
   echo "Discord channel deleted."
 fi
 
-# Archive worker directory
-ARCHIVE_DIR="$ONKOL_DIR/workers/.archive/${DATE}-${WORKER_NAME}"
+# Archive worker directory (append counter if archive already exists from same day)
+ARCHIVE_BASE="$ONKOL_DIR/workers/.archive/${DATE}-${WORKER_NAME}"
+ARCHIVE_DIR="$ARCHIVE_BASE"
+COUNTER=1
+while [ -d "$ARCHIVE_DIR" ]; do
+  ARCHIVE_DIR="${ARCHIVE_BASE}-${COUNTER}"
+  COUNTER=$((COUNTER + 1))
+done
 mkdir -p "$ONKOL_DIR/workers/.archive"
 mv "$WORKER_DIR" "$ARCHIVE_DIR"
 echo "Worker directory archived to $ARCHIVE_DIR"

package/src/plugin/discord-client.ts

@@ -1,4 +1,14 @@
 import { Client, GatewayIntentBits, type Message, type Attachment } from 'discord.js'
+import { mkdirSync, writeFileSync } from 'fs'
+import { join } from 'path'
+import { tmpdir } from 'os'
+
+const ATTACHMENT_DIR = join(tmpdir(), 'onkol-attachments')
+mkdirSync(ATTACHMENT_DIR, { recursive: true })
+
+// Small text files get inlined into the message content
+const INLINE_MAX_BYTES = 10_000
+const INLINE_CONTENT_TYPES = ['text/', 'application/json', 'application/xml', 'application/csv']
 
 export interface DiscordClientConfig {
   botToken: string
@@ -19,27 +29,53 @@ 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 shouldInline(a: Attachment): boolean {
+  if (a.name?.endsWith('.txt') || a.name?.endsWith('.csv') || a.name?.endsWith('.json') || a.name?.endsWith('.md')) {
+    return (a.size || 0) <= INLINE_MAX_BYTES
+  }
+  const ct = a.contentType || ''
+  return INLINE_CONTENT_TYPES.some(t => ct.startsWith(t)) && (a.size || 0) <= INLINE_MAX_BYTES
+}
+
+// Download all attachments: small text gets inlined, everything else saved to disk.
+// Claude Code's Read tool handles images, PDFs, CSVs, notebooks, etc. natively.
+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) {
+      if (!res.ok) continue
+
+      if (shouldInline(attachment)) {
+        // Inline small text files directly into the message
         const text = await res.text()
-        content = content ? `${content}\n\n${text}` : text
+        const label = attachment.name ? `[${attachment.name}]` : ''
+        content = content ? `${content}\n\n${label}\n${text}` : `${label}\n${text}`
+      } else {
+        // Save to disk — Claude Code can Read images, PDFs, CSVs, etc.
+        const buffer = Buffer.from(await res.arrayBuffer())
+        const filename = `${message.id}-${attachment.name || 'file'}`
+        const filepath = join(ATTACHMENT_DIR, filename)
+        writeFileSync(filepath, buffer)
+        const note = `[User sent a file: ${attachment.name || 'file'} (${formatSize(buffer.length)}). Saved to ${filepath} — use the Read tool to view it.]`
+        content = content ? `${content}\n\n${note}` : note
+        console.error(`[discord-filtered] Downloaded: ${filepath} (${formatSize(buffer.length)})`)
       }
     } catch (err) {
       console.error(`[discord-filtered] Failed to fetch attachment ${attachment.name}: ${err}`)
     }
   }
+
   return content
 }
 
+function formatSize(bytes: number): string {
+  if (bytes < 1024) return `${bytes}B`
+  if (bytes < 1024 * 1024) return `${(bytes / 1024).toFixed(1)}KB`
+  return `${(bytes / (1024 * 1024)).toFixed(1)}MB`
+}
+
 export function createDiscordClient(
   config: DiscordClientConfig,
   onMessage: (content: string, message: Message) => void
@@ -62,7 +98,7 @@ export function createDiscordClient(
         config.allowedUsers
       )
     ) {
-      const content = await resolveTextAttachments(message)
+      const content = await resolveAttachments(message)
       if (content) {
         onMessage(content, message)
       }