@darajs/core 1.3.2 → 1.4.0

package/dist/shared/interactivity/use-variable-value.js

@@ -3,12 +3,60 @@ import { useContext } from 'react';
 import { useLocation } from 'react-router-dom';
 import { useRecoilCallback } from 'recoil';
 import { useDeepCompare } from '@darajs/ui-utils';
-// eslint-disable-next-line import/no-cycle
 import { useSessionToken } from '../../auth/auth-context';
 import { WebSocketCtx, useTaskContext } from '../../shared/context';
 import { normalizeRequest } from '../../shared/utils/normalization';
 import { isDataVariable, isDerivedDataVariable, isDerivedVariable, isResolvedDataVariable, isResolvedDerivedDataVariable, isResolvedDerivedVariable, isVariable, } from '../../types';
 import { fetchDataVariable, fetchDerivedDataVariable, fetchDerivedVariable, formatDerivedVariableRequest, isTaskResponse, resolveVariable, } from './internal';
+/**
+ * Helper function that returns the current value of a variable.
+ *
+ * Plain variables are always resolved to their value.
+ * Computed (server-side) variables are resolved to their uid and dependency values, unless shouldFetchVariable is true.
+ */
+export function getVariableValue(variable, shouldFetchVariable = false, ctx) {
+    // Using loadable since the resolver is only used for simple atoms and shouldn't cause problems
+    const resolved = resolveVariable(variable, ctx.client, ctx.taskContext, ctx.search, ctx.token, (v) => ctx.snapshot.getLoadable(v).getValue());
+    // if we're NOT forced to fetch, or if it's not a DV/DDV/data variable, return the resolved value
+    // variable is plain/url
+    if (!shouldFetchVariable ||
+        (!isDerivedVariable(variable) && !isDataVariable(variable) && !isDerivedDataVariable(variable))) {
+        return resolved;
+    }
+    // we're forced to fetch but the resolved variable is not a resolved DV/data var, return the resolved value
+    // variable is plain/url
+    if (!isResolvedDerivedVariable(resolved) &&
+        !isResolvedDataVariable(resolved) &&
+        !isResolvedDerivedDataVariable(resolved)) {
+        return resolved;
+    }
+    // data variable
+    if (isResolvedDataVariable(resolved)) {
+        return fetchDataVariable(resolved.uid, ctx.token, resolved.filters);
+    }
+    // derived variable
+    return fetchDerivedVariable({
+        cache: variable.cache,
+        force: false,
+        token: ctx.token,
+        uid: resolved.uid,
+        values: normalizeRequest(formatDerivedVariableRequest(resolved.values), variable.variables),
+        wsClient: ctx.client,
+    }).then((resp) => {
+        // This is really only used in DownloadVariable currently; we can add support for tasks
+        // if it is requested in the future
+        if (isTaskResponse(resp)) {
+            throw new Error('Task DerivedVariables are not supported in this context');
+        }
+        // for derived data variables we need to make another request to retrieve the filtered value
+        if (isDerivedDataVariable(variable)) {
+            return ctx.client
+                .getChannel()
+                .then((chan) => fetchDerivedDataVariable(variable.uid, ctx.token, resp.cache_key, chan, variable.filters));
+        }
+        return resp.value;
+    });
+}
 /**
  * A helper hook that turns a Variable class into the actual value.
  * As opposed to the `useVariable` hook, this one returns a callback to retrieve the latest value
@@ -30,46 +78,12 @@ export default function useVariableValue(variable, shouldFetchVariable = false)
     }
     return useRecoilCallback(({ snapshot }) => {
         return () => {
-            // Using loadable since the resolver is only used for simple atoms and shouldn't cause problems
-            const resolved = resolveVariable(variable, client, taskContext, search, token, (v) => snapshot.getLoadable(v).getValue());
-            // if we're NOT forced to fetch, or if it's not a DV/DDV/data variable, return the resolved value
-            // variable is plain/url
-            if (!shouldFetchVariable ||
-                (!isDerivedVariable(variable) && !isDataVariable(variable) && !isDerivedDataVariable(variable))) {
-                return resolved;
-            }
-            // we're forced to fetch but the resolved variable is not a resolved DV/data var, return the resolved value
-            // variable is plain/url
-            if (!isResolvedDerivedVariable(resolved) &&
-                !isResolvedDataVariable(resolved) &&
-                !isResolvedDerivedDataVariable(resolved)) {
-                return resolved;
-            }
-            // data variable
-            if (isResolvedDataVariable(resolved)) {
-                return fetchDataVariable(resolved.uid, token, resolved.filters);
-            }
-            // derived variable
-            return fetchDerivedVariable({
-                cache: variable.cache,
-                force: false,
+            return getVariableValue(variable, shouldFetchVariable, {
+                client,
+                search,
+                snapshot,
+                taskContext,
                 token,
-                uid: resolved.uid,
-                values: normalizeRequest(formatDerivedVariableRequest(resolved.values), variable.variables),
-                wsClient: client,
-            }).then((resp) => {
-                // This is really only used in DownloadVariable currently; we can add support for tasks
-                // if it is requested in the future
-                if (isTaskResponse(resp)) {
-                    throw new Error('Task DerivedVariables are not supported in this context');
-                }
-                // for derived data variables we need to make another request to retrieve the filtered value
-                if (isDerivedDataVariable(variable)) {
-                    return client
-                        .getChannel()
-                        .then((chan) => fetchDerivedDataVariable(variable.uid, token, resp.cache_key, chan, variable.filters));
-                }
-                return resp.value;
             });
         };
     }, [variable.uid, useDeepCompare(taskContext), client, search, token]);

package/dist/shared/interactivity/use-variable-value.js.map

@@ -1 +1 @@
-{"version":3,"file":"use-variable-value.js","sourceRoot":"","sources":["../../../js/shared/interactivity/use-variable-value.tsx"],"names":[],"mappings":"AAAA,+CAA+C;AAC/C,OAAO,EAAE,UAAU,EAAE,MAAM,OAAO,CAAC;AACnC,OAAO,EAAE,WAAW,EAAE,MAAM,kBAAkB,CAAC;AAC/C,OAAO,EAAE,iBAAiB,EAAE,MAAM,QAAQ,CAAC;AAE3C,OAAO,EAAE,cAAc,EAAE,MAAM,kBAAkB,CAAC;AAElD,2CAA2C;AAC3C,OAAO,EAAE,eAAe,EAAE,MAAM,qBAAqB,CAAC;AACtD,OAAO,EAAE,YAAY,EAAE,cAAc,EAAE,MAAM,kBAAkB,CAAC;AAChE,OAAO,EAAE,gBAAgB,EAAE,MAAM,8BAA8B,CAAC;AAChE,OAAO,EAQH,cAAc,EACd,qBAAqB,EACrB,iBAAiB,EACjB,sBAAsB,EACtB,6BAA6B,EAC7B,yBAAyB,EACzB,UAAU,GACb,MAAM,SAAS,CAAC;AAEjB,OAAO,EACH,iBAAiB,EACjB,wBAAwB,EACxB,oBAAoB,EACpB,4BAA4B,EAC5B,cAAc,EACd,eAAe,GAClB,MAAM,YAAY,CAAC;AA0BpB;;;;;;;;;;GAUG;AACH,MAAM,CAAC,OAAO,UAAU,gBAAgB,CACpC,QAAkF,EAClF,sBAAyB,KAAU;IAEnC,MAAM,WAAW,GAAG,cAAc,EAAE,CAAC;IACrC,MAAM,EAAE,MAAM,EAAE,GAAG,UAAU,CAAC,YAAY,CAAC,CAAC;IAC5C,MAAM,EAAE,MAAM,EAAE,GAAG,WAAW,EAAE,CAAC;IACjC,MAAM,KAAK,GAAG,eAAe,EAAE,CAAC;IAEhC,IAAI,CAAC,UAAU,CAAK,QAAQ,CAAC,EAAE;QAC3B,OAAO,GAAG,EAAE,CAAC,QAAQ,CAAC;KACzB;IAED,OAAO,iBAAiB,CACpB,CAAC,EAAE,QAAQ,EAAE,EAAE,EAAE;QACb,OAAO,GAAG,EAAE;YACR,+FAA+F;YAC/F,MAAM,QAAQ,GAAG,eAAe,CAAM,QAAQ,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,CAAC,EAAE,EAAE,CACtF,QAAQ,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC,QAAQ,EAAE,CACrC,CAAC;YAEF,iGAAiG;YACjG,wBAAwB;YACxB,IACI,CAAC,mBAAmB;gBACpB,CAAC,CAAC,iBAAiB,CAAC,QAAQ,CAAC,IAAI,CAAC,cAAc,CAAC,QAAQ,CAAC,IAAI,CAAC,qBAAqB,CAAC,QAAQ,CAAC,CAAC,EACjG;gBACE,OAAO,QAAQ,CAAC;aACnB;YAED,2GAA2G;YAC3G,wBAAwB;YACxB,IACI,CAAC,yBAAyB,CAAC,QAAQ,CAAC;gBACpC,CAAC,sBAAsB,CAAC,QAAQ,CAAC;gBACjC,CAAC,6BAA6B,CAAC,QAAQ,CAAC,EAC1C;gBACE,OAAO,QAAQ,CAAC;aACnB;YAED,gBAAgB;YAChB,IAAI,sBAAsB,CAAC,QAAQ,CAAC,EAAE;gBAClC,OAAO,iBAAiB,CAAC,QAAQ,CAAC,GAAG,EAAE,KAAK,EAAE,QAAQ,CAAC,OAAO,CAAC,CAAC;aACnE;YAED,mBAAmB;YACnB,OAAO,oBAAoB,CAAC;gBACxB,KAAK,EAAG,QAAkD,CAAC,KAAK;gBAChE,KAAK,EAAE,KAAK;gBACZ,KAAK;gBACL,GAAG,EAAE,QAAQ,CAAC,GAAG;gBACjB,MAAM,EAAE,gBAAgB,CACpB,4BAA4B,CAAC,QAAQ,CAAC,MAAM,CAAC,EAC5C,QAAkD,CAAC,SAAS,CAChE;gBACD,QAAQ,EAAE,MAAM;aACnB,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE;gBACb,uFAAuF;gBACvF,mCAAmC;gBACnC,IAAI,cAAc,CAAC,IAAI,CAAC,EAAE;oBACtB,MAAM,IAAI,KAAK,CAAC,yDAAyD,CAAC,CAAC;iBAC9E;gBAED,4FAA4F;gBAC5F,IAAI,qBAAqB,CAAC,QAAQ,CAAC,EAAE;oBACjC,OAAO,MAAM;yBACR,UAAU,EAAE;yBACZ,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CACX,wBAAwB,CAAC,QAAQ,CAAC,GAAG,EAAE,KAAK,EAAE,IAAI,CAAC,SAAS,EAAE,IAAI,EAAE,QAAQ,CAAC,OAAO,CAAC,CACxF,CAAC;iBACT;gBAED,OAAO,IAAI,CAAC,KAAK,CAAC;YACtB,CAAC,CAAC,CAAC;QACP,CAAC,CAAC;IACN,CAAC,EACD,CAAC,QAAQ,CAAC,GAAG,EAAE,cAAc,CAAC,WAAW,CAAC,EAAE,MAAM,EAAE,MAAM,EAAE,KAAK,CAAC,CACrE,CAAC;AACN,CAAC"}
+{"version":3,"file":"use-variable-value.js","sourceRoot":"","sources":["../../../js/shared/interactivity/use-variable-value.tsx"],"names":[],"mappings":"AAAA,+CAA+C;AAC/C,OAAO,EAAE,UAAU,EAAE,MAAM,OAAO,CAAC;AACnC,OAAO,EAAE,WAAW,EAAE,MAAM,kBAAkB,CAAC;AAC/C,OAAO,EAAY,iBAAiB,EAAE,MAAM,QAAQ,CAAC;AAErD,OAAO,EAAE,cAAc,EAAE,MAAM,kBAAkB,CAAC;AAIlD,OAAO,EAAE,eAAe,EAAE,MAAM,qBAAqB,CAAC;AACtD,OAAO,EAAE,YAAY,EAAE,cAAc,EAAE,MAAM,kBAAkB,CAAC;AAChE,OAAO,EAAE,gBAAgB,EAAE,MAAM,8BAA8B,CAAC;AAChE,OAAO,EAUH,cAAc,EACd,qBAAqB,EACrB,iBAAiB,EACjB,sBAAsB,EACtB,6BAA6B,EAC7B,yBAAyB,EACzB,UAAU,GACb,MAAM,SAAS,CAAC;AAGjB,OAAO,EACH,iBAAiB,EACjB,wBAAwB,EACxB,oBAAoB,EACpB,4BAA4B,EAC5B,cAAc,EACd,eAAe,GAClB,MAAM,YAAY,CAAC;AAUpB;;;;;GAKG;AACH,MAAM,UAAU,gBAAgB,CAC5B,QAAyB,EACzB,sBAAyB,KAAU,EACnC,GAAwB;IASxB,+FAA+F;IAC/F,MAAM,QAAQ,GAAG,eAAe,CAAM,QAAQ,EAAE,GAAG,CAAC,MAAM,EAAE,GAAG,CAAC,WAAW,EAAE,GAAG,CAAC,MAAM,EAAE,GAAG,CAAC,KAAK,EAAE,CAAC,CAAC,EAAE,EAAE,CACtG,GAAG,CAAC,QAAQ,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC,QAAQ,EAAE,CACzC,CAAC;IAEF,iGAAiG;IACjG,wBAAwB;IACxB,IACI,CAAC,mBAAmB;QACpB,CAAC,CAAC,iBAAiB,CAAC,QAAQ,CAAC,IAAI,CAAC,cAAc,CAAC,QAAQ,CAAC,IAAI,CAAC,qBAAqB,CAAC,QAAQ,CAAC,CAAC,EACjG;QACE,OAAO,QAAQ,CAAC;KACnB;IAED,2GAA2G;IAC3G,wBAAwB;IACxB,IACI,CAAC,yBAAyB,CAAC,QAAQ,CAAC;QACpC,CAAC,sBAAsB,CAAC,QAAQ,CAAC;QACjC,CAAC,6BAA6B,CAAC,QAAQ,CAAC,EAC1C;QACE,OAAO,QAAQ,CAAC;KACnB;IAED,gBAAgB;IAChB,IAAI,sBAAsB,CAAC,QAAQ,CAAC,EAAE;QAClC,OAAO,iBAAiB,CAAC,QAAQ,CAAC,GAAG,EAAE,GAAG,CAAC,KAAK,EAAE,QAAQ,CAAC,OAAO,CAAC,CAAC;KACvE;IAED,mBAAmB;IACnB,OAAO,oBAAoB,CAAC;QACxB,KAAK,EAAG,QAAkD,CAAC,KAAK;QAChE,KAAK,EAAE,KAAK;QACZ,KAAK,EAAE,GAAG,CAAC,KAAK;QAChB,GAAG,EAAE,QAAQ,CAAC,GAAG;QACjB,MAAM,EAAE,gBAAgB,CACpB,4BAA4B,CAAC,QAAQ,CAAC,MAAM,CAAC,EAC5C,QAAkD,CAAC,SAAS,CAChE;QACD,QAAQ,EAAE,GAAG,CAAC,MAAM;KACvB,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE;QACb,uFAAuF;QACvF,mCAAmC;QACnC,IAAI,cAAc,CAAC,IAAI,CAAC,EAAE;YACtB,MAAM,IAAI,KAAK,CAAC,yDAAyD,CAAC,CAAC;SAC9E;QAED,4FAA4F;QAC5F,IAAI,qBAAqB,CAAC,QAAQ,CAAC,EAAE;YACjC,OAAO,GAAG,CAAC,MAAM;iBACZ,UAAU,EAAE;iBACZ,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CACX,wBAAwB,CAAC,QAAQ,CAAC,GAAG,EAAE,GAAG,CAAC,KAAK,EAAE,IAAI,CAAC,SAAS,EAAE,IAAI,EAAE,QAAQ,CAAC,OAAO,CAAC,CAC5F,CAAC;SACT;QAED,OAAO,IAAI,CAAC,KAAK,CAAC;IACtB,CAAC,CAAgB,CAAC;AACtB,CAAC;AAED;;;;;;;;;;GAUG;AACH,MAAM,CAAC,OAAO,UAAU,gBAAgB,CACpC,QAAkF,EAClF,sBAAyB,KAAU;IAEnC,MAAM,WAAW,GAAG,cAAc,EAAE,CAAC;IACrC,MAAM,EAAE,MAAM,EAAE,GAAG,UAAU,CAAC,YAAY,CAAC,CAAC;IAC5C,MAAM,EAAE,MAAM,EAAE,GAAG,WAAW,EAAE,CAAC;IACjC,MAAM,KAAK,GAAG,eAAe,EAAE,CAAC;IAEhC,IAAI,CAAC,UAAU,CAAK,QAAQ,CAAC,EAAE;QAC3B,OAAO,GAAG,EAAE,CAAC,QAAQ,CAAC;KACzB;IAED,OAAO,iBAAiB,CACpB,CAAC,EAAE,QAAQ,EAAE,EAAE,EAAE;QACb,OAAO,GAAG,EAAE;YACR,OAAO,gBAAgB,CAAQ,QAAQ,EAAE,mBAAmB,EAAE;gBAC1D,MAAM;gBACN,MAAM;gBACN,QAAQ;gBACR,WAAW;gBACX,KAAK;aACR,CAAC,CAAC;QACP,CAAC,CAAC;IACN,CAAC,EACD,CAAC,QAAQ,CAAC,GAAG,EAAE,cAAc,CAAC,WAAW,CAAC,EAAE,MAAM,EAAE,MAAM,EAAE,KAAK,CAAC,CACrE,CAAC;AACN,CAAC"}

package/dist/shared/private-route/private-route.d.ts

@@ -1,16 +1,12 @@
 import { ReactNode } from 'react';
-import { Variable } from '../../types';
+import { Action } from '../../types';
 interface PrivateRouteProps {
     /** The children to wrap */
     children: ReactNode;
