quickjs 0.2.3 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5f57f31bd8921f897f9d1c81d1c50d24774bbfba5d31a99528b138e82d3fad58
-  data.tar.gz: 28216a7b1711fe08ad3e2a5b021abebaadd2b5d3808584ff259d0129abf0adf6
+  metadata.gz: ecc8aa4eddd6aca97baf0e773c4880b4479b4ac4597e0edf811d256eeb61aeb9
+  data.tar.gz: 37c8eeef2cad6e0d7773e89759c7d2d1659219247883cfd1817f2603ffbf1543
 SHA512:
-  metadata.gz: 355b53297b3c27269b71dca93e057ecfe02eac70755b5bd336b212092fb6da3c14ff3b2dd2f77214a62891b28d860e33f961329374ddef485e168dea41b50097
-  data.tar.gz: 7ff16986a8ef0efeff790389805fa6d36f6b03b5983e8f8131ff33f8369514344cc9f9b9f103f036ad305441e06dfdfdacd125bdb8b061967337235d4913c525
+  metadata.gz: 2728bcbae59fae1138bdb5aef4e2f22290db93ce6c1e81f031e25c43108287a4f4d93d20afcfd9cb19a0e55593e1ccdadf1d83ebfbb430ee802368b73994cc2b
+  data.tar.gz: d06939ece851f9f4fb1311846826cad5c92e594511948ac9659bb190ebe6f905fdce15b4b9d16cf5df3b08bc5483a6f27d847f3ddf752aaa18bfcf74fec087e1

data/ext/quickjsrb/quickjsrb.c

@@ -1,22 +1,22 @@
 #include "quickjsrb.h"
 
-JSValue to_js_value(JSContext *ctx, VALUE r_value)
+JSValue j_error_from_ruby_error(JSContext *ctx, VALUE r_error)
 {
-  if (RTEST(rb_funcall(
-          r_value,
-          rb_intern("is_a?"),
-          1, rb_const_get(rb_cClass, rb_intern("Exception")))))
-  {
-    JSValue j_error = JS_NewError(ctx);
-    VALUE r_str = rb_funcall(r_value, rb_intern("message"), 0, NULL);
-    char *exceptionMessage = StringValueCStr(r_str);
-    VALUE r_exception_name = rb_funcall(rb_funcall(r_value, rb_intern("class"), 0, NULL), rb_intern("name"), 0, NULL);
-    char *exceptionName = StringValueCStr(r_exception_name);
-    JS_SetPropertyStr(ctx, j_error, "name", JS_NewString(ctx, exceptionName));
-    JS_SetPropertyStr(ctx, j_error, "message", JS_NewString(ctx, exceptionMessage));
-    return JS_Throw(ctx, j_error);
-  }
+  JSValue j_error = JS_NewError(ctx); // may wanna have custom error class to determine in JS' end
+
+  VALUE r_object_id = rb_funcall(r_error, rb_intern("object_id"), 0, NULL);
+  int objectId = NUM2INT(r_object_id);
+  JS_SetPropertyStr(ctx, j_error, "rb_object_id", JS_NewInt32(ctx, objectId));
+
+  VALUE r_exception_message = rb_funcall(r_error, rb_intern("message"), 0, NULL);
+  const char *errorMessage = StringValueCStr(r_exception_message);
+  JS_SetPropertyStr(ctx, j_error, "message", JS_NewString(ctx, errorMessage));
 
+  return j_error;
+}
+
+JSValue to_js_value(JSContext *ctx, VALUE r_value)
+{
   switch (TYPE(r_value))
   {
   case T_NIL:
@@ -72,6 +72,13 @@ JSValue to_js_value(JSContext *ctx, VALUE r_value)
   }
   default:
   {
+    if (TYPE(r_value) == T_OBJECT && RTEST(rb_funcall(
+                                         r_value,
+                                         rb_intern("is_a?"),
+                                         1, rb_const_get(rb_cClass, rb_intern("Exception")))))
+    {
+      return j_error_from_ruby_error(ctx, r_value);
+    }
     VALUE r_inspect_str = rb_funcall(r_value, rb_intern("inspect"), 0, NULL);
     char *str = StringValueCStr(r_inspect_str);
 
@@ -80,6 +87,11 @@ JSValue to_js_value(JSContext *ctx, VALUE r_value)
   }
 }
 
