fix process .isTTY detection (#3240)

Effect-TS · Jul 12, 2024 · dabd028 · dabd028
1 parent edb0da3
commit dabd028
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/.changeset/six-kangaroos-jog.md b/.changeset/six-kangaroos-jog.md
@@ -0,0 +1,5 @@
+---
+"effect": patch
+---
+
+fix process .isTTY detection
diff --git a/packages/effect/src/internal/logger.ts b/packages/effect/src/internal/logger.ts
@@ -416,9 +416,16 @@ const defaultDateFormat = (date: Date): string =>
     date.getSeconds().toString().padStart(2, "0")
   }.${date.getMilliseconds().toString().padStart(3, "0")}`
 
-const processStdoutIsTTY = typeof process === "object" && "stdout" in process && process.stdout.isTTY === true
+const processStdoutIsTTY = typeof process === "object" &&
+  process !== null &&
+  typeof process.stdout === "object" &&
+  process.stdout !== null &&
+  process.stdout.isTTY === true
 const hasWindow = typeof window === "object"
-const isWorker = typeof self === "object" && self.constructor && self.constructor.name.includes("Worker")
+const isWorker = typeof self === "object" &&
+  self !== null &&
+  typeof self.constructor === "function" &&
+  self.constructor.name.includes("Worker")
 
 /** @internal */
 export const prettyLogger = (options?: {