test js pdk v1.3.3 (#14)

* test js pdk v1.3.3

* install v1.3.3 in release pipeline too

* fix build

.github/workflows/release.yml

@@ -15,7 +15,7 @@ jobs:
 
       - name: Setup Extism
         run: |
-          curl https://raw.githubusercontent.com/extism/js-pdk/main/install.sh | bash
+          curl -L https://raw.githubusercontent.com/extism/js-pdk/main/install.sh | bash -s v1.3.3
 
       - name: Set up Node.js
         uses: actions/setup-node@v2

.github/workflows/test.yml

@@ -15,7 +15,7 @@ jobs:
 
       - name: Setup Extism and XTP
         run: |
-          curl https://raw.githubusercontent.com/extism/js-pdk/main/install.sh | bash
+          curl -L https://raw.githubusercontent.com/extism/js-pdk/main/install.sh | bash -s v1.3.3
           curl https://static.dylibso.com/cli/install.sh | bash
 
       - name: Set up Node.js
@@ -47,6 +47,8 @@ jobs:
 
       - name: Run Bindgen Test
         run: |
+          extism-js --version
+          
           sh bindgen-test.sh install
           sh bindgen-test.sh run
 

package.json

@@ -17,7 +17,7 @@
     "typescript": "^5.3.2"
   },
   "dependencies": {
-    "@dylibso/xtp-bindgen": "1.0.0-rc.8",
+    "@dylibso/xtp-bindgen": "1.0.0-rc.16",
     "ejs": "^3.1.10"
   }
 }

src/index.ts

@@ -45,10 +45,7 @@ function addStdImport(schema: XtpSchema) {
   // in the generated `main.zig` this would include a reference to
   // std.json.ArrayHashMap and std.json.Value, so we import "std".
   const exportHasJsonObject = schema.exports.some((f) => {
-    return (f.input?.contentType === "application/json" &&
-      f.input.type === "object") ||
-      (f.output?.contentType === "application/json" &&
-        f.output.type === "object");
+    return (helpers.isJsonEncoded(f.input!)) || (helpers.isJsonEncoded(f.output!));
   });
 
   return exportHasJsonObject /* || others here */