@runtime-type-inspector/transpiler 3.3.5 → 3.3.6

package/index.cjs

@@ -964,7 +964,7 @@
 	function parseJSDoc(src) {
 	  var expandType = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : expandTypeDepFree;
 	  // Parse something like: @param {Object} [kwargs={}] Optional arguments.
-	  var regex = /@param \{(.*?)\} ([\[\]a-zA-Z0-9_$=\{\}\.'" ]+)/g;
+	  var regex = /@param \{(.*?)\} ([\[\]a-zA-Z0-9_$=\-\{\}\.'" ]+)/g;
 	  var matches = _toConsumableArray(src.matchAll(regex));
 	  /** @type {Record<string, ExpandTypeReturnType>} */
 	  var params = Object.create(null);
@@ -3610,6 +3610,12 @@
 	        var tmp = JSON.stringify(templates, null, 2).replaceAll('\n', '\n' + spaces);
 	        out += "\n".concat(spaces, "const rtiTemplates = ").concat(tmp, ";");
 	      }
+	      function newlineBeforeFirst() {
+	        if (first) {
+	          out += '\n';
+	          first = false;
+	        }
+	      }
 	      //out += `${spaces}/*${spaces}  node.type=${node.type}\n${spaces}
 	      //  ${JSON.stringify(jsdoc)}\n${parent}\n${spaces}*/\n`;
 	      var _loop = function _loop(name) {
@@ -3653,11 +3659,19 @@
 	                  try {
 	                    for (_iterator.s(); !(_step = _iterator.n()).done;) {
 	                      var element = _step.value;
+	                      if (element.type === 'RestElement') {
+	                        // Case for:
+	                        // function test([a, b, ...rest] = [1, 2, 3]) {return rest;}
+	                        /** @todo We could typecheck the rest element too */
+	                        // console.log("Ignore RestElement", element);
+	                        continue;
+	                      }
 	                      if (element.type !== 'Identifier') {
 	                        _this2.warn('Only Identifier case handled right now');
 	                        continue;
 	                      }
 	                      var _t = JSON.stringify(type.elementType, null, 2).replaceAll('\n', '\n' + spaces);
+	                      newlineBeforeFirst();
 	                      if (templates) {
 	                        out += "".concat(spaces, "if (!inspectTypeWithTemplates(").concat(element.name, ", ").concat(_t, ", '").concat(_loc, "', '").concat(nameFancy, "', rtiTemplates)) {\n");
 	                      } else {
@@ -3679,6 +3693,11 @@
 	                    try {
 	                      for (_iterator2.s(); !(_step2 = _iterator2.n()).done;) {
 	                        var property = _step2.value;
+	                        if (property.type !== 'ObjectProperty') {
+	                          // E.g. case of RestElement, {a, b, c, ...kwargs}
+	                          // console.log("Ignore property", property);
+	                          continue;
+	                        }
 	                        if (property.key.type !== 'Identifier') {
 	                          _this2.warn('ObjectPattern> Only Identifier case handled right now');
 	                          continue;
@@ -3694,6 +3713,7 @@
 	                          continue;
 	                        }
 	                        var _t2 = JSON.stringify(subType, null, 2).replaceAll('\n', '\n' + spaces);
+	                        newlineBeforeFirst();
 	                        if (templates) {
 	                          out += "".concat(spaces, "if (!inspectTypeWithTemplates(").concat(keyName, ", ").concat(_t2, ", '").concat(_loc, "', '").concat(nameFancy, "', rtiTemplates)) {\n");
 	                        } else {
@@ -3737,10 +3757,7 @@
 	          if (loc === 'ContactPoint#constructor' || loc === 'ContactResult#constructor' || loc === 'SingleContactResult#constructor') {
 	            prevCheck = 'arguments.length !== 0 && ';
 	          }
-	          if (first) {
-	            out += '\n';
-	            first = false;
-	          }
+	          newlineBeforeFirst();
 	          if (templates) {
 	            out += "".concat(spaces, "if (").concat(prevCheck, "!inspectTypeWithTemplates(").concat(name, ", ").concat(t, ", '").concat(loc, "', '").concat(nameFancy, "', rtiTemplates)) {\n");
 	          } else {

package/index.mjs

@@ -744,7 +744,7 @@ function simplifyType(type, optional) {
  */
 function parseJSDoc(src, expandType = expandTypeDepFree) {
   // Parse something like: @param {Object} [kwargs={}] Optional arguments.
-  const regex = /@param \{(.*?)\} ([\[\]a-zA-Z0-9_$=\{\}\.'" ]+)/g;
+  const regex = /@param \{(.*?)\} ([\[\]a-zA-Z0-9_$=\-\{\}\.'" ]+)/g;
   const matches = [...src.matchAll(regex)];
   /** @type {Record<string, ExpandTypeReturnType>} */
   const params = Object.create(null);
@@ -3301,6 +3301,12 @@ class Asserter extends Stringifier {
       const tmp = JSON.stringify(templates, null, 2).replaceAll('\n', '\n' + spaces);
       out += `\n${spaces}const rtiTemplates = ${tmp};`;
     }
+    function newlineBeforeFirst() {
+      if (first) {
+        out += '\n';
+        first = false;
+      }
+    }
     //out += `${spaces}/*${spaces}  node.type=${node.type}\n${spaces}
     //  ${JSON.stringify(jsdoc)}\n${parent}\n${spaces}*/\n`;
     for (let name in params) {
@@ -3338,11 +3344,19 @@ class Asserter extends Stringifier {
             if (param.left.type === 'ArrayPattern' && type.type === 'array') {
               // Add a type assertion for each element of the ArrayPattern
               for (const element of param.left.elements) {
+                if (element.type === 'RestElement') {
+                  // Case for:
+                  // function test([a, b, ...rest] = [1, 2, 3]) {return rest;}
+                  /** @todo We could typecheck the rest element too */
+                  // console.log("Ignore RestElement", element);
+                  continue;
+                }
                 if (element.type !== 'Identifier') {
                   this.warn('Only Identifier case handled right now');
                   continue;
                 }
                 const _t = JSON.stringify(type.elementType, null, 2).replaceAll('\n', '\n' + spaces);
+                newlineBeforeFirst();
                 if (templates) {
                   out += `${spaces}if (!inspectTypeWithTemplates(${element.name}, ${_t}, '${_loc}', '${nameFancy}', rtiTemplates)) {\n`;
                 } else {
@@ -3355,6 +3369,11 @@ class Asserter extends Stringifier {
               if (type.type === 'object') {
                 // Add a type assertion for each property of the ObjectPattern
                 for (const property of param.left.properties) {
+                  if (property.type !== 'ObjectProperty') {
+                    // E.g. case of RestElement, {a, b, c, ...kwargs}
+                    // console.log("Ignore property", property);
+                    continue;
+                  }
                   if (property.key.type !== 'Identifier') {
                     this.warn('ObjectPattern> Only Identifier case handled right now');
                     continue;
@@ -3370,6 +3389,7 @@ class Asserter extends Stringifier {
                     continue;
                   }
                   const _t2 = JSON.stringify(subType, null, 2).replaceAll('\n', '\n' + spaces);
+                  newlineBeforeFirst();
                   if (templates) {
                     out += `${spaces}if (!inspectTypeWithTemplates(${keyName}, ${_t2}, '${_loc}', '${nameFancy}', rtiTemplates)) {\n`;
                   } else {
@@ -3408,10 +3428,7 @@ class Asserter extends Stringifier {
       if (_loc3 === 'ContactPoint#constructor' || _loc3 === 'ContactResult#constructor' || _loc3 === 'SingleContactResult#constructor') {
         prevCheck = 'arguments.length !== 0 && ';
       }
-      if (first) {
-        out += '\n';
-        first = false;
-      }
+      newlineBeforeFirst();
       if (templates) {
         out += `${spaces}if (${prevCheck}!inspectTypeWithTemplates(${name}, ${t}, '${_loc3}', '${nameFancy}', rtiTemplates)) {\n`;
       } else {

package/package.json

@@ -1,7 +1,7 @@
 {
   "bin": "bin.js",
   "name": "@runtime-type-inspector/transpiler",
-  "version": "3.3.5",
+  "version": "3.3.6",
   "description": "Validating JSDoc types at runtime for high-quality types - Trust is good, control is better.",
   "main": "index.cjs",
   "module": "index.mjs",
@@ -19,7 +19,7 @@
   "homepage": "https://github.com/kungfooman/RuntimeTypeInspector.js#readme",
   "dependencies": {
     "@babel/parser": "^7.23.0",
-    "@runtime-type-inspector/runtime": "^3.3.5",
+    "@runtime-type-inspector/runtime": "^3.3.6",
     "typescript": "^5.1.6"
   },
   "files": [