obol-ai 0.3.36 → 0.3.37

package/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.3.37
+- changelog
+- fix media group race condition — buffer before download
+
 ## 0.3.36
 - save media assets instantly in date-based folders
 - fix agentic event raw JSON output and update_event wiping instructions

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "obol-ai",
-  "version": "0.3.36",
+  "version": "0.3.37",
   "description": "Self-evolving AI assistant that learns, remembers, and acts on its own. Persistent vector memory, self-rewriting personality, proactive heartbeats.",
   "main": "src/index.js",
   "bin": {

package/src/telegram/constants.js

@@ -9,7 +9,7 @@ const TEXT_BUFFER_GAP_MS = 1500;
 const TEXT_BUFFER_MAX_PARTS = 12;
 const TEXT_BUFFER_MAX_CHARS = 50000;
 const TEXT_BUFFER_THRESHOLD = 4000;
-const MEDIA_GROUP_DELAY_MS = 500;
+const MEDIA_GROUP_DELAY_MS = 5000;
 const MAX_MEDIA_SIZE = 50 * 1024 * 1024;
 const TERM_SEP = '━'.repeat(TERM_WIDTH);
 

package/src/telegram/handlers/media.js

@@ -16,6 +16,29 @@ async function downloadMediaItem(ctx, fileInfo, telegramToken) {
   return { buffer, filename, fileInfo, caption: ctx.message.caption || '' };
 }
 
+async function downloadAndProcess(ctx, entries, deps, token) {
+  const userId = ctx.from.id;
+  const userDir = ensureUserDir(userId);
+  const tz = getUserTimezone(deps.config, userId);
+  const today = new Date().toLocaleDateString('en-CA', { timeZone: tz });
+  const assetsDir = path.join(userDir, 'assets', today);
+
+  const items = (await Promise.all(
+    entries.map(({ ctx: entryCtx, fileInfo }) =>
+      downloadMediaItem(entryCtx, fileInfo, token).then(item => {
+        if (item) item.savedPath = media.saveFile(item.buffer, assetsDir, item.filename);
+        return item;
+      }).catch(e => {
+        console.error('Media download error:', e.message);
+        return null;
+      })
+    )
+  )).filter(Boolean);
+
+  if (items.length === 0) return;
+  await processMediaItems(ctx, items, deps);
+}
+
 async function processMediaItems(ctx, items, { config, allowedUsers, bot, createAsk }) {
   if (!ctx.from) return;
   const userId = ctx.from.id;
@@ -154,38 +177,26 @@ function registerMediaHandler(bot, telegramConfig, deps) {
       return;
     }
 
-    const item = await downloadMediaItem(ctx, fileInfo, telegramConfig.token).catch(e => {
-      console.error('Media download error:', e.message);
-      return null;
-    });
-    if (!item) return;
-
-    const userDir = ensureUserDir(userId);
-    const tz = getUserTimezone(deps.config, userId);
-    const today = new Date().toLocaleDateString('en-CA', { timeZone: tz });
-    const assetsDir = path.join(userDir, 'assets', today);
-    item.savedPath = media.saveFile(item.buffer, assetsDir, item.filename);
-
     const groupId = ctx.message.media_group_id;
     if (groupId) {
       const existing = mediaGroups.get(groupId);
       if (existing) {
         clearTimeout(existing.timer);
-        existing.items.push(item);
-        existing.ctx = ctx;
+        existing.entries.push({ ctx, fileInfo });
+        existing.latestCtx = ctx;
         existing.timer = setTimeout(() => {
           mediaGroups.delete(groupId);
-          processMediaItems(existing.ctx, existing.items, deps).catch(e =>
+          downloadAndProcess(existing.latestCtx, existing.entries, deps, telegramConfig.token).catch(e =>
             console.error('Media group error:', e.message)
           );
         }, MEDIA_GROUP_DELAY_MS);
       } else {
         const group = {
-          items: [item],
-          ctx,
+          entries: [{ ctx, fileInfo }],
+          latestCtx: ctx,
           timer: setTimeout(() => {
             mediaGroups.delete(groupId);
-            processMediaItems(ctx, [item], deps).catch(e =>
+            downloadAndProcess(ctx, [{ ctx, fileInfo }], deps, telegramConfig.token).catch(e =>
               console.error('Media group error:', e.message)
             );
           }, MEDIA_GROUP_DELAY_MS),
@@ -195,6 +206,18 @@ function registerMediaHandler(bot, telegramConfig, deps) {
       return;
     }
 
+    const item = await downloadMediaItem(ctx, fileInfo, telegramConfig.token).catch(e => {
+      console.error('Media download error:', e.message);
+      return null;
+    });
+    if (!item) return;
+
+    const userDir = ensureUserDir(userId);
+    const tz = getUserTimezone(deps.config, userId);
+    const today = new Date().toLocaleDateString('en-CA', { timeZone: tz });
+    const assetsDir = path.join(userDir, 'assets', today);
+    item.savedPath = media.saveFile(item.buffer, assetsDir, item.filename);
+
     await processMediaItems(ctx, [item], deps);
   }