datagrok-tools 6.7.2 → 6.7.4

package/.turbo/turbo-build.log

@@ -1,5 +1,5 @@
 
-> datagrok-tools@6.7.2 build /home/runner/work/public/public/tools
+> datagrok-tools@6.7.4 build /home/runner/work/public/public/tools
 > node build.js
 
 Successfully compiled 51 files with Babel.

package/CHANGELOG.md

@@ -1,6 +1,15 @@
 # Datagrok-tools changelog
 
-## 6.7.2 (WIP)
+## 6.7.4 (2026-09-17)
+
+* `grok create`, `grok publish`, `grok test` and `grok run` work inside the pnpm workspace
+
+## 6.7.3 (2026-09-17)
+
+* The function-metadata plugin emits decorator roles in camel case again (`fileHandler`, `app`, `viewer`, ...); since 6.7.1 it capitalised them, which broke JS code comparing `options.role` with `DG.FUNC_TYPES`.
+* `grok check` accepts `## v.next` as the top changelog heading, the section the repository keeps unreleased changes in.
+
+## 6.7.2 (2026-09-16)
 
 * `grok publish` marks a bundled package for servers older than 1.28.0, which detect one only by a `webpack.config.js` in the archive; the marker is generated at publish time and never written to the package folder.
 
@@ -11,6 +20,7 @@
 * `grok config add` — `--key` is now optional: a server reached with a keypair has no developer key to record.
 * Against a server older than 1.28 — which has no keypair endpoints and refuses their paths before routing — the CLI names the version needed instead of reporting a bare 401, and falls back to the developer key when one is still configured for that server.
 * `grok login` reports its progress step by step (spinner on a terminal, one line per step in a log), including while it waits for the browser approval.
+* `grok check` — warns when a package pins its own version of a platform-served library (`build-config/platform-deps.json`) instead of `catalog:`; the `common/*.js` shared-library map is read from the same manifest.
 * `grok s push/migrate` — a migrated entity is stamped in its `metaParams` with `migrated_from` (the source stand URL) and `migrated_on`, so on the target it is clear the entity came from another stand. Written wherever the entity already carries `metaParams` (as `sync_id` is) and stripped before the idempotency comparison, so a re-push of an unchanged entity still reads as identical.
 
 ## 6.6.0 (2026-09-13)

package/bin/commands/check.js

@@ -21,7 +21,7 @@ var utils = _interopRequireWildcard(require("../utils/utils"));
 var color = _interopRequireWildcard(require("../utils/color-utils"));
 var _const = require("../utils/const");
 function _interopRequireWildcard(e, t) { if ("function" == typeof WeakMap) var r = new WeakMap(), n = new WeakMap(); return (_interopRequireWildcard = function (e, t) { if (!t && e && e.__esModule) return e; var o, i, f = { __proto__: null, default: e }; if (null === e || "object" != typeof e && "function" != typeof e) return f; if (o = t ? n : r) { if (o.has(e)) return o.get(e); o.set(e, f); } for (const t in e) "default" !== t && {}.hasOwnProperty.call(e, t) && ((i = (o = Object.defineProperty) && Object.getOwnPropertyDescriptor(e, t)) && (i.get || i.set) ? o(f, t, i) : f[t] = e[t]); return f; })(e, t); }