+VALUE r_try_json_parse(VALUE r_str)
+{
+  return rb_funcall(rb_const_get(rb_cClass, rb_intern("JSON")), rb_intern("parse"), 1, r_str);
+}
+
 VALUE to_rb_value(JSContext *ctx, JSValue j_val)
 {
   switch (JS_VALUE_GET_NORM_TAG(j_val))
@@ -121,6 +133,23 @@ VALUE to_rb_value(JSContext *ctx, JSValue j_val)
       return Qnil;
     }
 
+    if (JS_IsError(ctx, j_val))
+    {
+      JSValue j_errorOriginalRubyObjectId = JS_GetPropertyStr(ctx, j_val, "rb_object_id");
+      int errorOriginalRubyObjectId = 0;
+      if (JS_VALUE_GET_NORM_TAG(j_errorOriginalRubyObjectId) == JS_TAG_INT)
+      {
+        JS_ToInt32(ctx, &errorOriginalRubyObjectId, j_errorOriginalRubyObjectId);
+        JS_FreeValue(ctx, j_errorOriginalRubyObjectId);
+        if (errorOriginalRubyObjectId > 0)
+        {
+          // may be nice if cover the case of object is missing
+          return rb_funcall(rb_const_get(rb_cClass, rb_intern("ObjectSpace")), rb_intern("_id2ref"), 1, INT2NUM(errorOriginalRubyObjectId));
+        }
+      }
+      // will support other errors
+    }
+
     JSValue j_global = JS_GetGlobalObject(ctx);
     JSValue j_jsonClass = JS_GetPropertyStr(ctx, j_global, "JSON");
     JSValue j_stringifyFunc = JS_GetPropertyStr(ctx, j_jsonClass, "stringify");
@@ -135,7 +164,21 @@ VALUE to_rb_value(JSContext *ctx, JSValue j_val)
     JS_FreeValue(ctx, j_jsonClass);
     JS_FreeValue(ctx, j_global);
 
-    return rb_funcall(rb_const_get(rb_cClass, rb_intern("JSON")), rb_intern("parse"), 1, r_str);
+    if (rb_funcall(r_str, rb_intern("=="), 1, rb_str_new2("undefined")))
+    {
+      return QUICKJSRB_SYM(undefinedId);
+    }
+
+    int couldntParse;
+    VALUE r_result = rb_protect(r_try_json_parse, r_str, &couldntParse);
+    if (couldntParse)
+    {
+      return Qnil;
+    }
+    else
+    {
+      return r_result;
+    }
   }
   case JS_TAG_NULL:
     return Qnil;
@@ -146,6 +189,21 @@ VALUE to_rb_value(JSContext *ctx, JSValue j_val)
     JSValue j_exceptionVal = JS_GetException(ctx);
     if (JS_IsError(ctx, j_exceptionVal))
     {
+      JSValue j_errorOriginalRubyObjectId = JS_GetPropertyStr(ctx, j_exceptionVal, "rb_object_id");
+      int errorOriginalRubyObjectId = 0;
+      if (JS_VALUE_GET_NORM_TAG(j_errorOriginalRubyObjectId) == JS_TAG_INT)
+      {
+        JS_ToInt32(ctx, &errorOriginalRubyObjectId, j_errorOriginalRubyObjectId);
+        JS_FreeValue(ctx, j_errorOriginalRubyObjectId);
+        if (errorOriginalRubyObjectId > 0)
+        {
+          JS_FreeValue(ctx, j_exceptionVal);
+          // may be nice if cover the case of object is missing
+          rb_exc_raise(rb_funcall(rb_const_get(rb_cClass, rb_intern("ObjectSpace")), rb_intern("_id2ref"), 1, INT2NUM(errorOriginalRubyObjectId)));
+          return Qnil;
+        }
+      }
+
       JSValue j_errorClassName = JS_GetPropertyStr(ctx, j_exceptionVal, "name");
       const char *errorClassName = JS_ToCString(ctx, j_errorClassName);
 
@@ -165,11 +223,6 @@ VALUE to_rb_value(JSContext *ctx, JSValue j_val)
         r_error_class = QUICKJSRB_ERROR_FOR(QUICKJSRB_INTERRUPTED_ERROR);
         r_error_message = rb_str_new2("Code evaluation is interrupted by the timeout or something");
       }
