@nocobase/flow-engine 2.1.0-alpha.37 → 2.1.0-alpha.38

package/lib/executor/FlowExecutor.js

@@ -153,7 +153,7 @@ const _FlowExecutor = class _FlowExecutor {
         const stepDefaultParams = await (0, import_utils.resolveDefaultParams)(step.defaultParams, runtimeCtx);
         combinedParams = { ...stepDefaultParams };
       } else {
-        flowContext.logger.error(
+        flowContext.logger.warn(
           `BaseModel.applyFlow: Step '${stepKey}' in flow '${flowKey}' has neither 'use' nor 'handler'. Skipping.`
         );
         continue;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nocobase/flow-engine",
-  "version": "2.1.0-alpha.37",
+  "version": "2.1.0-alpha.38",
   "private": false,
   "description": "A standalone flow engine for NocoBase, managing workflows, models, and actions.",
   "main": "lib/index.js",
@@ -8,8 +8,8 @@
   "dependencies": {
     "@formily/antd-v5": "1.x",
     "@formily/reactive": "2.x",
-    "@nocobase/sdk": "2.1.0-alpha.37",
-    "@nocobase/shared": "2.1.0-alpha.37",
+    "@nocobase/sdk": "2.1.0-alpha.38",
+    "@nocobase/shared": "2.1.0-alpha.38",
     "ahooks": "^3.7.2",
     "axios": "^1.7.0",
     "dayjs": "^1.11.9",
@@ -37,5 +37,5 @@
   ],
   "author": "NocoBase Team",
   "license": "Apache-2.0",
-  "gitHead": "8b45f4586ea5b386b376188cfc1012ec12e9bc8b"
+  "gitHead": "cc99815fc9ae0612d6db0db5ebbc87a774fff8ed"
 }

package/src/executor/FlowExecutor.ts

@@ -158,7 +158,7 @@ export class FlowExecutor {
         const stepDefaultParams = await resolveDefaultParams(step.defaultParams, runtimeCtx);
         combinedParams = { ...stepDefaultParams };
       } else {
-        flowContext.logger.error(
+        flowContext.logger.warn(
           `BaseModel.applyFlow: Step '${stepKey}' in flow '${flowKey}' has neither 'use' nor 'handler'. Skipping.`,
         );
         continue;

package/src/models/__tests__/flowModel.test.ts

@@ -546,6 +546,34 @@ describe('FlowModel', () => {
 
         loggerSpy.mockRestore();
       });
+
+      test('should warn and skip step when use and handler are both missing', async () => {
+        const warnSpy = vi.spyOn(model.context.logger, 'warn').mockImplementation(() => {});
+        const errorSpy = vi.spyOn(model.context.logger, 'error').mockImplementation(() => {});
+
+        TestFlowModel.registerFlow({
+          key: 'settingsOnlyFlow',
+          steps: {
+            edit: {
+              title: 'Edit',
+              uiSchema: {},
+            },
+          },
+        });
+
+        const result = await model.applyFlow('settingsOnlyFlow');
+
+        expect(result).toEqual({});
+        expect(warnSpy).toHaveBeenCalledWith(
+          expect.stringContaining("Step 'edit' in flow 'settingsOnlyFlow' has neither 'use' nor 'handler'"),
+        );
+        expect(errorSpy).not.toHaveBeenCalledWith(
+          expect.stringContaining("Step 'edit' in flow 'settingsOnlyFlow' has neither 'use' nor 'handler'"),
+        );
+
+        warnSpy.mockRestore();
+        errorSpy.mockRestore();
+      });
     });
 
     describe('beforeRender flows', () => {