-const warns = ['Latest package version', 'Datagrok API version should contain'];
+const warns = ['Latest package version', 'Datagrok API version should contain', 'is served by the platform'];
 const forbiddenNames = ['function', 'class', 'export'];
 /** Static imports of these libraries significantly increase the main bundle size. */
 const HEAVY_IMPORT_RULES = [
@@ -375,26 +375,29 @@ function checkFuncSignatures(packagePath, files) {
 
   return [warnings, errors];
 }
-const sharedLibExternals = {
-  'common/html2canvas.min.js': {
-    'exceljs': 'ExcelJS'
-  },
-  'common/exceljs.min.js': {
-    'html2canvas': 'html2canvas'
-  },
-  'common/ngl_viewer/ngl.js': {
-    'NGL': 'NGL'
-  },
-  'common/openchemlib-full.js': {
-    'openchemlib/full': 'OCL'
-  },
-  'common/codemirror/codemirror.js': {
-    'codemirror': 'CodeMirror'
-  },
-  'common/vue.js': {
-    'vue': 'Vue'
+
+/** Libraries the platform serves at runtime; versions live in build-config/platform-deps.json. */
+function platformDeps(packagePath) {
+  try {
+    return require(require.resolve('@datagrok/build-config/platform-deps.json', {
+      paths: [packagePath]
+    }));
+  } catch {
+    return {};
   }
-};
+}
+
+/** For each `common/*.js` source: the import specifiers a bundled package must externalize, with their globals. */
+function sharedLibExternals(packagePath) {
+  const result = {};
+  for (const [name, d] of Object.entries(platformDeps(packagePath))) {
+    if (d.source) result[d.source] = {
+      [name]: d.global,
+      ...d.imports
+    };
+  }
+  return result;
+}
 function checkPackageFile(packagePath, json, options) {
   const warnings = [];
   const isPublicPackage = _path.default.basename(_path.default.dirname(packagePath)) === 'packages' && _path.default.basename(_path.default.dirname(_path.default.dirname(packagePath))) === 'public';
@@ -416,6 +419,14 @@ function checkPackageFile(packagePath, json, options) {
   // if (dt && dt !== 'latest')
   //   warnings.push('File "package.json": "datagrok-tools" dependency must be "latest" version.');
 
+  const platform = platformDeps(packagePath);
+  for (const [name, spec] of Object.entries({
+    ...json.dependencies,
+    ...json.devDependencies
+  })) {
+    if (name in platform && !spec.startsWith('catalog:')) warnings.push(`File "package.json": "${name}" is served by the platform (${platform[name].version}, ` + `see build-config/platform-deps.json); "${spec}" overrides it. Use "catalog:" unless the override is intended.`);
+  }
+  const shared = sharedLibExternals(packagePath);
   if (Array.isArray(json.sources) && json.sources.length > 0) {
     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}`);
@@ -423,14 +434,15 @@ function checkPackageFile(packagePath, json, options) {
       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)) {
+            if (source in shared) {
+              const expected = Object.entries(shared[source]);
+              if (!expected.some(([lib, name]) => options.externals[lib] === name)) {
+                const [lib, name] = expected[0];
                 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` : ''));
+            warnings.push('Webpack config parsing: External modules not found.\n' + `Consider adding source "${source}" to webpack externals` + (source in shared ? ':\n' + `'${Object.keys(shared[source])[0]}': '${Object.values(shared[source])[0]}'\n` : ''));
           }
         }
         continue;
@@ -475,9 +487,9 @@ function checkChangelog(packagePath, json) {
   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)\)$/;
+  regex = /^## (v\.next|\d+\.\d+\.\d+ \((\d{4}-\d{2}-\d{2}|WIP)\))$/;
   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`);
+    if (!regex.test(h)) warnings.push(`CHANGELOG: '${h}' does not match the h2 format, expected: ## <version> (<yyyy-mm-dd> | WIP) or ## v.next\n`);
   }
   regex = /^## (\d+\.\d+\.\d+)/;
   const v1 = h2[0].match(regex)?.[1];

package/bin/commands/create.js

@@ -21,12 +21,6 @@ const curFolder = _path.default.basename(curDir);
 const grokDir = _path.default.join(_os.default.homedir(), '.grok');
 const confPath = _path.default.join(grokDir, 'config.yaml');
 const templateDir = _path.default.join(_path.default.dirname(_path.default.dirname(__dirname)), 'package-template');
