Clean up for review

KittyCAD · Jan 10, 2025 · 0bfb6bc · 0bfb6bc
1 parent 44490da
commit 0bfb6bc
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 7 deletions.
diff --git a/src/lang/std/engineConnection.ts b/src/lang/std/engineConnection.ts
@@ -2098,7 +2098,6 @@ export class EngineCommandManager extends EventTarget {
       ast,
       execStateArtifacts,
     })
-    console.log(this.artifactGraph)
     // TODO check if these still need to be deferred once e2e tests are working again.
     if (this.artifactGraph.size) {
       this.deferredArtifactEmptied(null)

diff --git a/src/lib/commandBarConfigs/modelingCommandConfig.ts b/src/lib/commandBarConfigs/modelingCommandConfig.ts
@@ -308,6 +308,7 @@ export const modelingMachineCommandConfig: StateMachineCommandSetConfig<
         selectionTypes: ['solid2D'],
         required: true,
         skip: true,
+        multiple: false,
         // TODO: add dry-run validation
         warningMessage:
           'The sweep workflow is new and under tested. Please break it and report issues.',
@@ -317,6 +318,7 @@ export const modelingMachineCommandConfig: StateMachineCommandSetConfig<
         selectionTypes: ['segment', 'path'],
         required: true,
         skip: true,
+        multiple: false,
         // TODO: add dry-run validation
       },
     },

diff --git a/src/lib/selections.ts b/src/lib/selections.ts
@@ -208,7 +208,6 @@ export async function getEventForSelectWithPoint({
   }
 
   let _artifact = engineCommandManager.artifactGraph.get(data.entity_id)
-  console.log('_artifact', _artifact)
   const codeRefs = getCodeRefsByArtifactId(
     data.entity_id,
     engineCommandManager.artifactGraph

diff --git a/src/machines/modelingMachine.ts b/src/machines/modelingMachine.ts
@@ -1572,11 +1572,6 @@ export const modelingMachine = setup({
         }
         const profileDeclarator = profileNode.node
 
-        // TODO: add better validation on selection
-        if (!profileDeclarator) {
-          trap('Not enough sketches selected')
-        }
-
         // Find the path declaration
         const pathNodePath = getNodePathFromSourceRange(
           ast,