Merge pull request #270 from Portkey-AI/feat/envCheck

FS Module Check

Author: csgulati09

src/apis/audio.ts

@@ -17,7 +17,7 @@ import { SpeechCreateParams } from 'openai/resources/audio/speech';
 import { Stream } from '../streaming';
 import { AUDIO_FILE_DURATION_HEADER } from '../constants';
 import getAudioFileDuration from '../getAudioDuration';
-import { isRunningInBrowser } from '../core';
+import { isFsModuleAvailable } from '../core';
 
 export class Audio extends ApiResource {
   transcriptions: transcriptions;
@@ -78,7 +78,8 @@ export class transcriptions extends ApiResource {
   > {
     // @ts-ignore
     const path = body.file?.path;
-    if (path && this.client.calculateAudioDuration && !isRunningInBrowser()) {
+
+    if (path && this.client.calculateAudioDuration && isFsModuleAvailable()) {
       const duration = await getAudioFileDuration(path);
       if (duration) {
         params = {
@@ -110,7 +111,8 @@ export class translations extends ApiResource {
   ): Promise<any> {
     const body: any = _body;
     const path = body.file?.path;
-    if (path && this.client.calculateAudioDuration && !isRunningInBrowser()) {
+
+    if (path && this.client.calculateAudioDuration && isFsModuleAvailable()) {
       const duration = await getAudioFileDuration(path);
       if (duration) {
         params = {

src/core.ts

@@ -59,3 +59,12 @@ export const isRunningInBrowser = () => {
     typeof navigator !== 'undefined'
   );
 };
+
+export const isFsModuleAvailable = () => {
+  try {
+    require('fs');
+    return true;
+  } catch (e) {
+    return false;
+  }
+};

src/getAudioDuration.ts

@@ -1,14 +1,15 @@
-import { isRunningInBrowser } from './core';
+import { isFsModuleAvailable } from './core';
 
-const isBrowser = isRunningInBrowser();
+const fsAvailable = isFsModuleAvailable();
 
 let fs: any;
 let open: any;
 let read: any;
 let stat: any;
 let close: any;
 
-if (!isBrowser) {
+// Check if we're in Node.js AND fs module is available
+if (fsAvailable) {
   try {
     fs = require('fs');
     const { promisify } = require('util');
@@ -27,7 +28,8 @@ if (!isBrowser) {
  * Uses optimized file reading to avoid loading entire files into memory
  */
 async function getAudioFileDuration(filePath: string): Promise<string | null> {
-  if (isBrowser || !fs) {
+  // Only proceed if we're in Node.js, fs is available, and fs module loaded successfully
+  if (!fsAvailable || !fs) {
     return null;
   }