build: use @slack/logger for debugging parsechunks

srtaalej · srtaalej · commit dbca9c953d18 · 2026-01-22T13:27:39.000-05:00
diff --git a/packages/types/package.json b/packages/types/package.json
@@ -27,6 +27,9 @@
   "bugs": {
     "url": "https://github.com/slackapi/node-slack-sdk/issues"
   },
+  "dependencies": {
+    "@slack/logger": "^4.0.0"
+  },
   "scripts": {
     "prepare": "npm run build",
     "build": "npm run build:clean && tsc",
diff --git a/packages/types/src/chunk.ts b/packages/types/src/chunk.ts
@@ -1,3 +1,8 @@
+import { ConsoleLogger, LogLevel } from '@slack/logger';
+
+const logger = new ConsoleLogger();
+logger.setLevel(LogLevel.DEBUG);
+
 /**
  * Base interface for streaming message chunks.
  * https://docs.slack.dev/messaging/sending-and-scheduling-messages#text-streaming
@@ -65,7 +70,7 @@ export function parseChunk(chunk: unknown): AnyChunk | null {
   const chunkObj = chunk as Record<string, unknown>;
 
   if (!('type' in chunkObj) || typeof chunkObj.type !== 'string') {
-    console.warn('Unknown chunk detected and skipped (missing type)', chunk);
+    logger.debug('Unknown chunk detected and skipped (missing type)', chunk);
     return null;
   }
 
@@ -75,15 +80,15 @@ export function parseChunk(chunk: unknown): AnyChunk | null {
     if (typeof chunkObj.text === 'string') {
       return chunkObj as unknown as MarkdownTextChunk;
     }
-    console.warn('Invalid MarkdownTextChunk (missing text property)', chunk);
+    logger.debug('Invalid MarkdownTextChunk (missing text property)', chunk);
     return null;
   }
 
-  if(type === 'plan_update') {
+  if (type === 'plan_update') {
     if (typeof chunkObj.title === 'string') {
       return chunkObj as unknown as PlanUpdateChunk;
     }
-    console.warn('Invalid PlanUpdateChunk (missing title property)', chunk);
+    logger.debug('Invalid PlanUpdateChunk (missing title property)', chunk);
     return null;
   }
 
@@ -97,10 +102,10 @@ export function parseChunk(chunk: unknown): AnyChunk | null {
     ) {
       return chunkObj as unknown as TaskUpdateChunk;
     }
-    console.warn('Invalid TaskUpdateChunk (missing required properties)', chunk);
+    logger.debug('Invalid TaskUpdateChunk (missing required properties)', chunk);
     return null;
   }
 
-  console.warn(`Unknown chunk type detected and skipped: ${type}`, chunk);
+  logger.debug(`Unknown chunk type detected and skipped: ${type}`, chunk);
   return null;
 }

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,8 @@`
	`1`	`+import { ConsoleLogger, LogLevel } from '@slack/logger';`
	`2`	`+`
	`3`	`+const logger = new ConsoleLogger();`
	`4`	`+logger.setLevel(LogLevel.DEBUG);`
	`5`	`+`
`1`	`6`	`/**`
`2`	`7`	`* Base interface for streaming message chunks.`
`3`	`8`	`* https://docs.slack.dev/messaging/sending-and-scheduling-messages#text-streaming`
`@@ -65,7 +70,7 @@ export function parseChunk(chunk: unknown): AnyChunk \| null {`
`65`	`70`	`const chunkObj = chunk as Record<string, unknown>;`
`66`	`71`
`67`	`72`	`if (!('type' in chunkObj) \|\| typeof chunkObj.type !== 'string') {`
`68`		`- console.warn('Unknown chunk detected and skipped (missing type)', chunk);`
	`73`	`+ logger.debug('Unknown chunk detected and skipped (missing type)', chunk);`
`69`	`74`	`return null;`
`70`	`75`	`}`
`71`	`76`
`@@ -75,15 +80,15 @@ export function parseChunk(chunk: unknown): AnyChunk \| null {`
`75`	`80`	`if (typeof chunkObj.text === 'string') {`
`76`	`81`	`return chunkObj as unknown as MarkdownTextChunk;`
`77`	`82`	`}`
`78`		`- console.warn('Invalid MarkdownTextChunk (missing text property)', chunk);`
	`83`	`+ logger.debug('Invalid MarkdownTextChunk (missing text property)', chunk);`
`79`	`84`	`return null;`
`80`	`85`	`}`
`81`	`86`
`82`		`- if(type === 'plan_update') {`
	`87`	`+ if (type === 'plan_update') {`
`83`	`88`	`if (typeof chunkObj.title === 'string') {`
`84`	`89`	`return chunkObj as unknown as PlanUpdateChunk;`
`85`	`90`	`}`
`86`		`- console.warn('Invalid PlanUpdateChunk (missing title property)', chunk);`
	`91`	`+ logger.debug('Invalid PlanUpdateChunk (missing title property)', chunk);`
`87`	`92`	`return null;`
`88`	`93`	`}`
`89`	`94`
`@@ -97,10 +102,10 @@ export function parseChunk(chunk: unknown): AnyChunk \| null {`
`97`	`102`	`) {`
`98`	`103`	`return chunkObj as unknown as TaskUpdateChunk;`
`99`	`104`	`}`
`100`		`- console.warn('Invalid TaskUpdateChunk (missing required properties)', chunk);`
	`105`	`+ logger.debug('Invalid TaskUpdateChunk (missing required properties)', chunk);`
`101`	`106`	`return null;`
`102`	`107`	`}`
`103`	`108`
`104`		- console.warn(`Unknown chunk type detected and skipped: ${type}`, chunk);
	`109`	+ logger.debug(`Unknown chunk type detected and skipped: ${type}`, chunk);
`105`	`110`	`return null;`
`106`	`111`	`}`