Compare commits

...

11 Commits

Author SHA1 Message Date
b17e61d963 Cut release v0.10.0 (#803)
Co-authored-by: Frank Noirot <frank@kittycad.io>
2023-10-06 11:09:54 -04:00
d31d07d9c8 Make "Replay Onboarding" button available on home settings page (#804)
* Fix unrelated bug, settings button in the home sidebar
doesn't go to the home settings after my previous fixes to routes

* Turn on "Replay Onboarding" button in home settings

* Use ONBOARDING_PROJECT_NAME in both places

* Fix formatting
2023-10-06 10:00:35 -04:00
7aa2d63c21 Hide planes (#797)
* hide planes in one go

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* update hide;

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* fixes

Signed-off-by: Jess Frazelle <github@jessfraz.com>

---------

Signed-off-by: Jess Frazelle <github@jessfraz.com>
2023-10-05 19:54:31 -07:00
e1081b0ee6 turning back on all planes (#720)
* updates

Signed-off-by: Jess Frazelle <github@jessfraz.com>

tests

Signed-off-by: Jess Frazelle <github@jessfraz.com>

fix more tests

Signed-off-by: Jess Frazelle <github@jessfraz.com>

fixes

Signed-off-by: Jess Frazelle <github@jessfraz.com>

fix stdlib

Signed-off-by: Jess Frazelle <github@jessfraz.com>

fix tests

Signed-off-by: Jess Frazelle <github@jessfraz.com>

fixes

Signed-off-by: Jess Frazelle <github@jessfraz.com>

updates

Signed-off-by: Jess Frazelle <github@jessfraz.com>

updates

Signed-off-by: Jess Frazelle <github@jessfraz.com>

compile

Signed-off-by: Jess Frazelle <github@jessfraz.com>

update sample code

Signed-off-by: Jess Frazelle <github@jessfraz.com>

re-enable the planes

Signed-off-by: Jess Frazelle <github@jessfraz.com>

fix tests

Signed-off-by: Jess Frazelle <github@jessfraz.com>

fix tests

Signed-off-by: Jess Frazelle <github@jessfraz.com>

fix tests

Signed-off-by: Jess Frazelle <github@jessfraz.com>

fix tests

Signed-off-by: Jess Frazelle <github@jessfraz.com>

fix tests

Signed-off-by: Jess Frazelle <github@jessfraz.com>

fix tests

Signed-off-by: Jess Frazelle <github@jessfraz.com>

fix tests

Signed-off-by: Jess Frazelle <github@jessfraz.com>

updates

Signed-off-by: Jess Frazelle <github@jessfraz.com>

fix all tests

Signed-off-by: Jess Frazelle <github@jessfraz.com>

updates

Signed-off-by: Jess Frazelle <github@jessfraz.com>

boilerplate

Signed-off-by: Jess Frazelle <github@jessfraz.com>

Cut release v0.9.2 (#714)

rust make default planes

Signed-off-by: Jess Frazelle <github@jessfraz.com>

updates

Signed-off-by: Jess Frazelle <github@jessfraz.com>

use the planes from engine

Signed-off-by: Jess Frazelle <github@jessfraz.com>

fixups

Signed-off-by: Jess Frazelle <github@jessfraz.com>

updates

Signed-off-by: Jess Frazelle <github@jessfraz.com>

fixes

Signed-off-by: Jess Frazelle <github@jessfraz.com>

updates

Signed-off-by: Jess Frazelle <github@jessfraz.com>

fixes

Signed-off-by: Jess Frazelle <github@jessfraz.com>

fixes

Signed-off-by: Jess Frazelle <github@jessfraz.com>

negative args

Signed-off-by: Jess Frazelle <github@jessfraz.com>

diable camera

Signed-off-by: Jess Frazelle <github@jessfraz.com>

hide planes

Signed-off-by: Jess Frazelle <github@jessfraz.com>

updates

Signed-off-by: Jess Frazelle <github@jessfraz.com>

updates

Signed-off-by: Jess Frazelle <github@jessfraz.com>

updatress

Signed-off-by: Jess Frazelle <github@jessfraz.com>

fmt

Signed-off-by: Jess Frazelle <github@jessfraz.com>

Update src/hooks/useAppMode.ts

Co-authored-by: Kurt Hutten <k.hutten@protonmail.ch>

Update src/hooks/useAppMode.ts

Co-authored-by: Kurt Hutten <k.hutten@protonmail.ch>

cleanups

Signed-off-by: Jess Frazelle <github@jessfraz.com>

Bump kittycad from 0.2.26 to 0.2.27 in /src-tauri (#726)

Bumps [kittycad](https://github.com/KittyCAD/kittycad.rs) from 0.2.26 to 0.2.27.
- [Release notes](https://github.com/KittyCAD/kittycad.rs/releases)
- [Commits](https://github.com/KittyCAD/kittycad.rs/compare/v0.2.26...v0.2.27)

---
updated-dependencies:
- dependency-name: kittycad
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump tauri-plugin-fs-extra from `b04bde3` to `6c7a4c0` in /src-tauri (#725)

Bumps [tauri-plugin-fs-extra](https://github.com/tauri-apps/plugins-workspace) from `b04bde3` to `6c7a4c0`.
- [Release notes](https://github.com/tauri-apps/plugins-workspace/releases)
- [Commits](b04bde3461...6c7a4c0984)

---
updated-dependencies:
- dependency-name: tauri-plugin-fs-extra
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump toml from 0.8.0 to 0.8.1 in /src-tauri (#724)

Bumps [toml](https://github.com/toml-rs/toml) from 0.8.0 to 0.8.1.
- [Commits](https://github.com/toml-rs/toml/compare/toml-v0.8.0...toml-v0.8.1)

---
updated-dependencies:
- dependency-name: toml
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump openapitor from `61a1605` to `d3e98c4` in /src/wasm-lib (#723)

Bumps [openapitor](https://github.com/KittyCAD/kittycad.rs) from `61a1605` to `d3e98c4`.
- [Release notes](https://github.com/KittyCAD/kittycad.rs/releases)
- [Commits](61a16059b3...d3e98c4ec0)

---
updated-dependencies:
- dependency-name: openapitor
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump kittycad from 0.2.26 to 0.2.27 in /src/wasm-lib (#722)

Bumps [kittycad](https://github.com/KittyCAD/kittycad.rs) from 0.2.26 to 0.2.27.
- [Release notes](https://github.com/KittyCAD/kittycad.rs/releases)
- [Commits](https://github.com/KittyCAD/kittycad.rs/compare/v0.2.26...v0.2.27)

---
updated-dependencies:
- dependency-name: kittycad
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump thiserror from 1.0.48 to 1.0.49 in /src/wasm-lib (#721)

Bumps [thiserror](https://github.com/dtolnay/thiserror) from 1.0.48 to 1.0.49.
- [Release notes](https://github.com/dtolnay/thiserror/releases)
- [Commits](https://github.com/dtolnay/thiserror/compare/1.0.48...1.0.49)

---
updated-dependencies:
- dependency-name: thiserror
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump expectorate from 1.0.7 to 1.1.0 in /src/wasm-lib (#712)

Bumps [expectorate](https://github.com/oxidecomputer/expectorate) from 1.0.7 to 1.1.0.
- [Release notes](https://github.com/oxidecomputer/expectorate/releases)
- [Commits](https://github.com/oxidecomputer/expectorate/compare/v1.0.7...v1.1.0)

---
updated-dependencies:
- dependency-name: expectorate
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump clap from 4.4.4 to 4.4.5 in /src/wasm-lib (#711)

Bumps [clap](https://github.com/clap-rs/clap) from 4.4.4 to 4.4.5.
- [Release notes](https://github.com/clap-rs/clap/releases)
- [Changelog](https://github.com/clap-rs/clap/blob/master/CHANGELOG.md)
- [Commits](https://github.com/clap-rs/clap/compare/v4.4.4...v4.4.5)

---
updated-dependencies:
- dependency-name: clap
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

refactor cleanup

Signed-off-by: Jess Frazelle <github@jessfraz.com>

updates

Signed-off-by: Jess Frazelle <github@jessfraz.com>

updates

Signed-off-by: Jess Frazelle <github@jessfraz.com>

type improvements

* use new sketchmode no camera

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* js working better

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* start of negative planes

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* tests and neg

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* updates

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* images

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* norma;s

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* better initial load of planes

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* ts

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* updates

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* fixes

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* updates

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* fixes

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* fix tsc

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* fix edit sketch

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* add regression test for 2d solid issue

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* updates

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* show planes

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* fix clippy

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* fix tests

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* canecel in progress

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* fix ci as well

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* updates

Signed-off-by: Jess Frazelle <github@jessfraz.com>

---------

Signed-off-by: Jess Frazelle <github@jessfraz.com>
2023-10-05 14:27:48 -07:00
59223279b7 Type Error: Type error engineConnection.ts (#786)
* TypeError Fix

* removed not needed console log

* took into account adamchalmers feedback and fixed cl type error

* pretty up

* Irev-dev feedback fixes issue
2023-10-05 16:18:50 +11:00
8a4e717565 Use absolute URLs to settings to avoid relative URL edge cases (#781)
* Create useAbsoluteFilePath hook

* Fix "report bug" link on Error page

* Replace relative URL to settings with absolute URL

* Replace other absolute file URLs to use common hook

* Use named const for default browser file name

* Fix UI tests that now rely on useRouteLoaderData()

Signed-off-by: Frank Noirot <frank@kittycad.io>

---------

Signed-off-by: Frank Noirot <frank@kittycad.io>
2023-10-04 18:00:55 -04:00
80b542ca18 Allow Vertical Scroll Wheel to affect Horizontal Scroll for Toolbar (#780) 2023-10-04 09:35:50 -07:00
e4bfc863ea Bump re-resizable from 6.9.9 to 6.9.11 (#777)
Bumps [re-resizable](https://github.com/bokuweb/react-resizable-box) from 6.9.9 to 6.9.11.
- [Release notes](https://github.com/bokuweb/react-resizable-box/releases)
- [Changelog](https://github.com/bokuweb/re-resizable/blob/master/CHANGELOG.md)
- [Commits](https://github.com/bokuweb/react-resizable-box/commits)

---
updated-dependencies:
- dependency-name: re-resizable
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2023-10-04 10:08:10 -05:00
77ef255de4 Bump tauri from 1.5.0 to 1.5.1 in /src-tauri (#761)
Bumps [tauri](https://github.com/tauri-apps/tauri) from 1.5.0 to 1.5.1.
- [Release notes](https://github.com/tauri-apps/tauri/releases)
- [Commits](https://github.com/tauri-apps/tauri/compare/tauri-v1.5...tauri-v1.5.1)

---
updated-dependencies:
- dependency-name: tauri
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2023-10-04 09:21:58 -05:00
64c3841079 Bump tauri-plugin-fs-extra from 9b96996 to fa32d1a in /src-tauri (#776)
Bumps [tauri-plugin-fs-extra](https://github.com/tauri-apps/plugins-workspace) from `9b96996` to `fa32d1a`.
- [Release notes](https://github.com/tauri-apps/plugins-workspace/releases)
- [Commits](9b96996b5a...fa32d1afa9)

---
updated-dependencies:
- dependency-name: tauri-plugin-fs-extra
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2023-10-03 23:23:43 -05:00
c7bb6bc845 Bump reqwest from 0.11.21 to 0.11.22 in /src/wasm-lib (#775)
Bumps [reqwest](https://github.com/seanmonstar/reqwest) from 0.11.21 to 0.11.22.
- [Release notes](https://github.com/seanmonstar/reqwest/releases)
- [Changelog](https://github.com/seanmonstar/reqwest/blob/master/CHANGELOG.md)
- [Commits](https://github.com/seanmonstar/reqwest/compare/v0.11.21...v0.11.22)

---
updated-dependencies:
- dependency-name: reqwest
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2023-10-03 23:23:18 -05:00
71 changed files with 3061 additions and 887 deletions

View File

@ -15,6 +15,9 @@ on:
- '**/Cargo.lock' - '**/Cargo.lock'
- '**/rust-toolchain.toml' - '**/rust-toolchain.toml'
- .github/workflows/cargo-build.yml - .github/workflows/cargo-build.yml
concurrency:
group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
cancel-in-progress: true
name: cargo build name: cargo build
jobs: jobs:
cargobuild: cargobuild:

View File

@ -15,6 +15,9 @@ on:
- '**/rust-toolchain.toml' - '**/rust-toolchain.toml'
- '**.rs' - '**.rs'
- .github/workflows/cargo-build.yml - .github/workflows/cargo-build.yml
concurrency:
group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
cancel-in-progress: true
name: cargo clippy name: cargo clippy
jobs: jobs:
cargoclippy: cargoclippy:

View File

@ -17,6 +17,9 @@ on:
- .github/workflows/cargo-criterion.yml - .github/workflows/cargo-criterion.yml
workflow_dispatch: workflow_dispatch:
permissions: read-all permissions: read-all
concurrency:
group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
cancel-in-progress: true
name: cargo criterion name: cargo criterion
jobs: jobs:
cargocriterion: cargocriterion:

View File

@ -18,6 +18,9 @@ on:
permissions: permissions:
packages: read packages: read
contents: read contents: read
concurrency:
group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
cancel-in-progress: true
name: cargo fmt name: cargo fmt
jobs: jobs:
cargofmt: cargofmt:

View File

@ -17,6 +17,9 @@ on:
- .github/workflows/cargo-test.yml - .github/workflows/cargo-test.yml
workflow_dispatch: workflow_dispatch:
permissions: read-all permissions: read-all
concurrency:
group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
cancel-in-progress: true
name: cargo test name: cargo test
jobs: jobs:
cargotest: cargotest:

View File

@ -8,6 +8,9 @@ on:
release: release:
types: [published] types: [published]
concurrency:
group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
cancel-in-progress: true
jobs: jobs:
check-format: check-format:
runs-on: 'ubuntu-20.04' runs-on: 'ubuntu-20.04'

File diff suppressed because it is too large Load Diff

View File

@ -48,7 +48,9 @@
* [`show`](#show) * [`show`](#show)
* [`sin`](#sin) * [`sin`](#sin)
* [`sqrt`](#sqrt) * [`sqrt`](#sqrt)
* [`startProfileAt`](#startProfileAt)
* [`startSketchAt`](#startSketchAt) * [`startSketchAt`](#startSketchAt)
* [`startSketchOn`](#startSketchOn)
* [`tan`](#tan) * [`tan`](#tan)
* [`tangentalArc`](#tangentalArc) * [`tangentalArc`](#tangentalArc)
* [`tangentalArcTo`](#tangentalArcTo) * [`tangentalArcTo`](#tangentalArcTo)
@ -120,6 +122,8 @@ angleToMatchLengthX(segment_name: string, to: number, sketch_group: SketchGroup)
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -204,6 +208,8 @@ angleToMatchLengthY(segment_name: string, to: number, sketch_group: SketchGroup)
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -298,6 +304,8 @@ angledLine(data: AngledLineData, sketch_group: SketchGroup) -> SketchGroup
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -364,6 +372,8 @@ angledLine(data: AngledLineData, sketch_group: SketchGroup) -> SketchGroup
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -454,6 +464,8 @@ angledLineOfXLength(data: AngledLineData, sketch_group: SketchGroup) -> SketchGr
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -520,6 +532,8 @@ angledLineOfXLength(data: AngledLineData, sketch_group: SketchGroup) -> SketchGr
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -610,6 +624,8 @@ angledLineOfYLength(data: AngledLineData, sketch_group: SketchGroup) -> SketchGr
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -676,6 +692,8 @@ angledLineOfYLength(data: AngledLineData, sketch_group: SketchGroup) -> SketchGr
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -767,6 +785,8 @@ angledLineThatIntersects(data: AngeledLineThatIntersectsData, sketch_group: Sket
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -833,6 +853,8 @@ angledLineThatIntersects(data: AngeledLineThatIntersectsData, sketch_group: Sket
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -923,6 +945,8 @@ angledLineToX(data: AngledLineToData, sketch_group: SketchGroup) -> SketchGroup
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -989,6 +1013,8 @@ angledLineToX(data: AngledLineToData, sketch_group: SketchGroup) -> SketchGroup
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -1079,6 +1105,8 @@ angledLineToY(data: AngledLineToData, sketch_group: SketchGroup) -> SketchGroup
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -1145,6 +1173,8 @@ angledLineToY(data: AngledLineToData, sketch_group: SketchGroup) -> SketchGroup
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -1262,6 +1292,8 @@ arc(data: ArcData, sketch_group: SketchGroup) -> SketchGroup
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -1328,6 +1360,8 @@ arc(data: ArcData, sketch_group: SketchGroup) -> SketchGroup
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -1467,6 +1501,8 @@ bezierCurve(data: BezierData, sketch_group: SketchGroup) -> SketchGroup
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -1533,6 +1569,8 @@ bezierCurve(data: BezierData, sketch_group: SketchGroup) -> SketchGroup
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -1631,6 +1669,8 @@ close(sketch_group: SketchGroup) -> SketchGroup
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -1697,6 +1737,8 @@ close(sketch_group: SketchGroup) -> SketchGroup
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -1815,6 +1857,8 @@ extrude(length: number, sketch_group: SketchGroup) -> ExtrudeGroup
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -1996,6 +2040,8 @@ lastSegX(sketch_group: SketchGroup) -> number
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -2078,6 +2124,8 @@ lastSegY(sketch_group: SketchGroup) -> number
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -2233,6 +2281,8 @@ line(data: LineData, sketch_group: SketchGroup) -> SketchGroup
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -2299,6 +2349,8 @@ line(data: LineData, sketch_group: SketchGroup) -> SketchGroup
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -2387,6 +2439,8 @@ lineTo(data: LineToData, sketch_group: SketchGroup) -> SketchGroup
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -2453,6 +2507,8 @@ lineTo(data: LineToData, sketch_group: SketchGroup) -> SketchGroup
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -2693,6 +2749,8 @@ segAng(segment_name: string, sketch_group: SketchGroup) -> number
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -2776,6 +2834,8 @@ segEndX(segment_name: string, sketch_group: SketchGroup) -> number
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -2859,6 +2919,8 @@ segEndY(segment_name: string, sketch_group: SketchGroup) -> number
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -2942,6 +3004,8 @@ segLen(segment_name: string, sketch_group: SketchGroup) -> number
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -3024,6 +3088,8 @@ show(sketch: SketchGroup)
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -3125,14 +3191,14 @@ sqrt(num: number) -> number
### startSketchAt ### startProfileAt
Start a sketch at a given point. Start a profile at a given point.
``` ```
startSketchAt(data: LineData) -> SketchGroup startProfileAt(data: LineData, plane: Plane) -> SketchGroup
``` ```
#### Arguments #### Arguments
@ -3147,6 +3213,40 @@ startSketchAt(data: LineData) -> SketchGroup
} | } |
[number] [number]
``` ```
* `plane`: `Plane` - A plane.
```
{
// The id of the plane.
id: uuid,
// Origin of the plane.
origin: {
x: number,
y: number,
z: number,
},
// Type for a plane.
value: string |
string,
// What should the planes X axis be?
xAxis: {
x: number,
y: number,
z: number,
},
// What should the planes Y axis be?
yAxis: {
x: number,
y: number,
z: number,
},
// The z-axis (normal).
zAxis: {
x: number,
y: number,
z: number,
},
}
```
#### Returns #### Returns
@ -3155,6 +3255,8 @@ startSketchAt(data: LineData) -> SketchGroup
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -3216,6 +3318,188 @@ startSketchAt(data: LineData) -> SketchGroup
### startSketchAt
Start a sketch at a given point on the 'XY' plane.
```
startSketchAt(data: LineData) -> SketchGroup
```
#### Arguments
* `data`: `LineData` - Data to draw a line.
```
{
// The tag.
tag: string,
// The to point.
to: [number],
} |
[number]
```
#### Returns
* `SketchGroup` - A sketch group is a collection of paths.
```
{
// The id of the sketch group.
id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group.
position: [number],
// The rotation of the sketch group.
rotation: [number],
// The starting path.
start: {
// The from point.
from: [number],
// The name of the path.
name: string,
// The to point.
to: [number],
},
// The paths in the sketch group.
value: [{
// The from point.
from: [number],
// The name of the path.
name: string,
// The to point.
to: [number],
type: string,
} |
{
// The from point.
from: [number],
// The name of the path.
name: string,
// The to point.
to: [number],
type: string,
// The x coordinate.
x: number,
} |
{
// The from point.
from: [number],
// The name of the path.
name: string,
// The to point.
to: [number],
type: string,
// The x coordinate.
x: number,
// The y coordinate.
y: number,
} |
{
// The from point.
from: [number],
// The name of the path.
name: string,
// The to point.
to: [number],
type: string,
}],
}
```
### startSketchOn
Start a sketch at a given point.
```
startSketchOn(data: PlaneData) -> Plane
```
#### Arguments
* `data`: `PlaneData` - Data for a plane.
```
string |
string |
string |
string |
string |
string |
{
plane: {
// Origin of the plane.
origin: {
x: number,
y: number,
z: number,
},
// What should the planes X axis be?
x_axis: {
x: number,
y: number,
z: number,
},
// What should the planes Y axis be?
y_axis: {
x: number,
y: number,
z: number,
},
// The z-axis (normal).
z_axis: {
x: number,
y: number,
z: number,
},
},
}
```
#### Returns
* `Plane` - A plane.
```
{
// The id of the plane.
id: uuid,
// Origin of the plane.
origin: {
x: number,
y: number,
z: number,
},
// Type for a plane.
value: string |
string,
// What should the planes X axis be?
xAxis: {
x: number,
y: number,
z: number,
},
// What should the planes Y axis be?
yAxis: {
x: number,
y: number,
z: number,
},
// The z-axis (normal).
zAxis: {
x: number,
y: number,
z: number,
},
}
```
### tan ### tan
Computes the tangent of a number (in radians). Computes the tangent of a number (in radians).
@ -3269,6 +3553,8 @@ tangentalArc(data: TangentalArcData, sketch_group: SketchGroup) -> SketchGroup
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -3335,6 +3621,8 @@ tangentalArc(data: TangentalArcData, sketch_group: SketchGroup) -> SketchGroup
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -3423,6 +3711,8 @@ tangentalArcTo(data: TangentalArcToData, sketch_group: SketchGroup) -> SketchGro
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -3489,6 +3779,8 @@ tangentalArcTo(data: TangentalArcToData, sketch_group: SketchGroup) -> SketchGro
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -3596,6 +3888,8 @@ number
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -3662,6 +3956,8 @@ number
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -3750,6 +4046,8 @@ number
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -3816,6 +4114,8 @@ number
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -3904,6 +4204,8 @@ number
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -3970,6 +4272,8 @@ number
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -4058,6 +4362,8 @@ number
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.
@ -4124,6 +4430,8 @@ number
{ {
// The id of the sketch group. // The id of the sketch group.
id: uuid, id: uuid,
// The plane id of the sketch group.
planeId: uuid,
// The position of the sketch group. // The position of the sketch group.
position: [number], position: [number],
// The rotation of the sketch group. // The rotation of the sketch group.

View File

@ -1,6 +1,6 @@
{ {
"name": "untitled-app", "name": "untitled-app",
"version": "0.9.5", "version": "0.10.0",
"private": true, "private": true,
"dependencies": { "dependencies": {
"@codemirror/autocomplete": "^6.9.0", "@codemirror/autocomplete": "^6.9.0",
@ -10,7 +10,7 @@
"@fortawesome/react-fontawesome": "^0.2.0", "@fortawesome/react-fontawesome": "^0.2.0",
"@headlessui/react": "^1.7.13", "@headlessui/react": "^1.7.13",
"@headlessui/tailwindcss": "^0.2.0", "@headlessui/tailwindcss": "^0.2.0",
"@kittycad/lib": "^0.0.40", "@kittycad/lib": "^0.0.43",
"@lezer/javascript": "^1.4.7", "@lezer/javascript": "^1.4.7",
"@open-rpc/client-js": "^1.8.1", "@open-rpc/client-js": "^1.8.1",
"@react-hook/resize-observer": "^1.2.6", "@react-hook/resize-observer": "^1.2.6",
@ -32,7 +32,7 @@
"fuse.js": "^6.6.2", "fuse.js": "^6.6.2",
"http-server": "^14.1.1", "http-server": "^14.1.1",
"json-rpc-2.0": "^1.6.0", "json-rpc-2.0": "^1.6.0",
"re-resizable": "^6.9.9", "re-resizable": "^6.9.11",
"react": "^18.2.0", "react": "^18.2.0",
"react-dom": "^18.2.0", "react-dom": "^18.2.0",
"react-hot-toast": "^2.4.1", "react-hot-toast": "^2.4.1",

6
src-tauri/Cargo.lock generated
View File

@ -3712,9 +3712,9 @@ dependencies = [
[[package]] [[package]]
name = "tauri" name = "tauri"
version = "1.5.0" version = "1.5.1"
source = "registry+https://github.com/rust-lang/crates.io-index" source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "72aee3277d0a0df01472cc704ab5934a51a1f25348838df17bfb3c5cb727880c" checksum = "0238c5063bf9613054149a1b6bce4935922e532b7d8211f36989a490a79806be"
dependencies = [ dependencies = [
"anyhow", "anyhow",
"base64 0.21.2", "base64 0.21.2",
@ -3828,7 +3828,7 @@ dependencies = [
[[package]] [[package]]
name = "tauri-plugin-fs-extra" name = "tauri-plugin-fs-extra"
version = "0.0.0" version = "0.0.0"
source = "git+https://github.com/tauri-apps/plugins-workspace?branch=v1#9b96996b5a90a6a57d587ce4312975f13a4d8bc2" source = "git+https://github.com/tauri-apps/plugins-workspace?branch=v1#fa32d1afa97f52f74d814c5619b8d95da3268e3e"
dependencies = [ dependencies = [
"log", "log",
"serde", "serde",

View File

@ -20,7 +20,7 @@ kittycad = "0.2.28"
oauth2 = "4.4.2" oauth2 = "4.4.2"
serde = { version = "1.0", features = ["derive"] } serde = { version = "1.0", features = ["derive"] }
serde_json = "1.0" serde_json = "1.0"
tauri = { version = "1.5.0", features = [ "os-all", "dialog-all", "fs-all", "http-request", "path-all", "shell-open", "shell-open-api", "updater", "devtools"] } tauri = { version = "1.5.1", features = [ "os-all", "dialog-all", "fs-all", "http-request", "path-all", "shell-open", "shell-open-api", "updater", "devtools"] }
tauri-plugin-fs-extra = { git = "https://github.com/tauri-apps/plugins-workspace", branch = "v1" } tauri-plugin-fs-extra = { git = "https://github.com/tauri-apps/plugins-workspace", branch = "v1" }
tokio = { version = "1.32.0", features = ["time"] } tokio = { version = "1.32.0", features = ["time"] }
toml = "0.8.2" toml = "0.8.2"

View File

@ -8,7 +8,7 @@
}, },
"package": { "package": {
"productName": "kittycad-modeling", "productName": "kittycad-modeling",
"version": "0.9.5" "version": "0.10.0"
}, },
"tauri": { "tauri": {
"allowlist": { "allowlist": {

View File

@ -1,9 +1,15 @@
import { render, screen } from '@testing-library/react' import { render, screen } from '@testing-library/react'
import { App } from './App' import { App } from './App'
import { describe, test, vi } from 'vitest' import { describe, test, vi } from 'vitest'
import { BrowserRouter } from 'react-router-dom' import {
Route,
RouterProvider,
createMemoryRouter,
createRoutesFromElements,
} from 'react-router-dom'
import { GlobalStateProvider } from './components/GlobalStateProvider' import { GlobalStateProvider } from './components/GlobalStateProvider'
import CommandBarProvider from 'components/CommandBar' import CommandBarProvider from 'components/CommandBar'
import { BROWSER_FILE_NAME } from 'Router'
let listener: ((rect: any) => void) | undefined = undefined let listener: ((rect: any) => void) | undefined = undefined
;(global as any).ResizeObserver = class ResizeObserver { ;(global as any).ResizeObserver = class ResizeObserver {
@ -24,7 +30,7 @@ describe('App tests', () => {
> >
return { return {
...actual, ...actual,
useParams: () => ({ id: 'new' }), useParams: () => ({ id: BROWSER_FILE_NAME }),
useLoaderData: () => ({ code: null }), useLoaderData: () => ({ code: null }),
} }
}) })
@ -41,12 +47,24 @@ describe('App tests', () => {
}) })
function TestWrap({ children }: { children: React.ReactNode }) { function TestWrap({ children }: { children: React.ReactNode }) {
// wrap in router and xState context // We have to use a memory router in the testing environment,
return ( // and we have to use the createMemoryRouter function instead of <MemoryRouter /> as of react-router v6.4:
<BrowserRouter> // https://reactrouter.com/en/6.16.0/routers/picking-a-router#using-v64-data-apis
<CommandBarProvider> const router = createMemoryRouter(
<GlobalStateProvider>{children}</GlobalStateProvider> createRoutesFromElements(
</CommandBarProvider> <Route
</BrowserRouter> path="/file/:id"
element={
<CommandBarProvider>
<GlobalStateProvider>{children}</GlobalStateProvider>
</CommandBarProvider>
}
/>
),
{
initialEntries: ['/file/new'],
initialIndex: 0,
}
) )
return <RouterProvider router={router} />
} }

View File

@ -94,6 +94,8 @@ export const paths = {
) as typeof onboardingPaths, ) as typeof onboardingPaths,
} }
export const BROWSER_FILE_NAME = 'new'
export type IndexLoaderData = { export type IndexLoaderData = {
code: string | null code: string | null
project?: ProjectWithEntryPointMetadata project?: ProjectWithEntryPointMetadata
@ -129,7 +131,9 @@ const router = createBrowserRouter(
{ {
path: paths.INDEX, path: paths.INDEX,
loader: () => loader: () =>
isTauri() ? redirect(paths.HOME) : redirect(paths.FILE + '/new'), isTauri()
? redirect(paths.HOME)
: redirect(paths.FILE + '/' + BROWSER_FILE_NAME),
errorElement: <ErrorPage />, errorElement: <ErrorPage />,
}, },
{ {
@ -167,7 +171,7 @@ const router = createBrowserRouter(
) )
} }
if (params.id && params.id !== 'new') { if (params.id && params.id !== BROWSER_FILE_NAME) {
// Note that PROJECT_ENTRYPOINT is hardcoded until we support multiple files // Note that PROJECT_ENTRYPOINT is hardcoded until we support multiple files
const code = await readTextFile(params.id + '/' + PROJECT_ENTRYPOINT) const code = await readTextFile(params.id + '/' + PROJECT_ENTRYPOINT)
const entrypoint_metadata = await metadata( const entrypoint_metadata = await metadata(
@ -212,7 +216,7 @@ const router = createBrowserRouter(
), ),
loader: async () => { loader: async () => {
if (!isTauri()) { if (!isTauri()) {
return redirect(paths.FILE + '/new') return redirect(paths.FILE + '/' + BROWSER_FILE_NAME)
} }
const fetchedStorage = localStorage?.getItem(SETTINGS_PERSIST_KEY) const fetchedStorage = localStorage?.getItem(SETTINGS_PERSIST_KEY)
const persistedSettings = JSON.parse(fetchedStorage || '{}') as Partial< const persistedSettings = JSON.parse(fetchedStorage || '{}') as Partial<

View File

@ -10,7 +10,7 @@ import { SetHorzVertDistance } from './components/Toolbar/SetHorzVertDistance'
import { SetAngleLength } from './components/Toolbar/setAngleLength' import { SetAngleLength } from './components/Toolbar/setAngleLength'
import { SetAbsDistance } from './components/Toolbar/SetAbsDistance' import { SetAbsDistance } from './components/Toolbar/SetAbsDistance'
import { SetAngleBetween } from './components/Toolbar/SetAngleBetween' import { SetAngleBetween } from './components/Toolbar/SetAngleBetween'
import { Fragment, useEffect } from 'react' import { Fragment, WheelEvent, useRef } from 'react'
import { FontAwesomeIcon } from '@fortawesome/react-fontawesome' import { FontAwesomeIcon } from '@fortawesome/react-fontawesome'
import { faSearch, faX } from '@fortawesome/free-solid-svg-icons' import { faSearch, faX } from '@fortawesome/free-solid-svg-icons'
import { Popover, Transition } from '@headlessui/react' import { Popover, Transition } from '@headlessui/react'
@ -62,10 +62,24 @@ export const Toolbar = () => {
executeAst: s.executeAst, executeAst: s.executeAst,
})) }))
useAppMode() useAppMode()
const toolbarButtonsRef = useRef<HTMLSpanElement>(null)
function handleToolbarButtonsWheelEvent(ev: WheelEvent<HTMLSpanElement>) {
const span = toolbarButtonsRef.current
if (!span) {
return
}
span.scrollLeft = span.scrollLeft += ev.deltaY
}
function ToolbarButtons({ className }: React.HTMLAttributes<HTMLElement>) { function ToolbarButtons({ className }: React.HTMLAttributes<HTMLElement>) {
return ( return (
<span className={styles.toolbarButtons + ' ' + className}> <span
ref={toolbarButtonsRef}
onWheel={handleToolbarButtonsWheelEvent}
className={styles.toolbarButtons + ' ' + className}
>
{guiMode.mode === 'default' && ( {guiMode.mode === 'default' && (
<button <button
onClick={() => { onClick={() => {

View File

@ -91,11 +91,14 @@ export function useCalc({
newVariableInsertIndex: number newVariableInsertIndex: number
setNewVariableName: (a: string) => void setNewVariableName: (a: string) => void
} { } {
const { ast, programMemory, selectionRange } = useStore((s) => ({ const { ast, programMemory, selectionRange, defaultPlanes } = useStore(
ast: s.ast, (s) => ({
programMemory: s.programMemory, ast: s.ast,
selectionRange: s.selectionRanges.codeBasedSelections[0].range, programMemory: s.programMemory,
})) selectionRange: s.selectionRanges.codeBasedSelections[0].range,
defaultPlanes: s.defaultPlanes,
})
)
const inputRef = useRef<HTMLInputElement>(null) const inputRef = useRef<HTMLInputElement>(null)
const [availableVarInfo, setAvailableVarInfo] = useState< const [availableVarInfo, setAvailableVarInfo] = useState<
ReturnType<typeof findAllPreviousVariables> ReturnType<typeof findAllPreviousVariables>
@ -143,19 +146,22 @@ export function useCalc({
availableVarInfo.variables.forEach(({ key, value }) => { availableVarInfo.variables.forEach(({ key, value }) => {
_programMem.root[key] = { type: 'userVal', value, __meta: [] } _programMem.root[key] = { type: 'userVal', value, __meta: [] }
}) })
executor(ast, _programMem, engineCommandManager).then((programMemory) => { if (!defaultPlanes) return
const resultDeclaration = ast.body.find( executor(ast, _programMem, engineCommandManager, defaultPlanes!).then(
(a) => (programMemory) => {
a.type === 'VariableDeclaration' && const resultDeclaration = ast.body.find(
a.declarations?.[0]?.id?.name === '__result__' (a) =>
) a.type === 'VariableDeclaration' &&
const init = a.declarations?.[0]?.id?.name === '__result__'
resultDeclaration?.type === 'VariableDeclaration' && )
resultDeclaration?.declarations?.[0]?.init const init =
const result = programMemory?.root?.__result__?.value resultDeclaration?.type === 'VariableDeclaration' &&
setCalcResult(typeof result === 'number' ? String(result) : 'NAN') resultDeclaration?.declarations?.[0]?.init
init && setValueNode(init) const result = programMemory?.root?.__result__?.value
}) setCalcResult(typeof result === 'number' ? String(result) : 'NAN')
init && setValueNode(init)
}
)
} catch (e) { } catch (e) {
setCalcResult('NAN') setCalcResult('NAN')
setValueNode(null) setValueNode(null)

View File

@ -47,11 +47,9 @@ export const ErrorPage = () => {
Clear storage Clear storage
</ActionButton> </ActionButton>
<ActionButton <ActionButton
Element="link" Element="externalLink"
icon={{ icon: faBug }} icon={{ icon: faBug }}
target="_blank" to="https://github.com/KittyCAD/modeling-app/issues/new"
rel="noopener noreferrer"
to="https://discord.com/channels/915388055236509727/1138967922614743060"
> >
Report Bug Report Bug
</ActionButton> </ActionButton>

View File

@ -14,12 +14,14 @@ describe('processMemory', () => {
} }
const otherVar = myFn(5) const otherVar = myFn(5)
const theExtrude = startSketchAt([0, 0]) const theExtrude = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> lineTo([-2.4, myVar], %) |> lineTo([-2.4, myVar], %)
|> lineTo([-0.76, otherVar], %) |> lineTo([-0.76, otherVar], %)
|> extrude(4, %) |> extrude(4, %)
const theSketch = startSketchAt([0, 0]) const theSketch = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> lineTo([-3.35, 0.17], %) |> lineTo([-3.35, 0.17], %)
|> lineTo([0.98, 5.16], %) |> lineTo([0.98, 5.16], %)
|> lineTo([2.15, 4.32], %) |> lineTo([2.15, 4.32], %)

View File

@ -46,6 +46,8 @@ export const Stream = ({ className = '' }) => {
updateAst, updateAst,
setGuiMode, setGuiMode,
programMemory, programMemory,
defaultPlanes,
currentPlane,
} = useStore((s) => ({ } = useStore((s) => ({
mediaStream: s.mediaStream, mediaStream: s.mediaStream,
setButtonDownInStream: s.setButtonDownInStream, setButtonDownInStream: s.setButtonDownInStream,
@ -59,6 +61,8 @@ export const Stream = ({ className = '' }) => {
updateAst: s.updateAst, updateAst: s.updateAst,
setGuiMode: s.setGuiMode, setGuiMode: s.setGuiMode,
programMemory: s.programMemory, programMemory: s.programMemory,
defaultPlanes: s.defaultPlanes,
currentPlane: s.currentPlane,
})) }))
const { const {
settings: { settings: {
@ -252,10 +256,26 @@ export const Stream = ({ className = '' }) => {
let engineId = guiMode.pathId let engineId = guiMode.pathId
// Get the current plane string for plane we are on.
let currentPlaneString = ''
if (currentPlane === defaultPlanes?.xy) {
currentPlaneString = 'XY'
} else if (currentPlane === defaultPlanes?.yz) {
currentPlaneString = 'YZ'
} else if (currentPlane === defaultPlanes?.xz) {
currentPlaneString = 'XZ'
}
// Do not supporting editing/moving lines on a non-default plane.
// Eventually we can support this but for now we will just throw an
// error.
if (currentPlaneString === '') return
const updatedAst: Program = await modifyAstForSketch( const updatedAst: Program = await modifyAstForSketch(
engineCommandManager, engineCommandManager,
ast, ast,
variableName, variableName,
currentPlaneString,
engineId engineId
) )
@ -282,8 +302,46 @@ export const Stream = ({ className = '' }) => {
}) })
const coords: { x: number; y: number }[] = const coords: { x: number; y: number }[] =
curve.data.data.control_points curve.data.data.control_points
// We need the normal for the plane we are on.
const plane = await engineCommandManager.sendSceneCommand({
type: 'modeling_cmd_req',
cmd_id: uuidv4(),
cmd: {
type: 'get_sketch_mode_plane',
},
})
const z_axis = plane.data.data.z_axis
// Get the current axis.
let currentAxis: 'xy' | 'xz' | 'yz' | '-xy' | '-xz' | '-yz' | null =
null
if (currentPlane === defaultPlanes?.xy) {
if (z_axis.z === -1) {
currentAxis = '-xy'
} else {
currentAxis = 'xy'
}
} else if (currentPlane === defaultPlanes?.yz) {
if (z_axis.x === -1) {
currentAxis = '-yz'
} else {
currentAxis = 'yz'
}
} else if (currentPlane === defaultPlanes?.xz) {
if (z_axis.y === -1) {
currentAxis = '-xz'
} else {
currentAxis = 'xz'
}
}
// Do not support starting a new sketch on a non-default plane.
if (!currentAxis) return
const _addStartSketch = addStartSketch( const _addStartSketch = addStartSketch(
ast, ast,
currentAxis,
[roundOff(coords[0].x), roundOff(coords[0].y)], [roundOff(coords[0].x), roundOff(coords[0].y)],
[ [
roundOff(coords[1].x - coords[0].x), roundOff(coords[1].x - coords[0].x),

View File

@ -1,6 +1,11 @@
import { fireEvent, render, screen } from '@testing-library/react' import { fireEvent, render, screen } from '@testing-library/react'
import UserSidebarMenu from './UserSidebarMenu' import UserSidebarMenu from './UserSidebarMenu'
import { BrowserRouter } from 'react-router-dom' import {
Route,
RouterProvider,
createMemoryRouter,
createRoutesFromElements,
} from 'react-router-dom'
import { Models } from '@kittycad/lib' import { Models } from '@kittycad/lib'
import { GlobalStateProvider } from './GlobalStateProvider' import { GlobalStateProvider } from './GlobalStateProvider'
import CommandBarProvider from './CommandBar' import CommandBarProvider from './CommandBar'
@ -93,11 +98,24 @@ describe('UserSidebarMenu tests', () => {
function TestWrap({ children }: { children: React.ReactNode }) { function TestWrap({ children }: { children: React.ReactNode }) {
// wrap in router and xState context // wrap in router and xState context
return ( // We have to use a memory router in the testing environment,
<BrowserRouter> // and we have to use the createMemoryRouter function instead of <MemoryRouter /> as of react-router v6.4:
<CommandBarProvider> // https://reactrouter.com/en/6.16.0/routers/picking-a-router#using-v64-data-apis
<GlobalStateProvider>{children}</GlobalStateProvider> const router = createMemoryRouter(
</CommandBarProvider> createRoutesFromElements(
</BrowserRouter> <Route
path="/file/:id"
element={
<CommandBarProvider>
<GlobalStateProvider>{children}</GlobalStateProvider>
</CommandBarProvider>
}
/>
),
{
initialEntries: ['/file/new'],
initialIndex: 0,
}
) )
return <RouterProvider router={router} />
} }

View File

@ -10,14 +10,15 @@ import { faGithub } from '@fortawesome/free-brands-svg-icons'
import { useLocation, useNavigate } from 'react-router-dom' import { useLocation, useNavigate } from 'react-router-dom'
import { Fragment, useState } from 'react' import { Fragment, useState } from 'react'
import { paths } from '../Router' import { paths } from '../Router'
import makeUrlPathRelative from '../lib/makeUrlPathRelative'
import { Models } from '@kittycad/lib' import { Models } from '@kittycad/lib'
import { useGlobalStateContext } from 'hooks/useGlobalStateContext' import { useGlobalStateContext } from 'hooks/useGlobalStateContext'
import { useAbsoluteFilePath } from 'hooks/useAbsoluteFilePath'
type User = Models['User_type'] type User = Models['User_type']
const UserSidebarMenu = ({ user }: { user?: User }) => { const UserSidebarMenu = ({ user }: { user?: User }) => {
const location = useLocation() const location = useLocation()
const filePath = useAbsoluteFilePath()
const displayedName = getDisplayName(user) const displayedName = getDisplayName(user)
const [imageLoadFailed, setImageLoadFailed] = useState(false) const [imageLoadFailed, setImageLoadFailed] = useState(false)
const navigate = useNavigate() const navigate = useNavigate()
@ -132,11 +133,10 @@ const UserSidebarMenu = ({ user }: { user?: User }) => {
// since /settings is a nested route the sidebar doesn't close // since /settings is a nested route the sidebar doesn't close
// automatically when navigating to it // automatically when navigating to it
close() close()
navigate( const targetPath = location.pathname.includes(paths.FILE)
(location.pathname.endsWith('/') ? filePath + paths.SETTINGS
? location.pathname.slice(0, -1) : paths.HOME + paths.SETTINGS
: location.pathname) + paths.SETTINGS navigate(targetPath)
)
}} }}
> >
Settings Settings

View File

@ -0,0 +1,12 @@
import { BROWSER_FILE_NAME, IndexLoaderData, paths } from 'Router'
import { useRouteLoaderData } from 'react-router-dom'
export function useAbsoluteFilePath() {
const routeData = useRouteLoaderData(paths.FILE) as IndexLoaderData
return (
paths.FILE +
'/' +
encodeURIComponent(routeData?.project?.path || BROWSER_FILE_NAME)
)
}

View File

@ -2,30 +2,37 @@
// Once we have xState this should be removed // Once we have xState this should be removed
import { useStore, Selections } from 'useStore' import { useStore, Selections } from 'useStore'
import { useEffect, useState } from 'react' import { useEffect } from 'react'
import { v4 as uuidv4 } from 'uuid' import { v4 as uuidv4 } from 'uuid'
import { ArtifactMap, EngineCommandManager } from 'lang/std/engineConnection' import { ArtifactMap, EngineCommandManager } from 'lang/std/engineConnection'
import { Models } from '@kittycad/lib/dist/types/src' import { Models } from '@kittycad/lib/dist/types/src'
import { isReducedMotion } from 'lang/util' import { isReducedMotion } from 'lang/util'
import { isOverlap } from 'lib/utils' import { isOverlap } from 'lib/utils'
import { engineCommandManager } from '../lang/std/engineConnection' import { engineCommandManager } from '../lang/std/engineConnection'
import { DefaultPlanes } from '../wasm-lib/kcl/bindings/DefaultPlanes'
interface DefaultPlanes { import { getNodeFromPath } from '../lang/queryAst'
xy: string import { CallExpression, PipeExpression } from '../lang/wasm'
// TODO re-enable
// yz: string
// xz: string
}
export function useAppMode() { export function useAppMode() {
const { guiMode, setGuiMode, selectionRanges, selectionRangeTypeMap } = const {
useStore((s) => ({ guiMode,
guiMode: s.guiMode, setGuiMode,
setGuiMode: s.setGuiMode, selectionRanges,
selectionRanges: s.selectionRanges, selectionRangeTypeMap,
selectionRangeTypeMap: s.selectionRangeTypeMap, defaultPlanes,
})) setDefaultPlanes,
const [defaultPlanes, setDefaultPlanes] = useState<DefaultPlanes | null>(null) setCurrentPlane,
ast,
} = useStore((s) => ({
guiMode: s.guiMode,
setGuiMode: s.setGuiMode,
selectionRanges: s.selectionRanges,
selectionRangeTypeMap: s.selectionRangeTypeMap,
defaultPlanes: s.defaultPlanes,
setDefaultPlanes: s.setDefaultPlanes,
setCurrentPlane: s.setCurrentPlane,
ast: s.ast,
}))
useEffect(() => { useEffect(() => {
if ( if (
guiMode.mode === 'sketch' && guiMode.mode === 'sketch' &&
@ -35,8 +42,10 @@ export function useAppMode() {
const createAndShowPlanes = async () => { const createAndShowPlanes = async () => {
let localDefaultPlanes: DefaultPlanes let localDefaultPlanes: DefaultPlanes
if (!defaultPlanes) { if (!defaultPlanes) {
localDefaultPlanes = await initDefaultPlanes(engineCommandManager) const newDefaultPlanes = await initDefaultPlanes(engineCommandManager)
setDefaultPlanes(localDefaultPlanes) if (!newDefaultPlanes) return
setDefaultPlanes(newDefaultPlanes)
localDefaultPlanes = newDefaultPlanes
} else { } else {
localDefaultPlanes = defaultPlanes localDefaultPlanes = defaultPlanes
} }
@ -52,20 +61,52 @@ export function useAppMode() {
const enableSketchMode = async () => { const enableSketchMode = async () => {
let localDefaultPlanes: DefaultPlanes let localDefaultPlanes: DefaultPlanes
if (!defaultPlanes) { if (!defaultPlanes) {
localDefaultPlanes = await initDefaultPlanes(engineCommandManager) const newDefaultPlanes = await initDefaultPlanes(engineCommandManager)
setDefaultPlanes(localDefaultPlanes) if (!newDefaultPlanes) return
setDefaultPlanes(newDefaultPlanes)
localDefaultPlanes = newDefaultPlanes
} else { } else {
localDefaultPlanes = defaultPlanes localDefaultPlanes = defaultPlanes
} }
setDefaultPlanesHidden(engineCommandManager, localDefaultPlanes, true) setDefaultPlanesHidden(engineCommandManager, localDefaultPlanes, true)
// TODO figure out the plane to use based on the sketch
// maybe it's easier to make a new plane than rely on the defaults const pipeExpression = getNodeFromPath<PipeExpression>(
ast,
guiMode.pathToNode,
'PipeExpression'
).node
if (pipeExpression.type !== 'PipeExpression') return /// bad bad bad
const sketchCallExpression = pipeExpression.body.find(
(e) =>
e.type === 'CallExpression' && e.callee.name === 'startSketchOn'
) as CallExpression
if (!sketchCallExpression) return // also bad bad bad
const firstArg = sketchCallExpression.arguments[0]
let planeId = ''
if (firstArg.type === 'Literal' && firstArg.value) {
const planeStrCleaned = firstArg.value
.toString()
.toLowerCase()
.replace('-', '')
if (
planeStrCleaned === 'xy' ||
planeStrCleaned === 'xz' ||
planeStrCleaned === 'yz'
) {
planeId = localDefaultPlanes[planeStrCleaned]
}
}
if (!planeId) return // they are on some non default plane, which we don't support yet
setCurrentPlane(planeId)
await engineCommandManager.sendSceneCommand({ await engineCommandManager.sendSceneCommand({
type: 'modeling_cmd_req', type: 'modeling_cmd_req',
cmd_id: uuidv4(), cmd_id: uuidv4(),
cmd: { cmd: {
type: 'sketch_mode_enable', type: 'sketch_mode_enable',
plane_id: localDefaultPlanes.xy, plane_id: planeId,
ortho: true, ortho: true,
animated: !isReducedMotion(), animated: !isReducedMotion(),
}, },
@ -139,6 +180,7 @@ export function useAppMode() {
// user clicked something else in the scene // user clicked something else in the scene
return return
} }
setCurrentPlane(data.entity_id)
const sketchModeResponse = await engineCommandManager.sendSceneCommand({ const sketchModeResponse = await engineCommandManager.sendSceneCommand({
type: 'modeling_cmd_req', type: 'modeling_cmd_req',
cmd_id: uuidv4(), cmd_id: uuidv4(),
@ -194,10 +236,12 @@ async function createPlane(
x_axis, x_axis,
y_axis, y_axis,
color, color,
hidden,
}: { }: {
x_axis: Models['Point3d_type'] x_axis: Models['Point3d_type']
y_axis: Models['Point3d_type'] y_axis: Models['Point3d_type']
color: Models['Color_type'] color: Models['Color_type']
hidden: boolean
} }
) { ) {
const planeId = uuidv4() const planeId = uuidv4()
@ -210,6 +254,7 @@ async function createPlane(
x_axis, x_axis,
y_axis, y_axis,
clobber: false, clobber: false,
hide: hidden,
}, },
cmd_id: planeId, cmd_id: planeId,
}) })
@ -225,61 +270,82 @@ async function createPlane(
return planeId return planeId
} }
function setDefaultPlanesHidden( export function setDefaultPlanesHidden(
engineCommandManager: EngineCommandManager, engineCommandManager: EngineCommandManager,
defaultPlanes: DefaultPlanes, defaultPlanes: DefaultPlanes,
hidden: boolean hidden: boolean
) { ) {
Object.values(defaultPlanes).forEach((planeId) => { Object.values(defaultPlanes).forEach((planeId) => {
engineCommandManager.sendSceneCommand({ hidePlane(engineCommandManager, planeId, hidden)
type: 'modeling_cmd_req',
cmd_id: uuidv4(),
cmd: {
type: 'object_visible',
object_id: planeId,
hidden: hidden,
},
})
}) })
} }
async function initDefaultPlanes( function hidePlane(
engineCommandManager: EngineCommandManager engineCommandManager: EngineCommandManager,
): Promise<DefaultPlanes> { planeId: string,
hidden: boolean
) {
engineCommandManager.sendSceneCommand({
type: 'modeling_cmd_req',
cmd_id: uuidv4(),
cmd: {
type: 'object_visible',
object_id: planeId,
hidden: hidden,
},
})
}
export async function initDefaultPlanes(
engineCommandManager: EngineCommandManager,
hidePlanes?: boolean
): Promise<DefaultPlanes | null> {
if (!engineCommandManager.engineConnection?.isReady()) {
return null
}
const xy = await createPlane(engineCommandManager, { const xy = await createPlane(engineCommandManager, {
x_axis: { x: 1, y: 0, z: 0 }, x_axis: { x: 1, y: 0, z: 0 },
y_axis: { x: 0, y: 1, z: 0 }, y_axis: { x: 0, y: 1, z: 0 },
color: { r: 0.7, g: 0.28, b: 0.28, a: 0.4 }, color: { r: 0.7, g: 0.28, b: 0.28, a: 0.4 },
hidden: hidePlanes ? true : false,
}) })
// TODO re-enable if (hidePlanes) {
// const yz = createPlane(engineCommandManager, { hidePlane(engineCommandManager, xy, true)
// x_axis: { x: 0, y: 1, z: 0 }, }
// y_axis: { x: 0, y: 0, z: 1 }, const yz = await createPlane(engineCommandManager, {
// color: { r: 0.28, g: 0.7, b: 0.28, a: 0.4 }, x_axis: { x: 0, y: 1, z: 0 },
// }) y_axis: { x: 0, y: 0, z: 1 },
// const xz = createPlane(engineCommandManager, { color: { r: 0.28, g: 0.7, b: 0.28, a: 0.4 },
// x_axis: { x: 1, y: 0, z: 0 }, hidden: hidePlanes ? true : false,
// y_axis: { x: 0, y: 0, z: 1 }, })
// color: { r: 0.28, g: 0.28, b: 0.7, a: 0.4 }, if (hidePlanes) {
// }) hidePlane(engineCommandManager, yz, true)
return { xy } }
const xz = await createPlane(engineCommandManager, {
x_axis: { x: 1, y: 0, z: 0 },
y_axis: { x: 0, y: 0, z: 1 },
color: { r: 0.28, g: 0.28, b: 0.7, a: 0.4 },
hidden: hidePlanes ? true : false,
})
return { xy, yz, xz }
} }
function isCursorInSketchCommandRange( function isCursorInSketchCommandRange(
artifactMap: ArtifactMap, artifactMap: ArtifactMap,
selectionRanges: Selections selectionRanges: Selections
): string | false { ): string | false {
const overlapingEntries = Object.entries(artifactMap || {}).filter( const overlapingEntries: [string, ArtifactMap[string]][] = Object.entries(
([id, artifact]) => artifactMap
selectionRanges.codeBasedSelections.some( ).filter(([id, artifact]: [string, ArtifactMap[string]]) =>
(selection) => selectionRanges.codeBasedSelections.some(
Array.isArray(selection?.range) && (selection) =>
Array.isArray(artifact?.range) && Array.isArray(selection?.range) &&
isOverlap(selection.range, artifact.range) && Array.isArray(artifact?.range) &&
(artifact.commandType === 'start_path' || isOverlap(selection.range, artifact.range) &&
artifact.commandType === 'extend_path' || (artifact.commandType === 'start_path' ||
artifact.commandType === 'close_path') artifact.commandType === 'extend_path' ||
) artifact.commandType === 'close_path')
)
) )
return overlapingEntries.length && overlapingEntries[0][1].parentId return overlapingEntries.length && overlapingEntries[0][1].parentId
? overlapingEntries[0][1].parentId ? overlapingEntries[0][1].parentId

View File

@ -3,7 +3,6 @@ import { _executor } from '../lang/wasm'
import { useStore } from '../useStore' import { useStore } from '../useStore'
import { engineCommandManager } from '../lang/std/engineConnection' import { engineCommandManager } from '../lang/std/engineConnection'
import { deferExecution } from 'lib/utils' import { deferExecution } from 'lib/utils'
import { v4 as uuidv4 } from 'uuid'
export function useSetupEngineManager( export function useSetupEngineManager(
streamRef: React.RefObject<HTMLDivElement>, streamRef: React.RefObject<HTMLDivElement>,

View File

@ -7,7 +7,8 @@ describe('testing artifacts', () => {
// Enable rotations #152 // Enable rotations #152
test('sketch artifacts', async () => { test('sketch artifacts', async () => {
const code = ` const code = `
const mySketch001 = startSketchAt([0, 0]) const mySketch001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> lineTo([-1.59, -1.54], %) |> lineTo([-1.59, -1.54], %)
|> lineTo([0.46, -5.82], %) |> lineTo([0.46, -5.82], %)
// |> rx(45, %) // |> rx(45, %)
@ -27,7 +28,7 @@ show(mySketch001)`
name: '', name: '',
__geoMeta: { __geoMeta: {
id: expect.any(String), id: expect.any(String),
sourceRange: [21, 42], sourceRange: [46, 71],
}, },
}, },
value: [ value: [
@ -37,7 +38,7 @@ show(mySketch001)`
to: [-1.59, -1.54], to: [-1.59, -1.54],
from: [0, 0], from: [0, 0],
__geoMeta: { __geoMeta: {
sourceRange: [48, 73], sourceRange: [77, 102],
id: expect.any(String), id: expect.any(String),
}, },
}, },
@ -47,7 +48,7 @@ show(mySketch001)`
from: [-1.59, -1.54], from: [-1.59, -1.54],
name: '', name: '',
__geoMeta: { __geoMeta: {
sourceRange: [79, 103], sourceRange: [108, 132],
id: expect.any(String), id: expect.any(String),
}, },
}, },
@ -55,14 +56,16 @@ show(mySketch001)`
position: [0, 0, 0], position: [0, 0, 0],
rotation: [0, 0, 0, 1], rotation: [0, 0, 0, 1],
id: expect.any(String), id: expect.any(String),
__meta: [{ sourceRange: [21, 42] }], planeId: expect.any(String),
__meta: [{ sourceRange: [46, 71] }],
}, },
]) ])
}) })
test('extrude artifacts', async () => { test('extrude artifacts', async () => {
// Enable rotations #152 // Enable rotations #152
const code = ` const code = `
const mySketch001 = startSketchAt([0, 0]) const mySketch001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> lineTo([-1.59, -1.54], %) |> lineTo([-1.59, -1.54], %)
|> lineTo([0.46, -5.82], %) |> lineTo([0.46, -5.82], %)
// |> rx(45, %) // |> rx(45, %)
@ -82,7 +85,7 @@ show(mySketch001)`
height: 2, height: 2,
position: [0, 0, 0], position: [0, 0, 0],
rotation: [0, 0, 0, 1], rotation: [0, 0, 0, 1],
__meta: [{ sourceRange: [21, 42] }], __meta: [{ sourceRange: [46, 71] }],
}, },
]) ])
}) })
@ -90,7 +93,8 @@ show(mySketch001)`
// Enable rotations #152 // Enable rotations #152
// TODO #153 in order for getExtrudeWallTransform to work we need to query the engine for the location of a face. // TODO #153 in order for getExtrudeWallTransform to work we need to query the engine for the location of a face.
const code = ` const code = `
const sk1 = startSketchAt([0, 0]) const sk1 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> lineTo([-2.5, 0], %) |> lineTo([-2.5, 0], %)
|> lineTo({ to: [0, 10], tag: "p" }, %) |> lineTo({ to: [0, 10], tag: "p" }, %)
|> lineTo([2.5, 0], %) |> lineTo([2.5, 0], %)
@ -99,7 +103,8 @@ const sk1 = startSketchAt([0, 0])
// |> ry(5, %) // |> ry(5, %)
const theExtrude = extrude(2, sk1) const theExtrude = extrude(2, sk1)
// const theTransf = getExtrudeWallTransform('p', theExtrude) // const theTransf = getExtrudeWallTransform('p', theExtrude)
const sk2 = startSketchAt([0, 0]) const sk2 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> lineTo([-2.5, 0], %) |> lineTo([-2.5, 0], %)
|> lineTo({ to: [0, 3], tag: "p" }, %) |> lineTo({ to: [0, 3], tag: "p" }, %)
|> lineTo([2.5, 0], %) |> lineTo([2.5, 0], %)
@ -122,7 +127,7 @@ show(theExtrude, sk2)`
height: 2, height: 2,
position: [0, 0, 0], position: [0, 0, 0],
rotation: [0, 0, 0, 1], rotation: [0, 0, 0, 1],
__meta: [{ sourceRange: [13, 34] }], __meta: [{ sourceRange: [38, 63] }],
}, },
{ {
type: 'ExtrudeGroup', type: 'ExtrudeGroup',
@ -131,7 +136,7 @@ show(theExtrude, sk2)`
height: 2, height: 2,
position: [0, 0, 0], position: [0, 0, 0],
rotation: [0, 0, 0, 1], rotation: [0, 0, 0, 1],
__meta: [{ sourceRange: [302, 323] }], __meta: [{ sourceRange: [356, 381] }],
}, },
]) ])
}) })

View File

@ -41,7 +41,8 @@ const newVar = myVar + 1`
expect(root.magicNum.value).toBe(69) expect(root.magicNum.value).toBe(69)
}) })
it('sketch declaration', async () => { it('sketch declaration', async () => {
let code = `const mySketch = startSketchAt([0,0]) let code = `const mySketch = startSketchOn('XY')
|> startProfileAt([0,0], %)
|> lineTo({to: [0,2], tag: "myPath"}, %) |> lineTo({to: [0,2], tag: "myPath"}, %)
|> lineTo([2,3], %) |> lineTo([2,3], %)
|> lineTo({ to: [5,-1], tag: "rightPath" }, %) |> lineTo({ to: [5,-1], tag: "rightPath" }, %)
@ -57,7 +58,7 @@ show(mySketch)
to: [0, 2], to: [0, 2],
from: [0, 0], from: [0, 0],
__geoMeta: { __geoMeta: {
sourceRange: [43, 80], sourceRange: [72, 109],
id: expect.any(String), id: expect.any(String),
}, },
name: 'myPath', name: 'myPath',
@ -68,7 +69,7 @@ show(mySketch)
from: [0, 2], from: [0, 2],
name: '', name: '',
__geoMeta: { __geoMeta: {
sourceRange: [86, 102], sourceRange: [115, 131],
id: expect.any(String), id: expect.any(String),
}, },
}, },
@ -77,7 +78,7 @@ show(mySketch)
to: [5, -1], to: [5, -1],
from: [2, 3], from: [2, 3],
__geoMeta: { __geoMeta: {
sourceRange: [108, 151], sourceRange: [137, 180],
id: expect.any(String), id: expect.any(String),
}, },
name: 'rightPath', name: 'rightPath',
@ -87,8 +88,8 @@ show(mySketch)
expect(_return).toEqual([ expect(_return).toEqual([
{ {
type: 'Identifier', type: 'Identifier',
start: 174, start: 203,
end: 182, end: 211,
name: 'mySketch', name: 'mySketch',
}, },
]) ])
@ -132,7 +133,8 @@ show(mySketch)
it('execute pipe sketch into call expression', async () => { it('execute pipe sketch into call expression', async () => {
// Enable rotations #152 // Enable rotations #152
const code = [ const code = [
'const mySk1 = startSketchAt([0,0])', "const mySk1 = startSketchOn('XY')",
' |> startProfileAt([0,0], %)',
' |> lineTo([1,1], %)', ' |> lineTo([1,1], %)',
' |> lineTo({to: [0, 1], tag: "myPath"}, %)', ' |> lineTo({to: [0, 1], tag: "myPath"}, %)',
' |> lineTo([1,1], %)', ' |> lineTo([1,1], %)',
@ -147,7 +149,7 @@ show(mySketch)
name: '', name: '',
__geoMeta: { __geoMeta: {
id: expect.any(String), id: expect.any(String),
sourceRange: [14, 34], sourceRange: [39, 63],
}, },
}, },
value: [ value: [
@ -157,7 +159,7 @@ show(mySketch)
from: [0, 0], from: [0, 0],
name: '', name: '',
__geoMeta: { __geoMeta: {
sourceRange: [40, 56], sourceRange: [69, 85],
id: expect.any(String), id: expect.any(String),
}, },
}, },
@ -166,7 +168,7 @@ show(mySketch)
to: [0, 1], to: [0, 1],
from: [1, 1], from: [1, 1],
__geoMeta: { __geoMeta: {
sourceRange: [62, 100], sourceRange: [91, 129],
id: expect.any(String), id: expect.any(String),
}, },
name: 'myPath', name: 'myPath',
@ -177,7 +179,7 @@ show(mySketch)
from: [0, 1], from: [0, 1],
name: '', name: '',
__geoMeta: { __geoMeta: {
sourceRange: [106, 122], sourceRange: [135, 151],
id: expect.any(String), id: expect.any(String),
}, },
}, },
@ -185,7 +187,8 @@ show(mySketch)
position: [0, 0, 0], position: [0, 0, 0],
rotation: [0, 0, 0, 1], rotation: [0, 0, 0, 1],
id: expect.any(String), id: expect.any(String),
__meta: [{ sourceRange: [14, 34] }], planeId: expect.any(String),
__meta: [{ sourceRange: [39, 63] }],
}) })
}) })
it('execute array expression', async () => { it('execute array expression', async () => {
@ -329,7 +332,8 @@ describe('testing math operators', () => {
}) })
it('with unaryExpression in ArrayExpression in CallExpression, checking nothing funny happens when used in a sketch', async () => { it('with unaryExpression in ArrayExpression in CallExpression, checking nothing funny happens when used in a sketch', async () => {
const code = [ const code = [
'const part001 = startSketchAt([0, 0])', "const part001 = startSketchOn('XY')",
' |> startProfileAt([0, 0], %)',
'|> line([-2.21, -legLen(5, min(3, 999))], %)', '|> line([-2.21, -legLen(5, min(3, 999))], %)',
].join('\n') ].join('\n')
const { root } = await exe(code) const { root } = await exe(code)
@ -341,7 +345,8 @@ describe('testing math operators', () => {
it('test that % substitution feeds down CallExp->ArrExp->UnaryExp->CallExp', async () => { it('test that % substitution feeds down CallExp->ArrExp->UnaryExp->CallExp', async () => {
const code = [ const code = [
`const myVar = 3`, `const myVar = 3`,
`const part001 = startSketchAt([0, 0])`, `const part001 = startSketchOn('XY')`,
` |> startProfileAt([0, 0], %)`,
` |> line({ to: [3, 4], tag: 'seg01' }, %)`, ` |> line({ to: [3, 4], tag: 'seg01' }, %)`,
` |> line([`, ` |> line([`,
` min(segLen('seg01', %), myVar),`, ` min(segLen('seg01', %), myVar),`,
@ -377,7 +382,8 @@ describe('testing math operators', () => {
describe('Testing Errors', () => { describe('Testing Errors', () => {
it('should throw an error when a variable is not defined', async () => { it('should throw an error when a variable is not defined', async () => {
const code = `const myVar = 5 const code = `const myVar = 5
const theExtrude = startSketchAt([0, 0]) const theExtrude = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> line([-2.4, 5], %) |> line([-2.4, 5], %)
|> line([-0.76], myVarZ, %) |> line([-0.76], myVarZ, %)
|> line([5,5], %) |> line([5,5], %)
@ -388,7 +394,7 @@ show(theExtrude)`
new KCLError( new KCLError(
'undefined_value', 'undefined_value',
'memory item key `myVarZ` is not defined', 'memory item key `myVarZ` is not defined',
[[100, 106]] [[129, 135]]
) )
) )
}) })

View File

@ -109,8 +109,8 @@ describe('Testing addSketchTo', () => {
'yz' 'yz'
) )
const str = recast(result.modifiedAst) const str = recast(result.modifiedAst)
expect(str).toBe(`const part001 = startSketchAt('default') expect(str).toBe(`const part001 = startSketchOn('YZ')
|> ry(90, %) |> startProfileAt('default', %)
|> line('default', %) |> line('default', %)
show(part001) show(part001)
`) `)
@ -133,7 +133,8 @@ function giveSketchFnCallTagTestHelper(
} }
describe('Testing giveSketchFnCallTag', () => { describe('Testing giveSketchFnCallTag', () => {
const code = `const part001 = startSketchAt([0, 0]) const code = `const part001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> line([-2.57, -0.13], %) |> line([-2.57, -0.13], %)
|> line([0, 0.83], %) |> line([0, 0.83], %)
|> line([0.82, 0.34], %) |> line([0.82, 0.34], %)
@ -185,7 +186,8 @@ fn ghi = (x) => {
const abc = 3 const abc = 3
const identifierGuy = 5 const identifierGuy = 5
const yo = 5 + 6 const yo = 5 + 6
const part001 = startSketchAt([-1.2, 4.83]) const part001 = startSketchOn('XY')
|> startProfileAt([-1.2, 4.83], %)
|> line([2.8, 0], %) |> line([2.8, 0], %)
|> angledLine([100 + 100, 3.09], %) |> angledLine([100 + 100, 3.09], %)
|> angledLine([abc, 3.09], %) |> angledLine([abc, 3.09], %)

View File

@ -32,21 +32,26 @@ import { isLiteralArrayOrStatic } from './std/sketchcombos'
export function addStartSketch( export function addStartSketch(
node: Program, node: Program,
axis: 'xy' | 'xz' | 'yz' | '-xy' | '-xz' | '-yz',
start: [number, number], start: [number, number],
end: [number, number] end: [number, number]
): { modifiedAst: Program; id: string; pathToNode: PathToNode } { ): { modifiedAst: Program; id: string; pathToNode: PathToNode } {
const _node = { ...node } const _node = { ...node }
const _name = findUniqueName(node, 'part') const _name = findUniqueName(node, 'part')
const startSketchAt = createCallExpression('startSketchAt', [ const startSketchOn = createCallExpressionStdLib('startSketchOn', [
createLiteral(axis.toUpperCase()),
])
const startProfileAt = createCallExpressionStdLib('startProfileAt', [
createArrayExpression([createLiteral(start[0]), createLiteral(start[1])]), createArrayExpression([createLiteral(start[0]), createLiteral(start[1])]),
createPipeSubstitution(),
]) ])
const initialLineTo = createCallExpression('line', [ const initialLineTo = createCallExpression('line', [
createArrayExpression([createLiteral(end[0]), createLiteral(end[1])]), createArrayExpression([createLiteral(end[0]), createLiteral(end[1])]),
createPipeSubstitution(), createPipeSubstitution(),
]) ])
const pipeBody = [startSketchAt, initialLineTo] const pipeBody = [startSketchOn, startProfileAt, initialLineTo]
const variableDeclaration = createVariableDeclaration( const variableDeclaration = createVariableDeclaration(
_name, _name,
@ -79,11 +84,11 @@ export function addSketchTo(
const _node = { ...node } const _node = { ...node }
const _name = name || findUniqueName(node, 'part') const _name = name || findUniqueName(node, 'part')
const startSketchAt = createCallExpressionStdLib('startSketchAt', [ const startSketchOn = createCallExpressionStdLib('startSketchOn', [
createLiteral('default'), createLiteral(axis.toUpperCase()),
]) ])
const rotate = createCallExpression(axis === 'xz' ? 'rx' : 'ry', [ const startProfileAt = createCallExpressionStdLib('startProfileAt', [
createLiteral(90), createLiteral('default'),
createPipeSubstitution(), createPipeSubstitution(),
]) ])
const initialLineTo = createCallExpressionStdLib('line', [ const initialLineTo = createCallExpressionStdLib('line', [
@ -91,10 +96,7 @@ export function addSketchTo(
createPipeSubstitution(), createPipeSubstitution(),
]) ])
const pipeBody = const pipeBody = [startSketchOn, startProfileAt, initialLineTo]
axis !== 'xy'
? [startSketchAt, rotate, initialLineTo]
: [startSketchAt, initialLineTo]
const variableDeclaration = createVariableDeclaration( const variableDeclaration = createVariableDeclaration(
_name, _name,

View File

@ -26,7 +26,8 @@ const halfArmAngle = armAngle / 2
const arrExpShouldNotBeIncluded = [1, 2, 3] const arrExpShouldNotBeIncluded = [1, 2, 3]
const objExpShouldNotBeIncluded = { a: 1, b: 2, c: 3 } const objExpShouldNotBeIncluded = { a: 1, b: 2, c: 3 }
const part001 = startSketchAt([0, 0]) const part001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> yLineTo(1, %) |> yLineTo(1, %)
|> xLine(3.84, %) // selection-range-7ish-before-this |> xLine(3.84, %) // selection-range-7ish-before-this
@ -57,7 +58,8 @@ show(part001)`
}) })
describe('testing argIsNotIdentifier', () => { describe('testing argIsNotIdentifier', () => {
const code = `const part001 = startSketchAt([-1.2, 4.83]) const code = `const part001 = startSketchOn('XY')
|> startProfileAt([-1.2, 4.83], %)
|> line([2.8, 0], %) |> line([2.8, 0], %)
|> angledLine([100 + 100, 3.09], %) |> angledLine([100 + 100, 3.09], %)
|> angledLine([abc, 3.09], %) |> angledLine([abc, 3.09], %)
@ -194,7 +196,8 @@ show(part001)`
}) })
describe('testing getNodePathFromSourceRange', () => { describe('testing getNodePathFromSourceRange', () => {
const code = `const part001 = startSketchAt([0.39, -0.05]) const code = `const part001 = startSketchOn('XY')
|> startProfileAt([0.39, -0.05], %)
|> line([0.94, 2.61], %) |> line([0.94, 2.61], %)
|> line([-0.21, -1.4], %) |> line([-0.21, -1.4], %)
show(part001)` show(part001)`
@ -210,7 +213,7 @@ show(part001)`
[0, 'index'], [0, 'index'],
['init', ''], ['init', ''],
['body', 'PipeExpression'], ['body', 'PipeExpression'],
[1, 'index'], [2, 'index'],
]) ])
}) })
it('finds the last line when cursor is put at the end', () => { it('finds the last line when cursor is put at the end', () => {
@ -225,7 +228,7 @@ show(part001)`
[0, 'index'], [0, 'index'],
['init', ''], ['init', ''],
['body', 'PipeExpression'], ['body', 'PipeExpression'],
[2, 'index'], [3, 'index'],
] ]
expect(result).toEqual(expected) expect(result).toEqual(expected)
// expect similar result for start of line // expect similar result for start of line

View File

@ -725,7 +725,11 @@ export class EngineCommandManager {
message.request_id message.request_id
) { ) {
this.handleModelingCommand(message.resp, message.request_id) this.handleModelingCommand(message.resp, message.request_id)
} else if (!message.success && message.request_id) { } else if (
!message.success &&
message.request_id &&
this.artifactMap[message.request_id]
) {
this.handleFailedModelingCommand(message) this.handleFailedModelingCommand(message)
} }
} }

View File

@ -96,7 +96,8 @@ describe('testing changeSketchArguments', () => {
const lineAfterChange = 'lineTo([2, 3], %)' const lineAfterChange = 'lineTo([2, 3], %)'
test('changeSketchArguments', async () => { test('changeSketchArguments', async () => {
// Enable rotations #152 // Enable rotations #152
const genCode = (line: string) => `const mySketch001 = startSketchAt([0, 0]) const genCode = (line: string) => `const mySketch001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> ${line} |> ${line}
|> lineTo([0.46, -5.82], %) |> lineTo([0.46, -5.82], %)
// |> rx(45, %) // |> rx(45, %)
@ -137,7 +138,8 @@ describe('testing addNewSketchLn', () => {
test('addNewSketchLn', async () => { test('addNewSketchLn', async () => {
// Enable rotations #152 // Enable rotations #152
const code = ` const code = `
const mySketch001 = startSketchAt([0, 0]) const mySketch001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
// |> rx(45, %) // |> rx(45, %)
|> lineTo([-1.59, -1.54], %) |> lineTo([-1.59, -1.54], %)
|> lineTo([0.46, -5.82], %) |> lineTo([0.46, -5.82], %)
@ -145,7 +147,7 @@ show(mySketch001)`
const ast = parse(code) const ast = parse(code)
const programMemory = await enginelessExecutor(ast) const programMemory = await enginelessExecutor(ast)
const sourceStart = code.indexOf(lineToChange) const sourceStart = code.indexOf(lineToChange)
expect(sourceStart).toBe(66) expect(sourceStart).toBe(95)
let { modifiedAst } = addNewSketchLn({ let { modifiedAst } = addNewSketchLn({
node: ast, node: ast,
programMemory, programMemory,
@ -160,7 +162,8 @@ show(mySketch001)`
], ],
}) })
// Enable rotations #152 // Enable rotations #152
let expectedCode = `const mySketch001 = startSketchAt([0, 0]) let expectedCode = `const mySketch001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
// |> rx(45, %) // |> rx(45, %)
|> lineTo([-1.59, -1.54], %) |> lineTo([-1.59, -1.54], %)
|> lineTo([0.46, -5.82], %) |> lineTo([0.46, -5.82], %)
@ -181,7 +184,8 @@ show(mySketch001)
], ],
}) })
expectedCode = `const mySketch001 = startSketchAt([0, 0]) expectedCode = `const mySketch001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
// |> rx(45, %) // |> rx(45, %)
|> lineTo([-1.59, -1.54], %) |> lineTo([-1.59, -1.54], %)
|> lineTo([0.46, -5.82], %) |> lineTo([0.46, -5.82], %)
@ -196,7 +200,8 @@ describe('testing addTagForSketchOnFace', () => {
it('needs to be in it', async () => { it('needs to be in it', async () => {
const originalLine = 'lineTo([-1.59, -1.54], %)' const originalLine = 'lineTo([-1.59, -1.54], %)'
// Enable rotations #152 // Enable rotations #152
const genCode = (line: string) => `const mySketch001 = startSketchAt([0, 0]) const genCode = (line: string) => `const mySketch001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
// |> rx(45, %) // |> rx(45, %)
|> ${line} |> ${line}
|> lineTo([0.46, -5.82], %) |> lineTo([0.46, -5.82], %)

View File

@ -52,7 +52,8 @@ async function testingSwapSketchFnCall({
describe('testing swaping out sketch calls with xLine/xLineTo', () => { describe('testing swaping out sketch calls with xLine/xLineTo', () => {
const bigExampleArr = [ const bigExampleArr = [
`const part001 = startSketchAt([0, 0])`, `const part001 = startSketchOn('XY')`,
` |> startProfileAt([0, 0], %)`,
` |> lineTo({ to: [1, 1], tag: 'abc1' }, %)`, ` |> lineTo({ to: [1, 1], tag: 'abc1' }, %)`,
` |> line({ to: [-2.04, -0.7], tag: 'abc2' }, %)`, ` |> line({ to: [-2.04, -0.7], tag: 'abc2' }, %)`,
` |> angledLine({`, ` |> angledLine({`,
@ -277,7 +278,8 @@ describe('testing swaping out sketch calls with xLine/xLineTo while keeping vari
`const angledLineOfYLengthY = 0.89`, `const angledLineOfYLengthY = 0.89`,
`const angledLineToXx = -1.86`, `const angledLineToXx = -1.86`,
`const angledLineToYy = -0.76`, `const angledLineToYy = -0.76`,
`const part001 = startSketchAt([0, 0])`, `const part001 = startSketchOn('XY')`,
` |> startProfileAt([0, 0], %)`,
// ` |> rx(90, %)`, // ` |> rx(90, %)`,
` |> lineTo([1, 1], %)`, ` |> lineTo([1, 1], %)`,
` |> line([lineX, 2.13], %)`, ` |> line([lineX, 2.13], %)`,
@ -371,7 +373,8 @@ describe('testing swaping out sketch calls with xLine/xLineTo while keeping vari
describe('testing getSketchSegmentIndexFromSourceRange', () => { describe('testing getSketchSegmentIndexFromSourceRange', () => {
const code = ` const code = `
const part001 = startSketchAt([0, 0.04]) // segment-in-start const part001 = startSketchOn('XY')
|> startProfileAt([0, 0.04], %) // segment-in-start
|> line([0, 0.4], %) |> line([0, 0.4], %)
|> xLine(3.48, %) |> xLine(3.48, %)
|> line([2.14, 1.35], %) // normal-segment |> line([2.14, 1.35], %) // normal-segment

View File

@ -92,7 +92,8 @@ const myVar2 = 5
const myVar3 = 6 const myVar3 = 6
const myAng = 40 const myAng = 40
const myAng2 = 134 const myAng2 = 134
const part001 = startSketchAt([0, 0]) const part001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> line([1, 3.82], %) // ln-should-get-tag |> line([1, 3.82], %) // ln-should-get-tag
|> lineTo([myVar, 1], %) // ln-lineTo-xAbsolute should use angleToMatchLengthX helper |> lineTo([myVar, 1], %) // ln-lineTo-xAbsolute should use angleToMatchLengthX helper
|> lineTo([1, myVar], %) // ln-lineTo-yAbsolute should use angleToMatchLengthY helper |> lineTo([1, myVar], %) // ln-lineTo-yAbsolute should use angleToMatchLengthY helper
@ -128,7 +129,8 @@ const myVar2 = 5
const myVar3 = 6 const myVar3 = 6
const myAng = 40 const myAng = 40
const myAng2 = 134 const myAng2 = 134
const part001 = startSketchAt([0, 0]) const part001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> line({ to: [1, 3.82], tag: 'seg01' }, %) // ln-should-get-tag |> line({ to: [1, 3.82], tag: 'seg01' }, %) // ln-should-get-tag
|> angledLineToX([ |> angledLineToX([
-angleToMatchLengthX('seg01', myVar, %), -angleToMatchLengthX('seg01', myVar, %),
@ -231,7 +233,8 @@ describe('testing transformAstForSketchLines for vertical and horizontal constra
const inputScript = `const myVar = 2 const inputScript = `const myVar = 2
const myVar2 = 12 const myVar2 = 12
const myVar3 = -10 const myVar3 = -10
const part001 = startSketchAt([0, 0]) const part001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> lineTo([1, 1], %) |> lineTo([1, 1], %)
|> line([-6.28, 1.4], %) // select for horizontal constraint 1 |> line([-6.28, 1.4], %) // select for horizontal constraint 1
|> line([-1.07, myVar], %) // select for vertical constraint 1 |> line([-1.07, myVar], %) // select for vertical constraint 1
@ -259,7 +262,8 @@ show(part001)
const expectModifiedScript = `const myVar = 2 const expectModifiedScript = `const myVar = 2
const myVar2 = 12 const myVar2 = 12
const myVar3 = -10 const myVar3 = -10
const part001 = startSketchAt([0, 0]) const part001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> lineTo([1, 1], %) |> lineTo([1, 1], %)
|> xLine(-6.28, %) // select for horizontal constraint 1 |> xLine(-6.28, %) // select for horizontal constraint 1
|> line([-1.07, myVar], %) // select for vertical constraint 1 |> line([-1.07, myVar], %) // select for vertical constraint 1
@ -317,7 +321,8 @@ show(part001)
const expectModifiedScript = `const myVar = 2 const expectModifiedScript = `const myVar = 2
const myVar2 = 12 const myVar2 = 12
const myVar3 = -10 const myVar3 = -10
const part001 = startSketchAt([0, 0]) const part001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> lineTo([1, 1], %) |> lineTo([1, 1], %)
|> line([-6.28, 1.4], %) // select for horizontal constraint 1 |> line([-6.28, 1.4], %) // select for horizontal constraint 1
|> yLine(myVar, %) // select for vertical constraint 1 |> yLine(myVar, %) // select for vertical constraint 1
@ -376,7 +381,8 @@ show(part001)
describe('testing transformAstForSketchLines for vertical and horizontal distance constraints', () => { describe('testing transformAstForSketchLines for vertical and horizontal distance constraints', () => {
describe('testing setHorzDistance for line', () => { describe('testing setHorzDistance for line', () => {
const inputScript = `const myVar = 1 const inputScript = `const myVar = 1
const part001 = startSketchAt([0, 0]) const part001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> line([0.31, 1.67], %) // base selection |> line([0.31, 1.67], %) // base selection
|> line([0.45, 1.46], %) |> line([0.45, 1.46], %)
|> line([0.45, 1.46], %) // free |> line([0.45, 1.46], %) // free
@ -477,7 +483,8 @@ const baseThickHalf = baseThick / 2
const halfHeight = totalHeight / 2 const halfHeight = totalHeight / 2
const halfArmAngle = armAngle / 2 const halfArmAngle = armAngle / 2
const part001 = startSketchAt([-0.01, -0.05]) const part001 = startSketchOn('XY')
|> startProfileAt([-0.01, -0.05], %)
|> line([0.01, 0.94 + 0], %) // partial |> line([0.01, 0.94 + 0], %) // partial
|> xLine(3.03, %) // partial |> xLine(3.03, %) // partial
|> angledLine({ |> angledLine({

View File

@ -5,7 +5,8 @@ beforeAll(() => initPromise)
describe('testing angledLineThatIntersects', () => { describe('testing angledLineThatIntersects', () => {
it('angledLineThatIntersects should intersect with another line', async () => { it('angledLineThatIntersects should intersect with another line', async () => {
const code = (offset: string) => `const part001 = startSketchAt([0, 0]) const code = (offset: string) => `const part001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> lineTo({to:[2, 2], tag: "yo"}, %) |> lineTo({to:[2, 2], tag: "yo"}, %)
|> lineTo([3, 1], %) |> lineTo([3, 1], %)
|> angledLineThatIntersects({ |> angledLineThatIntersects({

View File

@ -16,6 +16,7 @@ import { ProgramReturn } from '../wasm-lib/kcl/bindings/ProgramReturn'
import { MemoryItem } from '../wasm-lib/kcl/bindings/MemoryItem' import { MemoryItem } from '../wasm-lib/kcl/bindings/MemoryItem'
import type { Program } from '../wasm-lib/kcl/bindings/Program' import type { Program } from '../wasm-lib/kcl/bindings/Program'
import type { Token } from '../wasm-lib/kcl/bindings/Token' import type { Token } from '../wasm-lib/kcl/bindings/Token'
import { DefaultPlanes } from '../wasm-lib/kcl/bindings/DefaultPlanes'
export type { Program } from '../wasm-lib/kcl/bindings/Program' export type { Program } from '../wasm-lib/kcl/bindings/Program'
export type { Value } from '../wasm-lib/kcl/bindings/Value' export type { Value } from '../wasm-lib/kcl/bindings/Value'
@ -118,6 +119,7 @@ export const executor = async (
node: Program, node: Program,
programMemory: ProgramMemory = { root: {}, return: null }, programMemory: ProgramMemory = { root: {}, return: null },
engineCommandManager: EngineCommandManager, engineCommandManager: EngineCommandManager,
planes: DefaultPlanes,
// work around while the gemotry is still be stored on the frontend // work around while the gemotry is still be stored on the frontend
// will be removed when the stream UI is added. // will be removed when the stream UI is added.
tempMapCallback: (a: { tempMapCallback: (a: {
@ -129,7 +131,8 @@ export const executor = async (
const _programMemory = await _executor( const _programMemory = await _executor(
node, node,
programMemory, programMemory,
engineCommandManager engineCommandManager,
planes
) )
const { artifactMap, sourceRangeMap } = const { artifactMap, sourceRangeMap } =
await engineCommandManager.waitForAllCommands(node, _programMemory) await engineCommandManager.waitForAllCommands(node, _programMemory)
@ -142,13 +145,15 @@ export const executor = async (
export const _executor = async ( export const _executor = async (
node: Program, node: Program,
programMemory: ProgramMemory = { root: {}, return: null }, programMemory: ProgramMemory = { root: {}, return: null },
engineCommandManager: EngineCommandManager engineCommandManager: EngineCommandManager,
planes: DefaultPlanes
): Promise<ProgramMemory> => { ): Promise<ProgramMemory> => {
try { try {
const memory: ProgramMemory = await execute_wasm( const memory: ProgramMemory = await execute_wasm(
JSON.stringify(node), JSON.stringify(node),
JSON.stringify(programMemory), JSON.stringify(programMemory),
engineCommandManager engineCommandManager,
JSON.stringify(planes)
) )
return memory return memory
} catch (e: any) { } catch (e: any) {
@ -190,6 +195,7 @@ export const modifyAstForSketch = async (
engineCommandManager: EngineCommandManager, engineCommandManager: EngineCommandManager,
ast: Program, ast: Program,
variableName: string, variableName: string,
currentPlane: string,
engineId: string engineId: string
): Promise<Program> => { ): Promise<Program> => {
try { try {
@ -197,6 +203,7 @@ export const modifyAstForSketch = async (
engineCommandManager, engineCommandManager,
JSON.stringify(ast), JSON.stringify(ast),
variableName, variableName,
JSON.stringify(currentPlane),
engineId engineId
) )

View File

@ -8,7 +8,8 @@ const filletR = thickness * 2
const shelfMountL = 9 const shelfMountL = 9
const wallMountL = 8 const wallMountL = 8
const bracket = startSketchAt([0, 0]) const bracket = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> line([0, wallMountL], %) |> line([0, wallMountL], %)
|> tangentalArc({ |> tangentalArc({
radius: filletR, radius: filletR,

View File

@ -4,6 +4,7 @@ import {
EngineCommand, EngineCommand,
} from '../lang/std/engineConnection' } from '../lang/std/engineConnection'
import { Models } from '@kittycad/lib' import { Models } from '@kittycad/lib'
import { v4 as uuidv4 } from 'uuid'
type WebSocketResponse = Models['OkWebSocketResponseData_type'] type WebSocketResponse = Models['OkWebSocketResponseData_type']
@ -64,7 +65,11 @@ export async function enginelessExecutor(
}) as any as EngineCommandManager }) as any as EngineCommandManager
await mockEngineCommandManager.waitForReady await mockEngineCommandManager.waitForReady
mockEngineCommandManager.startNewSession() mockEngineCommandManager.startNewSession()
const programMemory = await _executor(ast, pm, mockEngineCommandManager) const programMemory = await _executor(ast, pm, mockEngineCommandManager, {
xy: uuidv4(),
yz: uuidv4(),
xz: uuidv4(),
})
await mockEngineCommandManager.waitForAllCommands() await mockEngineCommandManager.waitForAllCommands()
return programMemory return programMemory
} }
@ -83,7 +88,11 @@ export async function executor(
}) })
await engineCommandManager.waitForReady await engineCommandManager.waitForReady
engineCommandManager.startNewSession() engineCommandManager.startNewSession()
const programMemory = await _executor(ast, pm, engineCommandManager) const programMemory = await _executor(ast, pm, engineCommandManager, {
xy: uuidv4(),
yz: uuidv4(),
xz: uuidv4(),
})
await engineCommandManager.waitForAllCommands(ast, programMemory) await engineCommandManager.waitForAllCommands(ast, programMemory)
return programMemory return programMemory
} }

View File

@ -1,6 +1,11 @@
import { faArrowRight, faXmark } from '@fortawesome/free-solid-svg-icons' import { faArrowRight, faXmark } from '@fortawesome/free-solid-svg-icons'
import { ActionButton } from '../../components/ActionButton' import { ActionButton } from '../../components/ActionButton'
import { onboardingPaths, useDismiss, useNextClick } from '.' import {
ONBOARDING_PROJECT_NAME,
onboardingPaths,
useDismiss,
useNextClick,
} from '.'
import { useGlobalStateContext } from 'hooks/useGlobalStateContext' import { useGlobalStateContext } from 'hooks/useGlobalStateContext'
import { Themes, getSystemTheme } from 'lib/theme' import { Themes, getSystemTheme } from 'lib/theme'
import { bracket } from 'lib/exampleKcl' import { bracket } from 'lib/exampleKcl'
@ -25,14 +30,20 @@ function OnboardingWithNewFile() {
})) }))
const { const {
settings: { settings: {
context: { defaultDirectory, defaultProjectName }, context: { defaultDirectory },
}, },
} = useGlobalStateContext() } = useGlobalStateContext()
async function createAndOpenNewProject() { async function createAndOpenNewProject() {
const projects = await getProjectsInDir(defaultDirectory) const projects = await getProjectsInDir(defaultDirectory)
const nextIndex = await getNextProjectIndex(defaultProjectName, projects) const nextIndex = await getNextProjectIndex(
const name = interpolateProjectNameWithIndex(defaultProjectName, nextIndex) ONBOARDING_PROJECT_NAME,
projects
)
const name = interpolateProjectNameWithIndex(
ONBOARDING_PROJECT_NAME,
nextIndex
)
const newFile = await createNewProject(defaultDirectory + '/' + name) const newFile = await createNewProject(defaultDirectory + '/' + name)
navigate(`${paths.FILE}/${encodeURIComponent(newFile.path)}`) navigate(`${paths.FILE}/${encodeURIComponent(newFile.path)}`)
} }

View File

@ -1,5 +1,5 @@
import { useHotkeys } from 'react-hotkeys-hook' import { useHotkeys } from 'react-hotkeys-hook'
import { Outlet, useRouteLoaderData, useNavigate } from 'react-router-dom' import { Outlet, useNavigate } from 'react-router-dom'
import Introduction from './Introduction' import Introduction from './Introduction'
import Camera from './Camera' import Camera from './Camera'
import Sketching from './Sketching' import Sketching from './Sketching'
@ -15,7 +15,10 @@ import UserMenu from './UserMenu'
import ProjectMenu from './ProjectMenu' import ProjectMenu from './ProjectMenu'
import Export from './Export' import Export from './Export'
import FutureWork from './FutureWork' import FutureWork from './FutureWork'
import { IndexLoaderData, paths } from 'Router' import { paths } from 'Router'
import { useAbsoluteFilePath } from 'hooks/useAbsoluteFilePath'
export const ONBOARDING_PROJECT_NAME = 'Tutorial Project $nn'
export const onboardingPaths = { export const onboardingPaths = {
INDEX: '/', INDEX: '/',
@ -86,29 +89,23 @@ export const onboardingRoutes = [
] ]
export function useNextClick(newStatus: string) { export function useNextClick(newStatus: string) {
const filePath = useAbsoluteFilePath()
const { const {
settings: { send }, settings: { send },
} = useGlobalStateContext() } = useGlobalStateContext()
const navigate = useNavigate() const navigate = useNavigate()
const { project } = useRouteLoaderData(paths.FILE) as IndexLoaderData
return useCallback(() => { return useCallback(() => {
send({ send({
type: 'Set Onboarding Status', type: 'Set Onboarding Status',
data: { onboardingStatus: newStatus }, data: { onboardingStatus: newStatus },
}) })
navigate( navigate(filePath + paths.ONBOARDING.INDEX.slice(0, -1) + newStatus)
paths.FILE + }, [filePath, newStatus, send, navigate])
'/' +
encodeURIComponent(project?.path || 'new') +
paths.ONBOARDING.INDEX.slice(0, -1) +
newStatus
)
}, [project, newStatus, send, navigate])
} }
export function useDismiss() { export function useDismiss() {
const routeData = useRouteLoaderData(paths.FILE) as IndexLoaderData const filePath = useAbsoluteFilePath()
const { const {
settings: { send }, settings: { send },
} = useGlobalStateContext() } = useGlobalStateContext()
@ -119,10 +116,8 @@ export function useDismiss() {
type: 'Set Onboarding Status', type: 'Set Onboarding Status',
data: { onboardingStatus: 'dismissed' }, data: { onboardingStatus: 'dismissed' },
}) })
navigate( navigate(filePath)
paths.FILE + '/' + encodeURIComponent(routeData?.project?.path || 'new') }, [send, navigate, filePath])
)
}, [send, navigate, routeData])
} }
const Onboarding = () => { const Onboarding = () => {

View File

@ -24,11 +24,19 @@ import {
} from 'lib/cameraControls' } from 'lib/cameraControls'
import { UnitSystem } from 'machines/settingsMachine' import { UnitSystem } from 'machines/settingsMachine'
import { useDotDotSlash } from 'hooks/useDotDotSlash' import { useDotDotSlash } from 'hooks/useDotDotSlash'
import {
createNewProject,
getNextProjectIndex,
getProjectsInDir,
interpolateProjectNameWithIndex,
} from 'lib/tauriFS'
import { ONBOARDING_PROJECT_NAME } from './Onboarding'
export const Settings = () => { export const Settings = () => {
const loaderData = useRouteLoaderData(paths.FILE) as IndexLoaderData const loaderData = useRouteLoaderData(paths.FILE) as IndexLoaderData
const navigate = useNavigate() const navigate = useNavigate()
const location = useLocation() const location = useLocation()
const isFileSettings = location.pathname.includes(paths.FILE)
const dotDotSlash = useDotDotSlash() const dotDotSlash = useDotDotSlash()
useHotkeys('esc', () => navigate(dotDotSlash())) useHotkeys('esc', () => navigate(dotDotSlash()))
const { const {
@ -63,6 +71,33 @@ export const Settings = () => {
} }
} }
function restartOnboarding() {
send({
type: 'Set Onboarding Status',
data: { onboardingStatus: '' },
})
if (isFileSettings) {
navigate(dotDotSlash(1) + paths.ONBOARDING.INDEX)
} else {
createAndOpenNewProject()
}
}
async function createAndOpenNewProject() {
const projects = await getProjectsInDir(defaultDirectory)
const nextIndex = await getNextProjectIndex(
ONBOARDING_PROJECT_NAME,
projects
)
const name = interpolateProjectNameWithIndex(
ONBOARDING_PROJECT_NAME,
nextIndex
)
const newFile = await createNewProject(defaultDirectory + '/' + name)
navigate(`${paths.FILE}/${encodeURIComponent(newFile.path)}`)
}
return ( return (
<div className="fixed inset-0 z-40 overflow-auto body-bg"> <div className="fixed inset-0 z-40 overflow-auto body-bg">
<AppHeader showToolbar={false} project={loaderData?.project}> <AppHeader showToolbar={false} project={loaderData?.project}>
@ -257,26 +292,18 @@ export const Settings = () => {
))} ))}
</select> </select>
</SettingsSection> </SettingsSection>
{location.pathname.includes(paths.FILE) && ( <SettingsSection
<SettingsSection title="Onboarding"
title="Onboarding" description="Replay the onboarding process"
description="Replay the onboarding process" >
<ActionButton
Element="button"
onClick={restartOnboarding}
icon={{ icon: faArrowRotateBack }}
> >
<ActionButton Replay Onboarding
Element="button" </ActionButton>
onClick={() => { </SettingsSection>
send({
type: 'Set Onboarding Status',
data: { onboardingStatus: '' },
})
navigate(dotDotSlash(1) + paths.ONBOARDING.INDEX)
}}
icon={{ icon: faArrowRotateBack }}
>
Replay Onboarding
</ActionButton>
</SettingsSection>
)}
</div> </div>
</div> </div>
) )

View File

@ -20,6 +20,8 @@ import { KCLError } from './lang/errors'
import { deferExecution } from 'lib/utils' import { deferExecution } from 'lib/utils'
import { bracket } from 'lib/exampleKcl' import { bracket } from 'lib/exampleKcl'
import { engineCommandManager } from './lang/std/engineConnection' import { engineCommandManager } from './lang/std/engineConnection'
import { DefaultPlanes } from './wasm-lib/kcl/bindings/DefaultPlanes'
import { initDefaultPlanes } from './hooks/useAppMode'
export type Selection = { export type Selection = {
type: 'default' | 'line-end' | 'line-mid' type: 'default' | 'line-end' | 'line-mid'
@ -182,6 +184,10 @@ export interface StoreState {
}) => void }) => void
isExecuting: boolean isExecuting: boolean
setIsExecuting: (isExecuting: boolean) => void setIsExecuting: (isExecuting: boolean) => void
defaultPlanes: DefaultPlanes | null
setDefaultPlanes: (defaultPlanes: DefaultPlanes) => void
currentPlane: string | null
setCurrentPlane: (currentPlane: string) => void
showHomeMenu: boolean showHomeMenu: boolean
setHomeShowMenu: (showMenu: boolean) => void setHomeShowMenu: (showMenu: boolean) => void
@ -222,10 +228,20 @@ export const useStore = create<StoreState>()(
} }
}, },
executeCode: async (code, force) => { executeCode: async (code, force) => {
if (!get().defaultPlanes) {
let defaultPlanes = await initDefaultPlanes(
engineCommandManager,
true
)
if (!defaultPlanes) return
get().setDefaultPlanes(defaultPlanes)
}
const result = await executeCode({ const result = await executeCode({
code: code || get().code, code: code || get().code,
lastAst: get().ast, lastAst: get().ast,
engineCommandManager: engineCommandManager, engineCommandManager: engineCommandManager,
defaultPlanes: get().defaultPlanes!,
force, force,
}) })
if (!result.isChange) { if (!result.isChange) {
@ -332,11 +348,20 @@ export const useStore = create<StoreState>()(
executeAst: async (ast) => { executeAst: async (ast) => {
const _ast = ast || get().ast const _ast = ast || get().ast
if (!get().isStreamReady) return if (!get().isStreamReady) return
if (!get().defaultPlanes) {
let defaultPlanes = await initDefaultPlanes(
engineCommandManager,
true
)
if (!defaultPlanes) return
get().setDefaultPlanes(defaultPlanes)
}
set({ isExecuting: true }) set({ isExecuting: true })
const { logs, errors, programMemory } = await executeAst({ const { logs, errors, programMemory } = await executeAst({
ast: _ast, ast: _ast,
engineCommandManager, engineCommandManager,
defaultPlanes: get().defaultPlanes!,
}) })
set({ set({
programMemory, programMemory,
@ -349,10 +374,20 @@ export const useStore = create<StoreState>()(
const _ast = ast || get().ast const _ast = ast || get().ast
if (!get().isStreamReady) return if (!get().isStreamReady) return
if (!get().defaultPlanes) {
let defaultPlanes = await initDefaultPlanes(
engineCommandManager,
true
)
if (!defaultPlanes) return
get().setDefaultPlanes(defaultPlanes)
}
const { logs, errors, programMemory } = await executeAst({ const { logs, errors, programMemory } = await executeAst({
ast: _ast, ast: _ast,
engineCommandManager, engineCommandManager,
useFakeExecutor: true, useFakeExecutor: true,
defaultPlanes: get().defaultPlanes!,
}) })
set({ set({
programMemory, programMemory,
@ -453,6 +488,10 @@ export const useStore = create<StoreState>()(
}, },
isExecuting: false, isExecuting: false,
setIsExecuting: (isExecuting) => set({ isExecuting }), setIsExecuting: (isExecuting) => set({ isExecuting }),
defaultPlanes: null,
setDefaultPlanes: (defaultPlanes) => set({ defaultPlanes }),
currentPlane: null,
setCurrentPlane: (currentPlane) => set({ currentPlane }),
// tauri specific app settings // tauri specific app settings
defaultDir: { defaultDir: {
@ -512,11 +551,13 @@ async function executeCode({
engineCommandManager, engineCommandManager,
code, code,
lastAst, lastAst,
defaultPlanes,
force, force,
}: { }: {
code: string code: string
lastAst: Program lastAst: Program
engineCommandManager: EngineCommandManager engineCommandManager: EngineCommandManager
defaultPlanes: DefaultPlanes
force?: boolean force?: boolean
}): Promise< }): Promise<
| { | {
@ -566,6 +607,7 @@ async function executeCode({
const { logs, errors, programMemory } = await executeAst({ const { logs, errors, programMemory } = await executeAst({
ast, ast,
engineCommandManager, engineCommandManager,
defaultPlanes,
}) })
return { return {
ast, ast,
@ -579,10 +621,12 @@ async function executeCode({
async function executeAst({ async function executeAst({
ast, ast,
engineCommandManager, engineCommandManager,
defaultPlanes,
useFakeExecutor = false, useFakeExecutor = false,
}: { }: {
ast: Program ast: Program
engineCommandManager: EngineCommandManager engineCommandManager: EngineCommandManager
defaultPlanes: DefaultPlanes
useFakeExecutor?: boolean useFakeExecutor?: boolean
}): Promise<{ }): Promise<{
logs: string[] logs: string[]
@ -605,7 +649,8 @@ async function executeAst({
root: defaultProgramMemory, root: defaultProgramMemory,
return: null, return: null,
}, },
engineCommandManager engineCommandManager,
defaultPlanes
)) ))
await engineCommandManager.waitForAllCommands(ast, programMemory) await engineCommandManager.waitForAllCommands(ast, programMemory)

View File

@ -1390,7 +1390,7 @@ dependencies = [
[[package]] [[package]]
name = "kcl-lib" name = "kcl-lib"
version = "0.1.32" version = "0.1.33"
dependencies = [ dependencies = [
"anyhow", "anyhow",
"async-recursion", "async-recursion",
@ -1426,9 +1426,9 @@ dependencies = [
[[package]] [[package]]
name = "kittycad" name = "kittycad"
version = "0.2.28" version = "0.2.31"
source = "registry+https://github.com/rust-lang/crates.io-index" source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "35b2f9302648dbb06fd7121687f9505fc3179eba84111a06d76b246e3158f5dc" checksum = "539b323537b877fc8dd130362b8f1af9af8051c19208bb8bfd816ab7c330f2bb"
dependencies = [ dependencies = [
"anyhow", "anyhow",
"async-trait", "async-trait",
@ -2217,9 +2217,9 @@ checksum = "dbb5fb1acd8a1a18b3dd5be62d25485eb770e05afb408a9627d14d451bae12da"
[[package]] [[package]]
name = "reqwest" name = "reqwest"
version = "0.11.21" version = "0.11.22"
source = "registry+https://github.com/rust-lang/crates.io-index" source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "78fdbab6a7e1d7b13cc8ff10197f47986b41c639300cc3c8158cac7847c9bbef" checksum = "046cd98826c46c2ac8ddecae268eb5c2e58628688a5fc7a2643704a73faba95b"
dependencies = [ dependencies = [
"base64 0.21.4", "base64 0.21.4",
"bytes", "bytes",

View File

@ -11,7 +11,7 @@ crate-type = ["cdylib"]
bson = { version = "2.7.0", features = ["uuid-1", "chrono"] } bson = { version = "2.7.0", features = ["uuid-1", "chrono"] }
gloo-utils = "0.2.0" gloo-utils = "0.2.0"
kcl-lib = { path = "kcl" } kcl-lib = { path = "kcl" }
kittycad = { version = "0.2.27", default-features = false, features = ["js"] } kittycad = { version = "0.2.31", default-features = false, features = ["js"] }
serde_json = "1.0.107" serde_json = "1.0.107"
uuid = { version = "1.4.1", features = ["v4", "js", "serde"] } uuid = { version = "1.4.1", features = ["v4", "js", "serde"] }
wasm-bindgen = "0.2.87" wasm-bindgen = "0.2.87"
@ -20,9 +20,9 @@ wasm-bindgen-futures = "0.4.37"
[dev-dependencies] [dev-dependencies]
anyhow = "1" anyhow = "1"
image = "0.24.7" image = "0.24.7"
kittycad = "0.2.27" kittycad = "0.2.31"
pretty_assertions = "1.4.0" pretty_assertions = "1.4.0"
reqwest = { version = "0.11.21", default-features = false } reqwest = { version = "0.11.22", default-features = false }
tokio = { version = "1.32.0", features = ["rt-multi-thread", "macros", "time"] } tokio = { version = "1.32.0", features = ["rt-multi-thread", "macros", "time"] }
twenty-twenty = "0.6.1" twenty-twenty = "0.6.1"
uuid = { version = "1.4.1", features = ["v4", "js", "serde"] } uuid = { version = "1.4.1", features = ["v4", "js", "serde"] }

View File

@ -1,7 +1,7 @@
[package] [package]
name = "kcl-lib" name = "kcl-lib"
description = "KittyCAD Language" description = "KittyCAD Language"
version = "0.1.32" version = "0.1.33"
edition = "2021" edition = "2021"
license = "MIT" license = "MIT"
@ -15,7 +15,7 @@ clap = { version = "4.4.6", features = ["cargo", "derive", "env", "unicode"], op
dashmap = "5.5.3" dashmap = "5.5.3"
derive-docs = { version = "0.1.4" } derive-docs = { version = "0.1.4" }
#derive-docs = { path = "../derive-docs" } #derive-docs = { path = "../derive-docs" }
kittycad = { version = "0.2.27", default-features = false, features = ["js"] } kittycad = { version = "0.2.31", default-features = false, features = ["js"] }
lazy_static = "1.4.0" lazy_static = "1.4.0"
parse-display = "0.8.2" parse-display = "0.8.2"
schemars = { version = "0.8", features = ["impl_json_schema", "url", "uuid1"] } schemars = { version = "0.8", features = ["impl_json_schema", "url", "uuid1"] }
@ -36,7 +36,7 @@ web-sys = { version = "0.3.64", features = ["console"] }
[target.'cfg(not(target_arch = "wasm32"))'.dependencies] [target.'cfg(not(target_arch = "wasm32"))'.dependencies]
bson = { version = "2.7.0", features = ["uuid-1", "chrono"] } bson = { version = "2.7.0", features = ["uuid-1", "chrono"] }
futures = { version = "0.3.28" } futures = { version = "0.3.28" }
reqwest = { version = "0.11.21", default-features = false } reqwest = { version = "0.11.22", default-features = false }
tokio = { version = "1.32.0", features = ["full"] } tokio = { version = "1.32.0", features = ["full"] }
tokio-tungstenite = { version = "0.20.0", features = ["rustls-tls-native-roots"] } tokio-tungstenite = { version = "0.20.0", features = ["rustls-tls-native-roots"] }
tower-lsp = { version = "0.20.0", features = ["proposed"] } tower-lsp = { version = "0.20.0", features = ["proposed"] }

View File

@ -1 +1 @@
enum-variant-size-threshold = 24 enum-variant-size-threshold = 48

View File

@ -31,6 +31,8 @@ pub async fn modify_ast_for_sketch(
program: &mut Program, program: &mut Program,
// The name of the sketch. // The name of the sketch.
sketch_name: &str, sketch_name: &str,
// The type of plane the sketch is on. `XY` or `XZ`, etc
plane: crate::executor::PlaneType,
// The ID of the parent sketch. // The ID of the parent sketch.
sketch_id: uuid::Uuid, sketch_id: uuid::Uuid,
) -> Result<String, KclError> { ) -> Result<String, KclError> {
@ -153,10 +155,11 @@ pub async fn modify_ast_for_sketch(
y: (first_control_points.points[1].y - first_control_points.points[0].y), y: (first_control_points.points[1].y - first_control_points.points[0].y),
z: (first_control_points.points[1].z - first_control_points.points[0].z), z: (first_control_points.points[1].z - first_control_points.points[0].z),
}; };
let sketch = create_start_sketch_at( let sketch = create_start_sketch_on(
sketch_name, sketch_name,
[first_control_points.points[0].x, first_control_points.points[0].y], [first_control_points.points[0].x, first_control_points.points[0].y],
[start_sketch_at_end.x, start_sketch_at_end.y], [start_sketch_at_end.x, start_sketch_at_end.y],
plane,
additional_lines, additional_lines,
)?; )?;
@ -174,19 +177,24 @@ pub async fn modify_ast_for_sketch(
} }
/// Create a pipe expression that starts a sketch at the given point and draws a line to the given point. /// Create a pipe expression that starts a sketch at the given point and draws a line to the given point.
fn create_start_sketch_at( fn create_start_sketch_on(
name: &str, name: &str,
start: [f64; 2], start: [f64; 2],
end: [f64; 2], end: [f64; 2],
plane: crate::executor::PlaneType,
additional_lines: Vec<[f64; 2]>, additional_lines: Vec<[f64; 2]>,
) -> Result<VariableDeclarator, KclError> { ) -> Result<VariableDeclarator, KclError> {
let start_sketch_at = CallExpression::new( let start_sketch_on = CallExpression::new("startSketchOn", vec![Literal::new(plane.to_string().into()).into()])?;
"startSketchAt", let start_profile_at = CallExpression::new(
vec![ArrayExpression::new(vec![ "startProfileAt",
Literal::new(round_before_recast(start[0]).into()).into(), vec![
Literal::new(round_before_recast(start[1]).into()).into(), ArrayExpression::new(vec![
]) Literal::new(round_before_recast(start[0]).into()).into(),
.into()], Literal::new(round_before_recast(start[1]).into()).into(),
])
.into(),
PipeSubstitution::new().into(),
],
)?; )?;
// Keep track of where we are so we can close the sketch if we need to. // Keep track of where we are so we can close the sketch if we need to.
@ -209,7 +217,7 @@ fn create_start_sketch_at(
], ],
)?; )?;
let mut pipe_body = vec![start_sketch_at.into(), initial_line.into()]; let mut pipe_body = vec![start_sketch_on.into(), start_profile_at.into(), initial_line.into()];
for (index, line) in additional_lines.iter().enumerate() { for (index, line) in additional_lines.iter().enumerate() {
current_position.x += line[0]; current_position.x += line[0];

View File

@ -1,6 +1,6 @@
//! Data types for the AST. //! Data types for the AST.
use std::collections::HashMap; use std::{collections::HashMap, fmt::Write};
use anyhow::Result; use anyhow::Result;
use parse_display::{Display, FromStr}; use parse_display::{Display, FromStr};
@ -10,9 +10,8 @@ use serde_json::Map;
use tower_lsp::lsp_types::{CompletionItem, CompletionItemKind, DocumentSymbol, Range as LspRange, SymbolKind}; use tower_lsp::lsp_types::{CompletionItem, CompletionItemKind, DocumentSymbol, Range as LspRange, SymbolKind};
use crate::{ use crate::{
engine::EngineConnection,
errors::{KclError, KclErrorDetails}, errors::{KclError, KclErrorDetails},
executor::{MemoryItem, Metadata, PipeInfo, ProgramMemory, SourceRange, UserVal}, executor::{ExecutorContext, MemoryItem, Metadata, PipeInfo, ProgramMemory, SourceRange, UserVal},
parser::PIPE_OPERATOR, parser::PIPE_OPERATOR,
}; };
@ -29,73 +28,74 @@ pub struct Program {
impl Program { impl Program {
pub fn recast(&self, options: &FormatOptions, indentation_level: usize) -> String { pub fn recast(&self, options: &FormatOptions, indentation_level: usize) -> String {
let indentation = options.get_indentation(indentation_level); let indentation = options.get_indentation(indentation_level);
let result = self let result =
.body self.body
.iter() .iter()
.map(|statement| match statement.clone() { .map(|statement| match statement.clone() {
BodyItem::ExpressionStatement(expression_statement) => { BodyItem::ExpressionStatement(expression_statement) => {
expression_statement expression_statement
.expression .expression
.recast(options, indentation_level, false) .recast(options, indentation_level, false)
}
BodyItem::VariableDeclaration(variable_declaration) => variable_declaration
.declarations
.iter()
.map(|declaration| {
format!(
"{}{} {} = {}",
indentation,
variable_declaration.kind,
declaration.id.name,
declaration.init.recast(options, 0, false)
)
})
.collect::<String>(),
BodyItem::ReturnStatement(return_statement) => {
format!(
"{}return {}",
indentation,
return_statement.argument.recast(options, 0, false)
)
}
})
.enumerate()
.map(|(index, recast_str)| {
let start_string = if index == 0 {
// We need to indent.
if let Some(start) = self.non_code_meta.start.clone() {
start.format(&indentation)
} else {
indentation.to_string()
} }
} else { BodyItem::VariableDeclaration(variable_declaration) => variable_declaration
// Do nothing, we already applied the indentation elsewhere. .declarations
String::new() .iter()
}; .fold(String::new(), |mut output, declaration| {
let _ = write!(
output,
"{}{} {} = {}",
indentation,
variable_declaration.kind,
declaration.id.name,
declaration.init.recast(options, 0, false)
);
output
}),
BodyItem::ReturnStatement(return_statement) => {
format!(
"{}return {}",
indentation,
return_statement.argument.recast(options, 0, false)
)
}
})
.enumerate()
.fold(String::new(), |mut output, (index, recast_str)| {
let start_string = if index == 0 {
// We need to indent.
if let Some(start) = self.non_code_meta.start.clone() {
start.format(&indentation)
} else {
indentation.to_string()
}
} else {
// Do nothing, we already applied the indentation elsewhere.
String::new()
};
// determine the value of the end string // determine the value of the end string
// basically if we are inside a nested function we want to end with a new line // basically if we are inside a nested function we want to end with a new line
let maybe_line_break: String = if index == self.body.len() - 1 && indentation_level == 0 { let maybe_line_break: String = if index == self.body.len() - 1 && indentation_level == 0 {
String::new() String::new()
} else { } else {
"\n".to_string() "\n".to_string()
}; };
let custom_white_space_or_comment = match self.non_code_meta.non_code_nodes.get(&index) { let custom_white_space_or_comment = match self.non_code_meta.non_code_nodes.get(&index) {
Some(custom_white_space_or_comment) => custom_white_space_or_comment.format(&indentation), Some(custom_white_space_or_comment) => custom_white_space_or_comment.format(&indentation),
None => String::new(), None => String::new(),
}; };
let end_string = if custom_white_space_or_comment.is_empty() { let end_string = if custom_white_space_or_comment.is_empty() {
maybe_line_break maybe_line_break
} else { } else {
custom_white_space_or_comment custom_white_space_or_comment
}; };
format!("{}{}{}", start_string, recast_str, end_string) let _ = write!(output, "{}{}{}", start_string, recast_str, end_string);
}) output
.collect::<String>() })
.trim() .trim()
.to_string(); .to_string();
// Insert a final new line if the user wants it. // Insert a final new line if the user wants it.
if options.insert_final_newline { if options.insert_final_newline {
@ -631,7 +631,7 @@ impl BinaryPart {
&self, &self,
memory: &mut ProgramMemory, memory: &mut ProgramMemory,
pipe_info: &mut PipeInfo, pipe_info: &mut PipeInfo,
engine: &EngineConnection, ctx: &ExecutorContext,
) -> Result<MemoryItem, KclError> { ) -> Result<MemoryItem, KclError> {
// We DO NOT set this gloablly because if we did and this was called inside a pipe it would // We DO NOT set this gloablly because if we did and this was called inside a pipe it would
// stop the execution of the pipe. // stop the execution of the pipe.
@ -646,13 +646,13 @@ impl BinaryPart {
Ok(value.clone()) Ok(value.clone())
} }
BinaryPart::BinaryExpression(binary_expression) => { BinaryPart::BinaryExpression(binary_expression) => {
binary_expression.get_result(memory, &mut new_pipe_info, engine).await binary_expression.get_result(memory, &mut new_pipe_info, ctx).await
} }
BinaryPart::CallExpression(call_expression) => { BinaryPart::CallExpression(call_expression) => {
call_expression.execute(memory, &mut new_pipe_info, engine).await call_expression.execute(memory, &mut new_pipe_info, ctx).await
} }
BinaryPart::UnaryExpression(unary_expression) => { BinaryPart::UnaryExpression(unary_expression) => {
unary_expression.get_result(memory, &mut new_pipe_info, engine).await unary_expression.get_result(memory, &mut new_pipe_info, ctx).await
} }
BinaryPart::MemberExpression(member_expression) => member_expression.get_result(memory), BinaryPart::MemberExpression(member_expression) => member_expression.get_result(memory),
} }
@ -879,7 +879,7 @@ impl CallExpression {
&self, &self,
memory: &mut ProgramMemory, memory: &mut ProgramMemory,
pipe_info: &mut PipeInfo, pipe_info: &mut PipeInfo,
engine: &EngineConnection, ctx: &ExecutorContext,
) -> Result<MemoryItem, KclError> { ) -> Result<MemoryItem, KclError> {
let fn_name = self.callee.name.clone(); let fn_name = self.callee.name.clone();
@ -893,7 +893,7 @@ impl CallExpression {
value.clone() value.clone()
} }
Value::BinaryExpression(binary_expression) => { Value::BinaryExpression(binary_expression) => {
binary_expression.get_result(memory, pipe_info, engine).await? binary_expression.get_result(memory, pipe_info, ctx).await?
} }
Value::CallExpression(call_expression) => { Value::CallExpression(call_expression) => {
// We DO NOT set this gloablly because if we did and this was called inside a pipe it would // We DO NOT set this gloablly because if we did and this was called inside a pipe it would
@ -901,15 +901,11 @@ impl CallExpression {
// THIS IS IMPORTANT. // THIS IS IMPORTANT.
let mut new_pipe_info = pipe_info.clone(); let mut new_pipe_info = pipe_info.clone();
new_pipe_info.is_in_pipe = false; new_pipe_info.is_in_pipe = false;
call_expression.execute(memory, &mut new_pipe_info, engine).await? call_expression.execute(memory, &mut new_pipe_info, ctx).await?
} }
Value::UnaryExpression(unary_expression) => { Value::UnaryExpression(unary_expression) => unary_expression.get_result(memory, pipe_info, ctx).await?,
unary_expression.get_result(memory, pipe_info, engine).await? Value::ObjectExpression(object_expression) => object_expression.execute(memory, pipe_info, ctx).await?,
} Value::ArrayExpression(array_expression) => array_expression.execute(memory, pipe_info, ctx).await?,
Value::ObjectExpression(object_expression) => {
object_expression.execute(memory, pipe_info, engine).await?
}
Value::ArrayExpression(array_expression) => array_expression.execute(memory, pipe_info, engine).await?,
Value::PipeExpression(pipe_expression) => { Value::PipeExpression(pipe_expression) => {
return Err(KclError::Semantic(KclErrorDetails { return Err(KclError::Semantic(KclErrorDetails {
message: format!("PipeExpression not implemented here: {:?}", pipe_expression), message: format!("PipeExpression not implemented here: {:?}", pipe_expression),
@ -941,12 +937,12 @@ impl CallExpression {
match &self.function { match &self.function {
Function::StdLib { func } => { Function::StdLib { func } => {
// Attempt to call the function. // Attempt to call the function.
let args = crate::std::Args::new(fn_args, self.into(), engine.clone()); let args = crate::std::Args::new(fn_args, self.into(), ctx.clone());
let result = func.std_lib_fn()(args).await?; let result = func.std_lib_fn()(args).await?;
if pipe_info.is_in_pipe { if pipe_info.is_in_pipe {
pipe_info.index += 1; pipe_info.index += 1;
pipe_info.previous_results.push(result); pipe_info.previous_results.push(result);
execute_pipe_body(memory, &pipe_info.body.clone(), pipe_info, self.into(), engine).await execute_pipe_body(memory, &pipe_info.body.clone(), pipe_info, self.into(), ctx).await
} else { } else {
Ok(result) Ok(result)
} }
@ -954,7 +950,7 @@ impl CallExpression {
Function::InMemory => { Function::InMemory => {
let func = memory.get(&fn_name, self.into())?; let func = memory.get(&fn_name, self.into())?;
let result = func let result = func
.call_fn(fn_args, memory.clone(), engine.clone()) .call_fn(fn_args, memory.clone(), ctx.clone())
.await? .await?
.ok_or_else(|| { .ok_or_else(|| {
KclError::UndefinedValue(KclErrorDetails { KclError::UndefinedValue(KclErrorDetails {
@ -969,7 +965,7 @@ impl CallExpression {
pipe_info.index += 1; pipe_info.index += 1;
pipe_info.previous_results.push(result); pipe_info.previous_results.push(result);
execute_pipe_body(memory, &pipe_info.body.clone(), pipe_info, self.into(), engine).await execute_pipe_body(memory, &pipe_info.body.clone(), pipe_info, self.into(), ctx).await
} else { } else {
Ok(result) Ok(result)
} }
@ -1507,7 +1503,7 @@ impl ArrayExpression {
&self, &self,
memory: &mut ProgramMemory, memory: &mut ProgramMemory,
pipe_info: &mut PipeInfo, pipe_info: &mut PipeInfo,
engine: &EngineConnection, ctx: &ExecutorContext,
) -> Result<MemoryItem, KclError> { ) -> Result<MemoryItem, KclError> {
let mut results = Vec::with_capacity(self.elements.len()); let mut results = Vec::with_capacity(self.elements.len());
@ -1519,7 +1515,7 @@ impl ArrayExpression {
value.clone() value.clone()
} }
Value::BinaryExpression(binary_expression) => { Value::BinaryExpression(binary_expression) => {
binary_expression.get_result(memory, pipe_info, engine).await? binary_expression.get_result(memory, pipe_info, ctx).await?
} }
Value::CallExpression(call_expression) => { Value::CallExpression(call_expression) => {
// We DO NOT set this gloablly because if we did and this was called inside a pipe it would // We DO NOT set this gloablly because if we did and this was called inside a pipe it would
@ -1527,16 +1523,12 @@ impl ArrayExpression {
// THIS IS IMPORTANT. // THIS IS IMPORTANT.
let mut new_pipe_info = pipe_info.clone(); let mut new_pipe_info = pipe_info.clone();
new_pipe_info.is_in_pipe = false; new_pipe_info.is_in_pipe = false;
call_expression.execute(memory, &mut new_pipe_info, engine).await? call_expression.execute(memory, &mut new_pipe_info, ctx).await?
} }
Value::UnaryExpression(unary_expression) => { Value::UnaryExpression(unary_expression) => unary_expression.get_result(memory, pipe_info, ctx).await?,
unary_expression.get_result(memory, pipe_info, engine).await? Value::ObjectExpression(object_expression) => object_expression.execute(memory, pipe_info, ctx).await?,
} Value::ArrayExpression(array_expression) => array_expression.execute(memory, pipe_info, ctx).await?,
Value::ObjectExpression(object_expression) => { Value::PipeExpression(pipe_expression) => pipe_expression.get_result(memory, pipe_info, ctx).await?,
object_expression.execute(memory, pipe_info, engine).await?
}
Value::ArrayExpression(array_expression) => array_expression.execute(memory, pipe_info, engine).await?,
Value::PipeExpression(pipe_expression) => pipe_expression.get_result(memory, pipe_info, engine).await?,
Value::PipeSubstitution(pipe_substitution) => { Value::PipeSubstitution(pipe_substitution) => {
return Err(KclError::Semantic(KclErrorDetails { return Err(KclError::Semantic(KclErrorDetails {
message: format!("PipeSubstitution not implemented here: {:?}", pipe_substitution), message: format!("PipeSubstitution not implemented here: {:?}", pipe_substitution),
@ -1663,7 +1655,7 @@ impl ObjectExpression {
&self, &self,
memory: &mut ProgramMemory, memory: &mut ProgramMemory,
pipe_info: &mut PipeInfo, pipe_info: &mut PipeInfo,
engine: &EngineConnection, ctx: &ExecutorContext,
) -> Result<MemoryItem, KclError> { ) -> Result<MemoryItem, KclError> {
let mut object = Map::new(); let mut object = Map::new();
for property in &self.properties { for property in &self.properties {
@ -1674,7 +1666,7 @@ impl ObjectExpression {
value.clone() value.clone()
} }
Value::BinaryExpression(binary_expression) => { Value::BinaryExpression(binary_expression) => {
binary_expression.get_result(memory, pipe_info, engine).await? binary_expression.get_result(memory, pipe_info, ctx).await?
} }
Value::CallExpression(call_expression) => { Value::CallExpression(call_expression) => {
// We DO NOT set this gloablly because if we did and this was called inside a pipe it would // We DO NOT set this gloablly because if we did and this was called inside a pipe it would
@ -1682,16 +1674,12 @@ impl ObjectExpression {
// THIS IS IMPORTANT. // THIS IS IMPORTANT.
let mut new_pipe_info = pipe_info.clone(); let mut new_pipe_info = pipe_info.clone();
new_pipe_info.is_in_pipe = false; new_pipe_info.is_in_pipe = false;
call_expression.execute(memory, &mut new_pipe_info, engine).await? call_expression.execute(memory, &mut new_pipe_info, ctx).await?
} }
Value::UnaryExpression(unary_expression) => { Value::UnaryExpression(unary_expression) => unary_expression.get_result(memory, pipe_info, ctx).await?,
unary_expression.get_result(memory, pipe_info, engine).await? Value::ObjectExpression(object_expression) => object_expression.execute(memory, pipe_info, ctx).await?,
} Value::ArrayExpression(array_expression) => array_expression.execute(memory, pipe_info, ctx).await?,
Value::ObjectExpression(object_expression) => { Value::PipeExpression(pipe_expression) => pipe_expression.get_result(memory, pipe_info, ctx).await?,
object_expression.execute(memory, pipe_info, engine).await?
}
Value::ArrayExpression(array_expression) => array_expression.execute(memory, pipe_info, engine).await?,
Value::PipeExpression(pipe_expression) => pipe_expression.get_result(memory, pipe_info, engine).await?,
Value::PipeSubstitution(pipe_substitution) => { Value::PipeSubstitution(pipe_substitution) => {
return Err(KclError::Semantic(KclErrorDetails { return Err(KclError::Semantic(KclErrorDetails {
message: format!("PipeSubstitution not implemented here: {:?}", pipe_substitution), message: format!("PipeSubstitution not implemented here: {:?}", pipe_substitution),
@ -2109,7 +2097,7 @@ impl BinaryExpression {
&self, &self,
memory: &mut ProgramMemory, memory: &mut ProgramMemory,
pipe_info: &mut PipeInfo, pipe_info: &mut PipeInfo,
engine: &EngineConnection, ctx: &ExecutorContext,
) -> Result<MemoryItem, KclError> { ) -> Result<MemoryItem, KclError> {
// We DO NOT set this gloablly because if we did and this was called inside a pipe it would // We DO NOT set this gloablly because if we did and this was called inside a pipe it would
// stop the execution of the pipe. // stop the execution of the pipe.
@ -2119,12 +2107,12 @@ impl BinaryExpression {
let left_json_value = self let left_json_value = self
.left .left
.get_result(memory, &mut new_pipe_info, engine) .get_result(memory, &mut new_pipe_info, ctx)
.await? .await?
.get_json_value()?; .get_json_value()?;
let right_json_value = self let right_json_value = self
.right .right
.get_result(memory, &mut new_pipe_info, engine) .get_result(memory, &mut new_pipe_info, ctx)
.await? .await?
.get_json_value()?; .get_json_value()?;
@ -2283,7 +2271,7 @@ impl UnaryExpression {
&self, &self,
memory: &mut ProgramMemory, memory: &mut ProgramMemory,
pipe_info: &mut PipeInfo, pipe_info: &mut PipeInfo,
engine: &EngineConnection, ctx: &ExecutorContext,
) -> Result<MemoryItem, KclError> { ) -> Result<MemoryItem, KclError> {
// We DO NOT set this gloablly because if we did and this was called inside a pipe it would // We DO NOT set this gloablly because if we did and this was called inside a pipe it would
// stop the execution of the pipe. // stop the execution of the pipe.
@ -2294,7 +2282,7 @@ impl UnaryExpression {
let num = parse_json_number_as_f64( let num = parse_json_number_as_f64(
&self &self
.argument .argument
.get_result(memory, &mut new_pipe_info, engine) .get_result(memory, &mut new_pipe_info, ctx)
.await? .await?
.get_json_value()?, .get_json_value()?,
self.into(), self.into(),
@ -2427,12 +2415,12 @@ impl PipeExpression {
&self, &self,
memory: &mut ProgramMemory, memory: &mut ProgramMemory,
pipe_info: &mut PipeInfo, pipe_info: &mut PipeInfo,
engine: &EngineConnection, ctx: &ExecutorContext,
) -> Result<MemoryItem, KclError> { ) -> Result<MemoryItem, KclError> {
// Reset the previous results. // Reset the previous results.
pipe_info.previous_results = vec![]; pipe_info.previous_results = vec![];
pipe_info.index = 0; pipe_info.index = 0;
execute_pipe_body(memory, &self.body, pipe_info, self.into(), engine).await execute_pipe_body(memory, &self.body, pipe_info, self.into(), ctx).await
} }
/// Rename all identifiers that have the old name to the new given name. /// Rename all identifiers that have the old name to the new given name.
@ -2449,7 +2437,7 @@ async fn execute_pipe_body(
body: &[Value], body: &[Value],
pipe_info: &mut PipeInfo, pipe_info: &mut PipeInfo,
source_range: SourceRange, source_range: SourceRange,
engine: &EngineConnection, ctx: &ExecutorContext,
) -> Result<MemoryItem, KclError> { ) -> Result<MemoryItem, KclError> {
if pipe_info.index == body.len() { if pipe_info.index == body.len() {
pipe_info.is_in_pipe = false; pipe_info.is_in_pipe = false;
@ -2474,15 +2462,15 @@ async fn execute_pipe_body(
match expression { match expression {
Value::BinaryExpression(binary_expression) => { Value::BinaryExpression(binary_expression) => {
let result = binary_expression.get_result(memory, pipe_info, engine).await?; let result = binary_expression.get_result(memory, pipe_info, ctx).await?;
pipe_info.previous_results.push(result); pipe_info.previous_results.push(result);
pipe_info.index += 1; pipe_info.index += 1;
execute_pipe_body(memory, body, pipe_info, source_range, engine).await execute_pipe_body(memory, body, pipe_info, source_range, ctx).await
} }
Value::CallExpression(call_expression) => { Value::CallExpression(call_expression) => {
pipe_info.is_in_pipe = true; pipe_info.is_in_pipe = true;
pipe_info.body = body.to_vec(); pipe_info.body = body.to_vec();
call_expression.execute(memory, pipe_info, engine).await call_expression.execute(memory, pipe_info, ctx).await
} }
_ => { _ => {
// Return an error this should not happen. // Return an error this should not happen.
@ -2775,7 +2763,8 @@ mod tests {
#[test] #[test]
fn test_get_lsp_symbols() { fn test_get_lsp_symbols() {
let code = r#"const part001 = startSketchAt([0.0000000000, 5.0000000000]) let code = r#"const part001 = startSketchOn('XY')
|> startProfileAt([0.0000000000, 5.0000000000], %)
|> line([0.4900857016, -0.0240763666], %) |> line([0.4900857016, -0.0240763666], %)
const part002 = "part002" const part002 = "part002"
@ -2796,25 +2785,10 @@ show(part001)"#;
assert_eq!(symbols.len(), 7); assert_eq!(symbols.len(), 7);
} }
#[test]
fn test_recast_with_std_and_non_stdlib() {
let some_program_string = r#"{"body":[{"type":"VariableDeclaration","start":0,"end":0,"declarations":[{"type":"VariableDeclarator","start":0,"end":0,"id":{"type":"Identifier","start":0,"end":0,"name":"part001"},"init":{"type":"PipeExpression","start":0,"end":0,"body":[{"type":"CallExpression","start":0,"end":0,"callee":{"type":"Identifier","start":0,"end":0,"name":"startSketchAt"},"function":{"type":"StdLib","func":{"name":"startSketchAt","summary":"","description":"","tags":[],"returnValue":{"type":"","required":false,"name":"","schema":{}},"args":[],"unpublished":false,"deprecated":false}},"optional":false,"arguments":[{"type":"Literal","start":0,"end":0,"value":"default","raw":"default"}]},{"type":"CallExpression","start":0,"end":0,"callee":{"type":"Identifier","start":0,"end":0,"name":"ry"},"function":{"type":"InMemory"},"optional":false,"arguments":[{"type":"Literal","start":0,"end":0,"value":90,"raw":"90"},{"type":"PipeSubstitution","start":0,"end":0}]},{"type":"CallExpression","start":0,"end":0,"callee":{"type":"Identifier","start":0,"end":0,"name":"line"},"function":{"type":"StdLib","func":{"name":"line","summary":"","description":"","tags":[],"returnValue":{"type":"","required":false,"name":"","schema":{}},"args":[],"unpublished":false,"deprecated":false}},"optional":false,"arguments":[{"type":"Literal","start":0,"end":0,"value":"default","raw":"default"},{"type":"PipeSubstitution","start":0,"end":0}]}],"nonCodeMeta":{"nonCodeNodes":{},"start":null}}}],"kind":"const"},{"type":"ExpressionStatement","start":0,"end":0,"expression":{"type":"CallExpression","start":0,"end":0,"callee":{"type":"Identifier","start":0,"end":0,"name":"show"},"function":{"type":"StdLib","func":{"name":"show","summary":"","description":"","tags":[],"returnValue":{"type":"","required":false,"name":"","schema":{}},"args":[],"unpublished":false,"deprecated":false}},"optional":false,"arguments":[{"type":"Identifier","start":0,"end":0,"name":"part001"}]}}],"start":0,"end":0,"nonCodeMeta":{"nonCodeNodes":{},"start":null}}"#;
let some_program: crate::ast::types::Program = serde_json::from_str(some_program_string).unwrap();
let recasted = some_program.recast(&Default::default(), 0);
assert_eq!(
recasted,
r#"const part001 = startSketchAt('default')
|> ry(90, %)
|> line('default', %)
show(part001)
"#
);
}
#[test] #[test]
fn test_recast_with_bad_indentation() { fn test_recast_with_bad_indentation() {
let some_program_string = r#"const part001 = startSketchAt([0.0, 5.0]) let some_program_string = r#"const part001 = startSketchOn('XY')
|> startProfileAt([0.0, 5.0], %)
|> line([0.4900857016, -0.0240763666], %) |> line([0.4900857016, -0.0240763666], %)
|> line([0.6804562304, 0.9087880491], %)"#; |> line([0.6804562304, 0.9087880491], %)"#;
let tokens = crate::token::lexer(some_program_string); let tokens = crate::token::lexer(some_program_string);
@ -2824,7 +2798,8 @@ show(part001)
let recasted = program.recast(&Default::default(), 0); let recasted = program.recast(&Default::default(), 0);
assert_eq!( assert_eq!(
recasted, recasted,
r#"const part001 = startSketchAt([0.0, 5.0]) r#"const part001 = startSketchOn('XY')
|> startProfileAt([0.0, 5.0], %)
|> line([0.4900857016, -0.0240763666], %) |> line([0.4900857016, -0.0240763666], %)
|> line([0.6804562304, 0.9087880491], %) |> line([0.6804562304, 0.9087880491], %)
"# "#
@ -2833,7 +2808,8 @@ show(part001)
#[test] #[test]
fn test_recast_with_bad_indentation_and_inline_comment() { fn test_recast_with_bad_indentation_and_inline_comment() {
let some_program_string = r#"const part001 = startSketchAt([0.0, 5.0]) let some_program_string = r#"const part001 = startSketchOn('XY')
|> startProfileAt([0.0, 5.0], %)
|> line([0.4900857016, -0.0240763666], %) // hello world |> line([0.4900857016, -0.0240763666], %) // hello world
|> line([0.6804562304, 0.9087880491], %)"#; |> line([0.6804562304, 0.9087880491], %)"#;
let tokens = crate::token::lexer(some_program_string); let tokens = crate::token::lexer(some_program_string);
@ -2843,7 +2819,8 @@ show(part001)
let recasted = program.recast(&Default::default(), 0); let recasted = program.recast(&Default::default(), 0);
assert_eq!( assert_eq!(
recasted, recasted,
r#"const part001 = startSketchAt([0.0, 5.0]) r#"const part001 = startSketchOn('XY')
|> startProfileAt([0.0, 5.0], %)
|> line([0.4900857016, -0.0240763666], %) // hello world |> line([0.4900857016, -0.0240763666], %) // hello world
|> line([0.6804562304, 0.9087880491], %) |> line([0.6804562304, 0.9087880491], %)
"# "#
@ -2851,7 +2828,8 @@ show(part001)
} }
#[test] #[test]
fn test_recast_with_bad_indentation_and_line_comment() { fn test_recast_with_bad_indentation_and_line_comment() {
let some_program_string = r#"const part001 = startSketchAt([0.0, 5.0]) let some_program_string = r#"const part001 = startSketchOn('XY')
|> startProfileAt([0.0, 5.0], %)
|> line([0.4900857016, -0.0240763666], %) |> line([0.4900857016, -0.0240763666], %)
// hello world // hello world
|> line([0.6804562304, 0.9087880491], %)"#; |> line([0.6804562304, 0.9087880491], %)"#;
@ -2862,7 +2840,8 @@ show(part001)
let recasted = program.recast(&Default::default(), 0); let recasted = program.recast(&Default::default(), 0);
assert_eq!( assert_eq!(
recasted, recasted,
r#"const part001 = startSketchAt([0.0, 5.0]) r#"const part001 = startSketchOn('XY')
|> startProfileAt([0.0, 5.0], %)
|> line([0.4900857016, -0.0240763666], %) |> line([0.4900857016, -0.0240763666], %)
// hello world // hello world
|> line([0.6804562304, 0.9087880491], %) |> line([0.6804562304, 0.9087880491], %)
@ -2904,7 +2883,8 @@ show(part001)
#[test] #[test]
fn test_recast_lots_of_comments() { fn test_recast_lots_of_comments() {
let some_program_string = r#"// comment at start let some_program_string = r#"// comment at start
const mySk1 = startSketchAt([0, 0]) const mySk1 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> lineTo([1, 1], %) |> lineTo([1, 1], %)
// comment here // comment here
|> lineTo({ to: [0, 1], tag: 'myTag' }, %) |> lineTo({ to: [0, 1], tag: 'myTag' }, %)
@ -2926,7 +2906,8 @@ const mySk1 = startSketchAt([0, 0])
assert_eq!( assert_eq!(
recasted, recasted,
r#"// comment at start r#"// comment at start
const mySk1 = startSketchAt([0, 0]) const mySk1 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> lineTo([1, 1], %) |> lineTo([1, 1], %)
// comment here // comment here
|> lineTo({ to: [0, 1], tag: 'myTag' }, %) |> lineTo({ to: [0, 1], tag: 'myTag' }, %)
@ -2946,7 +2927,8 @@ a comment between pipe expression statements */
#[test] #[test]
fn test_recast_multiline_object() { fn test_recast_multiline_object() {
let some_program_string = r#"const part001 = startSketchAt([-0.01, -0.08]) let some_program_string = r#"const part001 = startSketchOn('XY')
|> startProfileAt([-0.01, -0.08], %)
|> line({ to: [0.62, 4.15], tag: 'seg01' }, %) |> line({ to: [0.62, 4.15], tag: 'seg01' }, %)
|> line([2.77, -1.24], %) |> line([2.77, -1.24], %)
|> angledLineThatIntersects({ |> angledLineThatIntersects({
@ -3034,7 +3016,8 @@ const myVar2 = 5
const myVar3 = 6 const myVar3 = 6
const myAng = 40 const myAng = 40
const myAng2 = 134 const myAng2 = 134
const part001 = startSketchAt([0, 0]) const part001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> line({ to: [1, 3.82], tag: 'seg01' }, %) // ln-should-get-tag |> line({ to: [1, 3.82], tag: 'seg01' }, %) // ln-should-get-tag
|> angledLineToX([ |> angledLineToX([
-angleToMatchLengthX('seg01', myVar, %), -angleToMatchLengthX('seg01', myVar, %),
@ -3059,7 +3042,8 @@ const myVar2 = 5
const myVar3 = 6 const myVar3 = 6
const myAng = 40 const myAng = 40
const myAng2 = 134 const myAng2 = 134
const part001 = startSketchAt([0, 0]) const part001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> line({ to: [1, 3.82], tag: 'seg01' }, %) // ln-should-get-tag |> line({ to: [1, 3.82], tag: 'seg01' }, %) // ln-should-get-tag
|> angledLineToX([ |> angledLineToX([
-angleToMatchLengthX('seg01', myVar, %), -angleToMatchLengthX('seg01', myVar, %),
@ -3087,7 +3071,8 @@ const part001 = startSketchAt([0, 0])
#[test] #[test]
fn test_recast_after_rename_std() { fn test_recast_after_rename_std() {
let some_program_string = r#"const part001 = startSketchAt([0.0000000000, 5.0000000000]) let some_program_string = r#"const part001 = startSketchOn('XY')
|> startProfileAt([0.0000000000, 5.0000000000], %)
|> line([0.4900857016, -0.0240763666], %) |> line([0.4900857016, -0.0240763666], %)
const part002 = "part002" const part002 = "part002"
@ -3109,7 +3094,8 @@ show(part001)"#;
let recasted = program.recast(&Default::default(), 0); let recasted = program.recast(&Default::default(), 0);
assert_eq!( assert_eq!(
recasted, recasted,
r#"const mySuperCoolPart = startSketchAt([0.0, 5.0]) r#"const mySuperCoolPart = startSketchOn('XY')
|> startProfileAt([0.0, 5.0], %)
|> line([0.4900857016, -0.0240763666], %) |> line([0.4900857016, -0.0240763666], %)
const part002 = "part002" const part002 = "part002"
@ -3153,7 +3139,8 @@ show(mySuperCoolPart)
const l = 8 const l = 8
const h = 10 const h = 10
const firstExtrude = startSketchAt([0,0]) const firstExtrude = startSketchOn('XY')
|> startProfileAt([0,0], %)
|> line([0, l], %) |> line([0, l], %)
|> line([w, 0], %) |> line([w, 0], %)
|> line([0, -l], %) |> line([0, -l], %)
@ -3172,7 +3159,8 @@ show(firstExtrude)"#;
const l = 8 const l = 8
const h = 10 const h = 10
const firstExtrude = startSketchAt([0, 0]) const firstExtrude = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> line([0, l], %) |> line([0, l], %)
|> line([w, 0], %) |> line([w, 0], %)
|> line([0, -l], %) |> line([0, -l], %)

View File

@ -3,13 +3,15 @@
use std::collections::HashMap; use std::collections::HashMap;
use anyhow::Result; use anyhow::Result;
use kittycad::types::{Color, ModelingCmd, Point3D};
use parse_display::{Display, FromStr};
use schemars::JsonSchema; use schemars::JsonSchema;
use serde::{Deserialize, Serialize}; use serde::{Deserialize, Serialize};
use tower_lsp::lsp_types::{Position as LspPosition, Range as LspRange}; use tower_lsp::lsp_types::{Position as LspPosition, Range as LspRange};
use crate::{ use crate::{
ast::types::{BodyItem, Function, FunctionExpression, Value}, ast::types::{BodyItem, Function, FunctionExpression, Value},
engine::EngineConnection, engine::{EngineConnection, EngineManager},
errors::{KclError, KclErrorDetails}, errors::{KclError, KclErrorDetails},
}; };
@ -101,6 +103,7 @@ impl ProgramReturn {
#[serde(tag = "type")] #[serde(tag = "type")]
pub enum MemoryItem { pub enum MemoryItem {
UserVal(UserVal), UserVal(UserVal),
Plane(Box<Plane>),
SketchGroup(Box<SketchGroup>), SketchGroup(Box<SketchGroup>),
ExtrudeGroup(Box<ExtrudeGroup>), ExtrudeGroup(Box<ExtrudeGroup>),
#[ts(skip)] #[ts(skip)]
@ -115,6 +118,166 @@ pub enum MemoryItem {
}, },
} }
/// A plane.
#[derive(Debug, Clone, Deserialize, Serialize, PartialEq, ts_rs::TS, JsonSchema)]
#[ts(export)]
#[serde(rename_all = "camelCase")]
pub struct Plane {
/// The id of the plane.
pub id: uuid::Uuid,
// The code for the plane either a string or custom.
pub value: PlaneType,
/// Origin of the plane.
pub origin: Point3d,
/// What should the planes X axis be?
pub x_axis: Point3d,
/// What should the planes Y axis be?
pub y_axis: Point3d,
/// The z-axis (normal).
pub z_axis: Point3d,
#[serde(rename = "__meta")]
pub meta: Vec<Metadata>,
}
/// Type for a plane.
#[derive(Debug, Clone, Deserialize, Serialize, PartialEq, ts_rs::TS, JsonSchema, FromStr, Display)]
#[ts(export)]
#[serde(rename_all = "camelCase")]
#[display(style = "camelCase")]
pub enum PlaneType {
#[serde(rename = "XY", alias = "xy")]
#[display("XY")]
XY,
#[serde(rename = "XZ", alias = "xz")]
#[display("XZ")]
XZ,
#[serde(rename = "YZ", alias = "yz")]
#[display("YZ")]
YZ,
/// A custom plane.
#[serde(rename = "Custom")]
#[display("Custom")]
Custom,
}
#[derive(Debug, Clone, Deserialize, Serialize, PartialEq, ts_rs::TS, JsonSchema)]
#[ts(export)]
#[serde(rename_all = "camelCase")]
pub struct DefaultPlanes {
pub xy: uuid::Uuid,
pub xz: uuid::Uuid,
pub yz: uuid::Uuid,
}
impl DefaultPlanes {
pub async fn new(engine: &EngineConnection) -> Result<Self, KclError> {
// Create new default planes.
let default_size = 60.0;
let default_origin = Point3D { x: 0.0, y: 0.0, z: 0.0 };
// Create xy plane.
let xy = uuid::Uuid::new_v4();
engine
.send_modeling_cmd(
xy,
SourceRange::default(),
ModelingCmd::MakePlane {
clobber: false,
origin: default_origin.clone(),
size: default_size,
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?;
// Set the color.
engine
.send_modeling_cmd(
uuid::Uuid::new_v4(),
SourceRange::default(),
ModelingCmd::PlaneSetColor {
color: Color {
r: 0.7,
g: 0.28,
b: 0.28,
a: 0.4,
},
plane_id: xy,
},
)
.await?;
// Create yz plane.
let yz = uuid::Uuid::new_v4();
engine
.send_modeling_cmd(
yz,
SourceRange::default(),
ModelingCmd::MakePlane {
clobber: false,
origin: default_origin.clone(),
size: default_size,
x_axis: Point3D { x: 0.0, y: 1.0, z: 0.0 },
y_axis: Point3D { x: 0.0, y: 0.0, z: 1.0 },
hide: Some(true),
},
)
.await?;
// Set the color.
engine
.send_modeling_cmd(
uuid::Uuid::new_v4(),
SourceRange::default(),
ModelingCmd::PlaneSetColor {
color: Color {
r: 0.28,
g: 0.7,
b: 0.28,
a: 0.4,
},
plane_id: yz,
},
)
.await?;
// Create xz plane.
let xz = uuid::Uuid::new_v4();
engine
.send_modeling_cmd(
xz,
SourceRange::default(),
ModelingCmd::MakePlane {
clobber: false,
origin: default_origin,
size: default_size,
x_axis: Point3D { x: 1.0, y: 0.0, z: 0.0 },
y_axis: Point3D { x: 0.0, y: 0.0, z: 1.0 },
hide: Some(true),
},
)
.await?;
// Set the color.
engine
.send_modeling_cmd(
uuid::Uuid::new_v4(),
SourceRange::default(),
ModelingCmd::PlaneSetColor {
color: Color {
r: 0.28,
g: 0.28,
b: 0.7,
a: 0.4,
},
plane_id: xz,
},
)
.await?;
Ok(Self { xy, xz, yz })
}
}
#[derive(Debug, Clone, Deserialize, Serialize, PartialEq, ts_rs::TS, JsonSchema)] #[derive(Debug, Clone, Deserialize, Serialize, PartialEq, ts_rs::TS, JsonSchema)]
#[ts(export)] #[ts(export)]
#[serde(tag = "type", rename_all = "camelCase")] #[serde(tag = "type", rename_all = "camelCase")]
@ -140,7 +303,7 @@ pub type MemoryFunction =
memory: ProgramMemory, memory: ProgramMemory,
expression: Box<FunctionExpression>, expression: Box<FunctionExpression>,
metadata: Vec<Metadata>, metadata: Vec<Metadata>,
engine: EngineConnection, ctx: ExecutorContext,
) -> std::pin::Pin<Box<dyn std::future::Future<Output = Result<Option<ProgramReturn>, KclError>>>>; ) -> std::pin::Pin<Box<dyn std::future::Future<Output = Result<Option<ProgramReturn>, KclError>>>>;
fn force_memory_function< fn force_memory_function<
@ -149,7 +312,7 @@ fn force_memory_function<
ProgramMemory, ProgramMemory,
Box<FunctionExpression>, Box<FunctionExpression>,
Vec<Metadata>, Vec<Metadata>,
EngineConnection, ExecutorContext,
) -> std::pin::Pin<Box<dyn std::future::Future<Output = Result<Option<ProgramReturn>, KclError>>>>, ) -> std::pin::Pin<Box<dyn std::future::Future<Output = Result<Option<ProgramReturn>, KclError>>>>,
>( >(
f: F, f: F,
@ -165,6 +328,7 @@ impl From<MemoryItem> for Vec<SourceRange> {
MemoryItem::ExtrudeGroup(e) => e.meta.iter().map(|m| m.source_range).collect(), MemoryItem::ExtrudeGroup(e) => e.meta.iter().map(|m| m.source_range).collect(),
MemoryItem::ExtrudeTransform(e) => e.meta.iter().map(|m| m.source_range).collect(), MemoryItem::ExtrudeTransform(e) => e.meta.iter().map(|m| m.source_range).collect(),
MemoryItem::Function { meta, .. } => meta.iter().map(|m| m.source_range).collect(), MemoryItem::Function { meta, .. } => meta.iter().map(|m| m.source_range).collect(),
MemoryItem::Plane(p) => p.meta.iter().map(|m| m.source_range).collect(),
} }
} }
} }
@ -187,11 +351,11 @@ impl MemoryItem {
&self, &self,
args: Vec<MemoryItem>, args: Vec<MemoryItem>,
memory: ProgramMemory, memory: ProgramMemory,
engine: EngineConnection, ctx: ExecutorContext,
) -> Result<Option<ProgramReturn>, KclError> { ) -> Result<Option<ProgramReturn>, KclError> {
if let MemoryItem::Function { func, expression, meta } = &self { if let MemoryItem::Function { func, expression, meta } = &self {
if let Some(func) = func { if let Some(func) = func {
func(args, memory, expression.clone(), meta.clone(), engine).await func(args, memory, expression.clone(), meta.clone(), ctx).await
} else { } else {
Err(KclError::Semantic(KclErrorDetails { Err(KclError::Semantic(KclErrorDetails {
message: format!("Not a function: {:?}", expression), message: format!("Not a function: {:?}", expression),
@ -222,6 +386,8 @@ pub struct SketchGroup {
pub position: Position, pub position: Position,
/// The rotation of the sketch group. /// The rotation of the sketch group.
pub rotation: Rotation, pub rotation: Rotation,
/// The plane id of the sketch group.
pub plane_id: Option<uuid::Uuid>,
/// Metadata. /// Metadata.
#[serde(rename = "__meta")] #[serde(rename = "__meta")]
pub meta: Vec<Metadata>, pub meta: Vec<Metadata>,
@ -414,6 +580,18 @@ pub struct Point3d {
pub z: f64, pub z: f64,
} }
impl Point3d {
pub fn new(x: f64, y: f64, z: f64) -> Self {
Self { x, y, z }
}
}
impl From<Point3d> for kittycad::types::Point3D {
fn from(p: Point3d) -> Self {
Self { x: p.x, y: p.y, z: p.z }
}
}
/// Metadata. /// Metadata.
#[derive(Debug, Clone, Deserialize, Serialize, PartialEq, ts_rs::TS, JsonSchema)] #[derive(Debug, Clone, Deserialize, Serialize, PartialEq, ts_rs::TS, JsonSchema)]
#[ts(export)] #[ts(export)]
@ -593,12 +771,19 @@ impl Default for PipeInfo {
} }
} }
/// The executor context.
#[derive(Debug, Clone)]
pub struct ExecutorContext {
pub engine: EngineConnection,
pub planes: DefaultPlanes,
}
/// Execute a AST's program. /// Execute a AST's program.
pub async fn execute( pub async fn execute(
program: crate::ast::types::Program, program: crate::ast::types::Program,
memory: &mut ProgramMemory, memory: &mut ProgramMemory,
options: BodyType, options: BodyType,
engine: &EngineConnection, ctx: &ExecutorContext,
) -> Result<ProgramMemory, KclError> { ) -> Result<ProgramMemory, KclError> {
let mut pipe_info = PipeInfo::default(); let mut pipe_info = PipeInfo::default();
@ -617,23 +802,23 @@ pub async fn execute(
args.push(memory_item.clone()); args.push(memory_item.clone());
} }
Value::CallExpression(call_expr) => { Value::CallExpression(call_expr) => {
let result = call_expr.execute(memory, &mut pipe_info, engine).await?; let result = call_expr.execute(memory, &mut pipe_info, ctx).await?;
args.push(result); args.push(result);
} }
Value::BinaryExpression(binary_expression) => { Value::BinaryExpression(binary_expression) => {
let result = binary_expression.get_result(memory, &mut pipe_info, engine).await?; let result = binary_expression.get_result(memory, &mut pipe_info, ctx).await?;
args.push(result); args.push(result);
} }
Value::UnaryExpression(unary_expression) => { Value::UnaryExpression(unary_expression) => {
let result = unary_expression.get_result(memory, &mut pipe_info, engine).await?; let result = unary_expression.get_result(memory, &mut pipe_info, ctx).await?;
args.push(result); args.push(result);
} }
Value::ObjectExpression(object_expression) => { Value::ObjectExpression(object_expression) => {
let result = object_expression.execute(memory, &mut pipe_info, engine).await?; let result = object_expression.execute(memory, &mut pipe_info, ctx).await?;
args.push(result); args.push(result);
} }
Value::ArrayExpression(array_expression) => { Value::ArrayExpression(array_expression) => {
let result = array_expression.execute(memory, &mut pipe_info, engine).await?; let result = array_expression.execute(memory, &mut pipe_info, ctx).await?;
args.push(result); args.push(result);
} }
// We do nothing for the rest. // We do nothing for the rest.
@ -651,7 +836,7 @@ pub async fn execute(
memory.return_ = Some(ProgramReturn::Arguments(call_expr.arguments.clone())); memory.return_ = Some(ProgramReturn::Arguments(call_expr.arguments.clone()));
} else if let Some(func) = memory.clone().root.get(&fn_name) { } else if let Some(func) = memory.clone().root.get(&fn_name) {
let result = func.call_fn(args.clone(), memory.clone(), engine.clone()).await?; let result = func.call_fn(args.clone(), memory.clone(), ctx.clone()).await?;
memory.return_ = result; memory.return_ = result;
} else { } else {
@ -677,7 +862,7 @@ pub async fn execute(
memory.add(&var_name, value.clone(), source_range)?; memory.add(&var_name, value.clone(), source_range)?;
} }
Value::BinaryExpression(binary_expression) => { Value::BinaryExpression(binary_expression) => {
let result = binary_expression.get_result(memory, &mut pipe_info, engine).await?; let result = binary_expression.get_result(memory, &mut pipe_info, ctx).await?;
memory.add(&var_name, result, source_range)?; memory.add(&var_name, result, source_range)?;
} }
Value::FunctionExpression(function_expression) => { Value::FunctionExpression(function_expression) => {
@ -686,7 +871,7 @@ pub async fn execute(
memory: ProgramMemory, memory: ProgramMemory,
function_expression: Box<FunctionExpression>, function_expression: Box<FunctionExpression>,
_metadata: Vec<Metadata>, _metadata: Vec<Metadata>,
engine: EngineConnection| { ctx: ExecutorContext| {
Box::pin(async move { Box::pin(async move {
let mut fn_memory = memory.clone(); let mut fn_memory = memory.clone();
@ -714,7 +899,7 @@ pub async fn execute(
function_expression.body.clone(), function_expression.body.clone(),
&mut fn_memory, &mut fn_memory,
BodyType::Block, BodyType::Block,
&engine, &ctx,
) )
.await?; .await?;
@ -733,11 +918,11 @@ pub async fn execute(
)?; )?;
} }
Value::CallExpression(call_expression) => { Value::CallExpression(call_expression) => {
let result = call_expression.execute(memory, &mut pipe_info, engine).await?; let result = call_expression.execute(memory, &mut pipe_info, ctx).await?;
memory.add(&var_name, result, source_range)?; memory.add(&var_name, result, source_range)?;
} }
Value::PipeExpression(pipe_expression) => { Value::PipeExpression(pipe_expression) => {
let result = pipe_expression.get_result(memory, &mut pipe_info, engine).await?; let result = pipe_expression.get_result(memory, &mut pipe_info, ctx).await?;
memory.add(&var_name, result, source_range)?; memory.add(&var_name, result, source_range)?;
} }
Value::PipeSubstitution(pipe_substitution) => { Value::PipeSubstitution(pipe_substitution) => {
@ -750,11 +935,11 @@ pub async fn execute(
})); }));
} }
Value::ArrayExpression(array_expression) => { Value::ArrayExpression(array_expression) => {
let result = array_expression.execute(memory, &mut pipe_info, engine).await?; let result = array_expression.execute(memory, &mut pipe_info, ctx).await?;
memory.add(&var_name, result, source_range)?; memory.add(&var_name, result, source_range)?;
} }
Value::ObjectExpression(object_expression) => { Value::ObjectExpression(object_expression) => {
let result = object_expression.execute(memory, &mut pipe_info, engine).await?; let result = object_expression.execute(memory, &mut pipe_info, ctx).await?;
memory.add(&var_name, result, source_range)?; memory.add(&var_name, result, source_range)?;
} }
Value::MemberExpression(member_expression) => { Value::MemberExpression(member_expression) => {
@ -762,7 +947,7 @@ pub async fn execute(
memory.add(&var_name, result, source_range)?; memory.add(&var_name, result, source_range)?;
} }
Value::UnaryExpression(unary_expression) => { Value::UnaryExpression(unary_expression) => {
let result = unary_expression.get_result(memory, &mut pipe_info, engine).await?; let result = unary_expression.get_result(memory, &mut pipe_info, ctx).await?;
memory.add(&var_name, result, source_range)?; memory.add(&var_name, result, source_range)?;
} }
} }
@ -770,11 +955,11 @@ pub async fn execute(
} }
BodyItem::ReturnStatement(return_statement) => match &return_statement.argument { BodyItem::ReturnStatement(return_statement) => match &return_statement.argument {
Value::BinaryExpression(bin_expr) => { Value::BinaryExpression(bin_expr) => {
let result = bin_expr.get_result(memory, &mut pipe_info, engine).await?; let result = bin_expr.get_result(memory, &mut pipe_info, ctx).await?;
memory.return_ = Some(ProgramReturn::Value(result)); memory.return_ = Some(ProgramReturn::Value(result));
} }
Value::UnaryExpression(unary_expr) => { Value::UnaryExpression(unary_expr) => {
let result = unary_expr.get_result(memory, &mut pipe_info, engine).await?; let result = unary_expr.get_result(memory, &mut pipe_info, ctx).await?;
memory.return_ = Some(ProgramReturn::Value(result)); memory.return_ = Some(ProgramReturn::Value(result));
} }
Value::Identifier(identifier) => { Value::Identifier(identifier) => {
@ -785,15 +970,15 @@ pub async fn execute(
memory.return_ = Some(ProgramReturn::Value(literal.into())); memory.return_ = Some(ProgramReturn::Value(literal.into()));
} }
Value::ArrayExpression(array_expr) => { Value::ArrayExpression(array_expr) => {
let result = array_expr.execute(memory, &mut pipe_info, engine).await?; let result = array_expr.execute(memory, &mut pipe_info, ctx).await?;
memory.return_ = Some(ProgramReturn::Value(result)); memory.return_ = Some(ProgramReturn::Value(result));
} }
Value::ObjectExpression(obj_expr) => { Value::ObjectExpression(obj_expr) => {
let result = obj_expr.execute(memory, &mut pipe_info, engine).await?; let result = obj_expr.execute(memory, &mut pipe_info, ctx).await?;
memory.return_ = Some(ProgramReturn::Value(result)); memory.return_ = Some(ProgramReturn::Value(result));
} }
Value::CallExpression(call_expr) => { Value::CallExpression(call_expr) => {
let result = call_expr.execute(memory, &mut pipe_info, engine).await?; let result = call_expr.execute(memory, &mut pipe_info, ctx).await?;
memory.return_ = Some(ProgramReturn::Value(result)); memory.return_ = Some(ProgramReturn::Value(result));
} }
Value::MemberExpression(member_expr) => { Value::MemberExpression(member_expr) => {
@ -801,7 +986,7 @@ pub async fn execute(
memory.return_ = Some(ProgramReturn::Value(result)); memory.return_ = Some(ProgramReturn::Value(result));
} }
Value::PipeExpression(pipe_expr) => { Value::PipeExpression(pipe_expr) => {
let result = pipe_expr.get_result(memory, &mut pipe_info, engine).await?; let result = pipe_expr.get_result(memory, &mut pipe_info, ctx).await?;
memory.return_ = Some(ProgramReturn::Value(result)); memory.return_ = Some(ProgramReturn::Value(result));
} }
Value::PipeSubstitution(_) => {} Value::PipeSubstitution(_) => {}
@ -825,7 +1010,9 @@ mod tests {
let program = parser.ast()?; let program = parser.ast()?;
let mut mem: ProgramMemory = Default::default(); let mut mem: ProgramMemory = Default::default();
let engine = EngineConnection::new().await?; let engine = EngineConnection::new().await?;
let memory = execute(program, &mut mem, BodyType::Root, &engine).await?; let planes = DefaultPlanes::new(&engine).await?;
let ctx = ExecutorContext { engine, planes };
let memory = execute(program, &mut mem, BodyType::Root, &ctx).await?;
Ok(memory) Ok(memory)
} }
@ -849,7 +1036,8 @@ const newVar = myVar + 1"#;
async fn test_execute_angled_line_that_intersects() { async fn test_execute_angled_line_that_intersects() {
let ast_fn = |offset: &str| -> String { let ast_fn = |offset: &str| -> String {
format!( format!(
r#"const part001 = startSketchAt([0, 0]) r#"const part001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> lineTo({{to:[2, 2], tag: "yo"}}, %) |> lineTo({{to:[2, 2], tag: "yo"}}, %)
|> lineTo([3, 1], %) |> lineTo([3, 1], %)
|> angledLineThatIntersects({{ |> angledLineThatIntersects({{
@ -896,7 +1084,8 @@ const yo = 5 + 6
const abc = 3 const abc = 3
const identifierGuy = 5 const identifierGuy = 5
const part001 = startSketchAt([-1.2, 4.83]) const part001 = startSketchOn('XY')
|> startProfileAt([-1.2, 4.83], %)
|> line([2.8, 0], %) |> line([2.8, 0], %)
|> angledLine([100 + 100, 3.01], %) |> angledLine([100 + 100, 3.01], %)
|> angledLine([abc, 3.02], %) |> angledLine([abc, 3.02], %)
@ -913,7 +1102,8 @@ show(part001)"#;
#[tokio::test(flavor = "multi_thread")] #[tokio::test(flavor = "multi_thread")]
async fn test_execute_with_pipe_substitutions_unary() { async fn test_execute_with_pipe_substitutions_unary() {
let ast = r#"const myVar = 3 let ast = r#"const myVar = 3
const part001 = startSketchAt([0, 0]) const part001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> line({ to: [3, 4], tag: 'seg01' }, %) |> line({ to: [3, 4], tag: 'seg01' }, %)
|> line([ |> line([
min(segLen('seg01', %), myVar), min(segLen('seg01', %), myVar),
@ -928,7 +1118,8 @@ show(part001)"#;
#[tokio::test(flavor = "multi_thread")] #[tokio::test(flavor = "multi_thread")]
async fn test_execute_with_pipe_substitutions() { async fn test_execute_with_pipe_substitutions() {
let ast = r#"const myVar = 3 let ast = r#"const myVar = 3
const part001 = startSketchAt([0, 0]) const part001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> line({ to: [3, 4], tag: 'seg01' }, %) |> line({ to: [3, 4], tag: 'seg01' }, %)
|> line([ |> line([
min(segLen('seg01', %), myVar), min(segLen('seg01', %), myVar),
@ -951,7 +1142,8 @@ const halfArmAngle = armAngle / 2
const arrExpShouldNotBeIncluded = [1, 2, 3] const arrExpShouldNotBeIncluded = [1, 2, 3]
const objExpShouldNotBeIncluded = { a: 1, b: 2, c: 3 } const objExpShouldNotBeIncluded = { a: 1, b: 2, c: 3 }
const part001 = startSketchAt([0, 0]) const part001 = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> yLineTo(1, %) |> yLineTo(1, %)
|> xLine(3.84, %) // selection-range-7ish-before-this |> xLine(3.84, %) // selection-range-7ish-before-this
@ -972,7 +1164,8 @@ fn thing = () => {
return -8 return -8
} }
const firstExtrude = startSketchAt([0,0]) const firstExtrude = startSketchOn('XY')
|> startProfileAt([0,0], %)
|> line([0, l], %) |> line([0, l], %)
|> line([w, 0], %) |> line([w, 0], %)
|> line([0, thing()], %) |> line([0, thing()], %)
@ -994,7 +1187,8 @@ fn thing = (x) => {
return -x return -x
} }
const firstExtrude = startSketchAt([0,0]) const firstExtrude = startSketchOn('XY')
|> startProfileAt([0,0], %)
|> line([0, l], %) |> line([0, l], %)
|> line([w, 0], %) |> line([w, 0], %)
|> line([0, thing(8)], %) |> line([0, thing(8)], %)
@ -1016,7 +1210,8 @@ fn thing = (x) => {
return [0, -x] return [0, -x]
} }
const firstExtrude = startSketchAt([0,0]) const firstExtrude = startSketchOn('XY')
|> startProfileAt([0,0], %)
|> line([0, l], %) |> line([0, l], %)
|> line([w, 0], %) |> line([w, 0], %)
|> line(thing(8), %) |> line(thing(8), %)
@ -1042,7 +1237,8 @@ fn thing = (x) => {
return other_thing(x) return other_thing(x)
} }
const firstExtrude = startSketchAt([0,0]) const firstExtrude = startSketchOn('XY')
|> startProfileAt([0,0], %)
|> line([0, l], %) |> line([0, l], %)
|> line([w, 0], %) |> line([w, 0], %)
|> line([0, thing(8)], %) |> line([0, thing(8)], %)
@ -1057,7 +1253,8 @@ show(firstExtrude)"#;
#[tokio::test(flavor = "multi_thread")] #[tokio::test(flavor = "multi_thread")]
async fn test_execute_with_function_sketch() { async fn test_execute_with_function_sketch() {
let ast = r#"fn box = (h, l, w) => { let ast = r#"fn box = (h, l, w) => {
const myBox = startSketchAt([0,0]) const myBox = startSketchOn('XY')
|> startProfileAt([0,0], %)
|> line([0, l], %) |> line([0, l], %)
|> line([w, 0], %) |> line([w, 0], %)
|> line([0, -l], %) |> line([0, -l], %)
@ -1077,7 +1274,8 @@ show(fnBox)"#;
#[tokio::test(flavor = "multi_thread")] #[tokio::test(flavor = "multi_thread")]
async fn test_get_member_of_object_with_function_period() { async fn test_get_member_of_object_with_function_period() {
let ast = r#"fn box = (obj) => { let ast = r#"fn box = (obj) => {
let myBox = startSketchAt(obj.start) let myBox = startSketchOn('XY')
|> startProfileAt(obj.start, %)
|> line([0, obj.l], %) |> line([0, obj.l], %)
|> line([obj.w, 0], %) |> line([obj.w, 0], %)
|> line([0, -obj.l], %) |> line([0, -obj.l], %)
@ -1097,7 +1295,8 @@ show(thisBox)
#[tokio::test(flavor = "multi_thread")] #[tokio::test(flavor = "multi_thread")]
async fn test_get_member_of_object_with_function_brace() { async fn test_get_member_of_object_with_function_brace() {
let ast = r#"fn box = (obj) => { let ast = r#"fn box = (obj) => {
let myBox = startSketchAt(obj["start"]) let myBox = startSketchOn('XY')
|> startProfileAt(obj["start"], %)
|> line([0, obj["l"]], %) |> line([0, obj["l"]], %)
|> line([obj["w"], 0], %) |> line([obj["w"], 0], %)
|> line([0, -obj["l"]], %) |> line([0, -obj["l"]], %)
@ -1117,7 +1316,8 @@ show(thisBox)
#[tokio::test(flavor = "multi_thread")] #[tokio::test(flavor = "multi_thread")]
async fn test_get_member_of_object_with_function_mix_period_brace() { async fn test_get_member_of_object_with_function_mix_period_brace() {
let ast = r#"fn box = (obj) => { let ast = r#"fn box = (obj) => {
let myBox = startSketchAt(obj["start"]) let myBox = startSketchOn('XY')
|> startProfileAt(obj["start"], %)
|> line([0, obj["l"]], %) |> line([0, obj["l"]], %)
|> line([obj["w"], 0], %) |> line([obj["w"], 0], %)
|> line([10 - obj["w"], -obj.l], %) |> line([10 - obj["w"], -obj.l], %)
@ -1138,7 +1338,8 @@ show(thisBox)
#[ignore] // ignore til we get loops #[ignore] // ignore til we get loops
async fn test_execute_with_function_sketch_loop_objects() { async fn test_execute_with_function_sketch_loop_objects() {
let ast = r#"fn box = (obj) => { let ast = r#"fn box = (obj) => {
let myBox = startSketchAt(obj.start) let myBox = startSketchOn('XY')
|> startProfileAt(obj.start, %)
|> line([0, obj.l], %) |> line([0, obj.l], %)
|> line([obj.w, 0], %) |> line([obj.w, 0], %)
|> line([0, -obj.l], %) |> line([0, -obj.l], %)
@ -1160,7 +1361,8 @@ for var in [{start: [0,0], l: 6, w: 10, h: 3}, {start: [-10,-10], l: 3, w: 5, h:
#[ignore] // ignore til we get loops #[ignore] // ignore til we get loops
async fn test_execute_with_function_sketch_loop_array() { async fn test_execute_with_function_sketch_loop_array() {
let ast = r#"fn box = (h, l, w, start) => { let ast = r#"fn box = (h, l, w, start) => {
const myBox = startSketchAt([0,0]) const myBox = startSketchOn('XY')
|> startProfileAt([0,0], %)
|> line([0, l], %) |> line([0, l], %)
|> line([w, 0], %) |> line([w, 0], %)
|> line([0, -l], %) |> line([0, -l], %)
@ -1182,7 +1384,8 @@ for var in [[3, 6, 10, [0,0]], [1.5, 3, 5, [-10,-10]]] {
#[tokio::test(flavor = "multi_thread")] #[tokio::test(flavor = "multi_thread")]
async fn test_get_member_of_array_with_function() { async fn test_get_member_of_array_with_function() {
let ast = r#"fn box = (array) => { let ast = r#"fn box = (array) => {
let myBox = startSketchAt(array[0]) let myBox =startSketchOn('XY')
|> startProfileAt(array[0], %)
|> line([0, array[1]], %) |> line([0, array[1]], %)
|> line([array[2], 0], %) |> line([array[2], 0], %)
|> line([0, -array[1]], %) |> line([0, -array[1]], %)
@ -1256,7 +1459,8 @@ const leg1 = 5 // inches
const leg2 = 8 // inches const leg2 = 8 // inches
fn thickness = () => { return 0.56 } fn thickness = () => { return 0.56 }
const bracket = startSketchAt([0,0]) const bracket = startSketchOn('XY')
|> startProfileAt([0,0], %)
|> line([0, leg1], %) |> line([0, leg1], %)
|> line([leg2, 0], %) |> line([leg2, 0], %)
|> line([0, -thickness()], %) |> line([0, -thickness()], %)
@ -1280,7 +1484,8 @@ const leg2 = 8 // inches
const thickness_squared = distance * p * FOS * 6 / sigmaAllow const thickness_squared = distance * p * FOS * 6 / sigmaAllow
const thickness = 0.56 // inches. App does not support square root function yet const thickness = 0.56 // inches. App does not support square root function yet
const bracket = startSketchAt([0,0]) const bracket = startSketchOn('XY')
|> startProfileAt([0,0], %)
|> line([0, leg1], %) |> line([0, leg1], %)
|> line([leg2, 0], %) |> line([leg2, 0], %)
|> line([0, -thickness], %) |> line([0, -thickness], %)
@ -1300,7 +1505,8 @@ const leg1 = 5 // inches
const leg2 = 8 // inches const leg2 = 8 // inches
const thickness_squared = (distance * p * FOS * 6 / (sigmaAllow - width)) const thickness_squared = (distance * p * FOS * 6 / (sigmaAllow - width))
const thickness = 0.32 // inches. App does not support square root function yet const thickness = 0.32 // inches. App does not support square root function yet
const bracket = startSketchAt([0,0]) const bracket = startSketchOn('XY')
|> startProfileAt([0,0], %)
|> line([0, leg1], %) |> line([0, leg1], %)
|> line([leg2, 0], %) |> line([leg2, 0], %)
|> line([0, -thickness], %) |> line([0, -thickness], %)
@ -1324,7 +1530,8 @@ const leg1 = 5 // inches
const leg2 = 8 // inches const leg2 = 8 // inches
const thickness_squared = distance * p * FOS * 6 / (sigmaAllow - width) const thickness_squared = distance * p * FOS * 6 / (sigmaAllow - width)
const thickness = 0.32 // inches. App does not support square root function yet const thickness = 0.32 // inches. App does not support square root function yet
const bracket = startSketchAt([0,0]) const bracket = startSketchOn('XY')
|> startProfileAt([0,0], %)
|> line([0, leg1], %) |> line([0, leg1], %)
|> line([leg2, 0], %) |> line([leg2, 0], %)
|> line([0, -thickness], %) |> line([0, -thickness], %)

View File

@ -552,7 +552,7 @@ impl Parser {
_previous_keys: Option<Vec<ObjectKeyInfo>>, _previous_keys: Option<Vec<ObjectKeyInfo>>,
has_opening_brace: bool, has_opening_brace: bool,
) -> Result<Vec<ObjectKeyInfo>, KclError> { ) -> Result<Vec<ObjectKeyInfo>, KclError> {
let previous_keys = _previous_keys.unwrap_or(vec![]); let previous_keys = _previous_keys.unwrap_or_default();
let next_token = self.next_meaningful_token(index, None)?; let next_token = self.next_meaningful_token(index, None)?;
if next_token.index == self.tokens.len() - 1 { if next_token.index == self.tokens.len() - 1 {
return Ok(previous_keys); return Ok(previous_keys);
@ -1918,28 +1918,6 @@ const key = 'c'"#,
31, 31,
); );
assert_eq!(parser.make_non_code_node(index).unwrap(), expected_output); assert_eq!(parser.make_non_code_node(index).unwrap(), expected_output);
let tokens = crate::token::lexer(
r#"const mySketch = startSketchAt([0,0])
|> lineTo({ to: [0, 1], tag: 'myPath' }, %)
|> lineTo([1, 1], %) /* this is
a comment
spanning a few lines */
|> lineTo({ to: [1,0], tag: "rightPath" }, %)
|> close(%)"#,
);
let parser = Parser::new(tokens);
let index = 57;
let expected_output = (
Some(NonCodeNode {
start: 106,
end: 166,
value: NonCodeValue::BlockComment {
value: "this is\n a comment\n spanning a few lines".to_string(),
},
}),
59,
);
assert_eq!(parser.make_non_code_node(index).unwrap(), expected_output);
} }
#[test] #[test]
@ -2070,7 +2048,8 @@ const key = 'c'"#,
fn test_next_meaningful_token() { fn test_next_meaningful_token() {
let _offset = 1; let _offset = 1;
let tokens = crate::token::lexer( let tokens = crate::token::lexer(
r#"const mySketch = startSketchAt([0,0]) r#"const mySketch = startSketchOn('XY')
|> startProfileAt([0,0], %)
|> lineTo({ to: [0, 1], tag: 'myPath' }, %) |> lineTo({ to: [0, 1], tag: 'myPath' }, %)
|> lineTo([1, 1], %) /* this is |> lineTo([1, 1], %) /* this is
a comment a comment
@ -2082,10 +2061,10 @@ const key = 'c'"#,
let index = 17; let index = 17;
let expected_output = TokenReturnWithNonCode { let expected_output = TokenReturnWithNonCode {
token: Some(Token { token: Some(Token {
token_type: TokenType::Brace, token_type: TokenType::Number,
start: 49, start: 60,
end: 50, end: 61,
value: "(".to_string(), value: "0".to_string(),
}), }),
index: 18, index: 18,
non_code_node: None, non_code_node: None,
@ -2095,9 +2074,9 @@ const key = 'c'"#,
let expected_output = TokenReturnWithNonCode { let expected_output = TokenReturnWithNonCode {
token: Some(Token { token: Some(Token {
token_type: TokenType::Brace, token_type: TokenType::Brace,
start: 50, start: 61,
end: 51, end: 62,
value: "{".to_string(), value: "]".to_string(),
}), }),
index: 19, index: 19,
non_code_node: None, non_code_node: None,
@ -2106,10 +2085,10 @@ const key = 'c'"#,
let index = 21; let index = 21;
let expected_output = TokenReturnWithNonCode { let expected_output = TokenReturnWithNonCode {
token: Some(Token { token: Some(Token {
token_type: TokenType::Colon, token_type: TokenType::Operator,
start: 54, start: 64,
end: 55, end: 65,
value: ":".to_string(), value: "%".to_string(),
}), }),
index: 22, index: 22,
non_code_node: None, non_code_node: None,
@ -2118,10 +2097,10 @@ const key = 'c'"#,
let index = 24; let index = 24;
let expected_output = TokenReturnWithNonCode { let expected_output = TokenReturnWithNonCode {
token: Some(Token { token: Some(Token {
token_type: TokenType::Number, token_type: TokenType::Operator,
start: 57, start: 69,
end: 58, end: 71,
value: "0".to_string(), value: "|>".to_string(),
}), }),
index: 25, index: 25,
non_code_node: None, non_code_node: None,
@ -2130,324 +2109,12 @@ const key = 'c'"#,
let index = 25; let index = 25;
let expected_output = TokenReturnWithNonCode { let expected_output = TokenReturnWithNonCode {
token: Some(Token { token: Some(Token {
token_type: TokenType::Comma, token_type: TokenType::Word,
start: 58, start: 72,
end: 59, end: 78,
value: ",".to_string(), value: "lineTo".to_string(),
}), }),
index: 26, index: 27,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 28;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Brace,
start: 61,
end: 62,
value: "]".to_string(),
}),
index: 29,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 29;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Comma,
start: 62,
end: 63,
value: ",".to_string(),
}),
index: 30,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 32;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Colon,
start: 67,
end: 68,
value: ":".to_string(),
}),
index: 33,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 37;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Comma,
start: 79,
end: 80,
value: ",".to_string(),
}),
index: 38,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 40;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Brace,
start: 82,
end: 83,
value: ")".to_string(),
}),
index: 41,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 45;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Brace,
start: 95,
end: 96,
value: "(".to_string(),
}),
index: 46,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 46;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Brace,
start: 96,
end: 97,
value: "[".to_string(),
}),
index: 47,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 47;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Number,
start: 97,
end: 98,
value: "1".to_string(),
}),
index: 48,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 48;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Comma,
start: 98,
end: 99,
value: ",".to_string(),
}),
index: 49,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 51;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Brace,
start: 101,
end: 102,
value: "]".to_string(),
}),
index: 52,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 52;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Comma,
start: 102,
end: 103,
value: ",".to_string(),
}),
index: 53,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 55;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Brace,
start: 105,
end: 106,
value: ")".to_string(),
}),
index: 56,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 62;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Brace,
start: 175,
end: 176,
value: "(".to_string(),
}),
index: 63,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 63;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Brace,
start: 176,
end: 177,
value: "{".to_string(),
}),
index: 64,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 66;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Colon,
start: 180,
end: 181,
value: ":".to_string(),
}),
index: 67,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 69;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Number,
start: 183,
end: 184,
value: "1".to_string(),
}),
index: 70,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 70;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Comma,
start: 184,
end: 185,
value: ",".to_string(),
}),
index: 71,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 71;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Number,
start: 185,
end: 186,
value: "0".to_string(),
}),
index: 72,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 72;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Brace,
start: 186,
end: 187,
value: "]".to_string(),
}),
index: 73,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 73;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Comma,
start: 187,
end: 188,
value: ",".to_string(),
}),
index: 74,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 76;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Colon,
start: 192,
end: 193,
value: ":".to_string(),
}),
index: 77,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 81;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Comma,
start: 207,
end: 208,
value: ",".to_string(),
}),
index: 82,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 84;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Brace,
start: 210,
end: 211,
value: ")".to_string(),
}),
index: 85,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 89;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Brace,
start: 222,
end: 223,
value: "(".to_string(),
}),
index: 90,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 90;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Operator,
start: 223,
end: 224,
value: "%".to_string(),
}),
index: 91,
non_code_node: None,
};
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
let index = 91;
let expected_output = TokenReturnWithNonCode {
token: Some(Token {
token_type: TokenType::Brace,
start: 224,
end: 225,
value: ")".to_string(),
}),
index: 92,
non_code_node: None, non_code_node: None,
}; };
assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output); assert_eq!(parser.next_meaningful_token(index, None).unwrap(), expected_output);
@ -2456,7 +2123,8 @@ const key = 'c'"#,
#[test] #[test]
fn test_find_closing_brace() { fn test_find_closing_brace() {
let tokens = crate::token::lexer( let tokens = crate::token::lexer(
r#"const mySketch = startSketchAt([0,0]) r#"const mySketch = startSketchOn('XY')
|> startProfileAt([0,0], %)
|> lineTo({ to: [0, 1], tag: 'myPath' }, %) |> lineTo({ to: [0, 1], tag: 'myPath' }, %)
|> lineTo([1, 1], %) /* this is |> lineTo([1, 1], %) /* this is
a comment a comment
@ -2465,11 +2133,10 @@ const key = 'c'"#,
|> close(%)"#, |> close(%)"#,
); );
let parser = Parser::new(tokens); let parser = Parser::new(tokens);
assert_eq!(parser.find_closing_brace(7, 0, "").unwrap(), 13); assert_eq!(parser.find_closing_brace(7, 0, "").unwrap(), 9);
assert_eq!(parser.find_closing_brace(18, 0, "").unwrap(), 41); assert_eq!(parser.find_closing_brace(14, 0, "").unwrap(), 23);
assert_eq!(parser.find_closing_brace(46, 0, "").unwrap(), 56); assert_eq!(parser.find_closing_brace(29, 0, "").unwrap(), 47);
assert_eq!(parser.find_closing_brace(63, 0, "").unwrap(), 85); assert_eq!(parser.find_closing_brace(57, 0, "").unwrap(), 62);
assert_eq!(parser.find_closing_brace(90, 0, "").unwrap(), 92);
let basic = "( hey )"; let basic = "( hey )";
let parser = Parser::new(crate::token::lexer(basic)); let parser = Parser::new(crate::token::lexer(basic));
@ -2490,7 +2157,8 @@ const key = 'c'"#,
#[test] #[test]
fn test_is_call_expression() { fn test_is_call_expression() {
let tokens = crate::token::lexer( let tokens = crate::token::lexer(
r#"const mySketch = startSketchAt([0,0]) r#"const mySketch = startSketchOn('XY')
|> startProfileAt([0,0], %)
|> lineTo({ to: [0, 1], tag: 'myPath' }, %) |> lineTo({ to: [0, 1], tag: 'myPath' }, %)
|> lineTo([1, 1], %) /* this is |> lineTo([1, 1], %) /* this is
a comment a comment
@ -2501,8 +2169,8 @@ const key = 'c'"#,
let parser = Parser::new(tokens); let parser = Parser::new(tokens);
assert_eq!(parser.is_call_expression(4).unwrap(), None); assert_eq!(parser.is_call_expression(4).unwrap(), None);
assert_eq!(parser.is_call_expression(6).unwrap(), Some(13)); assert_eq!(parser.is_call_expression(6).unwrap(), Some(9));
assert_eq!(parser.is_call_expression(15).unwrap(), None); assert_eq!(parser.is_call_expression(9).unwrap(), None);
assert_eq!(parser.is_call_expression(43).unwrap(), None); assert_eq!(parser.is_call_expression(43).unwrap(), None);
assert_eq!(parser.is_call_expression(60).unwrap(), None); assert_eq!(parser.is_call_expression(60).unwrap(), None);
assert_eq!(parser.is_call_expression(87).unwrap(), None); assert_eq!(parser.is_call_expression(87).unwrap(), None);
@ -2511,7 +2179,8 @@ const key = 'c'"#,
#[test] #[test]
fn test_find_next_declaration_keyword() { fn test_find_next_declaration_keyword() {
let tokens = crate::token::lexer( let tokens = crate::token::lexer(
r#"const mySketch = startSketchAt([0,0]) r#"const mySketch = startSketchOn('XY')
|> startProfileAt([0,0], %)
|> lineTo({ to: [0, 1], tag: 'myPath' }, %) |> lineTo({ to: [0, 1], tag: 'myPath' }, %)
|> lineTo([1, 1], %) /* this is |> lineTo([1, 1], %) /* this is
a comment a comment
@ -2522,7 +2191,10 @@ const key = 'c'"#,
let parser = Parser::new(tokens); let parser = Parser::new(tokens);
assert_eq!( assert_eq!(
parser.find_next_declaration_keyword(4).unwrap(), parser.find_next_declaration_keyword(4).unwrap(),
TokenReturn { token: None, index: 92 } TokenReturn {
token: None,
index: 102
}
); );
let tokens = crate::token::lexer( let tokens = crate::token::lexer(
@ -2986,7 +2658,8 @@ show(mySk1)"#;
#[test] #[test]
fn test_parse_half_pipe_small() { fn test_parse_half_pipe_small() {
let tokens = crate::token::lexer( let tokens = crate::token::lexer(
"const secondExtrude = startSketchAt([0,0]) "const secondExtrude = startSketchOn('XY')
|> startProfileAt([0,0], %)
|", |",
); );
let parser = Parser::new(tokens); let parser = Parser::new(tokens);
@ -3067,7 +2740,8 @@ const height = [obj["a"] -1, 0]"#,
let tokens = crate::token::lexer( let tokens = crate::token::lexer(
"const height = 10 "const height = 10
const firstExtrude = startSketchAt([0,0]) const firstExtrude = startSketchOn('XY')
|> startProfileAt([0,0], %)
|> line([0, 8], %) |> line([0, 8], %)
|> line([20, 0], %) |> line([20, 0], %)
|> line([0, -8], %) |> line([0, -8], %)
@ -3076,7 +2750,8 @@ const firstExtrude = startSketchAt([0,0])
show(firstExtrude) show(firstExtrude)
const secondExtrude = startSketchAt([0,0]) const secondExtrude = startSketchOn('XY')
|> startProfileAt([0,0], %)
|", |",
); );
let parser = Parser::new(tokens); let parser = Parser::new(tokens);
@ -3457,7 +3132,8 @@ thing(false)
#[test] #[test]
fn test_member_expression_sketch_group() { fn test_member_expression_sketch_group() {
let some_program_string = r#"fn cube = (pos, scale) => { let some_program_string = r#"fn cube = (pos, scale) => {
const sg = startSketchAt(pos) const sg = startSketchOn('XY')
|> startProfileAt(pos, %)
|> line([0, scale], %) |> line([0, scale], %)
|> line([scale, 0], %) |> line([scale, 0], %)
|> line([0, -scale], %) |> line([0, -scale], %)
@ -3499,7 +3175,8 @@ let other_thing = 2 * cos(3)"#;
#[test] #[test]
fn test_negative_arguments() { fn test_negative_arguments() {
let some_program_string = r#"fn box = (p, h, l, w) => { let some_program_string = r#"fn box = (p, h, l, w) => {
const myBox = startSketchAt(p) const myBox = startSketchOn('XY')
|> startProfileAt(p, %)
|> line([0, l], %) |> line([0, l], %)
|> line([w, 0], %) |> line([w, 0], %)
|> line([0, -l], %) |> line([0, -l], %)

View File

@ -17,9 +17,9 @@ use serde::{Deserialize, Serialize};
use crate::{ use crate::{
ast::types::parse_json_number_as_f64, ast::types::parse_json_number_as_f64,
engine::{EngineConnection, EngineManager}, engine::EngineManager,
errors::{KclError, KclErrorDetails}, errors::{KclError, KclErrorDetails},
executor::{ExtrudeGroup, MemoryItem, Metadata, SketchGroup, SourceRange}, executor::{ExecutorContext, ExtrudeGroup, MemoryItem, Metadata, Plane, SketchGroup, SourceRange},
}; };
pub type StdFn = fn(Args) -> std::pin::Pin<Box<dyn std::future::Future<Output = Result<MemoryItem, KclError>>>>; pub type StdFn = fn(Args) -> std::pin::Pin<Box<dyn std::future::Future<Output = Result<MemoryItem, KclError>>>>;
@ -59,6 +59,8 @@ impl StdLib {
Box::new(crate::std::sketch::AngledLineOfYLength), Box::new(crate::std::sketch::AngledLineOfYLength),
Box::new(crate::std::sketch::AngledLineThatIntersects), Box::new(crate::std::sketch::AngledLineThatIntersects),
Box::new(crate::std::sketch::StartSketchAt), Box::new(crate::std::sketch::StartSketchAt),
Box::new(crate::std::sketch::StartSketchOn),
Box::new(crate::std::sketch::StartProfileAt),
Box::new(crate::std::sketch::Close), Box::new(crate::std::sketch::Close),
Box::new(crate::std::sketch::Arc), Box::new(crate::std::sketch::Arc),
Box::new(crate::std::sketch::TangentalArc), Box::new(crate::std::sketch::TangentalArc),
@ -109,15 +111,15 @@ impl Default for StdLib {
pub struct Args { pub struct Args {
pub args: Vec<MemoryItem>, pub args: Vec<MemoryItem>,
pub source_range: SourceRange, pub source_range: SourceRange,
engine: EngineConnection, pub ctx: ExecutorContext,
} }
impl Args { impl Args {
pub fn new(args: Vec<MemoryItem>, source_range: SourceRange, engine: EngineConnection) -> Self { pub fn new(args: Vec<MemoryItem>, source_range: SourceRange, ctx: ExecutorContext) -> Self {
Self { Self {
args, args,
source_range, source_range,
engine, ctx,
} }
} }
@ -126,7 +128,7 @@ impl Args {
id: uuid::Uuid, id: uuid::Uuid,
cmd: kittycad::types::ModelingCmd, cmd: kittycad::types::ModelingCmd,
) -> Result<OkWebSocketResponseData, KclError> { ) -> Result<OkWebSocketResponseData, KclError> {
self.engine.send_modeling_cmd(id, self.source_range, cmd).await self.ctx.engine.send_modeling_cmd(id, self.source_range, cmd).await
} }
fn make_user_val_from_json(&self, j: serde_json::Value) -> Result<MemoryItem, KclError> { fn make_user_val_from_json(&self, j: serde_json::Value) -> Result<MemoryItem, KclError> {
@ -308,6 +310,44 @@ impl Args {
Ok((data, sketch_group)) Ok((data, sketch_group))
} }
fn get_data_and_plane<T: serde::de::DeserializeOwned>(&self) -> Result<(T, Box<Plane>), KclError> {
let first_value = self
.args
.first()
.ok_or_else(|| {
KclError::Type(KclErrorDetails {
message: format!("Expected a struct as the first argument, found `{:?}`", self.args),
source_ranges: vec![self.source_range],
})
})?
.get_json_value()?;
let data: T = serde_json::from_value(first_value).map_err(|e| {
KclError::Type(KclErrorDetails {
message: format!("Failed to deserialize struct from JSON: {}", e),
source_ranges: vec![self.source_range],
})
})?;
let second_value = self.args.get(1).ok_or_else(|| {
KclError::Type(KclErrorDetails {
message: format!("Expected a Plane as the second argument, found `{:?}`", self.args),
source_ranges: vec![self.source_range],
})
})?;
let plane = if let MemoryItem::Plane(p) = second_value {
p.clone()
} else {
return Err(KclError::Type(KclErrorDetails {
message: format!("Expected a Plane as the second argument, found `{:?}`", self.args),
source_ranges: vec![self.source_range],
}));
};
Ok((data, plane))
}
fn get_segment_name_to_number_sketch_group(&self) -> Result<(String, f64, Box<SketchGroup>), KclError> { fn get_segment_name_to_number_sketch_group(&self) -> Result<(String, f64, Box<SketchGroup>), KclError> {
// Iterate over our args, the first argument should be a UserVal with a string value. // Iterate over our args, the first argument should be a UserVal with a string value.
// The second argument should be a number. // The second argument should be a number.

View File

@ -9,7 +9,9 @@ use serde::{Deserialize, Serialize};
use super::utils::Angle; use super::utils::Angle;
use crate::{ use crate::{
errors::{KclError, KclErrorDetails}, errors::{KclError, KclErrorDetails},
executor::{BasePath, GeoMeta, MemoryItem, Path, Point2d, Position, Rotation, SketchGroup}, executor::{
BasePath, GeoMeta, MemoryItem, Path, Plane, PlaneType, Point2d, Point3d, Position, Rotation, SketchGroup,
},
std::{ std::{
utils::{arc_angles, arc_center_and_end, get_x_component, get_y_component, intersection_with_parallel_line}, utils::{arc_angles, arc_center_and_end, get_x_component, get_y_component, intersection_with_parallel_line},
Args, Args,
@ -649,11 +651,202 @@ pub async fn start_sketch_at(args: Args) -> Result<MemoryItem, KclError> {
Ok(MemoryItem::SketchGroup(sketch_group)) Ok(MemoryItem::SketchGroup(sketch_group))
} }
/// Start a sketch at a given point. /// Start a sketch at a given point on the 'XY' plane.
#[stdlib { #[stdlib {
name = "startSketchAt", name = "startSketchAt",
}] }]
async fn inner_start_sketch_at(data: LineData, args: Args) -> Result<Box<SketchGroup>, KclError> { async fn inner_start_sketch_at(data: LineData, args: Args) -> Result<Box<SketchGroup>, KclError> {
// Let's assume it's the XY plane for now, this is just for backwards compatibility.
let xy_plane = PlaneData::XY;
let plane = inner_start_sketch_on(xy_plane, args.clone()).await?;
let sketch_group = inner_start_profile_at(data, plane, args).await?;
Ok(sketch_group)
}
/// Data for a plane.
#[derive(Debug, Clone, Deserialize, Serialize, PartialEq, ts_rs::TS, JsonSchema)]
#[ts(export)]
#[serde(rename_all = "camelCase")]
pub enum PlaneData {
/// The XY plane.
#[serde(rename = "XY", alias = "xy")]
XY,
/// The opposite side of the XY plane.
#[serde(rename = "-XY", alias = "-xy")]
NegXY,
/// The XZ plane.
#[serde(rename = "XZ", alias = "xz")]
XZ,
/// The opposite side of the XZ plane.
#[serde(rename = "-XZ", alias = "-xz")]
NegXZ,
/// The YZ plane.
#[serde(rename = "YZ", alias = "yz")]
YZ,
/// The opposite side of the YZ plane.
#[serde(rename = "-YZ", alias = "-yz")]
NegYZ,
/// A defined plane.
Plane {
/// Origin of the plane.
origin: Box<Point3d>,
/// What should the planes X axis be?
x_axis: Box<Point3d>,
/// What should the planes Y axis be?
y_axis: Box<Point3d>,
/// The z-axis (normal).
z_axis: Box<Point3d>,
},
}
impl From<PlaneData> for Plane {
fn from(value: PlaneData) -> Self {
let id = uuid::Uuid::new_v4();
match value {
PlaneData::XY => Plane {
id,
origin: Point3d::new(0.0, 0.0, 0.0),
x_axis: Point3d::new(1.0, 0.0, 0.0),
y_axis: Point3d::new(0.0, 1.0, 0.0),
z_axis: Point3d::new(0.0, 0.0, 1.0),
value: PlaneType::XY,
meta: vec![],
},
PlaneData::NegXY => Plane {
id,
origin: Point3d::new(0.0, 0.0, 0.0),
x_axis: Point3d::new(1.0, 0.0, 0.0),
y_axis: Point3d::new(0.0, 1.0, 0.0),
z_axis: Point3d::new(0.0, 0.0, -1.0),
value: PlaneType::XY,
meta: vec![],
},
PlaneData::XZ => Plane {
id,
origin: Point3d::new(0.0, 0.0, 0.0),
x_axis: Point3d::new(1.0, 0.0, 0.0),
y_axis: Point3d::new(0.0, 0.0, 1.0),
z_axis: Point3d::new(0.0, 1.0, 0.0),
value: PlaneType::XZ,
meta: vec![],
},
PlaneData::NegXZ => Plane {
id,
origin: Point3d::new(0.0, 0.0, 0.0),
x_axis: Point3d::new(1.0, 0.0, 0.0),
y_axis: Point3d::new(0.0, 0.0, 1.0),
z_axis: Point3d::new(0.0, -1.0, 0.0),
value: PlaneType::XZ,
meta: vec![],
},
PlaneData::YZ => Plane {
id,
origin: Point3d::new(0.0, 0.0, 0.0),
x_axis: Point3d::new(0.0, 1.0, 0.0),
y_axis: Point3d::new(0.0, 0.0, 1.0),
z_axis: Point3d::new(1.0, 0.0, 0.0),
value: PlaneType::YZ,
meta: vec![],
},
PlaneData::NegYZ => Plane {
id,
origin: Point3d::new(0.0, 0.0, 0.0),
x_axis: Point3d::new(0.0, 1.0, 0.0),
y_axis: Point3d::new(0.0, 0.0, 1.0),
z_axis: Point3d::new(-1.0, 0.0, 0.0),
value: PlaneType::YZ,
meta: vec![],
},
PlaneData::Plane {
origin,
x_axis,
y_axis,
z_axis,
} => Plane {
id,
origin: *origin,
x_axis: *x_axis,
y_axis: *y_axis,
z_axis: *z_axis,
value: PlaneType::Custom,
meta: vec![],
},
}
}
}
/// Start a sketch on a specific plane.
pub async fn start_sketch_on(args: Args) -> Result<MemoryItem, KclError> {
let data: PlaneData = args.get_data()?;
let plane = inner_start_sketch_on(data, args).await?;
Ok(MemoryItem::Plane(plane))
}
/// Start a sketch at a given point.
#[stdlib {
name = "startSketchOn",
}]
async fn inner_start_sketch_on(data: PlaneData, args: Args) -> Result<Box<Plane>, KclError> {
let mut plane: Plane = data.clone().into();
plane.id = match data {
PlaneData::XY | PlaneData::NegXY => args.ctx.planes.xy,
PlaneData::XZ | PlaneData::NegXZ => args.ctx.planes.xz,
PlaneData::YZ | PlaneData::NegYZ => args.ctx.planes.yz,
PlaneData::Plane {
origin,
x_axis,
y_axis,
z_axis: _,
} => {
let id = uuid::Uuid::new_v4();
// Create the plane.
args.send_modeling_cmd(
id,
ModelingCmd::MakePlane {
clobber: false,
origin: (*origin).into(),
size: 60.0,
x_axis: (*x_axis).into(),
y_axis: (*y_axis).into(),
hide: Some(true),
},
)
.await?;
id
}
};
// Enter sketch mode on the plane.
args.send_modeling_cmd(
uuid::Uuid::new_v4(),
ModelingCmd::SketchModeEnable {
animated: false,
ortho: false,
plane_id: plane.id,
// We pass in the normal for the plane here.
disable_camera_with_plane: Some(plane.z_axis.clone().into()),
},
)
.await?;
Ok(Box::new(plane))
}
/// Start a profile at a given point.
pub async fn start_profile_at(args: Args) -> Result<MemoryItem, KclError> {
let (data, plane): (LineData, Box<Plane>) = args.get_data_and_plane()?;
let sketch_group = inner_start_profile_at(data, plane, args).await?;
Ok(MemoryItem::SketchGroup(sketch_group))
}
/// Start a profile at a given point.
#[stdlib {
name = "startProfileAt",
}]
async fn inner_start_profile_at(data: LineData, plane: Box<Plane>, args: Args) -> Result<Box<SketchGroup>, KclError> {
let to = match &data { let to = match &data {
LineData::PointWithTag { to, .. } => *to, LineData::PointWithTag { to, .. } => *to,
LineData::Point(to) => *to, LineData::Point(to) => *to,
@ -694,6 +887,7 @@ async fn inner_start_sketch_at(data: LineData, args: Args) -> Result<Box<SketchG
id: path_id, id: path_id,
position: Position([0.0, 0.0, 0.0]), position: Position([0.0, 0.0, 0.0]),
rotation: Rotation([0.0, 0.0, 0.0, 1.0]), rotation: Rotation([0.0, 0.0, 0.0, 1.0]),
plane_id: Some(plane.id),
value: vec![], value: vec![],
start: current_path, start: current_path,
meta: vec![args.source_range.into()], meta: vec![args.source_range.into()],
@ -728,6 +922,13 @@ async fn inner_close(sketch_group: Box<SketchGroup>, args: Args) -> Result<Box<S
) )
.await?; .await?;
// Exit sketch mode, since if we were in a plane we'd want to disable the sketch mode after.
if sketch_group.plane_id.is_some() {
// We were on a plane, disable the sketch mode.
args.send_modeling_cmd(uuid::Uuid::new_v4(), ModelingCmd::SketchModeDisable {})
.await?;
}
let mut new_sketch_group = sketch_group.clone(); let mut new_sketch_group = sketch_group.clone();
new_sketch_group.value.push(Path::ToPoint { new_sketch_group.value.push(Path::ToPoint {
base: BasePath { base: BasePath {
@ -1199,7 +1400,7 @@ mod tests {
use pretty_assertions::assert_eq; use pretty_assertions::assert_eq;
use crate::std::sketch::LineData; use crate::std::sketch::{LineData, PlaneData};
#[test] #[test]
fn test_deserialize_line_data() { fn test_deserialize_line_data() {
@ -1221,4 +1422,23 @@ mod tests {
} }
); );
} }
#[test]
fn test_deserialize_plane_data() {
let data = PlaneData::XY;
let mut str_json = serde_json::to_string(&data).unwrap();
assert_eq!(str_json, "\"XY\"");
str_json = "\"YZ\"".to_string();
let data: PlaneData = serde_json::from_str(&str_json).unwrap();
assert_eq!(data, PlaneData::YZ);
str_json = "\"-YZ\"".to_string();
let data: PlaneData = serde_json::from_str(&str_json).unwrap();
assert_eq!(data, PlaneData::NegYZ);
str_json = "\"-xz\"".to_string();
let data: PlaneData = serde_json::from_str(&str_json).unwrap();
assert_eq!(data, PlaneData::NegXZ);
}
} }

View File

@ -1434,13 +1434,13 @@ const things = "things"
fn test_kitt() { fn test_kitt() {
let program = include_str!("../../../tests/executor/inputs/kittycad_svg.kcl"); let program = include_str!("../../../tests/executor/inputs/kittycad_svg.kcl");
let actual = lexer(program).unwrap(); let actual = lexer(program).unwrap();
assert_eq!(actual.len(), 5088); assert_eq!(actual.len(), 5098);
} }
#[test] #[test]
fn test_pipes_on_pipes() { fn test_pipes_on_pipes() {
let program = include_str!("../../../tests/executor/inputs/pipes_on_pipes.kcl"); let program = include_str!("../../../tests/executor/inputs/pipes_on_pipes.kcl");
let actual = lexer(program).unwrap(); let actual = lexer(program).unwrap();
assert_eq!(actual.len(), 17836); assert_eq!(actual.len(), 17846);
} }
#[test] #[test]
fn test_lexer_negative_word() { fn test_lexer_negative_word() {

View File

@ -16,16 +16,21 @@ pub async fn execute_wasm(
program_str: &str, program_str: &str,
memory_str: &str, memory_str: &str,
manager: kcl_lib::engine::conn_wasm::EngineCommandManager, manager: kcl_lib::engine::conn_wasm::EngineCommandManager,
planes_str: &str,
) -> Result<JsValue, String> { ) -> Result<JsValue, String> {
// deserialize the ast from a stringified json // deserialize the ast from a stringified json
use kcl_lib::executor::ExecutorContext;
let program: kcl_lib::ast::types::Program = serde_json::from_str(program_str).map_err(|e| e.to_string())?; let program: kcl_lib::ast::types::Program = serde_json::from_str(program_str).map_err(|e| e.to_string())?;
let planes: kcl_lib::executor::DefaultPlanes = serde_json::from_str(planes_str).map_err(|e| e.to_string())?;
let mut mem: kcl_lib::executor::ProgramMemory = serde_json::from_str(memory_str).map_err(|e| e.to_string())?; let mut mem: kcl_lib::executor::ProgramMemory = serde_json::from_str(memory_str).map_err(|e| e.to_string())?;
let engine = kcl_lib::engine::EngineConnection::new(manager) let engine = kcl_lib::engine::EngineConnection::new(manager)
.await .await
.map_err(|e| format!("{:?}", e))?; .map_err(|e| format!("{:?}", e))?;
let ctx = ExecutorContext { engine, planes };
let memory = kcl_lib::executor::execute(program, &mut mem, kcl_lib::executor::BodyType::Root, &engine) let memory = kcl_lib::executor::execute(program, &mut mem, kcl_lib::executor::BodyType::Root, &ctx)
.await .await
.map_err(String::from)?; .map_err(String::from)?;
// The serde-wasm-bindgen does not work here because of weird HashMap issues so we use the // The serde-wasm-bindgen does not work here because of weird HashMap issues so we use the
@ -40,11 +45,14 @@ pub async fn modify_ast_for_sketch_wasm(
manager: kcl_lib::engine::conn_wasm::EngineCommandManager, manager: kcl_lib::engine::conn_wasm::EngineCommandManager,
program_str: &str, program_str: &str,
sketch_name: &str, sketch_name: &str,
plane_type: &str,
sketch_id: &str, sketch_id: &str,
) -> Result<JsValue, String> { ) -> Result<JsValue, String> {
// deserialize the ast from a stringified json // deserialize the ast from a stringified json
let mut program: kcl_lib::ast::types::Program = serde_json::from_str(program_str).map_err(|e| e.to_string())?; let mut program: kcl_lib::ast::types::Program = serde_json::from_str(program_str).map_err(|e| e.to_string())?;
let plane: kcl_lib::executor::PlaneType = serde_json::from_str(plane_type).map_err(|e| e.to_string())?;
let mut engine = kcl_lib::engine::EngineConnection::new(manager) let mut engine = kcl_lib::engine::EngineConnection::new(manager)
.await .await
.map_err(|e| format!("{:?}", e))?; .map_err(|e| format!("{:?}", e))?;
@ -53,6 +61,7 @@ pub async fn modify_ast_for_sketch_wasm(
&mut engine, &mut engine,
&mut program, &mut program,
sketch_name, sketch_name,
plane,
uuid::Uuid::parse_str(sketch_id).map_err(|e| e.to_string())?, uuid::Uuid::parse_str(sketch_id).map_err(|e| e.to_string())?,
) )
.await .await

View File

@ -1,4 +1,5 @@
const svg = startSketchAt([0, 0]) const svg = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> lineTo([2.52, -26.04], %) // MoveAbsolute |> lineTo([2.52, -26.04], %) // MoveAbsolute
|> lineTo([2.52, -25.2], %) // VerticalLineAbsolute |> lineTo([2.52, -25.2], %) // VerticalLineAbsolute

View File

@ -1,4 +1,5 @@
const svg = startSketchAt([0, 0]) const svg = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> lineTo([22.687663, -2.7664351], %) // MoveRelative |> lineTo([22.687663, -2.7664351], %) // MoveRelative
|> lineTo([15.687664000000002, -5.7664351], %) // MoveRelative |> lineTo([15.687664000000002, -5.7664351], %) // MoveRelative

View File

@ -37,10 +37,13 @@ async fn execute_and_snapshot(code: &str) -> Result<image::DynamicImage> {
let program = parser.ast()?; let program = parser.ast()?;
let mut mem: kcl_lib::executor::ProgramMemory = Default::default(); let mut mem: kcl_lib::executor::ProgramMemory = Default::default();
let engine = kcl_lib::engine::EngineConnection::new(ws).await?; let engine = kcl_lib::engine::EngineConnection::new(ws).await?;
let _ = kcl_lib::executor::execute(program, &mut mem, kcl_lib::executor::BodyType::Root, &engine).await?; let planes = kcl_lib::executor::DefaultPlanes::new(&engine).await?;
let ctx = kcl_lib::executor::ExecutorContext { engine, planes };
let _ = kcl_lib::executor::execute(program, &mut mem, kcl_lib::executor::BodyType::Root, &ctx).await?;
// Send a snapshot request to the engine. // Send a snapshot request to the engine.
let resp = engine let resp = ctx
.engine
.send_modeling_cmd( .send_modeling_cmd(
uuid::Uuid::new_v4(), uuid::Uuid::new_v4(),
kcl_lib::executor::SourceRange::default(), kcl_lib::executor::SourceRange::default(),
@ -68,7 +71,8 @@ async fn execute_and_snapshot(code: &str) -> Result<image::DynamicImage> {
#[tokio::test(flavor = "multi_thread")] #[tokio::test(flavor = "multi_thread")]
async fn serial_test_execute_with_function_sketch() { async fn serial_test_execute_with_function_sketch() {
let code = r#"fn box = (h, l, w) => { let code = r#"fn box = (h, l, w) => {
const myBox = startSketchAt([0,0]) const myBox = startSketchOn('XY')
|> startProfileAt([0,0], %)
|> line([0, l], %) |> line([0, l], %)
|> line([w, 0], %) |> line([w, 0], %)
|> line([0, -l], %) |> line([0, -l], %)
@ -89,7 +93,8 @@ show(fnBox)"#;
#[tokio::test(flavor = "multi_thread")] #[tokio::test(flavor = "multi_thread")]
async fn serial_test_execute_with_function_sketch_with_position() { async fn serial_test_execute_with_function_sketch_with_position() {
let code = r#"fn box = (p, h, l, w) => { let code = r#"fn box = (p, h, l, w) => {
const myBox = startSketchAt(p) const myBox = startSketchOn('XY')
|> startProfileAt(p, %)
|> line([0, l], %) |> line([0, l], %)
|> line([w, 0], %) |> line([w, 0], %)
|> line([0, -l], %) |> line([0, -l], %)
@ -107,7 +112,8 @@ show(box([0,0], 3, 6, 10))"#;
#[tokio::test(flavor = "multi_thread")] #[tokio::test(flavor = "multi_thread")]
async fn serial_test_execute_with_angled_line() { async fn serial_test_execute_with_angled_line() {
let code = r#"const part001 = startSketchAt([4.83, 12.56]) let code = r#"const part001 = startSketchOn('XY')
|> startProfileAt([4.83, 12.56], %)
|> line([15.1, 2.48], %) |> line([15.1, 2.48], %)
|> line({ to: [3.15, -9.85], tag: 'seg01' }, %) |> line({ to: [3.15, -9.85], tag: 'seg01' }, %)
|> line([-15.17, -4.1], %) |> line([-15.17, -4.1], %)
@ -133,7 +139,8 @@ const FOS = 2
const leg1 = 5 // inches const leg1 = 5 // inches
const leg2 = 8 // inches const leg2 = 8 // inches
const thickness = sqrt(distance * p * FOS * 6 / sigmaAllow / width) // inches const thickness = sqrt(distance * p * FOS * 6 / sigmaAllow / width) // inches
const bracket = startSketchAt([0, 0]) const bracket = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> line([0, leg1], %) |> line([0, leg1], %)
|> line([leg2, 0], %) |> line([leg2, 0], %)
|> line([0, -thickness], %) |> line([0, -thickness], %)
@ -185,7 +192,8 @@ show(bracket)"#;
#[tokio::test(flavor = "multi_thread")] #[tokio::test(flavor = "multi_thread")]
async fn serial_test_execute_engine_error_return() { async fn serial_test_execute_engine_error_return() {
let code = r#"const part001 = startSketchAt([5.5229, 5.25217]) let code = r#"const part001 = startSketchOn('XY')
|> startProfileAt([5.5229, 5.25217], %)
|> line([10.50433, -1.19122], %) |> line([10.50433, -1.19122], %)
|> line([8.01362, -5.48731], %) |> line([8.01362, -5.48731], %)
|> line([-1.02877, -6.76825], %) |> line([-1.02877, -6.76825], %)
@ -197,7 +205,7 @@ async fn serial_test_execute_engine_error_return() {
assert!(result.is_err()); assert!(result.is_err());
assert_eq!( assert_eq!(
result.err().unwrap().to_string(), result.err().unwrap().to_string(),
r#"engine: KclErrorDetails { source_ranges: [SourceRange([193, 206])], message: "Modeling command failed: Some([ApiError { error_code: BadRequest, message: \"The path is not closed. Solid2D construction requires a closed path!\" }])" }"#, r#"engine: KclErrorDetails { source_ranges: [SourceRange([222, 235])], message: "Modeling command failed: Some([ApiError { error_code: BadRequest, message: \"The path is not closed. Solid2D construction requires a closed path!\" }])" }"#,
); );
} }
@ -221,7 +229,8 @@ async fn serial_test_execute_kittycad_svg() {
#[tokio::test(flavor = "multi_thread")] #[tokio::test(flavor = "multi_thread")]
async fn test_member_expression_sketch_group() { async fn test_member_expression_sketch_group() {
let code = r#"fn cube = (pos, scale) => { let code = r#"fn cube = (pos, scale) => {
const sg = startSketchAt(pos) const sg = startSketchOn('XY')
|> startProfileAt(pos, %)
|> line([0, scale], %) |> line([0, scale], %)
|> line([scale, 0], %) |> line([scale, 0], %)
|> line([0, -scale], %) |> line([0, -scale], %)
@ -252,7 +261,8 @@ async fn test_close_arc() {
const radius = 40 const radius = 40
const height = 3 const height = 3
const body = startSketchAt([center[0]+radius, center[1]]) const body = startSketchOn('XY')
|> startProfileAt([center[0]+radius, center[1]], %)
|> arc({angle_end: 360, angle_start: 0, radius: radius}, %) |> arc({angle_end: 360, angle_start: 0, radius: radius}, %)
|> close(%) |> close(%)
|> extrude(height, %) |> extrude(height, %)
@ -270,7 +280,8 @@ const height = 10
const length = 12 const length = 12
fn box = (sk1, sk2, scale) => { fn box = (sk1, sk2, scale) => {
const boxSketch = startSketchAt([sk1, sk2]) const boxSketch = startSketchOn('XY')
|> startProfileAt([sk1, sk2], %)
|> line([0, scale], %) |> line([0, scale], %)
|> line([scale, 0], %) |> line([scale, 0], %)
|> line([0, -scale], %) |> line([0, -scale], %)
@ -326,3 +337,89 @@ async fn test_basic_tangental_arc_to() {
let result = execute_and_snapshot(code).await.unwrap(); let result = execute_and_snapshot(code).await.unwrap();
twenty_twenty::assert_image("tests/executor/outputs/tangental_arc_to.png", &result, 1.0); twenty_twenty::assert_image("tests/executor/outputs/tangental_arc_to.png", &result, 1.0);
} }
#[tokio::test(flavor = "multi_thread")]
async fn test_different_planes_same_drawing() {
let code = r#"const width = 5
const height = 10
const length = 12
fn box = (sk1, sk2, scale, plane) => {
const boxsketch = startSketchOn(plane)
|> startProfileAt([sk1, sk2], %)
|> line([0, scale], %)
|> line([scale, 0], %)
|> line([0, -scale], %)
|> close(%)
|> extrude(scale, %)
return boxsketch
}
box(0, 0, 5, 'xy')
box(10, 23, 8, 'xz')
box(30, 43, 18, '-xy')
let thing = box(-12, -15, 10, 'yz')
box(-20, -5, 10, 'xy')"#;
let result = execute_and_snapshot(code).await.unwrap();
twenty_twenty::assert_image("tests/executor/outputs/different_planes_same_drawing.png", &result, 1.0);
}
#[tokio::test(flavor = "multi_thread")]
async fn test_lots_of_planes() {
let code = r#"const sigmaAllow = 15000 // psi
const width = 11 // inch
const p = 150 // Force on shelf - lbs
const distance = 12 // inches
const FOS = 2
const thickness = sqrt(distance * p * FOS * 6 / (sigmaAllow * width))
const filletR = thickness * 2
const shelfMountL = 9
const wallMountL = 8
const bracket = startSketchOn('XY')
|> startProfileAt([0, 0], %)
|> line([0, wallMountL], %)
|> tangentalArc({ radius: filletR, offset: 90 }, %)
|> line([-shelfMountL, 0], %)
|> line([0, -thickness], %)
|> line([shelfMountL, 0], %)
|> tangentalArc({
radius: filletR - thickness,
offset: -90
}, %)
|> line([0, -wallMountL], %)
|> close(%)
|> extrude(width, %)
show(bracket)
const part001 = startSketchOn('XY')
|> startProfileAt([-15.53, -10.28], %)
|> line([10.49, -2.08], %)
|> line([10.42, 8.47], %)
|> line([-19.16, 5.1], %)
|> close(%)
|> extrude(4, %)
const part002 = startSketchOn('-XZ')
|> startProfileAt([-9.35, 19.18], %)
|> line([32.14, -2.47], %)
|> line([8.39, -3.73], %)
|> close(%)
const part003 = startSketchOn('-XZ')
|> startProfileAt([13.82, 16.51], %)
|> line([-6.24, -30.82], %)
|> line([8.39, -3.73], %)
|> close(%)
const part004 = startSketchOn('YZ')
|> startProfileAt([19.04, 20.22], %)
|> line([9.44, -30.16], %)
|> line([8.39, -3.73], %)
|> close(%)
"#;
let result = execute_and_snapshot(code).await.unwrap();
twenty_twenty::assert_image("tests/executor/outputs/lots_of_planes.png", &result, 1.0);
}

Binary file not shown.

Before

Width:  |  Height:  |  Size: 78 KiB

After

Width:  |  Height:  |  Size: 77 KiB

Binary file not shown.

Before

Width:  |  Height:  |  Size: 96 KiB

After

Width:  |  Height:  |  Size: 94 KiB

Binary file not shown.

After

Width:  |  Height:  |  Size: 88 KiB

Binary file not shown.

Before

Width:  |  Height:  |  Size: 70 KiB

After

Width:  |  Height:  |  Size: 70 KiB

Binary file not shown.

Before

Width:  |  Height:  |  Size: 70 KiB

After

Width:  |  Height:  |  Size: 70 KiB

Binary file not shown.

After

Width:  |  Height:  |  Size: 84 KiB

Binary file not shown.

Before

Width:  |  Height:  |  Size: 69 KiB

After

Width:  |  Height:  |  Size: 69 KiB

Binary file not shown.

Before

Width:  |  Height:  |  Size: 78 KiB

After

Width:  |  Height:  |  Size: 78 KiB

Binary file not shown.

Before

Width:  |  Height:  |  Size: 71 KiB

After

Width:  |  Height:  |  Size: 71 KiB

Binary file not shown.

Before

Width:  |  Height:  |  Size: 70 KiB

After

Width:  |  Height:  |  Size: 70 KiB

Binary file not shown.

Before

Width:  |  Height:  |  Size: 70 KiB

After

Width:  |  Height:  |  Size: 70 KiB

Binary file not shown.

Before

Width:  |  Height:  |  Size: 70 KiB

After

Width:  |  Height:  |  Size: 70 KiB

View File

@ -1,14 +1,14 @@
use anyhow::Result; use anyhow::Result;
use kcl_lib::{ use kcl_lib::{
ast::{modify::modify_ast_for_sketch, types::Program}, ast::{modify::modify_ast_for_sketch, types::Program},
engine::{EngineConnection, EngineManager}, engine::EngineManager,
executor::{MemoryItem, SourceRange}, executor::{ExecutorContext, MemoryItem, PlaneType, SourceRange},
}; };
use kittycad::types::{ModelingCmd, Point3D}; use kittycad::types::{ModelingCmd, Point3D};
use pretty_assertions::assert_eq; use pretty_assertions::assert_eq;
/// Setup the engine and parse code for an ast. /// Setup the engine and parse code for an ast.
async fn setup(code: &str, name: &str) -> Result<(EngineConnection, Program, uuid::Uuid)> { async fn setup(code: &str, name: &str) -> Result<(ExecutorContext, Program, uuid::Uuid)> {
let user_agent = concat!(env!("CARGO_PKG_NAME"), ".rs/", env!("CARGO_PKG_VERSION"),); let user_agent = concat!(env!("CARGO_PKG_NAME"), ".rs/", env!("CARGO_PKG_VERSION"),);
let http_client = reqwest::Client::builder() let http_client = reqwest::Client::builder()
.user_agent(user_agent) .user_agent(user_agent)
@ -38,8 +38,9 @@ async fn setup(code: &str, name: &str) -> Result<(EngineConnection, Program, uui
let program = parser.ast()?; let program = parser.ast()?;
let mut mem: kcl_lib::executor::ProgramMemory = Default::default(); let mut mem: kcl_lib::executor::ProgramMemory = Default::default();
let engine = kcl_lib::engine::EngineConnection::new(ws).await?; let engine = kcl_lib::engine::EngineConnection::new(ws).await?;
let memory = let planes = kcl_lib::executor::DefaultPlanes::new(&engine).await?;
kcl_lib::executor::execute(program.clone(), &mut mem, kcl_lib::executor::BodyType::Root, &engine).await?; let ctx = ExecutorContext { engine, planes };
let memory = kcl_lib::executor::execute(program.clone(), &mut mem, kcl_lib::executor::BodyType::Root, &ctx).await?;
// We need to get the sketch ID. // We need to get the sketch ID.
// Get the sketch group ID from memory. // Get the sketch group ID from memory.
@ -49,7 +50,7 @@ async fn setup(code: &str, name: &str) -> Result<(EngineConnection, Program, uui
let sketch_id = sketch_group.id; let sketch_id = sketch_group.id;
let plane_id = uuid::Uuid::new_v4(); let plane_id = uuid::Uuid::new_v4();
engine ctx.engine
.send_modeling_cmd( .send_modeling_cmd(
plane_id, plane_id,
SourceRange::default(), SourceRange::default(),
@ -59,6 +60,7 @@ async fn setup(code: &str, name: &str) -> Result<(EngineConnection, Program, uui
size: 60.0, size: 60.0,
x_axis: Point3D { x: 1.0, y: 0.0, z: 0.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 }, y_axis: Point3D { x: 0.0, y: 1.0, z: 0.0 },
hide: Some(true),
}, },
) )
.await?; .await?;
@ -66,7 +68,7 @@ async fn setup(code: &str, name: &str) -> Result<(EngineConnection, Program, uui
// Enter sketch mode. // Enter sketch mode.
// We can't get control points without being in sketch mode. // We can't get control points without being in sketch mode.
// You can however get path info without sketch mode. // You can however get path info without sketch mode.
engine ctx.engine
.send_modeling_cmd( .send_modeling_cmd(
uuid::Uuid::new_v4(), uuid::Uuid::new_v4(),
SourceRange::default(), SourceRange::default(),
@ -74,13 +76,14 @@ async fn setup(code: &str, name: &str) -> Result<(EngineConnection, Program, uui
animated: false, animated: false,
ortho: true, ortho: true,
plane_id, plane_id,
disable_camera_with_plane: Some(Point3D { x: 0.0, y: 0.0, z: 1.0 }),
}, },
) )
.await?; .await?;
// Enter edit mode. // Enter edit mode.
// We can't get control points of an existing sketch without being in edit mode. // We can't get control points of an existing sketch without being in edit mode.
engine ctx.engine
.send_modeling_cmd( .send_modeling_cmd(
uuid::Uuid::new_v4(), uuid::Uuid::new_v4(),
SourceRange::default(), SourceRange::default(),
@ -88,14 +91,15 @@ async fn setup(code: &str, name: &str) -> Result<(EngineConnection, Program, uui
) )
.await?; .await?;
Ok((engine, program, sketch_id)) Ok((ctx, program, sketch_id))
} }
#[tokio::test(flavor = "multi_thread")] #[tokio::test(flavor = "multi_thread")]
async fn serial_test_modify_sketch_part001() { async fn serial_test_modify_sketch_part001() {
let name = "part001"; let name = "part001";
let code = format!( let code = format!(
r#"const {} = startSketchAt([8.41, 5.78]) r#"const {} = startSketchOn("XY")
|> startProfileAt([8.41, 5.78], %)
|> line([7.37, -11.0], %) |> line([7.37, -11.0], %)
|> line([-8.69, -3.75], %) |> line([-8.69, -3.75], %)
|> line([-5.0, 4.25], %) |> line([-5.0, 4.25], %)
@ -103,9 +107,9 @@ async fn serial_test_modify_sketch_part001() {
name name
); );
let (mut engine, program, sketch_id) = setup(&code, name).await.unwrap(); let (mut ctx, program, sketch_id) = setup(&code, name).await.unwrap();
let mut new_program = program.clone(); let mut new_program = program.clone();
let new_code = modify_ast_for_sketch(&mut engine, &mut new_program, name, sketch_id) let new_code = modify_ast_for_sketch(&mut ctx.engine, &mut new_program, name, PlaneType::XY, sketch_id)
.await .await
.unwrap(); .unwrap();
@ -119,7 +123,8 @@ async fn serial_test_modify_sketch_part001() {
async fn serial_test_modify_sketch_part002() { async fn serial_test_modify_sketch_part002() {
let name = "part002"; let name = "part002";
let code = format!( let code = format!(
r#"const {} = startSketchAt([8.41, 5.78]) r#"const {} = startSketchOn("XY")
|> startProfileAt([8.41, 5.78], %)
|> line([7.42, -8.62], %) |> line([7.42, -8.62], %)
|> line([-6.38, -3.51], %) |> line([-6.38, -3.51], %)
|> line([-3.77, 3.56], %) |> line([-3.77, 3.56], %)
@ -127,9 +132,9 @@ async fn serial_test_modify_sketch_part002() {
name name
); );
let (mut engine, program, sketch_id) = setup(&code, name).await.unwrap(); let (mut ctx, program, sketch_id) = setup(&code, name).await.unwrap();
let mut new_program = program.clone(); let mut new_program = program.clone();
let new_code = modify_ast_for_sketch(&mut engine, &mut new_program, name, sketch_id) let new_code = modify_ast_for_sketch(&mut ctx.engine, &mut new_program, name, PlaneType::XY, sketch_id)
.await .await
.unwrap(); .unwrap();
@ -144,7 +149,8 @@ async fn serial_test_modify_sketch_part002() {
async fn serial_test_modify_close_sketch() { async fn serial_test_modify_close_sketch() {
let name = "part002"; let name = "part002";
let code = format!( let code = format!(
r#"const {} = startSketchAt([7.91, 3.89]) r#"const {} = startSketchOn("XY")
|> startProfileAt([7.91, 3.89], %)
|> line([7.42, -8.62], %) |> line([7.42, -8.62], %)
|> line([-6.38, -3.51], %) |> line([-6.38, -3.51], %)
|> line([-3.77, 3.56], %) |> line([-3.77, 3.56], %)
@ -153,9 +159,9 @@ async fn serial_test_modify_close_sketch() {
name name
); );
let (mut engine, program, sketch_id) = setup(&code, name).await.unwrap(); let (mut ctx, program, sketch_id) = setup(&code, name).await.unwrap();
let mut new_program = program.clone(); let mut new_program = program.clone();
let new_code = modify_ast_for_sketch(&mut engine, &mut new_program, name, sketch_id) let new_code = modify_ast_for_sketch(&mut ctx.engine, &mut new_program, name, PlaneType::XY, sketch_id)
.await .await
.unwrap(); .unwrap();
@ -169,7 +175,8 @@ async fn serial_test_modify_close_sketch() {
async fn serial_test_modify_line_to_close_sketch() { async fn serial_test_modify_line_to_close_sketch() {
let name = "part002"; let name = "part002";
let code = format!( let code = format!(
r#"const {} = startSketchAt([7.91, 3.89]) r#"const {} = startSketchOn("XY")
|> startProfileAt([7.91, 3.89], %)
|> line([7.42, -8.62], %) |> line([7.42, -8.62], %)
|> line([-6.38, -3.51], %) |> line([-6.38, -3.51], %)
|> line([-3.77, 3.56], %) |> line([-3.77, 3.56], %)
@ -178,9 +185,9 @@ async fn serial_test_modify_line_to_close_sketch() {
name name
); );
let (mut engine, program, sketch_id) = setup(&code, name).await.unwrap(); let (mut ctx, program, sketch_id) = setup(&code, name).await.unwrap();
let mut new_program = program.clone(); let mut new_program = program.clone();
let new_code = modify_ast_for_sketch(&mut engine, &mut new_program, name, sketch_id) let new_code = modify_ast_for_sketch(&mut ctx.engine, &mut new_program, name, PlaneType::XY, sketch_id)
.await .await
.unwrap(); .unwrap();
@ -188,7 +195,8 @@ async fn serial_test_modify_line_to_close_sketch() {
assert_eq!( assert_eq!(
new_code, new_code,
format!( format!(
r#"const {} = startSketchAt([7.91, 3.89]) r#"const {} = startSketchOn("XY")
|> startProfileAt([7.91, 3.89], %)
|> line([7.42, -8.62], %) |> line([7.42, -8.62], %)
|> line([-6.38, -3.51], %) |> line([-6.38, -3.51], %)
|> line([-3.77, 3.56], %) |> line([-3.77, 3.56], %)
@ -204,7 +212,8 @@ async fn serial_test_modify_with_constraint() {
let name = "part002"; let name = "part002";
let code = format!( let code = format!(
r#"const thing = 12 r#"const thing = 12
const {} = startSketchAt([7.91, 3.89]) const {} = startSketchOn("XY")
|> startProfileAt([7.91, 3.89], %)
|> line([7.42, -8.62], %) |> line([7.42, -8.62], %)
|> line([-6.38, -3.51], %) |> line([-6.38, -3.51], %)
|> line([-3.77, 3.56], %) |> line([-3.77, 3.56], %)
@ -213,14 +222,14 @@ const {} = startSketchAt([7.91, 3.89])
name name
); );
let (mut engine, program, sketch_id) = setup(&code, name).await.unwrap(); let (mut ctx, program, sketch_id) = setup(&code, name).await.unwrap();
let mut new_program = program.clone(); let mut new_program = program.clone();
let result = modify_ast_for_sketch(&mut engine, &mut new_program, name, sketch_id).await; let result = modify_ast_for_sketch(&mut ctx.engine, &mut new_program, name, PlaneType::XY, sketch_id).await;
assert!(result.is_err()); assert!(result.is_err());
assert_eq!( assert_eq!(
result.unwrap_err().to_string(), result.unwrap_err().to_string(),
r#"engine: KclErrorDetails { source_ranges: [SourceRange([159, 164])], message: "Sketch part002 is constrained `partial` and cannot be modified" }"# r#"engine: KclErrorDetails { source_ranges: [SourceRange([188, 193])], message: "Sketch part002 is constrained `partial` and cannot be modified" }"#
); );
} }
@ -228,7 +237,8 @@ const {} = startSketchAt([7.91, 3.89])
async fn serial_test_modify_line_should_close_sketch() { async fn serial_test_modify_line_should_close_sketch() {
let name = "part003"; let name = "part003";
let code = format!( let code = format!(
r#"const {} = startSketchAt([13.69, 3.8]) r#"const {} = startSketchOn("XY")
|> startProfileAt([13.69, 3.8], %)
|> line([4.23, -11.79], %) |> line([4.23, -11.79], %)
|> line([-10.7, -1.16], %) |> line([-10.7, -1.16], %)
|> line([-3.72, 8.69], %) |> line([-3.72, 8.69], %)
@ -237,9 +247,9 @@ async fn serial_test_modify_line_should_close_sketch() {
name name
); );
let (mut engine, program, sketch_id) = setup(&code, name).await.unwrap(); let (mut ctx, program, sketch_id) = setup(&code, name).await.unwrap();
let mut new_program = program.clone(); let mut new_program = program.clone();
let new_code = modify_ast_for_sketch(&mut engine, &mut new_program, name, sketch_id) let new_code = modify_ast_for_sketch(&mut ctx.engine, &mut new_program, name, PlaneType::XY, sketch_id)
.await .await
.unwrap(); .unwrap();
@ -247,7 +257,8 @@ async fn serial_test_modify_line_should_close_sketch() {
assert_eq!( assert_eq!(
new_code, new_code,
format!( format!(
r#"const {} = startSketchAt([13.69, 3.8]) r#"const {} = startSketchOn("XY")
|> startProfileAt([13.69, 3.8], %)
|> line([4.23, -11.79], %) |> line([4.23, -11.79], %)
|> line([-10.7, -1.16], %) |> line([-10.7, -1.16], %)
|> line([-3.72, 8.69], %) |> line([-3.72, 8.69], %)

View File

@ -1,6 +1,9 @@
{ {
"compilerOptions": { "compilerOptions": {
"baseUrl": "src", "baseUrl": "src",
"paths": {
"/*": ["src/*"]
},
"types": ["vite/client", "@types/wicg-file-system-access"], "types": ["vite/client", "@types/wicg-file-system-access"],
"target": "esnext", "target": "esnext",
"lib": [ "lib": [

View File

@ -1530,10 +1530,10 @@
resolved "https://registry.yarnpkg.com/@juggle/resize-observer/-/resize-observer-3.4.0.tgz#08d6c5e20cf7e4cc02fd181c4b0c225cd31dbb60" resolved "https://registry.yarnpkg.com/@juggle/resize-observer/-/resize-observer-3.4.0.tgz#08d6c5e20cf7e4cc02fd181c4b0c225cd31dbb60"
integrity sha512-dfLbk+PwWvFzSxwk3n5ySL0hfBog779o8h68wK/7/APo/7cgyWp5jcXockbxdk5kFRkbeXWm4Fbi9FrdN381sA== integrity sha512-dfLbk+PwWvFzSxwk3n5ySL0hfBog779o8h68wK/7/APo/7cgyWp5jcXockbxdk5kFRkbeXWm4Fbi9FrdN381sA==
"@kittycad/lib@^0.0.40": "@kittycad/lib@^0.0.43":
version "0.0.40" version "0.0.43"
resolved "https://registry.yarnpkg.com/@kittycad/lib/-/lib-0.0.40.tgz#0ba00c642e76648fb7cb1337e799b9d24724312d" resolved "https://registry.yarnpkg.com/@kittycad/lib/-/lib-0.0.43.tgz#b93c0961200cf327f5ac6491be1ca01f9edc321c"
integrity sha512-R8sQKLWe3lQC7l7cyY49oFgeiMvRh8+bCaaoLiIVYT+YiE9TaS+uwwF1+sR7MiX6YZp/YCBBPFGj4Ci0VHC9Bg== integrity sha512-Pe/PQfZ8BWEDOm4dkY4tcPcuCSh2mQPe/W8RvJ6PQNTlB7bmjAj0234pVyG+8zrKKsJC9nq4ye7CZoaXEfUSTg==
dependencies: dependencies:
node-fetch "3.3.2" node-fetch "3.3.2"
openapi-types "^12.0.0" openapi-types "^12.0.0"
@ -4814,10 +4814,10 @@ queue-microtask@^1.2.2:
resolved "https://registry.yarnpkg.com/queue-microtask/-/queue-microtask-1.2.3.tgz#4929228bbc724dfac43e0efb058caf7b6cfb6243" resolved "https://registry.yarnpkg.com/queue-microtask/-/queue-microtask-1.2.3.tgz#4929228bbc724dfac43e0efb058caf7b6cfb6243"
integrity sha512-NuaNSa6flKT5JaSYQzJok04JzTL1CA6aGhv5rfLW3PgqA+M2ChpZQnAC8h8i4ZFkBS8X5RqkDBHA7r4hej3K9A== integrity sha512-NuaNSa6flKT5JaSYQzJok04JzTL1CA6aGhv5rfLW3PgqA+M2ChpZQnAC8h8i4ZFkBS8X5RqkDBHA7r4hej3K9A==
re-resizable@^6.9.9: re-resizable@^6.9.11:
version "6.9.9" version "6.9.11"
resolved "https://registry.yarnpkg.com/re-resizable/-/re-resizable-6.9.9.tgz#99e8b31c67a62115dc9c5394b7e55892265be216" resolved "https://registry.yarnpkg.com/re-resizable/-/re-resizable-6.9.11.tgz#f356e27877f12d926d076ab9ad9ff0b95912b475"
integrity sha512-l+MBlKZffv/SicxDySKEEh42hR6m5bAHfNu3Tvxks2c4Ah+ldnWjfnVRwxo/nxF27SsUsxDS0raAzFuJNKABXA== integrity sha512-a3hiLWck/NkmyLvGWUuvkAmN1VhwAz4yOhS6FdMTaxCUVN9joIWkT11wsO68coG/iEYuwn+p/7qAmfQzRhiPLQ==
react-base16-styling@^0.6.0: react-base16-styling@^0.6.0:
version "0.6.0" version "0.6.0"