datagrok-tools 6.2.1 → 6.2.3

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Datagrok-tools changelog
 
+## 6.2.3 (2026-05-06)
+
+* `grok test` — added `--skip-puppeteer` flag (symmetric counterpart of `--skip-playwright`). Bypasses `loadPackages()` and the Puppeteer/`DG.Test` runner so Playwright-only test directories (e.g. `public/playwright-public`) can run end-to-end via `grok test --skip-puppeteer` without tripping the Dart/JS package loader.
+
+## 6.2.2 (2026-05-05)
+
+* `grok test` — Playwright runner now writes `test-report-playwright.csv` next to the existing merged `test-report.csv`, so CI can ship Playwright rows to a dedicated Datlas reporting bucket without disturbing the legacy `package` flow.
+
 ## 6.2.1 (2026-05-05)
 
 * Reports: `grok report attach <ticket> <file>` — upload a file as a JIRA issue attachment via REST v2 multipart POST.

package/bin/commands/help.js

@@ -205,6 +205,8 @@ Options:
 --report            Report failed tests to audit, notifies package author (default=false)
 --skip-build        Skip the package build step
 --skip-publish      Skip the package publication step
+--skip-puppeteer    Skip the Puppeteer/DG.Test pass; only run Playwright (for playwright-only test directories)
+--skip-playwright   Skip the Playwright pass; only run Puppeteer/DG.Test
 --link  	        Link the package to local utils
 --record            Records the test execution process in mp4 format
 --platform          Runs only platform tests (applicable for ApiTests package only)

package/bin/commands/test.js

@@ -26,7 +26,7 @@ function _interopRequireWildcard(e, t) { if ("function" == typeof WeakMap) var r
 const execAsync = (0, _util.promisify)(_child_process.exec);
 const execFileAsync = (0, _util.promisify)(_child_process.execFile);
 const testInvocationTimeout = 3600000;
-const availableCommandOptions = ['host', 'package', 'csv', 'gui', 'catchUnhandled', 'platform', 'core', 'report', 'skip-build', 'skip-publish', 'path', 'record', 'verbose', 'benchmark', 'category', 'test', 'stress-test', 'link', 'tag', 'ci-cd', 'debug', 'no-retry', 'dartium', 'f', 'params', 'logfailed', 'skip-playwright'];
+const availableCommandOptions = ['host', 'package', 'csv', 'gui', 'catchUnhandled', 'platform', 'core', 'report', 'skip-build', 'skip-publish', 'path', 'record', 'verbose', 'benchmark', 'category', 'test', 'stress-test', 'link', 'tag', 'ci-cd', 'debug', 'no-retry', 'dartium', 'f', 'params', 'logfailed', 'skip-playwright', 'skip-puppeteer'];
 const curDir = process.cwd();
 
 /** Expands camelCase to space-separated lowercase: "dataManipulation" → "data manipulation" */
@@ -216,7 +216,7 @@ async function test(args) {
   // if (args.core && packageName !== 'DevTools')
   //   color.warn('--core flag can only be used in the DevTools package');
 
-  if (!args.package) {
+  if (!args.package && !args['skip-puppeteer']) {
     try {
       await testUtils.loadPackages(packagesDir, packageName, args.host, args['skip-publish'], args['skip-build'], args.link);
     } catch (e) {
@@ -226,24 +226,41 @@ async function test(args) {
   }
   process.env.TARGET_PACKAGE = packageName;
   let res;
-  try {
-    res = await runTesting(args);
-  } catch (e) {
-    // Don't let Puppeteer-side failures (login error, browser crash) skip the
-    // Playwright pass — the two suites have independent auth and runtime paths,
-    // and we want at least one half of the run reported.
-    color.error(`Puppeteer pass failed: ${e?.message || e}`);
+  if (args['skip-puppeteer']) {
+    // Playwright-only mode: skip the Puppeteer browser launch + DG.Test runner.
+    // Used by Playwright-only test directories (e.g. public/playwright-public)
+    // that have a `playwrightTests` field in package.json but no Dart/JS package
+    // tests on the server.
     res = {
-      failed: true,
+      failed: false,
       verbosePassed: '',
       verboseSkipped: '',
-      verboseFailed: `Puppeteer pass failed: ${e?.message || e}\n`,
+      verboseFailed: '',
       passedAmount: 0,
       skippedAmount: 0,
-      failedAmount: 1,
-      csv: '',
-      error: String(e?.message || e)
+      failedAmount: 0,
+      csv: ''
     };
+  } else {
+    try {
+      res = await runTesting(args);
+    } catch (e) {
+      // Don't let Puppeteer-side failures (login error, browser crash) skip the
+      // Playwright pass — the two suites have independent auth and runtime paths,
+      // and we want at least one half of the run reported.
+      color.error(`Puppeteer pass failed: ${e?.message || e}`);
+      res = {
+        failed: true,
+        verbosePassed: '',
+        verboseSkipped: '',
+        verboseFailed: `Puppeteer pass failed: ${e?.message || e}\n`,
+        passedAmount: 0,
+        skippedAmount: 0,
+        failedAmount: 1,
+        csv: '',
+        error: String(e?.message || e)
+      };
+    }
   }
   if (!args['skip-playwright']) {
     const ptDir = playwrightRunner.hasPlaywrightTests(curDir);

package/bin/utils/playwright-runner.js

@@ -246,6 +246,15 @@ async function runPlaywrightTests(pkgDir, testDir, args, hostKey) {
       verboseFailed += `${r.category}: ${r.name} (${r.ms} ms) :  ${r.result}\n`;
     }
   }
+  const csv = rowsToCsv(rows);
+  // Persist a Playwright-only CSV so the pipeline can ship it to the Datlas
+  // 'playwright' bucket, separate from the merged Puppeteer+Playwright
+  // test-report.csv that feeds the legacy 'package' bucket and JUnit.
+  try {
+    _fs.default.writeFileSync(_path.default.join(pkgDir, 'test-report-playwright.csv'), csv, 'utf-8');
+  } catch (e) {
+    color.warn(`Playwright: failed to write test-report-playwright.csv: ${e.message || e}`);
+  }
   return {
     failed: failedAmount > 0,
     passedAmount: passedAmount,
@@ -254,6 +263,6 @@ async function runPlaywrightTests(pkgDir, testDir, args, hostKey) {
     verbosePassed: verbosePassed,
     verboseFailed: verboseFailed,
     verboseSkipped: verboseSkipped,
-    csv: rowsToCsv(rows)
+    csv: csv
   };
 }

package/package.json

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