@architect/inventory 3.2.1 → 3.3.0

package/changelog.md

@@ -2,6 +2,22 @@
 
 ---
 
+## [3.3.0] 2022-09-06
+
+### Changed
+
+- Node 14+ Lambda handler selection now defaults to ESM (unless otherwise specified)
+
+---
+
+## [3.2.2] 2022-08-20
+
+### Changed
+
+- Enabled `hydrate` plugin API support
+
+---
+
 ## [3.2.1] 2022-08-10
 
 ### Changed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@architect/inventory",
-  "version": "3.2.1",
+  "version": "3.3.0",
   "description": "Architect project resource enumeration utility",
   "main": "src/index.js",
   "scripts": {

package/src/config/pragmas/plugins.js

@@ -4,7 +4,7 @@ let { is, normalizeSrc, pragmas, tidyError, validationPatterns } = require('../.
 let { lambdas } = pragmas
 let nonLambdaSetters = [ 'customLambdas', 'env', 'proxy', 'runtimes', 'shared', 'static', 'views', 'tables', 'tables-indexes' ]
 let setters = [ ...lambdas, ...nonLambdaSetters ]
-let pluginMethods = [ 'deploy', 'sandbox' ]
+let pluginMethods = [ 'deploy', 'hydrate', 'sandbox' ]
 let reservedNames = [ '_methods' ]
 
 module.exports = function getPluginModules ({ arc, inventory, errors }) {

package/src/config/pragmas/populate-lambda/get-handler.js

@@ -32,6 +32,7 @@ module.exports = function getHandler ({ config, src, build, errors }) {
   return handlerConfig
 }
 
+// As of nodejs14.x, CJS remains the default over ESM when both are present
 let nodeHandlers = [ 'index.js', 'index.mjs', 'index.cjs' ]
 let denoHandlers = [ 'mod.ts', 'mod.js' ]
   // TODO: these are all prob going away
@@ -43,14 +44,22 @@ function getExt ({ runtime, src, errors }) {
       if (runtime === 'nodejs12.x') {
         return { ext: 'js', handlerModuleSystem: 'cjs' }
       }
-      // This presumes Node.js 14.x+ Lambda releases use the same CJS/ESM pattern
+      // This presumes Node.js ≥14 Lambda releases use the same CJS/ESM pattern
+      // Generally in Lambda: CJS wins, but in Architect-land we attempt to default to ESM
       else {
-        let { file, ext = 'js' } = findHandler(nodeHandlers, src)
+        let { file, ext = 'mjs' } = findHandler(nodeHandlers, src)
         let handlerModuleSystem = ext === 'mjs' ? 'esm' : 'cjs'
         let pkgFile = join(src, 'package.json')
         if (existsSync(pkgFile)) {
           let pkg = JSON.parse(readFileSync(pkgFile))
-          handlerModuleSystem = getModSystem(pkg)
+
+          /**/ if (pkg?.type === 'module') handlerModuleSystem = 'esm'
+          else if (pkg?.type === 'commonjs') handlerModuleSystem = 'cjs'
+          else if (pkg?.type) throw Error(`Invalid 'type' field: ${pkg.type}`)
+          else handlerModuleSystem = 'cjs' // Lambda's default, not ours
+
+          // We always get to make this a .js file, even if it's ESM!
+          ext = 'js'
         }
         return { file, ext, handlerModuleSystem }
       }
@@ -78,10 +87,3 @@ function findHandler (arr, src){
   }
   return {}
 }
-
-function getModSystem (pkg) {
-  if (pkg?.type === 'module') return 'esm'
-  else if (pkg?.type === 'commonjs') return 'cjs'
-  else if (pkg?.type) throw Error(`Invalid 'type' field: ${pkg.type}`)
-  return 'cjs'
-}