feat: remove amino-json patch (#199)

* chore: yarn updated yarn.lock

* feat: remove amino-json patch

* chore: generate proto without amino-json patch

* fix: wallet callmethod type

* fix: prettify all files + improve script logs

* fix: deps issue

* test: add e2e jest test + edit CI workflow

Author: aeddi

.github/workflows/test.yaml

@@ -19,5 +19,14 @@ jobs:
       - name: Install modules
         run: yarn install
 
+      - name: Install Go
+        uses: actions/setup-go@v6
+        with:
+          go-version: '1.23'
+
+      # Install gnodev only when this PR will be merged: https://github.com/gnolang/gno/pull/4763
+      - name: Install gnodev
+        run: curl -sSL https://raw.githubusercontent.com/gnolang/gno/master/misc/install.sh | bash
+
       - name: Tests
-        run: ./node_modules/.bin/jest --passWithNoTests
+        run: ./node_modules/.bin/jest

package.json

@@ -54,5 +54,8 @@
     "@gnolang/tm2-js-client": "^1.3.3",
     "long": "^5.3.2",
     "protobufjs": "^7.5.3"
+  },
+  "resolutions": {
+    "@noble/hashes": "^1.8.0"
   }
 }

scripts/amino-json-compatibility.patch

@@ -8,6 +8,7 @@ FILES=$(find proto -type f -name "*.proto")
 
 mkdir -p ${OUT_DIR}
 
+echo "Generating TypeScript files from proto definitions..."
 for x in ${FILES}; do
   protoc \
     --plugin="./node_modules/.bin/protoc-gen-ts_proto" \
@@ -17,18 +18,7 @@ for x in ${FILES}; do
     ${x}
 done
 
-# Apply amino JSON compatibility changes to patch proto generated files.
-# Patch fixes two issues:
-# - Empty `max_deposit` is represented in JSON as an empty string in proto while omitted completely in Amino JSON
-# - Empty `google.protobuf.Any` fields are omitted in proto JSON while explicitly set to null in Amino JSON
-#
-# See:
-#   https://github.com/gnolang/gno-js-client/pull/184
-#   https://github.com/gnolang/gno-js-client/pull/46
-#
-# This is a short term solution that potentially requires maintaining the patch updated.
+echo "Prettifying generated files..."
 if [ $? -eq 0 ]; then
-  # First prettify generated code to allow the patch to be applied
-  ${PRETTIER} --write ${OUT_DIR}/gno/bank.ts ${OUT_DIR}/gno/vm.ts && \
-  git apply ./scripts/amino-json-compatibility.patch
+  find ${OUT_DIR} -name "*.ts" -exec ${PRETTIER} --write {} +
 fi