@nocobase/cli-v1 2.1.10 → 2.1.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nocobase/cli-v1",
-  "version": "2.1.10",
+  "version": "2.1.11",
   "description": "",
   "license": "Apache-2.0",
   "main": "./src/index.js",
@@ -9,9 +9,9 @@
     "nocobase-v1": "./bin/index.js"
   },
   "dependencies": {
-    "@nocobase/cli": "2.1.10",
+    "@nocobase/cli": "2.1.11",
     "@nocobase/license-kit": "^0.3.8",
-    "@nocobase/utils": "2.1.10",
+    "@nocobase/utils": "2.1.11",
     "chalk": "^4.1.1",
     "commander": "^9.2.0",
     "deepmerge": "^4.3.1",
@@ -33,5 +33,5 @@
     "url": "git+https://github.com/nocobase/nocobase.git",
     "directory": "packages/core/cli"
   },
-  "gitHead": "65d5bc38d72d5a6fca8bcf9798a0b976e382cac9"
+  "gitHead": "1ccf891d837e21089f65f84b892407b34a0a0cb9"
 }

package/src/__tests__/app-dev-utils.test.js

@@ -82,6 +82,7 @@ describe('cli-v1 app-dev utils', () => {
       }),
     ).toEqual([
       'watch',
+      '--clear-screen=false',
       `--ignore=${path.resolve(process.cwd(), 'storage/plugins')}/**`,
       '--tsconfig',
       './tsconfig.server.json',

package/src/__tests__/util.test.js

@@ -0,0 +1,33 @@
+/**
+ * This file is part of the NocoBase (R) project.
+ * Copyright (c) 2020-2024 NocoBase Co., Ltd.
+ * Authors: NocoBase Team.
+ *
+ * This project is dual-licensed under AGPL-3.0 and NocoBase Commercial License.
+ * For more information, please refer to: https://www.nocobase.com/agreement.
+ */
+
+/* eslint-env jest */
+
+const { colorizedDevLogEnv, createRunWithPrefixLabel } = require('../util')._test;
+
+describe('cli-v1 util helpers', () => {
+  test('colorizedDevLogEnv enables color for dev child output by default', () => {
+    expect(colorizedDevLogEnv({})).toEqual({ FORCE_COLOR: '1' });
+  });
+
+  test('colorizedDevLogEnv keeps explicit FORCE_COLOR', () => {
+    expect(colorizedDevLogEnv({ FORCE_COLOR: '3' })).toEqual({ FORCE_COLOR: '3' });
+  });
+
+  test('colorizedDevLogEnv removes inherited NO_COLOR so dev logs stay colored', () => {
+    expect(colorizedDevLogEnv({ NO_COLOR: '1', TERM: 'dumb' })).toEqual({
+      FORCE_COLOR: '1',
+      TERM: 'dumb',
+    });
+  });
+
+  test('createRunWithPrefixLabel colorizes prefixed output even when the parent disables colors', () => {
+    expect(createRunWithPrefixLabel('client', 'cyan')).toBe('\u001b[36m[client]\u001b[39m');
+  });
+});

package/src/commands/app-dev-utils.js

@@ -150,7 +150,7 @@ function buildAppDevServerArgs({
   argv = process.argv,
   serverTsconfigPath = process.env.SERVER_TSCONFIG_PATH,
 } = {}) {
-  const args = ['watch', `--ignore=${resolvePluginStoragePath()}/**`];
+  const args = ['watch', '--clear-screen=false', `--ignore=${resolvePluginStoragePath()}/**`];
 
   if (serverTsconfigPath) {
     args.push('--tsconfig', serverTsconfigPath);

package/src/commands/dev.js

@@ -13,6 +13,7 @@ const {
   hasCorePackages,
   run,
   runWithPrefix,
+  colorizedDevLogEnv,
   postCheck,
   nodeCheck,
   promptForTs,
@@ -198,16 +199,13 @@ module.exports = (cli) => {
         }
         subprocessRef.cancel();
         let i = 0;
-        while (true) {
+        while (i <= 10) {
           ++i;
           const result = await isPortReachable(port);
           if (!result) {
             break;
           }
           await sleep(500);
-          if (i > 10) {
-            break;
-          }
         }
         start();
       };
@@ -216,7 +214,7 @@ module.exports = (cli) => {
         const storagePluginPath = resolvePluginStoragePath();
         const watcher = chokidar.watch(`${storagePluginPath}/**/*`, {
           cwd: process.cwd(),
-          ignored: /(^|[\/\\])\../, // 忽略隐藏文件
+          ignored: /(^|[/\\])\../, // 忽略隐藏文件
           persistent: true,
           depth: 1, // 只监听第一层目录
         });
@@ -260,6 +258,7 @@ module.exports = (cli) => {
 
         const argv = [
           'watch',
+          '--clear-screen=false',
           ...(inspect ? [`--inspect=${inspect === true ? 9229 : inspect}`] : []),
           `--ignore=${resolvePluginStoragePath()}/**`,
           '--tsconfig',
@@ -278,9 +277,9 @@ module.exports = (cli) => {
 
         const runDevServer = () => {
           run('tsx', argv, {
-            env: {
+            env: colorizedDevLogEnv(process.env, {
               APP_PORT: serverPort,
-            },
+            }),
           }).catch((err) => {
             if (err.exitCode == 100) {
               console.log('Restarting server...');

package/src/util.js

@@ -84,6 +84,24 @@ exports.run = (command, args, options = {}) => {
   });
 };
 
+function colorizedDevLogEnv(env = process.env, overrides = {}) {
+  const nextEnv = {
+    ...env,
+    ...overrides,
+  };
+  delete nextEnv.NO_COLOR;
+  if (!nextEnv.FORCE_COLOR) {
+    nextEnv.FORCE_COLOR = '1';
+  }
+  return nextEnv;
+}
+
+function createRunWithPrefixLabel(prefix, color) {
+  const forcedChalk = new chalk.Instance({ level: 1 });
+  const colorize = forcedChalk[color] || forcedChalk.cyan;
+  return colorize(`[${prefix}]`);
+}
+
 exports.runWithPrefix = (command, args, options = {}) => {
   if (command === 'tsx') {
     command = 'node';
@@ -92,15 +110,12 @@ exports.runWithPrefix = (command, args, options = {}) => {
 
   const prefix = options.prefix || 'process';
   const color = options.color || 'cyan';
-  const label = chalk[color](`[${prefix}]`);
+  const label = createRunWithPrefixLabel(prefix, color);
   const subprocess = execa(command, args, {
     shell: true,
     stdio: 'pipe',
     ...options,
-    env: {
-      ...process.env,
-      ...options.env,
-    },
+    env: colorizedDevLogEnv(process.env, options.env),
   });
 
   const writePrefixed = (chunk, writer) => {
@@ -122,6 +137,13 @@ exports.runWithPrefix = (command, args, options = {}) => {
   return subprocess;
 };
 
+exports._test = {
+  createRunWithPrefixLabel,
+  colorizedDevLogEnv,
+};
+
+exports.colorizedDevLogEnv = colorizedDevLogEnv;
+
 exports.isPortReachable = async (port, { timeout = 1000, host } = {}) => {
   const promise = new Promise((resolve, reject) => {
     const socket = new net.Socket();