datagrok-tools 6.4.2 → 6.4.4

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Datagrok-tools changelog
 
+## 6.4.4 (2026-06-22)
+
+* `grok publish` — fixed every publish failing with a silent `exit 1` after a successful upload: a stray `fs.unlinkSync('zip')` threw `ENOENT` (the archive is streamed in-memory, no `zip` file is ever written), and the surrounding `catch` only logged under `--verbose`. The errant unlink is removed and publish errors are now always surfaced.
+
+## 6.4.3 (2026-06-22)
+
+* `grok api` — numeric IVP model inputs are now generated with `nullable: false`, so an emptied input field fails form validation instead of running the solver with a null.
+
 ## 6.4.2 (2026-06-18)
 
 * `grok publish` — registry-aware Docker fallback: when a package's image isn't built locally and the target server has no compatible record, `grok publish` now checks the configured registry and Docker Hub (`docker manifest inspect`) for the expected `datagrok/<name>:<version>` (and content-hashed) tag and uses it, instead of reporting "No fallback available" and failing. Fixes dependency publishes (e.g. Bio → @datagrok/chem) on CI runners where the image exists in the registry but not locally.

package/bin/commands/publish.js

@@ -457,7 +457,7 @@ async function processPackage(debug, rebuild, host, devKey, packageName, dropDb,
     }
     if (debug) timestamps = checkData;
   } catch (error) {
-    if (utils.isConnectivityError(error)) color.error(`Server is possibly offline: ${host}`);
+    color.error(utils.isConnectivityError(error) ? `Server is possibly offline: ${host}` : `Failed to validate package on ${host}: ${error?.message ?? error}`);
     if (color.isVerbose()) console.error(error);
     return 1;
   }
@@ -582,7 +582,6 @@ async function processPackage(debug, rebuild, host, devKey, packageName, dropDb,
       body: zipBuffer
     });
     const log = JSON.parse(await body.text());
-    _fs.default.unlinkSync('zip');
     if (log != undefined) {
       if (log['#type'] === 'ApiError') {
         color.error(log['message']);
@@ -595,7 +594,7 @@ async function processPackage(debug, rebuild, host, devKey, packageName, dropDb,
       }
     }
   } catch (error) {
-    if (utils.isConnectivityError(error)) color.error(`Server is possibly offline: ${url}`);
+    color.error(utils.isConnectivityError(error) ? `Server is possibly offline: ${url}` : `Publish failed: ${error?.message ?? error}`);
     if (color.isVerbose()) console.error(error);
     return 1;
   }

package/bin/utils/playwright-runner.js

@@ -157,7 +157,13 @@ async function runPlaywrightTests(pkgDir, testDir, args, hostKey) {
     const candidate = _path.default.join(testDir, args.category);
     if (_fs.default.existsSync(candidate)) testDirFinal = candidate;
   }
-  if (testDirFinal !== testDir) cliArgs.push(testDirFinal);
+  if (testDirFinal !== testDir)
+    // Normalize Windows backslashes to forward slashes — Playwright CLI on Windows
+    // treats backslash positional args as regex patterns and emits "No tests found"
+    // when the resolved path contains backslashes. Linux path.join already emits
+    // forward slashes; this no-ops there. Per
+    // .claude/plan/investigator-handback-autocomplete-no-tests-2026-05-24.md.
+    cliArgs.push(testDirFinal.replace(/\\/g, '/'));
   const env = {
     ...process.env,
     DATAGROK_URL: webUrl,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "datagrok-tools",
-  "version": "6.4.2",
+  "version": "6.4.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

@@ -51,9 +51,19 @@ function preparseIvpModel(parser, text) {
   // Input names use the script-form names (arg bounds/step `_`-prefixed, loop count `_count`) so the
   // run path, the diff-grok fitting/SA pipeline, and `propagateChoice` lookups all agree on one set
   // of names. `scriptKey` mirrors `name`; it is kept only for the value-forwarding map below.
+  // Numeric IVP inputs are always required: mark them `nullable: false` so an emptied field fails
+  // form validation instead of running with a null (which throws in the solver).
+  const withNullable = (annot) => {
+    const a = (annot ?? '').trim();
+    if (!a.startsWith('{')) return '{nullable: false}';     // no options block
+    if (/[{;\s]nullable\s*:/.test(a)) return a;              // already declares nullable (any value)
+    if (/^\{\s*\}$/.test(a)) return '{nullable: false}';     // empty {}
+    return `{nullable: false; ${a.slice(1).trimStart()}`;    // everything after '{' untouched
+  };
+
   const mk = (type, name, scriptKey, input) =>
     ({tsType: 'number', name, scriptKey,
-      annotation: `//input: ${type} ${name} = ${input.value} ${input.annot ?? ''}`.trim()});
+      annotation: `//input: ${type} ${name} = ${input.value} ${withNullable(input.annot)}`.trim()});
 
   const inputs = [];
   const a = ivp.arg.name;