chore(deps): bump @slack/web-api from 6.12.0 to 7.3.4 (#280)

Fixes #279 --------- Signed-off-by: github-actions <[email protected]> Co-authored-by: github-actions <[email protected]>
jogold · Aug 19, 2024 · fe6e7b6 · fe6e7b6
1 parent e5b615e
commit fe6e7b6
Show file tree

Hide file tree

Showing 5 changed files with 67 additions and 80 deletions.
diff --git a/package.json b/package.json
diff --git a/src/slack-textract/detect.lambda.ts b/src/slack-textract/detect.lambda.ts
@@ -1,28 +1,13 @@
 /* eslint-disable no-console */
 import { DetectDocumentTextCommand, TextractClient } from '@aws-sdk/client-textract';
-import { WebClient, WebAPICallResult } from '@slack/web-api';
+import { WebClient } from '@slack/web-api';
 import got from 'got';
 
 export interface SlackEvent {
   channel_id: string;
   file_id: string;
 }
 
-export interface FilesInfoResult extends WebAPICallResult {
-  file: {
-    mimetype: string;
-    filetype: string;
-    url_private: string;
-    shares: {
-      public: {
-        [channel: string]: [{
-          ts: string;
-        }];
-      };
-    };
-  };
-}
-
 const textractClient = new TextractClient({});
 
 export async function handler(event: SlackEvent): Promise<void> {
@@ -33,14 +18,24 @@ export async function handler(event: SlackEvent): Promise<void> {
   // Get file info
   const info = await slackClient.files.info({
     file: event.file_id,
-  }) as FilesInfoResult;
+  });
   console.log('File info: %j', info);
 
-  if (!info.file.mimetype.startsWith('image')) {
+  if (!info.file) {
+    console.log('No file');
+    return;
+  }
+
+  if (!info.file.mimetype?.startsWith('image')) {
     console.log('Not an image');
     return;
   }
 
+  if (!info.file.url_private) {
+    console.log('No private URL');
+    return;
+  }
+
   // Get file
   const file = await got(info.file.url_private, {
     headers: {
@@ -62,7 +57,7 @@ export async function handler(event: SlackEvent): Promise<void> {
   const postMessage = await slackClient.chat.postMessage({
     channel: event.channel_id,
     text: data.Blocks.filter((b) => b.BlockType === 'LINE').map((b) => b.Text).join('\n'),
-    thread_ts: info.file.shares.public[event.channel_id][0].ts,
+    thread_ts: info.file.shares?.public?.[event.channel_id][0].ts,
   });
   console.log('Post message: %j', postMessage);
 }
diff --git a/test/slack-textract/__snapshots__/slack-textract.test.ts.snap b/test/slack-textract/__snapshots__/slack-textract.test.ts.snap
diff --git a/test/slack-textract/handler.test.ts b/test/slack-textract/handler.test.ts
@@ -1,9 +1,9 @@
 import 'aws-sdk-client-mock-jest';
 import { DetectDocumentTextCommand, TextractClient } from '@aws-sdk/client-textract';
-import { WebClient } from '@slack/web-api';
+import { FilesInfoResponse, WebClient } from '@slack/web-api';
 import { mockClient } from 'aws-sdk-client-mock';
 import got from 'got';
-import { handler, FilesInfoResult } from '../../src/slack-textract/detect.lambda';
+import { handler } from '../../src/slack-textract/detect.lambda';
 
 process.env.SLACK_TOKEN = 'token';
 
@@ -18,7 +18,7 @@ beforeEach(() => {
 });
 
 test('handler', async () => {
-  const fileInfo: FilesInfoResult = {
+  const fileInfo: FilesInfoResponse = {
     ok: true,
     file: {
       mimetype: 'image/jpg',

diff --git a/yarn.lock b/yarn.lock