Add debouncing settle detection

Add debounce function
Add registry for background promises
2025-04-01 13:33:49 -04:00 · 2025-04-01 13:33:49 -04:00 · 2025-04-01 13:33:49 -04:00 · 2025-04-01 10:21:31 -07:00 · 2025-04-01 12:43:27 -04:00 · 2025-04-01 12:42:11 -04:00
329 changed files with 6894 additions and 6266 deletions
--- a/.eslintrc
+++ b/.eslintrc
@ -26,11 +26,17 @@
      "@typescript-eslint/no-duplicate-enum-values": "error",
      "@typescript-eslint/no-duplicate-type-constituents": "error",
      "@typescript-eslint/no-empty-object-type": "error",
+      "@typescript-eslint/no-extra-non-null-assertion": "error",
      "@typescript-eslint/no-floating-promises": "error",
+      "@typescript-eslint/no-for-in-array": "error",
      "no-implied-eval": "off", // This is wrong; use the @typescript-eslint one instead.
      "@typescript-eslint/no-implied-eval": "error",
+      "@typescript-eslint/no-misused-new": "error",
      "@typescript-eslint/no-misused-promises": "error",
      "@typescript-eslint/no-namespace": "error",
+      "@typescript-eslint/no-non-null-asserted-optional-chain": "error",
+      "@typescript-eslint/no-redundant-type-constituents": "error",
+      "@typescript-eslint/no-this-alias": "warn",
      "@typescript-eslint/no-unnecessary-type-assertion": "error",
      "@typescript-eslint/no-unnecessary-type-constraint": "error",
      "no-unused-vars": "off", // This is wrong; use the @typescript-eslint one instead.
@ -41,7 +47,13 @@
        "vars": "all",
        "args": "none"
      }],
+      "@typescript-eslint/no-unsafe-unary-minus": "error",
+      "@typescript-eslint/no-wrapper-object-types": "error",
+      "no-throw-literal": "off", // Use @typescript-eslint/only-throw-error instead.
+      "@typescript-eslint/only-throw-error": "error",
      "@typescript-eslint/prefer-as-const": "warn",
+      "@typescript-eslint/prefer-namespace-keyword": "error",
+      "@typescript-eslint/restrict-plus-operands": "error",
      "jsx-a11y/click-events-have-key-events": "off",
      "jsx-a11y/no-autofocus": "off",
      "jsx-a11y/no-noninteractive-element-interactions": "off",
@ -50,7 +62,7 @@
        {
          "name": "isNaN",
          "message": "Use Number.isNaN() instead."
-        },
+        }
      ],
      "no-restricted-syntax": [
        "error",
@ -64,7 +76,7 @@
        "never"
      ],
      "react-hooks/exhaustive-deps": "off",
-      "suggest-no-throw/suggest-no-throw": "warn",
+      "suggest-no-throw/suggest-no-throw": "error"
    },
    "overrides": [
      {
@ -84,7 +96,7 @@
          "plugin:testing-library/react"
        ],
        "rules": {
-          "suggest-no-throw/suggest-no-throw": "off",
+          "suggest-no-throw/suggest-no-throw": "off"
        }
      }
    ]
--- a/.github/workflows/e2e-tests.yml
+++ b/.github/workflows/e2e-tests.yml
@ -229,10 +229,6 @@ jobs:
          timeout_minutes: 30
          max_attempts: 3
        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 }}

@ -377,11 +373,7 @@ jobs:
          timeout_minutes: 45
          max_attempts: 15
        env:
-          CI: true
          FAIL_ON_CONSOLE_ERRORS: true
-          NODE_ENV: development
-          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
--- a/docs/kcl/std-sketch-circle.md
+++ b/docs/kcl/std-sketch-circle.md
@ -17,7 +17,7 @@ circle(@sketch_or_surface: Sketch | Plane | Face, center: Point2d, radius: numbe

 | Name | Type | Description | Required |
 |----------|------|-------------|----------|
-| `sketch_or_surface` | [`Sketch`](/docs/kcl/types/Sketch) `|` [`Plane`](/docs/kcl/types/Face) `|` [`Plane`](/docs/kcl/types/Face) | Sketch to extend, or plane or surface to sketch on. | Yes |
+| `sketch_or_surface` | [`Sketch`](/docs/kcl/types/Sketch) OR [`Plane`](/docs/kcl/types/Plane) OR [`Face`](/docs/kcl/types/Face) | Sketch to extend, or plane or surface to sketch on. | Yes |
 | `center` | [`Point2d`](/docs/kcl/types/Point2d) | The center of the circle. | Yes |
 | `radius` | [`number`](/docs/kcl/types/number) | The radius of the circle. | Yes |
 | [`tag`](/docs/kcl/types/tag) | [`tag`](/docs/kcl/types/tag) | Create a new tag which refers to this circle. | No |
--- a/docs/kcl/std.json
+++ b/docs/kcl/std.json
@ -207302,372 +207302,8 @@
          },
          "definitions": {
            "TyF64": {
-              "type": "object",
-              "required": [
-                "n",
-                "ty"
-              ],
-              "properties": {
-                "n": {
-                  "type": "number",
-                  "format": "double"
-                },
-                "ty": {
-                  "$ref": "#/components/schemas/NumericType"
-                }
-              }
-            },
-            "NumericType": {
-              "oneOf": [
-                {
-                  "type": "object",
-                  "oneOf": [
-                    {
-                      "type": "object",
-                      "required": [
-                        "type"
-                      ],
-                      "properties": {
-                        "type": {
-                          "type": "string",
-                          "enum": [
-                            "Count"
-                          ]
-                        }
-                      }
-                    },
-                    {
-                      "description": "A unit of length.",
-                      "type": "object",
-                      "oneOf": [
-                        {
-                          "type": "object",
-                          "required": [
-                            "type"
-                          ],
-                          "properties": {
-                            "type": {
-                              "type": "string",
-                              "enum": [
-                                "Mm"
-                              ]
-                            }
-                          }
-                        },
-                        {
-                          "type": "object",
-                          "required": [
-                            "type"
-                          ],
-                          "properties": {
-                            "type": {
-                              "type": "string",
-                              "enum": [
-                                "Cm"
-                              ]
-                            }
-                          }
-                        },
-                        {
-                          "type": "object",
-                          "required": [
-                            "type"
-                          ],
-                          "properties": {
-                            "type": {
-                              "type": "string",
-                              "enum": [
-                                "M"
-                              ]
-                            }
-                          }
-                        },
-                        {
-                          "type": "object",
-                          "required": [
-                            "type"
-                          ],
-                          "properties": {
-                            "type": {
-                              "type": "string",
-                              "enum": [
-                                "Inches"
-                              ]
-                            }
-                          }
-                        },
-                        {
-                          "type": "object",
-                          "required": [
-                            "type"
-                          ],
-                          "properties": {
-                            "type": {
-                              "type": "string",
-                              "enum": [
-                                "Feet"
-                              ]
-                            }
-                          }
-                        },
-                        {
-                          "type": "object",
-                          "required": [
-                            "type"
-                          ],
-                          "properties": {
-                            "type": {
-                              "type": "string",
-                              "enum": [
-                                "Yards"
-                              ]
-                            }
-                          }
-                        }
-                      ],
-                      "required": [
-                        "type"
-                      ],
-                      "properties": {
-                        "type": {
-                          "type": "string",
-                          "enum": [
-                            "Length"
-                          ]
-                        }
-                      }
-                    },
-                    {
-                      "description": "A unit of angle.",
-                      "type": "object",
-                      "oneOf": [
-                        {
-                          "type": "object",
-                          "required": [
-                            "type"
-                          ],
-                          "properties": {
-                            "type": {
-                              "type": "string",
-                              "enum": [
-                                "Degrees"
-                              ]
-                            }
-                          }
-                        },
-                        {
-                          "type": "object",
-                          "required": [
-                            "type"
-                          ],
-                          "properties": {
-                            "type": {
-                              "type": "string",
-                              "enum": [
-                                "Radians"
-                              ]
-                            }
-                          }
-                        }
-                      ],
-                      "required": [
-                        "type"
-                      ],
-                      "properties": {
-                        "type": {
-                          "type": "string",
-                          "enum": [
-                            "Angle"
-                          ]
-                        }
-                      }
-                    }
-                  ],
-                  "required": [
-                    "type"
-                  ],
-                  "properties": {
-                    "type": {
-                      "type": "string",
-                      "enum": [
-                        "Known"
-                      ]
-                    }
-                  }
-                },
-                {
-                  "type": "object",
-                  "required": [
-                    "angle",
-                    "len",
-                    "type"
-                  ],
-                  "properties": {
-                    "type": {
-                      "type": "string",
-                      "enum": [
-                        "Default"
-                      ]
-                    },
-                    "len": {
-                      "$ref": "#/components/schemas/UnitLen"
-                    },
-                    "angle": {
-                      "$ref": "#/components/schemas/UnitAngle"
-                    }
-                  }
-                },
-                {
-                  "type": "object",
-                  "required": [
-                    "type"
-                  ],
-                  "properties": {
-                    "type": {
-                      "type": "string",
-                      "enum": [
-                        "Unknown"
-                      ]
-                    }
-                  }
-                },
-                {
-                  "type": "object",
-                  "required": [
-                    "type"
-                  ],
-                  "properties": {
-                    "type": {
-                      "type": "string",
-                      "enum": [
-                        "Any"
-                      ]
-                    }
-                  }
-                }
-              ]
-            },
-            "UnitLen": {
-              "description": "A unit of length.",
-              "oneOf": [
-                {
-                  "type": "object",
-                  "required": [
-                    "type"
-                  ],
-                  "properties": {
-                    "type": {
-                      "type": "string",
-                      "enum": [
-                        "Mm"
-                      ]
-                    }
-                  }
-                },
-                {
-                  "type": "object",
-                  "required": [
-                    "type"
-                  ],
-                  "properties": {
-                    "type": {
-                      "type": "string",
-                      "enum": [
-                        "Cm"
-                      ]
-                    }
-                  }
-                },
-                {
-                  "type": "object",
-                  "required": [
-                    "type"
-                  ],
-                  "properties": {
-                    "type": {
-                      "type": "string",
-                      "enum": [
-                        "M"
-                      ]
-                    }
-                  }
-                },
-                {
-                  "type": "object",
-                  "required": [
-                    "type"
-                  ],
-                  "properties": {
-                    "type": {
-                      "type": "string",
-                      "enum": [
-                        "Inches"
-                      ]
-                    }
-                  }
-                },
-                {
-                  "type": "object",
-                  "required": [
-                    "type"
-                  ],
-                  "properties": {
-                    "type": {
-                      "type": "string",
-                      "enum": [
-                        "Feet"
-                      ]
-                    }
-                  }
-                },
-                {
-                  "type": "object",
-                  "required": [
-                    "type"
-                  ],
-                  "properties": {
-                    "type": {
-                      "type": "string",
-                      "enum": [
-                        "Yards"
-                      ]
-                    }
-                  }
-                }
-              ]
-            },
-            "UnitAngle": {
-              "description": "A unit of angle.",
-              "oneOf": [
-                {
-                  "type": "object",
-                  "required": [
-                    "type"
-                  ],
-                  "properties": {
-                    "type": {
-                      "type": "string",
-                      "enum": [
-                        "Degrees"
-                      ]
-                    }
-                  }
-                },
-                {
-                  "type": "object",
-                  "required": [
-                    "type"
-                  ],
-                  "properties": {
-                    "type": {
-                      "type": "string",
-                      "enum": [
-                        "Radians"
-                      ]
-                    }
-                  }
-                }
-              ]
+              "type": "number",
+              "format": "double"
            }
          }
        },
--- a/docs/kcl/types/ExtrudeSurface.md
+++ b/docs/kcl/types/ExtrudeSurface.md
@ -28,7 +28,7 @@ An extrude plane.
 | `faceId` |[`string`](/docs/kcl/types/string)| The face id for the extrude plane. | No |
 | [`tag`](/docs/kcl/types/tag) |[`TagDeclarator`](/docs/kcl/types#tag-declaration)| The tag. | No |
 | `id` |[`string`](/docs/kcl/types/string)| The id of the geometry. | No |
-| `sourceRange` |[`SourceRange`](/docs/kcl/types/SourceRange)| The source range. | No |
+| `sourceRange` |`[integer, integer, integer]`| The source range. | No |


 ----
@ -48,7 +48,7 @@ An extruded arc.
 | `faceId` |[`string`](/docs/kcl/types/string)| The face id for the extrude plane. | No |
 | [`tag`](/docs/kcl/types/tag) |[`TagDeclarator`](/docs/kcl/types#tag-declaration)| The tag. | No |
 | `id` |[`string`](/docs/kcl/types/string)| The id of the geometry. | No |
-| `sourceRange` |[`SourceRange`](/docs/kcl/types/SourceRange)| The source range. | No |
+| `sourceRange` |`[integer, integer, integer]`| The source range. | No |


 ----
@ -68,7 +68,7 @@ Geometry metadata.
 | `faceId` |[`string`](/docs/kcl/types/string)| The id for the chamfer surface. | No |
 | [`tag`](/docs/kcl/types/tag) |[`TagDeclarator`](/docs/kcl/types#tag-declaration)| The tag. | No |
 | `id` |[`string`](/docs/kcl/types/string)| The id of the geometry. | No |
-| `sourceRange` |[`SourceRange`](/docs/kcl/types/SourceRange)| The source range. | No |
+| `sourceRange` |`[integer, integer, integer]`| The source range. | No |


 ----
@ -88,7 +88,7 @@ Geometry metadata.
 | `faceId` |[`string`](/docs/kcl/types/string)| The id for the fillet surface. | No |
 | [`tag`](/docs/kcl/types/tag) |[`TagDeclarator`](/docs/kcl/types#tag-declaration)| The tag. | No |
 | `id` |[`string`](/docs/kcl/types/string)| The id of the geometry. | No |
-| `sourceRange` |[`SourceRange`](/docs/kcl/types/SourceRange)| The source range. | No |
+| `sourceRange` |`[integer, integer, integer]`| The source range. | No |


 ----
--- a/docs/kcl/types/GeoMeta.md
+++ b/docs/kcl/types/GeoMeta.md
@ -17,6 +17,6 @@ Geometry metadata.
 | Property | Type | Description | Required |
 |----------|------|-------------|----------|
 | `id` |[`string`](/docs/kcl/types/string)| The id of the geometry. | No |
-| `sourceRange` |[`SourceRange`](/docs/kcl/types/SourceRange)| The source range. | No |
+| `sourceRange` |`[integer, integer, integer]`| The source range. | No |


--- a/docs/kcl/types/HelixValue.md
+++ b/docs/kcl/types/HelixValue.md
@ -17,7 +17,7 @@ A helix.
 | Property | Type | Description | Required |
 |----------|------|-------------|----------|
 | `value` |[`string`](/docs/kcl/types/string)| The id of the helix. | No |
-| `artifactId` |[`ArtifactId`](/docs/kcl/types/ArtifactId)| The artifact ID. | No |
+| `artifactId` |[`string`](/docs/kcl/types/string)| The artifact ID. | No |
 | `revolutions` |[`number`](/docs/kcl/types/number)| Number of revolutions. | No |
 | `angleStart` |[`number`](/docs/kcl/types/number)| Start angle (in degrees). | No |
 | `ccw` |`boolean`| Is the helix rotation counter clockwise? | No |
--- a/docs/kcl/types/KclValue.md
+++ b/docs/kcl/types/KclValue.md
@ -285,7 +285,7 @@ Data for an imported geometry.
 | Property | Type | Description | Required |
 |----------|------|-------------|----------|
 | `type` |enum: `Module`|  | No |
-| `value` |[`ModuleId`](/docs/kcl/types/ModuleId)| Identifier of a source file.  Uses a u32 to keep the size small. | No |
+| `value` |`integer`| Identifier of a source file.  Uses a u32 to keep the size small. | No |


 ----
--- a/docs/kcl/types/PolarCoordsData.md
+++ b/docs/kcl/types/PolarCoordsData.md
@ -17,6 +17,6 @@ Data for polar coordinates.
 | Property | Type | Description | Required |
 |----------|------|-------------|----------|
 | `angle` |[`number`](/docs/kcl/types/number)| The angle of the line (in degrees). | No |
-| `length` |[`TyF64`](/docs/kcl/types/TyF64)| The length of the line. | No |
+| `length` |[`number`](/docs/kcl/types/number)| The length of the line. | No |


--- a/docs/kcl/types/SketchSurface.md
+++ b/docs/kcl/types/SketchSurface.md
@ -25,7 +25,7 @@ A sketch type.
 |----------|------|-------------|----------|
 | `type` |enum: `plane`|  | No |
 | `id` |[`string`](/docs/kcl/types/string)| The id of the plane. | No |
-| `artifactId` |[`ArtifactId`](/docs/kcl/types/ArtifactId)| The artifact ID. | No |
+| `artifactId` |[`string`](/docs/kcl/types/string)| The artifact ID. | No |
 | `value` |[`PlaneType`](/docs/kcl/types/PlaneType)| Type for a plane. | No |
 | `origin` |[`Point3d`](/docs/kcl/types/Point3d)| Origin of the plane. | No |
 | `xAxis` |[`Point3d`](/docs/kcl/types/Point3d)| What should the plane's X axis be? | No |
@ -49,7 +49,7 @@ A face.
 |----------|------|-------------|----------|
 | `type` |enum: `face`|  | No |
 | `id` |[`string`](/docs/kcl/types/string)| The id of the face. | No |
-| `artifactId` |[`ArtifactId`](/docs/kcl/types/ArtifactId)| The artifact ID. | No |
+| `artifactId` |[`string`](/docs/kcl/types/string)| The artifact ID. | No |
 | `value` |[`string`](/docs/kcl/types/string)| The tag of the face. | No |
 | `xAxis` |[`Point3d`](/docs/kcl/types/Point3d)| What should the face's X axis be? | No |
 | `yAxis` |[`Point3d`](/docs/kcl/types/Point3d)| What should the face's Y axis be? | No |
--- a/docs/kcl/types/SourceRange.md
+++ b/docs/kcl/types/SourceRange.md
@ -1,15 +0,0 @@
---
-title: "SourceRange"
-excerpt: ""
-layout: manual
---
-
-
-**Type:** `integer` (`uint`)
-
-
-
-
-
-
-
--- a/docs/kcl/types/TyF64.md
+++ b/docs/kcl/types/TyF64.md
@ -5,17 +5,11 @@ layout: manual
 ---


-**Type:** `object`
+**Type:** [`number`](/docs/kcl/types/number) (`double`)





-## Properties
-
-| Property | Type | Description | Required |
-|----------|------|-------------|----------|
-| `n` |[`number`](/docs/kcl/types/number)|  | No |
-| `ty` |[`NumericType`](/docs/kcl/types/NumericType)|  | No |


--- a/e2e/playwright/basic-sketch.spec.ts
+++ b/e2e/playwright/basic-sketch.spec.ts
@ -85,7 +85,7 @@ async function doBasicSketch(
  await page.mouse.click(startXPx, 500 - PUR * 20)
  if (openPanes.includes('code')) {
    await expect(u.codeLocator)
-      .toHaveText(`sketch001 = startSketchOn(XZ)profile001 = startProfileAt(${
+      .toHaveText(`@settings(defaultLengthUnit = in)sketch001 = startSketchOn(XZ)profile001 = startProfileAt(${
      commonPoints.startAt
    }, sketch001)
  |> xLine(length = ${commonPoints.num1})
@ -119,10 +119,7 @@ async function doBasicSketch(
  await page.waitForTimeout(100)

  if (openPanes.includes('code')) {
-    await expect(
-      await u.getGreatestPixDiff(line1, TEST_COLORS.BLUE)
-    ).toBeLessThan(3)
-    await expect(await u.getGreatestPixDiff(line1, [0, 0, 255])).toBeLessThan(3)
+    expect(await u.getGreatestPixDiff(line1, TEST_COLORS.BLUE)).toBeLessThan(3)
  }

  // hold down shift
@ -145,7 +142,7 @@ async function doBasicSketch(
  // Open the code pane.
  await u.openKclCodePanel()
  await expect(u.codeLocator)
-    .toHaveText(`sketch001 = startSketchOn(XZ)profile001 = startProfileAt(${
+    .toHaveText(`@settings(defaultLengthUnit = in)sketch001 = startSketchOn(XZ)profile001 = startProfileAt(${
    commonPoints.startAt
  }, sketch001)
  |> xLine(length = ${commonPoints.num1}, tag = $seg01)
--- a/e2e/playwright/can-create-sketches-on-all-planes-and-their-back-sides.spec.ts
+++ b/e2e/playwright/can-create-sketches-on-all-planes-and-their-back-sides.spec.ts
@ -46,7 +46,7 @@ test.describe(
        },
      }

-      const code = `sketch001 = startSketchOn(${plane})profile001 = startProfileAt([0.91, -1.22], sketch001)`
+      const code = `@settings(defaultLengthUnit = in)sketch001 = startSketchOn(${plane})profile001 = startProfileAt([0.91, -1.22], sketch001)`

      await u.openDebugPanel()

--- a/e2e/playwright/code-pane-and-errors.spec.ts
+++ b/e2e/playwright/code-pane-and-errors.spec.ts
@ -251,11 +251,11 @@ test(
      ])
      await Promise.all([
        fsp.copyFile(
-          executorInputPath('router-template-slate.kcl'),
+          executorInputPath('cylinder-inches.kcl'),
          join(routerTemplateDir, 'main.kcl')
        ),
        fsp.copyFile(
-          executorInputPath('focusrite_scarlett_mounting_braket.kcl'),
+          executorInputPath('e2e-can-sketch-on-chamfer.kcl'),
          join(bracketDir, 'main.kcl')
        ),
      ])
--- a/e2e/playwright/desktop-export.spec.ts
+++ b/e2e/playwright/desktop-export.spec.ts
@ -20,11 +20,11 @@ test(
      await Promise.all([fsp.mkdir(bracketDir, { recursive: true })])
      await Promise.all([
        fsp.copyFile(
-          executorInputPath('router-template-slate.kcl'),
+          executorInputPath('cylinder-inches.kcl'),
          path.join(bracketDir, 'other.kcl')
        ),
        fsp.copyFile(
-          executorInputPath('focusrite_scarlett_mounting_braket.kcl'),
+          executorInputPath('e2e-can-sketch-on-chamfer.kcl'),
          path.join(bracketDir, 'main.kcl')
        ),
      ])
@ -107,7 +107,7 @@ test(
            },
            { timeout: 15_000 }
          )
-          .toBeGreaterThan(300_000)
+          .toBeGreaterThan(30_000)
      })
    })

@ -187,7 +187,7 @@ test(
            },
            { timeout: 15_000 }
          )
-          .toBeGreaterThan(70_000)
+          .toBeGreaterThan(50_000)
      })
    })
  }
