Update snapshots from the upload artifact run

.codespellrc

@@ -1,3 +1,3 @@
 [codespell]
 ignore-words-list: crate,everytime,inout,co-ordinate,ot,nwo,atleast,ue,afterall,ser
-skip: **/target,node_modules,build,dist,./out,**/Cargo.lock,./docs/kcl/*.md,pnpm-lock.yaml,**/yarn.lock,./openapi/*.json,./packages/codemirror-lang-kcl/test/all.test.ts,tsconfig.tsbuildinfo,src/lib/machine-api.d.ts
+skip: **/target,node_modules,build,dist,./out,**/Cargo.lock,./docs/kcl/*.md,.yarn.lock,**/yarn.lock,./openapi/*.json,./packages/codemirror-lang-kcl/test/all.test.ts,tsconfig.tsbuildinfo

.env.production

@@ -1,4 +1,5 @@
 NODE_ENV=production
+DEV=false
 VITE_KC_API_WS_MODELING_URL=wss://api.zoo.dev/ws/modeling/commands
 VITE_KC_API_BASE_URL=https://api.zoo.dev
 VITE_KC_SITE_BASE_URL=https://zoo.dev

.eslintignore

@@ -1,4 +1,3 @@
 src/wasm-lib/*
 *.typegen.ts
 packages/codemirror-lsp-client/dist/*
-e2e/playwright/snapshots/prompt-to-edit/*

.gitattributes

@@ -1,13 +0,0 @@
-# Set default behavior to automatically normalize line endings.
-* text=auto
-
-# Force batch scripts to always use CRLF line endings so that if a repo is accessed
-# in Windows via a file share from Linux, the scripts will work.
-*.{cmd,[cC][mM][dD]} text working-tree-encoding=UTF-16LE eol=CRLF
-*.{bat,[bB][aA][tT]} text working-tree-encoding=UTF-16LE eol=CRLF
-*.{ics,[iI][cC][sS]} text working-tree-encoding=UTF-16LE eol=CRLF
-*.{ps1,[iP][sS][1]} text working-tree-encoding=UTF-16LE eol=CRLF
-
-# Force bash scripts to always use LF line endings so that if a repo is accessed
-# in Unix via a file share from Windows, the scripts will work.
-*.sh text eol=lfol=lf

.github/ci-cd-scripts/playwright-electron.sh

@@ -7,11 +7,11 @@ if [[ ! -f "test-results/.last-run.json" ]]; then
     # if no last run artifact, than run plawright normally
     echo "run playwright normally"
         if [[ "$3" == *ubuntu* ]]; then
-            xvfb-run --auto-servernum --server-args="-screen 0 1280x960x24" -- pnpm test:playwright:electron:ubuntu -- --shard=$1/$2 || true
+            xvfb-run --auto-servernum --server-args="-screen 0 1280x960x24" -- yarn test:playwright:electron:ubuntu -- --shard=$1/$2 || true
         elif [[ "$3" == *windows* ]]; then
-            pnpm test:playwright:electron:windows -- --shard=$1/$2 || true
+            yarn test:playwright:electron:windows -- --shard=$1/$2 || true
         elif [[ "$3" == *macos* ]]; then
-            pnpm test:playwright:electron:macos  -- --shard=$1/$2 || true
+            yarn test:playwright:electron:macos  -- --shard=$1/$2 || true
         else
             echo "Do not run playwright. Unable to detect os runtime."
             exit 1
@@ -21,7 +21,7 @@ if [[ ! -f "test-results/.last-run.json" ]]; then
 fi
 
 retry=1
-max_retrys=1
+max_retrys=5
 
 # retry failed tests, doing our own retries because using inbuilt playwright retries causes connection issues
 while [[ $retry -le $max_retrys ]]; do
@@ -31,11 +31,11 @@ while [[ $retry -le $max_retrys ]]; do
             echo "retried=true" >>$GITHUB_OUTPUT
             echo "run playwright with last failed tests and retry $retry"
             if [[ "$3" == *ubuntu* ]]; then
-                xvfb-run --auto-servernum --server-args="-screen 0 1280x960x24" -- pnpm test:playwright:electron:ubuntu -- --last-failed || true
+                xvfb-run --auto-servernum --server-args="-screen 0 1280x960x24" -- yarn test:playwright:electron:ubuntu -- --last-failed || true
             elif [[ "$3" == *windows* ]]; then
-                pnpm test:playwright:electron:windows -- --last-failed || true
+                yarn test:playwright:electron:windows -- --last-failed || true
             elif [[ "$3" == *macos* ]]; then
-                pnpm test:playwright:electron:macos -- --last-failed || true
+                yarn test:playwright:electron:macos -- --last-failed || true
             else
                 echo "Do not run playwright. Unable to detect os runtime."
                 exit 1

.github/workflows/build-and-store-wasm.yml

@@ -10,24 +10,20 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-      - uses: pnpm/action-setup@v4
       - uses: actions/setup-node@v4
         with:
           node-version-file: '.nvmrc'
-          cache: 'pnpm'
+          cache: 'yarn'
       - name: Install dependencies
-        run: pnpm install
+        run: yarn
       - name: Setup Rust
         uses: dtolnay/rust-toolchain@stable
       - name: Cache wasm
         uses: Swatinem/rust-cache@v2
         with:
           workspaces: './src/wasm-lib'
-      - uses: taiki-e/install-action@2dbeb927f58939d3aa13bf06ba0c0a34b76b9bfb
-        with:
-          tool: wasm-pack
       - name: build wasm
-        run: pnpm build:wasm
+        run: yarn build:wasm
 
 
       # Upload the WASM bundle as an artifact

.github/workflows/build-apps.yml

@@ -5,14 +5,16 @@ on:
   push:
     branches:
       - main
-      - pierremtb/adhoc/pnpm
     tags:
       - 'v[0-9]+.[0-9]+.[0-9]+'
-      - 'nightly-v[0-9]+.[0-9]+.[0-9]+'
+  schedule:
+    - cron: '0 4 * * *'
+  # Daily at 04:00 AM UTC
+  # Will checkout the last commit from the default branch (main as of 2023-10-04)
 
 env:
-  IS_RELEASE: ${{ github.ref_type == 'tag' && startsWith(github.ref_name, 'v') }}
-  IS_NIGHTLY: ${{ github.ref_type == 'tag' && startsWith(github.ref_name, 'nightly-v') }}
+  IS_RELEASE: ${{ github.ref_type == 'tag' }}
+  IS_NIGHTLY: ${{ github.event_name == 'schedule' }}
 
 concurrency:
   group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
@@ -27,14 +29,12 @@ jobs:
     steps:
       - uses: actions/checkout@v4
 
-      - uses: pnpm/action-setup@v4
-
       - uses: actions/setup-node@v4
         with:
           node-version-file: '.nvmrc'
-          cache: 'pnpm'
+          cache: 'yarn'
 
-      - run: pnpm install
+      - run: yarn install
 
       - name: Setup Rust
         uses: dtolnay/rust-toolchain@stable
@@ -44,25 +44,18 @@ jobs:
           workspaces: './src/wasm-lib'
 
       # TODO: see if we can fetch from main instead if no diff at src/wasm-lib
-      - uses: taiki-e/install-action@2dbeb927f58939d3aa13bf06ba0c0a34b76b9bfb
-        with:
-          tool: wasm-pack
-
       - name: Run build:wasm
-        run: "pnpm build:wasm"
+        run: "yarn build:wasm"
 
       - name: Set nightly version, product name, release notes, and icons
         if: ${{ env.IS_NIGHTLY == 'true' }}
-        run: |
-          export VERSION=${GITHUB_REF_NAME#nightly-v}
-          pnpm files:set-version
-          pnpm files:flip-to-nightly
+        run: yarn files:flip-to-nightly
 
       - name: Set release version
         if: ${{ env.IS_RELEASE == 'true' }}
         run: |
           export VERSION=${GITHUB_REF_NAME#v}
-          pnpm files:set-version
+          yarn files:set-version
 
       - uses: actions/upload-artifact@v4
         with:
@@ -127,21 +120,19 @@ jobs:
           cp prepared-files/assets/icon.ico assets/icon.ico
           cp prepared-files/assets/icon.png assets/icon.png
 
-      - uses: pnpm/action-setup@v4
-
       - name: Sync node version and setup cache
         uses: actions/setup-node@v4
         with:
           node-version-file: '.nvmrc'
-          cache: 'pnpm' # Set this to npm, pnpm or pnpm.
+          cache: 'yarn' # Set this to npm, yarn or pnpm.
 
-      - name: pnpm install
+      - name: yarn install
         # Windows is picky sometimes and fails on fetch. Step takes about ~30s
-        uses: nick-fields/retry@v3.0.1
+        uses: nick-fields/retry@v3.0.0
         with:
           timeout_minutes: 2
           max_attempts: 3
-          command: pnpm install
+          command: yarn install
 
       - name: Prepare certificate and variables (Windows only)
         if: ${{ (env.IS_RELEASE == 'true' || env.IS_NIGHTLY == 'true') && matrix.os == 'windows-2022' }}
@@ -173,7 +164,7 @@ jobs:
         if: ${{ env.IS_RELEASE == 'false' && env.IS_NIGHTLY == 'false' }}
         # electron-builder doesn't have a concept of release vs debug,
         # this is just not doing any codesign or release yml generation, and points to dev infra
-        run: pnpm tronb:package:dev
+        run: yarn tronb:package:dev
 
       - name: Build the app (release)
         if: ${{ env.IS_RELEASE == 'true' || env.IS_NIGHTLY == 'true' }}
@@ -188,11 +179,11 @@ jobs:
           WINDOWS_CERTIFICATE_THUMBPRINT: ${{ secrets.WINDOWS_CERTIFICATE_THUMBPRINT }}
           DEBUG: "electron-notarize*"
         # TODO: Fix electron-notarize flakes. The logs above should help gather more data on failures
-        uses: nick-fields/retry@v3.0.1
+        uses: nick-fields/retry@v3.0.0
         with:
           timeout_minutes: 10
           max_attempts: 3
-          command: pnpm tronb:package:prod
+          command: yarn tronb:package:prod
 
       - name: List artifacts in out/
         run: ls -R out
@@ -249,11 +240,11 @@ jobs:
           WINDOWS_CERTIFICATE_THUMBPRINT: ${{ secrets.WINDOWS_CERTIFICATE_THUMBPRINT }}
           DEBUG: "electron-notarize*"
         # TODO: Fix electron-notarize flakes. The logs above should help gather more data on failures
-        uses: nick-fields/retry@v3.0.1
+        uses: nick-fields/retry@v3.0.0
         with:
           timeout_minutes: 10
           max_attempts: 3
-          command: pnpm tronb:package:prod
+          command: yarn tronb:package:prod
 
       - uses: actions/upload-artifact@v4
         if: ${{ env.IS_RELEASE == 'true' }}
@@ -278,7 +269,7 @@ jobs:
     runs-on: ubuntu-22.04
     permissions:
       contents: write
-    if: ${{ github.ref_type == 'tag' }}
+    if: ${{ github.ref_type == 'tag' || github.event_name == 'schedule' }}
     env:
       VERSION_NO_V: ${{ needs.prepare-files.outputs.version }}
       VERSION: ${{ format('v{0}', needs.prepare-files.outputs.version) }}
@@ -335,8 +326,8 @@ jobs:
         env:
           NOTES: ${{ needs.prepare-files.outputs.notes }}
           PUB_DATE: ${{ github.event.repository.updated_at }}
-          WEBSITE_DIR: ${{ env.IS_NIGHTLY == 'true' && 'dl.zoo.dev/releases/modeling-app/nightly' || 'dl.zoo.dev/releases/modeling-app' }}
-          URL_CODED_NAME: ${{ env.IS_NIGHTLY == 'true' && 'Zoo%20Modeling%20App%20%28Nightly%29' || 'Zoo%20Modeling%20App' }}
+          WEBSITE_DIR: ${{ github.event_name == 'schedule' && 'dl.zoo.dev/releases/modeling-app/nightly' || 'dl.zoo.dev/releases/modeling-app' }}
+          URL_CODED_NAME: ${{ github.event_name == 'schedule' && 'Zoo%20Modeling%20App%20%28Nightly%29' || 'Zoo%20Modeling%20App' }}
         run: |
           RELEASE_DIR=https://${WEBSITE_DIR}
           jq --null-input \
@@ -391,9 +382,9 @@ jobs:
           # see https://github.com/actions/checkout/issues/1471
           git fetch --prune --unshallow --tags
           export TAG="nightly-${VERSION}"
-          export PREVIOUS_TAG=$(git tag --list --sort=-committerdate "nightly-v[0-9]*" | head -n2 | tail -n1)
+          export PREVIOUS_TAG=$(git describe --tags --match="nightly-v[0-9]*" --abbrev=0)
           export NOTES=$(./scripts/get-nightly-changelog.sh)
-          pnpm files:set-notes
+          yarn files:set-notes
 
       - name: Authenticate to Google Cloud
         if: ${{ env.IS_NIGHTLY == 'true' }}
@@ -418,4 +409,15 @@ jobs:
 
       - name: Invalidate bucket cache on latest*.yml and last_download.json files
         if: ${{ env.IS_NIGHTLY == 'true' }}
-        run: pnpm files:invalidate-bucket:nightly
+        run: yarn files:invalidate-bucket:nightly
+
+      - name: Tag nightly commit
+        if: ${{ env.IS_NIGHTLY == 'true' }}
+        uses: actions/github-script@v7
+        with:
+          script: |
+            const { VERSION } = process.env          
+            const { owner, repo } = context.repo
+            const { sha } = context
+            const ref = `refs/tags/nightly-${VERSION}`
+            github.rest.git.createRef({ owner, repo, sha, ref })

.github/workflows/cargo-test.yml

@@ -46,7 +46,7 @@ jobs:
         shell: bash
         run: |-
           cd "${{ matrix.dir }}"
-          cargo llvm-cov nextest --workspace --lcov --output-path lcov.info --test-threads=1 --retries=2 --no-fail-fast -P ci 2>&1 | tee /tmp/github-actions.log
+          cargo llvm-cov nextest --workspace --lcov --output-path lcov.info --test-threads=1 --no-fail-fast -P ci 2>&1 | tee /tmp/github-actions.log
         env:
           KITTYCAD_API_TOKEN: ${{secrets.KITTYCAD_API_TOKEN}}
           RUST_MIN_STACK: 10485760000

.github/workflows/check-exampleKcl.yml

@@ -26,21 +26,11 @@ jobs:
             const issue_number = context.payload.pull_request.number;
             const owner = context.repo.owner;
             const repo = context.repo.repo;
-            
-            const { data: comments } = await github.rest.issues.listComments({
+
+            // Post a comment on the PR
+            await github.rest.issues.createComment({
               owner,
               repo,
-              issue_number
-            });
-
-            const commentExists = comments.some(comment => comment.body === message);
-
-            if (!commentExists) {
-              // Post a comment on the PR
-              await github.rest.issues.createComment({
-                owner,
-                repo,
-                issue_number,
-                body: message,
-              });
-            }
+              issue_number,
+              body: message,
+            });

.github/workflows/codemirror-lang-kcl.yml

@@ -16,19 +16,17 @@ jobs:
     steps:
       - uses: actions/checkout@v4
 
-      - uses: pnpm/action-setup@v4
-
       - uses: actions/setup-node@v4
         with:
           node-version-file: '.nvmrc'
-          cache: 'pnpm'
+          cache: 'yarn'
 
-      - run: pnpm install
+      - run: yarn install
         working-directory: packages/codemirror-lang-kcl
 
-      - run: pnpm tsc
+      - run: yarn tsc
         working-directory: packages/codemirror-lang-kcl
 
       - name: run unit tests
-        run: pnpm test
+        run: yarn test
         working-directory: packages/codemirror-lang-kcl

.github/workflows/e2e-flow-tests.yml

@@ -1,10 +1,9 @@
-name: E2E Tests
+name: E2E Flow Tests
 on:
   push:
-    branches: [ main, pierremtb/adhoc/pnpm]
+    branches: [ main ]
   pull_request:
-  schedule:
-    - cron: 0 * * * *  # hourly
+    branches: [ main ]
 
 concurrency:
   group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
@@ -17,7 +16,24 @@ permissions:
 
 
 jobs:
-  electron:
+
+  check-rust-changes:
+    runs-on: ubuntu-latest
+    outputs:
+      rust-changed: ${{ steps.filter.outputs.rust }}
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - id: filter
+        name: Check for Rust changes
+        uses: dorny/paths-filter@v3
+        with:
+          filters: |
+            rust:
+              - 'src/wasm-lib/**'
+
+  flow-tests:
     timeout-minutes: 60
     name: playwright:electron:${{ matrix.os }} ${{ matrix.shardIndex }} ${{ matrix.shardTotal }}
     strategy:
@@ -27,48 +43,36 @@ jobs:
         os: [namespace-profile-ubuntu-8-cores, namespace-profile-macos-8-cores, windows-16-cores]
         shardIndex: [1, 2, 3, 4]
         shardTotal: [4]
-        # TODO: add ref here for main and latest release tag
     runs-on: ${{ matrix.os }}
+    needs: check-rust-changes
     steps:
-    - uses: actions/create-github-app-token@v1
-      id: app-token
-      with:
-        app-id: ${{ secrets.MODELING_APP_GH_APP_ID }}
-        private-key: ${{ secrets.MODELING_APP_GH_APP_PRIVATE_KEY }}
-        owner: ${{ github.repository_owner }}
     - uses: actions/checkout@v4
-      with:
-        token: ${{ steps.app-token.outputs.token }}
-    - id: filter
-      name: Check for Rust changes
-      uses: dorny/paths-filter@v3
-      with:
-        filters: |
-          rust:
-            - 'src/wasm-lib/**'
-    - uses: pnpm/action-setup@v4
-    # TODO: remove this
-    - run: npm install --global --force corepack@latest
+
     - uses: actions/setup-node@v4
       with:
         node-version-file: '.nvmrc'
-        cache: 'pnpm'
+        cache: 'yarn'
+
     - uses: KittyCAD/action-install-cli@main
+
     - name: Install dependencies
       shell: bash
-      run: pnpm install
+      run: yarn
+
     - name: Cache Playwright Browsers
       uses: actions/cache@v4
       with:
         path: |
           ~/.cache/ms-playwright/
-        key: ${{ runner.os }}-playwright-${{ hashFiles('pnpm.lock') }}
+        key: ${{ runner.os }}-playwright-${{ hashFiles('yarn.lock') }}
+
     - name: Install Playwright Browsers
       shell: bash
-      run: pnpm playwright install --with-deps
+      run: yarn playwright install --with-deps
+
     - name: Download Wasm Cache
       id: download-wasm
-      if: github.event_name != 'schedule' && steps.filter.outputs.rust == 'false'
+      if: needs.check-rust-changes.outputs.rust-changed == 'false'
       uses: dawidd6/action-download-artifact@v7
       continue-on-error: true
       with:
@@ -77,32 +81,35 @@ jobs:
         workflow: build-and-store-wasm.yml
         branch: main
         path: src/wasm-lib/pkg
+
     - name: copy wasm blob
-      if: github.event_name != 'schedule' && steps.filter.outputs.rust == 'false'
+      if: needs.check-rust-changes.outputs.rust-changed == 'false'
       shell: bash
       run: cp src/wasm-lib/pkg/wasm_lib_bg.wasm public
       continue-on-error: true
+
     - name: Setup Rust
       uses: dtolnay/rust-toolchain@stable
-    - uses: taiki-e/install-action@2dbeb927f58939d3aa13bf06ba0c0a34b76b9bfb
-      with:
-        tool: wasm-pack
+
     - name: Cache Wasm (because rust diff)
-      if: github.event_name == 'schedule' || steps.filter.outputs.rust == 'true'
+      if: needs.check-rust-changes.outputs.rust-changed == 'true'
       uses: Swatinem/rust-cache@v2
       with:
         workspaces: './src/wasm-lib'
+
     - name: OR Cache Wasm (because wasm cache failed)
       if: steps.download-wasm.outcome == 'failure'
       uses: Swatinem/rust-cache@v2
       with:
         workspaces: './src/wasm-lib'
+
     - name: install good sed
       if:  ${{ startsWith(matrix.os, 'macos') }}
       shell: bash
       run: |
         brew install gnu-sed
         echo "/opt/homebrew/opt/gnu-sed/libexec/gnubin" >> $GITHUB_PATH
+
     - name: Install vector
       shell: bash
       # TODO: figure out what to do with this, it's failing
@@ -120,89 +127,39 @@ jobs:
         sed -i "s#GH_ACTIONS_AXIOM_TOKEN#${{secrets.GH_ACTIONS_AXIOM_TOKEN}}#g" /tmp/vector.toml
         cat /tmp/vector.toml
         ${HOME}/.vector/bin/vector --config /tmp/vector.toml &
+
     - name: Build Wasm (because rust diff)
-      if: github.event_name == 'schedule' || steps.filter.outputs.rust == 'true'
+      if: needs.check-rust-changes.outputs.rust-changed == 'true'
       shell: bash
-      run: pnpm build:wasm
+      run: yarn build:wasm
+
     - name: OR Build Wasm (because wasm cache failed)
       if: steps.download-wasm.outcome == 'failure'
       shell: bash
-      run: pnpm build:wasm
+      run: yarn build:wasm
+
     - name: build web
       shell: bash
-      run: pnpm tronb:vite:dev
-    - name: Run ubuntu/chrome snapshots
-      if: ${{ matrix.os == 'namespace-profile-ubuntu-8-cores' && matrix.shardIndex == 1 }}
-      shell: bash
-      # TODO: break this in its own job, for now it's not slowing down the overall execution as ubuntu is the quickest,
-      # but we could do better. This forces a large 1/1 shard of all 20 snapshot tests that runs in about 3 minutes.
-      run: |
-        PLATFORM=web pnpm playwright test --config=playwright.config.ts --retries="3" --update-snapshots --grep=@snapshot --trace=on --shard=1/1
-      env:
-        CI: true
-        NODE_ENV: development
-        VITE_KC_DEV_TOKEN: ${{ secrets.KITTYCAD_API_TOKEN_DEV }}
-        VITE_KC_SKIP_AUTH: true
-        token: ${{ secrets.KITTYCAD_API_TOKEN_DEV }}
-        snapshottoken: ${{ secrets.KITTYCAD_API_TOKEN }}
-    - uses: actions/upload-artifact@v4
-      if: ${{ !cancelled() && (success() || failure()) }}
-      with:
-        name: playwright-report-snapshots-${{ matrix.os }}-snapshot-${{ matrix.shardIndex }}-${{ github.sha }}
-        path: playwright-report/
-        include-hidden-files: true
-        retention-days: 30
-        overwrite: true
+      run: yarn tronb:vite:dev
+
     - name: Clean up test-results
       if: ${{ !cancelled() && (success() || failure()) }}
       continue-on-error: true
       run: rm -r test-results
-    - name: check for changes
-      if: ${{ matrix.os == 'namespace-profile-ubuntu-8-cores' && matrix.shardIndex == 1 && github.ref != 'refs/heads/main' }}
-      shell: bash
-      id: git-check
-      run: |
-          git add e2e/playwright/snapshot-tests.spec.ts-snapshots e2e/playwright/snapshots
-          if git status | grep -q "Changes to be committed"
-          then echo "modified=true" >> $GITHUB_OUTPUT
-          else echo "modified=false" >> $GITHUB_OUTPUT
-          fi
-    - name: Commit changes, if any
-      if: steps.git-check.outputs.modified == 'true'
-      shell: bash
-      run: |
-        git add e2e/playwright/snapshot-tests.spec.ts-snapshots e2e/playwright/snapshots
-        git config --local user.email "github-actions[bot]@users.noreply.github.com"
-        git config --local user.name "github-actions[bot]"
-        git remote set-url origin https://${{ github.actor }}:${{ secrets.GITHUB_TOKEN }}@github.com/${{ github.repository }}.git
-        git fetch origin
-        echo ${{ github.head_ref }}
-        git checkout ${{ github.head_ref }}
-        git commit -m "A snapshot a day keeps the bugs away! 📷🐛 (OS: ${{matrix.os}})" || true
-        git push
-        git push origin ${{ github.head_ref }}
-    # only upload artifacts if there's actually changes
-    - uses: actions/upload-artifact@v4
-      if: steps.git-check.outputs.modified == 'true'
-      with:
-        name: playwright-report-${{ matrix.os }}-${{ matrix.shardIndex }}-${{ github.sha }}
-        path: playwright-report/
-        include-hidden-files: true
-        retention-days: 30
+
     - uses: actions/download-artifact@v4
       if: ${{ !cancelled() && (success() || failure()) }}
       continue-on-error: true
       with:
         name: test-results-${{ matrix.os }}-${{ matrix.shardIndex }}-${{ github.sha }}
         path: test-results/
+
     - name: Run playwright/electron flow (with retries)
       id: retry
       if: ${{ !cancelled() && (success() || failure()) }}
-      uses: nick-fields/retry@v3.0.1
-      with:
-        command: .github/ci-cd-scripts/playwright-electron.sh ${{matrix.shardIndex}} ${{matrix.shardTotal}} ${{matrix.os}}
-        timeout_minutes: 30
-        max_attempts: 25
+      shell: bash
+      run: |
+        .github/ci-cd-scripts/playwright-electron.sh ${{matrix.shardIndex}} ${{matrix.shardTotal}} ${{matrix.os}}
       env:
         CI: true
         FAIL_ON_CONSOLE_ERRORS: true
@@ -210,6 +167,7 @@ jobs:
         VITE_KC_DEV_TOKEN: ${{ secrets.KITTYCAD_API_TOKEN_DEV }}
         VITE_KC_SKIP_AUTH: true
         token: ${{ secrets.KITTYCAD_API_TOKEN_DEV }}
+
     - uses: actions/upload-artifact@v4
       if: always()
       with:
@@ -218,6 +176,7 @@ jobs:
         include-hidden-files: true
         retention-days: 30
         overwrite: true
+
     - uses: actions/upload-artifact@v4
       if: always()
       with:
@@ -226,4 +185,3 @@ jobs:
         include-hidden-files: true
         retention-days: 30
         overwrite: true
-

.github/workflows/e2e-snapshot-tests.yml

@@ -0,0 +1,145 @@
+name: E2E Snapshot Tests
+on:
+  push:
+    branches: [ main ]
+  pull_request:
+    branches: [ main ]
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
+  cancel-in-progress: true
+
+permissions:
+  contents: write
+  pull-requests: write
+  actions: read
+
+
+jobs:
+
+  check-rust-changes:
+    runs-on: ubuntu-latest
+    outputs:
+      rust-changed: ${{ steps.filter.outputs.rust }}
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - id: filter
+        name: Check for Rust changes
+        uses: dorny/paths-filter@v3
+        with:
+          filters: |
+            rust:
+              - 'src/wasm-lib/**'
+
+  snapshot-tests:
+    runs-on: ubuntu-22.04
+    needs: check-rust-changes
+    steps:
+    - uses: actions/checkout@v4
+
+    - uses: actions/setup-node@v4
+      with:
+        node-version-file: '.nvmrc'
+        cache: 'yarn'
+
+    - name: Install dependencies
+      run: yarn
+
+    - name: Cache Playwright Browsers
+      uses: actions/cache@v4
+      with:
+        path: |
+          ~/.cache/ms-playwright/
+        key: ${{ runner.os }}-playwright-${{ hashFiles('yarn.lock') }}
+
+    - name: Install Playwright Browsers
+      run: yarn playwright install --with-deps
+
+    - name: Download Wasm Cache
+      id: download-wasm
+      if: needs.check-rust-changes.outputs.rust-changed == 'false'
+      uses: dawidd6/action-download-artifact@v7
+      continue-on-error: true
+      with:
+        github_token: ${{secrets.GITHUB_TOKEN}}
+        name: wasm-bundle
+        workflow: build-and-store-wasm.yml
+        branch: main
+        path: src/wasm-lib/pkg
+
+    - name: copy wasm blob
+      if: needs.check-rust-changes.outputs.rust-changed == 'false'
+      run: cp src/wasm-lib/pkg/wasm_lib_bg.wasm public
+      continue-on-error: true
+
+    - name: Setup Rust
+      if: needs.check-rust-changes.outputs.rust-changed == 'true'
+      uses: dtolnay/rust-toolchain@stable
+
+    - name: Cache Wasm (because rust diff)
+      if: needs.check-rust-changes.outputs.rust-changed == 'true'
+      uses: Swatinem/rust-cache@v2
+      with:
+        workspaces: './src/wasm-lib'
+
+    - name: OR Cache Wasm (because wasm cache failed)
+      if: steps.download-wasm.outcome == 'failure'
+      uses: Swatinem/rust-cache@v2
+      with:
+        workspaces: './src/wasm-lib'
+
+    - name: Build Wasm (because rust diff)
+      if: needs.check-rust-changes.outputs.rust-changed == 'true'
+      run: yarn build:wasm
+
+    - name: OR Build Wasm (because wasm cache failed)
+      if: steps.download-wasm.outcome == 'failure'
+      run: yarn build:wasm
+
+    - name: build web
+      run: yarn tronb:vite:dev
+
+    - name: Run chrome snapshots
+      run: |
+        PLATFORM=web yarn playwright test --config=playwright.config.ts --retries="3" --update-snapshots --grep=@snapshot
+      env:
+        CI: true
+        NODE_ENV: development
+        VITE_KC_DEV_TOKEN: ${{ secrets.KITTYCAD_API_TOKEN_DEV }}
+        VITE_KC_SKIP_AUTH: true
+        token: ${{ secrets.KITTYCAD_API_TOKEN_DEV }}
+        snapshottoken: ${{ secrets.KITTYCAD_API_TOKEN }}
+
+    - name: check for changes
+      id: git-check
+      run: |
+        {
+          echo 'changes<<EOF'
+          git diff --name-only e2e/playwright/snapshot-tests.spec.ts-snapshots
+          echo EOF
+        } >> "$GITHUB_OUTPUT"
+
+    # only upload artifacts if there's actually changes
+    - name: Upload changes, if any
+      if: steps.git-check.outputs.changes != ''
+      uses: actions/upload-artifact@v4
+      with:
+        name: playwright-snapshots-${{ runner.os }}-${{ github.sha }}
+        path: ${{ steps.git-check.outputs.changes }}
+
+    - name: Upload report, if any
+      uses: actions/upload-artifact@v4
+      if: steps.git-check.outputs.changes != ''
+      with:
+        name: playwright-report-${{ runner.os }}-${{ github.sha }}
+        path: playwright-report/
+        include-hidden-files: true
+        retention-days: 30
+
+    - name: Fail the run if we have snapshot updates
+      if: steps.git-check.outputs.changes != ''
+      run: exit 1
+
+    # TODO: check if we could comment on the PR as well

.github/workflows/generate-machine-api-types.yml

@@ -18,14 +18,13 @@ jobs:
     runs-on: 'ubuntu-latest'
     steps:
       - uses: actions/checkout@v4
-      - uses: pnpm/action-setup@v4
       - uses: actions/setup-node@v4
         with:
           node-version-file: '.nvmrc'
-          cache: 'pnpm'
-      - run: pnpm install
-      - run: pnpm generate:machine-api
-      - run: pnpm fmt
+          cache: 'yarn'
+      - run: yarn install
+      - run: yarn generate:machine-api
+      - run: yarn fmt
       - name: check for changes
         id: git-check
         run: |

.github/workflows/publish-apps-release.yml

@@ -16,12 +16,10 @@ jobs:
     steps:
       - uses: actions/checkout@v4
 
-      - uses: pnpm/action-setup@v4
-
       - uses: actions/setup-node@v4
         with:
           node-version-file: '.nvmrc'
-          cache: 'pnpm'
+          cache: 'yarn'
 
       - name: Find tag workflow id
         id: tag_workflow_id
@@ -107,7 +105,7 @@ jobs:
       - name: Override release notes
         env:
           NOTES: ${{ github.event.release.body }}
-        run: pnpm files:set-notes
+        run: yarn files:set-notes
 
       - name: Authenticate to Google Cloud
         uses: 'google-github-actions/auth@v2.1.8'
@@ -128,7 +126,7 @@ jobs:
           destination: 'dl.kittycad.io/releases/modeling-app'
 
       - name: Invalidate bucket cache on latest*.yml and last_download.json files
-        run: pnpm files:invalidate-bucket
+        run: yarn files:invalidate-bucket
 
       - name: Upload release files to Github
         if: ${{ github.event_name == 'release' }}

.github/workflows/static-analysis.yml

@@ -20,63 +20,53 @@ jobs:
     runs-on: 'ubuntu-22.04'
     steps:
       - uses: actions/checkout@v4
-      - uses: pnpm/action-setup@v4
       - uses: actions/setup-node@v4
         with:
           node-version-file: '.nvmrc'
-          cache: 'pnpm'
-      - run: pnpm install
-      - run: pnpm fmt-check
+          cache: 'yarn'
+      - run: yarn install
+      - run: yarn fmt-check
 
   yarn-build-wasm:
     runs-on: ubuntu-22.04
 
     steps:
       - uses: actions/checkout@v4
-      - uses: pnpm/action-setup@v4
       - uses: actions/setup-node@v4
         with:
           node-version-file: '.nvmrc'
-          cache: 'pnpm'
-      - run: pnpm install
-      - uses: taiki-e/install-action@2dbeb927f58939d3aa13bf06ba0c0a34b76b9bfb
-        with:
-          tool: wasm-pack
-      - run: pnpm build:wasm
+          cache: 'yarn'
+      - run: yarn install
+      - run: yarn build:wasm
 
   yarn-tsc:
     runs-on: ubuntu-22.04
 
     steps:
       - uses: actions/checkout@v4
-      - uses: pnpm/action-setup@v4
       - uses: actions/setup-node@v4
         with:
           node-version-file: '.nvmrc'
-          cache: 'pnpm'
-      - run: pnpm install
+          cache: 'yarn'
+      - run: yarn install
       - uses: Swatinem/rust-cache@v2
         with:
           workspaces: './src/wasm-lib'
 
-      - uses: taiki-e/install-action@2dbeb927f58939d3aa13bf06ba0c0a34b76b9bfb
-        with:
-          tool: wasm-pack
-      - run: pnpm build:wasm
-      - run: pnpm tsc
+      - run: yarn build:wasm
+      - run: yarn tsc
 
   yarn-lint:
     runs-on: ubuntu-22.04
 
     steps:
       - uses: actions/checkout@v4
-      - uses: pnpm/action-setup@v4
       - uses: actions/setup-node@v4
         with:
           node-version-file: '.nvmrc'
-          cache: 'pnpm'
-      - run: pnpm install
-      - run: pnpm lint
+          cache: 'yarn'
+      - run: yarn install
+      - run: yarn lint
 
   python-codespell:
     runs-on: ubuntu-22.04
@@ -96,29 +86,24 @@ jobs:
     steps:
       - uses: actions/checkout@v4
 
-      - uses: pnpm/action-setup@v4
-
       - uses: actions/setup-node@v4
         with:
           node-version-file: '.nvmrc'
-          cache: 'pnpm'
+          cache: 'yarn'
 
-      - run: pnpm install
-      - uses: taiki-e/install-action@2dbeb927f58939d3aa13bf06ba0c0a34b76b9bfb
-        with:
-          tool: wasm-pack
-      - run: pnpm build:wasm
+      - run: yarn install
+      - run: yarn build:wasm
 
-      - run: pnpm simpleserver:bg
+      - run: yarn simpleserver:bg
         if: ${{ github.event_name != 'release' && github.event_name != 'schedule' }}
 
       - name: Install Chromium Browser
         if: ${{ github.event_name != 'release' && github.event_name != 'schedule' }}
-        run: pnpm playwright install chromium --with-deps
+        run: yarn playwright install chromium --with-deps
 
       - name: run unit tests for kcl samples
         if: ${{ github.event_name != 'release' && github.event_name != 'schedule' }}
-        run: pnpm test:unit:kcl-samples
+        run: yarn test:unit:kcl-samples
         env:
           VITE_KC_DEV_TOKEN: ${{ secrets.KITTYCAD_API_TOKEN_DEV }}
 
@@ -127,29 +112,24 @@ jobs:
     steps:
       - uses: actions/checkout@v4
 
-      - uses: pnpm/action-setup@v4
-
       - uses: actions/setup-node@v4
         with:
           node-version-file: '.nvmrc'
-          cache: 'pnpm'
+          cache: 'yarn'
 
-      - run: pnpm install
-      - uses: taiki-e/install-action@2dbeb927f58939d3aa13bf06ba0c0a34b76b9bfb
-        with:
-          tool: wasm-pack
-      - run: pnpm build:wasm
+      - run: yarn install
+      - run: yarn build:wasm
 
-      - run: pnpm simpleserver:bg
+      - run: yarn simpleserver:bg
         if: ${{ github.event_name != 'release' && github.event_name != 'schedule' }}
 
       - name: Install Chromium Browser
         if: ${{ github.event_name != 'release' && github.event_name != 'schedule' }}
-        run: pnpm playwright install chromium --with-deps
+        run: yarn playwright install chromium --with-deps
 
       - name: run unit tests
         if: ${{ github.event_name != 'release' && github.event_name != 'schedule' }}
-        run: pnpm test:unit
+        run: yarn test:unit
         env:
           VITE_KC_DEV_TOKEN: ${{ secrets.KITTYCAD_API_TOKEN_DEV }}
 

.github/workflows/tag-nightly.yml

@@ -1,41 +0,0 @@
-name: tag-nightly
-
-permissions:
-  contents: write
-on:
-  schedule:
-    - cron: '0 4 * * *'
-  # Daily at 04:00 AM UTC
-  # Will checkout the last commit from the default branch (main as of 2023-10-04)
-  
-jobs:
-  tag-nightly:
-    runs-on: ubuntu-22.04
-    steps:
-      - uses: actions/create-github-app-token@v1
-        id: app-token
-        with:
-          app-id: ${{ secrets.MODELING_APP_GH_APP_ID }}
-          private-key: ${{ secrets.MODELING_APP_GH_APP_PRIVATE_KEY }}
-          owner: ${{ github.repository_owner }}
-
-      - uses: actions/checkout@v4
-        with:
-          token: ${{ steps.app-token.outputs.token }}
-
-      - uses: pnpm/action-setup@v4
-
-      - uses: actions/setup-node@v4
-        with:
-          node-version-file: '.nvmrc'
-
-      - run: pnpm install
-
-      - name: Push tag
-        run: |
-          VERSION_NO_V=$(date +'%-y.%-m.%-d')
-          TAG="nightly-v$VERSION_NO_V"
-          git config --local user.email "github-actions[bot]@users.noreply.github.com"
-          git config --local user.name "github-actions[bot]"
-          git tag $TAG
-          git push origin tag $TAG

.gitignore

@@ -41,12 +41,9 @@ e2e/playwright/playwright-secrets.env
 e2e/playwright/temp1.png
 e2e/playwright/temp2.png
 e2e/playwright/temp3.png
-# this will be overridden for specific directories
-e2e/playwright/**/*.png
 # exports from snapshot-tests.spec.ts "exports of each format should work"
 e2e/playwright/export-snapshots/*
 !e2e/playwright/export-snapshots/*.png
-!e2e/playwright/snapshot-tests.spec.ts-snapshots/*.png
 
 /kcl-samples
 /test-results/
@@ -71,4 +68,3 @@ venv
 
 # electron
 out/
-*.snap.new

.husky/pre-push

@@ -1,4 +1,4 @@
 #!/usr/bin/env sh
 . "$(dirname -- "$0")/_/husky.sh"
 
-pnpm fmt-check
+yarn fmt-check

.prettierignore

@@ -10,7 +10,6 @@ target
 src/wasm-lib/pkg
 src/wasm-lib/kcl/bindings
 e2e/playwright/export-snapshots
-e2e/playwright/snapshots/prompt-to-edit
 
 
 # XState generated files

Makefile

@@ -5,21 +5,21 @@ TS_SRC := $(wildcard src/**/*.tsx) $(wildcard src/**/*.ts)
 XSTATE_TYPEGENS := $(wildcard src/machines/*.typegen.ts)
 
 dev: node_modules public/wasm_lib_bg.wasm $(XSTATE_TYPEGENS)
-	pnpm start
+	yarn start
 
 # I'm sorry this is so specific to my setup you may as well ignore this.
 # This is so you don't have to deal with electron windows popping up constantly.
 # It should work for you other Linux users.
 lee-electron-test:
 	Xephyr -br -ac -noreset -screen 1200x500 :2 &
-	DISPLAY=:2 NODE_ENV=development PW_TEST_CONNECT_WS_ENDPOINT=ws://127.0.0.1:4444/ pnpm tron:test -g "when using the file tree"
+	DISPLAY=:2 NODE_ENV=development PW_TEST_CONNECT_WS_ENDPOINT=ws://127.0.0.1:4444/ yarn tron:test -g "when using the file tree"
 	killall Xephyr
 
 $(XSTATE_TYPEGENS): $(TS_SRC)
-	pnpm xstate typegen 'src/**/*.ts?(x)'
+	yarn xstate typegen 'src/**/*.ts?(x)'
 
 public/wasm_lib_bg.wasm: $(WASM_LIB_FILES)
-	pnpm build:wasm
+	yarn build:wasm
 
 node_modules: package.json yarn.lock
-	pnpm install
+	yarn install

README.md

@@ -48,49 +48,22 @@ We recommend downloading the latest application binary from [our Releases page](
 
 ## Running a development build
 
-Install a node version manager such as [fnm](https://github.com/Schniz/fnm?tab=readme-ov-#installation).
-
-On Windows, it's also recommended to [upgrade your PowerShell version](https://learn.microsoft.com/en-us/powershell/scripting/install/installing-powershell-on-windows?view=powershell-7.5#winget), we're using 7.
-
-Then in the repo run the following to install and use the node version specified in `.nvmrc`. You might need to specify your processor architecture with `--arch arm64` or `--arch x64` if it's not autodetected. 
-```
-fnm install --corepack-enabled
-fnm use
-```
-
-Install the NPM dependencies with:
-```
-pnpm install
-```
-
-This project uses a lot of Rust compiled to [WASM](https://webassembly.org/) within it. We have package scripts to run rustup, see `package.json` for reference:
-```
-# macOS/Linux
-pnpm install:rust
-pnpm install:wasm-pack:sh
-
-# Windows
-pnpm install:rust:windows
-pnpm install:wasm-pack:cargo
-```
-
-Then to build the WASM layer, run:
-```
-# macOS/Linux
-pnpm build:wasm
-
-# Windows
-pnpm build:wasm:windows
-```
-
-Or if you have the `gh` cli installed and want to download the latest main wasm bundle. Note that on Windows, you need to associate .ps1 files with PowerShell, which can be done via the right click menu, selecting `C:\Program Files\PowerShell\7\pwsh.exe`, and you can install tools like `gh` via `pnpm install:tools:windows`.
+First, [install Rust via `rustup`](https://www.rust-lang.org/tools/install). This project uses a lot of Rust compiled to [WASM](https://webassembly.org/) within it. We always use the latest stable version of Rust, so you may need to run `rustup update stable`. Then, run:
 
 ```
-# macOS/Linux
-pnpm fetch:wasm
+yarn install
+```
 
-# Windows
-pnpm fetch:wasm:windows
+followed by:
+
+```
+yarn build:wasm
+```
+
+or if you have the gh cli installed
+
+```
+./get-latest-wasm-bundle.sh # this will download the latest main wasm bundle
 ```
 
 That will build the WASM binary and put in the `public` dir (though gitignored).
@@ -98,10 +71,10 @@ That will build the WASM binary and put in the `public` dir (though gitignored).
 Finally, to run the web app only, run:
 
 ```
-pnpm start
+yarn start
 ```
 
-If you're not a Zoo employee you won't be able to access the dev environment, you should copy everything from `.env.production` to `.env.development` to make it point to production instead, then when you navigate to `localhost:3000` the easiest way to sign in is to paste `localStorage.setItem('TOKEN_PERSIST_KEY', "your-token-from-https://zoo.dev/account/api-tokens")` replacing the with a real token from https://zoo.dev/account/api-tokens of course, then navigate to localhost:3000 again. Note that navigating to `localhost:3000/signin` removes your token so you will need to set the token again.
+If you're not an KittyCAD employee you won't be able to access the dev environment, you should copy everything from `.env.production` to `.env.development` to make it point to production instead, then when you navigate to `localhost:3000` the easiest way to sign in is to paste `localStorage.setItem('TOKEN_PERSIST_KEY', "your-token-from-https://zoo.dev/account/api-tokens")` replacing the with a real token from https://zoo.dev/account/api-tokens of course, then navigate to localhost:3000 again. Note that navigating to `localhost:3000/signin` removes your token so you will need to set the token again.
 
 ### Development environment variables
 
@@ -118,17 +91,17 @@ Third-Party Cookies".
 
 ## Desktop
 
-To spin up the desktop app, `pnpm install` and `pnpm build:wasm` need to have been done before hand then
+To spin up the desktop app, `yarn install` and `yarn build:wasm` need to have been done before hand then
 
 ```
-pnpm tron:start
+yarn tron:start
 ```
 
 This will start the application and hot-reload on changes.
 
 Devtools can be opened with the usual Cmd-Opt-I (Mac) or Ctrl-Shift-I (Linux and Windows).
 
-To package the app for your platform with electron-builder, run `pnpm tronb:package:dev` (or `pnpm tronb:package:prod` to point to the .env.production variables)
+To build with electron-builder, run `yarn tronb:package:dev` (or `yarn tronb:package:prod` to point to the .env.production variables)
 
 ## Checking out commits / Bisecting
 
@@ -136,9 +109,9 @@ Which commands from setup are one off vs need to be run every time?
 
 The following will need to be run when checking out a new commit and guarantees the build is not stale:
 ```bash
-pnpm install
-pnpm build:wasm
-pnpm start # or pnpm build:local && pnpm serve for slower but more production-like build
+yarn install
+yarn build:wasm
+yarn start # or yarn build:local && yarn serve for slower but more production-like build
 ```
 
 ## Before submitting a PR
@@ -148,10 +121,10 @@ Before you submit a contribution PR to this repo, please ensure that:
 - There is a corresponding issue for the changes you want to make, so that discussion of approach can be had before work begins.
 - You have separated out refactoring commits from feature commits as much as possible
 - You have run all of the following commands locally:
-  - `pnpm fmt`
-  - `pnpm tsc`
-  - `pnpm test`
-  - Here they are all together: `pnpm fmt && pnpm tsc && pnpm test`
+  - `yarn fmt`
+  - `yarn tsc`
+  - `yarn test`
+  - Here they are all together: `yarn fmt && yarn tsc && yarn test`
 
 ## Release a new version
 
@@ -260,7 +233,7 @@ docker run --network host  --rm --init -it playwright/chrome:playwright-x.xx.x
 and in another terminal, run:
 
 ```bash
-PW_TEST_CONNECT_WS_ENDPOINT=ws://127.0.0.1:4444/ pnpm playwright test --project="Google Chrome" <test suite>
+PW_TEST_CONNECT_WS_ENDPOINT=ws://127.0.0.1:4444/ yarn playwright test --project="Google Chrome" <test suite>
 ```
 
 
@@ -275,7 +248,7 @@ docker run --network host  --rm --init -it playwright/chrome:playwright-1.46.0
 and in another terminal, run:
 
 ```bash
-PW_TEST_CONNECT_WS_ENDPOINT=ws://127.0.0.1:4444/ pnpm playwright test --project="Google Chrome" e2e/playwright/command-bar-tests.spec.ts
+PW_TEST_CONNECT_WS_ENDPOINT=ws://127.0.0.1:4444/ yarn playwright test --project="Google Chrome" e2e/playwright/command-bar-tests.spec.ts
 ```
 
 run a specific test change the test from `test('...` to `test.only('...`
@@ -287,13 +260,13 @@ run a specific test change the test from `test('...` to `test.only('...`
 run headed
 
 ```
-pnpm playwright test --headed
+yarn playwright test --headed
 ```
 
 run with step through debugger
 
 ```
-PWDEBUG=1 pnpm playwright test
+PWDEBUG=1 yarn playwright test
 ```
 
 However, if you want a debugger I recommend using VSCode and the `playwright` extension, as the above command is a cruder debugger that steps into every function call which is annoying.
@@ -304,7 +277,7 @@ Or comment out browsers in `playwright.config.ts`.
 
 note chromium has encoder compat issues which is why were testing against the branded 'Google Chrome'
 
-You may consider using the VSCode extension, it's useful for running individual threads, but some some reason the "record a test" is locked to chromium with we can't use. A work around is to us the CI `pnpm playwright codegen -b wk --load-storage ./store localhost:3000`
+You may consider using the VSCode extension, it's useful for running individual threads, but some some reason the "record a test" is locked to chromium with we can't use. A work around is to us the CI `yarn playwright codegen -b wk --load-storage ./store localhost:3000`
 
 <details>
 <summary>
@@ -347,21 +320,21 @@ However because much of our tests involve clicking in the stream at specific loc
 If you already haven't, run the following:
 
 ```
-pnpm install
-pnpm build:wasm
-pnpm start
+yarn
+yarn build:wasm
+yarn start
 ```
 
 and finally:
 
 ```
-pnpm test:unit
+yarn test:unit
 ```
 
 For individual testing:
 
 ```
-pnpm test abstractSyntaxTree -t "unexpected closed curly brace" --silent=false
+yarn test abstractSyntaxTree -t "unexpected closed curly brace" --silent=false
 ```
 
 Which will run our suite of [Vitest unit](https://vitest.dev/) and [React Testing Library E2E](https://testing-library.com/docs/react-testing-library/intro) tests, in interactive mode by default.
@@ -432,17 +405,17 @@ Part of the CI CD pipeline performs static analysis on the code. Use the followi
 The following set of commands should get us closer to one and done commands to instantly retest issues.
 
 ```
-pnpm test-setup
+yarn test-setup
 ```
 
 > Gotcha, are packages up to date and is the wasm built?
 
 
 ```
-pnpm tsc
-pnpm fmt-check
-pnpm lint
-pnpm test:unit:local
+yarn tsc
+yarn fmt-check
+yarn lint
+yarn test:unit:local
 ```
 
 > Gotcha: Our unit tests have integration tests in them. You need to run a localhost server to run the unit tests.
@@ -454,10 +427,10 @@ pnpm test:unit:local
 These E2E tests run in electron. There are tests that are skipped if they are ran in a windows, linux, or macos environment. We can use playwright tags to implement test skipping.
 
 ```
-pnpm test:playwright:electron:local
-pnpm test:playwright:electron:windows:local
-pnpm test:playwright:electron:macos:local
-pnpm test:playwright:electron:ubuntu:local
+yarn test:playwright:electron:local
+yarn test:playwright:electron:windows:local
+yarn test:playwright:electron:macos:local
+yarn test:playwright:electron:ubuntu:local
 ```
 
 > Why does it say local? The CI CD commands that run in the pipeline cannot be ran locally. A single command will not properly setup the testing environment locally.

docs/kcl/assert.md

@@ -9,10 +9,7 @@ Check a value at runtime, and raise an error if the argument provided
 is false.
 
 ```js
-assert(
-  data: bool,
-  message: string,
-) -> ()
+assert(data: bool, message: string) -> ()
 ```
 
 
@@ -25,7 +22,7 @@ assert(
 
 ### Returns
 
-`()`
+`()` 
 
 
 ### Examples

docs/kcl/assertEqual.md

@@ -9,12 +9,7 @@ Check that a numerical value equals another at runtime,
 otherwise raise an error.
 
 ```js
-assertEqual(
-  left: number,
-  right: number,
-  epsilon: number,
-  message: string,
-) -> ()
+assertEqual(left: number, right: number, epsilon: number, message: string) -> ()
 ```
 
 
@@ -29,7 +24,7 @@ assertEqual(
 
 ### Returns
 
-`()`
+`()` 
 
 
 ### Examples

docs/kcl/assertGreaterThan.md

@@ -9,11 +9,7 @@ Check that a numerical value is greater than another at runtime,
 otherwise raise an error.
 
 ```js
-assertGreaterThan(
-  left: number,
-  right: number,
-  message: string,
-) -> ()
+assertGreaterThan(left: number, right: number, message: string) -> ()
 ```
 
 
@@ -27,7 +23,7 @@ assertGreaterThan(
 
 ### Returns
 
-`()`
+`()` 
 
 
 ### Examples

docs/kcl/assertGreaterThanOrEq.md

@@ -9,11 +9,7 @@ Check that a numerical value is greater than or equal to another at runtime,
 otherwise raise an error.
 
 ```js
-assertGreaterThanOrEq(
-  left: number,
-  right: number,
-  message: string,
-) -> ()
+assertGreaterThanOrEq(left: number, right: number, message: string) -> ()
 ```
 
 
@@ -27,7 +23,7 @@ assertGreaterThanOrEq(
 
 ### Returns
 
-`()`
+`()` 
 
 
 ### Examples

docs/kcl/assertLessThan.md

@@ -9,11 +9,7 @@ Check that a numerical value is less than to another at runtime,
 otherwise raise an error.
 
 ```js
-assertLessThan(
-  left: number,
-  right: number,
-  message: string,
-) -> ()
+assertLessThan(left: number, right: number, message: string) -> ()
 ```
 
 
@@ -27,7 +23,7 @@ assertLessThan(
 
 ### Returns
 
-`()`
+`()` 
 
 
 ### Examples

docs/kcl/assertLessThanOrEq.md

@@ -9,11 +9,7 @@ Check that a numerical value is less than or equal to another at runtime,
 otherwise raise an error.
 
 ```js
-assertLessThanOrEq(
-  left: number,
-  right: number,
-  message: string,
-) -> ()
+assertLessThanOrEq(left: number, right: number, message: string) -> ()
 ```
 
 
@@ -27,7 +23,7 @@ assertLessThanOrEq(
 
 ### Returns
 
-`()`
+`()` 
 
 
 ### Examples

docs/kcl/cm.md

@@ -26,7 +26,7 @@ cm() -> number
 
 ### Returns
 
-`number`
+`number` 
 
 
 ### Examples

docs/kcl/const_std-HALF_TURN.md

@@ -1,15 +0,0 @@
----
-title: "std::HALF_TURN"
-excerpt: ""
-layout: manual
----
-
-
-
-
-
-```js
-std::HALF_TURN: number(deg) = 180deg
-```
-
-

docs/kcl/const_std-QUARTER_TURN.md

@@ -1,15 +0,0 @@
----
-title: "std::QUARTER_TURN"
-excerpt: ""
-layout: manual
----
-
-
-
-
-
-```js
-std::QUARTER_TURN: number(deg) = 90deg
-```
-
-

docs/kcl/const_std-THREE_QUARTER_TURN.md

@@ -1,15 +0,0 @@
----
-title: "std::THREE_QUARTER_TURN"
-excerpt: ""
-layout: manual
----
-
-
-
-
-
-```js
-std::THREE_QUARTER_TURN: number(deg) = 270deg
-```
-
-

docs/kcl/const_std-XY.md

@@ -1,15 +0,0 @@
----
-title: "std::XY"
-excerpt: ""
-layout: manual
----
-
-
-
-
-
-```js
-std::XY
-```
-
-

docs/kcl/const_std-XZ.md

@@ -1,15 +0,0 @@
----
-title: "std::XZ"
-excerpt: ""
-layout: manual
----
-
-
-
-
-
-```js
-std::XZ
-```
-
-

docs/kcl/const_std-YZ.md

@@ -1,15 +0,0 @@
----
-title: "std::YZ"
-excerpt: ""
-layout: manual
----
-
-
-
-
-
-```js
-std::YZ
-```
-
-

docs/kcl/const_std-ZERO.md

@@ -1,15 +0,0 @@
----
-title: "std::ZERO"
-excerpt: ""
-layout: manual
----
-
-
-
-
-
-```js
-std::ZERO: number = 0
-```
-
-

docs/kcl/ft.md

@@ -26,7 +26,7 @@ ft() -> number
 
 ### Returns
 
-`number`
+`number` 
 
 
 ### Examples

docs/kcl/inch.md

@@ -26,7 +26,7 @@ inch() -> number
 
 ### Returns
 
-`number`
+`number` 
 
 
 ### Examples

docs/kcl/index.md

@@ -6,131 +6,113 @@ layout: manual
 
 ## Table of Contents
 
-### Language
-
-* [`Types`](kcl/types)
-* [`Modules`](kcl/modules)
-* [`Settings`](kcl/settings)
-* [`Known Issues`](kcl/known-issues)
-
-### Standard library
-
-* **`std`**
-  * [`HALF_TURN`](kcl/const_std-HALF_TURN)
-  * [`QUARTER_TURN`](kcl/const_std-QUARTER_TURN)
-  * [`THREE_QUARTER_TURN`](kcl/const_std-THREE_QUARTER_TURN)
-  * [`XY`](kcl/const_std-XY)
-  * [`XZ`](kcl/const_std-XZ)
-  * [`YZ`](kcl/const_std-YZ)
-  * [`ZERO`](kcl/const_std-ZERO)
-  * [`abs`](kcl/abs)
-  * [`acos`](kcl/acos)
-  * [`angleToMatchLengthX`](kcl/angleToMatchLengthX)
-  * [`angleToMatchLengthY`](kcl/angleToMatchLengthY)
-  * [`angledLine`](kcl/angledLine)
-  * [`angledLineOfXLength`](kcl/angledLineOfXLength)
-  * [`angledLineOfYLength`](kcl/angledLineOfYLength)
-  * [`angledLineThatIntersects`](kcl/angledLineThatIntersects)
-  * [`angledLineToX`](kcl/angledLineToX)
-  * [`angledLineToY`](kcl/angledLineToY)
-  * [`appearance`](kcl/appearance)
-  * [`arc`](kcl/arc)
-  * [`arcTo`](kcl/arcTo)
-  * [`asin`](kcl/asin)
-  * [`assert`](kcl/assert)
-  * [`assertEqual`](kcl/assertEqual)
-  * [`assertGreaterThan`](kcl/assertGreaterThan)
-  * [`assertGreaterThanOrEq`](kcl/assertGreaterThanOrEq)
-  * [`assertLessThan`](kcl/assertLessThan)
-  * [`assertLessThanOrEq`](kcl/assertLessThanOrEq)
-  * [`atan`](kcl/atan)
-  * [`atan2`](kcl/atan2)
-  * [`bezierCurve`](kcl/bezierCurve)
-  * [`ceil`](kcl/ceil)
-  * [`chamfer`](kcl/chamfer)
-  * [`circle`](kcl/circle)
-  * [`circleThreePoint`](kcl/circleThreePoint)
-  * [`close`](kcl/close)
-  * [`cm`](kcl/cm)
-  * [`extrude`](kcl/extrude)
-  * [`fillet`](kcl/fillet)
-  * [`floor`](kcl/floor)
-  * [`ft`](kcl/ft)
-  * [`getNextAdjacentEdge`](kcl/getNextAdjacentEdge)
-  * [`getOppositeEdge`](kcl/getOppositeEdge)
-  * [`getPreviousAdjacentEdge`](kcl/getPreviousAdjacentEdge)
-  * [`helix`](kcl/helix)
-  * [`helixRevolutions`](kcl/helixRevolutions)
-  * [`hole`](kcl/hole)
-  * [`hollow`](kcl/hollow)
-  * [`inch`](kcl/inch)
-  * [`lastSegX`](kcl/lastSegX)
-  * [`lastSegY`](kcl/lastSegY)
-  * [`legAngX`](kcl/legAngX)
-  * [`legAngY`](kcl/legAngY)
-  * [`legLen`](kcl/legLen)
-  * [`line`](kcl/line)
-  * [`ln`](kcl/ln)
-  * [`loft`](kcl/loft)
-  * [`log`](kcl/log)
-  * [`log10`](kcl/log10)
-  * [`log2`](kcl/log2)
-  * [`m`](kcl/m)
-  * [`map`](kcl/map)
-  * [`max`](kcl/max)
-  * [`min`](kcl/min)
-  * [`mirror2d`](kcl/mirror2d)
-  * [`mm`](kcl/mm)
-  * [`offsetPlane`](kcl/offsetPlane)
-  * [`patternCircular2d`](kcl/patternCircular2d)
-  * [`patternCircular3d`](kcl/patternCircular3d)
-  * [`patternLinear2d`](kcl/patternLinear2d)
-  * [`patternLinear3d`](kcl/patternLinear3d)
-  * [`patternTransform`](kcl/patternTransform)
-  * [`patternTransform2d`](kcl/patternTransform2d)
-  * [`polar`](kcl/polar)
-  * [`polygon`](kcl/polygon)
-  * [`pop`](kcl/pop)
-  * [`pow`](kcl/pow)
-  * [`profileStart`](kcl/profileStart)
-  * [`profileStartX`](kcl/profileStartX)
-  * [`profileStartY`](kcl/profileStartY)
-  * [`push`](kcl/push)
-  * [`reduce`](kcl/reduce)
-  * [`rem`](kcl/rem)
-  * [`revolve`](kcl/revolve)
-  * [`rotate`](kcl/rotate)
-  * [`round`](kcl/round)
-  * [`scale`](kcl/scale)
-  * [`segAng`](kcl/segAng)
-  * [`segEnd`](kcl/segEnd)
-  * [`segEndX`](kcl/segEndX)
-  * [`segEndY`](kcl/segEndY)
-  * [`segLen`](kcl/segLen)
-  * [`segStart`](kcl/segStart)
-  * [`segStartX`](kcl/segStartX)
-  * [`segStartY`](kcl/segStartY)
-  * [`shell`](kcl/shell)
-  * [`sqrt`](kcl/sqrt)
-  * [`startProfileAt`](kcl/startProfileAt)
-  * [`startSketchOn`](kcl/startSketchOn)
-  * [`sweep`](kcl/sweep)
-  * [`tangentToEnd`](kcl/tangentToEnd)
-  * [`tangentialArc`](kcl/tangentialArc)
-  * [`tangentialArcTo`](kcl/tangentialArcTo)
-  * [`tangentialArcToRelative`](kcl/tangentialArcToRelative)
-  * [`toDegrees`](kcl/toDegrees)
-  * [`toRadians`](kcl/toRadians)
-  * [`translate`](kcl/translate)
-  * [`xLine`](kcl/xLine)
-  * [`xLineTo`](kcl/xLineTo)
-  * [`yLine`](kcl/yLine)
-  * [`yLineTo`](kcl/yLineTo)
-  * [`yd`](kcl/yd)
-* **`std::math`**
-  * [`E`](kcl/const_std-math-E)
-  * [`PI`](kcl/const_std-math-PI)
-  * [`TAU`](kcl/const_std-math-TAU)
-  * [`cos`](kcl/std-math-cos)
-  * [`sin`](kcl/std-math-sin)
-  * [`tan`](kcl/std-math-tan)
+* [Types](kcl/types)
+* [Modules](kcl/modules)
+* [Known Issues](kcl/KNOWN-ISSUES)
+* [`abs`](kcl/abs)
+* [`acos`](kcl/acos)
+* [`angleToMatchLengthX`](kcl/angleToMatchLengthX)
+* [`angleToMatchLengthY`](kcl/angleToMatchLengthY)
+* [`angledLine`](kcl/angledLine)
+* [`angledLineOfXLength`](kcl/angledLineOfXLength)
+* [`angledLineOfYLength`](kcl/angledLineOfYLength)
+* [`angledLineThatIntersects`](kcl/angledLineThatIntersects)
+* [`angledLineToX`](kcl/angledLineToX)
+* [`angledLineToY`](kcl/angledLineToY)
+* [`appearance`](kcl/appearance)
+* [`arc`](kcl/arc)
+* [`arcTo`](kcl/arcTo)
+* [`asin`](kcl/asin)
+* [`assert`](kcl/assert)
+* [`assertEqual`](kcl/assertEqual)
+* [`assertGreaterThan`](kcl/assertGreaterThan)
+* [`assertGreaterThanOrEq`](kcl/assertGreaterThanOrEq)
+* [`assertLessThan`](kcl/assertLessThan)
+* [`assertLessThanOrEq`](kcl/assertLessThanOrEq)
+* [`atan`](kcl/atan)
+* [`atan2`](kcl/atan2)
+* [`bezierCurve`](kcl/bezierCurve)
+* [`ceil`](kcl/ceil)
+* [`chamfer`](kcl/chamfer)
+* [`circle`](kcl/circle)
+* [`circleThreePoint`](kcl/circleThreePoint)
+* [`close`](kcl/close)
+* [`cm`](kcl/cm)
+* [`cos`](kcl/cos)
+* [`e`](kcl/e)
+* [`extrude`](kcl/extrude)
+* [`fillet`](kcl/fillet)
+* [`floor`](kcl/floor)
+* [`ft`](kcl/ft)
+* [`getNextAdjacentEdge`](kcl/getNextAdjacentEdge)
+* [`getOppositeEdge`](kcl/getOppositeEdge)
+* [`getPreviousAdjacentEdge`](kcl/getPreviousAdjacentEdge)
+* [`helix`](kcl/helix)
+* [`helixRevolutions`](kcl/helixRevolutions)
+* [`hole`](kcl/hole)
+* [`hollow`](kcl/hollow)
+* [`inch`](kcl/inch)
+* [`lastSegX`](kcl/lastSegX)
+* [`lastSegY`](kcl/lastSegY)
+* [`legAngX`](kcl/legAngX)
+* [`legAngY`](kcl/legAngY)
+* [`legLen`](kcl/legLen)
+* [`line`](kcl/line)
+* [`ln`](kcl/ln)
+* [`loft`](kcl/loft)
+* [`log`](kcl/log)
+* [`log10`](kcl/log10)
+* [`log2`](kcl/log2)
+* [`m`](kcl/m)
+* [`map`](kcl/map)
+* [`max`](kcl/max)
+* [`min`](kcl/min)
+* [`mirror2d`](kcl/mirror2d)
+* [`mm`](kcl/mm)
+* [`offsetPlane`](kcl/offsetPlane)
+* [`patternCircular2d`](kcl/patternCircular2d)
+* [`patternCircular3d`](kcl/patternCircular3d)
+* [`patternLinear2d`](kcl/patternLinear2d)
+* [`patternLinear3d`](kcl/patternLinear3d)
+* [`patternTransform`](kcl/patternTransform)
+* [`patternTransform2d`](kcl/patternTransform2d)
+* [`pi`](kcl/pi)
+* [`polar`](kcl/polar)
+* [`polygon`](kcl/polygon)
+* [`pop`](kcl/pop)
+* [`pow`](kcl/pow)
+* [`profileStart`](kcl/profileStart)
+* [`profileStartX`](kcl/profileStartX)
+* [`profileStartY`](kcl/profileStartY)
+* [`push`](kcl/push)
+* [`reduce`](kcl/reduce)
+* [`rem`](kcl/rem)
+* [`revolve`](kcl/revolve)
+* [`round`](kcl/round)
+* [`segAng`](kcl/segAng)
+* [`segEnd`](kcl/segEnd)
+* [`segEndX`](kcl/segEndX)
+* [`segEndY`](kcl/segEndY)
+* [`segLen`](kcl/segLen)
+* [`segStart`](kcl/segStart)
+* [`segStartX`](kcl/segStartX)
+* [`segStartY`](kcl/segStartY)
+* [`shell`](kcl/shell)
+* [`sin`](kcl/sin)
+* [`sqrt`](kcl/sqrt)
+* [`startProfileAt`](kcl/startProfileAt)
+* [`startSketchOn`](kcl/startSketchOn)
+* [`sweep`](kcl/sweep)
+* [`tan`](kcl/tan)
+* [`tangentToEnd`](kcl/tangentToEnd)
+* [`tangentialArc`](kcl/tangentialArc)
+* [`tangentialArcTo`](kcl/tangentialArcTo)
+* [`tangentialArcToRelative`](kcl/tangentialArcToRelative)
+* [`tau`](kcl/tau)
+* [`toDegrees`](kcl/toDegrees)
+* [`toRadians`](kcl/toRadians)
+* [`xLine`](kcl/xLine)
+* [`xLineTo`](kcl/xLineTo)
+* [`yLine`](kcl/yLine)
+* [`yLineTo`](kcl/yLineTo)
+* [`yd`](kcl/yd)

docs/kcl/legAngX.md

@@ -9,10 +9,7 @@ Compute the angle of the given leg for x.
 
 
 ```js
-legAngX(
-  hypotenuse: number,
-  leg: number,
-) -> number
+legAngX(hypotenuse: number, leg: number) -> number
 ```
 
 ### Tags
@@ -29,7 +26,7 @@ legAngX(
 
 ### Returns
 
-`number`
+`number` 
 
 
 ### Examples

docs/kcl/legAngY.md

@@ -9,10 +9,7 @@ Compute the angle of the given leg for y.
 
 
 ```js
-legAngY(
-  hypotenuse: number,
-  leg: number,
-) -> number
+legAngY(hypotenuse: number, leg: number) -> number
 ```
 
 ### Tags
@@ -29,7 +26,7 @@ legAngY(
 
 ### Returns
 
-`number`
+`number` 
 
 
 ### Examples

docs/kcl/legLen.md

@@ -9,10 +9,7 @@ Compute the length of the given leg.
 
 
 ```js
-legLen(
-  hypotenuse: number,
-  leg: number,
-) -> number
+legLen(hypotenuse: number, leg: number) -> number
 ```
 
 ### Tags
@@ -29,7 +26,7 @@ legLen(
 
 ### Returns
 
-`number`
+`number` 
 
 
 ### Examples

docs/kcl/m.md

@@ -26,7 +26,7 @@ m() -> number
 
 ### Returns
 
-`number`
+`number` 
 
 
 ### Examples

docs/kcl/map.md

@@ -9,10 +9,7 @@ Apply a function to every element of a list.
 Given a list like `[a, b, c]`, and a function like `f`, returns `[f(a), f(b), f(c)]`
 
 ```js
-map(
-  array: [KclValue],
-  mapFn: FunctionSource,
-) -> [KclValue]
+map(array: [KclValue], map_fn: FunctionParam) -> [KclValue]
 ```
 
 
@@ -21,11 +18,11 @@ map(
 | Name | Type | Description | Required |
 |----------|------|-------------|----------|
 | `array` | [`[KclValue]`](/docs/kcl/types/KclValue) |  | Yes |
-| `mapFn` | `FunctionSource` |  | Yes |
+| `map_fn` | `FunctionParam` |  | Yes |
 
 ### Returns
 
-[`[KclValue]`](/docs/kcl/types/KclValue)
+[`[KclValue]`](/docs/kcl/types/KclValue) 
 
 
 ### Examples

docs/kcl/mm.md

@@ -26,7 +26,7 @@ mm() -> number
 
 ### Returns
 
-`number`
+`number` 
 
 
 ### Examples

docs/kcl/modules.md

@@ -20,7 +20,7 @@ export fn increment(x) {
 Other files in the project can now import functions that have been exported.
 This makes them available to use in another file.
 
-```norun
+```
 // main.kcl
 import increment from "util.kcl"
 
@@ -48,64 +48,12 @@ export fn decrement(x) {
 
 When importing, you can import multiple functions at once.
 
-```norun
+```
 import increment, decrement from "util.kcl"
 ```
 
 Imported symbols can be renamed for convenience or to avoid name collisions.
 
-```norun
+```
 import increment as inc, decrement as dec from "util.kcl"
 ```
-
-## Importing files from other CAD systems
-
-`import` can also be used to import files from other CAD systems. The format of the statement is the
-same as for KCL files. You can only import the whole file, not items from it. E.g.,
-
-```norun
-import "tests/inputs/cube.obj"
-
-// Use `cube` just like a KCL object.
-```
-
-```norun
-import "tests/inputs/cube-2.sldprt" as cube
-
-// Use `cube` just like a KCL object.
-```
-
-You can make the file format explicit using a format attribute (useful if using a different
-extension), e.g.,
-
-```norun
-@(format = obj)
-import "tests/inputs/cube"
-```
-
-For formats lacking unit data (such as STL, OBJ, or PLY files), the default
-unit of measurement is millimeters. Alternatively you may specify the unit
-by using an attirbute. Likewise, you can also specify a coordinate system. E.g.,
-
-```norun
-@(unitLength = ft, coords = opengl)
-import "tests/inputs/cube.obj"
-```
-
-When importing a GLTF file, the bin file will be imported as well.
-
-Import paths are relative to the current project directory. Imports currently only work when
-using the native Modeling App, not in the browser.
-
-### Supported values
-
-File formats: `fbx`, `gltf`/`glb`, `obj`+, `ply`+, `sldprt`, `step`/`stp`, `stl`+. (Those marked with a
-'+' support customising the length unit and coordinate system).
-
-Length units: `mm` (the default), `cm`, `m`, `inch`, `ft`, `yd`.
-
-Coordinate systems:
-
-- `zoo` (the default), forward: -Y, up: +Z, handedness: right
-- `opengl`, forward: +Z, up: +Y, handedness: right
-- `vulkan`, forward: +Z, up: -Y, handedness: left