Merge pull request #1037 from lightpanda-io/upgrade_v8

Upgrade v8

Author: karlseguin

.github/actions/install/action.yml

@@ -17,11 +17,11 @@ inputs:
   zig-v8:
     description: 'zig v8 version to install'
     required: false
-    default: 'v0.1.28'
+    default: 'v0.1.30'
   v8:
     description: 'v8 version to install'
     required: false
-    default: '13.6.233.8'
+    default: '14.0.365.4'
   cache-dir:
     description: 'cache dir to use'
     required: false

Dockerfile

@@ -3,8 +3,8 @@ FROM debian:stable
 ARG MINISIG=0.12
 ARG ZIG=0.15.1
 ARG ZIG_MINISIG=RWSGOq2NVecA2UPNdBUZykf1CCb147pkmdtYxgb3Ti+JO/wCYvhbAb/U
-ARG V8=13.6.233.8
-ARG ZIG_V8=v0.1.28
+ARG V8=14.0.365.4
+ARG ZIG_V8=v0.1.30
 ARG TARGETPLATFORM
 
 RUN apt-get update -yq && \

build.zig.zon

@@ -5,9 +5,9 @@
     .fingerprint = 0xda130f3af836cea0,
     .dependencies = .{
         .v8 = .{
-            .url = "https://github.com/lightpanda-io/zig-v8-fork/archive/d3040a953e5a37290dae20e7ddf138b7aeb5e67d.tar.gz",
-            .hash = "v8-0.0.0-xddH6w_EAwA8vK0NAxfxfI7IcbnpkUAcXKNujn7qwnmY",
+            .url = "https://github.com/lightpanda-io/zig-v8-fork/archive/7177ee1ae267a44751a0e7e012e257177699a375.tar.gz",
+            .hash = "v8-0.0.0-xddH63TCAwC1D1hEiOtbEnLBbtz9ZPHrdiGWLcBcYQB7",
         },
-        //.v8 = .{ .path = "../zig-v8-fork" }
+        // .v8 = .{ .path = "../zig-v8-fork" }
     },
 }

src/runtime/test_object_types.zig

@@ -210,8 +210,8 @@ test "JS: object types" {
 
     defer runner.deinit();
 
-    // v8 has 5 default "own" properties
-    const own_base = "5";
+    // v8 has 3 default "own" properties
+    const own_base = "3";
 
     try runner.testCases(&.{
         .{ "Object.getOwnPropertyNames(MyObject).length;", own_base },