datagrok-tools 6.4.2 → 6.4.3

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Datagrok-tools changelog
 
+## 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/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.3",
   "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;