chore: Pull noir (#4546)

Co-authored-by: Tom French <tom@tomfren.ch>
AztecProtocol · Feb 12, 2024 · acf5cf2 · acf5cf2
1 parent 490ca26
commit acf5cf2
Show file tree

Hide file tree

Showing 429 changed files with 8,581 additions and 5,969 deletions.
diff --git a/avm-transpiler/Cargo.lock b/avm-transpiler/Cargo.lock
diff --git a/build-system/scripts/request_spot b/build-system/scripts/request_spot
@@ -57,7 +57,7 @@ launch_spec=$(cat <<EOF
     {
       "DeviceName": "/dev/sda1",
       "Ebs": {
-        "VolumeSize": 16
+        "VolumeSize": 32
       }
     }
   ]

diff --git a/noir/.github/ISSUE_TEMPLATE/bug_report.yml b/noir/.github/ISSUE_TEMPLATE/bug_report.yml
@@ -6,7 +6,7 @@ body:
     attributes:
       value: |
         # Description
-        Thanks for taking the time to create the Issue, and welcome to the Noirot family!
+        Thanks for taking the time to create the Issue and welcome to the Noir community!
   - type: textarea
     id: aim
     attributes:
@@ -38,45 +38,69 @@ body:
         2.
         3.
         4.
+  - type: dropdown
+    id: impact
+    attributes:
+      label: Project Impact
+      description: How does this affect a project you or others are working on? 
+      options:
+        - "Nice-to-have"
+        - "Blocker"
+  - type: textarea
+    id: impact_context
+    attributes:
+      label: Impact Context
+      description: If a nice-to-have / blocker, supplement how does this Issue affect the project.
+  - type: dropdown
+    id: workaround
+    attributes:
+      label: Workaround
+      description: Is there a workaround for this Issue? 
+      options:
+        - "Yes"
+  - type: textarea
+    id: workaround_description
+    attributes:
+      label: Workaround Description
+      description: If yes, supplement how could the Issue be worked around.
+  - type: textarea
+    id: additional
+    attributes:
+      label: Additional Context
+      description: Supplement further information if applicable.
   - type: markdown
     attributes:
       value: |
         # Environment
-        Specify your versions of Noir releases used.
+        Specify your version of Noir tooling used.
   - type: markdown
     attributes:
       value: |
-        ## Using Nargo?
+        ## Nargo (CLI)
   - type: dropdown
     id: nargo-install
     attributes:
       label: Installation Method
       description: How did you install Nargo?
-      multiple: false
       options:
-        - Binary
+        - Binary (`noirup` default)
         - Compiled from source
   - type: input
     id: nargo-version
     attributes:
       label: Nargo Version
-      description: What is the output of the `nargo --version` command?
-      placeholder: "nargo 0.6.0 (git version hash: 0181813203a9e3e46c6d8c3169ad5d25971d4282, is dirty: false)"
+      description: Output of running `nargo --version`
+      placeholder: "nargo version = 0.23.0 noirc version = 0.23.0+5be9f9d7e2f39ca228df10e5a530474af0331704 (git version hash: 5be9f9d7e2f39ca228df10e5a530474af0331704, is dirty: false)"
   - type: markdown
     attributes:
       value: |
-        ## Using TypeScript?
-        Please await for our new set of packages.
-        You can find our target release timeframe on the [Noir Roadmap](https://github.com/orgs/noir-lang/projects/1/views/16).
-  - type: markdown
-    attributes:
-      value: |
-        # Misc
-  - type: textarea
-    id: additional
+        ## NoirJS (JavaScript)
+  - type: input
+    id: noirjs-version
     attributes:
-      label: Additional Context
-      description: Supplement further information if applicable.
+      label: NoirJS Version
+      description: Version number of `noir_js` in `package.json`
+      placeholder: "0.23.0"
   - type: markdown
     attributes:
       value: |
@@ -87,11 +111,8 @@ body:
       label: Would you like to submit a PR for this Issue?
       description: Fellow contributors are happy to provide support where applicable.
       options:
-        - "No"
         - "Maybe"
         - "Yes"
-    validations:
-      required: true
   - type: textarea
     id: pr_support
     attributes:

diff --git a/noir/.github/ISSUE_TEMPLATE/feature_request.yml b/noir/.github/ISSUE_TEMPLATE/feature_request.yml
@@ -6,7 +6,7 @@ body:
     attributes:
       value: |
         ## Description
-        Thanks for taking the time to create the Issue, and welcome to the Noirot family!
+        Thanks for taking the time to create the Issue and welcome to the Noir community!
   - type: textarea
     id: problem
     attributes:
@@ -21,11 +21,31 @@ body:
       description: Describe how you think it should work. Supply pseudocode / step-by-step examples if applicable.
     validations:
       required: true
+  - type: dropdown
+    id: impact
+    attributes:
+      label: Project Impact
+      description: How does this affect a project you or others are working on? 
+      options:
+        - "Nice-to-have"
+        - "Blocker"
   - type: textarea
-    id: alternatives
+    id: impact_context
     attributes:
-      label: Alternatives Considered
-      description: Describe less-happy cases you have considered, if any.
+      label: Impact Context
+      description: If a nice-to-have / blocker, supplement how does this Issue affect the project.
+  - type: dropdown
+    id: workaround
+    attributes:
+      label: Workaround
+      description: Is there a workaround for this Issue? 
+      options:
+        - "Yes"
+  - type: textarea
+    id: workaround_description
+    attributes:
+      label: Workaround Description
+      description: If yes, supplement how could the Issue be worked around.
   - type: textarea
     id: additional
     attributes:
@@ -42,11 +62,8 @@ body:
       description: Fellow contributors are happy to provide support where applicable.
       multiple: false
       options:
-        - "No"
         - "Maybe"
         - "Yes"
-    validations:
-      required: true
   - type: textarea
     id: pr-support
     attributes:

diff --git a/noir/.github/actions/install-playwright/action.yml b/noir/.github/actions/install-playwright/action.yml
@@ -9,7 +9,7 @@ runs:
       run: echo "PLAYWRIGHT_VERSION=$(yarn workspace @noir-lang/noirc_abi info @web/test-runner-playwright --json | jq .children.Version | tr -d '"')" >> $GITHUB_ENV
 
     - name: Cache playwright binaries
-      uses: actions/cache@v3
+      uses: actions/cache@v4
       id: playwright-cache
       with:
         path: |

diff --git a/noir/.github/actions/nix/action.yml b/noir/.github/actions/nix/action.yml
diff --git a/noir/.github/scripts/integration-test.sh → ...ithub/scripts/integration-test-browser.sh b/noir/.github/scripts/integration-test.sh → ...ithub/scripts/integration-test-browser.sh
@@ -1,6 +1,5 @@
 #!/bin/bash
 set -eu
 
-apt-get install libc++-dev -y
 npx playwright install && npx playwright install-deps
-yarn workspace integration-tests test
+yarn workspace integration-tests test
diff --git a/noir/.github/scripts/integration-test-node.sh b/noir/.github/scripts/integration-test-node.sh
@@ -0,0 +1,5 @@
+#!/bin/bash
+set -eu
+
+apt-get install libc++-dev -y
+yarn workspace integration-tests test
diff --git a/noir/.github/scripts/noir-js-test.sh b/noir/.github/scripts/noir-js-test.sh
@@ -1,6 +1,4 @@
 #!/bin/bash
 set -eu
 
-./scripts/nargo_compile_noir_js_assert_lt.sh
-rm -rf /usr/src/noir/tooling/noir_js/test/noir_compiled_examples/assert_lt/target/debug_assert_lt.json
-yarn workspace @noir-lang/noir_js test
+yarn workspace @noir-lang/noir_js test
diff --git a/noir/.github/workflows/docker-test-flow.yml b/noir/.github/workflows/docker-test-flow.yml
@@ -720,7 +720,63 @@ jobs:
       - name: Test
         working-directory: /usr/src/noir
         run: |
-          ./.github/scripts/integration-test.sh
+          ./.github/scripts/integration-test-node.sh
+
+  test-integration-browser:
+    name: Integration test browser
+    runs-on: ubuntu-latest
+    needs: [
+      build-base-js,
+      build-noir-wasm,
+      build-noirc-abi,
+      build-acvm_js,
+      build-noir-js-types,
+      build-noir_js,
+      build-barretenberg-backend
+    ]
+    container:
+      image: ghcr.io/noir-lang/noir:${{ github.sha }}-js
+      credentials:
+        username: ${{ github.actor }}
+        password: ${{ secrets.github_token }}
+    steps:
+      - name: Download noir wasm
+        uses: actions/download-artifact@v4
+        with:
+          name: noir_wasm
+          path: /usr/src/noir/compiler/wasm
+      - name: Download noirc abi
+        uses: actions/download-artifact@v4
+        with:
+          name: noirc_abi_wasm
+          path: /usr/src/noir/tooling/noirc_abi_wasm
+      - name: Download acvm js
+        uses: actions/download-artifact@v4
+        with:
+          name: acvm_js
+          path: /usr/src/noir/acvm-repo/acvm_js
+      - name: Download noir js types
+        uses: actions/download-artifact@v4
+        with:
+          name: noir-js-types
+          path: |
+            /usr/src/noir/tooling/noir_js_types/lib
+      - name: Download noir js
+        uses: actions/download-artifact@v4
+        with:
+          name: noir_js
+          path:
+            /usr/src/noir/tooling/noir_js/lib
+      - name: Download Barretenberg backend
+        uses: actions/download-artifact@v4
+        with:
+          name: barretenberg-backend
+          path:
+            /usr/src/noir/tooling/noir_js_backend_barretenberg/lib
+      - name: Test
+        working-directory: /usr/src/noir
+        run: |
+          ./.github/scripts/integration-test-browser.sh
 
   tests-end:
     name: End
@@ -733,6 +789,7 @@ jobs:
       - test-noir-wasm
       - test-noir-wasm-browser
       - test-integration
+      - test-integration-browser
       - test-noir_codegen
       - test-acvm_js
       - test-acvm_js-browser