chore: improve check_interpreter_abort_condition error conversion

federico-stacks · federico-stacks · commit 55ef717f725a · 2026-07-06T10:58:14.000+02:00
diff --git a/clarity/src/vm/mod.rs b/clarity/src/vm/mod.rs
@@ -494,10 +494,9 @@ fn check_interpreter_abort_condition(
     if global_context.execution_time_tracker.is_expired() {
         return Err(RuntimeCheckErrorKind::ExecutionTimeExpired.into());
     }
+    
     if let Err(reason) = global_context.abort_callback.check() {
-        return Err(VmExecutionError::RuntimeCheck(
-            RuntimeCheckErrorKind::AbortedByExecutionHook(reason),
-        ));
+        return Err(RuntimeCheckErrorKind::AbortedByExecutionHook(reason).into());
     }
 
     Ok(())

Original file line number	Diff line number	Diff line change
`@@ -494,10 +494,9 @@ fn check_interpreter_abort_condition(`
`494`	`494`	`if global_context.execution_time_tracker.is_expired() {`
`495`	`495`	`return Err(RuntimeCheckErrorKind::ExecutionTimeExpired.into());`
`496`	`496`	`}`
	`497`	`+`
`497`	`498`	`if let Err(reason) = global_context.abort_callback.check() {`
`498`		`- return Err(VmExecutionError::RuntimeCheck(`
`499`		`- RuntimeCheckErrorKind::AbortedByExecutionHook(reason),`
`500`		`- ));`
	`499`	`+ return Err(RuntimeCheckErrorKind::AbortedByExecutionHook(reason).into());`
`501`	`500`	`}`
`502`	`501`
`503`	`502`	`Ok(())`