@jscad/core 2.5.3 → 2.5.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jscad/core",
-  "version": "2.5.3",
+  "version": "2.5.6",
   "description": "Core functionality for JSCAD Applications",
   "repository": "https://github.com/jscad/OpenJSCAD.org",
   "main": "src/index.js",
@@ -34,10 +34,10 @@
   ],
   "license": "MIT",
   "dependencies": {
-    "@jscad/array-utils": "2.1.1",
-    "@jscad/io": "2.2.3",
-    "@jscad/io-utils": "2.0.12",
-    "@jscad/modeling": "2.7.1",
+    "@jscad/array-utils": "2.1.2",
+    "@jscad/io": "2.2.6",
+    "@jscad/io-utils": "2.0.15",
+    "@jscad/modeling": "2.9.0",
     "json5": "2.2.0",
     "strip-bom": "4.0.0"
   },
@@ -52,5 +52,5 @@
     "url": "https://opencollective.com/openjscad",
     "logo": "https://opencollective.com/openjscad/logo.txt"
   },
-  "gitHead": "95f05c5f11a53bff469a305d69b99f674dc7d6d3"
+  "gitHead": "45fbcb7298fcacf3e45d67fdca32d0744ab7f2de"
 }

package/src/code-evaluation/rebuildGeometry.js

