@nocobase/cli-v1 2.1.0-alpha.35 → 2.1.0-alpha.37

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nocobase/cli-v1",
-  "version": "2.1.0-alpha.35",
+  "version": "2.1.0-alpha.37",
   "description": "",
   "license": "Apache-2.0",
   "main": "./src/index.js",
@@ -8,11 +8,9 @@
     "nocobase-v1": "./bin/index.js"
   },
   "dependencies": {
-    "@nocobase/cli": "2.1.0-alpha.35",
+    "@nocobase/cli": "2.1.0-alpha.37",
     "@nocobase/license-kit": "^0.3.8",
-    "@nocobase/utils": "2.1.0-alpha.35",
-    "@types/fs-extra": "^11.0.1",
-    "@umijs/utils": "3.5.20",
+    "@nocobase/utils": "2.1.0-alpha.37",
     "chalk": "^4.1.1",
     "commander": "^9.2.0",
     "deepmerge": "^4.3.1",
@@ -24,16 +22,16 @@
     "pm2": "^6.0.5",
     "portfinder": "^1.0.28",
     "tar": "^7.4.3",
-    "tree-kill": "^1.2.2",
-    "tsx": "^4.19.0"
+    "tree-kill": "^1.2.2"
   },
   "devDependencies": {
-    "@nocobase/devtools": "2.1.0-alpha.35"
+    "@nocobase/devtools": "2.1.0-alpha.37",
+    "@types/fs-extra": "^11.0.1"
   },
   "repository": {
     "type": "git",
     "url": "git+https://github.com/nocobase/nocobase.git",
     "directory": "packages/core/cli"
   },
-  "gitHead": "313a9e19e84a99924c86f9855c2ae48943f450b0"
+  "gitHead": "8b45f4586ea5b386b376188cfc1012ec12e9bc8b"
 }

package/src/__tests__/test-command.test.js

@@ -9,7 +9,12 @@
 
 const testCommandModule = require('../commands/test');
 
-const { buildVitestNodeArgs, requiresNoNodeSnapshot } = testCommandModule._test;
+const {
+  buildVitestNodeArgs,
+  requiresNoNodeSnapshot,
+  stripDelegatedWorkspaceArgs,
+  resolveWorkspaceTestDelegation,
+} = testCommandModule._test;
 
 describe('cli-v1 test command helpers', () => {
   test('requiresNoNodeSnapshot enables the Node 20+ compatibility flag', () => {
@@ -34,4 +39,32 @@ describe('cli-v1 test command helpers', () => {
       'foo.test.ts',
     ]);
   });
+
+  test('stripDelegatedWorkspaceArgs removes workspace-only routing flags while preserving extra test args', () => {
+    expect(
+      stripDelegatedWorkspaceArgs(
+        [
+          'packages/core/cli',
+          '--server',
+          '--single-thread=true',
+          '--runInBand',
+          'src/__tests__/skills-manager.test.ts',
+        ],
+        'packages/core/cli',
+      ),
+    ).toEqual(['--runInBand', 'src/__tests__/skills-manager.test.ts']);
+  });
+
+  test('resolveWorkspaceTestDelegation forwards workspace package roots that define a test script', () => {
+    const delegation = resolveWorkspaceTestDelegation(
+      ['packages/core/cli'],
+      ['packages/core/cli', '--server'],
+      '/Users/chen/t300/app5/source',
+    );
+
+    expect(delegation).toEqual({
+      packageDir: '/Users/chen/t300/app5/source/packages/core/cli',
+      forwardedArgv: [],
+    });
+  });
 });

package/src/commands/test.js

@@ -10,6 +10,7 @@
 const { Command } = require('commander');
 const { run, checkDBDialect } = require('../util');
 const path = require('path');
+const fs = require('fs');
 
 function stripSingleThreadArgs(argv) {
   const nextArgv = [];
@@ -35,6 +36,85 @@ function stripSingleThreadArgs(argv) {
   return nextArgv;
 }
 
+function stripDelegatedWorkspaceArgs(argv, delegatedPath) {
+  const nextArgv = [];
+  const normalizedDelegatedPath = String(delegatedPath || '')
+    .trim()
+    .split(path.sep)
+    .join('/');
+  let skippedPath = false;
+
+  for (let index = 0; index < argv.length; index += 1) {
+    const token = argv[index];
+    const normalizedToken = String(token || '')
+      .trim()
+      .split(path.sep)
+      .join('/');
+
+    if (
+      !skippedPath &&
+      normalizedDelegatedPath &&
+      !token.startsWith('-') &&
+      normalizedToken === normalizedDelegatedPath
+    ) {
+      skippedPath = true;
+      continue;
+    }
+
+    if (token === '--server' || token === '--client') {
+      continue;
+    }
+
+    if (token === '--single-thread') {
+      const next = argv[index + 1];
+      if (next && !next.startsWith('-')) {
+        index += 1;
+      }
+      continue;
+    }
+
+    if (token.startsWith('--single-thread=')) {
+      continue;
+    }
+
+    nextArgv.push(token);
+  }
+
+  return nextArgv;
+}
+
+function resolveWorkspaceTestDelegation(paths = [], argv = process.argv.slice(3), cwd = process.cwd()) {
+  const firstPath = String(paths?.[0] || '').trim();
+  if (!firstPath) {
+    return undefined;
+  }
+
+  const packageDir = path.resolve(cwd, firstPath);
+  if (packageDir === cwd) {
+    return undefined;
+  }
+
+  const packageJsonPath = path.join(packageDir, 'package.json');
+  if (!fs.existsSync(packageJsonPath)) {
+    return undefined;
+  }
+
+  try {
+    const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
+    const testScript = String(packageJson?.scripts?.test || '').trim();
+    if (!testScript) {
+      return undefined;
+    }
+
+    return {
+      packageDir,
+      forwardedArgv: stripDelegatedWorkspaceArgs(argv, firstPath),
+    };
+  } catch (error) {
+    return undefined;
+  }
+}
+
 function requiresNoNodeSnapshot(nodeVersion = process.versions.node) {
   const major = Number.parseInt(String(nodeVersion).split('.')[0], 10);
   return Number.isFinite(major) && major >= 20;
@@ -69,6 +149,12 @@ function addTestCommand(name, cli) {
     .arguments('[paths...]')
     .allowUnknownOption()
     .action(async (paths, opts) => {
+      const delegation = name === 'test' ? resolveWorkspaceTestDelegation(paths, process.argv.slice(3)) : undefined;
+      if (delegation) {
+        await run('yarn', ['--cwd', delegation.packageDir, 'test', ...delegation.forwardedArgv]);
+        return;
+      }
+
       checkDBDialect();
       if (name === 'test:server') {
         process.env.TEST_ENV = 'server-side';
@@ -144,6 +230,8 @@ module.exports = (cli) => {
 
 module.exports._test = {
   stripSingleThreadArgs,
+  stripDelegatedWorkspaceArgs,
+  resolveWorkspaceTestDelegation,
   requiresNoNodeSnapshot,
   buildVitestNodeArgs,
 };