@nocobase/flow-engine 2.1.0-beta.33 → 2.1.0-beta.34

package/lib/components/dnd/index.d.ts

@@ -39,4 +39,15 @@ export declare const Droppable: FC<{
     model: FlowModel<any>;
     children: React.ReactNode;
 }>;
-export declare const DndProvider: FC<DndContextProps & PersistOptions>;
+export interface DndProviderProps extends DndContextProps, PersistOptions {
+    /**
+     * Whether to render the built-in `DragOverlay` (the "Dragging" pill that
+     * follows the cursor). Defaults to `true` for backwards compatibility with
+     * existing flow-engine drag interactions. Set to `false` when the
+     * surrounding UI already gives clear visual feedback (e.g. a drag-sort
+     * table that highlights the drop position) and the floating pill would be
+     * redundant.
+     */
+    showDragOverlay?: boolean;
+}
+export declare const DndProvider: FC<DndProviderProps>;

package/lib/components/dnd/index.js

@@ -274,6 +274,7 @@ const Droppable = /* @__PURE__ */ __name(({ model, children }) => {
 }, "Droppable");
 const DndProvider = /* @__PURE__ */ __name(({
   persist = true,
+  showDragOverlay = true,
   children,
   onDragStart,
   onDragEnd,
@@ -337,7 +338,7 @@ const DndProvider = /* @__PURE__ */ __name(({
       ...restProps
     },
     children,
-    typeof document !== "undefined" ? (0, import_react_dom.createPortal)(
+    showDragOverlay && typeof document !== "undefined" ? (0, import_react_dom.createPortal)(
       /* @__PURE__ */ import_react.default.createElement(
         import_core.DragOverlay,
         {

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-beta.33",
+  "version": "2.1.0-beta.34",
   "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-beta.33",
-    "@nocobase/shared": "2.1.0-beta.33",
+    "@nocobase/sdk": "2.1.0-beta.34",
+    "@nocobase/shared": "2.1.0-beta.34",
     "ahooks": "^3.7.2",
     "axios": "^1.7.0",
     "dayjs": "^1.11.9",
@@ -37,5 +37,5 @@
   ],
   "author": "NocoBase Team",
   "license": "Apache-2.0",
-  "gitHead": "4815c394e80a264fa8ed619246280923c47aeb72"
+  "gitHead": "ca804833299c547f8d49f8d58f73273a4bfcd03c"
 }

package/src/components/dnd/index.tsx

@@ -283,9 +283,22 @@ export const Droppable: FC<{ model: FlowModel<any>; children: React.ReactNode }>
   );
 };
 
+export interface DndProviderProps extends DndContextProps, PersistOptions {
+  /**
+   * Whether to render the built-in `DragOverlay` (the "Dragging" pill that
+   * follows the cursor). Defaults to `true` for backwards compatibility with
+   * existing flow-engine drag interactions. Set to `false` when the
+   * surrounding UI already gives clear visual feedback (e.g. a drag-sort
+   * table that highlights the drop position) and the floating pill would be
+   * redundant.
+   */
+  showDragOverlay?: boolean;
+}
+
 // 提供一个封装了 DragOverlay 的 DndProvider 组件，继承 DndContext 的所有 props
-export const DndProvider: FC<DndContextProps & PersistOptions> = ({
+export const DndProvider: FC<DndProviderProps> = ({
   persist = true,
+  showDragOverlay = true,
   children,
   onDragStart,
   onDragEnd,
@@ -355,7 +368,7 @@ export const DndProvider: FC<DndContextProps & PersistOptions> = ({
       {...restProps}
     >
       {children}
-      {typeof document !== 'undefined'
+      {showDragOverlay && typeof document !== 'undefined'
         ? createPortal(
             <DragOverlay
               dropAnimation={null}

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', () => {