-function isPnpmWorkspace(dir) {
-  for (let d = _path.default.resolve(dir);; d = _path.default.dirname(d)) {
-    if (_fs.default.existsSync(_path.default.join(d, 'pnpm-workspace.yaml'))) return true;
-    if (_path.default.dirname(d) === d) return false;
-  }
-}
 const confTemplateDir = _path.default.join(_path.default.dirname(_path.default.dirname(__dirname)), 'config-template.yaml');
 const confTemplate = _jsYaml.default.load(_fs.default.readFileSync(confTemplateDir, {
   encoding: 'utf-8'
@@ -62,7 +56,7 @@ function createDirectoryContents(name, friendlyName, config, templateDir, packag
         _package.devDependencies = _package.devDependencies ?? {};
         // Outside the public/ pnpm workspace the template's workspace:/catalog: specifiers mean
         // nothing to npm: pin published ranges and bring the toolchain in as a devDependency.
-        if (!isPnpmWorkspace(packageDir)) {
+        if (!utils.isPnpmWorkspace(packageDir)) {
           const published = {
             'datagrok-api': `^${require('datagrok-api/package.json').version}`,
             '@datagrok-libraries/test': '^1.4.0',
@@ -216,8 +210,9 @@ function create(args) {
     color.success('Successfully created package ' + name);
     console.log(_entHelpers.help.package(ts));
     console.log(`\nThe package has the following dependencies:\n${dependencies.join(' ')}\n`);
-    console.log('Running `npm install` to get the required dependencies...\n');
-    (0, _child_process.exec)('npm install', {
+    const install = utils.isPnpmWorkspace(packageDir) ? 'pnpm install' : 'npm install';
+    console.log(`Running \`${install}\` to get the required dependencies...\n`);
+    (0, _child_process.exec)(install, {
       cwd: packageDir
     }, (err, stdout, stderr) => {
       if (err) throw err;else console.log(stderr, stdout);

package/bin/commands/help.js

@@ -210,7 +210,7 @@ Options:
 
 --all                  Publish all available packages (run in packages directory)
 --refresh              Publish all available already loaded packages (run in packages directory)
---link                 Link the package to local packages
+--link                 Link the package to local packages (no effect in a pnpm workspace checkout)
 --build                Builds the package (the default; kept for compatibility)
 --skip-build           Upload the existing dist/ without rebuilding
 --release              Publish package as release version
@@ -255,7 +255,7 @@ Options:
 --skip-playwright   Skip the Playwright pass; only run Puppeteer/DG.Test
 --skip-node         Skip the Node (browserless) pass; run all tests in the browser
 --node-only         Run only tests annotated {node: true} headless under Node, no browser
---link  	        Link the package to local utils
+--link  	        Link the package to local utils (no effect in a pnpm workspace checkout)
 --record            Records the test execution process in mp4 format
 --platform          Runs only platform tests (applicable for ApiTests package only)
 --core              Runs package & auto tests & core tests (core tests run only from DevTools package)

package/bin/commands/link.js

@@ -30,18 +30,12 @@ while (_path.default.dirname(dirStep) !== dirStep) {
   }
   dirStep = _path.default.dirname(dirStep);
 }
-function isPnpmWorkspace(dir) {
-  for (let d = _path.default.resolve(dir);; d = _path.default.dirname(d)) {
-    if (_fs.default.existsSync(_path.default.join(d, 'pnpm-workspace.yaml'))) return true;
-    if (_path.default.dirname(d) === d) return false;
-  }
-}
 let verbose = false;
 let pathMode = false;
 let devMode = false;
 let unlink = false;
 async function link(args) {
-  if (isPnpmWorkspace(curDir)) {
+  if ((0, _utils.isPnpmWorkspace)(curDir)) {
     console.log('This checkout is a pnpm workspace: in-repo dependencies are linked by `pnpm install` ' + '(workspace:^), and `grok link` is not needed. Run `pnpm install` at the repository root.');
     return true;
   }
@@ -49,7 +43,7 @@ async function link(args) {
   devMode = args.dev ?? false;
   pathMode = args.path ?? false;
   unlink = args.unlink ?? false;
-  let collectedPackages = collectAvaliablePackages(args['repo-only']);
+  const collectedPackages = collectAvaliablePackages(args['repo-only']);
   packagesInRepo = collectedPackages.packagesInRepo;
   packagesOutOfRepo = collectedPackages.packagesOutOfRepo;
   localPackageDependencies = [];
@@ -83,16 +77,13 @@ function collectPackagesData(packagePath = curDir) {
 }
 function collectPacakgeDataFromJsonObject(object) {
   let result = [];
-  for (let dependencyName of Object.keys(object ?? {})) {
+  for (const dependencyName of Object.keys(object ?? {})) {
     if (packagesInRepo[dependencyName]) result = result.concat(parsePackageDependencies(dependencyName, _path.default.dirname(packagesInRepo[dependencyName].path)));else if (packagesOutOfRepo[dependencyName]) result = result.concat(parsePackageDependencies(dependencyName, _path.default.dirname(packagesOutOfRepo[dependencyName].path)));
   }
   return result;
 }
-function toCamelCase(input) {
-  return input.split('-').map(word => word.charAt(0).toUpperCase() + word.slice(1)).join('');
-}
 function parsePackageDependencies(dependencyName, pathToLink) {
-  let result = [];
+  const result = [];
   if (!packagesToLink.has(dependencyName)) {
     packagesToLink.add(dependencyName);
     localPackageDependencies.push(new PackageData(pathToLink));
@@ -101,8 +92,8 @@ function parsePackageDependencies(dependencyName, pathToLink) {
   return result;
 }
 function collectAvaliablePackages(noOutLink = false) {
-  let repositoryPackages = collectAvaliablePackagesPathesFromDir(repositoryDir);
-  let commonPakcages = collectAvaliablePackagesPathesFromDir(containerDir, repositoryDir);
+  const repositoryPackages = collectAvaliablePackagesPathesFromDir(repositoryDir);
+  const commonPakcages = collectAvaliablePackagesPathesFromDir(containerDir, repositoryDir);
   const packagesInRepoBaseInfo = parsePackages(repositoryPackages);
   const packagesOutOfRepoBaseInfo = noOutLink ? [] : parsePackages(commonPakcages);
   const packagesInRepo = Object.fromEntries(packagesInRepoBaseInfo.map(({
@@ -127,8 +118,8 @@ function collectAvaliablePackagesPathesFromDir(dir, dirToExclude = '') {
   return res;
 }
 function parsePackages(packagPathes) {
-  let res = packagPathes.map(e => {
-    let packageData = {};
+  const res = packagPathes.map(e => {
+    const packageData = {};
     if (_fs.default.existsSync(e)) {
       packageData.path = e;
       const packageJson = JSON.parse(_fs.default.readFileSync(e, 'utf8'));
@@ -142,7 +133,7 @@ function parsePackages(packagPathes) {
 async function unlinkPackages() {
   const packages = [...localPackageDependencies];
   if (currentPackage) packages.push(currentPackage);
-  for (let packageData of packages) {
+  for (const packageData of packages) {
     if (excludedPackages.includes(packageData.name)) continue;
     if (verbose) console.log(`Package ${packageData.name} is unlinking`);
     const packageJsonPath = _path.default.join(packageData.packagePath, 'package.json');
@@ -154,29 +145,27 @@ async function unlinkPackages() {
   }
 }
 function updateDependenciesToVersion(packageData, dependecyNode) {
-  for (let dependency of packageData.dependencies) {
+  for (const dependency of packageData.dependencies) {
     if (excludedPackages.includes(dependency)) continue;
     if (dependecyNode[dependency] && !versionDependencyRegex.test(dependecyNode[dependency])) {
       const packageToLink = (localPackageDependencies.filter(e => e.name === dependency) ?? [])[0];
-      if (packageToLink) {
-        dependecyNode[dependency] = `^${packagesInRepo[dependency]?.version ?? packagesOutOfRepo[dependency]?.version}`;
-      }
+      if (packageToLink) dependecyNode[dependency] = `^${packagesInRepo[dependency]?.version ?? packagesOutOfRepo[dependency]?.version}`;
     }
   }
   return dependecyNode;
 }
 async function linkPackages() {
   let anyChanges = true;
-  for (let element of excludedPackages) packagesToLink.delete(element);
+  for (const element of excludedPackages) packagesToLink.delete(element);
   if (verbose) {
     console.log('Packages to link:');
     console.log(localPackageDependencies);
   }
   while (anyChanges && packagesToLink.size > 0) {
     anyChanges = false;
-    let mapElements = localPackageDependencies.filter(x => x.dependencies.every(i => !packagesToLink.has(i)) && packagesToLink.has(x.name) && !excludedPackages.includes(x.name));
+    const mapElements = localPackageDependencies.filter(x => x.dependencies.every(i => !packagesToLink.has(i)) && packagesToLink.has(x.name) && !excludedPackages.includes(x.name));
     if (mapElements.length === 0) break;
-    for (let element of mapElements) {
+    for (const element of mapElements) {
       if (verbose) console.log(`Package ${element.name} is linking`);
       if (pathMode) await linkPathMode(element);
       await (0, _utils.runScript)(`npm install`, element.packagePath);
@@ -187,7 +176,7 @@ async function linkPackages() {
     }
     if (anyChanges === false) throw new Error(`There is loop with next packages: ${JSON.stringify(Array.from(packagesToLink)).toString()}`);
   }
-  let names = localPackageDependencies.map(x => x.name);
+  const names = localPackageDependencies.map(x => x.name);
   if (currentPackage && pathMode) await linkPathMode(currentPackage);
   await (0, _utils.runScript)(`npm install`, curDir);
   if (pathMode === false) await (0, _utils.runScript)(`npm link ${names.join(' ')}`, curDir);
@@ -208,7 +197,7 @@ async function linkPathMode(packageData) {
 function updateDependenciesToLocal(packageData, dependecyNode) {
   const backRouteForCont = `../`.repeat(packageData.packagePath.replace(containerDir, '').split(_path.default.sep).length - 1);
   const backRouteForRepo = `../`.repeat(packageData.packagePath.replace(repositoryDir, '').split(_path.default.sep).length - 1);
-  for (let dependency of packageData.dependencies) {
+  for (const dependency of packageData.dependencies) {
     if (excludedPackages.includes(dependency)) continue;
     if (dependecyNode[dependency]) {
       const packageToLink = (localPackageDependencies.filter(e => e.name === dependency) ?? [])[0];
@@ -222,7 +211,7 @@ function updateDependenciesToLocal(packageData, dependecyNode) {
 class PackageData {
   dependencies = [];
   constructor(packagePath) {
-    let packageJsonData = collectPackagesData(packagePath);
+    const packageJsonData = collectPackagesData(packagePath);
     this.name = packageJsonData.name;
     this.packagePath = packagePath;
     this.dependencies = packageJsonData.dependencies;

package/bin/commands/publish.js

@@ -374,7 +374,9 @@ async function processDockerImages(packageName, version, registry, devKey, host,
         result = pushImage(img.imageName, registryTag, registry);
         if (!result) {
           const fallback = await fallbackImage(img, host, devKey, registry, version, contentHash);
-          if (fallback.serverError) color.error(`Cannot resolve fallback: ${fallback.serverError}`);else if (fallback.image) color.warn(`Falling back to ${fallback.image}` + `${fallback.hashMatch === false ? ' (hash mismatch — container will run older code)' : ''}`);else color.error(`No published image available for ${img.imageName}. No container will be available.`);
+          if (fallback.serverError) color.error(`Cannot resolve fallback: ${fallback.serverError}`);else if (fallback.image) {
+            color.warn(`Falling back to ${fallback.image}` + `${fallback.hashMatch === false ? ' (hash mismatch — container will run older code)' : ''}`);
+          } else color.error(`No published image available for ${img.imageName}. No container will be available.`);
           result = fallback;
         }
       } else {
@@ -700,28 +702,24 @@ async function publish(args) {
     }
     color.log('Loading packages:');
     await (0, _testUtils.loadPackages)(curDir, packagesToLoad.join(' '), host, false, false, args.link, args.release);
-  } else {
-    if (args.link) {
-      color.log('Linking');
-      await utils.runScript(`npm install`, curDir);
-      await utils.runScript(`grok link`, curDir);
-      await utils.runScript(`npm run build`, curDir);
-    }
-    return await publishPackage(args);
-  }
+  } else return await publishPackage(args);
   return true;
 }
+
+// --link means nothing inside the pnpm workspace: the root install already linked every in-repo dependency
+async function buildPackage(args) {
+  const workspace = utils.isPnpmWorkspace(curDir);
+  const link = args.link && !workspace;
+  if (!link && (args['skip-build'] || args.rebuild || !_fs.default.existsSync(_path.default.join(curDir, 'src')))) return;
+  color.log(link ? 'Linking and building' : 'Building');
+  if (link || !workspace && !_fs.default.existsSync(_path.default.join(curDir, 'node_modules'))) await utils.runScript('npm install', curDir);
+  if (link) await utils.runScript(`${utils.grokCommand} link`, curDir);
+  await utils.runScript(workspace ? 'pnpm run build' : 'npm run build', curDir);
+}
 async function publishPackage(args) {
   const nArgs = args['_'].length;
-
-  // A bundled package is built locally before upload unless --skip-build. Inside the pnpm
-  // workspace the install already happened at the root; standalone packages install first.
-  if (!args.link && !args['skip-build'] && !args.rebuild && _fs.default.existsSync(_path.default.join(curDir, 'src'))) {
-    color.log('Building');
-    const workspace = utils.isPnpmWorkspace(curDir);
-    if (!workspace && !_fs.default.existsSync(_path.default.join(curDir, 'node_modules'))) await utils.runScript('npm install', curDir, false);
-    await utils.runScript(workspace ? 'pnpm run build' : 'npm run build', curDir, false);
-  }
+  await buildPackage(args);
+  if (args['skip-build'] && !args.rebuild && _fs.default.existsSync(_path.default.join(curDir, 'src')) && !_fs.default.existsSync(_path.default.join(curDir, 'dist', 'package.js'))) return color.error('dist/package.js not found: build the package first, or run without --skip-build');
   if (args.debug && args.release) {
     color.error('Incompatible options: --debug and --release');
     return false;
@@ -755,7 +753,9 @@ async function publishPackage(args) {
 
   // Update the developer key
   if (args.key) key = args.key;
-  if (!key && !keypair.keypairFor(url)) return color.warn(`No credentials for ${url}. Run \`grok login ${host}\`, ` + 'or pass a developer key with `--key` (deprecated).');
+  if (!key && !keypair.keypairFor(url)) {
+    return color.warn(`No credentials for ${url}. Run \`grok login ${host}\`, ` + 'or pass a developer key with `--key` (deprecated).');
+  }
 
   // Get the package name
   if (!_fs.default.existsSync(packDir)) return color.error('`package.json` doesn\'t exist');

package/bin/commands/run.js

@@ -39,7 +39,8 @@ function openBrowser(url) {
 }
 const MISSING_MODULE_PATTERNS = ['cannot find module', 'module not found', 'can\'t resolve'];
 async function buildPackage(dir) {
-  const buildCmd = 'npm run build -- --env incremental';
+  const workspace = utils.isPnpmWorkspace(dir);
+  const buildCmd = workspace ? 'pnpm run build' : 'npm run build -- --env incremental';
   const packageJson = JSON.parse(_fs.default.readFileSync(_path.default.join(dir, 'package.json'), 'utf-8'));
   const name = packageJson.friendlyName || packageJson.name;
   console.log(`Building ${name}...`);
@@ -50,9 +51,10 @@ async function buildPackage(dir) {
   } catch (error) {
     const msg = (error?.message ?? '').toLowerCase();
     if (MISSING_MODULE_PATTERNS.some(p => msg.includes(p))) {
-      color.warn('Missing modules detected, running npm install...');
+      const install = workspace ? 'pnpm install --frozen-lockfile' : 'npm install';
+      color.warn(`Missing modules detected, running ${install}...`);
       try {
-        await utils.runScript('npm install', dir, color.isVerbose());
+        await utils.runScript(install, dir, color.isVerbose());
         await utils.runScript(buildCmd, dir, color.isVerbose());
         color.success(`Successfully built ${name}`);
         return true;
@@ -70,7 +72,7 @@ async function buildPackage(dir) {
 async function run(args) {
   color.setVerbose(args.verbose || false);
 
-  // Step 1: Build (skip npm install to preserve npm link; retry with npm install only if needed)
+  // Step 1: Build (skip the install to preserve npm link; retry with an install only if needed)
   const built = await buildPackage(process.cwd());
   if (!built) return false;
 

package/bin/utils/test-utils.js

@@ -251,10 +251,8 @@ async function loadPackage(packageDir, dirName, hostString, skipPublish, skipBui
   if (skipPublish != true) {
     process.stdout.write(`Building and publishing ${dirName}...`);
     try {
-      await utils.runScript(`npm install`, packageDir);
-      if (linkPackage) await utils.runScript(`grok link`, packageDir);
-      if (skipBuild != true) await utils.runScript(`npm run build`, packageDir);
-      await utils.runScript(`grok publish ${hostString}${release ? ' --release' : ''}`, packageDir);
+      const flags = `${release ? ' --release' : ''}${linkPackage ? ' --link' : ''}${skipBuild ? ' --skip-build' : ''}`;
+      await utils.runScript(`${utils.grokCommand} publish ${hostString}${flags}`, packageDir);
     } catch (e) {
       process.stdout.write(' FAILED\n');
       throw e;

package/bin/utils/utils.js

@@ -19,7 +19,7 @@ exports.getScriptDescription = getScriptDescription;
 exports.getScriptInputs = getScriptInputs;
 exports.getScriptName = getScriptName;
 exports.getScriptOutputType = getScriptOutputType;
-exports.headerTags = void 0;
+exports.headerTags = exports.grokCommand = void 0;
 exports.isConnectivityError = isConnectivityError;
 exports.isEmpty = isEmpty;
 exports.isPackageDir = isPackageDir;
@@ -310,6 +310,7 @@ function isPnpmWorkspace(dir) {
     if (_path.default.dirname(d) === d) return false;
   }
 }
+const grokCommand = exports.grokCommand = `"${process.execPath}" "${_path.default.join(__dirname, '..', 'grok.js')}"`;
 async function runScript(script, path, verbose = false) {
   try {
     const {

package/package.json

@@ -4,7 +4,7 @@
     "type": "git",
     "url": "https://github.com/datagrok-ai/public.git"
   },
-  "version": "6.7.2",
+  "version": "6.7.4",
   "description": "Utility to upload and publish packages to Datagrok",
   "homepage": "https://github.com/datagrok-ai/public/tree/master/tools#readme",
   "dependencies": {

package/plugins/func-gen-plugin.js

@@ -19,7 +19,7 @@ const {
 } = require('../bin/utils/func-generation');
 
 // Not imported from commands/migrate: that module loads ts-morph (~1.5 s) at require time.
-const toCamelCase = (s) => s.split('-').map((w) => w.charAt(0).toUpperCase() + w.slice(1)).join('');
+const toCamelCase = (s) => s.replace(/[-_ ]+(\w)/g, (_, c) => c.toUpperCase()).replace(/^[A-Z]/, (c) => c.toLowerCase());
 
 let _api;
 const api = (...a) => (_api ??= require('../bin/commands/api').api)(...a);