datagrok-tools 4.13.74 → 4.13.76

package/bin/commands/check.js

@@ -1,7 +1,6 @@
 "use strict";
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
-var _typeof3 = require("@babel/runtime/helpers/typeof");
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
@@ -13,25 +12,19 @@ exports.checkNpmIgnore = checkNpmIgnore;
 exports.checkPackageFile = checkPackageFile;
 exports.checkSourceMap = checkSourceMap;
 exports.extractExternals = extractExternals;
-var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime/helpers/slicedToArray"));
-var _typeof2 = _interopRequireDefault(require("@babel/runtime/helpers/typeof"));
-var _toConsumableArray2 = _interopRequireDefault(require("@babel/runtime/helpers/toConsumableArray"));
 var _fs = _interopRequireDefault(require("fs"));
 var _path = _interopRequireDefault(require("path"));
 var _ignoreWalk = _interopRequireDefault(require("ignore-walk"));
 var utils = _interopRequireWildcard(require("../utils/utils"));
 var color = _interopRequireWildcard(require("../utils/color-utils"));
 var testUtils = _interopRequireWildcard(require("../utils/test-utils"));
-function _getRequireWildcardCache(e) { if ("function" != typeof WeakMap) return null; var r = new WeakMap(), t = new WeakMap(); return (_getRequireWildcardCache = function _getRequireWildcardCache(e) { return e ? t : r; })(e); }
-function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e; if (null === e || "object" != _typeof3(e) && "function" != typeof e) return { "default": e }; var t = _getRequireWildcardCache(r); if (t && t.has(e)) return t.get(e); var n = { __proto__: null }, a = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var u in e) if ("default" !== u && Object.prototype.hasOwnProperty.call(e, u)) { var i = a ? Object.getOwnPropertyDescriptor(e, u) : null; i && (i.get || i.set) ? Object.defineProperty(n, u, i) : n[u] = e[u]; } return n["default"] = e, t && t.set(e, n), n; }
-function _createForOfIteratorHelper(o, allowArrayLike) { var it = typeof Symbol !== "undefined" && o[Symbol.iterator] || o["@@iterator"]; if (!it) { if (Array.isArray(o) || (it = _unsupportedIterableToArray(o)) || allowArrayLike && o && typeof o.length === "number") { if (it) o = it; var i = 0; var F = function F() {}; return { s: F, n: function n() { if (i >= o.length) return { done: true }; return { done: false, value: o[i++] }; }, e: function e(_e) { throw _e; }, f: F }; } throw new TypeError("Invalid attempt to iterate non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); } var normalCompletion = true, didErr = false, err; return { s: function s() { it = it.call(o); }, n: function n() { var step = it.next(); normalCompletion = step.done; return step; }, e: function e(_e2) { didErr = true; err = _e2; }, f: function f() { try { if (!normalCompletion && it["return"] != null) it["return"](); } finally { if (didErr) throw err; } } }; }
-function _unsupportedIterableToArray(o, minLen) { if (!o) return; if (typeof o === "string") return _arrayLikeToArray(o, minLen); var n = Object.prototype.toString.call(o).slice(8, -1); if (n === "Object" && o.constructor) n = o.constructor.name; if (n === "Map" || n === "Set") return Array.from(o); if (n === "Arguments" || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(n)) return _arrayLikeToArray(o, minLen); }
-function _arrayLikeToArray(arr, len) { if (len == null || len > arr.length) len = arr.length; for (var i = 0, arr2 = new Array(len); i < len; i++) arr2[i] = arr[i]; return arr2; }
-var warns = ['Latest package version', 'Datagrok API version should contain'];
+function _getRequireWildcardCache(e) { if ("function" != typeof WeakMap) return null; var r = new WeakMap(), t = new WeakMap(); return (_getRequireWildcardCache = function (e) { return e ? t : r; })(e); }
+function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e; if (null === e || "object" != typeof e && "function" != typeof e) return { default: e }; var t = _getRequireWildcardCache(r); if (t && t.has(e)) return t.get(e); var n = { __proto__: null }, a = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var u in e) if ("default" !== u && Object.prototype.hasOwnProperty.call(e, u)) { var i = a ? Object.getOwnPropertyDescriptor(e, u) : null; i && (i.get || i.set) ? Object.defineProperty(n, u, i) : n[u] = e[u]; } return n.default = e, t && t.set(e, n), n; }
+const warns = ['Latest package version', 'Datagrok API version should contain'];
 function check(args) {
-  var nOptions = Object.keys(args).length - 1;
+  const nOptions = Object.keys(args).length - 1;
   if (args['_'].length !== 1 || nOptions > 2 || nOptions > 0 && !args.r && !args.recursive) return false;
-  var curDir = process.cwd();
+  const curDir = process.cwd();
   if (args.recursive) return runChecksRec(curDir);else {
     if (!utils.isPackageDir(curDir)) {
       color.error('File `package.json` not found. Run the command from the package directory');
@@ -41,90 +34,75 @@ function check(args) {
   }
 }
 function runChecks(packagePath) {
-  var files = _ignoreWalk["default"].sync({
+  const files = _ignoreWalk.default.sync({
     path: packagePath,
     ignoreFiles: ['.npmignore', '.gitignore']
   });
-  var jsTsFiles = files.filter(function (f) {
-    return !f.startsWith('dist/') && (f.endsWith('.js') || f.endsWith('.ts') || f.endsWith('.sql') || f.endsWith('.py'));
-  });
-  var packageFiles = ['src/package.ts', 'src/detectors.ts', 'src/package.js', 'src/detectors.js', 'src/package-test.ts', 'src/package-test.js', 'package.js', 'detectors.js'];
+  const jsTsFiles = files.filter(f => !f.startsWith('dist/') && (f.endsWith('.js') || f.endsWith('.ts') || f.endsWith('.sql') || f.endsWith('.py')));
+  const packageFiles = ['src/package.ts', 'src/detectors.ts', 'src/package.js', 'src/detectors.js', 'src/package-test.ts', 'src/package-test.js', 'package.js', 'detectors.js'];
   // const funcFiles = jsTsFiles.filter((f) => packageFiles.includes(f)); 
-  var errors = [];
-  var warnings = [];
-  var packageFilePath = _path["default"].join(packagePath, 'package.json');
-  var json = JSON.parse(_fs["default"].readFileSync(packageFilePath, {
+  const errors = [];
+  const warnings = [];
+  const packageFilePath = _path.default.join(packagePath, 'package.json');
+  const json = JSON.parse(_fs.default.readFileSync(packageFilePath, {
     encoding: 'utf-8'
   }));
-  var webpackConfigPath = _path["default"].join(packagePath, 'webpack.config.js');
-  var isWebpack = _fs["default"].existsSync(webpackConfigPath);
-  var isReleaseCandidateVersion = false;
-  var externals = null;
+  const webpackConfigPath = _path.default.join(packagePath, 'webpack.config.js');
+  const isWebpack = _fs.default.existsSync(webpackConfigPath);
+  let isReleaseCandidateVersion = false;
+  let externals = null;
   if (/\d+.\d+.\d+-rc(.[A-Za-z0-9]*.[A-Za-z0-9]*)?/.test(json.version)) isReleaseCandidateVersion = true;
   if (isWebpack) {
-    var content = _fs["default"].readFileSync(webpackConfigPath, {
+    const content = _fs.default.readFileSync(webpackConfigPath, {
       encoding: 'utf-8'
     });
     externals = extractExternals(content);
-    if (externals) errors.push.apply(errors, (0, _toConsumableArray2["default"])(checkImportStatements(packagePath, jsTsFiles, externals)));
+    if (externals) errors.push(...checkImportStatements(packagePath, jsTsFiles, externals));
   }
-  errors.push.apply(errors, (0, _toConsumableArray2["default"])(checkSourceMap(packagePath)));
-  errors.push.apply(errors, (0, _toConsumableArray2["default"])(checkNpmIgnore(packagePath)));
-  warnings.push.apply(warnings, (0, _toConsumableArray2["default"])(checkScriptNames(packagePath)));
-  errors.push.apply(errors, (0, _toConsumableArray2["default"])(checkFuncSignatures(packagePath, jsTsFiles)));
-  errors.push.apply(errors, (0, _toConsumableArray2["default"])(checkPackageFile(packagePath, json, {
-    isWebpack: isWebpack,
-    externals: externals,
-    isReleaseCandidateVersion: isReleaseCandidateVersion
-  })));
-  if (!isReleaseCandidateVersion) warnings.push.apply(warnings, (0, _toConsumableArray2["default"])(checkChangelog(packagePath, json)));
+  errors.push(...checkSourceMap(packagePath));
+  errors.push(...checkNpmIgnore(packagePath));
+  warnings.push(...checkScriptNames(packagePath));
+  errors.push(...checkFuncSignatures(packagePath, jsTsFiles));
+  errors.push(...checkPackageFile(packagePath, json, {
+    isWebpack,
+    externals,
+    isReleaseCandidateVersion
+  }));
+  if (!isReleaseCandidateVersion) warnings.push(...checkChangelog(packagePath, json));
   if (warnings.length) {
-    console.log("".concat(_path["default"].basename(packagePath), " warnings"));
+    console.log(`${_path.default.basename(packagePath)} warnings`);
     warn(warnings);
   }
   if (errors.length) {
-    console.log("Checking package ".concat(_path["default"].basename(packagePath), "..."));
+    console.log(`Checking package ${_path.default.basename(packagePath)}...`);
     showError(errors);
-    if (json.version.startsWith('0') || errors.every(function (w) {
-      return warns.some(function (ww) {
-        return w.includes(ww);
-      });
-    })) return true;
+    if (json.version.startsWith('0') || errors.every(w => warns.some(ww => w.includes(ww)))) return true;
     testUtils.exitWithCode(1);
   }
-  console.log("Checking package ".concat(_path["default"].basename(packagePath), "...\t\t\t\u2713 OK"));
+  console.log(`Checking package ${_path.default.basename(packagePath)}...\t\t\t\u2713 OK`);
   return true;
 }
 function runChecksRec(dir) {
-  var files = _fs["default"].readdirSync(dir);
-  var _iterator = _createForOfIteratorHelper(files),
-    _step;
-  try {
-    for (_iterator.s(); !(_step = _iterator.n()).done;) {
-      var file = _step.value;
-      var filepath = _path["default"].join(dir, file);
-      var stats = _fs["default"].statSync(filepath);
-      if (stats.isDirectory()) {
-        if (utils.isPackageDir(filepath)) return runChecks(filepath);else {
-          if (file !== 'node_modules' && !file.startsWith('.')) runChecksRec(_path["default"].join(dir, file));
-        }
+  const files = _fs.default.readdirSync(dir);
+  for (const file of files) {
+    const filepath = _path.default.join(dir, file);
+    const stats = _fs.default.statSync(filepath);
+    if (stats.isDirectory()) {
+      if (utils.isPackageDir(filepath)) return runChecks(filepath);else {
+        if (file !== 'node_modules' && !file.startsWith('.')) runChecksRec(_path.default.join(dir, file));
       }
     }
-  } catch (err) {
-    _iterator.e(err);
-  } finally {
-    _iterator.f();
   }
   return false;
 }
 function extractExternals(config) {
-  var externalsRegex = /(?<=externals)\s*:\s*(\{[\S\s]*?\})/;
-  var match = config.match(externalsRegex);
+  const externalsRegex = /(?<=externals)\s*:\s*(\{[\S\s]*?\})/;
+  const match = config.match(externalsRegex);
   if (match) {
     // Replace single quotes, comments, and a trailing comma to make a string JSON-like
-    var externalStr = match[1].replace(/'/g, '"').replace(/\/\/.*(\r\n|\r|\n)/, '').replace(/(?<=[\S\s]),(?=\s*\})/, '');
+    const externalStr = match[1].replace(/'/g, '"').replace(/\/\/.*(\r\n|\r|\n)/, '').replace(/(?<=[\S\s]),(?=\s*\})/, '');
     try {
-      var externals = JSON.parse(externalStr);
+      const externals = JSON.parse(externalStr);
       return externals;
     } catch (e) {
       return null;
@@ -133,63 +111,46 @@ function extractExternals(config) {
   return null;
 }
 function checkImportStatements(packagePath, files, externals) {
-  var modules = [];
-  for (var _key in externals) {
-    modules.push(_key);
-    if (_key.includes('/')) modules.push(_key.split('/', 1)[0]);
+  const modules = [];
+  for (const key in externals) {
+    modules.push(key);
+    if (key.includes('/')) modules.push(key.split('/', 1)[0]);
   }
-  var importRegex = new RegExp("^(?!\\/{2})\\s*import\\s+.*(".concat(modules.join('|'), ").*(?=\\s+?)"), 'g');
-  var validImportRegex = new RegExp("import\\s+.*(".concat(Object.keys(externals).join('|'), ")['\"]{1}"));
-  var warnings = [];
+  const importRegex = new RegExp(`^(?!\\/{2})\\s*import\\s+.*(${modules.join('|')}).*(?=\\s+?)`, 'g');
+  const validImportRegex = new RegExp(`import\\s+.*(${Object.keys(externals).join('|')})['"]{1}`);
+  const warnings = [];
   function validateImport(file, s) {
-    var value = validImportRegex.test(s);
-    var message = value ? '' : 'Pay attention to file ' + file + ': import statement `' + s + '` differs from the path given in the webpack config as an external module. ' + 'It can increase the bundle size.';
+    const value = validImportRegex.test(s);
+    const message = value ? '' : 'Pay attention to file ' + file + ': import statement `' + s + '` differs from the path given in the webpack config as an external module. ' + 'It can increase the bundle size.';
     return {
-      value: value,
-      message: message
+      value,
+      message
     };
   }
-  var _iterator2 = _createForOfIteratorHelper(files),
-    _step2;
-  try {
-    for (_iterator2.s(); !(_step2 = _iterator2.n()).done;) {
-      var file = _step2.value;
-      var content = _fs["default"].readFileSync(_path["default"].join(packagePath, file), {
-        encoding: 'utf-8'
-      });
-      var matchedImports = content.match(importRegex);
-      if (matchedImports) {
-        var _iterator3 = _createForOfIteratorHelper(matchedImports),
-          _step3;
-        try {
-          for (_iterator3.s(); !(_step3 = _iterator3.n()).done;) {
-            var match = _step3.value;
-            var vr = validateImport(file, match);
-            if (!vr.value) warnings.push(vr.message);
-          }
-        } catch (err) {
-          _iterator3.e(err);
-        } finally {
-          _iterator3.f();
-        }
+  for (const file of files) {
+    const content = _fs.default.readFileSync(_path.default.join(packagePath, file), {
+      encoding: 'utf-8'
+    });
+    const matchedImports = content.match(importRegex);
+    if (matchedImports) {
+      for (const match of matchedImports) {
+        const vr = validateImport(file, match);
+        if (!vr.value) warnings.push(vr.message);
       }
     }
-  } catch (err) {
-    _iterator2.e(err);
-  } finally {
-    _iterator2.f();
   }
   return warnings;
 }
 function checkFuncSignatures(packagePath, files) {
-  var warnings = [];
-  var checkFunctions = {
-    app: function app(_ref) {
-      var name = _ref.name;
-      var value = true;
-      var message = '';
+  const warnings = [];
+  const checkFunctions = {
+    app: ({
+      name
+    }) => {
+      let value = true;
+      let message = '';
       if (name && typeof name === 'string') {
-        var lowerCaseName = name.toLocaleLowerCase();
+        const lowerCaseName = name.toLocaleLowerCase();
         if (lowerCaseName.startsWith('app')) {
           value = false;
           message += 'Prefix "App" is not needed. Consider removing it.\n';
@@ -200,15 +161,16 @@ function checkFuncSignatures(packagePath, files) {
         }
       }
       return {
-        value: value,
-        message: message
+        value,
+        message
       };
     },
-    semTypeDetector: function semTypeDetector(_ref2) {
-      var inputs = _ref2.inputs,
-        outputs = _ref2.outputs;
-      var value = true;
-      var message = '';
+    semTypeDetector: ({
+      inputs,
+      outputs
+    }) => {
+      let value = true;
+      let message = '';
       if (inputs.length !== 1 || inputs[0].type !== 'column') {
         value = false;
         message += 'Semantic type detectors must have one input of type "column"\n';
@@ -218,15 +180,16 @@ function checkFuncSignatures(packagePath, files) {
         message += 'Semantic type detectors must have one output of type "string"\n';
       }
       return {
-        value: value,
-        message: message
+        value,
+        message
       };
     },
-    cellRenderer: function cellRenderer(_ref3) {
-      var inputs = _ref3.inputs,
-        outputs = _ref3.outputs;
-      var value = true;
-      var message = '';
+    cellRenderer: ({
+      inputs,
+      outputs
+    }) => {
+      let value = true;
+      let message = '';
       if (inputs.length !== 0) {
         value = false;
         message += 'Cell renderer functions should take no arguments\n';
@@ -236,15 +199,16 @@ function checkFuncSignatures(packagePath, files) {
         message += 'Cell renderer functions must have one output of type "grid_cell_renderer"\n';
       }
       return {
-        value: value,
-        message: message
+        value,
+        message
       };
     },
-    viewer: function viewer(_ref4) {
-      var inputs = _ref4.inputs,
-        outputs = _ref4.outputs;
-      var value = true;
-      var message = '';
+    viewer: ({
+      inputs,
+      outputs
+    }) => {
+      let value = true;
+      let message = '';
       if (inputs.length !== 0) {
         value = false;
         message += 'Viewer functions should take no arguments\n';
@@ -254,16 +218,17 @@ function checkFuncSignatures(packagePath, files) {
         message += 'Viewers must have one output of type "viewer"\n';
       }
       return {
-        value: value,
-        message: message
+        value,
+        message
       };
     },
-    fileViewer: function fileViewer(_ref5) {
-      var inputs = _ref5.inputs,
-        outputs = _ref5.outputs,
-        tags = _ref5.tags;
-      var value = true;
-      var message = '';
+    fileViewer: ({
+      inputs,
+      outputs,
+      tags
+    }) => {
+      let value = true;
+      let message = '';
       if (tags == null || tags.length !== 1 && tags[0] !== 'fileViewer') {
         value = false;
         message += 'File viewers must have only one tag: "fileViewer"\n';
@@ -277,118 +242,86 @@ function checkFuncSignatures(packagePath, files) {
         message += 'File viewers must have one output of type "view"\n';
       }
       return {
-        value: value,
-        message: message
+        value,
+        message
       };
     },
-    fileExporter: function fileExporter(_ref6) {
-      var description = _ref6.description;
-      var value = true;
-      var message = '';
+    fileExporter: ({
+      description
+    }) => {
+      let value = true;
+      let message = '';
       if (description == null || description === '') {
         value = false;
         message += 'File exporters should have a description parameter\n';
       }
       return {
-        value: value,
-        message: message
+        value,
+        message
       };
     },
-    packageSettingsEditor: function packageSettingsEditor(_ref7) {
-      var outputs = _ref7.outputs;
-      var value = true;
-      var message = '';
+    packageSettingsEditor: ({
+      outputs
+    }) => {
+      let value = true;
+      let message = '';
       if (!(outputs.length === 1 && outputs[0].type === 'widget')) {
         value = false;
         message += 'Package settings editors must have one output of type "widget"\n';
       }
       return {
-        value: value,
-        message: message
+        value,
+        message
       };
     },
-    params: function params(_ref8) {
-      var inputs = _ref8.inputs,
-        outputs = _ref8.outputs;
-      var value = true;
-      var message = '';
-      var _iterator4 = _createForOfIteratorHelper(inputs),
-        _step4;
-      try {
-        for (_iterator4.s(); !(_step4 = _iterator4.n()).done;) {
-          var input = _step4.value;
-          if (!(input.name && input.type)) {
-            value = false;
-            message += "Function has no name or type of input parameter\n";
-          }
+    params: ({
+      inputs,
+      outputs
+    }) => {
+      let value = true;
+      let message = '';
+      for (const input of inputs) {
+        if (!(input.name && input.type)) {
+          value = false;
+          message += `Function has no name or type of input parameter\n`;
         }
-      } catch (err) {
-        _iterator4.e(err);
-      } finally {
-        _iterator4.f();
       }
-      var _iterator5 = _createForOfIteratorHelper(outputs),
-        _step5;
-      try {
-        for (_iterator5.s(); !(_step5 = _iterator5.n()).done;) {
-          var output = _step5.value;
-          if (!(output.name && output.type)) {
-            value = false;
-            message += "Function has no name or type of output parameter\n";
-          }
+      for (const output of outputs) {
+        if (!(output.name && output.type)) {
+          value = false;
+          message += `Function has no name or type of output parameter\n`;
         }
-      } catch (err) {
-        _iterator5.e(err);
-      } finally {
-        _iterator5.f();
       }
       return {
-        value: value,
-        message: message
+        value,
+        message
       };
     }
   };
-  var functionRoles = Object.keys(checkFunctions);
-  var _iterator6 = _createForOfIteratorHelper(files),
-    _step6;
-  try {
-    for (_iterator6.s(); !(_step6 = _iterator6.n()).done;) {
-      var _file$split$pop;
-      var file = _step6.value;
-      var content = _fs["default"].readFileSync(_path["default"].join(packagePath, file), {
-        encoding: 'utf-8'
-      });
-      var functions = getFuncMetadata(content, (_file$split$pop = file.split('.').pop()) !== null && _file$split$pop !== void 0 ? _file$split$pop : 'ts');
-      var _loop = function _loop() {
-        var f = _functions[_i];
-        var paramsCheck = checkFunctions.params(f);
-        if (!paramsCheck.value) warnings.push("File ".concat(file, ", function ").concat(f.name, ":\n").concat(paramsCheck.message));
-        var roles = functionRoles.filter(function (role) {
-          var _f$tags;
-          return (_f$tags = f.tags) === null || _f$tags === void 0 ? void 0 : _f$tags.includes(role);
-        });
-        if (roles.length > 1) warnings.push("File ".concat(file, ", function ").concat(f.name, ": several function roles are used (").concat(roles.join(', '), ")"));else if (roles.length === 1) {
-          var vr = checkFunctions[roles[0]](f);
-          if (!vr.value) {
-            warnings.push("File ".concat(file, ", function ").concat(f.name, ":\n").concat(vr.message));
-          }
+  const functionRoles = Object.keys(checkFunctions);
+  for (const file of files) {
+    const content = _fs.default.readFileSync(_path.default.join(packagePath, file), {
+      encoding: 'utf-8'
+    });
+    const functions = getFuncMetadata(content, file.split('.').pop() ?? 'ts');
+    for (const f of functions) {
+      const paramsCheck = checkFunctions.params(f);
+      if (!paramsCheck.value) warnings.push(`File ${file}, function ${f.name}:\n${paramsCheck.message}`);
+      const roles = functionRoles.filter(role => f.tags?.includes(role));
+      if (roles.length > 1) warnings.push(`File ${file}, function ${f.name}: several function roles are used (${roles.join(', ')})`);else if (roles.length === 1) {
+        const vr = checkFunctions[roles[0]](f);
+        if (!vr.value) {
+          warnings.push(`File ${file}, function ${f.name}:\n${vr.message}`);
         }
-        if (f.isInvalidateOnWithoutCache) warnings.push("File ".concat(file, ", function ").concat(f.name, ": Can't use invalidateOn without cache, please follow this example: 'meta.cache.invalidateOn'"));
-        if (f.cache) if (!utils.cahceValues.includes(f.cache)) warnings.push("File ".concat(file, ", function ").concat(f.name, ": unsupposed variable for cache : ").concat(f.cache));
-        if (f.invalidateOn) if (!utils.isValidCron(f.invalidateOn)) warnings.push("File ".concat(file, ", function ").concat(f.name, ": unsupposed variable for invalidateOn : ").concat(f.invalidateOn));
-      };
-      for (var _i = 0, _functions = functions; _i < _functions.length; _i++) {
-        _loop();
       }
+      if (f.isInvalidateOnWithoutCache) warnings.push(`File ${file}, function ${f.name}: Can't use invalidateOn without cache, please follow this example: 'meta.cache.invalidateOn'`);
+      if (f.cache) if (!utils.cahceValues.includes(f.cache)) warnings.push(`File ${file}, function ${f.name}: unsupposed variable for cache : ${f.cache}`);
+      if (f.invalidateOn) if (!utils.isValidCron(f.invalidateOn)) warnings.push(`File ${file}, function ${f.name}: unsupposed variable for invalidateOn : ${f.invalidateOn}`);
     }
-  } catch (err) {
-    _iterator6.e(err);
-  } finally {
-    _iterator6.f();
   }
   return warnings;
 }
-var sharedLibExternals = {
+const sharedLibExternals = {
   'common/html2canvas.min.js': {
     'exceljs': 'ExcelJS'
   },
@@ -409,86 +342,61 @@ var sharedLibExternals = {
   }
 };
 function checkPackageFile(packagePath, json, options) {
-  var _json$dependencies, _json$devDependencies, _json$devDependencies2, _json$dependencies2;
-  var warnings = [];
-  var isPublicPackage = _path["default"].basename(_path["default"].dirname(packagePath)) === 'packages' && _path["default"].basename(_path["default"].dirname(_path["default"].dirname(packagePath))) === 'public';
+  const warnings = [];
+  const isPublicPackage = _path.default.basename(_path.default.dirname(packagePath)) === 'packages' && _path.default.basename(_path.default.dirname(_path.default.dirname(packagePath))) === 'public';
   if (!json.description) warnings.push('File "package.json": "description" field is empty. Provide a package description.');
   if (Array.isArray(json.properties) && json.properties.length > 0) {
-    var _iterator7 = _createForOfIteratorHelper(json.properties),
-      _step7;
-    try {
-      for (_iterator7.s(); !(_step7 = _iterator7.n()).done;) {
-        var propInfo = _step7.value;
-        if ((0, _typeof2["default"])(propInfo) !== 'object') warnings.push('File "package.json": Invalid property annotation in the "properties" field.');else if (!propInfo.name) warnings.push('File "package.json": Add a property name for each property in the "properties" field.');else if (!utils.propertyTypes.includes(propInfo.propertyType)) warnings.push("File \"package.json\": Invalid property type for property ".concat(propInfo.name, "."));
-      }
-    } catch (err) {
-      _iterator7.e(err);
-    } finally {
-      _iterator7.f();
+    for (const propInfo of json.properties) {
+      if (typeof propInfo !== 'object') warnings.push('File "package.json": Invalid property annotation in the "properties" field.');else if (!propInfo.name) warnings.push('File "package.json": Add a property name for each property in the "properties" field.');else if (!utils.propertyTypes.includes(propInfo.propertyType)) warnings.push(`File "package.json": Invalid property type for property ${propInfo.name}.`);
     }
   }
   if (json.repository == null && isPublicPackage) warnings.push('File "package.json": add the "repository" field.');
   if (json.author == null && isPublicPackage) warnings.push('File "package.json": add the "author" field.');
   if (json.version.includes('beta') && isPublicPackage) warnings.push('File "package.json": public package cannot be beta version.');
-  var api = (_json$dependencies = json.dependencies) === null || _json$dependencies === void 0 ? void 0 : _json$dependencies['datagrok-api'];
+  const api = json.dependencies?.['datagrok-api'];
   if (api) {
-    if (api === '../../js-api') {} else if (api === 'latest') warnings.push('File "package.json": you should specify Datagrok API version constraint (for example ^1.16.0, >=1.16.0).');else if ((options === null || options === void 0 ? void 0 : options.isReleaseCandidateVersion) === false && !/^(\^|>|<|~).+/.test(api)) warnings.push('File "package.json": Datagrok API version should starts with > | >= | ~ | ^ | < | <=');
+    if (api === '../../js-api') {} else if (api === 'latest') warnings.push('File "package.json": you should specify Datagrok API version constraint (for example ^1.16.0, >=1.16.0).');else if (options?.isReleaseCandidateVersion === false && !/^(\^|>|<|~).+/.test(api)) warnings.push('File "package.json": Datagrok API version should starts with > | >= | ~ | ^ | < | <=');
   }
-  var dt = (_json$devDependencies = (_json$devDependencies2 = json.devDependencies) === null || _json$devDependencies2 === void 0 ? void 0 : _json$devDependencies2['datagrok-tools']) !== null && _json$devDependencies !== void 0 ? _json$devDependencies : (_json$dependencies2 = json.dependencies) === null || _json$dependencies2 === void 0 ? void 0 : _json$dependencies2['datagrok-tools'];
+  const dt = json.devDependencies?.['datagrok-tools'] ?? json.dependencies?.['datagrok-tools'];
   if (dt && dt !== 'latest') warnings.push('File "package.json": "datagrok-tools" dependency must be "latest" version.');
   if (Array.isArray(json.sources) && json.sources.length > 0) {
-    var _iterator8 = _createForOfIteratorHelper(json.sources),
-      _step8;
-    try {
-      for (_iterator8.s(); !(_step8 = _iterator8.n()).done;) {
-        var source = _step8.value;
-        if (typeof source !== 'string') warnings.push("File \"package.json\": Only file paths and URLs are allowed in sources. Modify the source ".concat(source));
-        if (utils.absUrlRegex.test(source)) continue;
-        if (source.startsWith('common/')) {
-          if (options !== null && options !== void 0 && options.isWebpack && source.endsWith('.js')) {
-            if (options !== null && options !== void 0 && options.externals) {
-              if (source in sharedLibExternals) {
-                var _Object$entries$ = (0, _slicedToArray2["default"])(Object.entries(sharedLibExternals[source])[0], 2),
-                  _lib = _Object$entries$[0],
-                  name = _Object$entries$[1];
-                if (!(_lib in options.externals && options.externals[_lib] === name)) {
-                  warnings.push("Webpack config parsing: Consider adding source \"".concat(source, "\" to webpack externals:\n") + "'".concat(_lib, "': '").concat(name, "'\n"));
-                }
-              } else warnings.push("File \"package.json\": source \"".concat(source, "\" not in the list of shared libraries"));
-            } else {
-              warnings.push('Webpack config parsing: External modules not found.\n' + "Consider adding source \"".concat(source, "\" to webpack externals") + (source in sharedLibExternals ? ':\n' + "'".concat(Object.keys(sharedLibExternals[source])[0], "': '").concat(Object.values(sharedLibExternals[source])[0], "'\n") : ''));
-            }
+    for (const source of json.sources) {
+      if (typeof source !== 'string') warnings.push(`File "package.json": Only file paths and URLs are allowed in sources. Modify the source ${source}`);
+      if (utils.absUrlRegex.test(source)) continue;
+      if (source.startsWith('common/')) {
+        if (options?.isWebpack && source.endsWith('.js')) {
+          if (options?.externals) {
+            if (source in sharedLibExternals) {
+              const [lib, name] = Object.entries(sharedLibExternals[source])[0];
+              if (!(lib in options.externals && options.externals[lib] === name)) {
+                warnings.push(`Webpack config parsing: Consider adding source "${source}" to webpack externals:\n` + `'${lib}': '${name}'\n`);
+              }
+            } else warnings.push(`File "package.json": source "${source}" not in the list of shared libraries`);
+          } else {
+            warnings.push('Webpack config parsing: External modules not found.\n' + `Consider adding source "${source}" to webpack externals` + (source in sharedLibExternals ? ':\n' + `'${Object.keys(sharedLibExternals[source])[0]}': '${Object.values(sharedLibExternals[source])[0]}'\n` : ''));
           }
-          continue;
         }
-        if (source.startsWith('src/') && _fs["default"].existsSync(_path["default"].join(packagePath, 'webpack.config.js'))) {
-          warnings.push('File "package.json": Sources cannot include files from the \`src/\` directory. ' + "Move file ".concat(source, " to another folder."));
-        }
-        if (!_fs["default"].existsSync(_path["default"].join(packagePath, source))) warnings.push("Source ".concat(source, " not found in the package."));
+        continue;
+      }
+      if (source.startsWith('src/') && _fs.default.existsSync(_path.default.join(packagePath, 'webpack.config.js'))) {
+        warnings.push('File "package.json": Sources cannot include files from the \`src/\` directory. ' + `Move file ${source} to another folder.`);
       }
-    } catch (err) {
-      _iterator8.e(err);
-    } finally {
-      _iterator8.f();
+      if (!_fs.default.existsSync(_path.default.join(packagePath, source))) warnings.push(`Source ${source} not found in the package.`);
     }
   }
-  if ((options === null || options === void 0 ? void 0 : options.isReleaseCandidateVersion) === true) {
-    var hasRCDependency = false;
-    for (var _i2 = 0, _Object$keys = Object.keys((_json$dependencies3 = json.dependencies) !== null && _json$dependencies3 !== void 0 ? _json$dependencies3 : {}); _i2 < _Object$keys.length; _i2++) {
-      var _json$dependencies3, _json$dependencies4;
-      var dependency = _Object$keys[_i2];
-      if (/\d+.\d+.\d+-rc(.[A-Za-z0-9]*.[A-Za-z0-9]*)?/.test(((_json$dependencies4 = json.dependencies) !== null && _json$dependencies4 !== void 0 ? _json$dependencies4 : {})[dependency])) {
+  if (options?.isReleaseCandidateVersion === true) {
+    let hasRCDependency = false;
+    for (let dependency of Object.keys(json.dependencies ?? {})) {
+      if (/\d+.\d+.\d+-rc(.[A-Za-z0-9]*.[A-Za-z0-9]*)?/.test((json.dependencies ?? {})[dependency])) {
         hasRCDependency = true;
         break;
       }
     }
     if (!hasRCDependency) {
-      for (var _i3 = 0, _Object$keys2 = Object.keys((_json$dependencies5 = json.dependencies) !== null && _json$dependencies5 !== void 0 ? _json$dependencies5 : {}); _i3 < _Object$keys2.length; _i3++) {
-        var _json$dependencies5, _json$dependencies6, _json$devDependencies3;
-        var _dependency = _Object$keys2[_i3];
-        console.log(_dependency);
-        console.log(((_json$dependencies6 = json.dependencies) !== null && _json$dependencies6 !== void 0 ? _json$dependencies6 : {})[_dependency]);
-        if (/\d+.\d+.\d+-rc(.[A-Za-z0-9]*.[A-Za-z0-9]*)?/.test(((_json$devDependencies3 = json.devDependencies) !== null && _json$devDependencies3 !== void 0 ? _json$devDependencies3 : {})[_dependency])) {
+      for (let dependency of Object.keys(json.dependencies ?? {})) {
+        console.log(dependency);
+        console.log((json.dependencies ?? {})[dependency]);
+        if (/\d+.\d+.\d+-rc(.[A-Za-z0-9]*.[A-Za-z0-9]*)?/.test((json.devDependencies ?? {})[dependency])) {
           hasRCDependency = true;
           break;
         }
@@ -499,185 +407,145 @@ function checkPackageFile(packagePath, json, options) {
   return warnings;
 }
 function checkChangelog(packagePath, json) {
-  var _h2$0$match, _h2$;
   if (json.servicePackage) return [];
-  var warnings = [];
-  var clf;
+  const warnings = [];
+  let clf;
   try {
-    clf = _fs["default"].readFileSync(_path["default"].join(packagePath, 'CHANGELOG.md'), {
+    clf = _fs.default.readFileSync(_path.default.join(packagePath, 'CHANGELOG.md'), {
       encoding: 'utf-8'
     });
   } catch (e) {
     return ['CHANGELOG.md file does not exist\n'];
   }
-  var regex = /^##[^#].*$/gm;
-  var h2 = clf.match(regex);
+  let regex = /^##[^#].*$/gm;
+  const h2 = clf.match(regex);
   if (!h2) return ['No versions found in CHANGELOG.md\n'];
   regex = /^## \d+\.\d+\.\d+ \((\d{4}-\d{2}-\d{2}|WIP)\)$/;
-  var _iterator9 = _createForOfIteratorHelper(h2),
-    _step9;
-  try {
-    for (_iterator9.s(); !(_step9 = _iterator9.n()).done;) {
-      var h = _step9.value;
-      if (!regex.test(h)) warnings.push("CHANGELOG: '".concat(h, "' does not match the h2 format, expected: ## <version> (<yyyy-mm-dd> | WIP)\n"));
-    }
-  } catch (err) {
-    _iterator9.e(err);
-  } finally {
-    _iterator9.f();
+  for (const h of h2) {
+    if (!regex.test(h)) warnings.push(`CHANGELOG: '${h}' does not match the h2 format, expected: ## <version> (<yyyy-mm-dd> | WIP)\n`);
   }
   regex = /^## (\d+\.\d+\.\d+)/;
-  var v1 = (_h2$0$match = h2[0].match(regex)) === null || _h2$0$match === void 0 ? void 0 : _h2$0$match[1];
-  var v2 = (_h2$ = h2[1]) === null || _h2$ === void 0 || (_h2$ = _h2$.match(regex)) === null || _h2$ === void 0 ? void 0 : _h2$[1];
-  if (v1 !== json.version && v2 !== json.version) warnings.push("Latest package version (".concat(json.version, ") is not in CHANGELOG\n"));
+  const v1 = h2[0].match(regex)?.[1];
+  const v2 = h2[1]?.match(regex)?.[1];
+  if (v1 !== json.version && v2 !== json.version) warnings.push(`Latest package version (${json.version}) is not in CHANGELOG\n`);
   if (warnings.length) warnings.push('Changelog guideline: https://datagrok.ai/help/develop/dev-process/changelog-policy#changelog-guideline');
   return warnings;
 }
 function checkSourceMap(packagePath) {
-  var warnings = [];
-  var tsconfigFilePath = _path["default"].join(packagePath, 'tsconfig.json');
-  var webpackConfigFilePath = _path["default"].join(packagePath, 'webpack.config.js');
-  if (_fs["default"].existsSync(tsconfigFilePath) && _fs["default"].existsSync(webpackConfigFilePath)) {
-    var configJson = _fs["default"].readFileSync(tsconfigFilePath, {
+  const warnings = [];
+  const tsconfigFilePath = _path.default.join(packagePath, 'tsconfig.json');
+  const webpackConfigFilePath = _path.default.join(packagePath, 'webpack.config.js');
+  if (_fs.default.existsSync(tsconfigFilePath) && _fs.default.existsSync(webpackConfigFilePath)) {
+    const configJson = _fs.default.readFileSync(tsconfigFilePath, {
       encoding: 'utf-8'
     }); // cant convert to json because file contains comments
 
     if (!new RegExp('"sourceMap"\\s*:\\s*true').test(configJson)) warnings.push('ts config doesnt contain source map');
-    var webpackConfigJson = _fs["default"].readFileSync(webpackConfigFilePath, {
+    const webpackConfigJson = _fs.default.readFileSync(webpackConfigFilePath, {
       encoding: 'utf-8'
     }); // cant convert to json because file contains comments
 
-    if (!new RegExp("devtool\\s*:\\s*(([^\\n]*?[^\\n]*source-map[^\\n]*:[^\\n]*source-map[^\\n]*)|('(inline-)?source-map'))\\s*").test(webpackConfigJson)) warnings.push('webpack config doesnt contain source map');
-    if (!_fs["default"].existsSync(packagePath + '/dist/package.js')) warnings.push('dist\\package.js file doesnt exists');
-    if (!_fs["default"].existsSync(packagePath + '/dist/package-test.js')) warnings.push('dist\\package-test.js file doesnt exists');
+    if (!new RegExp(`devtool\\s*:\\s*(([^\\n]*?[^\\n]*source-map[^\\n]*:[^\\n]*source-map[^\\n]*)|('(inline-)?source-map'))\\s*`).test(webpackConfigJson)) warnings.push('webpack config doesnt contain source map');
+    if (!_fs.default.existsSync(packagePath + '/dist/package.js')) warnings.push('dist\\package.js file doesnt exists');
+    if (!_fs.default.existsSync(packagePath + '/dist/package-test.js')) warnings.push('dist\\package-test.js file doesnt exists');
   }
   return warnings;
 }
 function checkNpmIgnore(packagePath) {
-  var warnings = [];
-  if (_path["default"].join.apply(_path["default"], [packagePath, '.npmignore'])) {
-    var npmIgnoreContent = _fs["default"].readFileSync(_path["default"].join.apply(_path["default"], [packagePath, '.npmignore']), {
+  const warnings = [];
+  if (_path.default.join(...[packagePath, '.npmignore'])) {
+    const npmIgnoreContent = _fs.default.readFileSync(_path.default.join(...[packagePath, '.npmignore']), {
       encoding: 'utf-8'
     });
-    var _iterator10 = _createForOfIteratorHelper(npmIgnoreContent.split('\n')),
-      _step10;
-    try {
-      for (_iterator10.s(); !(_step10 = _iterator10.n()).done;) {
-        var _row$match;
-        var row = _step10.value;
-        if ((((_row$match = row.match(new RegExp('\\s*dist\\/?\\s*$'))) === null || _row$match === void 0 ? void 0 : _row$match.length) || -1) > 0) {
-          warnings.push('there is dist directory in .npmignore');
-          break;
-        }
+    for (const row of npmIgnoreContent.split('\n')) {
+      if ((row.match(new RegExp('\\s*dist\\/?\\s*$'))?.length || -1) > 0) {
+        warnings.push('there is dist directory in .npmignore');
+        break;
       }
-    } catch (err) {
-      _iterator10.e(err);
-    } finally {
-      _iterator10.f();
     }
   } else warnings.push('.npmignore doesnt exists');
   return warnings;
 }
 function checkScriptNames(packagePath) {
-  var warnings = [];
+  const warnings = [];
   try {
-    if (_fs["default"].existsSync(packagePath)) {
-      var filesInDirectory = getAllFilesInDirectory(packagePath);
-      for (var _i4 = 0, _filesInDirectory = filesInDirectory; _i4 < _filesInDirectory.length; _i4++) {
-        var _fileName$match;
-        var fileName = _filesInDirectory[_i4];
-        if (((_fileName$match = fileName.match(new RegExp('^[A-Za-z0-9._-]*$'))) === null || _fileName$match === void 0 ? void 0 : _fileName$match.length) !== 1) warnings.push("".concat(fileName, ": file name contains inappropriate symbols"));
+    if (_fs.default.existsSync(packagePath)) {
+      const filesInDirectory = getAllFilesInDirectory(packagePath);
+      for (const fileName of filesInDirectory) {
+        if (fileName.match(new RegExp('^[A-Za-z0-9._-]*$'))?.length !== 1) warnings.push(`${fileName}: file name contains inappropriate symbols`);
       }
     }
   } catch (err) {}
   return warnings;
 }
 function getAllFilesInDirectory(directoryPath) {
-  var excludedFilesToCheck = ['node_modules', 'dist'];
-  var fileNames = [];
-  var entries = _fs["default"].readdirSync(directoryPath);
-  entries.forEach(function (entry) {
-    var entryPath = _path["default"].join(directoryPath, entry);
-    var stat = _fs["default"].statSync(entryPath);
+  const excludedFilesToCheck = ['node_modules', 'dist'];
+  let fileNames = [];
+  const entries = _fs.default.readdirSync(directoryPath);
+  entries.forEach(entry => {
+    const entryPath = _path.default.join(directoryPath, entry);
+    const stat = _fs.default.statSync(entryPath);
     if (stat.isFile()) {
       fileNames.push(entry);
     } else if (stat.isDirectory() && !excludedFilesToCheck.includes(entry)) {
-      var subDirectoryFiles = getAllFilesInDirectory(entryPath);
+      const subDirectoryFiles = getAllFilesInDirectory(entryPath);
       fileNames = fileNames.concat(subDirectoryFiles);
     }
   });
   return fileNames;
 }
 function showError(errors) {
-  errors.forEach(function (w) {
-    return color.error(w);
-  });
+  errors.forEach(w => color.error(w));
 }
 function warn(warnings) {
-  warnings.forEach(function (w) {
-    return color.warn(w);
-  });
+  warnings.forEach(w => color.warn(w));
 }
 function getFuncMetadata(script, fileExtention) {
-  var funcData = [];
-  var isHeader = false;
-  var data = {
+  const funcData = [];
+  let isHeader = false;
+  let data = {
     name: '',
     inputs: [],
     outputs: []
   };
-  var _iterator11 = _createForOfIteratorHelper(script.split('\n')),
-    _step11;
-  try {
-    for (_iterator11.s(); !(_step11 = _iterator11.n()).done;) {
-      var line = _step11.value;
-      if (!line) continue;
-      //@ts-ignore
-      var match = line.match(utils.fileParamRegex[fileExtention]);
-      if (match) {
-        var _line$match;
-        if (!isHeader) isHeader = true;
-        var param = match[1];
-        if (param === 'name') data.name = (_line$match = line.match(utils.nameAnnRegex)) === null || _line$match === void 0 ? void 0 : _line$match[2];else if (param === 'description') data.description = match[2];else if (param === 'input') {
-          data.inputs.push({
-            type: match[2],
-            name: match[3]
-          });
-        } else if (param === 'output') data.outputs.push({
+  for (const line of script.split('\n')) {
+    if (!line) continue;
+    //@ts-ignore
+    const match = line.match(utils.fileParamRegex[fileExtention]);
+    if (match) {
+      if (!isHeader) isHeader = true;
+      const param = match[1];
+      if (param === 'name') data.name = line.match(utils.nameAnnRegex)?.[2];else if (param === 'description') data.description = match[2];else if (param === 'input') {
+        data.inputs.push({
           type: match[2],
           name: match[3]
-        });else if (param === 'tags') {
-          data.tags = match.input && match[3] ? match.input.split(':')[1].split(',').map(function (t) {
-            return t.trim();
-          }) : [match[2]];
-        } else if (param === 'meta.cache') {
-          var _line$split$pop;
-          data.cache = (_line$split$pop = line.split(':').pop()) === null || _line$split$pop === void 0 ? void 0 : _line$split$pop.trim();
-        } else if (param === 'meta.cache.invalidateOn') {
-          var _line$split$pop2;
-          data.invalidateOn = (_line$split$pop2 = line.split(':').pop()) === null || _line$split$pop2 === void 0 ? void 0 : _line$split$pop2.trim();
-        } else if (param === 'meta.invalidateOn') {
-          data.isInvalidateOnWithoutCache = true;
-        }
+        });
+      } else if (param === 'output') data.outputs.push({
+        type: match[2],
+        name: match[3]
+      });else if (param === 'tags') {
+        data.tags = match.input && match[3] ? match.input.split(':')[1].split(',').map(t => t.trim()) : [match[2]];
+      } else if (param === 'meta.cache') {
+        data.cache = line.split(':').pop()?.trim();
+      } else if (param === 'meta.cache.invalidateOn') {
+        data.invalidateOn = line.split(':').pop()?.trim();
+      } else if (param === 'meta.invalidateOn') {
+        data.isInvalidateOnWithoutCache = true;
       }
-      if (isHeader) {
-        var nm = line.match(utils.nameRegex);
-        if (nm && !match) {
-          data.name = data.name || nm[1];
-          funcData.push(data);
-          data = {
-            name: '',
-            inputs: [],
-            outputs: []
-          };
-          isHeader = false;
-        }
+    }
+    if (isHeader) {
+      const nm = line.match(utils.nameRegex);
+      if (nm && !match) {
+        data.name = data.name || nm[1];
+        funcData.push(data);
+        data = {
+          name: '',
+          inputs: [],
+          outputs: []
+        };
+        isHeader = false;
       }
     }
-  } catch (err) {
-    _iterator11.e(err);
-  } finally {
-    _iterator11.f();
   }
   return funcData;
 }