chore(results): improve error messages for invalid JSON

crates/vosk/src/recognition/batch.rs

@@ -1,6 +1,6 @@
 use super::{
     errors::AcceptWaveformError,
-    results::{result_from_json_cstr, Word},
+    results::{result_from_json_c_str, Word},
 };
 use crate::models::BatchModel;
 use vosk_sys::*;
@@ -58,7 +58,7 @@ impl BatchRecognizer {
 
     /// Gets the front of the result queue
     pub fn front_result(&mut self) -> Word {
-        unsafe { result_from_json_cstr(vosk_batch_recognizer_front_result(self.0.as_ptr())) }
+        unsafe { result_from_json_c_str(vosk_batch_recognizer_front_result(self.0.as_ptr())) }
     }
 
     /// Removes the front of the result queue

crates/vosk/src/recognition/results.rs

@@ -186,9 +186,10 @@ pub struct PartialResult<'a> {
 }
 
 /// Generic function to retrieve a given type of result from the recognizer.
-pub(super) unsafe fn result_from_json_cstr<'de, T: Deserialize<'de>>(ptr: *const c_char) -> T {
+pub(super) unsafe fn result_from_json_c_str<'de, T: Deserialize<'de>>(ptr: *const c_char) -> T {
     // UNWRAP: Panics in here will never be the caller's fault, but rather some
     // edge case that was not thought of and should be reported, so it does not
     // make sense to return a Result.
-    serde_json::from_str(CStr::from_ptr(ptr).to_str().unwrap()).unwrap()
+    let string = CStr::from_ptr(ptr).to_str().unwrap();
+    serde_json::from_str(string).unwrap_or_else(|_| panic!("Invalid JSON: {string:?}"))
 }

crates/vosk/src/recognition/sequential.rs

@@ -1,5 +1,5 @@
 use super::{
-    result_from_json_cstr, AcceptWaveformError, CompleteResult, DecodingState, PartialResult,
+    result_from_json_c_str, AcceptWaveformError, CompleteResult, DecodingState, PartialResult,
 };
 use crate::models::{Model, SpeakerModel};
 
@@ -203,7 +203,7 @@ impl Recognizer {
     /// [`CompleteResult::Single`]: crate::CompleteResult::Single
     #[must_use]
     pub fn result(&mut self) -> CompleteResult {
-        unsafe { result_from_json_cstr(vosk_recognizer_result(self.0.as_ptr())) }
+        unsafe { result_from_json_c_str(vosk_recognizer_result(self.0.as_ptr())) }
     }
 
     /// Returns partial speech recognition, which is not yet finalized and may change after
@@ -214,7 +214,7 @@ impl Recognizer {
     /// [`set_partial_words`]: Self::set_partial_words
     #[must_use]
     pub fn partial_result(&mut self) -> PartialResult {
-        unsafe { result_from_json_cstr(vosk_recognizer_partial_result(self.0.as_ptr())) }
+        unsafe { result_from_json_c_str(vosk_recognizer_partial_result(self.0.as_ptr())) }
     }
 
     /// Returns speech recognition result. Like [`result`] but it does not
@@ -223,7 +223,7 @@ impl Recognizer {
     /// [`result`]: Self::result
     #[must_use]
     pub fn final_result(&mut self) -> CompleteResult {
-        unsafe { result_from_json_cstr(vosk_recognizer_final_result(self.0.as_ptr())) }
+        unsafe { result_from_json_c_str(vosk_recognizer_final_result(self.0.as_ptr())) }
     }
 
     /// Resets current results and data so the recognition can continue from scratch