| @@ -1,24 +1,12 @@ | ||
| 1 | 1 | import { isAbilityWorkflow } from '@agent/lib/abilities'; |
| 2 | -import { isChangeSiteDesignWorkflowAvailable } from '@agent/lib/util'; | |
| 3 | -import { AbilityRunGate } from '@agent/workflows/abilities/components/AbilityRunGate'; | |
| 4 | -import changeSiteDesignWorkflow from '@agent/workflows/theme/change-site-design'; | |
| 5 | -import variationsWorkflow from '@agent/workflows/theme/change-theme-variation'; | |
| 6 | -import { workflows } from '@agent/workflows/workflows'; | |
| 2 | +import { AbilityRun } from '@agent/workflows/abilities/components/run'; | |
| 3 | +import { abilityWorkflows, workflows } from '@agent/workflows/workflows'; | |
| 7 | 4 | import { useQuickEditStore } from '@quick-edit/state/store'; |
| 8 | 5 | import { deepMerge } from '@shared/lib/utils'; |
| 9 | 6 | import { create } from 'zustand'; |
| 10 | 7 | import { devtools, persist } from 'zustand/middleware'; |
| 11 | 8 | |
| 12 | -const onboarding = window.extAgentData.chatHistory?.length === 0; | |
| 13 | -const agentResponse = isChangeSiteDesignWorkflowAvailable() | |
| 14 | - ? changeSiteDesignWorkflow.example?.agentResponse | |
| 15 | - : variationsWorkflow.example?.agentResponse; | |
| 16 | -const onboardingToolProps = { | |
| 17 | - ...agentResponse.whenFinishedTool, | |
| 18 | - agentResponse, | |
| 19 | -}; | |
| 20 | - | |
| 21 | 9 | // Used to check case-insensitive matches for workflow examples |
| 22 | 10 | const collator = new Intl.Collator(undefined, { |
| 23 | 11 | sensitivity: 'base', |
| 24 | 12 | usage: 'search', |
| @@ -27,8 +15,10 @@ | ||
| 27 | 15 | const state = (set, get) => ({ |
| 28 | 16 | workflow: null, |
| 29 | 17 | // Data for the tool component that shows up at the end of a workflow |
| 30 | 18 | whenFinishedToolProps: null, |
| 19 | + // A staged tool restored from storage; its confirm never renders. | |
| 20 | + reloadedToolProps: null, | |
| 31 | 21 | getWorkflow: () => { |
| 32 | 22 | const curr = get().workflow; |
| 33 | 23 | // Workflows may define a "parent" workflow via templateId |
| 34 | 24 | const currId = curr?.templateId || curr?.id; |
| @@ -34,13 +24,15 @@ | ||
| 34 | 24 | const currId = curr?.templateId || curr?.id; |
| 35 | 25 | const wf = workflows.find(({ id }) => id === currId); |
| 36 | 26 | if (!wf?.id) { |
| 37 | 27 | if (!curr) return null; |
| 38 | - // Ability workflows are shaped per request, so they carry no static | |
| 39 | - // whenFinished component; give them the generic run gate. | |
| 40 | - return isAbilityWorkflow(curr.id) | |
| 41 | - ? { ...curr, whenFinished: { component: AbilityRunGate } } | |
| 42 | - : curr; | |
| 28 | + if (!isAbilityWorkflow(curr.id)) return curr; | |
| 29 | + // The backend never sends whenFinished, so this supplies the gate. | |
| 30 | + const ability = deepMerge( | |
| 31 | + { whenFinished: { component: AbilityRun } }, | |
| 32 | + abilityWorkflows[curr.id] ?? {}, | |
| 33 | + ); | |
| 34 | + return { ...deepMerge(curr, ability), id: curr.id }; | |
| 43 | 35 | } |
| 44 | 36 | return { ...deepMerge(curr, wf || {}), id: curr?.id }; |
| 45 | 37 | }, |
| 46 | 38 | getWorkflowByExample: (example) => { |
| @@ -47,8 +39,9 @@ | ||
| 47 | 39 | const wf = workflows.find( |
| 48 | 40 | ({ example: ex }) => collator.compare(ex?.text, example) === 0, |
| 49 | 41 | ); |
| 50 | 42 | if (!wf?.id) return null; |
| 43 | + if (!wf.available()) return null; | |
| 51 | 44 | return wf; |
| 52 | 45 | }, |
| 53 | 46 | // Gets the workflows available to the user |
| 54 | 47 | // TODO: maybe we need to have a way to include a |
| @@ -82,20 +75,41 @@ | ||
| 82 | 75 | }); |
| 83 | 76 | }, |
| 84 | 77 | setWorkflow: (workflow) => { |
| 85 | 78 | const agentBlockCode = useQuickEditStore.getState().agentBlockCode; |
| 79 | + const started = workflow | |
| 80 | + ? { ...workflow, startingPage: window.location.href } | |
| 81 | + : null; | |
| 86 | 82 | set({ |
| 87 | - workflow: workflow | |
| 88 | - ? { ...workflow, startingPage: window.location.href } | |
| 89 | - : null, | |
| 83 | + workflow: started, | |
| 90 | 84 | // If a block is selected, add it to the workflow data |
| 91 | 85 | // previousContent is named this way for legacy reasons |
| 92 | 86 | workflowData: agentBlockCode ? { previousContent: agentBlockCode } : null, |
| 93 | 87 | whenFinishedToolProps: null, |
| 94 | 88 | }); |
| 89 | + if (!started) return; | |
| 90 | + const workflowStarted = get().getWorkflow(); | |
| 91 | + window.dispatchEvent( | |
| 92 | + new CustomEvent('extendify-workflow::started', { | |
| 93 | + detail: workflowStarted, | |
| 94 | + }), | |
| 95 | + ); | |
| 96 | + try { | |
| 97 | + workflowStarted?.onStarted?.(workflowStarted); | |
| 98 | + } catch { | |
| 99 | + // A workflow that can't read the page still has to start. | |
| 100 | + } | |
| 95 | 101 | }, |
| 96 | 102 | setWhenFinishedToolProps: (whenFinishedToolProps) => |
| 97 | 103 | set({ whenFinishedToolProps }), |
| 104 | + // Declared up front it would hide the workflow while nothing is staged. | |
| 105 | + // Not setWorkflow: that resets workflowData and drops the run's data. | |
| 106 | + requireBlock: () => | |
| 107 | + set((state) => | |
| 108 | + state.workflow | |
| 109 | + ? { workflow: { ...state.workflow, requires: ['block'] } } | |
| 110 | + : {}, | |
| 111 | + ), | |
| 98 | 112 | }); |
| 99 | 113 | |
| 100 | 114 | export const useWorkflowStore = create()( |
| 101 | 115 | persist(devtools(state, { name: 'Extendify Agent Workflows' }), { |
| @@ -100,19 +114,23 @@ | ||
| 100 | 114 | export const useWorkflowStore = create()( |
| 101 | 115 | persist(devtools(state, { name: 'Extendify Agent Workflows' }), { |
| 102 | 116 | name: `extendify-agent-workflows-${window.extSharedData.siteId}`, |
| 103 | 117 | merge: (persistedState, currentState) => { |
| 104 | - // if we are in onboarding mode, add the starting workflow | |
| 105 | - if (onboarding) { | |
| 118 | + const merged = { ...currentState, ...persistedState }; | |
| 119 | + // A reload can't carry the staged block a block-patching workflow | |
| 120 | + // depends on, so a rehydrated-open one is always stale. A canvas | |
| 121 | + // session is stale too: its field values never persist. | |
| 122 | + const id = merged.workflow?.templateId || merged.workflow?.id; | |
| 123 | + const registered = workflows.find((wf) => wf.id === id); | |
| 124 | + if (id === 'block-patching' || registered?.whenFinished?.canvas) { | |
| 106 | 125 | return { |
| 107 | - ...currentState, | |
| 108 | - ...persistedState, | |
| 109 | - workflow: isChangeSiteDesignWorkflowAvailable() | |
| 110 | - ? changeSiteDesignWorkflow | |
| 111 | - : variationsWorkflow, | |
| 112 | - whenFinishedToolProps: onboardingToolProps, | |
| 126 | + ...merged, | |
| 127 | + workflow: null, | |
| 128 | + whenFinishedToolProps: null, | |
| 129 | + workflowData: null, | |
| 113 | 130 | }; |
| 114 | 131 | } |
| 115 | - return { ...currentState, ...persistedState }; | |
| 132 | + return { ...merged, reloadedToolProps: merged.whenFinishedToolProps }; | |
| 116 | 133 | }, |
| 134 | + partialize: ({ reloadedToolProps, ...persisted }) => persisted, | |
| 117 | 135 | }), |
| 118 | 136 | ); |