-    /** To have exact matching for the route path */
-    exact?: boolean;
     /** Name of the page this route links to */
     name?: string;
-    /** The route for the private route */
-    path: string;
     /** Variables which should be reset upon visiting the page */
-    reset_vars_on_load?: Variable<any>[];
+    on_load?: Action;
 }
 /**
  * The PrivateRoute component takes a private route object and checks for authentication.
@@ -18,6 +14,6 @@ interface PrivateRouteProps {
  *
  * @param props - the component props
  */
-declare function PrivateRoute({ children, path, exact, reset_vars_on_load, name, ...rest }: PrivateRouteProps): JSX.Element;
+declare function PrivateRoute({ children, on_load, name }: PrivateRouteProps): ReactNode;
 export default PrivateRoute;
 //# sourceMappingURL=private-route.d.ts.map

package/dist/shared/private-route/private-route.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"private-route.d.ts","sourceRoot":"","sources":["../../../js/shared/private-route/private-route.tsx"],"names":[],"mappings":"AAEA,OAAO,EAAE,SAAS,EAAa,MAAM,OAAO,CAAC;AAM7C,OAAO,EAAE,QAAQ,EAAE,MAAM,SAAS,CAAC;AAEnC,UAAU,iBAAiB;IACvB,2BAA2B;IAC3B,QAAQ,EAAE,SAAS,CAAC;IACpB,gDAAgD;IAChD,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB,2CAA2C;IAC3C,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,sCAAsC;IACtC,IAAI,EAAE,MAAM,CAAC;IACb,6DAA6D;IAC7D,kBAAkB,CAAC,EAAE,QAAQ,CAAC,GAAG,CAAC,EAAE,CAAC;CACxC;AAED;;;;;GAKG;AACH,iBAAS,YAAY,CAAC,EAAE,QAAQ,EAAE,IAAI,EAAE,KAAK,EAAE,kBAAkB,EAAE,IAAI,EAAE,GAAG,IAAI,EAAE,EAAE,iBAAiB,GAAG,GAAG,CAAC,OAAO,CA8BlH;AAED,eAAe,YAAY,CAAC"}
+{"version":3,"file":"private-route.d.ts","sourceRoot":"","sources":["../../../js/shared/private-route/private-route.tsx"],"names":[],"mappings":"AAEA,OAAO,EAAE,SAAS,EAAa,MAAM,OAAO,CAAC;AAO7C,OAAO,EAAE,MAAM,EAAE,MAAM,SAAS,CAAC;AAEjC,UAAU,iBAAiB;IACvB,2BAA2B;IAC3B,QAAQ,EAAE,SAAS,CAAC;IACpB,2CAA2C;IAC3C,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,6DAA6D;IAC7D,OAAO,CAAC,EAAE,MAAM,CAAC;CACpB;AAED;;;;;GAKG;AACH,iBAAS,YAAY,CAAC,EAAE,QAAQ,EAAE,OAAO,EAAE,IAAI,EAAE,EAAE,iBAAiB,GAAG,SAAS,CAiB/E;AAED,eAAe,YAAY,CAAC"}

