Sectional default update in point-and-click sweep

2025-05-14 12:38:47 -04:00
3 changed files with 6 additions and 5 deletions
--- a/src/lang/modifyAst/addSweep.ts
+++ b/src/lang/modifyAst/addSweep.ts
@ -152,9 +152,10 @@ export function addSweep({

  // Extra labeled args expressions
  const pathExpr = createLocalName(pathDeclaration.node.declaration.id.name)
-  const sectionalExpr = sectional
-    ? [createLabeledArg('sectional', createLiteral(sectional))]
-    : []
+  const sectionalExpr =
+    sectional == false
+      ? [createLabeledArg('sectional', createLiteral(sectional))]
+      : []

  const sketchesExpr = createSketchExpression(sketchesExprList)
  const call = createCallExpressionStdLibKw('sweep', sketchesExpr, [
--- a/src/lib/commandBarConfigs/modelingCommandConfig.ts
+++ b/src/lib/commandBarConfigs/modelingCommandConfig.ts
@ -434,7 +434,7 @@ export const modelingMachineCommandConfig: StateMachineCommandSetConfig<
      sectional: {
        inputType: 'options',
        skip: true,
-        defaultValue: false,
+        defaultValue: true,
        hidden: false,
        required: true,
        options: [
--- a/src/lib/commandBarConfigs/validators.ts
+++ b/src/lib/commandBarConfigs/validators.ts
@ -290,7 +290,7 @@ export const sweepValidator = async ({
  const command = async () => {
    // TODO: second look on defaults here
    const DEFAULT_TOLERANCE: Models['LengthUnit_type'] = 1e-7
-    const DEFAULT_SECTIONAL = false
+    const DEFAULT_SECTIONAL = false // the more permissive for validation
    const cmdArgs = {
      target,
      trajectory,