support returning object as strigified JSON

hmsk · Jun 14, 2024 · ba06c7f · ba06c7f
1 parent 0f7f436
commit ba06c7f
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/ext/quickjsrb/quickjsrb.c b/ext/quickjsrb/quickjsrb.c
@@ -53,6 +53,19 @@ VALUE rb_module_eval_js_code(
   if (JS_IsException(res)) {
     rb_raise(rb_eRuntimeError, "Something happened by evaluating as JavaScript code");
     result = Qnil;
+  } else if (JS_IsObject(res)) {
+    JSValue global = JS_GetGlobalObject(ctx);
+    JSValue jsonClass = JS_GetPropertyStr(ctx, global, "JSON");
+    JSValue stringifyFunc = JS_GetPropertyStr(ctx, jsonClass, "stringify");
+    JSValue strigified = JS_Call(ctx, stringifyFunc, jsonClass, 1, &res);
+
+    const char *msg = JS_ToCString(ctx, strigified);
+    result = rb_str_new2(msg);
+
+    JS_FreeValue(ctx, global);
+    JS_FreeValue(ctx, strigified);
+    JS_FreeValue(ctx, stringifyFunc);
+    JS_FreeValue(ctx, jsonClass);
   } else if (JS_VALUE_IS_NAN(res)) {
     result = ID2SYM(rb_intern(nanId));
   } else if (JS_IsNumber(res)) {

diff --git a/test/quickjs_test.rb b/test/quickjs_test.rb
@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "test_helper"
+require "json"
 
 class QuickjsTest < Test::Unit::TestCase
   test "VERSION" do
@@ -44,6 +45,10 @@ class QuickjsTest < Test::Unit::TestCase
     assert_equal(::Quickjs.evalCode("const func = () => 1 == 3; func();"), false)
   end
 
+  test "support returning plain object" do
+    assert_equal(::Quickjs.evalCode("const func = () => ({ a: '1' }); func();"), JSON.dump({ a: '1' }))
+  end
+
   class QuickjsTestFeatures < Test::Unit::TestCase
     test "std is disabled" do
       assert_equal(::Quickjs.evalCode("typeof std === 'undefined'"), true)