package/dist/shared/private-route/private-route.js

@@ -1,20 +1,10 @@
-var __rest = (this && this.__rest) || function (s, e) {
-    var t = {};
-    for (var p in s) if (Object.prototype.hasOwnProperty.call(s, p) && e.indexOf(p) < 0)
-        t[p] = s[p];
-    if (s != null && typeof Object.getOwnPropertySymbols === "function")
-        for (var i = 0, p = Object.getOwnPropertySymbols(s); i < p.length; i++) {
-            if (e.indexOf(p[i]) < 0 && Object.prototype.propertyIsEnumerable.call(s, p[i]))
-                t[p[i]] = s[p[i]];
-        }
-    return t;
-};
 import { jsx as _jsx } from "react/jsx-runtime";
 /* eslint-disable react-hooks/exhaustive-deps */
 import { useEffect } from 'react';
-import { Redirect, Route } from 'react-router-dom';
+import { Redirect } from 'react-router-dom';
 import { useSessionToken } from '../../auth/auth-context';
-import { useResetVariables } from '../../shared/interactivity';
+import DefaultFallback from '../../components/fallback/default';
+import useAction from '../../shared/utils/use-action';
 import useWindowTitle from '../../shared/utils/use-window-title';
 /**
  * The PrivateRoute component takes a private route object and checks for authentication.
@@ -22,19 +12,19 @@ import useWindowTitle from '../../shared/utils/use-window-title';
  *
  * @param props - the component props
  */
