feat(util): expose safe action wrapper as static import (#5466)

char0n · web-flow · commit 269944a774e2 · 2025-05-08T09:44:34.000+02:00
diff --git a/src/plugins/dropzone/wrap-actions.js b/src/plugins/dropzone/wrap-actions.js
@@ -1,4 +1,4 @@
-import createSafeActionWrapper from '../../utils/create-safe-action-wrapper.js';
+import { createSafeActionWrapper } from '../util/fn.js';
 
 // eslint-disable-next-line import/prefer-default-export
 export const dropFileSuccess = createSafeActionWrapper((oriAction, system) => ({ content }) => {
diff --git a/src/plugins/editor-content-from-file/extensions/dropzone/wrap-actions.js b/src/plugins/editor-content-from-file/extensions/dropzone/wrap-actions.js
@@ -1,4 +1,4 @@
-import createSafeActionWrapper from '../../../../utils/create-safe-action-wrapper.js';
+import { createSafeActionWrapper } from '../../../util/fn.js';
 
 // eslint-disable-next-line import/prefer-default-export
 export const dropFileSuccess = createSafeActionWrapper(
diff --git a/src/plugins/editor-content-from-file/extensions/top-bar/wrap-actions.js b/src/plugins/editor-content-from-file/extensions/top-bar/wrap-actions.js
@@ -1,4 +1,4 @@
-import createSafeActionWrapper from '../../../../utils/create-safe-action-wrapper.js';
+import { createSafeActionWrapper } from '../../../util/fn.js';
 
 export const importUrlSuccess = createSafeActionWrapper(
   (oriAction, system) =>
diff --git a/src/plugins/editor-content-origin/extensions/editor-textarea/wrap-actions.js b/src/plugins/editor-content-origin/extensions/editor-textarea/wrap-actions.js
@@ -1,4 +1,4 @@
-import createSafeActionWrapper from '../../../../utils/create-safe-action-wrapper.js';
+import { createSafeActionWrapper } from '../../../util/fn.js';
 
 // eslint-disable-next-line import/prefer-default-export
 export const setContent = createSafeActionWrapper(
diff --git a/src/plugins/editor-content-persistence/extensions/editor-textarea/wrap-actions.js b/src/plugins/editor-content-persistence/extensions/editor-textarea/wrap-actions.js
@@ -1,4 +1,4 @@
-import createSafeActionWrapper from '../../../../utils/create-safe-action-wrapper.js';
+import { createSafeActionWrapper } from '../../../util/fn.js';
 
 // eslint-disable-next-line import/prefer-default-export
 export const setContent = createSafeActionWrapper((oriAction, system) => (content) => {
diff --git a/src/plugins/editor-content-type/extensions/editor-textarea/wrap-actions.js b/src/plugins/editor-content-type/extensions/editor-textarea/wrap-actions.js
@@ -1,4 +1,4 @@
-import createSafeActionWrapper from '../../../../utils/create-safe-action-wrapper.js';
+import { createSafeActionWrapper } from '../../../util/fn.js';
 
 // eslint-disable-next-line import/prefer-default-export
 export const setContent = createSafeActionWrapper((oriAction, system) => (content) => {
diff --git a/src/plugins/editor-preview-api-design-systems/extensions/editor-content-type/wrap-actions.js b/src/plugins/editor-preview-api-design-systems/extensions/editor-content-type/wrap-actions.js
@@ -1,4 +1,4 @@
-import createSafeActionWrapper from '../../../../utils/create-safe-action-wrapper.js';
+import { createSafeActionWrapper } from '../../../util/fn.js';
 
 // eslint-disable-next-line import/prefer-default-export
 export const detectContentTypeSuccess = createSafeActionWrapper(
diff --git a/src/plugins/editor-preview-asyncapi/extensions/editor-content-type/wrap-actions.js b/src/plugins/editor-preview-asyncapi/extensions/editor-content-type/wrap-actions.js
@@ -1,4 +1,4 @@
-import createSafeActionWrapper from '../../../../utils/create-safe-action-wrapper.js';
+import { createSafeActionWrapper } from '../../../util/fn.js';
 
 // eslint-disable-next-line import/prefer-default-export
 export const detectContentTypeSuccess = createSafeActionWrapper(
diff --git a/src/plugins/editor-preview-swagger-ui/extensions/editor-content-type/wrap-actions.js b/src/plugins/editor-preview-swagger-ui/extensions/editor-content-type/wrap-actions.js
@@ -1,4 +1,4 @@
-import createSafeActionWrapper from '../../../../utils/create-safe-action-wrapper.js';
+import { createSafeActionWrapper } from '../../../util/fn.js';
 
 // eslint-disable-next-line import/prefer-default-export
 export const detectContentTypeSuccess = createSafeActionWrapper(
diff --git a/src/plugins/editor-preview-swagger-ui/wrap-actions.js b/src/plugins/editor-preview-swagger-ui/wrap-actions.js
@@ -1,4 +1,4 @@
-import createSafeActionWrapper from '../../utils/create-safe-action-wrapper.js';
+import { createSafeActionWrapper } from '../util/fn.js';
 
 export const previewUnmounted = createSafeActionWrapper((oriAction, system) => () => {
   system.specActions.updateUrl('');
diff --git a/src/plugins/editor-textarea/wrap-actions.js b/src/plugins/editor-textarea/wrap-actions.js
@@ -1,6 +1,6 @@
 import debounce from 'lodash/debounce.js';
 
-import createSafeActionWrapper from '../../utils/create-safe-action-wrapper.js';
+import { createSafeActionWrapper } from '../util/fn.js';
 
 export const setContentDebouncedImpl = debounce((content, contentOrigin, system) => {
   system.editorActions.setContent(content, contentOrigin);
diff --git a/src/plugins/top-bar/extensions/editor-content-type/wrap-actions.js b/src/plugins/top-bar/extensions/editor-content-type/wrap-actions.js
@@ -1,4 +1,4 @@
-import createSafeActionWrapper from '../../../../utils/create-safe-action-wrapper.js';
+import { createSafeActionWrapper } from '../../../util/fn.js';
 import { IDLE_STATUS, FAILURE_STATUS } from '../../reducers.js';
 
 // eslint-disable-next-line import/prefer-default-export
diff --git a/src/plugins/top-bar/wrap-actions.js b/src/plugins/top-bar/wrap-actions.js
@@ -1,4 +1,4 @@
-import createSafeActionWrapper from '../../utils/create-safe-action-wrapper.js';
+import { createSafeActionWrapper } from '../util/fn.js';
 
 export const importUrlSuccess = createSafeActionWrapper((oriAction, system) => ({ definition }) => {
   const { editorActions, EditorContentOrigin } = system;
diff --git a/src/plugins/util/fn.js b/src/plugins/util/fn.js
@@ -55,3 +55,29 @@ export const requestIdleCallback = globalThis.requestIdleCallback
 export const cancelIdleCallback = globalThis.cancelIdleCallback
   ? (...args) => globalThis.cancelIdleCallback(...args)
   : cancelIdleCallbackPolyfill;
+
+/**
+ * Creates action wrapper that avoids double dispatch.
+ * It is meant to be used via static import and not via a plugin system.
+ *
+ * @param wrapper
+ * @returns {function(*, *): function(...[*]): (*)}
+ */
+export const createSafeActionWrapper = (wrapper) => {
+  let lastFSA = null; // Flux Standard Action(FSA): action objects emitted through redux
+
+  return (origAction, system) =>
+    (...args) => {
+      const fsa = origAction(...args);
+
+      if (fsa === lastFSA) {
+        return fsa;
+      }
+
+      wrapper(origAction, system)(...args);
+
+      lastFSA = fsa;
+
+      return fsa;
+    };
+};
diff --git a/src/plugins/util/index.js b/src/plugins/util/index.js
@@ -1,4 +1,10 @@
-import { generateRequestId, waitUntil, requestIdleCallback, cancelIdleCallback } from './fn.js';
+import {
+  generateRequestId,
+  waitUntil,
+  requestIdleCallback,
+  cancelIdleCallback,
+  createSafeActionWrapper,
+} from './fn.js';
 
 const UtilPlugin = () => ({
   fn: {
@@ -9,4 +15,5 @@ const UtilPlugin = () => ({
   },
 });
 
+export { createSafeActionWrapper };
 export default UtilPlugin;
diff --git a/src/utils/create-safe-action-wrapper.js b/src/utils/create-safe-action-wrapper.js

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-import createSafeActionWrapper from '../../utils/create-safe-action-wrapper.js';`
	`1`	`+import { createSafeActionWrapper } from '../util/fn.js';`
`2`	`2`
`3`	`3`	`// eslint-disable-next-line import/prefer-default-export`
`4`	`4`	`export const dropFileSuccess = createSafeActionWrapper((oriAction, system) => ({ content }) => {`