-      else if (strcmp(errorClassName, "InternalError") == 0 && strstr(errorClassMessage, "Ruby") != NULL)
-      {
-        r_error_class = QUICKJSRB_ERROR_FOR(QUICKJSRB_RUBY_FUNCTION_ERROR);
-      }
-
       else if (strcmp(errorClassName, "Quickjs::InterruptedError") == 0)
       {
         r_error_class = QUICKJSRB_ERROR_FOR(QUICKJSRB_INTERRUPTED_ERROR);
@@ -251,7 +304,9 @@ static JSValue js_quickjsrb_call_global(JSContext *ctx, JSValueConst _this, int
   JSValue j_result;
   if (sadnessHappened)
   {
-    j_result = JS_ThrowInternalError(ctx, "unintentional error is raised while executing the function by Ruby: `%s`", funcName);
+    VALUE r_error = rb_errinfo();
+    JSValue j_error = j_error_from_ruby_error(ctx, r_error);
+    return JS_Throw(ctx, j_error);
   }
   else
   {
@@ -475,13 +530,14 @@ static VALUE vm_m_import(int argc, VALUE *argv, VALUE r_self)
   rb_scan_args(argc, argv, "10:", &r_import_string, &r_opts);
   if (NIL_P(r_opts))
     r_opts = rb_hash_new();
-  VALUE r_from = rb_hash_aref(r_opts, ID2SYM(rb_intern("from"))); // TODO: Use kwargs instead
+  VALUE r_from = rb_hash_aref(r_opts, ID2SYM(rb_intern("from")));
   if (NIL_P(r_from))
   {
     VALUE r_error_message = rb_str_new2("missing import source");
     rb_exc_raise(rb_funcall(QUICKJSRB_ERROR_FOR(QUICKJSRB_ROOT_RUNTIME_ERROR), rb_intern("new"), 2, r_error_message, Qnil));
     return Qnil;
   }
+  VALUE r_custom_exposure = rb_hash_aref(r_opts, ID2SYM(rb_intern("code_to_expose")));
 
   VMData *data;
   TypedData_Get_Struct(r_self, VMData, &vm_type, data);
@@ -499,8 +555,16 @@ static VALUE vm_m_import(int argc, VALUE *argv, VALUE r_self)
       r_import_string);
   VALUE r_import_name = rb_ary_entry(r_import_settings, 0);
   char *import_name = StringValueCStr(r_import_name);
-  VALUE r_globalize = rb_ary_entry(r_import_settings, 1);
-  char *globalize = StringValueCStr(r_globalize);
+  VALUE r_default_exposure = rb_ary_entry(r_import_settings, 1);
+  char *globalize;
+  if (RTEST(r_custom_exposure))
+  {
+    globalize = StringValueCStr(r_custom_exposure);
+  }
+  else
+  {
+    globalize = StringValueCStr(r_default_exposure);
+  }
 
   const char *importAndGlobalizeModule = "import %s from '%s';\n"
                                          "%s\n";

data/ext/quickjsrb/quickjsrb.h

@@ -183,7 +183,6 @@ static VALUE r_define_log_class(VALUE r_parent_class)
 #define QUICKJSRB_ROOT_RUNTIME_ERROR "RuntimeError"
 #define QUICKJSRB_INTERRUPTED_ERROR "InterruptedError"
 #define QUICKJSRB_NO_AWAIT_ERROR "NoAwaitError"
-#define QUICKJSRB_RUBY_FUNCTION_ERROR "RubyFunctionError"
 
 #define QUICKJSRB_ERROR_FOR(name) \
   (VALUE) { rb_const_get(rb_const_get(rb_cClass, rb_intern("Quickjs")), rb_intern(name)) }
@@ -212,7 +211,6 @@ static void r_define_exception_classes(VALUE r_parent_class)
   // quickjsrb specific errors
   rb_define_class_under(r_parent_class, QUICKJSRB_INTERRUPTED_ERROR, r_runtime_error);
   rb_define_class_under(r_parent_class, QUICKJSRB_NO_AWAIT_ERROR, r_runtime_error);
-  rb_define_class_under(r_parent_class, QUICKJSRB_RUBY_FUNCTION_ERROR, r_runtime_error);
 }
 
 #endif /* QUICKJSRB_H */

data/lib/quickjs/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Quickjs
-  VERSION = "0.2.3"
+  VERSION = "0.3.0"
 end

data/lib/quickjs.rb

@@ -16,7 +16,7 @@ module Quickjs
   def _with_timeout(msec, proc, args)
     Timeout.timeout(msec / 1_000.0) { proc.call(*args) }
   rescue Timeout::Error
-    Quickjs::InterruptedError.new('Ruby runtime got timeout', nil)
+    raise Quickjs::InterruptedError.new('Ruby runtime got timeout', nil)
   rescue
     raise
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: quickjs
 version: !ruby/object:Gem::Version
-  version: 0.2.3
+  version: 0.3.0
 platform: ruby
 authors:
 - hmsk
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-10-31 00:00:00.000000000 Z
+date: 2024-11-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json