@jbroll/jscad-modeling 2.12.7

package/.claude/settings.local.json

@@ -0,0 +1,30 @@
+{
+  "permissions": {
+    "allow": [
+      "Bash(wc:*)",
+      "Bash(git add:*)",
+      "Bash(git commit -m \"$\\(cat <<''EOF''\ndocs\\(modeling\\): add CLAUDE.md for AI assistant guidance\n\nCo-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>\nEOF\n\\)\")",
+      "Bash(gh auth:*)",
+      "Bash(gh issue create --title \"perf\\(modeling\\): O\\(n²\\) array concatenation in extrudeFromSlices.js\" --body \"$\\(cat <<''EOF''\n## Summary\n`extrudeFromSlices.js` uses `Array.concat\\(\\)` inside a loop, causing O\\(n²\\) memory operations.\n\n## Usage Frequency: VERY HIGH\nThis function is the core of all extrusion operations:\n- `extrudeLinearGeom2.js`\n- `extrudeRotate.js`  \n- `extrudeHelical.js`\n\nEvery extrusion operation goes through this code path.\n\n## Problem\n**File:** `packages/modeling/src/operations/extrusions/extrudeFromSlices.js`\n**Lines:** 84, 98, 102-103, 108\n\n```javascript\nfor \\(let s = 0; s < numberOfSlices; s++\\) {\n  // ...\n  if \\(prevSlice\\) {\n    polygons = polygons.concat\\(extrudeWalls\\(prevSlice, currentSlice\\)\\)  // Creates new array each iteration\n  }\n}\n// ...\npolygons = polygons.concat\\(endPolygons\\)\npolygons = polygons.concat\\(startPolygons\\)\n```\n\nEach `concat\\(\\)` creates a new array and copies all existing elements. For N slices, this performs O\\(n²\\) copy operations.\n\n## Suggested Fix\nUse `push\\(\\)` with spread operator:\n\n```javascript\nfor \\(let s = 0; s < numberOfSlices; s++\\) {\n  if \\(prevSlice\\) {\n    polygons.push\\(...extrudeWalls\\(prevSlice, currentSlice\\)\\)\n  }\n}\npolygons.push\\(...endPolygons\\)\npolygons.push\\(...startPolygons\\)\n```\n\n## Impact\nHigh - affects performance of all extrusion operations, especially with many slices.\nEOF\n\\)\")",
+      "Bash(gh issue create --repo jscad/OpenJSCAD.org --title \"perf\\(modeling\\): O\\(n²\\) array concatenation in extrudeFromSlices.js\" --body \"$\\(cat <<''EOF''\n## Summary\n`extrudeFromSlices.js` uses `Array.concat\\(\\)` inside a loop, causing O\\(n²\\) memory operations.\n\n## Usage Frequency: VERY HIGH\nThis function is the core of all extrusion operations:\n- `extrudeLinearGeom2.js`\n- `extrudeRotate.js`  \n- `extrudeHelical.js`\n\nEvery extrusion operation goes through this code path.\n\n## Problem\n**File:** `packages/modeling/src/operations/extrusions/extrudeFromSlices.js`  \n**Lines:** 84, 98, 102-103, 108\n\n```javascript\nfor \\(let s = 0; s < numberOfSlices; s++\\) {\n  // ...\n  if \\(prevSlice\\) {\n    polygons = polygons.concat\\(extrudeWalls\\(prevSlice, currentSlice\\)\\)  // Creates new array each iteration\n  }\n}\n// ...\npolygons = polygons.concat\\(endPolygons\\)\npolygons = polygons.concat\\(startPolygons\\)\n```\n\nEach `concat\\(\\)` creates a new array and copies all existing elements. For N slices, this performs O\\(n²\\) copy operations.\n\n## Suggested Fix\nUse `push\\(\\)` with spread operator:\n\n```javascript\nfor \\(let s = 0; s < numberOfSlices; s++\\) {\n  if \\(prevSlice\\) {\n    polygons.push\\(...extrudeWalls\\(prevSlice, currentSlice\\)\\)\n  }\n}\npolygons.push\\(...endPolygons\\)\npolygons.push\\(...startPolygons\\)\n```\n\n## Impact\nHigh - affects performance of all extrusion operations, especially with many slices.\nEOF\n\\)\")",
+      "Bash(gh issue create --repo jscad/OpenJSCAD.org --title \"perf\\(modeling\\): O\\(n²\\) array concatenation in scissionGeom3.js\" --body \"$\\(cat <<''EOF''\n## Summary\n`scissionGeom3.js` uses `Array.concat\\(\\)` inside nested loops, causing O\\(n²\\) memory operations.\n\n## Usage Frequency: LOW\nUsed only by `scission\\(\\)` operation which splits a geometry into disconnected pieces. Most typical models don''t use scission - it''s a specialized operation for:\n- Separating imported models that contain multiple parts\n- Post-processing boolean results that created separate islands\n\n**Typical model usage:** Rare - maybe 1% of models\n\n## Problem\n**File:** `packages/modeling/src/operations/booleans/scissionGeom3.js`  \n**Lines:** 40-42\n\n```javascript\nconst indexesPerPolygon = polygons.map\\(\\(polygon\\) => {\n  let indexes = []\n  polygon.vertices.forEach\\(\\(point\\) => {\n    indexes = indexes.concat\\(findMapping\\(indexesPerPoint, vec3.snap\\(temp, point, eps\\)\\)\\)\n  }\\)\n  return { e: 1, d: sortNb\\(indexes\\) }\n}\\)\n```\n\nFor each vertex of each polygon, `concat\\(\\)` creates a new array. With N polygons averaging V vertices, this is O\\(N*V\\) array allocations.\n\n## Suggested Fix\nUse `push\\(\\)` with spread:\n\n```javascript\nconst indexesPerPolygon = polygons.map\\(\\(polygon\\) => {\n  const indexes = []\n  polygon.vertices.forEach\\(\\(point\\) => {\n    indexes.push\\(...findMapping\\(indexesPerPoint, vec3.snap\\(temp, point, eps\\)\\)\\)\n  }\\)\n  return { e: 1, d: sortNb\\(indexes\\) }\n}\\)\n```\n\n## Impact\nLow overall \\(rare operation\\), but when used on complex geometry it can be slow.\nEOF\n\\)\")",
+      "Bash(gh issue create:*)",
+      "Bash(npm run bench:*)",
+      "Bash(gh issue view:*)",
+      "Bash(git commit -m \"$\\(cat <<''EOF''\nperf\\(modeling\\): add benchmark suite for performance testing\n\nAdds benchmarks for:\n- Booleans \\(union, subtract, intersect at varying complexity\\)\n- Extrusions \\(extrudeLinear, extrudeRotate with varying slices\\)\n- Measurements \\(boundingBox, volume, area\\)\n- Transforms \\(translate, rotate, scale, center\\)\n- Utils \\(flatten with varying array sizes\\)\n\nRun with: npm run bench\n\nCo-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>\nEOF\n\\)\")",
+      "Bash(gh issue comment:*)",
+      "Bash(npm test:*)",
+      "Bash(npx ava:*)",
+      "Bash(node -e:*)",
+      "Bash(git commit:*)",
+      "Bash(ls:*)",
+      "Bash(grep:*)",
+      "Bash(npm install)",
+      "Bash(git checkout:*)",
+      "Bash(git push:*)",
+      "Bash(gh pr create:*)",
+      "Bash(cat:*)",
+      "Bash(npm whoami:*)"
+    ]
+  }
+}