eniac-slack 0.1.14 → 0.1.16

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## [0.1.16] - 2026-03-12
+
+- fix(slack): Slack 파일 다운로드 인증 실패 - redirect 수동 처리 (#25)
+
+
+## [0.1.15] - 2026-03-12
+
+- fix(slack): 이미지 로깅 강화 + magic bytes 불일치 허용 (#24)
+
+
 ## [0.1.14] - 2026-03-12
 
 - fix(slack): sharp 정규화 + AsyncIterable로 이미지 직접 전달 (#23)

package/dist/utils/slack-files.d.ts

@@ -5,12 +5,14 @@ export interface SlackImageFile {
 }
 /**
  * Download image files from a Slack message's `files` array.
- *
- * Only images with supported media types are returned; non-image files are skipped.
  */
 export declare function downloadSlackImages(client: WebClient, files: Array<{
     url_private_download?: string;
     url_private?: string;
     mimetype?: string;
+    name?: string;
+    filetype?: string;
+    size?: number;
+    id?: string;
 }>): Promise<SlackImageFile[]>;
 //# sourceMappingURL=slack-files.d.ts.map

package/dist/utils/slack-files.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"slack-files.d.ts","sourceRoot":"","sources":["../../src/utils/slack-files.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,gBAAgB,CAAC;AAGhD,MAAM,WAAW,cAAc;IAC7B,MAAM,EAAE,MAAM,CAAC;IACf,SAAS,EAAE,YAAY,GAAG,WAAW,GAAG,WAAW,GAAG,YAAY,CAAC;CACpE;AA8CD;;;;GAIG;AACH,wBAAsB,mBAAmB,CACvC,MAAM,EAAE,SAAS,EACjB,KAAK,EAAE,KAAK,CAAC;IAAE,oBAAoB,CAAC,EAAE,MAAM,CAAC;IAAC,WAAW,CAAC,EAAE,MAAM,CAAC;IAAC,QAAQ,CAAC,EAAE,MAAM,CAAA;CAAE,CAAC,GACvF,OAAO,CAAC,cAAc,EAAE,CAAC,CAoD3B"}
+{"version":3,"file":"slack-files.d.ts","sourceRoot":"","sources":["../../src/utils/slack-files.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,gBAAgB,CAAC;AAGhD,MAAM,WAAW,cAAc;IAC7B,MAAM,EAAE,MAAM,CAAC;IACf,SAAS,EAAE,YAAY,GAAG,WAAW,GAAG,WAAW,GAAG,YAAY,CAAC;CACpE;AAmED;;GAEG;AACH,wBAAsB,mBAAmB,CACvC,MAAM,EAAE,SAAS,EACjB,KAAK,EAAE,KAAK,CAAC;IACX,oBAAoB,CAAC,EAAE,MAAM,CAAC;IAC9B,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,EAAE,CAAC,EAAE,MAAM,CAAC;CACb,CAAC,GACD,OAAO,CAAC,cAAc,EAAE,CAAC,CAkF3B"}

package/dist/utils/slack-files.js

@@ -5,94 +5,135 @@ const SUPPORTED_TYPES = new Set([
     "image/gif",
     "image/webp",
 ]);
+const MAX_IMAGE_DIMENSION = 1568; // Claude API recommended max
 /**
- * Validate that a buffer actually contains image data by checking magic bytes.
+ * Detect actual image format from magic bytes.
  */
-function isValidImage(buf, expectedMime) {
-    if (buf.length < 4)
-        return false;
-    switch (expectedMime) {
-        case "image/png":
-            return buf[0] === 0x89 && buf[1] === 0x50 && buf[2] === 0x4e && buf[3] === 0x47;
-        case "image/jpeg":
-            return buf[0] === 0xff && buf[1] === 0xd8 && buf[2] === 0xff;
-        case "image/gif":
-            return buf[0] === 0x47 && buf[1] === 0x49 && buf[2] === 0x46;
-        case "image/webp":
-            return buf.length >= 12 && buf[0] === 0x52 && buf[1] === 0x49 && buf[2] === 0x46 && buf[3] === 0x46
-                && buf[8] === 0x57 && buf[9] === 0x45 && buf[10] === 0x42 && buf[11] === 0x50;
-        default:
-            return false;
-    }
+function detectImageFormat(buf) {
+    if (buf.length < 12)
+        return null;
+    if (buf[0] === 0x89 && buf[1] === 0x50 && buf[2] === 0x4e && buf[3] === 0x47)
+        return "image/png";
+    if (buf[0] === 0xff && buf[1] === 0xd8 && buf[2] === 0xff)
+        return "image/jpeg";
+    if (buf[0] === 0x47 && buf[1] === 0x49 && buf[2] === 0x46)
+        return "image/gif";
+    if (buf[0] === 0x52 && buf[1] === 0x49 && buf[2] === 0x46 && buf[3] === 0x46
+        && buf[8] === 0x57 && buf[9] === 0x45 && buf[10] === 0x42 && buf[11] === 0x50)
+        return "image/webp";
+    return null;
 }
-const MAX_IMAGE_DIMENSION = 1568; // Claude API recommended max
 /**
- * Normalize an image buffer: resize to max dimensions and re-encode as PNG.
- * This fixes issues with exotic color spaces, corrupted chunks, and oversized images.
+ * Download a file from Slack, handling cross-origin redirects that strip the Authorization header.
+ * Slack's file URLs often redirect to a CDN, and fetch() strips auth headers on cross-origin redirects.
  */
-async function normalizeImage(buf) {
-    const normalized = await sharp(buf)
-        .resize(MAX_IMAGE_DIMENSION, MAX_IMAGE_DIMENSION, {
-        fit: "inside",
-        withoutEnlargement: true,
-    })
-        .png()
-        .toBuffer();
-    return { data: normalized, mediaType: "image/png" };
+async function fetchSlackFile(url, token) {
+    // First try: manual redirect to preserve auth through redirect chain
+    const resp = await fetch(url, {
+        headers: { Authorization: `Bearer ${token}` },
+        redirect: "manual",
+    });
+    console.log(`[slack-files] initial response: status=${resp.status}, content-type=${resp.headers.get("content-type")}, location=${resp.headers.get("location")?.substring(0, 80) ?? "none"}`);
+    // If redirect, follow it manually with auth header
+    if (resp.status >= 300 && resp.status < 400) {
+        const location = resp.headers.get("location");
+        if (location) {
+            console.log(`[slack-files] following redirect to: ${location.substring(0, 80)}...`);
+            const resp2 = await fetch(location, {
+                headers: { Authorization: `Bearer ${token}` },
+            });
+            console.log(`[slack-files] redirect response: status=${resp2.status}, content-type=${resp2.headers.get("content-type")}`);
+            const buf = Buffer.from(await resp2.arrayBuffer());
+            return { ok: resp2.ok, status: resp2.status, contentType: resp2.headers.get("content-type"), buffer: buf };
+        }
+    }
+    // No redirect or non-redirect response
+    if (resp.ok || resp.status === 200) {
+        const buf = Buffer.from(await resp.arrayBuffer());
+        return { ok: true, status: resp.status, contentType: resp.headers.get("content-type"), buffer: buf };
+    }
+    // If first approach failed (got HTML), try with token as query parameter
+    console.log(`[slack-files] Bearer auth failed (status=${resp.status}), trying token as query param`);
+    const urlWithToken = new URL(url);
+    urlWithToken.searchParams.set("t", token);
+    const resp3 = await fetch(urlWithToken.toString());
+    console.log(`[slack-files] query param response: status=${resp3.status}, content-type=${resp3.headers.get("content-type")}`);
+    const buf = Buffer.from(await resp3.arrayBuffer());
+    return { ok: resp3.ok, status: resp3.status, contentType: resp3.headers.get("content-type"), buffer: buf };
 }
 /**
  * Download image files from a Slack message's `files` array.
- *
- * Only images with supported media types are returned; non-image files are skipped.
  */
 export async function downloadSlackImages(client, files) {
     const token = client.token;
-    if (!token)
+    if (!token) {
+        console.warn("[slack-files] no token found on WebClient");
         return [];
+    }
+    console.log(`[slack-files] token type: ${token.substring(0, 5)}..., length=${token.length}`);
     const results = [];
     for (const file of files) {
         const mime = file.mimetype;
-        if (!mime || !SUPPORTED_TYPES.has(mime))
+        console.log(`[slack-files] processing file: id=${file.id}, name=${file.name}, filetype=${file.filetype}, mimetype=${mime}, size=${file.size}`);
+        if (!mime || !SUPPORTED_TYPES.has(mime)) {
+            console.log(`[slack-files] skipping unsupported mime: ${mime}`);
             continue;
+        }
         const url = file.url_private_download ?? file.url_private;
-        if (!url)
+        if (!url) {
+            console.warn(`[slack-files] no download URL for file ${file.id}`);
             continue;
+        }
+        console.log(`[slack-files] download URL: ${url}`);
         try {
-            const resp = await fetch(url, {
-                headers: { Authorization: `Bearer ${token}` },
-            });
-            if (!resp.ok) {
-                console.warn(`[slack-files] failed to download ${url}: ${resp.status}`);
+            const { ok, status, contentType, buffer: buf } = await fetchSlackFile(url, token);
+            if (!ok) {
+                console.warn(`[slack-files] download failed: status=${status}`);
                 continue;
             }
-            const buf = Buffer.from(await resp.arrayBuffer());
-            if (!isValidImage(buf, mime)) {
-                console.warn(`[slack-files] downloaded data is not a valid ${mime} image (${buf.length} bytes), skipping`);
+            const hexDump = buf.subarray(0, 16).toString("hex").match(/../g)?.join(" ") ?? "";
+            const detectedFormat = detectImageFormat(buf);
+            console.log(`[slack-files] downloaded ${buf.length} bytes, content-type=${contentType}`);
+            console.log(`[slack-files] first 16 bytes: ${hexDump}`);
+            console.log(`[slack-files] declared mime=${mime}, detected format=${detectedFormat}`);
+            // If the data looks like HTML (Slack error/login page), log and skip
+            const head = buf.subarray(0, 50).toString("utf8");
+            if (head.includes("<!DOCTYPE") || head.includes("<html")) {
+                const preview = buf.subarray(0, 300).toString("utf8");
+                console.warn(`[slack-files] downloaded data is HTML, not an image! Preview: ${preview}`);
                 continue;
             }
-            // Normalize: resize + re-encode to fix exotic formats and oversized images
+            // Normalize with sharp (handles format detection internally)
             try {
-                const { data, mediaType } = await normalizeImage(buf);
-                const b64 = data.toString("base64");
-                console.log(`[slack-files] normalized image: ${buf.length} → ${data.length} bytes, base64=${b64.length} chars, mime=${mediaType}`);
-                console.log(`[slack-files] PNG header check: ${data[0]?.toString(16)}-${data[1]?.toString(16)}-${data[2]?.toString(16)}-${data[3]?.toString(16)}`);
-                results.push({
-                    base64: b64,
-                    mediaType,
-                });
+                const metadata = await sharp(buf).metadata();
+                console.log(`[slack-files] sharp metadata: format=${metadata.format}, ${metadata.width}x${metadata.height}, channels=${metadata.channels}, space=${metadata.space}`);
+                const normalized = await sharp(buf)
+                    .resize(MAX_IMAGE_DIMENSION, MAX_IMAGE_DIMENSION, {
+                    fit: "inside",
+                    withoutEnlargement: true,
+                })
+                    .png()
+                    .toBuffer();
+                const b64 = normalized.toString("base64");
+                console.log(`[slack-files] normalized: ${buf.length} → ${normalized.length} bytes, base64=${b64.length} chars`);
+                results.push({ base64: b64, mediaType: "image/png" });
             }
-            catch (normErr) {
-                console.warn(`[slack-files] failed to normalize image, using raw:`, normErr);
-                results.push({
-                    base64: buf.toString("base64"),
-                    mediaType: mime,
-                });
+            catch (sharpErr) {
+                console.error(`[slack-files] sharp failed:`, sharpErr);
+                if (detectedFormat && SUPPORTED_TYPES.has(detectedFormat)) {
+                    console.log(`[slack-files] fallback: raw buffer with ${detectedFormat}`);
+                    results.push({
+                        base64: buf.toString("base64"),
+                        mediaType: detectedFormat,
+                    });
+                }
             }
         }
         catch (err) {
-            console.warn(`[slack-files] error downloading file:`, err);
+            console.error(`[slack-files] error downloading file ${file.id}:`, err);
         }
     }
+    console.log(`[slack-files] total images processed: ${results.length}`);
     return results;
 }
 //# sourceMappingURL=slack-files.js.map

package/dist/utils/slack-files.js.map

@@ -1 +1 @@
-{"version":3,"file":"slack-files.js","sourceRoot":"","sources":["../../src/utils/slack-files.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,MAAM,OAAO,CAAC;AAO1B,MAAM,eAAe,GAAG,IAAI,GAAG,CAAC;IAC9B,YAAY;IACZ,WAAW;IACX,WAAW;IACX,YAAY;CACb,CAAC,CAAC;AAEH;;GAEG;AACH,SAAS,YAAY,CAAC,GAAW,EAAE,YAAoB;IACrD,IAAI,GAAG,CAAC,MAAM,GAAG,CAAC;QAAE,OAAO,KAAK,CAAC;IACjC,QAAQ,YAAY,EAAE,CAAC;QACrB,KAAK,WAAW;YACd,OAAO,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,CAAC;QAClF,KAAK,YAAY;YACf,OAAO,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,CAAC;QAC/D,KAAK,WAAW;YACd,OAAO,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,CAAC;QAC/D,KAAK,YAAY;YACf,OAAO,GAAG,CAAC,MAAM,IAAI,EAAE,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI;mBAC9F,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,EAAE,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,EAAE,CAAC,KAAK,IAAI,CAAC;QAClF;YACE,OAAO,KAAK,CAAC;IACjB,CAAC;AACH,CAAC;AAED,MAAM,mBAAmB,GAAG,IAAI,CAAC,CAAC,6BAA6B;AAE/D;;;GAGG;AACH,KAAK,UAAU,cAAc,CAAC,GAAW;IACvC,MAAM,UAAU,GAAG,MAAM,KAAK,CAAC,GAAG,CAAC;SAChC,MAAM,CAAC,mBAAmB,EAAE,mBAAmB,EAAE;QAChD,GAAG,EAAE,QAAQ;QACb,kBAAkB,EAAE,IAAI;KACzB,CAAC;SACD,GAAG,EAAE;SACL,QAAQ,EAAE,CAAC;IACd,OAAO,EAAE,IAAI,EAAE,UAAU,EAAE,SAAS,EAAE,WAAW,EAAE,CAAC;AACtD,CAAC;AAED;;;;GAIG;AACH,MAAM,CAAC,KAAK,UAAU,mBAAmB,CACvC,MAAiB,EACjB,KAAwF;IAExF,MAAM,KAAK,GAAI,MAAwC,CAAC,KAAK,CAAC;IAC9D,IAAI,CAAC,KAAK;QAAE,OAAO,EAAE,CAAC;IAEtB,MAAM,OAAO,GAAqB,EAAE,CAAC;IAErC,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,MAAM,IAAI,GAAG,IAAI,CAAC,QAAQ,CAAC;QAC3B,IAAI,CAAC,IAAI,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC,IAAI,CAAC;YAAE,SAAS;QAElD,MAAM,GAAG,GAAG,IAAI,CAAC,oBAAoB,IAAI,IAAI,CAAC,WAAW,CAAC;QAC1D,IAAI,CAAC,GAAG;YAAE,SAAS;QAEnB,IAAI,CAAC;YACH,MAAM,IAAI,GAAG,MAAM,KAAK,CAAC,GAAG,EAAE;gBAC5B,OAAO,EAAE,EAAE,aAAa,EAAE,UAAU,KAAK,EAAE,EAAE;aAC9C,CAAC,CAAC;YAEH,IAAI,CAAC,IAAI,CAAC,EAAE,EAAE,CAAC;gBACb,OAAO,CAAC,IAAI,CAAC,oCAAoC,GAAG,KAAK,IAAI,CAAC,MAAM,EAAE,CAAC,CAAC;gBACxE,SAAS;YACX,CAAC;YAED,MAAM,GAAG,GAAG,MAAM,CAAC,IAAI,CAAC,MAAM,IAAI,CAAC,WAAW,EAAE,CAAC,CAAC;YAClD,IAAI,CAAC,YAAY,CAAC,GAAG,EAAE,IAAI,CAAC,EAAE,CAAC;gBAC7B,OAAO,CAAC,IAAI,CAAC,gDAAgD,IAAI,WAAW,GAAG,CAAC,MAAM,mBAAmB,CAAC,CAAC;gBAC3G,SAAS;YACX,CAAC;YAED,2EAA2E;YAC3E,IAAI,CAAC;gBACH,MAAM,EAAE,IAAI,EAAE,SAAS,EAAE,GAAG,MAAM,cAAc,CAAC,GAAG,CAAC,CAAC;gBACtD,MAAM,GAAG,GAAG,IAAI,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC;gBACpC,OAAO,CAAC,GAAG,CAAC,mCAAmC,GAAG,CAAC,MAAM,MAAM,IAAI,CAAC,MAAM,kBAAkB,GAAG,CAAC,MAAM,gBAAgB,SAAS,EAAE,CAAC,CAAC;gBACnI,OAAO,CAAC,GAAG,CAAC,mCAAmC,IAAI,CAAC,CAAC,CAAC,EAAE,QAAQ,CAAC,EAAE,CAAC,IAAI,IAAI,CAAC,CAAC,CAAC,EAAE,QAAQ,CAAC,EAAE,CAAC,IAAI,IAAI,CAAC,CAAC,CAAC,EAAE,QAAQ,CAAC,EAAE,CAAC,IAAI,IAAI,CAAC,CAAC,CAAC,EAAE,QAAQ,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC;gBACnJ,OAAO,CAAC,IAAI,CAAC;oBACX,MAAM,EAAE,GAAG;oBACX,SAAS;iBACV,CAAC,CAAC;YACL,CAAC;YAAC,OAAO,OAAO,EAAE,CAAC;gBACjB,OAAO,CAAC,IAAI,CAAC,qDAAqD,EAAE,OAAO,CAAC,CAAC;gBAC7E,OAAO,CAAC,IAAI,CAAC;oBACX,MAAM,EAAE,GAAG,CAAC,QAAQ,CAAC,QAAQ,CAAC;oBAC9B,SAAS,EAAE,IAAmC;iBAC/C,CAAC,CAAC;YACL,CAAC;QACH,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,OAAO,CAAC,IAAI,CAAC,uCAAuC,EAAE,GAAG,CAAC,CAAC;QAC7D,CAAC;IACH,CAAC;IAED,OAAO,OAAO,CAAC;AACjB,CAAC"}
+{"version":3,"file":"slack-files.js","sourceRoot":"","sources":["../../src/utils/slack-files.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,MAAM,OAAO,CAAC;AAO1B,MAAM,eAAe,GAAG,IAAI,GAAG,CAAC;IAC9B,YAAY;IACZ,WAAW;IACX,WAAW;IACX,YAAY;CACb,CAAC,CAAC;AAEH,MAAM,mBAAmB,GAAG,IAAI,CAAC,CAAC,6BAA6B;AAE/D;;GAEG;AACH,SAAS,iBAAiB,CAAC,GAAW;IACpC,IAAI,GAAG,CAAC,MAAM,GAAG,EAAE;QAAE,OAAO,IAAI,CAAC;IACjC,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI;QAAE,OAAO,WAAW,CAAC;IACjG,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI;QAAE,OAAO,YAAY,CAAC;IAC/E,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI;QAAE,OAAO,WAAW,CAAC;IAC9E,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI;WACvE,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,EAAE,CAAC,KAAK,IAAI,IAAI,GAAG,CAAC,EAAE,CAAC,KAAK,IAAI;QAAE,OAAO,YAAY,CAAC;IACrG,OAAO,IAAI,CAAC;AACd,CAAC;AAED;;;GAGG;AACH,KAAK,UAAU,cAAc,CAAC,GAAW,EAAE,KAAa;IACtD,qEAAqE;IACrE,MAAM,IAAI,GAAG,MAAM,KAAK,CAAC,GAAG,EAAE;QAC5B,OAAO,EAAE,EAAE,aAAa,EAAE,UAAU,KAAK,EAAE,EAAE;QAC7C,QAAQ,EAAE,QAAQ;KACnB,CAAC,CAAC;IAEH,OAAO,CAAC,GAAG,CAAC,0CAA0C,IAAI,CAAC,MAAM,kBAAkB,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,cAAc,CAAC,cAAc,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,UAAU,CAAC,EAAE,SAAS,CAAC,CAAC,EAAE,EAAE,CAAC,IAAI,MAAM,EAAE,CAAC,CAAC;IAE7L,mDAAmD;IACnD,IAAI,IAAI,CAAC,MAAM,IAAI,GAAG,IAAI,IAAI,CAAC,MAAM,GAAG,GAAG,EAAE,CAAC;QAC5C,MAAM,QAAQ,GAAG,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,UAAU,CAAC,CAAC;QAC9C,IAAI,QAAQ,EAAE,CAAC;YACb,OAAO,CAAC,GAAG,CAAC,wCAAwC,QAAQ,CAAC,SAAS,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,CAAC;YACpF,MAAM,KAAK,GAAG,MAAM,KAAK,CAAC,QAAQ,EAAE;gBAClC,OAAO,EAAE,EAAE,aAAa,EAAE,UAAU,KAAK,EAAE,EAAE;aAC9C,CAAC,CAAC;YACH,OAAO,CAAC,GAAG,CAAC,2CAA2C,KAAK,CAAC,MAAM,kBAAkB,KAAK,CAAC,OAAO,CAAC,GAAG,CAAC,cAAc,CAAC,EAAE,CAAC,CAAC;YAC1H,MAAM,GAAG,GAAG,MAAM,CAAC,IAAI,CAAC,MAAM,KAAK,CAAC,WAAW,EAAE,CAAC,CAAC;YACnD,OAAO,EAAE,EAAE,EAAE,KAAK,CAAC,EAAE,EAAE,MAAM,EAAE,KAAK,CAAC,MAAM,EAAE,WAAW,EAAE,KAAK,CAAC,OAAO,CAAC,GAAG,CAAC,cAAc,CAAC,EAAE,MAAM,EAAE,GAAG,EAAE,CAAC;QAC7G,CAAC;IACH,CAAC;IAED,uCAAuC;IACvC,IAAI,IAAI,CAAC,EAAE,IAAI,IAAI,CAAC,MAAM,KAAK,GAAG,EAAE,CAAC;QACnC,MAAM,GAAG,GAAG,MAAM,CAAC,IAAI,CAAC,MAAM,IAAI,CAAC,WAAW,EAAE,CAAC,CAAC;QAClD,OAAO,EAAE,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,CAAC,MAAM,EAAE,WAAW,EAAE,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,cAAc,CAAC,EAAE,MAAM,EAAE,GAAG,EAAE,CAAC;IACvG,CAAC;IAED,yEAAyE;IACzE,OAAO,CAAC,GAAG,CAAC,4CAA4C,IAAI,CAAC,MAAM,gCAAgC,CAAC,CAAC;IACrG,MAAM,YAAY,GAAG,IAAI,GAAG,CAAC,GAAG,CAAC,CAAC;IAClC,YAAY,CAAC,YAAY,CAAC,GAAG,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC;IAC1C,MAAM,KAAK,GAAG,MAAM,KAAK,CAAC,YAAY,CAAC,QAAQ,EAAE,CAAC,CAAC;IACnD,OAAO,CAAC,GAAG,CAAC,8CAA8C,KAAK,CAAC,MAAM,kBAAkB,KAAK,CAAC,OAAO,CAAC,GAAG,CAAC,cAAc,CAAC,EAAE,CAAC,CAAC;IAC7H,MAAM,GAAG,GAAG,MAAM,CAAC,IAAI,CAAC,MAAM,KAAK,CAAC,WAAW,EAAE,CAAC,CAAC;IACnD,OAAO,EAAE,EAAE,EAAE,KAAK,CAAC,EAAE,EAAE,MAAM,EAAE,KAAK,CAAC,MAAM,EAAE,WAAW,EAAE,KAAK,CAAC,OAAO,CAAC,GAAG,CAAC,cAAc,CAAC,EAAE,MAAM,EAAE,GAAG,EAAE,CAAC;AAC7G,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,mBAAmB,CACvC,MAAiB,EACjB,KAQE;IAEF,MAAM,KAAK,GAAI,MAAwC,CAAC,KAAK,CAAC;IAC9D,IAAI,CAAC,KAAK,EAAE,CAAC;QACX,OAAO,CAAC,IAAI,CAAC,2CAA2C,CAAC,CAAC;QAC1D,OAAO,EAAE,CAAC;IACZ,CAAC;IACD,OAAO,CAAC,GAAG,CAAC,6BAA6B,KAAK,CAAC,SAAS,CAAC,CAAC,EAAE,CAAC,CAAC,eAAe,KAAK,CAAC,MAAM,EAAE,CAAC,CAAC;IAE7F,MAAM,OAAO,GAAqB,EAAE,CAAC;IAErC,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,MAAM,IAAI,GAAG,IAAI,CAAC,QAAQ,CAAC;QAC3B,OAAO,CAAC,GAAG,CAAC,qCAAqC,IAAI,CAAC,EAAE,UAAU,IAAI,CAAC,IAAI,cAAc,IAAI,CAAC,QAAQ,cAAc,IAAI,UAAU,IAAI,CAAC,IAAI,EAAE,CAAC,CAAC;QAE/I,IAAI,CAAC,IAAI,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC;YACxC,OAAO,CAAC,GAAG,CAAC,4CAA4C,IAAI,EAAE,CAAC,CAAC;YAChE,SAAS;QACX,CAAC;QAED,MAAM,GAAG,GAAG,IAAI,CAAC,oBAAoB,IAAI,IAAI,CAAC,WAAW,CAAC;QAC1D,IAAI,CAAC,GAAG,EAAE,CAAC;YACT,OAAO,CAAC,IAAI,CAAC,0CAA0C,IAAI,CAAC,EAAE,EAAE,CAAC,CAAC;YAClE,SAAS;QACX,CAAC;QACD,OAAO,CAAC,GAAG,CAAC,+BAA+B,GAAG,EAAE,CAAC,CAAC;QAElD,IAAI,CAAC;YACH,MAAM,EAAE,EAAE,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,EAAE,GAAG,EAAE,GAAG,MAAM,cAAc,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC;YAElF,IAAI,CAAC,EAAE,EAAE,CAAC;gBACR,OAAO,CAAC,IAAI,CAAC,yCAAyC,MAAM,EAAE,CAAC,CAAC;gBAChE,SAAS;YACX,CAAC;YAED,MAAM,OAAO,GAAG,GAAG,CAAC,QAAQ,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,KAAK,CAAC,KAAK,CAAC,EAAE,IAAI,CAAC,GAAG,CAAC,IAAI,EAAE,CAAC;YAClF,MAAM,cAAc,GAAG,iBAAiB,CAAC,GAAG,CAAC,CAAC;YAC9C,OAAO,CAAC,GAAG,CAAC,4BAA4B,GAAG,CAAC,MAAM,wBAAwB,WAAW,EAAE,CAAC,CAAC;YACzF,OAAO,CAAC,GAAG,CAAC,iCAAiC,OAAO,EAAE,CAAC,CAAC;YACxD,OAAO,CAAC,GAAG,CAAC,+BAA+B,IAAI,qBAAqB,cAAc,EAAE,CAAC,CAAC;YAEtF,qEAAqE;YACrE,MAAM,IAAI,GAAG,GAAG,CAAC,QAAQ,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC;YAClD,IAAI,IAAI,CAAC,QAAQ,CAAC,WAAW,CAAC,IAAI,IAAI,CAAC,QAAQ,CAAC,OAAO,CAAC,EAAE,CAAC;gBACzD,MAAM,OAAO,GAAG,GAAG,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC;gBACtD,OAAO,CAAC,IAAI,CAAC,iEAAiE,OAAO,EAAE,CAAC,CAAC;gBACzF,SAAS;YACX,CAAC;YAED,6DAA6D;YAC7D,IAAI,CAAC;gBACH,MAAM,QAAQ,GAAG,MAAM,KAAK,CAAC,GAAG,CAAC,CAAC,QAAQ,EAAE,CAAC;gBAC7C,OAAO,CAAC,GAAG,CAAC,wCAAwC,QAAQ,CAAC,MAAM,KAAK,QAAQ,CAAC,KAAK,IAAI,QAAQ,CAAC,MAAM,cAAc,QAAQ,CAAC,QAAQ,WAAW,QAAQ,CAAC,KAAK,EAAE,CAAC,CAAC;gBAErK,MAAM,UAAU,GAAG,MAAM,KAAK,CAAC,GAAG,CAAC;qBAChC,MAAM,CAAC,mBAAmB,EAAE,mBAAmB,EAAE;oBAChD,GAAG,EAAE,QAAQ;oBACb,kBAAkB,EAAE,IAAI;iBACzB,CAAC;qBACD,GAAG,EAAE;qBACL,QAAQ,EAAE,CAAC;gBAEd,MAAM,GAAG,GAAG,UAAU,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC;gBAC1C,OAAO,CAAC,GAAG,CAAC,6BAA6B,GAAG,CAAC,MAAM,MAAM,UAAU,CAAC,MAAM,kBAAkB,GAAG,CAAC,MAAM,QAAQ,CAAC,CAAC;gBAEhH,OAAO,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,GAAG,EAAE,SAAS,EAAE,WAAW,EAAE,CAAC,CAAC;YACxD,CAAC;YAAC,OAAO,QAAQ,EAAE,CAAC;gBAClB,OAAO,CAAC,KAAK,CAAC,6BAA6B,EAAE,QAAQ,CAAC,CAAC;gBACvD,IAAI,cAAc,IAAI,eAAe,CAAC,GAAG,CAAC,cAAc,CAAC,EAAE,CAAC;oBAC1D,OAAO,CAAC,GAAG,CAAC,2CAA2C,cAAc,EAAE,CAAC,CAAC;oBACzE,OAAO,CAAC,IAAI,CAAC;wBACX,MAAM,EAAE,GAAG,CAAC,QAAQ,CAAC,QAAQ,CAAC;wBAC9B,SAAS,EAAE,cAA6C;qBACzD,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;QACH,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,OAAO,CAAC,KAAK,CAAC,wCAAwC,IAAI,CAAC,EAAE,GAAG,EAAE,GAAG,CAAC,CAAC;QACzE,CAAC;IACH,CAAC;IAED,OAAO,CAAC,GAAG,CAAC,yCAAyC,OAAO,CAAC,MAAM,EAAE,CAAC,CAAC;IACvE,OAAO,OAAO,CAAC;AACjB,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eniac-slack",
-  "version": "0.1.14",
+  "version": "0.1.16",
   "type": "module",
   "bin": "./dist/cli.js",
   "scripts": {

package/src/utils/slack-files.ts

@@ -13,101 +13,158 @@ const SUPPORTED_TYPES = new Set([
   "image/webp",
 ]);
 
+const MAX_IMAGE_DIMENSION = 1568; // Claude API recommended max
+
 /**
- * Validate that a buffer actually contains image data by checking magic bytes.
+ * Detect actual image format from magic bytes.
  */
-function isValidImage(buf: Buffer, expectedMime: string): boolean {
-  if (buf.length < 4) return false;
-  switch (expectedMime) {
-    case "image/png":
-      return buf[0] === 0x89 && buf[1] === 0x50 && buf[2] === 0x4e && buf[3] === 0x47;
-    case "image/jpeg":
-      return buf[0] === 0xff && buf[1] === 0xd8 && buf[2] === 0xff;
-    case "image/gif":
-      return buf[0] === 0x47 && buf[1] === 0x49 && buf[2] === 0x46;
-    case "image/webp":
-      return buf.length >= 12 && buf[0] === 0x52 && buf[1] === 0x49 && buf[2] === 0x46 && buf[3] === 0x46
-        && buf[8] === 0x57 && buf[9] === 0x45 && buf[10] === 0x42 && buf[11] === 0x50;
-    default:
-      return false;
-  }
+function detectImageFormat(buf: Buffer): string | null {
+  if (buf.length < 12) return null;
+  if (buf[0] === 0x89 && buf[1] === 0x50 && buf[2] === 0x4e && buf[3] === 0x47) return "image/png";
+  if (buf[0] === 0xff && buf[1] === 0xd8 && buf[2] === 0xff) return "image/jpeg";
+  if (buf[0] === 0x47 && buf[1] === 0x49 && buf[2] === 0x46) return "image/gif";
+  if (buf[0] === 0x52 && buf[1] === 0x49 && buf[2] === 0x46 && buf[3] === 0x46
+    && buf[8] === 0x57 && buf[9] === 0x45 && buf[10] === 0x42 && buf[11] === 0x50) return "image/webp";
+  return null;
 }
 
-const MAX_IMAGE_DIMENSION = 1568; // Claude API recommended max
-
 /**
- * Normalize an image buffer: resize to max dimensions and re-encode as PNG.
- * This fixes issues with exotic color spaces, corrupted chunks, and oversized images.
+ * Download a file from Slack, handling cross-origin redirects that strip the Authorization header.
+ * Slack's file URLs often redirect to a CDN, and fetch() strips auth headers on cross-origin redirects.
  */
-async function normalizeImage(buf: Buffer): Promise<{ data: Buffer; mediaType: SlackImageFile["mediaType"] }> {
-  const normalized = await sharp(buf)
-    .resize(MAX_IMAGE_DIMENSION, MAX_IMAGE_DIMENSION, {
-      fit: "inside",
-      withoutEnlargement: true,
-    })
-    .png()
-    .toBuffer();
-  return { data: normalized, mediaType: "image/png" };
+async function fetchSlackFile(url: string, token: string): Promise<{ ok: boolean; status: number; contentType: string | null; buffer: Buffer }> {
+  // First try: manual redirect to preserve auth through redirect chain
+  const resp = await fetch(url, {
+    headers: { Authorization: `Bearer ${token}` },
+    redirect: "manual",
+  });
+
+  console.log(`[slack-files] initial response: status=${resp.status}, content-type=${resp.headers.get("content-type")}, location=${resp.headers.get("location")?.substring(0, 80) ?? "none"}`);
+
+  // If redirect, follow it manually with auth header
+  if (resp.status >= 300 && resp.status < 400) {
+    const location = resp.headers.get("location");
+    if (location) {
+      console.log(`[slack-files] following redirect to: ${location.substring(0, 80)}...`);
+      const resp2 = await fetch(location, {
+        headers: { Authorization: `Bearer ${token}` },
+      });
+      console.log(`[slack-files] redirect response: status=${resp2.status}, content-type=${resp2.headers.get("content-type")}`);
+      const buf = Buffer.from(await resp2.arrayBuffer());
+      return { ok: resp2.ok, status: resp2.status, contentType: resp2.headers.get("content-type"), buffer: buf };
+    }
+  }
+
+  // No redirect or non-redirect response
+  if (resp.ok || resp.status === 200) {
+    const buf = Buffer.from(await resp.arrayBuffer());
+    return { ok: true, status: resp.status, contentType: resp.headers.get("content-type"), buffer: buf };
+  }
+
+  // If first approach failed (got HTML), try with token as query parameter
+  console.log(`[slack-files] Bearer auth failed (status=${resp.status}), trying token as query param`);
+  const urlWithToken = new URL(url);
+  urlWithToken.searchParams.set("t", token);
+  const resp3 = await fetch(urlWithToken.toString());
+  console.log(`[slack-files] query param response: status=${resp3.status}, content-type=${resp3.headers.get("content-type")}`);
+  const buf = Buffer.from(await resp3.arrayBuffer());
+  return { ok: resp3.ok, status: resp3.status, contentType: resp3.headers.get("content-type"), buffer: buf };
 }
 
 /**
  * Download image files from a Slack message's `files` array.
- *
- * Only images with supported media types are returned; non-image files are skipped.
  */
 export async function downloadSlackImages(
   client: WebClient,
-  files: Array<{ url_private_download?: string; url_private?: string; mimetype?: string }>
+  files: Array<{
+    url_private_download?: string;
+    url_private?: string;
+    mimetype?: string;
+    name?: string;
+    filetype?: string;
+    size?: number;
+    id?: string;
+  }>
 ): Promise<SlackImageFile[]> {
   const token = (client as unknown as { token?: string }).token;
-  if (!token) return [];
+  if (!token) {
+    console.warn("[slack-files] no token found on WebClient");
+    return [];
+  }
+  console.log(`[slack-files] token type: ${token.substring(0, 5)}..., length=${token.length}`);
 
   const results: SlackImageFile[] = [];
 
   for (const file of files) {
     const mime = file.mimetype;
-    if (!mime || !SUPPORTED_TYPES.has(mime)) continue;
+    console.log(`[slack-files] processing file: id=${file.id}, name=${file.name}, filetype=${file.filetype}, mimetype=${mime}, size=${file.size}`);
+
+    if (!mime || !SUPPORTED_TYPES.has(mime)) {
+      console.log(`[slack-files] skipping unsupported mime: ${mime}`);
+      continue;
+    }
 
     const url = file.url_private_download ?? file.url_private;
-    if (!url) continue;
+    if (!url) {
+      console.warn(`[slack-files] no download URL for file ${file.id}`);
+      continue;
+    }
+    console.log(`[slack-files] download URL: ${url}`);
 
     try {
-      const resp = await fetch(url, {
-        headers: { Authorization: `Bearer ${token}` },
-      });
+      const { ok, status, contentType, buffer: buf } = await fetchSlackFile(url, token);
 
-      if (!resp.ok) {
-        console.warn(`[slack-files] failed to download ${url}: ${resp.status}`);
+      if (!ok) {
+        console.warn(`[slack-files] download failed: status=${status}`);
         continue;
       }
 
-      const buf = Buffer.from(await resp.arrayBuffer());
-      if (!isValidImage(buf, mime)) {
-        console.warn(`[slack-files] downloaded data is not a valid ${mime} image (${buf.length} bytes), skipping`);
+      const hexDump = buf.subarray(0, 16).toString("hex").match(/../g)?.join(" ") ?? "";
+      const detectedFormat = detectImageFormat(buf);
+      console.log(`[slack-files] downloaded ${buf.length} bytes, content-type=${contentType}`);
+      console.log(`[slack-files] first 16 bytes: ${hexDump}`);
+      console.log(`[slack-files] declared mime=${mime}, detected format=${detectedFormat}`);
+
+      // If the data looks like HTML (Slack error/login page), log and skip
+      const head = buf.subarray(0, 50).toString("utf8");
+      if (head.includes("<!DOCTYPE") || head.includes("<html")) {
+        const preview = buf.subarray(0, 300).toString("utf8");
+        console.warn(`[slack-files] downloaded data is HTML, not an image! Preview: ${preview}`);
         continue;
       }
 
-      // Normalize: resize + re-encode to fix exotic formats and oversized images
+      // Normalize with sharp (handles format detection internally)
       try {
-        const { data, mediaType } = await normalizeImage(buf);
-        const b64 = data.toString("base64");
-        console.log(`[slack-files] normalized image: ${buf.length} → ${data.length} bytes, base64=${b64.length} chars, mime=${mediaType}`);
-        console.log(`[slack-files] PNG header check: ${data[0]?.toString(16)}-${data[1]?.toString(16)}-${data[2]?.toString(16)}-${data[3]?.toString(16)}`);
-        results.push({
-          base64: b64,
-          mediaType,
-        });
-      } catch (normErr) {
-        console.warn(`[slack-files] failed to normalize image, using raw:`, normErr);
-        results.push({
-          base64: buf.toString("base64"),
-          mediaType: mime as SlackImageFile["mediaType"],
-        });
+        const metadata = await sharp(buf).metadata();
+        console.log(`[slack-files] sharp metadata: format=${metadata.format}, ${metadata.width}x${metadata.height}, channels=${metadata.channels}, space=${metadata.space}`);
+
+        const normalized = await sharp(buf)
+          .resize(MAX_IMAGE_DIMENSION, MAX_IMAGE_DIMENSION, {
+            fit: "inside",
+            withoutEnlargement: true,
+          })
+          .png()
+          .toBuffer();
+
+        const b64 = normalized.toString("base64");
+        console.log(`[slack-files] normalized: ${buf.length} → ${normalized.length} bytes, base64=${b64.length} chars`);
+
+        results.push({ base64: b64, mediaType: "image/png" });
+      } catch (sharpErr) {
+        console.error(`[slack-files] sharp failed:`, sharpErr);
+        if (detectedFormat && SUPPORTED_TYPES.has(detectedFormat)) {
+          console.log(`[slack-files] fallback: raw buffer with ${detectedFormat}`);
+          results.push({
+            base64: buf.toString("base64"),
+            mediaType: detectedFormat as SlackImageFile["mediaType"],
+          });
+        }
       }
     } catch (err) {
-      console.warn(`[slack-files] error downloading file:`, err);
+      console.error(`[slack-files] error downloading file ${file.id}:`, err);
     }
   }
 
+  console.log(`[slack-files] total images processed: ${results.length}`);
   return results;
 }