update error handler

Picovoice · Oct 12, 2023 · b4533d8 · b4533d8
1 parent add74af
commit b4533d8
Showing 1 changed file with 13 additions and 3 deletions.
diff --git a/binding/nodejs/src/porcupine.ts b/binding/nodejs/src/porcupine.ts
@@ -132,6 +132,8 @@ export default class Porcupine {
     }
 
     const pvPorcupine = require(libraryPath); // eslint-disable-line
+    this._pvPorcupine = pvPorcupine;
+
     let porcupineHandleAndStatus: PorcupineHandleAndStatus | null = null;
     try {
       pvPorcupine.set_sdk("node");
@@ -149,11 +151,10 @@ export default class Porcupine {
 
     const status = porcupineHandleAndStatus!.status;
     if (status !== PvStatus.SUCCESS) {
-      pvStatusToException(status, "Porcupine failed to initialize", pvPorcupine.get_error_stack());
+      this.handlePvStatus(status, "Porcupine failed to initialize");
     }
 
     this._handle = porcupineHandleAndStatus!.handle;
-    this._pvPorcupine = pvPorcupine;
     this._frameLength = pvPorcupine.frame_length();
     this._sampleRate = pvPorcupine.sample_rate();
     this._version = pvPorcupine.version();
@@ -229,7 +230,7 @@ export default class Porcupine {
 
     const status = keywordAndStatus!.status;
     if (status !== PvStatus.SUCCESS) {
-      pvStatusToException(status, "Porcupine failed to process the frame", this._pvPorcupine.get_error_stack());
+      this.handlePvStatus(status, "Porcupine failed to process");
     }
     const keywordIndex = keywordAndStatus!.keyword_index;
 
@@ -251,4 +252,13 @@ export default class Porcupine {
       console.warn("Porcupine is not initialized; nothing to destroy");
     }
   }
+
+  private handlePvStatus(status: PvStatus, message: string) {
+    const errorObject = this._pvPorcupine.get_error_stack();
+    if (errorObject.status === PvStatus.SUCCESS) {
+      pvStatusToException(status, message, errorObject.message_stack);
+    } else {
+      pvStatusToException(status, "Unable to get Porcupine error state");
+    }
+  }
 }