--- a/e2e/playwright/editor-tests.spec.ts
+++ b/e2e/playwright/editor-tests.spec.ts
@ -32,26 +32,30 @@ test.describe('Editor tests', { tag: ['@skipWin'] }, () => {
    await page.keyboard.press('/')
    await page.keyboard.up('ControlOrMeta')

-    await expect(page.locator('.cm-content'))
-      .toHaveText(`sketch001 = startSketchOn(XY)
+    await expect(page.locator('.cm-content')).toHaveText(
+      `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XY)
  |> startProfileAt([-10, -10], %)
  |> line(end = [20, 0])
  |> line(end = [0, 20])
  |> line(end = [-20, 0])
-  // |> close()`)
+  // |> close()`.replaceAll('\n', '')
+    )

    // uncomment the code
    await page.keyboard.down('ControlOrMeta')
    await page.keyboard.press('/')
    await page.keyboard.up('ControlOrMeta')

-    await expect(page.locator('.cm-content'))
-      .toHaveText(`sketch001 = startSketchOn(XY)
+    await expect(page.locator('.cm-content')).toHaveText(
+      `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XY)
  |> startProfileAt([-10, -10], %)
  |> line(end = [20, 0])
  |> line(end = [0, 20])
  |> line(end = [-20, 0])
-  |> close()`)
+  |> close()`.replaceAll('\n', '')
+    )
  })

  test('ensure we use the cache, and do not re-execute', async ({
@ -178,13 +182,15 @@ test.describe('Editor tests', { tag: ['@skipWin'] }, () => {
    await page.locator('#code-pane button:first-child').click()
    await page.locator('button:has-text("Format code")').click()

-    await expect(page.locator('.cm-content'))
-      .toHaveText(`sketch001 = startSketchOn(XY)
+    await expect(page.locator('.cm-content')).toHaveText(
+      `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XY)
  |> startProfileAt([-10, -10], %)
  |> line(end = [20, 0])
  |> line(end = [0, 20])
  |> line(end = [-20, 0])
-  |> close()`)
+  |> close()`.replaceAll('\n', '')
+    )
  })

  test('if you click the format button it formats your code and executes so lints are still there', async ({
@ -227,13 +233,15 @@ test.describe('Editor tests', { tag: ['@skipWin'] }, () => {
    await u.expectCmdLog('[data-message-type="execution-done"]')
    await u.closeDebugPanel()

-    await expect(page.locator('.cm-content'))
-      .toHaveText(`sketch_001 = startSketchOn(XY)
+    await expect(page.locator('.cm-content')).toHaveText(
+      `@settings(defaultLengthUnit = in)
+sketch_001 = startSketchOn(XY)
  |> startProfileAt([-10, -10], %)
  |> line(end = [20, 0])
  |> line(end = [0, 20])
  |> line(end = [-20, 0])
-  |> close()`)
+  |> close()`.replaceAll('\n', '')
+    )

    // error in guter
    await expect(page.locator('.cm-lint-marker-info').first()).toBeVisible()
@ -471,6 +479,7 @@ test.describe('Editor tests', { tag: ['@skipWin'] }, () => {
  test('if you write kcl with lint errors you get lints', async ({
    page,
    homePage,
+    scene,
  }) => {
    const u = await getUtils(page)
    await page.setBodyDimensions({ width: 1000, height: 500 })
@ -490,10 +499,7 @@ test.describe('Editor tests', { tag: ['@skipWin'] }, () => {
    await page.keyboard.press('ArrowLeft')
    await page.keyboard.press('ArrowRight')

-    // FIXME: lsp errors do not propagate to the frontend until engine is connected and code is executed
-    // This timeout is to wait for engine connection. LSP and code execution errors should be handled differently
-    // LSP can emit errors as fast as it waits and show them in the editor
-    await page.waitForTimeout(10000)
+    await scene.waitForExecutionDone()

    // error in guter
    await expect(page.locator('.cm-lint-marker-info').first()).toBeVisible()
@ -815,10 +821,12 @@ test.describe('Editor tests', { tag: ['@skipWin'] }, () => {
      // there shouldn't be any auto complete options for 'lin' in the comment
      await expect(page.locator('.cm-completionLabel')).not.toBeVisible()

-      await expect(page.locator('.cm-content'))
-        .toHaveText(`sketch001 = startSketchOn(XZ)
+      await expect(page.locator('.cm-content')).toHaveText(
+        `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
        |> startProfileAt([3.14, 12], %)
-        |> xLine(%, length = 5) // lin`)
+        |> xLine(%, length = 5) // lin`.replaceAll('\n', '')
+      )

      // expect there to be no KCL errors
      await expect(page.locator('.cm-lint-marker-error')).toHaveCount(0)
@ -888,10 +896,12 @@ test.describe('Editor tests', { tag: ['@skipWin'] }, () => {
      // there shouldn't be any auto complete options for 'lin' in the comment
      await expect(page.locator('.cm-completionLabel')).not.toBeVisible()

-      await expect(page.locator('.cm-content'))
-        .toHaveText(`sketch001 = startSketchOn(XZ)
+      await expect(page.locator('.cm-content')).toHaveText(
+        `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
        |> startProfileAt([3.14, 12], %)
-        |> xLine(%, length = 5) // lin`)
+        |> xLine(%, length = 5) // lin`.replaceAll('\n', '')
+      )
    })
  })
  test('Can undo a click and point extrude with ctrl+z', async ({
@ -1206,4 +1216,55 @@ test.describe('Editor tests', { tag: ['@skipWin'] }, () => {
      })
    }
  )
+
+  test('Rectangle tool panning with middle click', async ({
+    page,
+    homePage,
+    toolbar,
+    scene,
+    cmdBar,
+    editor,
+  }) => {
+    await page.setBodyDimensions({ width: 1200, height: 900 })
+    await homePage.goToModelingScene()
+
+    // wait until scene is ready to be interacted with
+    await scene.connectionEstablished()
+    await scene.settled(cmdBar)
+
+    await page.getByRole('button', { name: 'Start Sketch' }).click()
+
+    // select an axis plane
+    await page.mouse.click(700, 200)
+
+    // Needed as we don't yet have a way to get a signal from the engine that the camera has animated to the sketch plane
+    await page.waitForTimeout(1000)
+
+    const middleMousePan = async (
+      startX: number,
+      startY: number,
+      endX: number,
+      endY: number
+    ) => {
+      const initialCode = await editor.getCurrentCode()
+
+      await page.mouse.click(startX, startY, { button: 'middle' })
+      await page.mouse.move(endX, endY, {
+        steps: 10,
+      })
+
+      // We expect the code to be the same, middle mouse click should not modify the code, only do panning
+      await editor.expectEditor.toBe(initialCode)
+    }
+
+    await test.step(`Verify corner rectangle panning`, async () => {
+      await page.getByTestId('corner-rectangle').click()
+      await middleMousePan(800, 500, 900, 600)
+    })
+
+    await test.step(`Verify center rectangle panning`, async () => {
+      await toolbar.selectCenterRectangle()
+      await middleMousePan(800, 200, 900, 300)
+    })
+  })
 })
--- a/e2e/playwright/fixtures/editorFixture.ts
+++ b/e2e/playwright/fixtures/editorFixture.ts
@ -81,6 +81,13 @@ export class EditorFixture {
  expectEditor = {
    toContain: this._expectEditorToContain(),
    not: { toContain: this._expectEditorToContain(true) },
+    toBe: async (code: string) => {
+      const currentCode = await this.getCurrentCode()
+      return expect(currentCode).toBe(code)
+    },
+  }
+  getCurrentCode = async () => {
+    return await this.codeContent.innerText()
  }
  snapshot = async (options?: { timeout?: number; name?: string }) => {
    const wasPaneOpen = await this.checkIfPaneIsOpen()
--- a/e2e/playwright/fixtures/fixtureSetup.ts
+++ b/e2e/playwright/fixtures/fixtureSetup.ts
@ -124,6 +124,7 @@ export class ElectronZoo {

    // We need to expose this in order for some tests that require folder
    // creation and some code below.
+    // eslint-disable-next-line @typescript-eslint/no-this-alias
    const that = this

    const options = {
--- a/e2e/playwright/fixtures/sceneFixture.ts
+++ b/e2e/playwright/fixtures/sceneFixture.ts
@ -310,7 +310,9 @@ export async function expectPixelColor(
    .toBeTruthy()
    .catch((cause) => {
      throw new Error(
-        `ExpectPixelColor: expecting ${colour} got ${finalValue}`,
+        `ExpectPixelColor: point ${JSON.stringify(
+          coords
+        )} was expecting ${colour} but got ${finalValue}`,
        { cause }
      )
    })
--- a/e2e/playwright/lib/console-error-whitelist.ts
+++ b/e2e/playwright/lib/console-error-whitelist.ts
@ -272,14 +272,6 @@ export const isErrorWhitelisted = (exception: Error) => {
      project: 'Google Chrome',
      foundInSpec: 'e2e/playwright/snapshot-tests.spec.ts',
    },
-    // TODO: fix this error in the code
-    {
-      name: 'ReferenceError',
-      message: 'createNewVariableCheckbox is not defined',
-      stack: '',
-      project: 'Google Chrome',
-      foundInSpec: 'e2e/playwright/testing-constraints.spec.ts',
-    },
    {
      name: 'Error',
      message: 'The "path" argument must be of type string. Received undefined',
--- a/e2e/playwright/machines.spec.ts
+++ b/e2e/playwright/machines.spec.ts
@ -11,7 +11,7 @@ test(
      const bracketDir = join(dir, 'bracket')
      await fsp.mkdir(bracketDir, { recursive: true })
      await fsp.copyFile(
-        executorInputPath('focusrite_scarlett_mounting_braket.kcl'),
+        executorInputPath('cylinder-inches.kcl'),
        join(bracketDir, 'main.kcl')
      )
    })
@ -51,7 +51,7 @@ test(
      const bracketDir = join(dir, 'bracket')
      await fsp.mkdir(bracketDir, { recursive: true })
      await fsp.copyFile(
-        executorInputPath('focusrite_scarlett_mounting_braket.kcl'),
+        executorInputPath('cylinder-inches.kcl'),
        join(bracketDir, 'main.kcl')
      )
    })
--- a/e2e/playwright/point-click.spec.ts
+++ b/e2e/playwright/point-click.spec.ts
@ -137,7 +137,7 @@ test.describe('Point-and-click tests', () => {

        await scene.moveCameraTo(cameraPos, cameraTarget)

-        await test.step('check chamfer selection changes cursor positon', async () => {
+        await test.step('check chamfer selection changes cursor position', async () => {
          await expect(async () => {
            // sometimes initial click doesn't register
            await clickChamfer()
@ -173,7 +173,7 @@ test.describe('Point-and-click tests', () => {
        })
        await test.step('Check there is no errors after code created in previous steps executes', async () => {
          await editor.expectState({
-            activeLines: ['sketch001 = startSketchOn(XZ)'],
+            activeLines: ['@settings(defaultLengthUnit = in)'],
            highlightedCode: '',
            diagnostics: [],
          })
@ -299,7 +299,8 @@ test.describe('Point-and-click tests', () => {

      await test.step('verify at the end of the test that final code is what is expected', async () => {
        await editor.expectEditor.toContain(
-          `sketch001 = startSketchOn(XZ)
+          `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
  |> startProfileAt([75.8, 317.2], %) // [$startCapTag, $EndCapTag]
  |> angledLine([0, 268.43], %, $rectangleSegmentA001)
  |> angledLine([
@ -369,7 +370,7 @@ profile001 = startProfileAt([205.96, 254.59], sketch002)
      })
    })

-    test('Works on chamfers that are non in a pipeExpression can break up multi edges in a chamfer array', async ({
+    test('Works on chamfers that are not in a pipeExpression can break up multi edges in a chamfer array', async ({
      context,
      page,
      homePage,
@ -418,7 +419,8 @@ profile001 = startProfileAt([205.96, 254.59], sketch002)
        |>close()`,
      })
      await editor.expectEditor.toContain(
-        `sketch001 = startSketchOn(XZ)
+        `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
  |> startProfileAt([75.8, 317.2], %)
  |> angledLine([0, 268.43], %, $rectangleSegmentA001)
  |> angledLine([
@ -1639,9 +1641,10 @@ loft001 = loft([sketch001, sketch002])
    {
      targetType: 'circle',
      testPoint: { x: 700, y: 250 },
-      initialCode: `sketch001 = startSketchOn('YZ')
+      initialCode: `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(YZ)
 profile001 = circle(sketch001, center = [0, 0], radius = 500)
-sketch002 = startSketchOn('XZ')
+sketch002 = startSketchOn(XZ)
  |> startProfileAt([0, 0], %)
  |> xLine(length = -500)
  |> tangentialArcTo([-2000, 500], %)`,
@ -1649,7 +1652,8 @@ sketch002 = startSketchOn('XZ')
    {
      targetType: 'rectangle',
      testPoint: { x: 710, y: 255 },
-      initialCode: `sketch001 = startSketchOn('YZ')
+      initialCode: `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(YZ)
 profile001 = startProfileAt([-400, -400], sketch001)
  |> angledLine([0, 800], %, $rectangleSegmentA001)
  |> angledLine([
@ -1662,7 +1666,7 @@ profile001 = startProfileAt([-400, -400], sketch001)
     ], %)
  |> line(endAbsolute = [profileStartX(%), profileStartY(%)])
  |> close()
-sketch002 = startSketchOn('XZ')
+sketch002 = startSketchOn(XZ)
  |> startProfileAt([0, 0], %)
  |> xLine(length = -500)
  |> tangentialArcTo([-2000, 500], %)`,
@ -1806,7 +1810,8 @@ sketch002 = startSketchOn('XZ')
    toolbar,
    cmdBar,
  }) => {
-    const initialCode = `sketch001 = startSketchOn(YZ)
+    const initialCode = `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(YZ)
  |> circle(
       center = [0, 0],
       radius = 500
@ -2475,7 +2480,8 @@ extrude001 = extrude(profile001, length = 5)
    cmdBar,
  }) => {
    // Code samples
-    const initialCode = `sketch001 = startSketchOn(XY)
+    const initialCode = `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XY)
  |> startProfileAt([-12, -6], %)
  |> line(end = [0, 12])
  |> line(end = [24, 0])
@ -2767,7 +2773,8 @@ extrude001 = extrude(sketch001, length = -12)
    toolbar,
  }) => {
    // Code samples
-    const initialCode = `sketch001 = startSketchOn(XY)
+    const initialCode = `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XY)
  |> startProfileAt([-12, -6], %)
  |> line(end = [0, 12])
  |> line(end = [24, 0], tag = $seg02)
@ -2921,7 +2928,8 @@ chamfer04 = chamfer(extrude001, length = 5, tags = [getOppositeEdge(seg02)])
      toolbar,
      cmdBar,
    }) => {
-      const initialCode = `sketch001 = startSketchOn(XZ)
+      const initialCode = `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
  |> circle(center = [0, 0], radius = 30)
 extrude001 = extrude(sketch001, length = 30)
    `
@ -3056,7 +3064,8 @@ extrude001 = extrude(sketch001, length = 30)
    toolbar,
    cmdBar,
  }) => {
-    const initialCode = `sketch001 = startSketchOn(XY)
+    const initialCode = `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XY)
  |> startProfileAt([-20, 20], %)
  |> xLine(length = 40)
  |> yLine(length = -60)
@ -3174,7 +3183,8 @@ extrude001 = extrude(sketch001, length = 40)
  })

  const shellSketchOnFacesCases = [
-    `sketch001 = startSketchOn(XZ)
+    `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
  |> circle(center = [0, 0], radius = 100)
  |> extrude(length = 100)

@ -3182,7 +3192,8 @@ sketch002 = startSketchOn(sketch001, 'END')
  |> circle(center = [0, 0], radius = 50)
  |> extrude(length = 50)
  `,
-    `sketch001 = startSketchOn(XZ)
+    `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
  |> circle(center = [0, 0], radius = 100)
 extrude001 = extrude(sketch001, length = 100)

@ -3465,6 +3476,39 @@ segAng(rectangleSegmentA002),

      const newCodeToFind = `revolve001 = revolve(sketch002, angle = 360, axis = 'X')`
      expect(editor.expectEditor.toContain(newCodeToFind)).toBeTruthy()
+
+      // Edit flow
+      const newAngle = '90'
+      await toolbar.openPane('feature-tree')
+      const operationButton = await toolbar.getFeatureTreeOperation(
+        'Revolve',
+        0
+      )
+      await operationButton.dblclick({ button: 'left' })
+      await cmdBar.expectState({
+        commandName: 'Revolve',
+        currentArgKey: 'angle',
+        currentArgValue: '360',
+        headerArguments: {
+          Angle: '360',
+        },
+        highlightedHeaderArg: 'angle',
+        stage: 'arguments',
+      })
+      await page.keyboard.insertText(newAngle)
+      await cmdBar.progressCmdBar()
+      await cmdBar.expectState({
+        stage: 'review',
+        headerArguments: {
+          Angle: newAngle,
+        },
+        commandName: 'Revolve',
+      })
+      await cmdBar.progressCmdBar()
+      await toolbar.closePane('feature-tree')
+      await editor.expectEditor.toContain(
+        newCodeToFind.replace('angle = 360', 'angle = ' + newAngle)
+      )
    })
    test('revolve surface around edge from an extruded solid2d', async ({
      context,
@ -3475,26 +3519,22 @@ segAng(rectangleSegmentA002),
      toolbar,
      cmdBar,
    }) => {
-      const initialCode = `
-sketch001 = startSketchOn(XZ)
-|> startProfileAt([-102.57, 101.72], %)
-|> angledLine([0, 202.6], %, $rectangleSegmentA001)
-|> angledLine([
-segAng(rectangleSegmentA001) - 90,
-202.6
-], %, $rectangleSegmentB001)
-|> angledLine([
-segAng(rectangleSegmentA001),
-segLen(rectangleSegmentA001)
-], %, $rectangleSegmentC001)
-|> line(endAbsolute = [profileStartX(%), profileStartY(%)])
-|> close()
+      const initialCode = `sketch001 = startSketchOn(XZ)
+  |> startProfileAt([-102.57, 101.72], %)
+  |> angledLine([0, 202.6], %, $rectangleSegmentA001)
+  |> angledLine([
+       segAng(rectangleSegmentA001) - 90,
+       202.6
+     ], %, $rectangleSegmentB001)
+  |> angledLine([
+       segAng(rectangleSegmentA001),
+       -segLen(rectangleSegmentA001)
+     ], %, $rectangleSegmentC001)
+  |> line(endAbsolute = [profileStartX(%), profileStartY(%)])
+  |> close()
 extrude001 = extrude(sketch001, length = 50)
 sketch002 = startSketchOn(extrude001, rectangleSegmentA001)
-|> circle(
-center = [-11.34, 10.0],
-radius = 8.69
-)
+  |> circle(center = [-11.34, 10.0], radius = 8.69)
 `
      await context.addInitScript((initialCode) => {
        localStorage.setItem('persistCode', initialCode)
@ -3512,9 +3552,49 @@ radius = 8.69
      const lineCodeToSelection = `|> angledLine([0, 202.6], %, $rectangleSegmentA001)`
      await page.getByText(lineCodeToSelection).click()
      await cmdBar.progressCmdBar()
+      await cmdBar.progressCmdBar()
+      await cmdBar.progressCmdBar()

-      const newCodeToFind = `revolve001 = revolve(sketch002, angle = 360, axis = getOppositeEdge(rectangleSegmentA001)) `
-      expect(editor.expectEditor.toContain(newCodeToFind)).toBeTruthy()
+      const newCodeToFind = `revolve001 = revolve(sketch002, angle = 360, axis = rectangleSegmentA001)`
+      await editor.expectEditor.toContain(newCodeToFind)
+
+      // Edit flow
+      const newAngle = '180'
+      await toolbar.openPane('feature-tree')
+      const operationButton = await toolbar.getFeatureTreeOperation(
+        'Revolve',
+        0
+      )
+      await operationButton.dblclick({ button: 'left' })
+      await cmdBar.expectState({
+        commandName: 'Revolve',
+        currentArgKey: 'angle',
+        currentArgValue: '360',
+        headerArguments: {
+          Angle: '360',
+        },
+        highlightedHeaderArg: 'angle',
+        stage: 'arguments',
+      })
+      await page.keyboard.insertText(newAngle)
+      await page.getByRole('button', { name: 'Create new variable' }).click()
+      await expect(page.getByPlaceholder('Variable name')).toHaveValue(
+        'angle001'
+      )
+      await cmdBar.progressCmdBar()
+      await cmdBar.expectState({
+        stage: 'review',
+        headerArguments: {
+          Angle: newAngle,
+        },
+        commandName: 'Revolve',
+      })
+      await cmdBar.progressCmdBar()
+      await toolbar.closePane('feature-tree')
+      await editor.expectEditor.toContain('angle001 = ' + newAngle)
+      await editor.expectEditor.toContain(
+        newCodeToFind.replace('angle = 360', 'angle = angle001')
+      )
    })
    test('revolve sketch circle around line segment from startProfileAt sketch', async ({
      context,
@ -3525,26 +3605,22 @@ radius = 8.69
      toolbar,
      cmdBar,
    }) => {
-      const initialCode = `
-    sketch002 = startSketchOn(XY)
-      |> startProfileAt([-2.02, 1.79], %)
-      |> xLine(length = 2.6)
-    sketch001 = startSketchOn('-XY')
-      |> startProfileAt([-0.48, 1.25], %)
-      |> angledLine([0, 2.38], %, $rectangleSegmentA001)
-      |> angledLine([segAng(rectangleSegmentA001) - 90, 2.4], %, $rectangleSegmentB001)
-      |> angledLine([
-        segAng(rectangleSegmentA001),
-          -segLen(rectangleSegmentA001)
-      ], %, $rectangleSegmentC001)
-      |> line(endAbsolute = [profileStartX(%), profileStartY(%)])
-      |> close()
-    extrude001 = extrude(sketch001, length = 5)
-    sketch003 = startSketchOn(extrude001, 'START')
-      |> circle(
-        center = [-0.69, 0.56],
-        radius = 0.28
-      )
+      const initialCode = `sketch002 = startSketchOn(XY)
+  |> startProfileAt([-2.02, 1.79], %)
+  |> xLine(length = 2.6)
+sketch001 = startSketchOn(-XY)
+  |> startProfileAt([-0.48, 1.25], %)
+  |> angledLine([0, 2.38], %, $rectangleSegmentA001)
+  |> angledLine([segAng(rectangleSegmentA001) - 90, 2.4], %, $rectangleSegmentB001)
+  |> angledLine([
+       segAng(rectangleSegmentA001),
+       -segLen(rectangleSegmentA001)
+     ], %, $rectangleSegmentC001)
+  |> line(endAbsolute = [profileStartX(%), profileStartY(%)])
+  |> close()
+extrude001 = extrude(sketch001, length = 5)
+sketch003 = startSketchOn(extrude001, 'START')
+  |> circle(center = [-0.69, 0.56], radius = 0.28)
 `

      await context.addInitScript((initialCode) => {
@ -3563,9 +3639,44 @@ radius = 8.69
      const lineCodeToSelection = `|> xLine(length = 2.6)`
      await page.getByText(lineCodeToSelection).click()
      await cmdBar.progressCmdBar()
+      await cmdBar.progressCmdBar()
+      await cmdBar.progressCmdBar()

      const newCodeToFind = `revolve001 = revolve(sketch003, angle = 360, axis = seg01)`
      expect(editor.expectEditor.toContain(newCodeToFind)).toBeTruthy()
+
+      // Edit flow
+      const newAngle = '270'
+      await toolbar.openPane('feature-tree')
+      const operationButton = await toolbar.getFeatureTreeOperation(
+        'Revolve',
+        0
+      )
+      await operationButton.dblclick({ button: 'left' })
+      await cmdBar.expectState({
+        commandName: 'Revolve',
+        currentArgKey: 'angle',
+        currentArgValue: '360',
+        headerArguments: {
+          Angle: '360',
+        },
+        highlightedHeaderArg: 'angle',
+        stage: 'arguments',
+      })
+      await page.keyboard.insertText(newAngle)
+      await cmdBar.progressCmdBar()
+      await cmdBar.expectState({
+        stage: 'review',
+        headerArguments: {
+          Angle: newAngle,
+        },
+        commandName: 'Revolve',
+      })
+      await cmdBar.progressCmdBar()
+      await toolbar.closePane('feature-tree')
+      await editor.expectEditor.toContain(
+        newCodeToFind.replace('angle = 360', 'angle = ' + newAngle)
+      )
    })
  })

@ -3578,7 +3689,8 @@ radius = 8.69
    toolbar,
    cmdBar,
  }) => {
-    const initialCode = `sketch001 = startSketchOn(XZ)
+    const initialCode = `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
 profile001 = circle(
  sketch001,
  center = [0, 0],
--- a/e2e/playwright/projects.spec.ts
+++ b/e2e/playwright/projects.spec.ts
@ -87,7 +87,7 @@ test(
      const bracketDir = path.join(dir, 'bracket')
      await fsp.mkdir(bracketDir, { recursive: true })
      await fsp.copyFile(
-        executorInputPath('focusrite_scarlett_mounting_braket.kcl'),
+        executorInputPath('cylinder-inches.kcl'),
        path.join(bracketDir, 'main.kcl')
      )
    })
@ -124,7 +124,7 @@ test(
      const bracketDir = path.join(dir, 'bracket')
      await fsp.mkdir(bracketDir, { recursive: true })
      await fsp.copyFile(
-        executorInputPath('focusrite_scarlett_mounting_braket.kcl'),
+        executorInputPath('cylinder-inches.kcl'),
        path.join(bracketDir, 'main.kcl')
      )
      const errorDir = path.join(dir, 'broken-code')
@ -162,7 +162,7 @@ test(
      // gray at this pixel means the stream has loaded in the most
      // user way we can verify it (pixel color)
      await expect
-        .poll(() => u.getGreatestPixDiff(pointOnModel, [85, 85, 85]), {
+        .poll(() => u.getGreatestPixDiff(pointOnModel, [110, 110, 110]), {
          timeout: 10_000,
        })
        .toBeLessThan(20)
@ -213,7 +213,7 @@ test(
      const bracketDir = path.join(dir, 'bracket')
      await fsp.mkdir(bracketDir, { recursive: true })
      await fsp.copyFile(
-        executorInputPath('focusrite_scarlett_mounting_braket.kcl'),
+        executorInputPath('cylinder-inches.kcl'),
        path.join(bracketDir, 'main.kcl')
      )
      const emptyDir = path.join(dir, 'empty')
@ -248,7 +248,7 @@ test(
      // gray at this pixel means the stream has loaded in the most
      // user way we can verify it (pixel color)
      await expect
-        .poll(() => u.getGreatestPixDiff(pointOnModel, [85, 85, 85]), {
+        .poll(() => u.getGreatestPixDiff(pointOnModel, [125, 125, 125]), {
          timeout: 10_000,
        })
        .toBeLessThan(15)
@ -290,7 +290,7 @@ test(
      const bracketDir = path.join(dir, 'bracket')
      await fsp.mkdir(bracketDir, { recursive: true })
      await fsp.copyFile(
-        executorInputPath('focusrite_scarlett_mounting_braket.kcl'),
+        executorInputPath('cylinder-inches.kcl'),
        path.join(bracketDir, 'main.kcl')
      )

@ -319,7 +319,7 @@ test(
      // gray at this pixel means the stream has loaded in the most
      // user way we can verify it (pixel color)
      await expect
-        .poll(() => u.getGreatestPixDiff(pointOnModel, [85, 85, 85]), {
+        .poll(() => u.getGreatestPixDiff(pointOnModel, [125, 125, 125]), {
          timeout: 10_000,
        })
        .toBeLessThan(15)
@ -359,7 +359,7 @@ test(
      const bracketDir = path.join(dir, 'bracket')
      await fsp.mkdir(bracketDir, { recursive: true })
      await fsp.copyFile(
-        executorInputPath('focusrite_scarlett_mounting_braket.kcl'),
+        executorInputPath('cylinder-inches.kcl'),
        path.join(bracketDir, 'main.kcl')
      )
      await fsp.copyFile(
@ -393,7 +393,7 @@ test(
      // gray at this pixel means the stream has loaded in the most
      // user way we can verify it (pixel color)
      await expect
-        .poll(() => u.getGreatestPixDiff(pointOnModel, [85, 85, 85]), {
+        .poll(() => u.getGreatestPixDiff(pointOnModel, [125, 125, 125]), {
          timeout: 10_000,
        })
        .toBeLessThan(15)
@ -443,7 +443,6 @@ test(
    await page.getByText('broken-code').click()

    // Gotcha: You can not use scene.waitForExecutionDone() since the KCL code is going to fail
-    await expect(page.getByTestId('loading')).toBeAttached()
    await expect(page.getByTestId('loading')).not.toBeAttached({
      timeout: 20_000,
    })
@ -481,7 +480,7 @@ test.describe('Can export from electron app', () => {
          const bracketDir = path.join(dir, 'bracket')
          await fsp.mkdir(bracketDir, { recursive: true })
          await fsp.copyFile(
-            executorInputPath('focusrite_scarlett_mounting_braket.kcl'),
+            executorInputPath('cylinder-inches.kcl'),
            path.join(bracketDir, 'main.kcl')
          )
        })
@ -513,7 +512,7 @@ test.describe('Can export from electron app', () => {
          // gray at this pixel means the stream has loaded in the most
          // user way we can verify it (pixel color)
          await expect
-            .poll(() => u.getGreatestPixDiff(pointOnModel, [85, 85, 85]), {
+            .poll(() => u.getGreatestPixDiff(pointOnModel, [125, 125, 125]), {
              timeout: 10_000,
            })
            .toBeLessThan(15)
@ -554,7 +553,7 @@ test.describe('Can export from electron app', () => {
              },
              { timeout: 15_000 }
            )
-            .toBeGreaterThan(300_000)
+            .toBeGreaterThan(50_000)

          // clean up exported file
          await fsp.rm(filepath)
@ -1507,7 +1506,12 @@ test(

    await u.waitForPageLoad()

-    await page.locator('.cm-content').fill(`sketch001 = startSketchOn(XZ)
+    // The file should be prepopulated with the user's unit settings.
+    await expect(page.locator('.cm-content')).toHaveText(
+      '@settings(defaultLengthUnit = in)'
+    )
+
+    await page.locator('.cm-content').fill(`sketch001 = startSketchOn('XZ')
  |> startProfileAt([-87.4, 282.92], %)
  |> line(end = [324.07, 27.199], tag = $seg01)
  |> line(end = [118.328, -291.754])
--- a/e2e/playwright/regression-tests.spec.ts
+++ b/e2e/playwright/regression-tests.spec.ts
@ -4,9 +4,9 @@ import path from 'path'
 import * as fsp from 'fs/promises'
 import {
  getUtils,
-  executorInputPath,
  TEST_COLORS,
  TestColor,
+  executorInputPath,
  orRunWhenFullSuiteEnabled,
 } from './test-utils'
 import { TEST_CODE_TRIGGER_ENGINE_EXPORT_ERROR } from './storageStates'
@ -331,7 +331,7 @@ extrude001 = extrude(sketch001, length = 50)
          localStorage.setItem(
            'persistCode',
            `@settings(defaultLengthUnit = mm)
-sketch002 = startSketchOn('XY')
+sketch002 = startSketchOn(XY)
 profile002 = startProfileAt([72.24, -52.05], sketch002)
  |> angledLine([0, 181.26], %, $rectangleSegmentA001)
  |> angledLine([
@ -582,7 +582,7 @@ extrude002 = extrude(profile002, length = 150)
        const bracketDir = path.join(dir, 'bracket')
        await fsp.mkdir(bracketDir, { recursive: true })
        await fsp.copyFile(
-          executorInputPath('focusrite_scarlett_mounting_braket.kcl'),
+          executorInputPath('cylinder-inches.kcl'),
          path.join(bracketDir, 'main.kcl')
        )
      })
@ -619,6 +619,7 @@ extrude002 = extrude(profile002, length = 150)
  test(`View gizmo stays visible even when zoomed out all the way`, async ({
    page,
    homePage,
+    scene,
  }) => {
    const u = await getUtils(page)

@ -632,7 +633,7 @@ extrude002 = extrude(profile002, length = 150)

    await test.step(`Load an empty file`, async () => {
      await page.addInitScript(async () => {
-        localStorage.setItem('persistCode', '')
+        localStorage.setItem('persistCode', '@settings(defaultLengthUnit = in)')
      })
      await page.setBodyDimensions({ width: 1200, height: 500 })
      await homePage.goToModelingScene()
@ -646,22 +647,31 @@ extrude002 = extrude(profile002, length = 150)
          timeout: 5000,
          message: 'Plane color is visible',
        })
-        .toBeLessThanOrEqual(15)
+        .toBeLessThanOrEqual(20)
+      await expect(scene.startEditSketchBtn).toBeEnabled()

      let maxZoomOuts = 10
      let middlePixelIsBackgroundColor =
        (await middlePixelIsColor(bgColor)) < 10
+
+      console.time('pressing control')
+      await page.keyboard.down('Control')
+
      while (!middlePixelIsBackgroundColor && maxZoomOuts > 0) {
-        await page.keyboard.down('Control')
-        await page.mouse.move(600, 460)
-        await page.mouse.down({ button: 'right' })
-        await page.mouse.move(600, 50, { steps: 20 })
-        await page.mouse.up({ button: 'right' })
-        await page.keyboard.up('Control')
        await page.waitForTimeout(100)
+        await page.mouse.move(650, 460)
+        console.time('moved to start point')
+        await page.mouse.down({ button: 'right' })
+        console.time('moused down')
+        await page.mouse.move(650, 50, { steps: 20 })
+        console.time('moved to end point')
+        await page.waitForTimeout(100)
+        await page.mouse.up({ button: 'right' })
+        console.time('moused up')
        maxZoomOuts--
-        middlePixelIsBackgroundColor = (await middlePixelIsColor(bgColor)) < 10
+        middlePixelIsBackgroundColor = (await middlePixelIsColor(bgColor)) < 15
      }
+      await page.keyboard.up('Control')

      expect(middlePixelIsBackgroundColor, {
        message: 'We should not see the default planes',
@ -678,13 +688,12 @@ extrude002 = extrude(profile002, length = 150)
    homePage,
    scene,
    toolbar,
-    viewport,
  }) => {
    await context.folderSetupFn(async (dir) => {
      const legoDir = path.join(dir, 'lego')
      await fsp.mkdir(legoDir, { recursive: true })
      await fsp.copyFile(
-        executorInputPath('lego.kcl'),
+        executorInputPath('e2e-can-sketch-on-chamfer.kcl'),
        path.join(legoDir, 'main.kcl')
      )
    })
@ -697,11 +706,8 @@ extrude002 = extrude(profile002, length = 150)
      await scene.loadingIndicator.waitFor({ state: 'detached' })
    })
    await test.step(`The part should start loading quickly, not waiting until execution is complete`, async () => {
-      await scene.expectPixelColor(
-        [143, 143, 143],
-        { x: (viewport?.width ?? 1200) / 2, y: (viewport?.height ?? 500) / 2 },
-        15
-      )
+      // TODO: use the viewport size to pick the center point, but the `viewport` fixture's values were wrong.
+      await scene.expectPixelColor([116, 116, 116], { x: 500, y: 250 }, 15)
    })
  })

--- a/e2e/playwright/sketch-tests.spec.ts
+++ b/e2e/playwright/sketch-tests.spec.ts
@ -113,7 +113,8 @@ test.describe('Sketch tests', { tag: ['@skipWin'] }, () => {
    await page.addInitScript(async () => {
      localStorage.setItem(
        'persistCode',
-        `sketch001 = startSketchOn(XZ)
+        `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
  |> startProfileAt([2.61, -4.01], %)
  |> xLine(length = 8.73)
  |> tangentialArcTo([8.33, -1.31], %)`
@ -159,7 +160,10 @@ test.describe('Sketch tests', { tag: ['@skipWin'] }, () => {
      await page.mouse.click(700, 200)

      await expect.poll(u.normalisedEditorCode, { timeout: 1000 })
-        .toBe(`sketch002 = startSketchOn(XZ)
+        .toBe(`@settings(defaultLengthUnit = in)
+
+
+sketch002 = startSketchOn(XZ)
 sketch001 = startProfileAt([12.34, -12.34], sketch002)
  |> yLine(length = 12.34)

@ -789,7 +793,8 @@ sketch001 = startProfileAt([12.34, -12.34], sketch002)
      200
    )

-    let codeStr = 'sketch001 = startSketchOn(XY)'
+    let codeStr =
+      '@settings(defaultLengthUnit = in)sketch001 = startSketchOn(XY)'

    await page.mouse.click(center.x, viewportSize.height * 0.55)
    await expect(u.codeLocator).toHaveText(codeStr)
@ -868,7 +873,8 @@ sketch001 = startProfileAt([12.34, -12.34], sketch002)

      await u.openDebugPanel()

-      const code = `sketch001 = startSketchOn(-XZ)
+      const code = `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(-XZ)
 profile001 = startProfileAt([${roundOff(scale * 69.6)}, ${roundOff(
        scale * 34.8
      )}], sketch001)
@ -898,7 +904,7 @@ profile001 = startProfileAt([${roundOff(scale * 69.6)}, ${roundOff(
      await page.mouse.move(700, 200, { steps: 10 })
      await page.mouse.click(700, 200, { delay: 200 })
      await expect(page.locator('.cm-content')).toHaveText(
-        `sketch001 = startSketchOn(-XZ)`
+        `@settings(defaultLengthUnit = in)sketch001 = startSketchOn(-XZ)`
      )

      let prevContent = await page.locator('.cm-content').innerText()
@ -1426,7 +1432,8 @@ test.describe(`Sketching with offset planes`, () => {
    await context.addInitScript(() => {
      localStorage.setItem(
        'persistCode',
-        `offsetPlane001 = offsetPlane(XY, offset = 10)`
+        `@settings(defaultLengthUnit = in)
+offsetPlane001 = offsetPlane(XY, offset = 10)`
      )
    })

@ -1440,7 +1447,7 @@ test.describe(`Sketching with offset planes`, () => {
      await test.step(`Hovering should highlight code`, async () => {
        await planeHover()
        await editor.expectState({
-          activeLines: [`offsetPlane001=offsetPlane(XY,offset=10)`],
+          activeLines: [`@settings(defaultLengthUnit = in)`],
          diagnostics: [],
          highlightedCode: 'offsetPlane(XY, offset = 10)',
        })
@ -1453,7 +1460,7 @@ test.describe(`Sketching with offset planes`, () => {
        await expect(toolbar.lineBtn).toBeEnabled()
        await editor.expectEditor.toContain('startSketchOn(offsetPlane001)')
        await editor.expectState({
-          activeLines: [`offsetPlane001=offsetPlane(XY,offset=10)`],
+          activeLines: [`@settings(defaultLengthUnit = in)`],
          diagnostics: [],
          highlightedCode: '',
        })
@ -1604,7 +1611,8 @@ profile002 = startProfileAt([117.2, 56.08], sketch001)
      await context.addInitScript(() => {
        localStorage.setItem(
          'persistCode',
-          `sketch001 = startSketchOn(XZ)
+          `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
 profile002 = startProfileAt([40.68, 87.67], sketch001)
  |> xLine(length = 239.17)
 profile003 = startProfileAt([206.63, -56.73], sketch001)
@ -2172,7 +2180,8 @@ profile003 = startProfileAt([206.63, -56.73], sketch001)
      await page.addInitScript(async () => {
        localStorage.setItem(
          'persistCode',
-          `sketch001 = startSketchOn(XZ)
+          `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
 profile001 = startProfileAt([6.24, 4.54], sketch001)
  |> line(end = [-0.41, 6.99])
  |> line(end = [8.61, 0.74])
@ -2317,7 +2326,8 @@ profile004 = circleThreePoint(sketch001, p1 = [13.44, -6.8], p2 = [13.39, -2.07]
      await page.addInitScript(async () => {
        localStorage.setItem(
          'persistCode',
-          `sketch001 = startSketchOn(XZ)
+          `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
 profile001 = startProfileAt([6.24, 4.54], sketch001)
  |> line(end = [-0.41, 6.99])
  |> line(end = [8.61, 0.74])
@ -2422,7 +2432,8 @@ profile003 = circle(sketch001, center = [6.92, -4.2], radius = 3.16)
      await page.addInitScript(async () => {
        localStorage.setItem(
          'persistCode',
-          `sketch001 = startSketchOn(XZ)
+          `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
 profile001 = startProfileAt([-63.43, 193.08], sketch001)
  |> line(end = [168.52, 149.87])
  |> line(end = [190.29, -39.18])
@ -2486,7 +2497,11 @@ extrude001 = extrude(profile003, length = 5)
    page,
  }) => {
    await page.addInitScript(async () => {
-      localStorage.setItem('persistCode', `myVar = 5`)
+      localStorage.setItem(
+        'persistCode',
+        `@settings(defaultLengthUnit = in)
+        myVar = 5`
+      )
    })

    await page.setBodyDimensions({ width: 1000, height: 500 })
@ -2533,7 +2548,8 @@ extrude001 = extrude(profile003, length = 5)
      await page.addInitScript(async () => {
        localStorage.setItem(
          'persistCode',
-          `sketch001 = startSketchOn(XZ)
+          `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
 profile001 = startProfileAt([85.19, 338.59], sketch001)
  |> line(end = [213.3, -94.52])
  |> line(end = [-230.09, -55.34])
@ -2575,7 +2591,8 @@ profile002 = startProfileAt([85.81, 52.55], sketch002)
      await page.addInitScript(async () => {
        localStorage.setItem(
          'persistCode',
-          `thePart = startSketchOn(XZ)
+          `@settings(defaultLengthUnit = in)
+thePart = startSketchOn(XZ)
  |> startProfileAt([7.53, 10.51], %)
  |> line(end = [12.54, 1.83])
  |> line(end = [6.65, -6.91])
@ -2636,7 +2653,8 @@ extrude001 = extrude(thePart, length = 75)
      await page.addInitScript(async () => {
        localStorage.setItem(
          'persistCode',
-          `sketch001 = startSketchOn(XZ)
+          `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
 profile001 = startProfileAt([6.71, -3.66], sketch001)
  |> line(end = [2.65, 9.02], tag = $seg02)
  |> line(end = [3.73, -9.36], tag = $seg01)
@ -2809,7 +2827,8 @@ extrude003 = extrude(profile011, length = 2.5)
      await page.addInitScript(async () => {
        localStorage.setItem(
          'persistCode',
-          `sketch001 = startSketchOn(XZ)
+          `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
 profile001 = startProfileAt([34, 42.66], sketch001)
  |> line(end = [102.65, 151.99])
  |> line(end = [76, -138.66])
--- a/e2e/playwright/snapshot-tests.spec.ts-snapshots/Client-side-scene-scale-should-match-engine-scale-Inch-scale-1-Google-Chrome-linux.png
+++ b/e2e/playwright/snapshot-tests.spec.ts-snapshots/Client-side-scene-scale-should-match-engine-scale-Inch-scale-1-Google-Chrome-linux.png
--- a/e2e/playwright/snapshot-tests.spec.ts-snapshots/Client-side-scene-scale-should-match-engine-scale-Inch-scale-2-Google-Chrome-linux.png
+++ b/e2e/playwright/snapshot-tests.spec.ts-snapshots/Client-side-scene-scale-should-match-engine-scale-Inch-scale-2-Google-Chrome-linux.png
--- a/e2e/playwright/snapshot-tests.spec.ts-snapshots/Draft-circle-should-look-right-1-Google-Chrome-linux.png
+++ b/e2e/playwright/snapshot-tests.spec.ts-snapshots/Draft-circle-should-look-right-1-Google-Chrome-linux.png
--- a/e2e/playwright/snapshot-tests.spec.ts-snapshots/Draft-rectangles-should-look-right-1-Google-Chrome-linux.png
+++ b/e2e/playwright/snapshot-tests.spec.ts-snapshots/Draft-rectangles-should-look-right-1-Google-Chrome-linux.png
--- a/e2e/playwright/snapshot-tests.spec.ts-snapshots/Draft-segments-should-look-right-1-Google-Chrome-linux.png
+++ b/e2e/playwright/snapshot-tests.spec.ts-snapshots/Draft-segments-should-look-right-1-Google-Chrome-linux.png
--- a/e2e/playwright/snapshot-tests.spec.ts-snapshots/Draft-segments-should-look-right-2-Google-Chrome-linux.png
+++ b/e2e/playwright/snapshot-tests.spec.ts-snapshots/Draft-segments-should-look-right-2-Google-Chrome-linux.png
--- a/e2e/playwright/snapshot-tests.spec.ts-snapshots/Draft-segments-should-look-right-3-Google-Chrome-linux.png
+++ b/e2e/playwright/snapshot-tests.spec.ts-snapshots/Draft-segments-should-look-right-3-Google-Chrome-linux.png
--- a/e2e/playwright/snapshot-tests.spec.ts-snapshots/Draft-segments-should-look-right-4-Google-Chrome-linux.png
+++ b/e2e/playwright/snapshot-tests.spec.ts-snapshots/Draft-segments-should-look-right-4-Google-Chrome-linux.png
--- a/e2e/playwright/snapshot-tests.spec.ts-snapshots/Grid-visibility-Grid-turned-off-1-Google-Chrome-linux.png
+++ b/e2e/playwright/snapshot-tests.spec.ts-snapshots/Grid-visibility-Grid-turned-off-1-Google-Chrome-linux.png
--- a/e2e/playwright/snapshot-tests.spec.ts-snapshots/Grid-visibility-Grid-turned-off-to-on-via-command-bar-1-Google-Chrome-linux.png
+++ b/e2e/playwright/snapshot-tests.spec.ts-snapshots/Grid-visibility-Grid-turned-off-to-on-via-command-bar-1-Google-Chrome-linux.png
--- a/e2e/playwright/snapshot-tests.spec.ts-snapshots/Sketch-on-face-with-none-z-up-1-Google-Chrome-linux.png
+++ b/e2e/playwright/snapshot-tests.spec.ts-snapshots/Sketch-on-face-with-none-z-up-1-Google-Chrome-linux.png
--- a/e2e/playwright/snapshot-tests.spec.ts-snapshots/Zoom-to-fit-on-load---solid-3d-1-Google-Chrome-linux.png
+++ b/e2e/playwright/snapshot-tests.spec.ts-snapshots/Zoom-to-fit-on-load---solid-3d-1-Google-Chrome-linux.png
--- a/e2e/playwright/snapshot-tests.spec.ts-snapshots/code-color-goober-code-color-goober-1-Google-Chrome-linux.png
+++ b/e2e/playwright/snapshot-tests.spec.ts-snapshots/code-color-goober-code-color-goober-1-Google-Chrome-linux.png
--- a/e2e/playwright/snapshot-tests.spec.ts-snapshots/code-color-goober-code-color-goober-opening-window-1-Google-Chrome-linux.png
+++ b/e2e/playwright/snapshot-tests.spec.ts-snapshots/code-color-goober-code-color-goober-opening-window-1-Google-Chrome-linux.png
--- a/e2e/playwright/snapshot-tests.spec.ts-snapshots/extrude-on-default-planes-should-be-stable--XY-1-Google-Chrome-linux.png
+++ b/e2e/playwright/snapshot-tests.spec.ts-snapshots/extrude-on-default-planes-should-be-stable--XY-1-Google-Chrome-linux.png
--- a/e2e/playwright/snapshot-tests.spec.ts-snapshots/extrude-on-default-planes-should-be-stable--XZ-1-Google-Chrome-linux.png
+++ b/e2e/playwright/snapshot-tests.spec.ts-snapshots/extrude-on-default-planes-should-be-stable--XZ-1-Google-Chrome-linux.png
--- a/e2e/playwright/snapshot-tests.spec.ts-snapshots/extrude-on-default-planes-should-be-stable--YZ-1-Google-Chrome-linux.png
+++ b/e2e/playwright/snapshot-tests.spec.ts-snapshots/extrude-on-default-planes-should-be-stable--YZ-1-Google-Chrome-linux.png
--- a/e2e/playwright/snapshot-tests.spec.ts-snapshots/extrude-on-default-planes-should-be-stable-XY-1-Google-Chrome-linux.png
+++ b/e2e/playwright/snapshot-tests.spec.ts-snapshots/extrude-on-default-planes-should-be-stable-XY-1-Google-Chrome-linux.png
--- a/e2e/playwright/snapshot-tests.spec.ts-snapshots/extrude-on-default-planes-should-be-stable-XZ-1-Google-Chrome-linux.png
+++ b/e2e/playwright/snapshot-tests.spec.ts-snapshots/extrude-on-default-planes-should-be-stable-XZ-1-Google-Chrome-linux.png
--- a/e2e/playwright/snapshot-tests.spec.ts-snapshots/extrude-on-default-planes-should-be-stable-YZ-1-Google-Chrome-linux.png
+++ b/e2e/playwright/snapshot-tests.spec.ts-snapshots/extrude-on-default-planes-should-be-stable-YZ-1-Google-Chrome-linux.png
--- a/e2e/playwright/snapshots/prompt-to-edit/prompt-to-edit-snapshot-tests-spec-ts--change-colour.snap.json
+++ b/e2e/playwright/snapshots/prompt-to-edit/prompt-to-edit-snapshot-tests-spec-ts--change-colour.snap.json
@ -1,5 +1,5 @@
 {
-  "original_source_code": "sketch001 = startSketchOn('XZ')\nprofile001 = startProfileAt([57.81, 250.51], sketch001)\n  |> line(end = [121.13, 56.63], tag = $seg02)\n  |> line(end = [83.37, -34.61], tag = $seg01)\n  |> line(end = [19.66, -116.4])\n  |> line(end = [-221.8, -41.69])\n  |> line(endAbsolute = [profileStartX(%), profileStartY(%)])\n  |> close()\nextrude001 = extrude(profile001, length = 200)\nsketch002 = startSketchOn('XZ')\n  |> startProfileAt([-73.64, -42.89], %)\n  |> xLine(length = 173.71)\n  |> line(end = [-22.12, -94.4])\n  |> xLine(length = -156.98)\n  |> line(endAbsolute = [profileStartX(%), profileStartY(%)])\n  |> close()\nextrude002 = extrude(sketch002, length = 50)\nsketch003 = startSketchOn('XY')\n  |> startProfileAt([52.92, 157.81], %)\n  |> angledLine([0, 176.4], %, $rectangleSegmentA001)\n  |> angledLine([\n       segAng(rectangleSegmentA001) - 90,\n       53.4\n     ], %, $rectangleSegmentB001)\n  |> angledLine([\n       segAng(rectangleSegmentA001),\n       -segLen(rectangleSegmentA001)\n     ], %, $rectangleSegmentC001)\n  |> line(endAbsolute = [profileStartX(%), profileStartY(%)])\n  |> close()\nextrude003 = extrude(sketch003, length = 20)\n",
+  "original_source_code": "sketch001 = startSketchOn('XZ')\nprofile001 = startProfileAt([57.81, 250.51], sketch001)\n  |> line(end = [121.13, 56.63], tag = $seg02)\n  |> line(end = [83.37, -34.61], tag = $seg01)\n  |> line(end = [19.66, -116.4])\n  |> line(end = [-221.8, -41.69])\n  |> line(endAbsolute = [profileStartX(%), profileStartY(%)])\n  |> close()\nextrude001 = extrude(profile001, length = 200)\nsketch002 = startSketchOn('XZ')\n  |> startProfileAt([-73.64, -42.89], %)\n  |> xLine(length = 173.71)\n  |> line(end = [-22.12, -94.4])\n  |> xLine(length = -156.98)\n  |> line(endAbsolute = [profileStartX(%), profileStartY(%)])\n  |> close()\nextrude002 = extrude(sketch002, length = 50)\nsketch003 = startSketchOn(XY)\n  |> startProfileAt([52.92, 157.81], %)\n  |> angledLine([0, 176.4], %, $rectangleSegmentA001)\n  |> angledLine([\n       segAng(rectangleSegmentA001) - 90,\n       53.4\n     ], %, $rectangleSegmentB001)\n  |> angledLine([\n       segAng(rectangleSegmentA001),\n       -segLen(rectangleSegmentA001)\n     ], %, $rectangleSegmentC001)\n  |> line(endAbsolute = [profileStartX(%), profileStartY(%)])\n  |> close()\nextrude003 = extrude(sketch003, length = 20)\n",
  "prompt": "make this neon green please, use #39FF14",
  "source_ranges": [
    {
@ -30,4 +30,4 @@
    }
  ],
  "kcl_version": "0.2.48"
-}
+}
--- a/e2e/playwright/testing-constraints.spec.ts
+++ b/e2e/playwright/testing-constraints.spec.ts
@ -4,7 +4,6 @@ import {
  getUtils,
  TEST_COLORS,
  pollEditorLinesSelectedLength,
-  executorInputPath,
  orRunWhenFullSuiteEnabled,
 } from './test-utils'
 import { XOR } from 'lib/utils'
@ -81,7 +80,8 @@ test.describe('Testing constraints', { tag: ['@skipWin'] }, () => {
    await page.addInitScript(async () => {
      localStorage.setItem(
        'persistCode',
-        `yo = 79
+        `@settings(defaultLengthUnit = in)
+  yo = 79
  part001 = startSketchOn(XZ)
    |> startProfileAt([-7.54, -26.74], %)
    |> line(end = [74.36, 130.4], tag = $seg01)
@ -145,7 +145,8 @@ test.describe('Testing constraints', { tag: ['@skipWin'] }, () => {
        await page.addInitScript(async () => {
          localStorage.setItem(
            'persistCode',
-            `yo = 5
+            `@settings(defaultLengthUnit = in)
+      yo = 5
      part001 = startSketchOn(XZ)
        |> startProfileAt([-7.54, -26.74], %)
        |> line(end = [74.36, 130.4], tag = $seg01)
@ -159,31 +160,6 @@ test.describe('Testing constraints', { tag: ['@skipWin'] }, () => {
        |> xLine(length = segLen(seg_what))
        |> line(endAbsolute = [profileStartX(%), profileStartY(%)])`
          )
-
-          const isChecked = await createNewVariableCheckbox.isChecked()
-          const addVariable = testName === 'Add variable'
-          XOR(isChecked, addVariable) && // XOR because no need to click the checkbox if the state is already correct
-            (await createNewVariableCheckbox.click())
-
-          await page
-            .getByRole('button', { name: 'Add constraining value' })
-            .click()
-
-          // Wait for the codemod to take effect
-          await expect(page.locator('.cm-content')).toContainText(`angle: -57,`)
-          await expect(page.locator('.cm-content')).toContainText(
-            `offset: ${offset},`
-          )
-
-          await pollEditorLinesSelectedLength(page, 2)
-          const activeLinesContent = await page.locator('.cm-activeLine').all()
-          await expect(activeLinesContent[0]).toHaveText(
-            `|> line(end = [74.36, 130.4], tag = $seg01)`
-          )
-          await expect(activeLinesContent[1]).toHaveText(`}, %)`)
-
-          // checking the count of the overlays is a good proxy check that the client sketch scene is in a good state
-          await expect(page.getByTestId('segment-overlay')).toHaveCount(4)
        })
        const u = await getUtils(page)
        await page.setBodyDimensions({ width: 1200, height: 500 })
@ -277,7 +253,8 @@ test.describe('Testing constraints', { tag: ['@skipWin'] }, () => {
        await page.addInitScript(async () => {
          localStorage.setItem(
            'persistCode',
-            `yo = 5
+            `@settings(defaultLengthUnit = in)
+      yo = 5
      part001 = startSketchOn(XZ)
        |> startProfileAt([-7.54, -26.74], %)
        |> line(end = [74.36, 130.4])
@ -387,7 +364,8 @@ test.describe('Testing constraints', { tag: ['@skipWin'] }, () => {
        await page.addInitScript(async () => {
          localStorage.setItem(
            'persistCode',
-            `yo = 5
+            `@settings(defaultLengthUnit = in)
+      yo = 5
      part001 = startSketchOn(XZ)
        |> startProfileAt([-7.54, -26.74], %)
        |> line(end = [74.36, 130.4])
@ -500,7 +478,8 @@ test.describe('Testing constraints', { tag: ['@skipWin'] }, () => {
        await page.addInitScript(async () => {
          localStorage.setItem(
            'persistCode',
-            `yo = 5
+            `@settings(defaultLengthUnit = in)
+      yo = 5
      part001 = startSketchOn(XZ)
        |> startProfileAt([-7.54, -26.74], %)
        |> line(end = [74.36, 130.4])
@ -602,7 +581,8 @@ test.describe('Testing constraints', { tag: ['@skipWin'] }, () => {
        await page.addInitScript(async () => {
          localStorage.setItem(
            'persistCode',
-            `yo = 5
+            `@settings(defaultLengthUnit = in)
+      yo = 5
      part001 = startSketchOn(XZ)
        |> startProfileAt([-7.54, -26.74], %)
        |> line(end = [74.36, 130.4])
@ -688,7 +668,8 @@ test.describe('Testing constraints', { tag: ['@skipWin'] }, () => {
        await page.addInitScript(async () => {
          localStorage.setItem(
            'persistCode',
-            `yo = 5
+            `@settings(defaultLengthUnit = in)
+yo = 5
 part001 = startSketchOn(XZ)
  |> startProfileAt([-7.54, -26.74], %)
  |> line(end = [74.36, 130.4])
@ -768,7 +749,8 @@ part002 = startSketchOn(XZ)
        await page.addInitScript(async (customCode) => {
          localStorage.setItem(
            'persistCode',
-            `yo = 5
+            `@settings(defaultLengthUnit = in)
+      yo = 5
      part001 = startSketchOn(XZ)
        |> startProfileAt([-7.54, -26.74], %)
        |> line(end = [74.36, 130.4])
@ -869,7 +851,8 @@ part002 = startSketchOn(XZ)
        await page.addInitScript(async () => {
          localStorage.setItem(
            'persistCode',
-            `yo = 5
+            `@settings(defaultLengthUnit = in)
+      yo = 5
      part001 = startSketchOn(XZ)
        |> startProfileAt([-7.54, -26.74], %)
        |> line(end = [74.36, 130.4])
@ -950,7 +933,8 @@ part002 = startSketchOn(XZ)
        await page.addInitScript(async () => {
          localStorage.setItem(
            'persistCode',
-            `yo = 5
+            `@settings(defaultLengthUnit = in)
+      yo = 5
      part001 = startSketchOn(XZ)
        |> startProfileAt([-7.54, -26.74], %)
        |> line(end = [74.36, 130.4])
@ -1117,9 +1101,19 @@ test.describe('Electron constraint tests', () => {
      await context.folderSetupFn(async (dir) => {
        const bracketDir = path.join(dir, 'test-sample')
        await fsp.mkdir(bracketDir, { recursive: true })
-        await fsp.copyFile(
-          executorInputPath('angled_line.kcl'),
-          path.join(bracketDir, 'main.kcl')
+        await fsp.writeFile(
+          path.join(bracketDir, 'main.kcl'),
+          `@settings(defaultLengthUnit = in)
+          const part001 = startSketchOn(XY)
+            |> startProfileAt([4.83, 12.56], %)
+            |> line(end = [15.1, 2.48])
+            |> line(end = [3.15, -9.85], tag = $seg01)
+            |> line(end = [-15.17, -4.1])
+            |> angledLine([segAng(seg01), 12.35], %)
+            |> line(end = [-13.02, 10.03])
+            |> close()
+            |> extrude(length = 4)`,
+          'utf-8'
        )
      })

--- a/e2e/playwright/testing-gizmo.spec.ts
+++ b/e2e/playwright/testing-gizmo.spec.ts
@ -255,7 +255,7 @@ test.describe(`Testing gizmo, fixture-based`, () => {
    await context.addInitScript(() => {
      localStorage.setItem(
        'persistCode',
-        `
+        `@settings(defaultLengthUnit = in)
        const sketch002 = startSketchOn(XZ)
          |> startProfileAt([-108.83, -57.48], %)
          |> angledLine([0, 105.13], %, $rectangleSegmentA001)
--- a/e2e/playwright/testing-samples-loading.spec.ts
+++ b/e2e/playwright/testing-samples-loading.spec.ts
@ -46,7 +46,7 @@ test.describe('Testing in-app sample loading', () => {
      page.getByRole('option', {
        name,
      })
-    const warningText = page.getByText('Overwrite current file and units?')
+    const warningText = page.getByText('Overwrite current file with sample?')
    const confirmButton = page.getByRole('button', { name: 'Submit command' })

    await test.step(`Precondition: check the initial code`, async () => {
@ -110,11 +110,9 @@ test.describe('Testing in-app sample loading', () => {
      const commandMethodOption = page.getByRole('option', {
        name: 'Overwrite',
      })
-      const newFileWarning = page.getByText(
-        'Create a new file, overwrite project units?'
-      )
+      const newFileWarning = page.getByText('Create a new file from sample?')
      const overwriteWarning = page.getByText(
-        'Overwrite current file and units?'
+        'Overwrite current file with sample?'
      )
      const confirmButton = page.getByRole('button', { name: 'Submit command' })
      const projectMenuButton = page.getByTestId('project-sidebar-toggle')
--- a/e2e/playwright/testing-segment-overlays.spec.ts
+++ b/e2e/playwright/testing-segment-overlays.spec.ts
@ -210,7 +210,8 @@ test.describe('Testing segment overlays', { tag: ['@skipWin'] }, () => {
      await page.addInitScript(async () => {
        localStorage.setItem(
          'persistCode',
-          `part001 = startSketchOn(XZ)
+          `@settings(defaultLengthUnit = in)
+      part001 = startSketchOn(XZ)
        |> startProfileAt([5 + 0, 20 + 0], %)
        |> line(end = [0.5, -14 + 0])
        |> angledLine({ angle = 3 + 0, length = 32 + 0 }, %)
@ -380,7 +381,8 @@ test.describe('Testing segment overlays', { tag: ['@skipWin'] }, () => {
      await page.addInitScript(async () => {
        localStorage.setItem(
          'persistCode',
-          `yRel001 = -14
+          `@settings(defaultLengthUnit = in)
+    yRel001 = -14
    xRel001 = 0.5
    angle001 = 3
    len001 = 32
@ -459,7 +461,8 @@ test.describe('Testing segment overlays', { tag: ['@skipWin'] }, () => {
      await page.addInitScript(async () => {
        localStorage.setItem(
          'persistCode',
-          `part001 = startSketchOn(XZ)
+          `@settings(defaultLengthUnit = in)
+      part001 = startSketchOn(XZ)
        |> startProfileAt([0, 0], %)
        |> line(end = [0.5, -14 + 0])
        |> angledLine({ angle = 3 + 0, length = 32 + 0 }, %)
@ -590,7 +593,8 @@ test.describe('Testing segment overlays', { tag: ['@skipWin'] }, () => {
      await page.addInitScript(async () => {
        localStorage.setItem(
          'persistCode',
-          `part001 = startSketchOn(XZ)
+          `@settings(defaultLengthUnit = in)
+      part001 = startSketchOn(XZ)
        |> startProfileAt([0, 0], %)
        |> line(end = [0.5, -14 + 0])
        |> angledLine({ angle = 3 + 0, length = 32 + 0 }, %)
@ -751,7 +755,8 @@ test.describe('Testing segment overlays', { tag: ['@skipWin'] }, () => {
      await page.addInitScript(async () => {
        localStorage.setItem(
          'persistCode',
-          `part001 = startSketchOn(XZ)
+          `@settings(defaultLengthUnit = in)
+      part001 = startSketchOn(XZ)
        |> startProfileAt([0, 0], %)
        |> line(end = [0.5, -14 + 0])
        |> angledLine({ angle = 3 + 0, length = 32 + 0 }, %)
@ -831,7 +836,8 @@ test.describe('Testing segment overlays', { tag: ['@skipWin'] }, () => {
      await page.addInitScript(async () => {
        localStorage.setItem(
          'persistCode',
-          `sketch001 = startSketchOn(XZ)
+          `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
 profile001 = startProfileAt([56.37, 120.33], sketch001)
  |> line(end = [162.86, 106.48])
  |> arcTo({
@ -957,7 +963,8 @@ profile001 = startProfileAt([56.37, 120.33], sketch001)
      await page.addInitScript(async () => {
        localStorage.setItem(
          'persistCode',
-          `part001 = startSketchOn(XZ)
+          `@settings(defaultLengthUnit = in)
+part001 = startSketchOn(XZ)
      |> circle(center = [1 + 0, 0], radius = 8)
    `
        )
@ -1077,7 +1084,8 @@ profile001 = startProfileAt([56.37, 120.33], sketch001)
      await page.addInitScript(async () => {
        localStorage.setItem(
          'persistCode',
-          `part001 = startSketchOn(XZ)
+          `@settings(defaultLengthUnit = in)
+part001 = startSketchOn(XZ)
  |>startProfileAt([0, 0], %)
  |> line(end = [0.5, -14 + 0])
  |> angledLine({ angle = 3 + 0, length = 32 + 0 }, %)
@ -1351,7 +1359,8 @@ profile001 = startProfileAt([56.37, 120.33], sketch001)
            async ({ lineToBeDeleted, extraLine }) => {
              localStorage.setItem(
                'persistCode',
-                `part001 = startSketchOn(XZ)
+                `@settings(defaultLengthUnit = in)
+        part001 = startSketchOn(XZ)
          |> startProfileAt([5, 6], %)
          |> ${lineToBeDeleted}
          |> line(end = [-10, -15])
@ -1516,7 +1525,8 @@ profile001 = startProfileAt([56.37, 120.33], sketch001)
          async ({ lineToBeDeleted }) => {
            localStorage.setItem(
              'persistCode',
-              `part001 = startSketchOn(XZ)
+              `@settings(defaultLengthUnit = in)
+      part001 = startSketchOn(XZ)
        |> startProfileAt([5, 6], %)
        |> ${lineToBeDeleted}
        |> line(end = [-10, -15])
--- a/e2e/playwright/testing-selections.spec.ts
+++ b/e2e/playwright/testing-selections.spec.ts
@ -68,20 +68,20 @@ test.describe('Testing selections', { tag: ['@skipWin'] }, () => {
    await u.closeDebugPanel()
    await page.mouse.click(startXPx + PUR * 10, 500 - PUR * 10)
    await expect(page.locator('.cm-content')).toHaveText(
-      `sketch001 = startSketchOn(XZ)profile001 = startProfileAt(${commonPoints.startAt}, sketch001)`
+      `@settings(defaultLengthUnit = in)sketch001 = startSketchOn(XZ)profile001 = startProfileAt(${commonPoints.startAt}, sketch001)`
    )

    await page.waitForTimeout(100)
    await page.mouse.click(startXPx + PUR * 20, 500 - PUR * 10)

    await expect(page.locator('.cm-content'))
-      .toHaveText(`sketch001 = startSketchOn(XZ)profile001 = startProfileAt(${commonPoints.startAt}, sketch001)
+      .toHaveText(`@settings(defaultLengthUnit = in)sketch001 = startSketchOn(XZ)profile001 = startProfileAt(${commonPoints.startAt}, sketch001)
    |> xLine(length = ${commonPoints.num1})`)

    await page.waitForTimeout(100)
    await page.mouse.click(startXPx + PUR * 20, 500 - PUR * 20)
    await expect(page.locator('.cm-content'))
-      .toHaveText(`sketch001 = startSketchOn(XZ)profile001 = startProfileAt(${
+      .toHaveText(`@settings(defaultLengthUnit = in)sketch001 = startSketchOn(XZ)profile001 = startProfileAt(${
      commonPoints.startAt
    }, sketch001)
    |> xLine(length = ${commonPoints.num1})
@ -89,7 +89,7 @@ test.describe('Testing selections', { tag: ['@skipWin'] }, () => {
    await page.waitForTimeout(100)
    await page.mouse.click(startXPx, 500 - PUR * 20)
    await expect(page.locator('.cm-content'))
-      .toHaveText(`sketch001 = startSketchOn(XZ)profile001 = startProfileAt(${
+      .toHaveText(`@settings(defaultLengthUnit = in)sketch001 = startSketchOn(XZ)profile001 = startProfileAt(${
      commonPoints.startAt
    }, sketch001)
    |> xLine(length = ${commonPoints.num1})
@ -260,7 +260,8 @@ test.describe('Testing selections', { tag: ['@skipWin'] }, () => {
    await page.addInitScript(async () => {
      localStorage.setItem(
        'persistCode',
-        `sketch001 = startSketchOn(XZ)
+        `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
  |> startProfileAt([-79.26, 95.04], %)
  |> line(end = [112.54, 127.64], tag = $seg02)
  |> line(end = [170.36, -121.61], tag = $seg01)
@ -528,7 +529,8 @@ profile001 = startProfileAt([7.49, 9.96], sketch001)
    await page.addInitScript(async (KCL_DEFAULT_LENGTH) => {
      localStorage.setItem(
        'persistCode',
-        `part001 = startSketchOn(XZ)
+        `@settings(defaultLengthUnit = in)
+part001 = startSketchOn(XZ)
  |> startProfileAt([20, 0], %)
  |> line(end = [7.13, 4 + 0])
  |> angledLine({ angle = 3 + 0, length = 3.14 + 0 }, %)
@ -747,7 +749,8 @@ profile001 = startProfileAt([7.49, 9.96], sketch001)
    await page.waitForTimeout(200)

    await u.removeCurrentCode()
-    await u.codeLocator.fill(`sketch001 = startSketchOn(XZ)
+    await u.codeLocator.fill(`@settings(defaultLengthUnit = in)
+    sketch001 = startSketchOn(XZ)
    |> startProfileAt([75.8, 317.2], %) // [$startCapTag, $EndCapTag]
    |> angledLine([0, 268.43], %, $rectangleSegmentA001)
    |> angledLine([
@ -965,7 +968,8 @@ profile001 = startProfileAt([7.49, 9.96], sketch001)
      async ({ cases }) => {
        localStorage.setItem(
          'persistCode',
-          `yo = 79
+          `@settings(defaultLengthUnit = in)
+  yo = 79
  part001 = startSketchOn(XZ)
    |> startProfileAt([-7.54, -26.74], %)
    |> ${cases[0].expectedCode}
@ -1020,7 +1024,8 @@ profile001 = startProfileAt([7.49, 9.96], sketch001)
    await page.addInitScript(async () => {
      localStorage.setItem(
        'persistCode',
-        `sketch001 = startSketchOn(XZ)
+        `@settings(defaultLengthUnit = in)
+  sketch001 = startSketchOn(XZ)
    |> startProfileAt([-79.26, 95.04], %)
    |> line(end = [112.54, 127.64])
    |> line(end = [170.36, -121.61], tag = $seg01)
@ -1253,7 +1258,7 @@ profile001 = startProfileAt([7.49, 9.96], sketch001)
    await page.mouse.click(700, 200)

    await expect(page.locator('.cm-content')).toHaveText(
-      `sketch001 = startSketchOn(XZ)`
+      `@settings(defaultLengthUnit = in)sketch001 = startSketchOn(XZ)`
    )

    await page.waitForTimeout(600)
--- a/e2e/playwright/testing-settings.spec.ts
+++ b/e2e/playwright/testing-settings.spec.ts
@ -271,7 +271,7 @@ test.describe('Testing settings', () => {
          const bracketDir = join(dir, projectName)
          await fsp.mkdir(bracketDir, { recursive: true })
          await fsp.copyFile(
-            executorInputPath('focusrite_scarlett_mounting_braket.kcl'),
+            executorInputPath('cylinder-inches.kcl'),
            join(bracketDir, 'main.kcl')
          )
        }
@ -699,19 +699,19 @@ test.describe('Testing settings', () => {
          name: 'Current units are: ',
        })
        await gizmo.click()
-        const button = page.getByRole('button', {
+        const button = page.locator('ul').getByRole('button', {
          name: copy,
          exact: true,
        })
        await button.click()
        const toastMessage = page.getByText(
-          `Set default unit to "${unitOfMeasure}" for this project`
+          `Updated per-file units to ${unitOfMeasure}`
        )
        await expect(toastMessage).toBeVisible()
      }

-      await changeUnitOfMeasureInGizmo('in', 'Inches')
      await changeUnitOfMeasureInGizmo('ft', 'Feet')
+      await changeUnitOfMeasureInGizmo('in', 'Inches')
      await changeUnitOfMeasureInGizmo('yd', 'Yards')
      await changeUnitOfMeasureInGizmo('mm', 'Millimeters')
      await changeUnitOfMeasureInGizmo('cm', 'Centimeters')
@ -951,9 +951,9 @@ test.describe('Testing settings', () => {
      )
    })

-    await test.step(`Initial units from settings`, async () => {
+    await test.step(`Initial units from settings are ignored`, async () => {
      await homePage.openProject('project-000')
-      await expect(unitsIndicator).toHaveText('Current units are: in')
+      await expect(unitsIndicator).toHaveText('Current units are: mm')
    })

    await test.step(`Manually write inline settings`, async () => {
--- a/e2e/playwright/various.spec.ts
+++ b/e2e/playwright/various.spec.ts
@ -483,7 +483,8 @@ test('Sketch on face', async ({ page, homePage, scene, cmdBar, toolbar }) => {
  await page.addInitScript(async () => {
    localStorage.setItem(
      'persistCode',
-      `sketch001 = startSketchOn(XZ)
+      `@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
 |> startProfileAt([3.29, 7.86], %)
 |> line(end = [2.48, 2.44])
 |> line(end = [2.66, 1.17])
--- a/package.json
+++ b/package.json
@ -13,20 +13,20 @@
  "license": "MIT",
  "dependencies": {
    "@codemirror/autocomplete": "^6.18.6",
-    "@codemirror/commands": "^6.8.0",
+    "@codemirror/commands": "^6.8.1",
    "@codemirror/language": "^6.11.0",
-    "@codemirror/lint": "^6.8.4",
+    "@codemirror/lint": "^6.8.5",
    "@codemirror/search": "^6.5.10",
-    "@codemirror/state": "^6.4.1",
+    "@codemirror/state": "^6.5.2",
    "@codemirror/theme-one-dark": "^6.1.2",
-    "@csstools/postcss-oklab-function": "^4.0.7",
+    "@csstools/postcss-oklab-function": "^4.0.8",
    "@fortawesome/fontawesome-svg-core": "^6.7.2",
    "@fortawesome/free-brands-svg-icons": "^6.7.2",
    "@fortawesome/free-solid-svg-icons": "^6.7.2",
    "@fortawesome/react-fontawesome": "^0.2.0",
    "@headlessui/react": "^1.7.19",
-    "@headlessui/tailwindcss": "^0.2.0",
-    "@kittycad/lib": "2.0.21",
+    "@headlessui/tailwindcss": "^0.2.2",
+    "@kittycad/lib": "2.0.23",
    "@lezer/highlight": "^1.2.1",
    "@lezer/lr": "^1.4.1",
    "@react-hook/resize-observer": "^2.0.1",
@ -37,11 +37,11 @@
    "@xstate/react": "^4.1.1",
    "bonjour-service": "^1.3.0",
    "bson": "^6.10.3",
-    "chokidar": "^4.0.1",
+    "chokidar": "^4.0.3",
    "codemirror": "^6.0.1",
    "decamelize": "^6.0.0",
    "diff": "^7.0.0",
-    "electron-updater": "^6.6.0",
+    "electron-updater": "^6.6.2",
    "fuse.js": "^7.1.0",
    "html2canvas-pro": "^1.5.8",
    "isomorphic-fetch": "^3.0.0",
@ -164,15 +164,15 @@
    "@iarna/toml": "^2.2.5",
    "@lezer/generator": "^1.7.2",
    "@nabla/vite-plugin-eslint": "^2.0.5",
-    "@playwright/test": "^1.51.0",
+    "@playwright/test": "^1.51.1",
    "@testing-library/jest-dom": "^5.14.1",
    "@testing-library/react": "^15.0.2",
-    "@types/diff": "^7.0.1",
+    "@types/diff": "^7.0.2",
    "@types/electron": "^1.6.10",
    "@types/isomorphic-fetch": "^0.0.39",
    "@types/minimist": "^1.2.5",
    "@types/mocha": "^10.0.10",
-    "@types/node": "^22.13.9",
+    "@types/node": "^22.13.14",
    "@types/pixelmatch": "^5.2.6",
    "@types/pngjs": "^6.0.4",
    "@types/react": "^18.3.4",
@ -182,13 +182,13 @@
    "@types/ua-parser-js": "^0.7.39",
    "@types/uuid": "^9.0.8",
    "@types/wicg-file-system-access": "^2023.10.5",
-    "@types/ws": "^8.5.13",
-    "@vitejs/plugin-react": "^4.3.0",
+    "@types/ws": "^8.18.0",
+    "@vitejs/plugin-react": "^4.3.4",
    "@vitest/web-worker": "^1.5.0",
    "@xstate/cli": "^0.5.17",
-    "autoprefixer": "^10.4.19",
+    "autoprefixer": "^10.4.21",
    "electron": "^34.1.1",
-    "electron-builder": "^26.0.6",
+    "electron-builder": "^26.0.12",
    "eslint": "^8.0.1",
    "eslint-plugin-css-modules": "^2.12.0",
    "eslint-plugin-import": "^2.31.0",
@ -220,7 +220,7 @@
    "vite-tsconfig-paths": "^4.3.2",
    "vitest": "^1.6.1",
    "vitest-webgl-canvas-mock": "^1.1.0",
-    "ws": "^8.17.0",
+    "ws": "^8.18.1",
    "yarn": "^1.22.22"
  },
  "packageManager": "yarn@1.22.22+sha512.a6b2f7906b721bba3d67d4aff083df04dad64c399707841b7acf00f6b133b7ac24255f2652fa22ae3534329dc6180534e98d17432037ff6fd140556e2bb3137e"
--- a/packages/codemirror-lsp-client/src/client/codec/index.ts
+++ b/packages/codemirror-lsp-client/src/client/codec/index.ts
@ -37,11 +37,11 @@ export class IntoServer
  implements AsyncGenerator<Uint8Array, never, void>
 {
  private worker: Worker | null = null
-  private type_: String | null = null
+  private type_: string | null = null

  private trace: boolean = false

-  constructor(type_?: String, worker?: Worker, trace?: boolean) {
+  constructor(type_?: string, worker?: Worker, trace?: boolean) {
    super()
    if (worker && type_) {
      this.worker = worker
--- a/public/kcl-samples/dual-basin-utility-sink/main.kcl
+++ b/public/kcl-samples/dual-basin-utility-sink/main.kcl
@ -1,6 +1,7 @@
 // Dual-Basin Utility Sink
 // A stainless steel sink unit with dual rectangular basins and six under-counter storage compartments.

+// set units
@settings(defaultLengthUnit = mm)

 // globals
@ -12,7 +13,7 @@ profileThickness = 13
 metalThickness = 2

 blockCount = 3
-blockWidth = (tableWidth-profileThickness) / 3
+blockWidth = (tableWidth - profileThickness) / 3
 blockHeight = tableHeight - metalThickness - 0.5
 blockDepth = tableDepth - profileThickness

@ -27,9 +28,9 @@ legHeight = blockHeight - profileThickness
 legCount = blockCount + 1

 legBody = startProfileAt([0, 0], floorPlane)
-  |> yLine(length=profileThickness)
-  |> xLine(length=profileThickness)
-  |> yLine(length=-profileThickness)
+  |> yLine(length = profileThickness)
+  |> xLine(length = profileThickness)
+  |> yLine(length = -profileThickness)
  |> line(endAbsolute = [profileStartX(%), profileStartY(%)])
  |> close()
  |> patternLinear2d(axis = [1, 0], instances = legCount, distance = blockWidth)
@ -42,9 +43,9 @@ lowerBeltLengthX = blockWidth - profileThickness

 lowerBeltPlane = startSketchOn(offsetPlane(XY, offset = lowerBeltHeightAboveTheFloor))
 lowerBeltBodyX = startProfileAt([profileThickness, 0], lowerBeltPlane)
-  |> yLine(length=profileThickness)
-  |> xLine(length=lowerBeltLengthX)
-  |> yLine(length=-profileThickness)
+  |> yLine(length = profileThickness)
+  |> xLine(length = lowerBeltLengthX)
+  |> yLine(length = -profileThickness)
  |> line(endAbsolute = [profileStartX(%), profileStartY(%)])
  |> close()
  |> patternLinear2d(axis = [1, 0], instances = blockCount, distance = blockWidth)
@ -53,12 +54,12 @@ lowerBeltBodyX = startProfileAt([profileThickness, 0], lowerBeltPlane)

 lowerBeltLengthY = blockDepth - profileThickness
 lowerBeltBodyY = startProfileAt([0, profileThickness], lowerBeltPlane)
-  |> yLine(length=lowerBeltLengthY)
-  |> xLine(length=profileThickness)
-  |> yLine(length=-lowerBeltLengthY)
+  |> yLine(length = lowerBeltLengthY)
+  |> xLine(length = profileThickness)
+  |> yLine(length = -lowerBeltLengthY)
  |> line(endAbsolute = [profileStartX(%), profileStartY(%)])
  |> close()
-  |> patternLinear2d(axis = [1, 0], instances = 2, distance = tableWidth-profileThickness)
+  |> patternLinear2d(axis = [1, 0], instances = 2, distance = tableWidth - profileThickness)
  |> extrude(length = profileThickness)

 // pillars
@ -67,9 +68,9 @@ pillarPlane = startSketchOn(offsetPlane(XY, offset = pillarHeightAboveTheFloor))
 pillarTotalHeight = blockHeight - profileThickness - pillarHeightAboveTheFloor

 pillarBody = startProfileAt([blockSubdivisionWidth, 0], pillarPlane)
-  |> yLine(length=profileThickness)
-  |> xLine(length=profileThickness)
-  |> yLine(length=-profileThickness)
+  |> yLine(length = profileThickness)
+  |> xLine(length = profileThickness)
+  |> yLine(length = -profileThickness)
  |> line(endAbsolute = [profileStartX(%), profileStartY(%)])
  |> close()
  |> patternLinear2d(axis = [1, 0], instances = blockCount, distance = blockWidth)
@ -80,9 +81,9 @@ pillarBody = startProfileAt([blockSubdivisionWidth, 0], pillarPlane)
 upperBeltPlane = startSketchOn(offsetPlane(XY, offset = blockHeight))

 upperBeltBodyX = startProfileAt([0, 0], upperBeltPlane)
-  |> yLine(length=profileThickness)
-  |> xLine(length=tableWidth)
-  |> yLine(length=-profileThickness)
+  |> yLine(length = profileThickness)
+  |> xLine(length = tableWidth)
+  |> yLine(length = -profileThickness)
  |> line(endAbsolute = [profileStartX(%), profileStartY(%)])
  |> close()
  |> patternLinear2d(axis = [0, 1], instances = 2, distance = blockDepth)
@ -90,20 +91,20 @@ upperBeltBodyX = startProfileAt([0, 0], upperBeltPlane)

 upperBeltLengthY = blockDepth - profileThickness
 upperBeltBodyY = startProfileAt([0, profileThickness], upperBeltPlane)
-  |> yLine(length=upperBeltLengthY)
-  |> xLine(length=profileThickness)
-  |> yLine(length=-upperBeltLengthY)
+  |> yLine(length = upperBeltLengthY)
+  |> xLine(length = profileThickness)
+  |> yLine(length = -upperBeltLengthY)
  |> line(endAbsolute = [profileStartX(%), profileStartY(%)])
  |> close()
-  |> patternLinear2d(axis = [1, 0], instances = 2, distance = tableWidth-profileThickness)
+  |> patternLinear2d(axis = [1, 0], instances = 2, distance = tableWidth - profileThickness)
  |> extrude(length = -profileThickness)

 // sink
 tableTopPlane = startSketchOn(offsetPlane(XY, offset = tableHeight))
 tableTopBody = startProfileAt([0, 0], tableTopPlane)
-  |> yLine(length=tableDepth)
-  |> xLine(length=tableWidth)
-  |> yLine(length=-tableDepth)
+  |> yLine(length = tableDepth)
+  |> xLine(length = tableWidth)
+  |> yLine(length = -tableDepth)
  |> line(endAbsolute = [profileStartX(%), profileStartY(%)])
  |> close()
  |> extrude(length = -metalThickness)
@ -114,59 +115,62 @@ sinkLength = 250
 sinkDepth = 200
 sinkOffsetFront = 40
 sinkOffsetLeft = 350
-sinkSpacing = tableWidth - sinkWidth - sinkOffsetLeft*2
+sinkSpacing = tableWidth - sinkWidth - (sinkOffsetLeft * 2)

 sinkPlaneOutside = startSketchOn(tableTopBody, 'START')
 sinkBodyOutside = startProfileAt([-sinkOffsetLeft, sinkOffsetFront], sinkPlaneOutside)
-  |> yLine(length=sinkLength)
-  |> xLine(length=-sinkWidth)
-  |> yLine(length=-sinkLength)
+  |> yLine(length = sinkLength)
+  |> xLine(length = -sinkWidth)
+  |> yLine(length = -sinkLength)
  |> line(endAbsolute = [profileStartX(%), profileStartY(%)])
  |> close()
  |> patternLinear2d(axis = [-1, 0], instances = sinkCount, distance = sinkSpacing)
  |> extrude(length = sinkDepth)

 sinkPlaneInside = startSketchOn(tableTopBody, 'END')
-sinkBodyInside = startProfileAt([sinkOffsetLeft+metalThickness, sinkOffsetFront+metalThickness], sinkPlaneInside)
-  |> yLine(length=sinkLength-metalThickness*2)
-  |> xLine(length=sinkWidth-metalThickness*2)
-  |> yLine(length=-sinkLength+metalThickness*2)
+sinkBodyInside = startProfileAt([
+       sinkOffsetLeft + metalThickness,
+       sinkOffsetFront + metalThickness
+     ], sinkPlaneInside)
+  |> yLine(length = sinkLength - (metalThickness * 2))
+  |> xLine(length = sinkWidth - (metalThickness * 2))
+  |> yLine(length = -sinkLength + metalThickness * 2)
  |> line(endAbsolute = [profileStartX(%), profileStartY(%)])
  |> close()
  |> patternLinear2d(axis = [1, 0], instances = sinkCount, distance = sinkSpacing)
  |> extrude(length = -sinkDepth)
-  
+
 // door panels
 doorGap = 2
-doorWidth = blockSubdivisionWidth - profileThickness - doorGap*2
-doorStart = profileThickness+doorGap
+doorWidth = blockSubdivisionWidth - profileThickness - (doorGap * 2)
+doorStart = profileThickness + doorGap
 doorHeightAboveTheFloor = pillarHeightAboveTheFloor + doorGap
 doorHeight = blockHeight - doorHeightAboveTheFloor - profileThickness - doorGap
 doorCount = blockCount * blockSubdivisionCount

 doorPlane = startSketchOn(offsetPlane(XY, offset = doorHeightAboveTheFloor))
 doorBody = startProfileAt([doorStart, 0], doorPlane)
-  |> yLine(length=profileThickness)
-  |> xLine(length=doorWidth)
-  |> yLine(length=-profileThickness)
+  |> yLine(length = profileThickness)
+  |> xLine(length = doorWidth)
+  |> yLine(length = -profileThickness)
  |> line(endAbsolute = [profileStartX(%), profileStartY(%)])
  |> close()
  |> patternLinear2d(axis = [1, 0], instances = doorCount, distance = blockSubdivisionWidth)
  |> extrude(length = doorHeight)

 // side panels
-panelWidth = blockDepth - profileThickness - doorGap*2
+panelWidth = blockDepth - profileThickness - (doorGap * 2)
 panelCount = doorCount + 1
 panelSpacing = tableWidth - profileThickness
 panelBody = startProfileAt([0, doorStart], doorPlane)
-  |> yLine(length=panelWidth)
-  |> xLine(length=profileThickness)
-  |> yLine(length=-panelWidth)
+  |> yLine(length = panelWidth)
+  |> xLine(length = profileThickness)
+  |> yLine(length = -panelWidth)
  |> line(endAbsolute = [profileStartX(%), profileStartY(%)])
  |> close()
  |> patternLinear2d(axis = [1, 0], instances = 2, distance = panelSpacing)
  |> extrude(length = doorHeight)
-  
+
 // handle
 handleDepth = 40
 handleWidth = 120
@ -179,22 +183,25 @@ handleLengthSegmentB = handleWidth - (handleFillet * 2)
 handlePlane = startSketchOn(offsetPlane(XY, offset = handleHeightAboveTheFloor))

 handleProfilePath = startProfileAt([0 + handleOffset, 0], handlePlane)
-  |> yLine(length=-handleLengthSegmentA)
+  |> yLine(length = -handleLengthSegmentA)
  |> tangentialArcTo([
       handleFillet + handleOffset,
       -handleDepth
     ], %)
-  |> xLine(length=handleLengthSegmentB)
+  |> xLine(length = handleLengthSegmentB)
  |> tangentialArcTo([
       handleOffset + handleWidth,
       -handleLengthSegmentA
     ], %)
-  |> yLine(length=handleLengthSegmentA)
+  |> yLine(length = handleLengthSegmentA)
 handleSectionPlane = startSketchOn(XZ)
 handleProfileSection = circle(
  handleSectionPlane,
-  center = [handleOffset, handleHeightAboveTheFloor],
-  radius = 2)
-  
+  center = [
+    handleOffset,
+    handleHeightAboveTheFloor
+  ],
+  radius = 2,
+)
 handleBody = sweep(handleProfileSection, path = handleProfilePath)
  |> patternLinear3d(axis = [1, 0, 0], instances = doorCount, distance = blockSubdivisionWidth)
--- a/public/kcl-samples/i-beam/main.kcl
+++ b/public/kcl-samples/i-beam/main.kcl
@ -5,7 +5,7 @@
@settings(defaultLengthUnit = in)

 // Define Beam Dimensions
-beamLength = 6*ft()
+beamLength = 6 * ft()
 beamHeight = 4
 flangeWidth = 2.663
 flangeThickness = 0.293
--- a/public/kcl-samples/makeup-mirror/main.kcl
+++ b/public/kcl-samples/makeup-mirror/main.kcl
@ -52,19 +52,22 @@ armPartB = armFn(XZ, armLength, hingeHeight * 2.5 + hingeGap * 2)

 // mirror
 fn mirrorFn(plane, offsetX, offsetY, altitude, radius, tiefe, gestellR, gestellD) {
-  armPlane = startSketchOn(offsetPlane(plane, offset = offsetY - (tiefe / 2)))
+  armPlane = startSketchOn(  offsetPlane(plane, offset = offsetY - (tiefe / 2)))
  armBody = circle(armPlane, center = [offsetX, altitude], radius = radius)
    |> extrude(length = tiefe)

-  archBody = startProfileAt([offsetX-gestellR, altitude], armPlane)
+  archBody = startProfileAt([offsetX - gestellR, altitude], armPlane)
    |> xLine(length = gestellD)
    |> arcTo({
-         interior = [offsetX, altitude-gestellR],
-         end = [offsetX+gestellR, altitude]
+         interior = [offsetX, altitude - gestellR],
+         end = [offsetX + gestellR, altitude]
       }, %)
    |> xLine(length = gestellD)
    |> arcTo({
-         interior = [offsetX, altitude-gestellR-gestellD],
+         interior = [
+           offsetX,
+           altitude - gestellR - gestellD
+         ],
         end = [profileStartX(%), profileStartY(%)]
       }, %)
    |> close()
@ -72,4 +75,4 @@ fn mirrorFn(plane, offsetX, offsetY, altitude, radius, tiefe, gestellR, gestellD
  return armBody
 }

-mirror = mirrorFn(XZ, armLength, armLength, hingeHeight * 4 + hingeGap * 3 + mirrorRadius+archToMirrorGap+archThickness, mirrorRadius, mirrorThickness, archRadius, archThickness)
+mirror = mirrorFn(XZ, armLength, armLength, hingeHeight * 4 + hingeGap * 3 + mirrorRadius + archToMirrorGap + archThickness, mirrorRadius, mirrorThickness, archRadius, archThickness)
--- a/public/kcl-samples/pipe-flange-assembly/91251a404-bolt.kcl
+++ b/public/kcl-samples/pipe-flange-assembly/91251a404-bolt.kcl
@ -4,7 +4,6 @@
 // set units
@settings(defaultLengthUnit = in)

-
 // import constants
 import boltDiameter, boltLength, boltHeadLength, boltHeadDiameter, boltHexDrive, boltHexFlatLength, boltThreadLength from "globals.kcl"

--- a/public/kcl-samples/pipe-flange-assembly/98017a257-washer.kcl
+++ b/public/kcl-samples/pipe-flange-assembly/98017a257-washer.kcl
@ -1,5 +1,5 @@
 // 98017A257 Washer
-// washer for the screws in the pipe flange assembly. 
+// washer for the screws in the pipe flange assembly.

 // set units
@settings(defaultLengthUnit = in)
--- a/public/kcl-samples/pipe-flange-assembly/main.kcl
+++ b/public/kcl-samples/pipe-flange-assembly/main.kcl
@ -19,27 +19,15 @@ import pipe from "1120t74-pipe.kcl"
 flange()
 flange()
  |> rotate(axis = [0, 1, 0], angle = 180)
-  |> translate(
-       x = 0,
-       y = 0,
-       z = flangeBackHeight * 2 + gasketThickness,
-     )
+  |> translate(x = 0, y = 0, z = flangeBackHeight * 2 + gasketThickness)

 // place gasket between the flanges
 gasket()
-  |> translate(
-       x = 0,
-       y = 0,
-       z = -flangeBackHeight - gasketThickness
-     )
+  |> translate(x = 0, y = 0, z = -flangeBackHeight - gasketThickness)

 // place eight washers (four front, four back)
 washer()
-  |> translate(
-       x = mountingHolePlacementDiameter / 2,
-       y = 0,
-       z = flangeBaseThickness
-     )
+  |> translate(x = mountingHolePlacementDiameter / 2, y = 0, z = flangeBaseThickness)
  |> patternCircular3d(
       %,
       instances = 4,
@ -57,11 +45,7 @@ washer()

 // place four bolts
 bolt()
-  |> translate(
-       x = mountingHolePlacementDiameter / 2,
-       y = 0,
-       z = flangeBaseThickness + washerThickness,
-     )
+  |> translate(x = mountingHolePlacementDiameter / 2, y = 0, z = flangeBaseThickness + washerThickness)
  |> rotate(roll = 90, pitch = 0, yaw = 0)
  |> patternCircular3d(
       %,
@ -74,11 +58,7 @@ bolt()

 // place four hex nuts
 hexNut()
-  |> translate(
-       x = mountingHolePlacementDiameter / 2,
-       y = 0,
-       z = -(flangeBackHeight * 2 + gasketThickness + flangeBaseThickness + washerThickness + hexNutThickness),
-     )
+  |> translate(x = mountingHolePlacementDiameter / 2, y = 0, z = -(flangeBackHeight * 2 + gasketThickness + flangeBaseThickness + washerThickness + hexNutThickness))
  |> patternCircular3d(
       %,
       instances = 4,
@ -97,11 +77,11 @@ pipe()
       yaw = 0,
     )
  |> translate(
-         %,
-         x = 0,
-         y = 0,
-         z = flangeBaseThickness + flangeFrontHeight - 0.5,
-         global = true,
+       %,
+       x = 0,
+       y = 0,
+       z = flangeBaseThickness + flangeFrontHeight - 0.5,
+       global = true,
     )

 pipe()
@ -112,9 +92,9 @@ pipe()
       yaw = 0,
     )
  |> translate(
-         %,
-         x = 0,
-         y = 0,
-         z = -(flangeBackHeight * 2 + gasketThickness + flangeBaseThickness + flangeFrontHeight - 0.5),
-         global = true,
+       %,
+       x = 0,
+       y = 0,
+       z = -(flangeBackHeight * 2 + gasketThickness + flangeBaseThickness + flangeFrontHeight - 0.5),
+       global = true,
     )
--- a/public/kcl-samples/walkie-talkie/main.kcl
+++ b/public/kcl-samples/walkie-talkie/main.kcl
@ -32,48 +32,36 @@ talkButton()

 // import the frequency knob
 knob()
-  |> translate(
-    x = width / 2 - 0.70,
-    y = -thickness / 2,
-    z = height / 2
-    )
+  |> translate(x = width / 2 - 0.70, y = -thickness / 2, z = height / 2)

 // import the buttons
 button()
-  |> translate(
-    x = -(screenWidth / 2 + tolerance),
-    y = -1,
-    z = screenYPosition
-    )
+  |> translate(x = -(screenWidth / 2 + tolerance), y = -1, z = screenYPosition)
 button()
+  |> translate(x = -(screenWidth / 2 + tolerance), y = -1, z = screenYPosition - buttonHeight - (tolerance * 2))
+button()
+  |> rotate(
+       %,
+       roll = 0,
+       pitch = 180,
+       yaw = 0,
+     )
  |> translate(
-       x = -(screenWidth / 2 + tolerance),
+       x = screenWidth / 2 + tolerance,
       y = -1,
-       z = screenYPosition - buttonHeight - (tolerance * 2)
-     )
-button()
-  |> rotate(
-       %,
-       roll = 0,
-       pitch = 180,
-       yaw = 0,
-     )
-  |> translate(
-        x = screenWidth / 2 + tolerance,
-        y = -1,
-        z = screenYPosition - buttonHeight,
-        global = true,
-     )
-button()
-  |> rotate(
-       %,
-       roll = 0,
-       pitch = 180,
-       yaw = 0,
-     )
-  |> translate(
-        x = screenWidth / 2 + tolerance,
-        y = -1,
-        z = screenYPosition - (buttonHeight * 2) - (tolerance * 2),
+       z = screenYPosition - buttonHeight,
+       global = true,
+     )
+button()
+  |> rotate(
+       %,
+       roll = 0,
+       pitch = 180,
+       yaw = 0,
+     )
+  |> translate(
+       x = screenWidth / 2 + tolerance,
+       y = -1,
+       z = screenYPosition - (buttonHeight * 2) - (tolerance * 2),
       global = true,
     )
--- a/rust/Cargo.lock
+++ b/rust/Cargo.lock
@ -1780,7 +1780,7 @@ dependencies = [

 [[package]]
 name = "kcl-bumper"
-version = "0.1.56"
+version = "0.1.57"
 dependencies = [
 "anyhow",
 "clap",
@ -1791,7 +1791,7 @@ dependencies = [

 [[package]]
 name = "kcl-derive-docs"
-version = "0.1.56"
+version = "0.1.57"
 dependencies = [
 "Inflector",
 "anyhow",
@ -1810,7 +1810,7 @@ dependencies = [

 [[package]]
 name = "kcl-directory-test-macro"
-version = "0.1.56"
+version = "0.1.57"
 dependencies = [
 "proc-macro2",
 "quote",
@ -1819,7 +1819,7 @@ dependencies = [

 [[package]]
 name = "kcl-language-server"
-version = "0.2.56"
+version = "0.2.57"
 dependencies = [
 "anyhow",
 "clap",
@ -1840,7 +1840,7 @@ dependencies = [

 [[package]]
 name = "kcl-language-server-release"
-version = "0.1.56"
+version = "0.1.57"
 dependencies = [
 "anyhow",
 "clap",
@ -1860,7 +1860,7 @@ dependencies = [

 [[package]]
 name = "kcl-lib"
-version = "0.2.56"
+version = "0.2.57"
 dependencies = [
 "anyhow",
 "approx 0.5.1",
@ -1928,7 +1928,7 @@ dependencies = [

 [[package]]
 name = "kcl-python-bindings"
-version = "0.3.56"
+version = "0.3.57"
 dependencies = [
 "anyhow",
 "kcl-lib",
@ -1943,7 +1943,7 @@ dependencies = [

 [[package]]
 name = "kcl-test-server"
-version = "0.1.56"
+version = "0.1.57"
 dependencies = [
 "anyhow",
 "hyper 0.14.32",
@ -1956,7 +1956,7 @@ dependencies = [

 [[package]]
 name = "kcl-to-core"
-version = "0.1.56"
+version = "0.1.57"
 dependencies = [
 "anyhow",
 "async-trait",
@ -1970,7 +1970,7 @@ dependencies = [

 [[package]]
 name = "kcl-wasm-lib"
-version = "0.1.56"
+version = "0.1.57"
 dependencies = [
 "bson",
 "console_error_panic_hook",
@ -2033,9 +2033,9 @@ dependencies = [

 [[package]]
 name = "kittycad-modeling-cmds"
-version = "0.2.107"
+version = "0.2.108"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "cb129c1f4906a76e3518e58f61968f16cb56f1279366415d2bae6059aa68fce8"
+checksum = "58f5fbcfb0fe3384592829cd1a5109aa45fb6a3ffc5ff2d164d2fd528ffd4641"
 dependencies = [
 "anyhow",
 "chrono",
--- a/rust/Cargo.toml
+++ b/rust/Cargo.toml
@ -36,7 +36,7 @@ dashmap = { version = "6.1.0" }
 http = "1"
 indexmap = "2.7.0"
 kittycad = { version = "0.3.36", default-features = false, features = ["js", "requests"] }
-kittycad-modeling-cmds = { version = "0.2.107", features = ["ts-rs", "websocket"] }
+kittycad-modeling-cmds = { version = "0.2.108", features = ["ts-rs", "websocket"] }
 lazy_static = "1.5.0"
 miette = "7.5.0"
 pyo3 = { version = "0.24.0" }
--- a/rust/kcl-bumper/Cargo.toml
+++ b/rust/kcl-bumper/Cargo.toml
@ -1,7 +1,7 @@

 [package]
 name = "kcl-bumper"
-version = "0.1.56"
+version = "0.1.57"
 edition = "2021"
 repository = "https://github.com/KittyCAD/modeling-api"
 rust-version = "1.76"
--- a/rust/kcl-derive-docs/Cargo.toml
+++ b/rust/kcl-derive-docs/Cargo.toml
@ -1,7 +1,7 @@
 [package]
 name = "kcl-derive-docs"
 description = "A tool for generating documentation from Rust derive macros"
-version = "0.1.56"
+version = "0.1.57"
 edition = "2021"
 license = "MIT"
 repository = "https://github.com/KittyCAD/modeling-app"
--- a/rust/kcl-derive-docs/src/lib.rs
+++ b/rust/kcl-derive-docs/src/lib.rs
@ -329,7 +329,7 @@ fn do_stdlib_inner(
        let camel_case_arg_name = to_camel_case(&arg_name);
        if ty_string != "ExecState" && ty_string != "Args" {
            let schema = quote! {
-                #docs_crate::cleanup_number_tuples_root(generator.root_schema_for::<#ty_ident>())
+                generator.root_schema_for::<#ty_ident>()
            };
            arg_types.push(quote! {
                #docs_crate::StdLibFnArg {
@ -394,7 +394,7 @@ fn do_stdlib_inner(
    let return_type = if !ret_ty_string.is_empty() || ret_ty_string != "()" {
        let ret_ty_string = rust_type_to_openapi_type(&ret_ty_string);
        quote! {
-            let schema = #docs_crate::cleanup_number_tuples_root(generator.root_schema_for::<#return_type_inner>());
+            let schema = generator.root_schema_for::<#return_type_inner>();
            Some(#docs_crate::StdLibFnArg {
                name: "".to_string(),
                type_: #ret_ty_string.to_string(),
@ -816,7 +816,7 @@ fn generate_code_block_test(fn_name: &str, code_block: &str, index: usize) -> pr
        async fn #test_name() -> miette::Result<()> {
            let code = #code_block;
            // Note, `crate` must be kcl_lib
-            let result = match crate::test_server::execute_and_snapshot(code, crate::settings::types::UnitLength::Mm, None).await {
+            let result = match crate::test_server::execute_and_snapshot(code, None).await {
                Err(crate::errors::ExecError::Kcl(e)) => {
                    return Err(miette::Report::new(crate::errors::Report {
                        error: e.error,
--- a/rust/kcl-derive-docs/tests/args_with_lifetime.gen
+++ b/rust/kcl-derive-docs/tests/args_with_lifetime.gen
@ -31,13 +31,7 @@ mod test_examples_someFn {
    #[tokio::test(flavor = "multi_thread", worker_threads = 5)]
    async fn kcl_test_example_someFn0() -> miette::Result<()> {
        let code = "someFn()";
-        let result = match crate::test_server::execute_and_snapshot(
-            code,
-            crate::settings::types::UnitLength::Mm,
-            None,
-        )
-        .await
-        {
+        let result = match crate::test_server::execute_and_snapshot(code, None).await {
            Err(crate::errors::ExecError::Kcl(e)) => {
                return Err(miette::Report::new(crate::errors::Report {
                    error: e.error,
@ -108,7 +102,7 @@ impl crate::docs::StdLibFn for SomeFn {
        vec![crate::docs::StdLibFnArg {
            name: "data".to_string(),
            type_: "Foo".to_string(),
-            schema: crate::docs::cleanup_number_tuples_root(generator.root_schema_for::<Foo>()),
+            schema: generator.root_schema_for::<Foo>(),
            required: true,
            label_required: true,
            description: String::new().to_string(),
@ -120,7 +114,7 @@ impl crate::docs::StdLibFn for SomeFn {
        let mut settings = schemars::gen::SchemaSettings::openapi3();
        settings.inline_subschemas = inline_subschemas;
        let mut generator = schemars::gen::SchemaGenerator::new(settings);
-        let schema = crate::docs::cleanup_number_tuples_root(generator.root_schema_for::<i32>());
+        let schema = generator.root_schema_for::<i32>();
        Some(crate::docs::StdLibFnArg {
            name: "".to_string(),
            type_: "i32".to_string(),
--- a/rust/kcl-derive-docs/tests/args_with_refs.gen
+++ b/rust/kcl-derive-docs/tests/args_with_refs.gen
@ -31,13 +31,7 @@ mod test_examples_someFn {
    #[tokio::test(flavor = "multi_thread", worker_threads = 5)]
    async fn kcl_test_example_someFn0() -> miette::Result<()> {
        let code = "someFn()";
-        let result = match crate::test_server::execute_and_snapshot(
-            code,
-            crate::settings::types::UnitLength::Mm,
-            None,
-        )
-        .await
-        {
+        let result = match crate::test_server::execute_and_snapshot(code, None).await {
            Err(crate::errors::ExecError::Kcl(e)) => {
                return Err(miette::Report::new(crate::errors::Report {
                    error: e.error,
@ -108,7 +102,7 @@ impl crate::docs::StdLibFn for SomeFn {
        vec![crate::docs::StdLibFnArg {
            name: "data".to_string(),
            type_: "string".to_string(),
-            schema: crate::docs::cleanup_number_tuples_root(generator.root_schema_for::<str>()),
+            schema: generator.root_schema_for::<str>(),
            required: true,
            label_required: true,
            description: String::new().to_string(),
@ -120,7 +114,7 @@ impl crate::docs::StdLibFn for SomeFn {
        let mut settings = schemars::gen::SchemaSettings::openapi3();
        settings.inline_subschemas = inline_subschemas;
        let mut generator = schemars::gen::SchemaGenerator::new(settings);
-        let schema = crate::docs::cleanup_number_tuples_root(generator.root_schema_for::<i32>());
+        let schema = generator.root_schema_for::<i32>();
        Some(crate::docs::StdLibFnArg {
            name: "".to_string(),
            type_: "i32".to_string(),
--- a/rust/kcl-derive-docs/tests/array.gen
+++ b/rust/kcl-derive-docs/tests/array.gen
@ -32,13 +32,7 @@ mod test_examples_show {
    #[tokio::test(flavor = "multi_thread", worker_threads = 5)]
    async fn kcl_test_example_show0() -> miette::Result<()> {
        let code = "This is another code block.\nyes sirrr.\nshow";
-        let result = match crate::test_server::execute_and_snapshot(
-            code,
-            crate::settings::types::UnitLength::Mm,
-            None,
-        )
-        .await
-        {
+        let result = match crate::test_server::execute_and_snapshot(code, None).await {
            Err(crate::errors::ExecError::Kcl(e)) => {
                return Err(miette::Report::new(crate::errors::Report {
                    error: e.error,
@ -109,9 +103,7 @@ impl crate::docs::StdLibFn for Show {
        vec![crate::docs::StdLibFnArg {
            name: "args".to_string(),
            type_: "[number]".to_string(),
-            schema: crate::docs::cleanup_number_tuples_root(
-                generator.root_schema_for::<[f64; 2usize]>(),
-            ),
+            schema: generator.root_schema_for::<[f64; 2usize]>(),
            required: true,
            label_required: true,
            description: String::new().to_string(),
@ -123,7 +115,7 @@ impl crate::docs::StdLibFn for Show {
        let mut settings = schemars::gen::SchemaSettings::openapi3();
        settings.inline_subschemas = inline_subschemas;
        let mut generator = schemars::gen::SchemaGenerator::new(settings);
-        let schema = crate::docs::cleanup_number_tuples_root(generator.root_schema_for::<f64>());
+        let schema = generator.root_schema_for::<f64>();
        Some(crate::docs::StdLibFnArg {
            name: "".to_string(),
            type_: "number".to_string(),
--- a/rust/kcl-derive-docs/tests/box.gen
+++ b/rust/kcl-derive-docs/tests/box.gen
@ -32,13 +32,7 @@ mod test_examples_show {
    #[tokio::test(flavor = "multi_thread", worker_threads = 5)]
    async fn kcl_test_example_show0() -> miette::Result<()> {
        let code = "This is code.\nIt does other shit.\nshow";
-        let result = match crate::test_server::execute_and_snapshot(
-            code,
-            crate::settings::types::UnitLength::Mm,
-            None,
-        )
-        .await
-        {
+        let result = match crate::test_server::execute_and_snapshot(code, None).await {
            Err(crate::errors::ExecError::Kcl(e)) => {
                return Err(miette::Report::new(crate::errors::Report {
                    error: e.error,
@ -109,7 +103,7 @@ impl crate::docs::StdLibFn for Show {
        vec![crate::docs::StdLibFnArg {
            name: "args".to_string(),
            type_: "number".to_string(),
-            schema: crate::docs::cleanup_number_tuples_root(generator.root_schema_for::<f64>()),
+            schema: generator.root_schema_for::<f64>(),
            required: true,
            label_required: true,
            description: String::new().to_string(),
@ -121,7 +115,7 @@ impl crate::docs::StdLibFn for Show {
        let mut settings = schemars::gen::SchemaSettings::openapi3();
        settings.inline_subschemas = inline_subschemas;
        let mut generator = schemars::gen::SchemaGenerator::new(settings);
-        let schema = crate::docs::cleanup_number_tuples_root(generator.root_schema_for::<f64>());
+        let schema = generator.root_schema_for::<f64>();
        Some(crate::docs::StdLibFnArg {
            name: "".to_string(),
            type_: "number".to_string(),
--- a/rust/kcl-derive-docs/tests/doc_comment_with_code.gen
+++ b/rust/kcl-derive-docs/tests/doc_comment_with_code.gen
@ -33,13 +33,7 @@ mod test_examples_my_func {
    #[tokio::test(flavor = "multi_thread", worker_threads = 5)]
    async fn kcl_test_example_my_func0() -> miette::Result<()> {
        let code = "This is another code block.\nyes sirrr.\nmyFunc";
-        let result = match crate::test_server::execute_and_snapshot(
-            code,
-            crate::settings::types::UnitLength::Mm,
-            None,
-        )
-        .await
-        {
+        let result = match crate::test_server::execute_and_snapshot(code, None).await {
            Err(crate::errors::ExecError::Kcl(e)) => {
                return Err(miette::Report::new(crate::errors::Report {
                    error: e.error,
@ -110,9 +104,7 @@ impl crate::docs::StdLibFn for MyFunc {
        vec![crate::docs::StdLibFnArg {
            name: "args".to_string(),
            type_: "kittycad::types::InputFormat".to_string(),
-            schema: crate::docs::cleanup_number_tuples_root(
-                generator.root_schema_for::<Option<kittycad::types::InputFormat>>(),
-            ),
+            schema: generator.root_schema_for::<Option<kittycad::types::InputFormat>>(),
            required: false,
            label_required: true,
            description: String::new().to_string(),
@ -124,8 +116,7 @@ impl crate::docs::StdLibFn for MyFunc {
        let mut settings = schemars::gen::SchemaSettings::openapi3();
        settings.inline_subschemas = inline_subschemas;
        let mut generator = schemars::gen::SchemaGenerator::new(settings);
-        let schema =
-            crate::docs::cleanup_number_tuples_root(generator.root_schema_for::<Vec<Sketch>>());
+        let schema = generator.root_schema_for::<Vec<Sketch>>();
        Some(crate::docs::StdLibFnArg {
            name: "".to_string(),
            type_: "[Sketch]".to_string(),
--- a/rust/kcl-derive-docs/tests/lineTo.gen
+++ b/rust/kcl-derive-docs/tests/lineTo.gen
@ -33,13 +33,7 @@ mod test_examples_line_to {
    #[tokio::test(flavor = "multi_thread", worker_threads = 5)]
    async fn kcl_test_example_line_to0() -> miette::Result<()> {
        let code = "This is another code block.\nyes sirrr.\nlineTo";
-        let result = match crate::test_server::execute_and_snapshot(
-            code,
-            crate::settings::types::UnitLength::Mm,
-            None,
-        )
-        .await
-        {
+        let result = match crate::test_server::execute_and_snapshot(code, None).await {
            Err(crate::errors::ExecError::Kcl(e)) => {
                return Err(miette::Report::new(crate::errors::Report {
                    error: e.error,
@ -111,9 +105,7 @@ impl crate::docs::StdLibFn for LineTo {
            crate::docs::StdLibFnArg {
                name: "data".to_string(),
                type_: "LineToData".to_string(),
-                schema: crate::docs::cleanup_number_tuples_root(
-                    generator.root_schema_for::<LineToData>(),
-                ),
+                schema: generator.root_schema_for::<LineToData>(),
                required: true,
                label_required: true,
                description: String::new().to_string(),
@ -122,9 +114,7 @@ impl crate::docs::StdLibFn for LineTo {
            crate::docs::StdLibFnArg {
                name: "sketch".to_string(),
                type_: "Sketch".to_string(),
-                schema: crate::docs::cleanup_number_tuples_root(
-                    generator.root_schema_for::<Sketch>(),
-                ),
+                schema: generator.root_schema_for::<Sketch>(),
                required: true,
                label_required: true,
                description: "the sketch you're adding the line to".to_string(),
@ -137,7 +127,7 @@ impl crate::docs::StdLibFn for LineTo {
        let mut settings = schemars::gen::SchemaSettings::openapi3();
        settings.inline_subschemas = inline_subschemas;
        let mut generator = schemars::gen::SchemaGenerator::new(settings);
-        let schema = crate::docs::cleanup_number_tuples_root(generator.root_schema_for::<Sketch>());
+        let schema = generator.root_schema_for::<Sketch>();
        Some(crate::docs::StdLibFnArg {
            name: "".to_string(),
            type_: "Sketch".to_string(),
--- a/rust/kcl-derive-docs/tests/min.gen
+++ b/rust/kcl-derive-docs/tests/min.gen
@ -32,13 +32,7 @@ mod test_examples_min {
    #[tokio::test(flavor = "multi_thread", worker_threads = 5)]
    async fn kcl_test_example_min0() -> miette::Result<()> {
        let code = "This is another code block.\nyes sirrr.\nmin";
-        let result = match crate::test_server::execute_and_snapshot(
-            code,
-            crate::settings::types::UnitLength::Mm,
-            None,
-        )
-        .await
-        {
+        let result = match crate::test_server::execute_and_snapshot(code, None).await {
            Err(crate::errors::ExecError::Kcl(e)) => {
                return Err(miette::Report::new(crate::errors::Report {
                    error: e.error,
@ -109,9 +103,7 @@ impl crate::docs::StdLibFn for Min {
        vec![crate::docs::StdLibFnArg {
            name: "args".to_string(),
            type_: "[number]".to_string(),
-            schema: crate::docs::cleanup_number_tuples_root(
-                generator.root_schema_for::<Vec<f64>>(),
-            ),
+            schema: generator.root_schema_for::<Vec<f64>>(),
            required: true,
            label_required: true,
            description: String::new().to_string(),
@ -123,7 +115,7 @@ impl crate::docs::StdLibFn for Min {
        let mut settings = schemars::gen::SchemaSettings::openapi3();
        settings.inline_subschemas = inline_subschemas;
        let mut generator = schemars::gen::SchemaGenerator::new(settings);
-        let schema = crate::docs::cleanup_number_tuples_root(generator.root_schema_for::<f64>());
+        let schema = generator.root_schema_for::<f64>();
        Some(crate::docs::StdLibFnArg {
            name: "".to_string(),
            type_: "number".to_string(),
--- a/rust/kcl-derive-docs/tests/option.gen
+++ b/rust/kcl-derive-docs/tests/option.gen
@ -32,13 +32,7 @@ mod test_examples_show {
    #[tokio::test(flavor = "multi_thread", worker_threads = 5)]
    async fn kcl_test_example_show0() -> miette::Result<()> {
        let code = "This is code.\nIt does other shit.\nshow";
-        let result = match crate::test_server::execute_and_snapshot(
-            code,
-            crate::settings::types::UnitLength::Mm,
-            None,
-        )
-        .await
-        {
+        let result = match crate::test_server::execute_and_snapshot(code, None).await {
            Err(crate::errors::ExecError::Kcl(e)) => {
                return Err(miette::Report::new(crate::errors::Report {
                    error: e.error,
@ -109,9 +103,7 @@ impl crate::docs::StdLibFn for Show {
        vec![crate::docs::StdLibFnArg {
            name: "args".to_string(),
            type_: "number".to_string(),
-            schema: crate::docs::cleanup_number_tuples_root(
-                generator.root_schema_for::<Option<f64>>(),
-            ),
+            schema: generator.root_schema_for::<Option<f64>>(),
            required: false,
            label_required: true,
            description: String::new().to_string(),
@ -123,7 +115,7 @@ impl crate::docs::StdLibFn for Show {
        let mut settings = schemars::gen::SchemaSettings::openapi3();
        settings.inline_subschemas = inline_subschemas;
        let mut generator = schemars::gen::SchemaGenerator::new(settings);
-        let schema = crate::docs::cleanup_number_tuples_root(generator.root_schema_for::<f64>());
+        let schema = generator.root_schema_for::<f64>();
        Some(crate::docs::StdLibFnArg {
            name: "".to_string(),
            type_: "number".to_string(),
--- a/rust/kcl-derive-docs/tests/option_input_format.gen
+++ b/rust/kcl-derive-docs/tests/option_input_format.gen
@ -32,13 +32,7 @@ mod test_examples_import {
    #[tokio::test(flavor = "multi_thread", worker_threads = 5)]
    async fn kcl_test_example_import0() -> miette::Result<()> {
        let code = "This is code.\nIt does other shit.\nimport";
-        let result = match crate::test_server::execute_and_snapshot(
-            code,
-            crate::settings::types::UnitLength::Mm,
-            None,
-        )
-        .await
-        {
+        let result = match crate::test_server::execute_and_snapshot(code, None).await {
            Err(crate::errors::ExecError::Kcl(e)) => {
                return Err(miette::Report::new(crate::errors::Report {
                    error: e.error,
@ -109,9 +103,7 @@ impl crate::docs::StdLibFn for Import {
        vec![crate::docs::StdLibFnArg {
            name: "args".to_string(),
            type_: "kittycad::types::InputFormat".to_string(),
-            schema: crate::docs::cleanup_number_tuples_root(
-                generator.root_schema_for::<Option<kittycad::types::InputFormat>>(),
-            ),
+            schema: generator.root_schema_for::<Option<kittycad::types::InputFormat>>(),
            required: false,
            label_required: true,
            description: String::new().to_string(),
@ -123,7 +115,7 @@ impl crate::docs::StdLibFn for Import {
        let mut settings = schemars::gen::SchemaSettings::openapi3();
        settings.inline_subschemas = inline_subschemas;
        let mut generator = schemars::gen::SchemaGenerator::new(settings);
-        let schema = crate::docs::cleanup_number_tuples_root(generator.root_schema_for::<f64>());
+        let schema = generator.root_schema_for::<f64>();
        Some(crate::docs::StdLibFnArg {
            name: "".to_string(),
            type_: "number".to_string(),
--- a/rust/kcl-derive-docs/tests/return_vec_box_sketch.gen
+++ b/rust/kcl-derive-docs/tests/return_vec_box_sketch.gen
@ -32,13 +32,7 @@ mod test_examples_import {
    #[tokio::test(flavor = "multi_thread", worker_threads = 5)]
    async fn kcl_test_example_import0() -> miette::Result<()> {
        let code = "This is code.\nIt does other shit.\nimport";
-        let result = match crate::test_server::execute_and_snapshot(
-            code,
-            crate::settings::types::UnitLength::Mm,
-            None,
-        )
-        .await
-        {
+        let result = match crate::test_server::execute_and_snapshot(code, None).await {
            Err(crate::errors::ExecError::Kcl(e)) => {
                return Err(miette::Report::new(crate::errors::Report {
                    error: e.error,
@ -109,9 +103,7 @@ impl crate::docs::StdLibFn for Import {
        vec![crate::docs::StdLibFnArg {
            name: "args".to_string(),
            type_: "kittycad::types::InputFormat".to_string(),
-            schema: crate::docs::cleanup_number_tuples_root(
-                generator.root_schema_for::<Option<kittycad::types::InputFormat>>(),
-            ),
+            schema: generator.root_schema_for::<Option<kittycad::types::InputFormat>>(),
            required: false,
            label_required: true,
            description: String::new().to_string(),
@ -123,8 +115,7 @@ impl crate::docs::StdLibFn for Import {
        let mut settings = schemars::gen::SchemaSettings::openapi3();
        settings.inline_subschemas = inline_subschemas;
        let mut generator = schemars::gen::SchemaGenerator::new(settings);
-        let schema =
-            crate::docs::cleanup_number_tuples_root(generator.root_schema_for::<Vec<Sketch>>());
+        let schema = generator.root_schema_for::<Vec<Sketch>>();
        Some(crate::docs::StdLibFnArg {
            name: "".to_string(),
            type_: "[Sketch]".to_string(),
--- a/rust/kcl-derive-docs/tests/return_vec_sketch.gen
+++ b/rust/kcl-derive-docs/tests/return_vec_sketch.gen
@ -32,13 +32,7 @@ mod test_examples_import {
    #[tokio::test(flavor = "multi_thread", worker_threads = 5)]
    async fn kcl_test_example_import0() -> miette::Result<()> {
        let code = "This is code.\nIt does other shit.\nimport";
-        let result = match crate::test_server::execute_and_snapshot(
-            code,
-            crate::settings::types::UnitLength::Mm,
-            None,
-        )
-        .await
-        {
+        let result = match crate::test_server::execute_and_snapshot(code, None).await {
            Err(crate::errors::ExecError::Kcl(e)) => {
                return Err(miette::Report::new(crate::errors::Report {
                    error: e.error,
@ -109,9 +103,7 @@ impl crate::docs::StdLibFn for Import {
        vec![crate::docs::StdLibFnArg {
            name: "args".to_string(),
            type_: "kittycad::types::InputFormat".to_string(),
-            schema: crate::docs::cleanup_number_tuples_root(
-                generator.root_schema_for::<Option<kittycad::types::InputFormat>>(),
-            ),
+            schema: generator.root_schema_for::<Option<kittycad::types::InputFormat>>(),
            required: false,
            label_required: true,
            description: String::new().to_string(),
@ -123,8 +115,7 @@ impl crate::docs::StdLibFn for Import {
        let mut settings = schemars::gen::SchemaSettings::openapi3();
        settings.inline_subschemas = inline_subschemas;
        let mut generator = schemars::gen::SchemaGenerator::new(settings);
-        let schema =
-            crate::docs::cleanup_number_tuples_root(generator.root_schema_for::<Vec<Sketch>>());
+        let schema = generator.root_schema_for::<Vec<Sketch>>();
        Some(crate::docs::StdLibFnArg {
            name: "".to_string(),
            type_: "[Sketch]".to_string(),
--- a/rust/kcl-derive-docs/tests/show.gen
+++ b/rust/kcl-derive-docs/tests/show.gen
@ -32,13 +32,7 @@ mod test_examples_show {
    #[tokio::test(flavor = "multi_thread", worker_threads = 5)]
    async fn kcl_test_example_show0() -> miette::Result<()> {
        let code = "This is code.\nIt does other shit.\nshow";
-        let result = match crate::test_server::execute_and_snapshot(
-            code,
-            crate::settings::types::UnitLength::Mm,
-            None,
-        )
-        .await
-        {
+        let result = match crate::test_server::execute_and_snapshot(code, None).await {
            Err(crate::errors::ExecError::Kcl(e)) => {
                return Err(miette::Report::new(crate::errors::Report {
                    error: e.error,
@ -109,9 +103,7 @@ impl crate::docs::StdLibFn for Show {
        vec![crate::docs::StdLibFnArg {
            name: "args".to_string(),
            type_: "[number]".to_string(),
-            schema: crate::docs::cleanup_number_tuples_root(
-                generator.root_schema_for::<Vec<f64>>(),
-            ),
+            schema: generator.root_schema_for::<Vec<f64>>(),
            required: true,
            label_required: true,
            description: String::new().to_string(),
@ -123,7 +115,7 @@ impl crate::docs::StdLibFn for Show {
        let mut settings = schemars::gen::SchemaSettings::openapi3();
        settings.inline_subschemas = inline_subschemas;
        let mut generator = schemars::gen::SchemaGenerator::new(settings);
-        let schema = crate::docs::cleanup_number_tuples_root(generator.root_schema_for::<()>());
+        let schema = generator.root_schema_for::<()>();
        Some(crate::docs::StdLibFnArg {
            name: "".to_string(),
            type_: "()".to_string(),
--- a/rust/kcl-derive-docs/tests/test_args_with_exec_state.gen
+++ b/rust/kcl-derive-docs/tests/test_args_with_exec_state.gen
@ -31,13 +31,7 @@ mod test_examples_some_function {
    #[tokio::test(flavor = "multi_thread", worker_threads = 5)]
    async fn kcl_test_example_some_function0() -> miette::Result<()> {
        let code = "someFunction()";
-        let result = match crate::test_server::execute_and_snapshot(
-            code,
-            crate::settings::types::UnitLength::Mm,
-            None,
-        )
-        .await
-        {
+        let result = match crate::test_server::execute_and_snapshot(code, None).await {
            Err(crate::errors::ExecError::Kcl(e)) => {
                return Err(miette::Report::new(crate::errors::Report {
                    error: e.error,
@ -112,7 +106,7 @@ impl crate::docs::StdLibFn for SomeFunction {
        let mut settings = schemars::gen::SchemaSettings::openapi3();
        settings.inline_subschemas = inline_subschemas;
        let mut generator = schemars::gen::SchemaGenerator::new(settings);
-        let schema = crate::docs::cleanup_number_tuples_root(generator.root_schema_for::<i32>());
+        let schema = generator.root_schema_for::<i32>();
        Some(crate::docs::StdLibFnArg {
            name: "".to_string(),
            type_: "i32".to_string(),
--- a/rust/kcl-directory-test-macro/Cargo.toml
+++ b/rust/kcl-directory-test-macro/Cargo.toml
@ -1,7 +1,7 @@
 [package]
 name = "kcl-directory-test-macro"
 description = "A tool for generating tests from a directory of kcl files"
-version = "0.1.56"
+version = "0.1.57"
 edition = "2021"
 license = "MIT"
 repository = "https://github.com/KittyCAD/modeling-app"
--- a/rust/kcl-language-server-release/Cargo.toml
+++ b/rust/kcl-language-server-release/Cargo.toml
@ -1,6 +1,6 @@
 [package]
 name = "kcl-language-server-release"
-version = "0.1.56"
+version = "0.1.57"
 edition = "2021"
 authors = ["KittyCAD Inc <kcl@kittycad.io>"]
 publish = false
--- a/rust/kcl-language-server/Cargo.toml
+++ b/rust/kcl-language-server/Cargo.toml
@ -2,7 +2,7 @@
 name = "kcl-language-server"
 description = "A language server for KCL."
 authors = ["KittyCAD Inc <kcl@kittycad.io>"]
-version = "0.2.56"
+version = "0.2.57"
 edition = "2021"
 license = "MIT"
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
--- a/rust/kcl-lib/Cargo.toml
+++ b/rust/kcl-lib/Cargo.toml
@ -1,7 +1,7 @@
 [package]
 name = "kcl-lib"
 description = "KittyCAD Language implementation and tools"
-version = "0.2.56"
+version = "0.2.57"
 edition = "2021"
 license = "MIT"
 repository = "https://github.com/KittyCAD/modeling-app"
@ -153,8 +153,3 @@ harness = false
 name = "executor"
 path = "e2e/executor/main.rs"
 required-features = ["engine"]
-
-[[test]]
-name = "modify"
-path = "e2e/modify/main.rs"
-required-features = ["engine"]
--- a/rust/kcl-lib/benches/benchmark_kcl_samples.rs
+++ b/rust/kcl-lib/benches/benchmark_kcl_samples.rs
@ -76,7 +76,7 @@ fn run_benchmarks(c: &mut Criterion) {
        group.bench_function(format!("execute_{}", dir_name), |b| {
            b.iter(|| {
                if let Err(err) = rt.block_on(async {
-                    let ctx = kcl_lib::ExecutorContext::new_with_default_client(Default::default()).await?;
+                    let ctx = kcl_lib::ExecutorContext::new_with_default_client().await?;
                    let mut exec_state = kcl_lib::ExecState::new(&ctx);
                    ctx.run(black_box(&program), &mut exec_state).await?;
                    ctx.close().await;
--- a/rust/kcl-lib/e2e/executor/cache.rs
+++ b/rust/kcl-lib/e2e/executor/cache.rs
@ -51,45 +51,6 @@ async fn cache_test(
    img_results
 }

-#[tokio::test(flavor = "multi_thread")]
-async fn kcl_test_cache_change_units_changes_output() {
-    let code = r#"part001 = startSketchOn('XY')
-  |> startProfileAt([5.5229, 5.25217], %)
-  |> line(end = [10.50433, -1.19122])
-  |> line(end = [8.01362, -5.48731])
-  |> line(end = [-1.02877, -6.76825])
-  |> line(end = [-11.53311, 2.81559])
-  |> close()
-  |> extrude(length = 4)
-"#;
-
-    let result = cache_test(
-        "change_units_changes_output",
-        vec![
-            Variation {
-                code,
-                settings: &kcl_lib::ExecutorSettings {
-                    units: kcl_lib::UnitLength::In,
-                    ..Default::default()
-                },
-            },
-            Variation {
-                code,
-                settings: &kcl_lib::ExecutorSettings {
-                    units: kcl_lib::UnitLength::Mm,
-                    ..Default::default()
-                },
-            },
-        ],
-    )
-    .await;
-
-    let first = result.first().unwrap();
-    let second = result.last().unwrap();
-
-    assert!(first.1 != second.1);
-}
-
 #[tokio::test(flavor = "multi_thread")]
 async fn kcl_test_cache_change_grid_visualizes_grid_off_to_on() {
    let code = r#"part001 = startSketchOn('XY')
@ -261,9 +222,7 @@ async fn kcl_test_cache_empty_file_pop_cache_empty_file_planes_work() {
    // Get the current working directory.
    let code = "";

-    let ctx = kcl_lib::ExecutorContext::new_with_default_client(Default::default())
-        .await
-        .unwrap();
+    let ctx = kcl_lib::ExecutorContext::new_with_default_client().await.unwrap();
    let program = kcl_lib::Program::parse_no_errs(code).unwrap();
    let outcome = ctx.run_with_caching(program).await.unwrap();

--- a/rust/kcl-lib/e2e/executor/inputs/cylinder-inches.kcl
+++ b/rust/kcl-lib/e2e/executor/inputs/cylinder-inches.kcl
@ -0,0 +1,4 @@
+@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
+  |> circle(center = [0, 0], radius = 20)
+extrude001 = extrude(sketch001, length = 10)
--- a/rust/kcl-lib/e2e/executor/inputs/e2e-can-sketch-on-chamfer-no-pipeExpr.kcl
+++ b/rust/kcl-lib/e2e/executor/inputs/e2e-can-sketch-on-chamfer-no-pipeExpr.kcl
@ -1,4 +1,5 @@
-const sketch001 = startSketchOn(XZ)
+@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
  |> startProfileAt([75.8, 317.2], %)
  |> angledLine([0, 268.43], %, $rectangleSegmentA001)
  |> angledLine([
@ -11,8 +12,8 @@ const sketch001 = startSketchOn(XZ)
     ], %, $yo)
  |> line(endAbsolute = [profileStartX(%), profileStartY(%)], tag = $seg02)
  |> close()
-const extrude001 = extrude(sketch001, length = 100)
-const chamf = chamfer(
+extrude001 = extrude(sketch001, length = 100)
+chamf = chamfer(
  extrude001,
  length = 30,
  tags = [
--- a/rust/kcl-lib/e2e/executor/inputs/e2e-can-sketch-on-chamfer.kcl
+++ b/rust/kcl-lib/e2e/executor/inputs/e2e-can-sketch-on-chamfer.kcl
@ -1,4 +1,5 @@
-const sketch001 = startSketchOn(XZ)
+@settings(defaultLengthUnit = in)
+sketch001 = startSketchOn(XZ)
  |> startProfileAt([75.8, 317.2], %) // [$startCapTag, $EndCapTag]
  |> angledLine([0, 268.43], %, $rectangleSegmentA001)
  |> angledLine([
@ -11,7 +12,7 @@ const sketch001 = startSketchOn(XZ)
     ], %, $yo)
  |> line(endAbsolute = [profileStartX(%), profileStartY(%)], tag = $seg02)
  |> close()
-const extrude001 = extrude(sketch001, length = 100)
+extrude001 = extrude(sketch001, length = 100)
  |> chamfer(
       length = 30,
       tags = [
--- a/rust/kcl-lib/e2e/executor/main.rs
+++ b/rust/kcl-lib/e2e/executor/main.rs
@ -2,7 +2,7 @@ mod cache;

 use kcl_lib::{
    test_server::{execute_and_export_step, execute_and_snapshot, execute_and_snapshot_no_auth},
-    ExecError, UnitLength,
+    ExecError,
 };

 /// The minimum permissible difference between asserted twenty-twenty images.
@ -26,7 +26,7 @@ pub(crate) fn assert_out(test_name: &str, result: &image::DynamicImage) -> Strin
 async fn kcl_test_fillet_duplicate_tags() {
    let code = kcl_input!("fillet_duplicate_tags");

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    let err = result.expect_err("Code should have failed due to the duplicate edges being filletted");

    let err = err.as_kcl_error().unwrap();
@ -48,7 +48,7 @@ async fn kcl_test_execute_engine_error_return() {
  |> extrude(length = 4)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    assert!(result.is_err());
    assert_eq!(
        result.err().unwrap().to_string(),
@ -61,7 +61,7 @@ async fn kcl_test_execute_i_shape() {
    // This is some code from lee that starts a pipe expression with a variable.
    let code = kcl_input!("i_shape");

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("i_shape", &result);
 }

@ -70,7 +70,7 @@ async fn kcl_test_execute_i_shape() {
 async fn kcl_test_execute_pipes_on_pipes() {
    let code = kcl_input!("pipes_on_pipes");

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("pipes_on_pipes", &result);
 }

@ -78,7 +78,7 @@ async fn kcl_test_execute_pipes_on_pipes() {
 async fn kcl_test_execute_cylinder() {
    let code = kcl_input!("cylinder");

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("cylinder", &result);
 }

@ -86,7 +86,7 @@ async fn kcl_test_execute_cylinder() {
 async fn kcl_test_execute_kittycad_svg() {
    let code = kcl_input!("kittycad_svg");

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("kittycad_svg", &result);
 }

@ -94,7 +94,7 @@ async fn kcl_test_execute_kittycad_svg() {
 async fn kcl_test_execute_lsystem() {
    let code = kcl_input!("lsystem");

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("lsystem", &result);
 }

@ -102,7 +102,7 @@ async fn kcl_test_execute_lsystem() {
 async fn kcl_test_member_expression_sketch() {
    let code = kcl_input!("member_expression_sketch");

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("member_expression_sketch", &result);
 }

@ -110,7 +110,7 @@ async fn kcl_test_member_expression_sketch() {
 async fn kcl_test_helix_defaults() {
    let code = kcl_input!("helix_defaults");

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("helix_defaults", &result);
 }

@ -118,7 +118,7 @@ async fn kcl_test_helix_defaults() {
 async fn kcl_test_helix_defaults_negative_extrude() {
    let code = kcl_input!("helix_defaults_negative_extrude");

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("helix_defaults_negative_extrude", &result);
 }

@ -126,7 +126,7 @@ async fn kcl_test_helix_defaults_negative_extrude() {
 async fn kcl_test_helix_with_length() {
    let code = kcl_input!("helix_with_length");

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("helix_with_length", &result);
 }

@ -134,7 +134,7 @@ async fn kcl_test_helix_with_length() {
 async fn kcl_test_dimensions_match() {
    let code = kcl_input!("dimensions_match");

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("dimensions_match", &result);
 }

@ -142,7 +142,7 @@ async fn kcl_test_dimensions_match() {
 async fn kcl_test_close_arc() {
    let code = kcl_input!("close_arc");

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("close_arc", &result);
 }

@ -150,7 +150,7 @@ async fn kcl_test_close_arc() {
 async fn kcl_test_negative_args() {
    let code = kcl_input!("negative_args");

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("negative_args", &result);
 }

@ -164,7 +164,7 @@ async fn kcl_test_basic_tangential_arc_with_point() {
    |> extrude(length = 10)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("tangential_arc_with_point", &result);
 }

@ -178,7 +178,7 @@ async fn kcl_test_basic_tangential_arc_to() {
    |> extrude(length = 10)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("tangential_arc_to", &result);
 }

@ -205,7 +205,7 @@ box(30, 43, 18, '-xy')
 let thing = box(-12, -15, 10, 'yz')
 box(-20, -5, 10, 'xy')"#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("different_planes_same_drawing", &result);
 }

@ -263,7 +263,7 @@ part004 = startSketchOn(YZ)
  |> close()
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("lots_of_planes", &result);
 }

@ -280,7 +280,7 @@ async fn kcl_test_holes() {
  |> extrude(length = 2)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("holes", &result);
 }

@ -299,7 +299,7 @@ async fn optional_params() {

 thing = other_circle([2, 2], 20)
 "#;
-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("optional_params", &result);
 }

@ -335,7 +335,7 @@ part = roundedRectangle([0, 0], 20, 20, 4)
  |> extrude(length = 2)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("rounded_with_holes", &result);
 }

@ -343,7 +343,7 @@ part = roundedRectangle([0, 0], 20, 20, 4)
 async fn kcl_test_top_level_expression() {
    let code = r#"startSketchOn(XY) |> circle(center = [0,0], radius= 22) |> extrude(length = 14)"#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("top_level_expression", &result);
 }

@ -357,7 +357,7 @@ part =  startSketchOn(XY)
    |> extrude(length = 1)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("patterns_linear_basic_with_math", &result);
 }

@ -369,7 +369,7 @@ async fn kcl_test_patterns_linear_basic() {
    |> extrude(length = 1)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("patterns_linear_basic", &result);
 }

@ -385,7 +385,7 @@ async fn kcl_test_patterns_linear_basic_3d() {
    |> patternLinear3d(axis = [1, 0, 1], instances = 4, distance = 6)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("patterns_linear_basic_3d", &result);
 }

@ -397,7 +397,7 @@ async fn kcl_test_patterns_linear_basic_negative_distance() {
    |> extrude(length = 1)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("patterns_linear_basic_negative_distance", &result);
 }

@ -409,7 +409,7 @@ async fn kcl_test_patterns_linear_basic_negative_axis() {
    |> extrude(length = 1)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("patterns_linear_basic_negative_axis", &result);
 }

@ -430,7 +430,7 @@ rectangle = startSketchOn(XY)

 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("patterns_linear_basic_holes", &result);
 }

@ -442,7 +442,7 @@ async fn kcl_test_patterns_circular_basic_2d() {
    |> extrude(length = 1)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("patterns_circular_basic_2d", &result);
 }

@ -458,7 +458,7 @@ async fn kcl_test_patterns_circular_basic_3d() {
    |> patternCircular3d(axis = [0,0, 1], center = [-20, -20, -20], instances = 41, arcDegrees = 360, rotateDuplicates = false)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("patterns_circular_basic_3d", &result);
 }

@ -474,7 +474,7 @@ async fn kcl_test_patterns_circular_3d_tilted_axis() {
    |> patternCircular3d(axis = [1,1,0], center = [10, 0, 10], instances = 11, arcDegrees = 360, rotateDuplicates = true)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("patterns_circular_3d_tilted_axis", &result);
 }

@ -483,7 +483,7 @@ async fn kcl_test_import_file_doesnt_exist() {
    let code = r#"import 'thing.obj'
 model = cube"#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    assert!(result.is_err());
    assert_eq!(
        result.err().unwrap().to_string(),
@ -496,7 +496,7 @@ async fn kcl_test_import_obj_with_mtl() {
    let code = r#"import 'e2e/executor/inputs/cube.obj'
 model = cube"#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("import_obj_with_mtl", &result);
 }

@ -506,7 +506,7 @@ async fn kcl_test_import_obj_with_mtl_units() {
 import 'e2e/executor/inputs/cube.obj'
 model = cube"#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("import_obj_with_mtl_units", &result);
 }

@ -515,7 +515,7 @@ async fn kcl_test_import_stl() {
    let code = r#"import 'e2e/executor/inputs/2-5-long-m8-chc-screw.stl' as screw
 model = screw"#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("import_stl", &result);
 }

@ -524,7 +524,7 @@ async fn kcl_test_import_gltf_with_bin() {
    let code = r#"import 'e2e/executor/inputs/cube.gltf'
 model = cube"#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("import_gltf_with_bin", &result);
 }

@ -533,7 +533,7 @@ async fn kcl_test_import_gltf_embedded() {
    let code = r#"import 'e2e/executor/inputs/cube-embedded.gltf' as cube
 model = cube"#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("import_gltf_embedded", &result);
 }

@ -542,7 +542,7 @@ async fn kcl_test_import_glb() {
    let code = r#"import 'e2e/executor/inputs/cube.glb'
 model = cube"#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("import_glb", &result);
 }

@ -551,7 +551,7 @@ async fn kcl_test_import_glb_no_assign() {
    let code = r#"import 'e2e/executor/inputs/cube.glb'
 cube"#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("import_glb_no_assign", &result);
 }

@ -561,7 +561,7 @@ async fn kcl_test_import_ext_doesnt_match() {
 import 'e2e/executor/inputs/cube.gltf'
 model = cube"#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    assert!(result.is_err());
    assert_eq!(
        result.err().unwrap().to_string(),
@ -586,14 +586,15 @@ async fn kcl_test_cube_mm() {
 myCube = cube([0,0], 10)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("cube_mm", &result);
 }

 #[tokio::test(flavor = "multi_thread")]
 async fn kcl_test_cube_cm() {
-    let code = r#"fn cube = (pos, scale) => {
-  sg = startSketchOn(XY)
+    let code = r#"@settings(defaultLengthUnit = cm)
+fn cube = (pos, scale) => {
+  sg = startSketchOn('XY')
    |> startProfileAt(pos, %)
    |> line(end = [0, scale])
    |> line(end = [scale, 0])
@ -607,14 +608,15 @@ async fn kcl_test_cube_cm() {
 myCube = cube([0,0], 10)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Cm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("cube_cm", &result);
 }

 #[tokio::test(flavor = "multi_thread")]
 async fn kcl_test_cube_m() {
-    let code = r#"fn cube = (pos, scale) => {
-  sg = startSketchOn(XY)
+    let code = r#"@settings(defaultLengthUnit = m)
+fn cube = (pos, scale) => {
+  sg = startSketchOn('XY')
    |> startProfileAt(pos, %)
    |> line(end = [0, scale])
    |> line(end = [scale, 0])
@ -628,14 +630,15 @@ async fn kcl_test_cube_m() {
 myCube = cube([0,0], 10)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::M, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("cube_m", &result);
 }

 #[tokio::test(flavor = "multi_thread")]
 async fn kcl_test_cube_in() {
-    let code = r#"fn cube = (pos, scale) => {
-  sg = startSketchOn(XY)
+    let code = r#"@settings(defaultLengthUnit = in)
+fn cube = (pos, scale) => {
+  sg = startSketchOn('XY')
    |> startProfileAt(pos, %)
    |> line(end = [0, scale])
    |> line(end = [scale, 0])
@ -649,14 +652,15 @@ async fn kcl_test_cube_in() {
 myCube = cube([0,0], 10)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::In, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("cube_in", &result);
 }

 #[tokio::test(flavor = "multi_thread")]
 async fn kcl_test_cube_ft() {
-    let code = r#"fn cube = (pos, scale) => {
-  sg = startSketchOn(XY)
+    let code = r#"@settings(defaultLengthUnit = ft)
+fn cube = (pos, scale) => {
+  sg = startSketchOn('XY')
    |> startProfileAt(pos, %)
    |> line(end = [0, scale])
    |> line(end = [scale, 0])
@ -670,14 +674,15 @@ async fn kcl_test_cube_ft() {
 myCube = cube([0,0], 10)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Ft, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("cube_ft", &result);
 }

 #[tokio::test(flavor = "multi_thread")]
 async fn kcl_test_cube_yd() {
-    let code = r#"fn cube = (pos, scale) => {
-  sg = startSketchOn(XY)
+    let code = r#"@settings(defaultLengthUnit = yd)
+fn cube = (pos, scale) => {
+  sg = startSketchOn('XY')
    |> startProfileAt(pos, %)
    |> line(end = [0, scale])
    |> line(end = [scale, 0])
@ -691,7 +696,7 @@ async fn kcl_test_cube_yd() {
 myCube = cube([0,0], 10)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Yd, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("cube_yd", &result);
 }

@ -719,7 +724,7 @@ part002 = startSketchOn(part001, part001.sketch.tags.here)
  |> extrude(length = 1)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;

    let err = result.err().unwrap();
    let ExecError::Kcl(err) = err else {
@ -763,7 +768,7 @@ part003 = startSketchOn(part002, "end")
  |> extrude(length = 5)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("sketch_on_face_of_face", &result);
 }

@ -780,7 +785,7 @@ async fn kcl_test_stdlib_kcl_error_right_code_path() {
  |> extrude(length = 2)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    let err = result.err().unwrap();
    let ExecError::Kcl(err) = err else {
        panic!("Expected KCL error, found {err}");
@ -811,7 +816,7 @@ part002 = startSketchOn(part001, "end")
  |> extrude(length = 5)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("sketch_on_face_circle", &result);
 }

@ -853,7 +858,7 @@ part = rectShape([0, 0], 20, 20)
     )
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    let err = result.err().unwrap();
    let ExecError::Kcl(err) = err else {
        panic!("Expected KCL error, found {err}");
@ -880,7 +885,7 @@ async fn kcl_test_simple_revolve() {

 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("simple_revolve", &result);
 }

@ -900,7 +905,7 @@ async fn kcl_test_simple_revolve_uppercase() {

 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("simple_revolve_uppercase", &result);
 }

@ -920,7 +925,7 @@ async fn kcl_test_simple_revolve_negative() {

 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("simple_revolve_negative", &result);
 }

@ -940,7 +945,7 @@ async fn kcl_test_revolve_bad_angle_low() {

 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;

    assert!(result.is_err());
    assert!(result
@ -966,7 +971,7 @@ async fn kcl_test_revolve_bad_angle_high() {

 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;

    assert!(result.is_err());
    assert!(result
@ -992,7 +997,7 @@ async fn kcl_test_simple_revolve_custom_angle() {

 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("simple_revolve_custom_angle", &result);
 }

@ -1012,7 +1017,7 @@ async fn kcl_test_simple_revolve_custom_axis() {

 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("simple_revolve_custom_axis", &result);
 }

@ -1036,7 +1041,7 @@ sketch001 = startSketchOn(box, "end")

 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("revolve_on_edge", &result);
 }

@ -1060,7 +1065,7 @@ sketch001 = startSketchOn(box, revolveAxis)

 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;

    result.unwrap_err();
    //this fails right now, but slightly differently, lets just say its enough for it to fail - mike
@ -1088,7 +1093,7 @@ sketch001 = startSketchOn(box, "END")
    )
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("revolve_on_face_circle_edge", &result);
 }

@ -1110,7 +1115,7 @@ sketch001 = startSketchOn(box, "END")
    )
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("revolve_on_face_circle", &result);
 }

@ -1136,7 +1141,7 @@ sketch001 = startSketchOn(box, "end")
  )
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("revolve_on_face", &result);
 }

@ -1150,7 +1155,7 @@ async fn kcl_test_basic_revolve_circle() {
    )
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("basic_revolve_circle", &result);
 }

@ -1177,7 +1182,7 @@ part002 = startSketchOn(part001, 'end')
    |> extrude(length = 5)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("simple_revolve_sketch_on_edge", &result);
 }

@ -1240,7 +1245,7 @@ plumbus1 = circle1
      )
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("plumbus_fillets", &result);
 }

@ -1248,7 +1253,7 @@ plumbus1 = circle1
 async fn kcl_test_empty_file_is_ok() {
    let code = r#""#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    result.unwrap();
 }

@ -1278,7 +1283,7 @@ async fn kcl_test_member_expression_in_params() {
 capScrew([0, 0.5, 0], 50, 37.5, 50, 25)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("member_expression_in_params", &result);
 }

@ -1323,7 +1328,7 @@ bracket = startSketchOn(XY)
     )
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    result.unwrap();
 }

@ -1343,7 +1348,7 @@ secondSketch = startSketchOn(part001, '')
  |> extrude(length = 20)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    assert!(result.is_err());
    assert_eq!(
        result.err().unwrap().to_string(),
@ -1374,7 +1379,7 @@ extrusion = startSketchOn(XY)
  |> extrude(length = height)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    assert!(result.is_err());
    assert_eq!(
        result.err().unwrap().to_string(),
@ -1407,7 +1412,7 @@ sketch001 = [profile001, profile002]
 extrude(sketch001, length = 10)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("array_of_sketches", &result);
 }

@ -1444,7 +1449,7 @@ pattn1 = patternLinear3d(
     )
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("pattern3d_array_of_extrudes", &result);
 }

@ -1492,7 +1497,7 @@ baseExtrusion = extrude(sketch001, length = width)
 )
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("fillets_referencing_other_fillets", &result);
 }

@ -1540,7 +1545,7 @@ baseExtrusion = extrude(sketch001, length = width)
   )
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("chamfers_referencing_other_chamfers", &result);
 }

@ -1560,7 +1565,7 @@ async fn kcl_test_shell_with_tag() {
  )
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("shell_with_tag", &result);
 }

@ -1591,7 +1596,7 @@ pattn1 = patternLinear3d(
 )
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("linear_pattern3d_filleted_sketch", &result);
 }

@ -1618,7 +1623,7 @@ pattn2 = patternCircular3d(part001, axis = [0,0, 1], center = [-20, -20, -20], i

 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("circular_pattern3d_filleted_sketch", &result);
 }

@ -1644,7 +1649,7 @@ part001 = cube([0,0], 20)
 pattn2 = patternCircular3d(part001, axis = [0,0, 1], center = [-20, -20, -20], instances = 5, arcDegrees = 360, rotateDuplicates = false)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("circular_pattern3d_chamfered_sketch", &result);
 }

@ -1671,7 +1676,7 @@ part001 = cube([0,0], 20)

 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    let err = result.err().unwrap();
    let ExecError::Kcl(err) = err else {
        panic!("Expected KCL error, found {err}");
@ -1701,7 +1706,7 @@ async fn kcl_test_duplicate_tags_should_error() {
 let p = triangle(200)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    assert!(result.is_err());
    assert_eq!(
        result.err().unwrap().to_string(),
@ -1712,49 +1717,49 @@ let p = triangle(200)
 #[tokio::test(flavor = "multi_thread")]
 async fn kcl_test_global_tags() {
    let code = kcl_input!("global-tags");
-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("global_tags", &result);
 }

 #[tokio::test(flavor = "multi_thread")]
 async fn kcl_test_extrude_inside_fn_with_tags() {
    let code = kcl_input!("extrude-inside-fn-with-tags");
-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("extrude-inside-fn-with-tags", &result);
 }

 #[tokio::test(flavor = "multi_thread")]
 async fn kcl_test_pattern_vase() {
    let code = kcl_input!("pattern_vase");
-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("pattern_vase", &result);
 }

 #[tokio::test(flavor = "multi_thread")]
 async fn kcl_test_scoped_tags() {
    let code = kcl_input!("scoped-tags");
-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("scoped_tags", &result);
 }

 #[tokio::test(flavor = "multi_thread")]
 async fn kcl_test_order_sketch_extrude_in_order() {
    let code = kcl_input!("order-sketch-extrude-in-order");
-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("order-sketch-extrude-in-order", &result);
 }

 #[tokio::test(flavor = "multi_thread")]
 async fn kcl_test_order_sketch_extrude_out_of_order() {
    let code = kcl_input!("order-sketch-extrude-out-of-order");
-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("order-sketch-extrude-out-of-order", &result);
 }

 #[tokio::test(flavor = "multi_thread")]
 async fn kcl_test_extrude_custom_plane() {
    let code = kcl_input!("extrude-custom-plane");
-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await.unwrap();
+    let result = execute_and_snapshot(code, None).await.unwrap();
    assert_out("extrude-custom-plane", &result);
 }

@ -1776,7 +1781,7 @@ async fn kcl_test_arc_error_same_start_end() {
     )
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    assert!(result.is_err());
    assert_eq!(
        result.err().unwrap().to_string(),
@ -1796,7 +1801,7 @@ async fn kcl_test_angled_line_to_x_90() {
 example = extrude(exampleSketch, length = 10)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    assert!(result.is_err());
    assert_eq!(
        result.err().unwrap().to_string(),
@ -1816,7 +1821,7 @@ async fn kcl_test_angled_line_to_x_270() {
 example = extrude(exampleSketch, length = 10)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    assert!(result.is_err());
    assert_eq!(
        result.err().unwrap().to_string(),
@ -1836,7 +1841,7 @@ async fn kcl_test_angled_line_to_y_0() {
 example = extrude(exampleSketch, length = 10)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    assert!(result.is_err());
    assert_eq!(
        result.err().unwrap().to_string(),
@ -1856,7 +1861,7 @@ async fn kcl_test_angled_line_to_y_180() {
 example = extrude(exampleSketch, length = 10)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    assert!(result.is_err());
    assert_eq!(
        result.err().unwrap().to_string(),
@ -1876,7 +1881,7 @@ async fn kcl_test_angled_line_of_x_length_90() {
 extrusion = extrude(sketch001, length = 10)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    assert!(result.is_err());
    assert_eq!(
        result.err().unwrap().to_string(),
@ -1896,7 +1901,7 @@ async fn kcl_test_angled_line_of_x_length_270() {
 extrusion = extrude(sketch001, length = 10)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    assert!(result.is_err());
    assert_eq!(
        result.err().unwrap().to_string(),
@ -1918,7 +1923,7 @@ async fn kcl_test_angled_line_of_y_length_0() {
 example = extrude(exampleSketch, length = 10)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    assert!(result.is_err());
    assert_eq!(
        result.err().unwrap().to_string(),
@ -1940,7 +1945,7 @@ async fn kcl_test_angled_line_of_y_length_180() {
 example = extrude(exampleSketch, length = 10)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    assert!(result.is_err());
    assert_eq!(
        result.err().unwrap().to_string(),
@ -1962,7 +1967,7 @@ async fn kcl_test_angled_line_of_y_length_negative_180() {
 example = extrude(exampleSketch, length = 10)
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    assert!(result.is_err());
    assert_eq!(
        result.err().unwrap().to_string(),
@ -1979,7 +1984,7 @@ async fn kcl_test_error_inside_fn_also_has_source_range_of_call_site() {
 someFunction('INVALID')
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    assert!(result.is_err());
    assert_eq!(
        result.err().unwrap().to_string(),
@ -2000,7 +2005,7 @@ async fn kcl_test_error_inside_fn_also_has_source_range_of_call_site_recursive()
 someFunction('INVALID')
 "#;

-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;
    assert!(result.is_err());
    assert_eq!(
        result.err().unwrap().to_string(),
@ -2024,7 +2029,7 @@ async fn kcl_test_error_no_auth_websocket() {
  )
 "#;

-    let result = execute_and_snapshot_no_auth(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot_no_auth(code, None).await;
    assert!(result.is_err());
    assert!(result
        .err()
@ -2050,9 +2055,7 @@ sketch000 = startSketchOn(XY)
    |> line(end = [0, innerDiameter / 2])
 "#;

-    let ctx = kcl_lib::ExecutorContext::new_with_default_client(Default::default())
-        .await
-        .unwrap();
+    let ctx = kcl_lib::ExecutorContext::new_with_default_client().await.unwrap();
    let mut exec_state = kcl_lib::ExecState::new(&ctx);
    let program = kcl_lib::Program::parse_no_errs(code).unwrap();
    ctx.run(&program, &mut exec_state).await.unwrap();
@ -2075,9 +2078,7 @@ async fn kcl_test_ensure_nothing_left_in_batch_multi_file() {
    // Change the current working directory to the test directory.
    std::env::set_current_dir(path.parent().unwrap()).unwrap();

-    let ctx = kcl_lib::ExecutorContext::new_with_default_client(Default::default())
-        .await
-        .unwrap();
+    let ctx = kcl_lib::ExecutorContext::new_with_default_client().await.unwrap();
    let mut exec_state = kcl_lib::ExecState::new(&ctx);
    let program = kcl_lib::Program::parse_no_errs(&code).unwrap();
    ctx.run(&program, &mut exec_state).await.unwrap();
@ -2095,7 +2096,7 @@ async fn kcl_test_better_type_names() {
  |> circle(center = [-95.51, -74.7], radius = 262.23)
  |> appearance(metalness = 0.9)
 "#;
-    let result = execute_and_snapshot(code, UnitLength::Mm, None).await;
+    let result = execute_and_snapshot(code, None).await;

    let err = match result.err() {
        Some(x) => match x {
@ -2114,7 +2115,7 @@ async fn kcl_test_exporting_step_file() {
    // This tests export like how we do it in cli and kcl.py.
    let code = kcl_input!("helix_defaults_negative_extrude");

-    let (_, _, files) = execute_and_export_step(code, UnitLength::Mm, None).await.unwrap();
+    let (_, _, files) = execute_and_export_step(code, None).await.unwrap();
    for file in files {
        expectorate::assert_contents(
            format!("e2e/executor/outputs/helix_defaults_negative_extrude_{}", file.name),
--- a/rust/kcl-lib/e2e/modify/main.rs
+++ b/rust/kcl-lib/e2e/modify/main.rs
@ -1,232 +0,0 @@
-use anyhow::Result;
-use kcl_lib::{
-    exec::{KclValue, PlaneType},
-    modify_ast_for_sketch, ExecState, ExecutorContext, ModuleId, Program, SourceRange,
-};
-use kittycad_modeling_cmds::{each_cmd as mcmd, length_unit::LengthUnit, shared::Point3d, ModelingCmd};
-use pretty_assertions::assert_eq;
-
-/// Setup the engine and parse code for an ast.
-async fn setup(code: &str, name: &str) -> Result<(ExecutorContext, Program, ModuleId, uuid::Uuid)> {
-    let program = Program::parse_no_errs(code)?;
-    let ctx = kcl_lib::ExecutorContext::new_with_default_client(Default::default()).await?;
-    let mut exec_state = ExecState::new(&ctx);
-    let result = ctx.run(&program, &mut exec_state).await?;
-    let outcome = exec_state.to_wasm_outcome(result.0).await;
-
-    // We need to get the sketch ID.
-    let KclValue::Sketch { value: sketch } = outcome.variables.get(name).unwrap() else {
-        anyhow::bail!("part001 not found in: {:?}", outcome.variables);
-    };
-    let sketch_id = sketch.id;
-
-    let plane_id = uuid::Uuid::new_v4();
-    ctx.engine
-        .send_modeling_cmd(
-            plane_id,
-            SourceRange::default(),
-            &ModelingCmd::from(mcmd::MakePlane {
-                clobber: false,
-                origin: Point3d::default(),
-                size: LengthUnit(60.0),
-                x_axis: Point3d { x: 1.0, y: 0.0, z: 0.0 },
-                y_axis: Point3d { x: 0.0, y: 1.0, z: 0.0 },
-                hide: Some(true),
-            }),
-        )
-        .await?;
-
-    // Enter sketch mode.
-    // We can't get control points without being in sketch mode.
-    // You can however get path info without sketch mode.
-    ctx.engine
-        .send_modeling_cmd(
-            uuid::Uuid::new_v4(),
-            SourceRange::default(),
-            &ModelingCmd::from(mcmd::EnableSketchMode {
-                animated: false,
-                ortho: true,
-                entity_id: plane_id,
-                planar_normal: Some(Point3d { x: 0.0, y: 0.0, z: 1.0 }),
-                adjust_camera: false,
-            }),
-        )
-        .await?;
-
-    Ok((ctx, program, ModuleId::default(), sketch_id))
-}
-
-#[tokio::test(flavor = "multi_thread")]
-async fn kcl_test_modify_sketch_part001() {
-    let name = "part001";
-    let code = format!(
-        r#"{} = startSketchOn(XY)
-  |> startProfileAt([8.41, 5.78], %)
-  |> line(end = [7.37, -11])
-  |> line(end = [-8.69, -3.75])
-  |> line(end = [-5, 4.25])
-"#,
-        name
-    );
-
-    let (ctx, program, module_id, sketch_id) = setup(&code, name).await.unwrap();
-    let mut new_program = program.clone();
-    let new_code = modify_ast_for_sketch(&ctx.engine, &mut new_program, module_id, name, PlaneType::XY, sketch_id)
-        .await
-        .unwrap();
-
-    // Make sure the code is the same.
-    assert_eq!(code, new_code);
-    // Make sure the program is the same.
-    assert_eq!(new_program, program);
-}
-
-#[tokio::test(flavor = "multi_thread")]
-async fn kcl_test_modify_sketch_part002() {
-    let name = "part002";
-    let code = format!(
-        r#"{} = startSketchOn(XY)
-  |> startProfileAt([8.41, 5.78], %)
-  |> line(end = [7.42, -8.62])
-  |> line(end = [-6.38, -3.51])
-  |> line(end = [-3.77, 3.56])
-"#,
-        name
-    );
-
-    let (ctx, program, module_id, sketch_id) = setup(&code, name).await.unwrap();
-    let mut new_program = program.clone();
-    let new_code = modify_ast_for_sketch(&ctx.engine, &mut new_program, module_id, name, PlaneType::XY, sketch_id)
-        .await
-        .unwrap();
-
-    // Make sure the code is the same.
-    assert_eq!(code, new_code);
-    // Make sure the program is the same.
-    assert_eq!(new_program, program);
-}
-
-#[tokio::test(flavor = "multi_thread")]
-#[ignore] // until KittyCAD/engine#1434 is fixed.
-async fn kcl_test_modify_close_sketch() {
-    let name = "part002";
-    let code = format!(
-        r#"{} = startSketchOn(XY)
-  |> startProfileAt([7.91, 3.89], %)
-  |> line(end = [7.42, -8.62])
-  |> line(end = [-6.38, -3.51])
-  |> line(end = [-3.77, 3.56])
-  |> close()
-"#,
-        name
-    );
-
-    let (ctx, program, module_id, sketch_id) = setup(&code, name).await.unwrap();
-    let mut new_program = program.clone();
-    let new_code = modify_ast_for_sketch(&ctx.engine, &mut new_program, module_id, name, PlaneType::XY, sketch_id)
-        .await
-        .unwrap();
-
-    // Make sure the code is the same.
-    assert_eq!(code, new_code);
-    // Make sure the program is the same.
-    assert_eq!(new_program, program);
-}
-
-#[tokio::test(flavor = "multi_thread")]
-async fn kcl_test_modify_line_to_close_sketch() {
-    let name = "part002";
-    let code = format!(
-        r#"const {} = startSketchOn(XY)
-  |> startProfileAt([7.91, 3.89], %)
-  |> line(end = [7.42, -8.62])
-  |> line(end = [-6.38, -3.51])
-  |> line(end = [-3.77, 3.56])
-  |> line(endAbsolute = [7.91, 3.89])
-"#,
-        name
-    );
-
-    let (ctx, program, module_id, sketch_id) = setup(&code, name).await.unwrap();
-    let mut new_program = program.clone();
-    let new_code = modify_ast_for_sketch(&ctx.engine, &mut new_program, module_id, name, PlaneType::XY, sketch_id)
-        .await
-        .unwrap();
-
-    // Make sure the code is the same.
-    assert_eq!(
-        new_code,
-        format!(
-            r#"{} = startSketchOn(XY)
-  |> startProfileAt([7.91, 3.89], %)
-  |> line(end = [7.42, -8.62])
-  |> line(end = [-6.38, -3.51])
-  |> line(end = [-3.77, 3.56])
-  |> close()
-"#,
-            name
-        )
-    );
-}
-
-#[tokio::test(flavor = "multi_thread")]
-async fn kcl_test_modify_with_constraint() {
-    let name = "part002";
-    let code = format!(
-        r#"const thing = 12
-const {} = startSketchOn(XY)
-  |> startProfileAt([7.91, 3.89], %)
-  |> line(end = [7.42, -8.62])
-  |> line(end = [-6.38, -3.51])
-  |> line(end = [-3.77, 3.56])
-  |> line(endAbsolute = [thing, 3.89])
-"#,
-        name
-    );
-
-    let (ctx, program, module_id, sketch_id) = setup(&code, name).await.unwrap();
-    let mut new_program = program.clone();
-    let result = modify_ast_for_sketch(&ctx.engine, &mut new_program, module_id, name, PlaneType::XY, sketch_id).await;
-
-    assert!(result.is_err());
-    assert_eq!(
-        result.unwrap_err().message(),
-        "Sketch part002 is constrained `partial` and cannot be modified",
-    );
-}
-
-#[tokio::test(flavor = "multi_thread")]
-async fn kcl_test_modify_line_should_close_sketch() {
-    let name = "part003";
-    let code = format!(
-        r#"const {} = startSketchOn(XY)
-  |> startProfileAt([13.69, 3.8], %)
-  |> line(end = [4.23, -11.79])
-  |> line(end = [-10.7, -1.16])
-  |> line(end = [-3.72, 8.69])
-  |> line(end = [10.19, 4.26])
-"#,
-        name
-    );
-
-    let (ctx, program, module_id, sketch_id) = setup(&code, name).await.unwrap();
-    let mut new_program = program.clone();
-    let new_code = modify_ast_for_sketch(&ctx.engine, &mut new_program, module_id, name, PlaneType::XY, sketch_id)
-        .await
-        .unwrap();
-
-    // Make sure the code is the same.
-    assert_eq!(
-        new_code,
-        format!(
-            r#"{} = startSketchOn(XY)
-  |> startProfileAt([13.69, 3.8], %)
-  |> line(end = [4.23, -11.79])
-  |> line(end = [-10.7, -1.16])
-  |> line(end = [-3.72, 8.69])
-  |> close()
-"#,
-            name
-        )
-    );
-}
--- a/rust/kcl-lib/src/docs/gen_std_tests.rs
+++ b/rust/kcl-lib/src/docs/gen_std_tests.rs
@ -10,6 +10,7 @@ use convert_case::Casing;
 use handlebars::Renderable;
 use indexmap::IndexMap;
 use itertools::Itertools;
+use schemars::schema::SingleOrVec;
 use serde_json::json;
 use tokio::task::JoinSet;

@ -673,6 +674,7 @@ fn cleanup_type_links<'a>(output: &str, types: impl Iterator<Item = &'a String>)
    // TODO: This is a hack for the handlebars template being too complex.
    cleaned_output = cleaned_output.replace("`[, `number`, `number`]`", "`[number, number]`");
    cleaned_output = cleaned_output.replace("`[, `number`, `number`, `number`]`", "`[number, number, number]`");
+    cleaned_output = cleaned_output.replace("`[, `integer`, `integer`, `integer`]`", "`[integer, integer, integer]`");

    // Fix the links to the types.
    for type_name in types.map(|s| &**s).chain(DECLARED_TYPES) {
@ -691,7 +693,7 @@ fn cleanup_type_links<'a>(output: &str, types: impl Iterator<Item = &'a String>)
    // TODO handle union types generically rather than special casing them.
    cleaned_output = cleaned_output.replace(
        "`Sketch | Plane | Face`",
-        "[`Sketch`](/docs/kcl/types/Sketch) `|` [`Plane`](/docs/kcl/types/Face) `|` [`Plane`](/docs/kcl/types/Face)",
+        "[`Sketch`](/docs/kcl/types/Sketch) OR [`Plane`](/docs/kcl/types/Plane) OR [`Face`](/docs/kcl/types/Face)",
    );

    cleanup_static_links(&cleaned_output)
@ -720,6 +722,11 @@ fn add_to_types(
        ));
    };

+    if name == "SourceRange" {
+        types.insert(name.to_string(), schema.clone());
+        return Ok(());
+    }
+
    // If we have an array we want to generate the type markdown files for each item in the
    // array.
    if let Some(array) = &o.array {
@ -727,7 +734,7 @@ fn add_to_types(
        if let Some(items) = &array.items {
            match items {
                schemars::schema::SingleOrVec::Single(item) => {
-                    if is_primitive(item)?.is_some() && name != "SourceRange" {
+                    if is_primitive(item)?.is_some() {
                        return Ok(());
                    }
                    return add_to_types(name.trim_start_matches('[').trim_end_matches(']'), item, types);
@ -816,9 +823,8 @@ fn generate_type(
    }

    let cleaned_schema = recurse_and_create_references(name, schema, types)?;
-    let new_schema = super::cleanup_number_tuples(&cleaned_schema);

-    let schemars::schema::Schema::Object(o) = new_schema else {
+    let schemars::schema::Schema::Object(o) = cleaned_schema else {
        return Err(anyhow::anyhow!(
            "Failed to get object schema, should have not been a primitive"
        ));
@ -950,6 +956,13 @@ fn recurse_and_create_references(
                "Failed to get object schema, should have not been a primitive"
            ));
        };
+
+        // If this is a primitive just use the primitive.
+        if to.subschemas.is_none() && to.object.is_none() && to.reference.is_none() {
+            return Ok(t.clone());
+        }
+
+        // Otherwise append the metadata to our reference.
        if let Some(metadata) = obj.metadata.as_mut() {
            if metadata.description.is_none() {
                metadata.description = to.metadata.as_ref().and_then(|m| m.description.clone());
@ -1052,6 +1065,13 @@ fn recurse_and_create_references(
                *item = new_item;
            }
        }
+
+        if subschema.one_of.is_none() && subschema.all_of.is_none() && subschema.any_of.is_none() && obj.array.is_none()
+        {
+            if let Some(SingleOrVec::Single(_)) = &o.instance_type {
+                return Ok(schema.clone());
+            }
+        }
    }

    let obj = cleanup_type_description(&obj)
@ -1172,7 +1192,7 @@ fn find_examples(text: &str, filename: &str) -> Vec<(String, String)> {

 async fn run_example(text: &str) -> Result<()> {
    let program = crate::Program::parse_no_errs(text)?;
-    let ctx = ExecutorContext::new_with_default_client(crate::UnitLength::Mm).await?;
+    let ctx = ExecutorContext::new_with_default_client().await?;
    let mut exec_state = crate::execution::ExecState::new(&ctx);
    ctx.run(&program, &mut exec_state).await?;
    Ok(())
--- a/rust/kcl-lib/src/docs/kcl_doc.rs
+++ b/rust/kcl-lib/src/docs/kcl_doc.rs
@ -1010,20 +1010,17 @@ mod test {
        let std = walk_prelude();
        for d in std {
            for (i, eg) in d.examples().enumerate() {
-                let result =
-                    match crate::test_server::execute_and_snapshot(eg, crate::settings::types::UnitLength::Mm, None)
-                        .await
-                    {
-                        Err(crate::errors::ExecError::Kcl(e)) => {
-                            return Err(miette::Report::new(crate::errors::Report {
-                                error: e.error,
-                                filename: format!("{}{i}", d.name()),
-                                kcl_source: eg.to_string(),
-                            }));
-                        }
-                        Err(other_err) => panic!("{}", other_err),
-                        Ok(img) => img,
-                    };
+                let result = match crate::test_server::execute_and_snapshot(eg, None).await {
+                    Err(crate::errors::ExecError::Kcl(e)) => {
+                        return Err(miette::Report::new(crate::errors::Report {
+                            error: e.error,
+                            filename: format!("{}{i}", d.name()),
+                            kcl_source: eg.to_string(),
+                        }));
+                    }
+                    Err(other_err) => panic!("{}", other_err),
+                    Ok(img) => img,
+                };
                twenty_twenty::assert_image(
                    format!("tests/outputs/serial_test_example_{}{i}.png", d.example_name()),
                    &result,
--- a/rust/kcl-lib/src/docs/mod.rs
+++ b/rust/kcl-lib/src/docs/mod.rs
@ -660,69 +660,6 @@ pub fn get_description_string_from_schema(schema: &schemars::schema::RootSchema)
    None
 }

-pub fn cleanup_number_tuples_root(mut schema: schemars::schema::RootSchema) -> schemars::schema::RootSchema {
-    cleanup_number_tuples_object(&mut schema.schema);
-    schema
-}
-
-fn cleanup_number_tuples_object(o: &mut schemars::schema::SchemaObject) {
-    if let Some(object) = &mut o.object {
-        for (_, value) in object.properties.iter_mut() {
-            *value = cleanup_number_tuples(value);
-        }
-    }
-
-    if let Some(array) = &mut o.array {
-        if let Some(items) = &mut array.items {
-            match items {
-                schemars::schema::SingleOrVec::Single(_) => {
-                    // Do nothing since its only a single item.
-                }
-                schemars::schema::SingleOrVec::Vec(items) => {
-                    if items.len() == 2 {
-                        // Get the second item and see if its a NumericType.
-
-                        if let Some(schemars::schema::Schema::Object(obj)) = items.get(1) {
-                            if let Some(reference) = &obj.reference {
-                                if reference == "#/components/schemas/NumericType" {
-                                    // Get the first item.
-                                    if let Some(schemars::schema::Schema::Object(obj2)) = items.first() {
-                                        let mut obj2 = obj2.clone();
-                                        obj2.metadata = o.metadata.clone();
-                                        // Replace the array with the first item.
-                                        *o = obj2;
-                                    }
-                                }
-                            } else if NUMERIC_TYPE_SCHEMA.object == obj.object {
-                                if let Some(schemars::schema::Schema::Object(obj2)) = items.first() {
-                                    let mut obj2 = obj2.clone();
-                                    obj2.metadata = o.metadata.clone();
-                                    // Replace the array with the first item.
-                                    *o = obj2;
-                                }
-                            }
-                        }
-                    }
-                }
-            }
-        }
-    }
-}
-
-/// Some numbers will be tuples of 2 where the second type is always "NumericType". We want to
-/// replace these with the first item in the array and not have an array as it messes
-/// with the docs generation which assumes if there is a tuple that you give 2 values not one
-/// in the form of an array.
-fn cleanup_number_tuples(schema: &schemars::schema::Schema) -> schemars::schema::Schema {
-    let mut schema = schema.clone();
-
-    if let schemars::schema::Schema::Object(o) = &mut schema {
-        cleanup_number_tuples_object(o);
-    }
-
-    schema
-}
-
 pub fn is_primitive(schema: &schemars::schema::Schema) -> Result<Option<Primitive>> {
    match schema {
        schemars::schema::Schema::Object(o) => {
--- a/rust/kcl-lib/src/engine/mod.rs
+++ b/rust/kcl-lib/src/engine/mod.rs
@ -170,12 +170,16 @@ pub trait EngineManager: std::fmt::Debug + Send + Sync + 'static {
        self.clear_queues().await;

        self.batch_modeling_cmd(
-            uuid::Uuid::new_v4(),
+            id_generator.next_uuid(),
            source_range,
            &ModelingCmd::SceneClearAll(mcmd::SceneClearAll::default()),
        )
        .await?;

+        // Reset to the default units.  Modules assume the engine starts in the
+        // default state.
+        self.set_units(Default::default(), source_range, id_generator).await?;
+
        // Flush the batch queue, so clear is run right away.
        // Otherwise the hooks below won't work.
        self.flush_batch(false, source_range).await?;
@ -195,9 +199,10 @@ pub trait EngineManager: std::fmt::Debug + Send + Sync + 'static {
        &self,
        visible: bool,
        source_range: SourceRange,
+        id_generator: &mut IdGenerator,
    ) -> Result<(), crate::errors::KclError> {
        self.batch_modeling_cmd(
-            uuid::Uuid::new_v4(),
+            id_generator.next_uuid(),
            source_range,
            &ModelingCmd::from(mcmd::EdgeLinesVisible { hidden: !visible }),
        )
@ -231,10 +236,11 @@ pub trait EngineManager: std::fmt::Debug + Send + Sync + 'static {
        &self,
        units: crate::UnitLength,
        source_range: SourceRange,
+        id_generator: &mut IdGenerator,
    ) -> Result<(), crate::errors::KclError> {
        // Before we even start executing the program, set the units.
        self.batch_modeling_cmd(
-            uuid::Uuid::new_v4(),
+            id_generator.next_uuid(),
            source_range,
            &ModelingCmd::from(mcmd::SetSceneUnits { unit: units.into() }),
        )
@ -248,15 +254,15 @@ pub trait EngineManager: std::fmt::Debug + Send + Sync + 'static {
        &self,
        settings: &crate::ExecutorSettings,
        source_range: SourceRange,
+        id_generator: &mut IdGenerator,
    ) -> Result<(), crate::errors::KclError> {
        // Set the edge visibility.
-        self.set_edge_visibility(settings.highlight_edges, source_range).await?;
-
-        // Change the units.
-        self.set_units(settings.units, source_range).await?;
+        self.set_edge_visibility(settings.highlight_edges, source_range, id_generator)
+            .await?;

        // Send the command to show the grid.
-        self.modify_grid(!settings.show_grid, source_range).await?;
+        self.modify_grid(!settings.show_grid, source_range, id_generator)
+            .await?;

        // We do not have commands for changing ssao on the fly.

@ -502,6 +508,7 @@ pub trait EngineManager: std::fmt::Debug + Send + Sync + 'static {
        y_axis: Point3d,
        color: Option<Color>,
        source_range: SourceRange,
+        id_generator: &mut IdGenerator,
    ) -> Result<uuid::Uuid, KclError> {
        // Create new default planes.
        let default_size = 100.0;
@ -524,7 +531,7 @@ pub trait EngineManager: std::fmt::Debug + Send + Sync + 'static {
        if let Some(color) = color {
            // Set the color.
            self.batch_modeling_cmd(
-                uuid::Uuid::new_v4(),
+                id_generator.next_uuid(),
                source_range,
                &ModelingCmd::from(mcmd::PlaneSetColor { color, plane_id }),
            )
@ -615,7 +622,7 @@ pub trait EngineManager: std::fmt::Debug + Send + Sync + 'static {
        for (name, plane_id, x_axis, y_axis, color) in plane_settings {
            planes.insert(
                name,
-                self.make_default_plane(plane_id, x_axis, y_axis, color, source_range)
+                self.make_default_plane(plane_id, x_axis, y_axis, color, source_range, id_generator)
                    .await?,
            );
        }
@ -701,10 +708,15 @@ pub trait EngineManager: std::fmt::Debug + Send + Sync + 'static {
        }))
    }

-    async fn modify_grid(&self, hidden: bool, source_range: SourceRange) -> Result<(), KclError> {
+    async fn modify_grid(
+        &self,
+        hidden: bool,
+        source_range: SourceRange,
+        id_generator: &mut IdGenerator,
+    ) -> Result<(), KclError> {
        // Hide/show the grid.
        self.batch_modeling_cmd(
-            uuid::Uuid::new_v4(),
+            id_generator.next_uuid(),
            source_range,
            &ModelingCmd::from(mcmd::ObjectVisible {
                hidden,
@ -715,7 +727,7 @@ pub trait EngineManager: std::fmt::Debug + Send + Sync + 'static {

        // Hide/show the grid scale text.
        self.batch_modeling_cmd(
-            uuid::Uuid::new_v4(),
+            id_generator.next_uuid(),
            source_range,
            &ModelingCmd::from(mcmd::ObjectVisible {
                hidden,
--- a/rust/kcl-lib/src/execution/cache.rs
+++ b/rust/kcl-lib/src/execution/cache.rs
@ -97,15 +97,6 @@ pub(super) async fn get_changed_program(old: CacheInformation<'_>, new: CacheInf
    // If the settings are different we might need to bust the cache.
    // We specifically do this before checking if they are the exact same.
    if old.settings != new.settings {
-        // If the units are different we need to re-execute the whole thing.
-        if old.settings.units != new.settings.units {
-            return CacheResult::ReExecute {
-                clear_scene: true,
-                reapply_settings: true,
-                program: new.ast.clone(),
-            };
-        }
-
        // If anything else is different we may not need to re-execute, but rather just
        // run the settings again.
        reapply_settings = true;
@ -424,50 +415,6 @@ shell(firstSketch, faces = ['end'], thickness = 0.25)"#;
        assert_eq!(result, CacheResult::NoAction(false));
    }

-    // Changing the units with the exact same file should bust the cache.
-    #[tokio::test(flavor = "multi_thread")]
-    async fn test_get_changed_program_same_code_but_different_units() {
-        let new = r#"// Remove the end face for the extrusion.
-firstSketch = startSketchOn('XY')
-  |> startProfileAt([-12, 12], %)
-  |> line(end = [24, 0])
-  |> line(end = [0, -24])
-  |> line(end = [-24, 0])
-  |> close()
-  |> extrude(length = 6)
-
-// Remove the end face for the extrusion.
-shell(firstSketch, faces = ['end'], thickness = 0.25)"#;
-
-        let ExecTestResults {
-            program, mut exec_ctxt, ..
-        } = parse_execute(new).await.unwrap();
-
-        // Change the settings to cm.
-        exec_ctxt.settings.units = crate::UnitLength::Cm;
-
-        let result = get_changed_program(
-            CacheInformation {
-                ast: &program.ast,
-                settings: &Default::default(),
-            },
-            CacheInformation {
-                ast: &program.ast,
-                settings: &exec_ctxt.settings,
-            },
-        )
-        .await;
-
-        assert_eq!(
-            result,
-            CacheResult::ReExecute {
-                clear_scene: true,
-                reapply_settings: true,
-                program: program.ast
-            }
-        );
-    }
-
    // Changing the grid settings with the exact same file should NOT bust the cache.
    #[tokio::test(flavor = "multi_thread")]
    async fn test_get_changed_program_same_code_but_different_grid_setting() {
@ -615,4 +562,42 @@ startSketchOn('XY')
            }
        );
    }
+
+    // Removing the units settings using an annotation, when it was non-default
+    // units, with the exact same file should bust the cache.
+    #[tokio::test(flavor = "multi_thread")]
+    async fn test_get_changed_program_same_code_but_removed_unit_setting_using_annotation() {
+        let old_code = r#"@settings(defaultLengthUnit = in)
+startSketchOn('XY')
+"#;
+        let new_code = r#"
+startSketchOn('XY')
+"#;
+
+        let ExecTestResults { program, exec_ctxt, .. } = parse_execute(old_code).await.unwrap();
+
+        let mut new_program = crate::Program::parse_no_errs(new_code).unwrap();
+        new_program.compute_digest();
+
+        let result = get_changed_program(
+            CacheInformation {
+                ast: &program.ast,
+                settings: &exec_ctxt.settings,
+            },
+            CacheInformation {
+                ast: &new_program.ast,
+                settings: &exec_ctxt.settings,
+            },
+        )
+        .await;
+
+        assert_eq!(
+            result,
+            CacheResult::ReExecute {
+                clear_scene: true,
+                reapply_settings: true,
+                program: new_program.ast
+            }
+        );
+    }
 }
--- a/rust/kcl-lib/src/execution/exec_ast.rs
+++ b/rust/kcl-lib/src/execution/exec_ast.rs
@ -64,7 +64,11 @@ impl ExecutorContext {
                    let new_units = exec_state.length_unit();
                    if !self.engine.execution_kind().await.is_isolated() {
                        self.engine
-                            .set_units(new_units.into(), annotation.as_source_range())
+                            .set_units(
+                                new_units.into(),
+                                annotation.as_source_range(),
+                                exec_state.id_generator(),
+                            )
                            .await?;
                    }
                } else {
@ -106,12 +110,7 @@ impl ExecutorContext {
        let old_units = exec_state.length_unit();
        let original_execution = self.engine.replace_execution_kind(exec_kind).await;

-        let mut local_state = ModuleState::new(
-            &self.settings,
-            path.std_path(),
-            exec_state.stack().memory.clone(),
-            Some(module_id),
-        );
+        let mut local_state = ModuleState::new(path.std_path(), exec_state.stack().memory.clone(), Some(module_id));
        if !preserve_mem {
            std::mem::swap(&mut exec_state.mod_local, &mut local_state);
        }
@ -143,7 +142,9 @@ impl ExecutorContext {
        // command and we'd need to flush the batch again.
        // This avoids that.
        if !exec_kind.is_isolated() && new_units != old_units && *path != ModulePath::Main {
-            self.engine.set_units(old_units.into(), Default::default()).await?;
+            self.engine
+                .set_units(old_units.into(), Default::default(), exec_state.id_generator())
+                .await?;
        }
        self.engine.replace_execution_kind(original_execution).await;

--- a/rust/kcl-lib/src/execution/id_generator.rs
+++ b/rust/kcl-lib/src/execution/id_generator.rs
@ -2,7 +2,7 @@

 use crate::execution::ModuleId;

-const NAMESPACE_KCL: uuid::Uuid = uuid::uuid!("efcd6508-4ce6-4a09-8317-e6a6994a3cd7");
+const NAMESPACE_KCL: uuid::Uuid = uuid::uuid!("8bda3118-75eb-58c7-a866-bef1dcb495e7");

 /// A generator for ArtifactIds that can be stable across executions.
 #[derive(Debug, Clone, Default, PartialEq)]
--- a/rust/kcl-lib/src/execution/mod.rs
+++ b/rust/kcl-lib/src/execution/mod.rs
@ -39,7 +39,6 @@ use crate::{
    fs::FileManager,
    modules::{ModuleId, ModulePath},
    parsing::ast::types::{Expr, ImportPath, NodeRef},
-    settings::types::UnitLength,
    source_range::SourceRange,
    std::StdLib,
    CompilationError, ExecError, ExecutionKind, KclErrorWithOutputs,
@ -265,8 +264,6 @@ pub struct ExecutorContext {
 #[derive(Debug, Clone, Deserialize, Serialize, PartialEq, ts_rs::TS, JsonSchema)]
 #[ts(export)]
 pub struct ExecutorSettings {
-    /// The project-default unit to use in modeling dimensions.
-    pub units: UnitLength,
    /// Highlight edges of 3D objects?
    pub highlight_edges: bool,
    /// Whether or not Screen Space Ambient Occlusion (SSAO) is enabled.
@ -287,7 +284,6 @@ pub struct ExecutorSettings {
 impl Default for ExecutorSettings {
    fn default() -> Self {
        Self {
-            units: Default::default(),
            highlight_edges: true,
            enable_ssao: false,
            show_grid: false,
@ -301,7 +297,6 @@ impl Default for ExecutorSettings {
 impl From<crate::settings::types::Configuration> for ExecutorSettings {
    fn from(config: crate::settings::types::Configuration) -> Self {
        Self {
-            units: config.settings.modeling.base_unit,
            highlight_edges: config.settings.modeling.highlight_edges.into(),
            enable_ssao: config.settings.modeling.enable_ssao.into(),
            show_grid: config.settings.modeling.show_scale_grid,
@ -315,7 +310,6 @@ impl From<crate::settings::types::Configuration> for ExecutorSettings {
 impl From<crate::settings::types::project::ProjectConfiguration> for ExecutorSettings {
    fn from(config: crate::settings::types::project::ProjectConfiguration) -> Self {
        Self {
-            units: config.settings.modeling.base_unit,
            highlight_edges: config.settings.modeling.highlight_edges.into(),
            enable_ssao: config.settings.modeling.enable_ssao.into(),
            show_grid: Default::default(),
@ -329,7 +323,6 @@ impl From<crate::settings::types::project::ProjectConfiguration> for ExecutorSet
 impl From<crate::settings::types::ModelingSettings> for ExecutorSettings {
    fn from(modeling: crate::settings::types::ModelingSettings) -> Self {
        Self {
-            units: modeling.base_unit,
            highlight_edges: modeling.highlight_edges.into(),
            enable_ssao: modeling.enable_ssao.into(),
            show_grid: modeling.show_scale_grid,
@ -343,7 +336,6 @@ impl From<crate::settings::types::ModelingSettings> for ExecutorSettings {
 impl From<crate::settings::types::project::ProjectModelingSettings> for ExecutorSettings {
    fn from(modeling: crate::settings::types::project::ProjectModelingSettings) -> Self {
        Self {
-            units: modeling.base_unit,
            highlight_edges: modeling.highlight_edges.into(),
            enable_ssao: modeling.enable_ssao.into(),
            show_grid: Default::default(),
@ -476,26 +468,17 @@ impl ExecutorContext {
    /// This allows for passing in `ZOO_API_TOKEN` and `ZOO_HOST` as environment
    /// variables.
    #[cfg(not(target_arch = "wasm32"))]
-    pub async fn new_with_default_client(units: UnitLength) -> Result<Self> {
+    pub async fn new_with_default_client() -> Result<Self> {
        // Create the client.
-        let ctx = Self::new_with_client(
-            ExecutorSettings {
-                units,
-                ..Default::default()
-            },
-            None,
-            None,
-        )
-        .await?;
+        let ctx = Self::new_with_client(Default::default(), None, None).await?;
        Ok(ctx)
    }

    /// For executing unit tests.
    #[cfg(not(target_arch = "wasm32"))]
-    pub async fn new_for_unit_test(units: UnitLength, engine_addr: Option<String>) -> Result<Self> {
+    pub async fn new_for_unit_test(engine_addr: Option<String>) -> Result<Self> {
        let ctx = ExecutorContext::new_with_client(
            ExecutorSettings {
-                units,
                highlight_edges: true,
                enable_ssao: false,
                show_grid: false,
@ -618,7 +601,7 @@ impl ExecutorContext {
                    if reapply_settings
                        && self
                            .engine
-                            .reapply_settings(&self.settings, Default::default())
+                            .reapply_settings(&self.settings, Default::default(), old_state.id_generator())
                            .await
                            .is_err()
                    {
@ -636,7 +619,7 @@ impl ExecutorContext {
                CacheResult::NoAction(true) => {
                    if self
                        .engine
-                        .reapply_settings(&self.settings, Default::default())
+                        .reapply_settings(&self.settings, Default::default(), old_state.id_generator())
                        .await
                        .is_ok()
                    {
@ -737,7 +720,7 @@ impl ExecutorContext {

        // Re-apply the settings, in case the cache was busted.
        self.engine
-            .reapply_settings(&self.settings, Default::default())
+            .reapply_settings(&self.settings, Default::default(), exec_state.id_generator())
            .await
            .map_err(KclErrorWithOutputs::no_outputs)?;

@ -862,11 +845,6 @@ impl ExecutorContext {
        Ok(())
    }

-    /// Update the units for the executor.
-    pub(crate) fn update_units(&mut self, units: UnitLength) {
-        self.settings.units = units;
-    }
-
    /// Get a snapshot of the current scene.
    pub async fn prepare_snapshot(&self) -> std::result::Result<TakeSnapshot, ExecError> {
        // Zoom to fit.
@ -1008,11 +986,7 @@ mod tests {
    use pretty_assertions::assert_eq;

    use super::*;
-    use crate::{
-        errors::{KclErrorDetails, Severity},
-        execution::memory::Stack,
-        ModuleId,
-    };
+    use crate::{errors::KclErrorDetails, execution::memory::Stack, ModuleId};

    /// Convenience function to get a JSON value from memory and unwrap.
    #[track_caller]
@ -1615,34 +1589,6 @@ const inInches = 2.0 * inch()"#;
        );
    }

-    #[tokio::test(flavor = "multi_thread")]
-    async fn test_unit_suggest() {
-        let src = "foo = 42";
-        let program = crate::Program::parse_no_errs(src).unwrap();
-        let ctx = ExecutorContext {
-            engine: Arc::new(Box::new(
-                crate::engine::conn_mock::EngineConnection::new().await.unwrap(),
-            )),
-            fs: Arc::new(crate::fs::FileManager::new()),
-            stdlib: Arc::new(crate::std::StdLib::new()),
-            settings: ExecutorSettings {
-                units: UnitLength::Ft,
-                ..Default::default()
-            },
-            context_type: ContextType::Mock,
-        };
-        let mut exec_state = ExecState::new(&ctx);
-        ctx.run(&program, &mut exec_state).await.unwrap();
-        let errs = exec_state.errors();
-        assert_eq!(errs.len(), 1, "{errs:?}");
-        let warn = &errs[0];
-        assert_eq!(warn.severity, Severity::Warning);
-        assert_eq!(
-            warn.apply_suggestion(src).unwrap(),
-            "@settings(defaultLengthUnit = ft)\nfoo = 42"
-        )
-    }
-
    #[tokio::test(flavor = "multi_thread")]
    async fn test_zero_param_fn() {
        let ast = r#"const sigmaAllow = 35000 // psi
@ -1971,9 +1917,7 @@ let w = f() + f()
 )
 "#;

-        let ctx = crate::test_server::new_context(UnitLength::Mm, true, None)
-            .await
-            .unwrap();
+        let ctx = crate::test_server::new_context(true, None).await.unwrap();
        let old_program = crate::Program::parse_no_errs(code).unwrap();

        // Execute the program.
@ -2026,9 +1970,7 @@ let w = f() + f()
 )
 "#;

-        let mut ctx = crate::test_server::new_context(UnitLength::Mm, true, None)
-            .await
-            .unwrap();
+        let mut ctx = crate::test_server::new_context(true, None).await.unwrap();
        let old_program = crate::Program::parse_no_errs(code).unwrap();

        // Execute the program.
@ -2066,7 +2008,7 @@ let w = f() + f()

    #[tokio::test(flavor = "multi_thread")]
    async fn mock_after_not_mock() {
-        let ctx = ExecutorContext::new_with_default_client(UnitLength::Mm).await.unwrap();
+        let ctx = ExecutorContext::new_with_default_client().await.unwrap();
        let program = crate::Program::parse_no_errs("x = 2").unwrap();
        let result = ctx.run_with_caching(program).await.unwrap();
        assert_eq!(result.variables.get("x").unwrap().as_f64().unwrap(), 2.0);
--- a/Show More
+++ b/Show More
Author	SHA1	Message	Date
Jonathan Tran	5ab814d153	Add debouncing settle detection	2025-04-01 13:33:49 -04:00
Jonathan Tran	fe83cd94ca	Add debounce function	2025-04-01 13:33:49 -04:00
Jonathan Tran	5178a72c52	Add registry for background promises	2025-04-01 13:33:49 -04:00
Jonathan Tran	f4e801351c	Add more TS lints (#6084 ) * Fix to not call onMouseLeave with no selected object * Add no this alias lint * Add more lints and fix JSON formatting * Fix to use lower-case string type * Add another namespace lint * Fix to not use plus on possibly non-string values	2025-04-01 10:21:31 -07:00
Jonathan Tran	7329753211	Bump kcl versions (#6089 )	2025-04-01 12:43:27 -04:00
Jess Frazelle	7d46e7e271	fix api for edge cuts (#6086 ) Signed-off-by: Jess Frazelle <github@jessfraz.com>	2025-04-01 12:42:11 -04:00
Jace Browning	0583eb07d5	Remove unused test code to fix console error (#6073 )	2025-04-01 03:08:04 +00:00
Jess Frazelle	73694563cf	change TyF64 to f64 according to JsonSchema and cleanup docs code (#6081 ) * cleanup gen_std Signed-off-by: Jess Frazelle <github@jessfraz.com> * cleanup docs Signed-off-by: Jess Frazelle <github@jessfraz.com> * updates Signed-off-by: Jess Frazelle <github@jessfraz.com> * fix table Signed-off-by: Jess Frazelle <github@jessfraz.com> --------- Signed-off-by: Jess Frazelle <github@jessfraz.com>	2025-04-01 01:02:48 +00:00
Jonathan Tran	bb4ed59191	Fix patterning in module to not fail when importing function from the module (#6082 ) * Add failing test * Update output to show execution error * Fix circular pattern to not error in isolated or mock mode * Update output after fix * Add failing test for pattern linear 2D * Add failing output * Fix isolated execution in linear patterns * Update output after linear fix	2025-03-31 23:46:29 +00:00
Pierre Jacquier	566143757f	Fix 404 when clicking on the car wheel sample (#6079 ) Fixes #6078	2025-03-31 23:09:59 +00:00
Pierre Jacquier	822f2ffc73	Add edit flow for Revolve point-and-click (#5951 ) * WIP: Add edge and segment selection in point-and-click Helix flow Fixes #5393 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * Working edge based helix edit * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * Add utility function for shared code between revolve and helix * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * Use updateModelingState in codemod * A snapshot a day keeps the bugs away! 📷🐛 * Add skip: true for edge helix to be consistent with axis as options * A snapshot a day keeps the bugs away! 📷🐛 * Add support for sweepEdge and tests * Lint * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * Clean up snapshots * WIP: Add edit flow for Revolve Fixes #5504 * A snapshot a day keeps the bugs away! 📷🐛 * Clean up, add edit steps to three e2e tests * Fix up tests after ccw change * Use displayName: 'CounterClockWise' cause ccw not cutting it * Fix tsc * Remove uneeded return * Update 2020 snapshots after helix change * Update 2020 snapshots after helix change * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * Another one :djkhaled: * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * Move to fromPromise actor, fix test * A snapshot a day keeps the bugs away! 📷🐛 * Clean up * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * A snapshot a day keeps the bugs away! 📷🐛 * Lint * Fix third test * A snapshot a day keeps the bugs away! 📷🐛 * Lint * Fix edit insert order * Fix axis, edge cases with new var, edit new var * Add test case for new variable creation * Clean up addRevolve with getSafeInsertIndex --------- Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>	2025-03-31 22:13:34 +00:00
Frank Noirot	eb3ceba497	Minor design tweaks to log in page (#6071 ) Bump subheadings, make link buttons primary colored	2025-03-31 15:33:27 -04:00
Andrew Varga	2c6d0621c9	Add test for #4236 : "Fix corner rect panning bug" (#6018 ) * add test for rect panning bug * tweak timeouts * cleanups, handle both center and corner rectangle * revert regression that was used for the test * apply PR feedback	2025-03-31 15:31:22 -04:00
Jonathan Tran	d8e84cb5e3	Change default tolerance value to not depend on units (#6055 )	2025-03-31 19:28:15 +00:00
dependabot[bot]	0b1e79871f	Bump the patch group across 1 directory with 16 updates (#6068 ) Bumps the patch group with 16 updates in the / directory: \| Package \| From \| To \| \| --- \| --- \| --- \| \| [@codemirror/commands](https://github.com/codemirror/commands) \| `6.8.0` \| `6.8.1` \| \| [@codemirror/lint](https://github.com/codemirror/lint) \| `6.8.4` \| `6.8.5` \| \| [@codemirror/state](https://github.com/codemirror/state) \| `6.5.0` \| `6.5.2` \| \| [@csstools/postcss-oklab-function](https://github.com/csstools/postcss-plugins/tree/HEAD/plugins/postcss-oklab-function) \| `4.0.7` \| `4.0.8` \| \| [@headlessui/tailwindcss](https://github.com/tailwindlabs/headlessui/tree/HEAD/packages/@headlessui-tailwindcss) \| `0.2.1` \| `0.2.2` \| \| [@kittycad/lib](https://github.com/KittyCAD/kittycad.ts) \| `2.0.21` \| `2.0.23` \| \| [chokidar](https://github.com/paulmillr/chokidar) \| `4.0.1` \| `4.0.3` \| \| [electron-updater](https://github.com/electron-userland/electron-builder/tree/HEAD/packages/electron-updater) \| `6.6.0` \| `6.6.2` \| \| [@playwright/test](https://github.com/microsoft/playwright) \| `1.51.0` \| `1.51.1` \| \| [@types/diff](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/diff) \| `7.0.1` \| `7.0.2` \| \| [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) \| `22.13.9` \| `22.13.14` \| \| [@vitejs/plugin-react](https://github.com/vitejs/vite-plugin-react/tree/HEAD/packages/plugin-react) \| `4.3.1` \| `4.3.4` \| \| [autoprefixer](https://github.com/postcss/autoprefixer) \| `10.4.19` \| `10.4.21` \| \| [electron-builder](https://github.com/electron-userland/electron-builder/tree/HEAD/packages/electron-builder) \| `26.0.7` \| `26.0.12` \| \| [ws](https://github.com/websockets/ws) \| `8.18.0` \| `8.18.1` \| \| [@types/ws](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/ws) \| `8.5.13` \| `8.18.0` \| Updates `@codemirror/commands` from 6.8.0 to 6.8.1 - [Changelog](https://github.com/codemirror/commands/blob/main/CHANGELOG.md) - [Commits](https://github.com/codemirror/commands/compare/6.8.0...6.8.1) Updates `@codemirror/lint` from 6.8.4 to 6.8.5 - [Changelog](https://github.com/codemirror/lint/blob/main/CHANGELOG.md) - [Commits](https://github.com/codemirror/lint/compare/6.8.4...6.8.5) Updates `@codemirror/state` from 6.5.0 to 6.5.2 - [Changelog](https://github.com/codemirror/state/blob/main/CHANGELOG.md) - [Commits](https://github.com/codemirror/state/compare/6.5.0...6.5.2) Updates `@csstools/postcss-oklab-function` from 4.0.7 to 4.0.8 - [Changelog](https://github.com/csstools/postcss-plugins/blob/main/plugins/postcss-oklab-function/CHANGELOG.md) - [Commits](https://github.com/csstools/postcss-plugins/commits/HEAD/plugins/postcss-oklab-function) Updates `@headlessui/tailwindcss` from 0.2.1 to 0.2.2 - [Release notes](https://github.com/tailwindlabs/headlessui/releases) - [Changelog](https://github.com/tailwindlabs/headlessui/blob/main/packages/@headlessui-tailwindcss/CHANGELOG.md) - [Commits](https://github.com/tailwindlabs/headlessui/commits/@headlessui/tailwindcss@v0.2.2/packages/@headlessui-tailwindcss) Updates `@kittycad/lib` from 2.0.21 to 2.0.23 - [Release notes](https://github.com/KittyCAD/kittycad.ts/releases) - [Commits](https://github.com/KittyCAD/kittycad.ts/compare/v2.0.21...v2.0.23) Updates `chokidar` from 4.0.1 to 4.0.3 - [Release notes](https://github.com/paulmillr/chokidar/releases) - [Commits](https://github.com/paulmillr/chokidar/compare/4.0.1...4.0.3) Updates `electron-updater` from 6.6.0 to 6.6.2 - [Release notes](https://github.com/electron-userland/electron-builder/releases) - [Changelog](https://github.com/electron-userland/electron-builder/blob/master/packages/electron-updater/CHANGELOG.md) - [Commits](https://github.com/electron-userland/electron-builder/commits/electron-updater@6.6.2/packages/electron-updater) Updates `@playwright/test` from 1.51.0 to 1.51.1 - [Release notes](https://github.com/microsoft/playwright/releases) - [Commits](https://github.com/microsoft/playwright/compare/v1.51.0...v1.51.1) Updates `@types/diff` from 7.0.1 to 7.0.2 - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/diff) Updates `@types/node` from 22.13.9 to 22.13.14 - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node) Updates `@vitejs/plugin-react` from 4.3.1 to 4.3.4 - [Release notes](https://github.com/vitejs/vite-plugin-react/releases) - [Changelog](https://github.com/vitejs/vite-plugin-react/blob/main/packages/plugin-react/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite-plugin-react/commits/v4.3.4/packages/plugin-react) Updates `autoprefixer` from 10.4.19 to 10.4.21 - [Release notes](https://github.com/postcss/autoprefixer/releases) - [Changelog](https://github.com/postcss/autoprefixer/blob/main/CHANGELOG.md) - [Commits](https://github.com/postcss/autoprefixer/compare/10.4.19...10.4.21) Updates `electron-builder` from 26.0.7 to 26.0.12 - [Release notes](https://github.com/electron-userland/electron-builder/releases) - [Changelog](https://github.com/electron-userland/electron-builder/blob/master/packages/electron-builder/CHANGELOG.md) - [Commits](https://github.com/electron-userland/electron-builder/commits/v26.0.12/packages/electron-builder) Updates `ws` from 8.18.0 to 8.18.1 - [Release notes](https://github.com/websockets/ws/releases) - [Commits](https://github.com/websockets/ws/compare/8.18.0...8.18.1) Updates `@types/ws` from 8.5.13 to 8.18.0 - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/ws) --- updated-dependencies: - dependency-name: "@codemirror/commands" dependency-type: direct:production update-type: version-update:semver-patch dependency-group: patch - dependency-name: "@codemirror/lint" dependency-type: direct:production update-type: version-update:semver-patch dependency-group: patch - dependency-name: "@codemirror/state" dependency-type: direct:production update-type: version-update:semver-patch dependency-group: patch - dependency-name: "@csstools/postcss-oklab-function" dependency-type: direct:production update-type: version-update:semver-patch dependency-group: patch - dependency-name: "@headlessui/tailwindcss" dependency-type: direct:production update-type: version-update:semver-patch dependency-group: patch - dependency-name: "@kittycad/lib" dependency-type: direct:production update-type: version-update:semver-patch dependency-group: patch - dependency-name: chokidar dependency-type: direct:production update-type: version-update:semver-patch dependency-group: patch - dependency-name: electron-updater dependency-type: direct:production update-type: version-update:semver-patch dependency-group: patch - dependency-name: "@playwright/test" dependency-type: direct:development update-type: version-update:semver-patch dependency-group: patch - dependency-name: "@types/diff" dependency-type: direct:development update-type: version-update:semver-patch dependency-group: patch - dependency-name: "@types/node" dependency-type: direct:development update-type: version-update:semver-patch dependency-group: patch - dependency-name: "@vitejs/plugin-react" dependency-type: direct:development update-type: version-update:semver-patch dependency-group: patch - dependency-name: autoprefixer dependency-type: direct:development update-type: version-update:semver-patch dependency-group: patch - dependency-name: electron-builder dependency-type: direct:development update-type: version-update:semver-patch dependency-group: patch - dependency-name: ws dependency-type: direct:development update-type: version-update:semver-patch dependency-group: patch - dependency-name: "@types/ws" dependency-type: direct:development update-type: version-update:semver-minor dependency-group: patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>	2025-03-31 18:56:37 +00:00
Frank Noirot	954fddf578	Show a toast if the user tries to save their work (#6051 ) * Show a toast if the user tries to save their work And store the fact that they've seen it in localStorage so we don't spam them every time they hit it after they see it. * Remove any localStorage logic, toast every time	2025-03-31 14:37:04 -04:00
Pierre Jacquier	f94f748339	Use waitForExecutionDone in lint errors test (#6070 )	2025-03-31 13:43:43 -04:00
Jess Frazelle	e2b7b22ca9	ctx.close in tests when error on execution (#6075 ) Signed-off-by: Jess Frazelle <github@jessfraz.com>	2025-03-31 10:24:42 -07:00
Jonathan Tran	efc8c82d8b	BREAKING: KCL @settings are the source of truth for units (#5808 )	2025-03-31 10:56:03 -04:00
Pierre Jacquier	eac5abba79	Clean up env vars in e2e-tests.yml (#6045 ) * pierremtb/adhoc/clean-up-e2e-env-vars-ci * Add back VITE_KC_SKIP_AUTH: true for snaps * Remove VITE_KC_SKIP_AUTH: true, add back token	2025-03-31 09:18:23 -04:00
Jess Frazelle	1956c14b8a	auto format kcl samples (#6065 ) * recast kcl samples; Signed-off-by: Jess Frazelle <github@jessfraz.com> * auto format the kcl-samples Signed-off-by: Jess Frazelle <github@jessfraz.com> --------- Signed-off-by: Jess Frazelle <github@jessfraz.com>	2025-03-30 17:06:36 +00:00
Jess Frazelle	017fac7041	use deterministic ids in more places (#6064 ) * dont redact the ids now that they are deterministic Signed-off-by: Jess Frazelle <github@jessfraz.com> pass arouund id generator more Signed-off-by: Jess Frazelle <github@jessfraz.com> change the anme space Signed-off-by: Jess Frazelle <github@jessfraz.com> updates and re-run Signed-off-by: Jess Frazelle <github@jessfraz.com> updates Signed-off-by: Jess Frazelle <github@jessfraz.com> * fixups Signed-off-by: Jess Frazelle <github@jessfraz.com> * fixes Signed-off-by: Jess Frazelle <github@jessfraz.com> * cleanup old files Signed-off-by: Jess Frazelle <github@jessfraz.com> * cleanup old files Signed-off-by: Jess Frazelle <github@jessfraz.com> --------- Signed-off-by: Jess Frazelle <github@jessfraz.com>	2025-03-30 04:23:11 +00:00