@@ -24,7 +24,6 @@ const applyParameterDefinitions = require('../parameters/applyParameterDefinitio
  * And transfering data back & forth is both complex (see transferables) and costly (time)
  **/
 const rebuildSolids = (data, callback) => {
-  console.log('rebuildSolids',data)
   const defaults = {
     mainPath: '',
     apiMainPath: '@jscad/modeling',
@@ -47,7 +46,6 @@ const rebuildSolids = (data, callback) => {
     parameterDefaults: designData.parameterValues,
     parameterDefinitions: designData.parameterDefinitions
   })
-  // console.warn(`loadDesignData`, new Date() - start)
   // make sure parameters are correct by applying parameter definitions
   // this might be redundant with ui-side logic, but it makes sure this core piece works regardless of ui
   parameterValues = applyParameterDefinitions(parameterValues, designData.parameterDefinitions)
@@ -59,7 +57,6 @@ const rebuildSolids = (data, callback) => {
     serialize
   }
   const solidsData = instanciateDesign(designData.rootModule, parameterValues, options)
-  // console.warn(`instanciateDesign`, new Date() - start)
 
   // send back solids & any other metadata
   callback(null, {

package/src/code-evaluation/rebuildGeometryCli.js

@@ -7,10 +7,9 @@ const makeWebRequire = require('../code-loading/webRequire')
 
 const rebuildSolids = (data) => {
   const defaults = {
-    apiMainPath: '@jscad/modeling',
-    serialize: false
+    apiMainPath: '@jscad/modeling'
   }
-  let { apiMainPath, serialize, mainPath, parameterValues, useFakeFs } = Object.assign({}, defaults, data)
+  let { apiMainPath, mainPath, parameterValues, useFakeFs } = Object.assign({}, defaults, data)
   // we need to update the source for our module
   let requireFn = require
 

package/src/code-evaluation/serializeSolids.js

@@ -11,7 +11,7 @@ const serializeSolids = (solids) => {
   // imcomplete support for transfering objects via web workers
 
   // NOTE: JSON.stringify was used at some point, but was removed because it was no longer needed
-  // for postMessage JavaScript engines now use an optimized structured clone alg. 
+  // for postMessage JavaScript engines now use an optimized structured clone alg.
   // which should be at least as fast as JSON.stringify
   solids = solids.map((object) => {
     // apply the transforms before serializing
@@ -23,5 +23,4 @@ const serializeSolids = (solids) => {
   return solids
 }
 
-
 module.exports = serializeSolids

package/src/code-loading/loadDesign.js

@@ -1,4 +1,3 @@
-// loading
 const { registerAllExtensions } = require('../io/registerExtensions')
 
 const transformSources = require('./transformSources')
@@ -16,27 +15,6 @@ const getAllParameterDefintionsAndValues = require('../parameters/getParameterDe
  * @param {Object} parameterValuesOverride - the values to use to override the defaults for the current design
  */
 const loadDesign = (mainPath, apiMainPath, filesAndFolders, parameterValuesOverride) => {
-  // console.log('***** loadDesign',mainPath)
-  // the root script is the main entry point in a design
-  // ie either the only file if there is only one
-  // OR the file in the 'main' entry of package.js, index.js, main.js or <folderName>.js
-
-  /*
-    - if the script is a common.js file already
-      > load as it is
-        - if we have real require() access (CLI, desktop)
-          use standard require() to load the rootScript
-        - if we do NOT have real require() access (web)
-          use fake require() to load the rootScript
-    - if the script is NOT a common.js file (implicit imports)
-      > add explicit api imports to the rootScript's source
-      > add explicit exports ie module.exports {main, getParameterDefinitions}
-        - if we have real require() access (CLI, desktop)
-          use standard require() to load the rootScript
-        - if we do NOT have real require() access (web)
-          use fake require() to load the rootScript
-  */
-
   // transform the source if passed non-javascript content, i.e. stl
   filesAndFolders = transformSources({ apiMainPath }, filesAndFolders)
 
@@ -52,8 +30,6 @@ const loadDesign = (mainPath, apiMainPath, filesAndFolders, parameterValuesOverr
       }
     ]
   }
-  // console.log('filesAndFolders',filesAndFolders)
-
   const fakeFs = makeFakeFs(filesAndFolders)
 
   const webRequire = makeWebRequire(filesAndFolders, { apiMainPath })
@@ -64,8 +40,6 @@ const loadDesign = (mainPath, apiMainPath, filesAndFolders, parameterValuesOverr
   // find the root module
   let rootModule = webRequire(filesAndFolders[0].fullPath)
 
-  // console.log('***** rootModule',rootModule)
-
   rootModule = normalizeDesignModule(rootModule)
 
   // rootModule SHOULD contain a main() entry and optionally a getParameterDefinitions entry

package/src/code-loading/makeFakeFs.js

@@ -13,7 +13,6 @@ const makeFakeFs = (filesAndFolders) => {
       }
     }
     return undefined
-    // return filesAndFolders
   }
 
   const statSync = (path) => {
@@ -27,16 +26,13 @@ const makeFakeFs = (filesAndFolders) => {
     statSync,
     existsSync: (path) => {
       const entry = findMatch(path)
-      // console.log('does ', path, 'exist ?', entry !== undefined)
       return entry !== undefined
     },
     readdirSync: (path) => {
       const entry = findMatch(path)
       return entry.children.map((x) => x.name)
-      // filesAndFolders
     },
     readDir: (path, callback) => {
-      // console.log('readDir', path, callback)
       const entry = findMatch(path)
       callback(null, entry)
     },
@@ -48,7 +44,6 @@ const makeFakeFs = (filesAndFolders) => {
       if (!statSync(path).isFile()) {
         callback(new Error(`${entry} is not a file, cannot read`))
       } else {
-        // console.log('readFile', path, entry)
         callback(null, entry.source)
       }
     },
@@ -60,7 +55,6 @@ const makeFakeFs = (filesAndFolders) => {
       if (!statSync(path).isFile()) {
         throw new Error(`${entry} is not a file, cannot read`)
       } else {
-        // console.log('readFile sync', path, entry)
         return entry.source
       }
     }

package/src/code-loading/modulifySource.js

@@ -4,7 +4,8 @@
  */
 const modulifySource = (source, apiMainPath) => {
   const getParamsString = source.includes('getParameterDefinitions')
-    ? 'module.exports.getParameterDefinitions = getParameterDefinitions' : ''
+    ? 'module.exports.getParameterDefinitions = getParameterDefinitions'
+    : ''
   const updatedSource = `
     const {geom2, geom3, path2, pol2, poly3} = require('${apiMainPath}').geometries
 

package/src/code-loading/transformSources.js

@@ -3,9 +3,7 @@ const { deserializers } = require('@jscad/io')
 /*
  * Transform the entry into a ready-to-use module.
  */
-const modulifyTransform = (options, entry) => {
-  return Object.assign({}, entry, { source: entry.source })
-}
+const modulifyTransform = (options, entry) => Object.assign({}, entry, { source: entry.source })
 
 /*
  * Create a new entry for a script (JSCAD) from the given entry and source
@@ -24,8 +22,6 @@ const createJscadEntry = (entry, source) => {
  * Transforms are NOT applied to projects.
  */
 const transformSources = (options, filesAndFolders) => {
-  // console.log('***** transformSources', options, filesAndFolders)
-
   if (filesAndFolders && filesAndFolders.length > 1) return filesAndFolders // skip projects
 
   const codeTransforms = {

package/src/code-loading/webRequire.js

@@ -106,12 +106,12 @@ const makeWebRequire = (filesAndFolders, options) => {
       if (entry.children) return null // directory
 
       if (extensions[baseExt]) {
-        if(moduleCache[requirePath]) return moduleCache[requirePath]
+        if (moduleCache[requirePath]) return moduleCache[requirePath]
         // evaluate the content
         const matchingModule = {
           exports: {},
           _compile: (content, fileName) => {
-            const moduleMakerFunction = new Function('require', 'module', content)
+            const moduleMakerFunction = new Function('require', 'module', content) // eslint-disable-line no-new-func
             moduleMakerFunction(_require.bind(null, entry.fullPath), matchingModule)
 
             const paramDefFromSource = content.includes('@jscad-params') ? getParameterDefinitionsFromSource(content, fileName) : []
@@ -123,7 +123,8 @@ const makeWebRequire = (filesAndFolders, options) => {
           }
         }
         extensions[baseExt](matchingModule, entry.fullPath)
-        return moduleCache[requirePath] = matchingModule.exports
+        moduleCache[requirePath] = matchingModule.exports
+        return moduleCache[requirePath]
       }
       return null
     }

package/src/parameters/getParameterDefinitionsFromSource.test.js

@@ -224,5 +224,5 @@ test('multiline error', (t) => {
 test('single line /* ...*/ ok', (t) => {
   t.deepEqual(getParameterDefinitionsFromSource(`function main({//@jscad-params
   /* group */
-  width=14})`), [{type:'group', caption:'group', name:'_group_1'},...sampleParams2])
+  width=14})`), [{ type: 'group', caption: 'group', name: '_group_1' }, ...sampleParams2])
 })

package/src/parameters/getParameterValuesFromUIControls.js

@@ -13,14 +13,6 @@ const getParameterValuesFromUIControls = (paramControls, parameterDefinitions, o
     switch (control.paramType) {
       case 'choice':
         value = control.options[control.selectedIndex].value
-        /* console.log('choice', control, control.paramName)
-        // we try to match values against captions, then parse as numbers if applicable, then fallback to original value
-        const valueIndex = !definition ? -1 : definition.captions.indexOf(value)
-        const valueInDefinition = valueIndex > -1
-        // const valueInDefintionCaptionsAndValue = valueInDefinition && definition.values.length >= valueIndex
-        value = definition.values.length > 0 && isNumber(definition.values[0]) ? parseFloat(value) : value
-        value = definition.values.length > 0 && typeof value === 'boolean' ? !!value : value
-        console.log('foo', value) */
         break
       case 'float':
       case 'number':
@@ -63,7 +55,6 @@ const getParameterValuesFromUIControls = (paramControls, parameterDefinitions, o
       }
     }
     parameterValues[control.paramName] = value
-  // console.log(control.paramName+":"+parameterValues[control.paramName])
   }
   return parameterValues
 }

package/src/web/walkFileTree.js

@@ -30,8 +30,6 @@ const binaryMimetypes = {
  * @returns {Promise} new promise to read and convert the file
  */
 const readFileAsync = (file, fileMeta) => {
-  // console.log('readFileAsync',file,fileMeta)
-
   const fullPath = file.fullPath ? file.fullPath : fileMeta.fullPath ? fileMeta.fullPath : ''
   const ext = getFileExtensionFromString(file.name)
   const mimetype = file.mimetype
@@ -43,7 +41,7 @@ const readFileAsync = (file, fileMeta) => {
       const result = event.target.result
       if (result.byteLength) {
         resolve({ name: file.name, ext, fullPath, mimetype, source: result })
-      } else if (typeof(result) === 'string') {
+      } else if (typeof result === 'string') {
         resolve({ name: file.name, ext, fullPath, mimetype, source: result })
       }
     }
@@ -87,7 +85,6 @@ const isEmpty = (x) => x !== null && x !== undefined // skip empty items
  * @returns {Promise} one promise to resolve them all
  */
 const processEntries = (items) => {
-  // console.log('processEntries',items)
   const results = pseudoArraytoArray(items.filter(isEmpty))
     .filter(isEmpty) // skip empty items
     .reduce((result, item) => {
@@ -117,7 +114,6 @@ const processEntries = (items) => {
  * @returns {Promise} new promise to read and process the file
  */
 const processFile = (fileItem) => {
-  // console.log('processFile',fileItem)
   const promiseFile = new Promise((resolve, reject) => {
     fileItem.file(
       (fileData) => {
@@ -138,7 +134,6 @@ const processFile = (fileItem) => {
  * @returns {Promise} new promise to read and process the directory
  */
 const processDirectory = (directory) => {
-  // console.log('processDirectory',directory)
   const promiseDirectory = new Promise((resolve, reject) => {
     if (directory.entries) {
       directory.entries.length ? processEntries(directory.entries).then(resolve) : resolve([])