datagrok-tools 6.4.1 → 6.4.3

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # 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.
+
 ## 6.4.1 (2026-06-18)
 
 * Fixed `grok` failing with `Cannot find module './commands/build'` — the `.npmignore` `build.js` rule was unanchored and excluded the compiled `bin/commands/build.js` from the published package; anchored it to `/build.js`.

package/bin/commands/publish.js

@@ -95,6 +95,30 @@ function localImageExists(fullName, checkPlatform = true) {
     return false;
   }
 }
+function imageExistsInRegistry(ref) {
+  try {
+    execSync(`docker manifest inspect ${ref}`, {
+      stdio: ['pipe', 'pipe', 'pipe']
+    });
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+// Looks for an already-published image in the configured registry and on Docker Hub,
+// without pulling it. Prefers the content-hashed release tag (exact dockerfile match),
+// then the plain version tag. Returns the canonical `datagrok/<name>:<tag>` reference
+// for image.json, or null when nothing usable is published.
+function resolveRegistryImage(imageName, registryTag, versionTag, registry) {
+  const tags = registryTag === versionTag ? [versionTag] : [registryTag, versionTag];
+  for (const tag of tags) {
+    const canonical = `datagrok/${imageName}:${tag}`;
+    if (registry && imageExistsInRegistry(`${registry}/${canonical}`)) return canonical;
+    if (imageExistsInRegistry(canonical)) return canonical;
+  }
+  return null;
+}
 function dockerLogin(registry, devKey) {
   try {
     dockerCommand(`login ${registry} -u any -p ${devKey}`);
@@ -333,24 +357,37 @@ async function processDockerImages(packageName, version, registry, devKey, host,
             requestedVersion: registryTag
           };
           if (!result || result.fallback) color.warn(`Build failed. Falling back to ${fallback.image} (hash mismatch)`);
-        } else if (skipDockerRebuild) {
-          color.warn(`No fallback available. Skipping docker build (--skip-docker-rebuild).`);
-          result = {
-            image: null,
-            fallback: true,
-            requestedVersion: registryTag
-          };
         } else {
-          // No fallback and no local image — must build
-          color.warn(`No fallback available. Building ${img.fullLocalName}...`);
-          const built = buildAndPush();
-          if (built) result = built;else {
+          // The server has no compatible record, but the image may already be
+          // published in the configured registry / Docker Hub (e.g. pushed by an
+          // earlier CI run). Use it directly rather than failing or rebuilding.
+          const registryImage = resolveRegistryImage(img.imageName, registryTag, img.imageTag, registry);
+          if (registryImage) {
+            result = {
+              image: registryImage,
+              fallback: true,
+              requestedVersion: registryTag
+            };
+            color.success(`Falling back to registry image ${registryImage}`);
+          } else if (skipDockerRebuild) {
+            color.warn(`No fallback available. Skipping docker build (--skip-docker-rebuild).`);
             result = {
               image: null,
               fallback: true,
               requestedVersion: registryTag
             };
-            color.error(`Failed to build ${img.fullLocalName}. No container will be available.`);
+          } else {
+            // No fallback and no local image — must build
+            color.warn(`No fallback available. Building ${img.fullLocalName}...`);
+            const built = buildAndPush();
+            if (built) result = built;else {
+              result = {
+                image: null,
+                fallback: true,
+                requestedVersion: registryTag
+              };
+              color.error(`Failed to build ${img.fullLocalName}. No container will be available.`);
+            }
           }
         }
       }

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.1",
+  "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;