datagrok-tools 6.4.3 → 6.4.5

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Datagrok-tools changelog
 
+## 6.4.5 (2026-06-23)
+
+* `grok test` — screen recording (`--record`) is now optional: if `page.screencast()` can't start (e.g. `ffmpeg` is missing on the runner, `spawnSync ffmpeg ENOENT`), the run warns and continues instead of failing the whole Puppeteer pass with 0 tests executed.
+
+## 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.

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/test-utils.js

@@ -256,9 +256,14 @@ async function loadTestsList(packages, core = false, record = false) {
       const suffix = process.env.BACKUP_SIZE && process.env.WORKER_ID && process.env.TOTAL_WORKERS ? `_${process.env.BACKUP_SIZE}_${process.env.WORKER_ID}_${process.env.TOTAL_WORKERS}` : '';
       const logsDir = `./load-test-console-output${suffix}.log`;
       const recordDir = `./load-test-record${suffix}.mp4`;
-      recorder = await page.screencast({
-        path: recordDir
-      });
+      try {
+        recorder = await page.screencast({
+          path: recordDir
+        });
+      } catch (e) {
+        recorder = null;
+        color.warn(`Screen recording disabled: ${e?.message || e}`);
+      }
       await page.exposeFunction('addLogsToFile', addLogsToFile);
       _fs.default.writeFileSync(logsDir, ``);
       page.on('console', msg => {
@@ -636,10 +641,17 @@ async function runBrowser(testExecutionData, browserOptions, browsersId, testInv
     const logsDir = `./test-console-output-${currentBrowserNum}.log`;
     const recordDir = `./test-record-${currentBrowserNum}.mp4`;
     if (browserOptions.record && !existingBrowserSession) {
-      // Only set up recording on initial browser creation, not on retry
-      recorder = await page.screencast({
-        path: recordDir
-      });
+      // Only set up recording on initial browser creation, not on retry.
+      // Recording is an optional diagnostic — if it can't start (e.g. ffmpeg is
+      // missing on the runner), warn and keep testing instead of failing the pass.
+      try {
+        recorder = await page.screencast({
+          path: recordDir
+        });
+      } catch (e) {
+        recorder = null;
+        color.warn(`Screen recording disabled: ${e?.message || e}`);
+      }
       await page.exposeFunction('addLogsToFile', addLogsToFile);
       _fs.default.writeFileSync(logsDir, ``);
       page.on('console', msg => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "datagrok-tools",
-  "version": "6.4.3",
+  "version": "6.4.5",
   "description": "Utility to upload and publish packages to Datagrok",
   "homepage": "https://github.com/datagrok-ai/public/tree/master/tools#readme",
   "dependencies": {