-function PrivateRoute(_a) {
-    var { children, path, exact, reset_vars_on_load, name } = _a, rest = __rest(_a, ["children", "path", "exact", "reset_vars_on_load", "name"]);
+function PrivateRoute({ children, on_load, name }) {
     const token = useSessionToken();
-    const resetVariables = useResetVariables(reset_vars_on_load !== null && reset_vars_on_load !== void 0 ? reset_vars_on_load : []);
+    const [onLoad, isLoading] = useAction(on_load);
     useWindowTitle(name);
     useEffect(() => {
-        // On mount, reset variables which were specified
-        resetVariables();
+        // On mount, call the on_load action
+        onLoad();
     }, []);
-    return (_jsx(Route, Object.assign({}, rest, { exact: exact, path: path, render: ({ location }) => token ? (children) : (_jsx(Redirect, { to: {
-                pathname: '/login',
-                state: { from: location },
-            } })) })));
+    if (!token) {
+        return _jsx(Redirect, { to: "/login" });
+    }
+    // Show fallback while the onLoad action is in progress
+    return isLoading ? _jsx(DefaultFallback, {}) : children;
 }
 export default PrivateRoute;
 //# sourceMappingURL=private-route.js.map

package/dist/shared/private-route/private-route.js.map

@@ -1 +1 @@
-{"version":3,"file":"private-route.js","sourceRoot":"","sources":["../../../js/shared/private-route/private-route.tsx"],"names":[],"mappings":";;;;;;;;;;;;AAAA,gDAAgD;AAEhD,OAAO,EAAa,SAAS,EAAE,MAAM,OAAO,CAAC;AAC7C,OAAO,EAAE,QAAQ,EAAE,KAAK,EAAE,MAAM,kBAAkB,CAAC;AAEnD,OAAO,EAAE,eAAe,EAAE,MAAM,qBAAqB,CAAC;AACtD,OAAO,EAAE,iBAAiB,EAAE,MAAM,wBAAwB,CAAC;AAC3D,OAAO,cAAc,MAAM,iCAAiC,CAAC;AAgB7D;;;;;GAKG;AACH,SAAS,YAAY,CAAC,EAA+E;QAA/E,EAAE,QAAQ,EAAE,IAAI,EAAE,KAAK,EAAE,kBAAkB,EAAE,IAAI,OAA8B,EAAzB,IAAI,cAA1D,2DAA4D,CAAF;IAC5E,MAAM,KAAK,GAAG,eAAe,EAAE,CAAC;IAChC,MAAM,cAAc,GAAG,iBAAiB,CAAC,kBAAkB,aAAlB,kBAAkB,cAAlB,kBAAkB,GAAI,EAAE,CAAC,CAAC;IAEnE,cAAc,CAAC,IAAI,CAAC,CAAC;IAErB,SAAS,CAAC,GAAG,EAAE;QACX,iDAAiD;QACjD,cAAc,EAAE,CAAC;IACrB,CAAC,EAAE,EAAE,CAAC,CAAC;IAEP,OAAO,CACH,KAAC,KAAK,oBACE,IAAI,IACR,KAAK,EAAE,KAAK,EACZ,IAAI,EAAE,IAAI,EACV,MAAM,EAAE,CAAC,EAAE,QAAQ,EAAE,EAAE,EAAE,CACrB,KAAK,CAAC,CAAC,CAAC,CACJ,QAAQ,CACX,CAAC,CAAC,CAAC,CACA,KAAC,QAAQ,IACL,EAAE,EAAE;gBACA,QAAQ,EAAE,QAAQ;gBAClB,KAAK,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE;aAC5B,GACH,CACL,IAEP,CACL,CAAC;AACN,CAAC;AAED,eAAe,YAAY,CAAC"}
+{"version":3,"file":"private-route.js","sourceRoot":"","sources":["../../../js/shared/private-route/private-route.tsx"],"names":[],"mappings":";AAAA,gDAAgD;AAEhD,OAAO,EAAa,SAAS,EAAE,MAAM,OAAO,CAAC;AAC7C,OAAO,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAE5C,OAAO,EAAE,eAAe,EAAE,MAAM,qBAAqB,CAAC;AACtD,OAAO,eAAe,MAAM,+BAA+B,CAAC;AAC5D,OAAO,SAAS,MAAM,2BAA2B,CAAC;AAClD,OAAO,cAAc,MAAM,iCAAiC,CAAC;AAY7D;;;;;GAKG;AACH,SAAS,YAAY,CAAC,EAAE,QAAQ,EAAE,OAAO,EAAE,IAAI,EAAqB;IAChE,MAAM,KAAK,GAAG,eAAe,EAAE,CAAC;IAChC,MAAM,CAAC,MAAM,EAAE,SAAS,CAAC,GAAG,SAAS,CAAC,OAAO,CAAC,CAAC;IAE/C,cAAc,CAAC,IAAI,CAAC,CAAC;IAErB,SAAS,CAAC,GAAG,EAAE;QACX,oCAAoC;QACpC,MAAM,EAAE,CAAC;IACb,CAAC,EAAE,EAAE,CAAC,CAAC;IAEP,IAAI,CAAC,KAAK,EAAE;QACR,OAAO,KAAC,QAAQ,IAAC,EAAE,EAAC,QAAQ,GAAG,CAAC;KACnC;IAED,uDAAuD;IACvD,OAAO,SAAS,CAAC,CAAC,CAAC,KAAC,eAAe,KAAG,CAAC,CAAC,CAAC,QAAQ,CAAC;AACtD,CAAC;AAED,eAAe,YAAY,CAAC"}

package/dist/shared/utils/use-action-registry.d.ts

@@ -1,6 +1,6 @@
-import { ActionDef, ActionInstance } from '../../types/core';
+import { ActionDef, ActionImpl } from '../../types/core';
 interface ActionRegistryInterface {
-    get: (instance: ActionInstance) => ActionDef;
+    get: (instance: ActionImpl) => ActionDef;
 }
 /**
  * The action registry pulls the full list of actions from the backend and then returns a function that allows a

package/dist/shared/utils/use-action-registry.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-action-registry.d.ts","sourceRoot":"","sources":["../../../js/shared/utils/use-action-registry.tsx"],"names":[],"mappings":"AAGA,OAAO,EAAE,SAAS,EAAE,cAAc,EAAE,MAAM,cAAc,CAAC;AAEzD,UAAU,uBAAuB;IAC7B,GAAG,EAAE,CAAC,QAAQ,EAAE,cAAc,KAAK,SAAS,CAAC;CAChD;AAED;;;GAGG;AACH,iBAAS,iBAAiB,IAAI,uBAAuB,CAcpD;AAED,eAAe,iBAAiB,CAAC"}
+{"version":3,"file":"use-action-registry.d.ts","sourceRoot":"","sources":["../../../js/shared/utils/use-action-registry.tsx"],"names":[],"mappings":"AAGA,OAAO,EAAE,SAAS,EAAE,UAAU,EAAE,MAAM,cAAc,CAAC;AAErD,UAAU,uBAAuB;IAC7B,GAAG,EAAE,CAAC,QAAQ,EAAE,UAAU,KAAK,SAAS,CAAC;CAC5C;AAED;;;GAGG;AACH,iBAAS,iBAAiB,IAAI,uBAAuB,CAapD;AAED,eAAe,iBAAiB,CAAC"}

package/dist/shared/utils/use-action-registry.js

@@ -1,4 +1,4 @@
-import { useCallback, useContext } from 'react';
+import { useCallback, useContext, useMemo } from 'react';
 import RegistriesCtx from '../../shared/context/registries-context';
 /**
  * The action registry pulls the full list of actions from the backend and then returns a function that allows a
@@ -6,13 +6,13 @@ import RegistriesCtx from '../../shared/context/registries-context';
  */
 function useActionRegistry() {
     const { actionRegistry: actions } = useContext(RegistriesCtx);
-    const get = useCallback((instance) => {
-        if (actions && actions[instance.name]) {
-            return actions[instance.name];
+    const get = useCallback((impl) => {
+        if (actions && actions[impl.name]) {
+            return actions[impl.name];
         }
-        throw new Error(`Attempted to load an action (${instance.name}) that is not in the registry`);
+        throw new Error(`Attempted to load an action (${impl.name}) that is not in the registry`);
     }, [actions]);
-    return { get };
+    return useMemo(() => ({ get }), [get]);
 }
 export default useActionRegistry;
 //# sourceMappingURL=use-action-registry.js.map

package/dist/shared/utils/use-action-registry.js.map

@@ -1 +1 @@
-{"version":3,"file":"use-action-registry.js","sourceRoot":"","sources":["../../../js/shared/utils/use-action-registry.tsx"],"names":[],"mappings":"AAAA,OAAO,EAAE,WAAW,EAAE,UAAU,EAAE,MAAM,OAAO,CAAC;AAEhD,OAAO,aAAa,MAAM,qCAAqC,CAAC;AAOhE;;;GAGG;AACH,SAAS,iBAAiB;IACtB,MAAM,EAAE,cAAc,EAAE,OAAO,EAAE,GAAG,UAAU,CAAC,aAAa,CAAC,CAAC;IAE9D,MAAM,GAAG,GAAG,WAAW,CACnB,CAAC,QAAwB,EAAa,EAAE;QACpC,IAAI,OAAO,IAAI,OAAO,CAAC,QAAQ,CAAC,IAAI,CAAC,EAAE;YACnC,OAAO,OAAO,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;SACjC;QACD,MAAM,IAAI,KAAK,CAAC,gCAAgC,QAAQ,CAAC,IAAI,+BAA+B,CAAC,CAAC;IAClG,CAAC,EACD,CAAC,OAAO,CAAC,CACZ,CAAC;IAEF,OAAO,EAAE,GAAG,EAAE,CAAC;AACnB,CAAC;AAED,eAAe,iBAAiB,CAAC"}
+{"version":3,"file":"use-action-registry.js","sourceRoot":"","sources":["../../../js/shared/utils/use-action-registry.tsx"],"names":[],"mappings":"AAAA,OAAO,EAAE,WAAW,EAAE,UAAU,EAAE,OAAO,EAAE,MAAM,OAAO,CAAC;AAEzD,OAAO,aAAa,MAAM,qCAAqC,CAAC;AAOhE;;;GAGG;AACH,SAAS,iBAAiB;IACtB,MAAM,EAAE,cAAc,EAAE,OAAO,EAAE,GAAG,UAAU,CAAC,aAAa,CAAC,CAAC;IAE9D,MAAM,GAAG,GAAG,WAAW,CACnB,CAAC,IAAgB,EAAa,EAAE;QAC5B,IAAI,OAAO,IAAI,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE;YAC/B,OAAO,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;SAC7B;QACD,MAAM,IAAI,KAAK,CAAC,gCAAgC,IAAI,CAAC,IAAI,+BAA+B,CAAC,CAAC;IAC9F,CAAC,EACD,CAAC,OAAO,CAAC,CACZ,CAAC;IACF,OAAO,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC,EAAE,GAAG,EAAE,CAAC,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;AAC3C,CAAC;AAED,eAAe,iBAAiB,CAAC"}

package/dist/shared/utils/use-action.d.ts

@@ -1,9 +1,21 @@
 import { Action } from '../../types';
+import { ActionContext, ActionImpl } from '../../types/core';
+interface UseActionOptions {
+    /**
+     * Callback to invoke when an unhandled action is encountered.
+     * When not defined, an error notification will be shown instead.
+     *
+     * @param action action implementation
+     */
+    onUnhandledAction?: (action: ActionImpl, actionCtx: ActionContext) => void | Promise<void>;
+}
 /**
  * Consume an action def from the framework and return a callback function that can be used to trigger it
  * and a loading state boolean indicating whether an action is currently in progress
  *
- * @param actionDefinition the action def passed in
+ * @param action the action passed in
+ * @param options optional extra options for the hook
  */
-export default function useAction(action: Action): [(value: any) => Promise<void>, boolean];
+export default function useAction(action: Action, options?: UseActionOptions): [(input?: any) => Promise<void>, boolean];
+export {};
 //# sourceMappingURL=use-action.d.ts.map

package/dist/shared/utils/use-action.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-action.d.ts","sourceRoot":"","sources":["../../../js/shared/utils/use-action.tsx"],"names":[],"mappings":"AAKA,OAAO,EAAE,MAAM,EAAyC,MAAM,SAAS,CAAC;AAuCxE;;;;;GAKG;AACH,MAAM,CAAC,OAAO,UAAU,SAAS,CAAC,MAAM,EAAE,MAAM,GAAG,CAAC,CAAC,KAAK,EAAE,GAAG,KAAK,OAAO,CAAC,IAAI,CAAC,EAAE,OAAO,CAAC,CA6C1F"}
+{"version":3,"file":"use-action.d.ts","sourceRoot":"","sources":["../../../js/shared/utils/use-action.tsx"],"names":[],"mappings":"AAcA,OAAO,EAAE,MAAM,EAAiB,MAAM,SAAS,CAAC;AAChD,OAAO,EAAE,aAAa,EAAa,UAAU,EAAmB,MAAM,cAAc,CAAC;AAoOrF,UAAU,gBAAgB;IACtB;;;;;OAKG;IACH,iBAAiB,CAAC,EAAE,CAAC,MAAM,EAAE,UAAU,EAAE,SAAS,EAAE,aAAa,KAAK,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;CAC9F;AAED;;;;;;GAMG;AACH,MAAM,CAAC,OAAO,UAAU,SAAS,CAC7B,MAAM,EAAE,MAAM,EACd,OAAO,CAAC,EAAE,gBAAgB,GAC3B,CAAC,CAAC,KAAK,CAAC,EAAE,GAAG,KAAK,OAAO,CAAC,IAAI,CAAC,EAAE,OAAO,CAAC,CA4F3C"}

package/dist/shared/utils/use-action.js

@@ -7,79 +7,273 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
         step((generator = generator.apply(thisArg, _arguments || [])).next());
     });
 };
-import { useCallback, useContext, useState } from 'react';
-import { useAsyncResource } from 'use-async-resource';
-import { useActionContext } from '../../actions/utils';
-import { ImportersCtx } from '../../shared/context';
+import { useContext, useRef, useState } from 'react';
+import { useHistory, useLocation } from 'react-router-dom';
+import { useRecoilCallback } from 'recoil';
+import { concatMap, takeWhile } from 'rxjs/operators';
+import shortid from 'shortid';
+import { useNotifications } from '@darajs/ui-notifications';
+import { HTTP_METHOD, Status, validateResponse } from '@darajs/ui-utils';
+import { fetchTaskResult } from '../../api';
+import { request } from '../../api/http';
+import { useSessionToken } from '../../auth/auth-context';
+import { ImportersCtx, WebSocketCtx, useTaskContext } from '../../shared/context';
+import { isActionImpl } from '../../types/utils';
+import { resolveVariable } from '../interactivity/resolve-variable';
+import { normalizeRequest } from './normalization';
 import useActionRegistry from './use-action-registry';
-// Disabling rules of hook since the followiing function are willingly breaking the rules, making the assumption that the components call
-// the exported functions with values which don't change the hook order etc
-/* eslint-disable react-hooks/rules-of-hooks */
-/* eslint-disable react-hooks/exhaustive-deps */
 /**
- * Helper method which converts an action into a list of hooks.
+ * Invoke a server-side action.
+ * This is used for annotated actions, sends a POST request to the server to start an action execution.
  *
- * @param action action(s) to import
- * @param getAction method to convert an action instance into its definition
+ * @param input input value for the action
+ * @param executionId unique id for the action execution; this should be used to subscribe to action messages **before** calling fetchAction
+ * @param annotatedAction annotated action instance
+ * @param actionCtx action context
+ */
+function invokeAction(input, executionId, annotatedAction, actionCtx) {
+    return __awaiter(this, void 0, void 0, function* () {
+        // resolve kwargs to primitives, this registers variables if not already registered
+        const resolvedKwargs = Object.keys(annotatedAction.dynamic_kwargs).reduce((acc, k) => {
+            const value = annotatedAction.dynamic_kwargs[k];
+            acc[k] = resolveVariable(value, actionCtx.wsClient, actionCtx.taskCtx, actionCtx.location.search, actionCtx.sessionToken, (v) => 
+            // This is only called for primitive variables so it should always resolve successfully
+            // hence not using a promise
+            actionCtx.snapshot.getLoadable(v).getValue());
+            return acc;
+        }, {});
+        const ws_channel = yield actionCtx.wsClient.getChannel();
+        const res = yield request(`/api/core/action/${annotatedAction.definition_uid}`, {
+            body: JSON.stringify({
+                execution_id: executionId,
+                input,
+                uid: annotatedAction.uid,
+                values: normalizeRequest(resolvedKwargs, annotatedAction.dynamic_kwargs),
+                ws_channel,
+            }),
+            method: HTTP_METHOD.POST,
+        }, actionCtx.sessionToken);
+        yield validateResponse(res, `Failed to fetch the action value with uid: ${annotatedAction.uid}`);
+        const resContent = yield res.json();
+        // for tasks, wait for it to finish and fetch the result
+        // this is only used to pick up errors in the task execution
+        if ('task_id' in resContent) {
+            const taskId = resContent.task_id;
+            actionCtx.taskCtx.startTask(taskId);
+            yield actionCtx.wsClient.waitForTask(taskId);
+            actionCtx.taskCtx.endTask(taskId);
+            // We don't need the result as the MetaTask will send WS messages with actions as per usual
+            // fetchTaskResult will pick up on errors and raise them
+            yield fetchTaskResult(taskId, actionCtx.sessionToken);
+        }
+    });
+}
+/**
+ * Global cache of action handlers by name. Used for perf so each action will only have to be
+ * resolved once.
+ */
+const ACTION_HANDLER_BY_NAME = {};
+/**
+ * Error thrown when an action is not handled by the app.
+ */
+class UnhandledActionError extends Error {
+    constructor(message, actionImpl) {
+        super(message);
+        this.actionImpl = actionImpl;
+    }
+}
+/**
+ * Resolve an action implementation into an action handler function.
+ *
+ * @param actionImpl action implementation
+ * @param getAction callback to get the action definition from the action implementation
  * @param importers importers available in the app
  */
-function getActionHooks(action, getAction, importers) {
+function resolveActionImpl(actionImpl, getAction, importers) {
     return __awaiter(this, void 0, void 0, function* () {
-        const hooks = [];
-        const actions = Array.isArray(action) ? action : [action];
-        for (const act of actions) {
-            const actDef = getAction(act);
-            // Doing this sequentially to preserve order and not break rules of hooks
-            // eslint-disable-next-line no-await-in-loop
-            const moduleContent = yield importers[actDef.py_module]();
-            const Hook = moduleContent[actDef.name];
-            hooks.push(Hook);
+        // cache action handler globally for performance, they are pure functions so it's safe to do so
+        if (!ACTION_HANDLER_BY_NAME[actionImpl.name]) {
+            // resolve action handler function by name
+            let actionDef;
+            try {
+                actionDef = getAction(actionImpl);
+            }
+            catch (_a) {
+                throw new UnhandledActionError(`Action definition for impl "${actionImpl.name}" not found`, actionImpl);
+            }
+            const moduleContent = yield importers[actionDef.py_module]();
+            ACTION_HANDLER_BY_NAME[actionImpl.name] = moduleContent[actionImpl.name];
         }
-        return hooks;
+        return ACTION_HANDLER_BY_NAME[actionImpl.name];
     });
 }
+/**
+ * Execute a given action.
+ *
+ * @param input component input value
+ * @param action action implementation or annotated action
+ * @param actionCtx action execution context
+ * @param getAction callback to get the action definition from the action implementation
+ * @param importers available importers in the app
+ */
+function executeAction(input, action, actionCtx, getAction, importers) {
+    return __awaiter(this, void 0, void 0, function* () {
+        // if it's a simple action implementation, run the handler directly
+        if (isActionImpl(action)) {
+            const handler = yield resolveActionImpl(action, getAction, importers);
+            const result = handler(actionCtx, action);
+            // handle async handlers
+            if (result instanceof Promise) {
+                yield result;
+            }
+            return;
+        }
+        // otherwise call back to the server to execute the annotated action
+        // subscribe to action messages before even calling the action to avoid races
+        const executionId = shortid.generate();
+        const observable = actionCtx.wsClient.actionMessages$(executionId);
+        return new Promise((resolve, reject) => {
+            let activeTasks = 0; // Counter to keep track of active tasks
+            let streamCompleted = false; // Flag to check if the stream is completed
+            let isSettled = false;
+            let sub;
+            const checkForCompletion = () => {
+                if (!isSettled && streamCompleted && activeTasks === 0) {
+                    isSettled = true;
+                    sub.unsubscribe();
+                    resolve();
+                }
+            };
+            const onError = (error) => {
+                // eslint-disable-next-line no-console
+                console.error('Error executing action:', error);
+                if (!isSettled) {
+                    isSettled = true;
+                    sub.unsubscribe();
+                    reject(error);
+                }
+            };
+            sub = observable
+                .pipe(concatMap((actionImpl) => __awaiter(this, void 0, void 0, function* () {
+                if (actionImpl) {
+                    const handler = yield resolveActionImpl(actionImpl, getAction, importers);
+                    return [handler, actionImpl];
+                }
+                return null;
+            })), takeWhile((res) => !!res) // stop when falsy is returned from concatMap
+            )
+                .subscribe({
+                complete: () => {
+                    streamCompleted = true; // Set the flag when the stream is complete
+                    checkForCompletion();
+                },
+                error: onError,
+                next: ([handler, actionImpl]) => __awaiter(this, void 0, void 0, function* () {
+                    try {
+                        activeTasks += 1;
+                        const result = handler(actionCtx, actionImpl);
+                        // If it's a promise, await it to ensure sequential execution
+                        if (result instanceof Promise) {
+                            yield result;
+                        }
+                    }
+                    catch (error) {
+                        onError(error);
+                    }
+                    finally {
+                        if (activeTasks > 0) {
+                            activeTasks -= 1;
+                        }
+                        checkForCompletion();
+                    }
+                }),
+            });
+            // now request the action to be executed
+            invokeAction(input, executionId, action, actionCtx).catch(onError);
+        });
+    });
+}
+const noop = () => Promise.resolve();
 /**
  * Consume an action def from the framework and return a callback function that can be used to trigger it
  * and a loading state boolean indicating whether an action is currently in progress
  *
- * @param actionDefinition the action def passed in
+ * @param action the action passed in
+ * @param options optional extra options for the hook
  */
-export default function useAction(action) {
-    // Sanity check - it's technically possible to pass an ActionInstance or empty array and pydantic accepts them and converts to an ActionInstance
-    if ((action === null || action === void 0 ? void 0 : action.name) === 'ActionInstance') {
-        throw new Error('Expected a registered sub-class of "ActionInstance", base class detected');
-    }
-    // No action passed - noop
-    if (!action || (Array.isArray(action) && action.length === 0)) {
-        return [() => Promise.resolve(), false];
-    }
-    const [loading, setLoading] = useState(false);
+export default function useAction(action, options) {
+    const { client: wsClient } = useContext(WebSocketCtx);
     const importers = useContext(ImportersCtx);
-    const actionContext = useActionContext();
     const { get: getAction } = useActionRegistry();
-    const [actionHooksReader] = useAsyncResource(getActionHooks, action, getAction, importers);
-    // Call all action hooks and collect the callbacks
-    const actionHooks = actionHooksReader();
-    const actionCallbacks = [];
-    const actions = Array.isArray(action) ? action : [action];
-    for (const [index, actionHook] of actionHooks.entries()) {
-        actionCallbacks.push(actionHook(actions[index], actionContext));
-    }
-    const actionCallback = useCallback((value) => __awaiter(this, void 0, void 0, function* () {
-        // Track loading state for all actions
-        setLoading(true);
-        try {
-            for (const actionCb of actionCallbacks) {
-                // Sequentially execute actions to ensure deterministic order of execution
+    const notificationCtx = useNotifications();
+    const sessionToken = useSessionToken();
+    const history = useHistory();
+    const taskCtx = useTaskContext();
+    const location = useLocation();
+    const [isLoading, setIsLoading] = useState(false);
+    // keep actionCtx in a ref to avoid re-creating the callbacks
+    const actionCtx = useRef();
+    actionCtx.current = {
+        history,
+        location,
+        notificationCtx,
+        sessionToken,
+        taskCtx,
+        wsClient,
+    };
+    const optionsRef = useRef(options);
+    optionsRef.current = options;
+    const callback = useRecoilCallback((cbInterface) => (input) => __awaiter(this, void 0, void 0, function* () {
+        var _a;
+        setIsLoading(true);
+        const actionsToExecute = Array.isArray(action) ? action : [action];
+        // execute actions sequentially
+        for (const actionToExecute of actionsToExecute) {
+            // this is redefined for each action to have up-to-date snapshot
+            /* eslint-disable sort-keys-fix/sort-keys-fix */
+            const fullActionContext = Object.assign(Object.assign({}, actionCtx.current), { input, 
+                // Recoil callback interface cannot be spread as it is a Proxy
+                gotoSnapshot: cbInterface.gotoSnapshot, refresh: cbInterface.refresh, reset: cbInterface.reset, set: cbInterface.set, snapshot: cbInterface.snapshot, transact_UNSTABLE: cbInterface.transact_UNSTABLE });
+            /* eslint-enable sort-keys-fix/sort-keys-fix */
+            try {
                 // eslint-disable-next-line no-await-in-loop
-                yield actionCb(value);
+                yield executeAction(input, actionToExecute, fullActionContext, getAction, importers);
+            }
+            catch (error) {
+                // Handle unhandled action errors separately
+                if (error instanceof UnhandledActionError) {
+                    // there is a callback defined for it, call it
+                    if ((_a = optionsRef.current) === null || _a === void 0 ? void 0 : _a.onUnhandledAction) {
+                        const result = optionsRef.current.onUnhandledAction(error.actionImpl, fullActionContext);
+                        if (result instanceof Promise) {
+                            // eslint-disable-next-line no-await-in-loop
+                            yield result;
+                        }
+                    }
+                    else {
+                        actionCtx.current.notificationCtx.pushNotification({
+                            key: '_actionError',
+                            message: `Action "${error.actionImpl.name}" not registered`,
+                            status: Status.ERROR,
+                            title: 'Error executing action',
+                        });
+                    }
+                    continue;
+                }
+                actionCtx.current.notificationCtx.pushNotification({
+                    key: '_actionError',
+                    message: 'Try again or contact the application owner',
+                    status: Status.ERROR,
+                    title: 'Error executing action',
+                });
             }
         }
-        finally {
-            // make sure to clean up loading state regardless of errors
-            setLoading(false);
-        }
-    }), [action, ...actionCallbacks]);
-    return [actionCallback, loading];
+        setIsLoading(false);
+    }), [action, getAction, importers]);
+    // return a noop if no action is passed
+    if (!action) {
+        return [noop, false];
+    }
+    return [callback, isLoading];
 }
 //# sourceMappingURL=use-action.js.map