@zeniai/client-epic-state 5.2.4 → 5.2.5

package/lib/epic.js

@@ -53,10 +53,16 @@ const acceptMasterTOSEpic_1 = require("./view/aiCfoView/epics/acceptMasterTOSEpi
 const createSessionAndSubmitEpic_1 = require("./view/aiCfoView/epics/createSessionAndSubmitEpic");
 const createSessionEpic_1 = require("./view/aiCfoView/epics/createSessionEpic");
 const deleteChatSessionEpic_1 = require("./view/aiCfoView/epics/deleteChatSessionEpic");
+const deleteSkillEpic_1 = require("./view/aiCfoView/epics/deleteSkillEpic");
 const fetchChatHistoryEpic_1 = require("./view/aiCfoView/epics/fetchChatHistoryEpic");
 const fetchChatSessionsForUserEpic_1 = require("./view/aiCfoView/epics/fetchChatSessionsForUserEpic");
+const fetchSkillDetailEpic_1 = require("./view/aiCfoView/epics/fetchSkillDetailEpic");
+const fetchSkillVersionBodyEpic_1 = require("./view/aiCfoView/epics/fetchSkillVersionBodyEpic");
+const fetchSkillVersionsEpic_1 = require("./view/aiCfoView/epics/fetchSkillVersionsEpic");
 const fetchSkillsEpic_1 = require("./view/aiCfoView/epics/fetchSkillsEpic");
 const fetchSuggestedQuestionsEpic_1 = require("./view/aiCfoView/epics/fetchSuggestedQuestionsEpic");
+const saveSkillEpic_1 = require("./view/aiCfoView/epics/saveSkillEpic");
+const setSkillVisibilityEpic_1 = require("./view/aiCfoView/epics/setSkillVisibilityEpic");
 const stopSubmitEpic_1 = require("./view/aiCfoView/epics/stopSubmitEpic");
 const stopSubmitQuestionEpic_1 = require("./view/aiCfoView/epics/stopSubmitQuestionEpic");
 const submitFeedbackEpic_1 = require("./view/aiCfoView/epics/submitFeedbackEpic");
@@ -802,6 +808,7 @@ const ALL_EPICS = [
     deleteRemiEpic_1.deleteRemiEpic,
     deleteScheduleAccruedDetailEpic_1.deleteScheduleAccruedDetailEpic,
     deleteScheduleDetailEpic_1.deleteScheduleDetailEpic,
+    deleteSkillEpic_1.deleteSkillEpic,
     deleteTagEpic_1.deleteTagEpic,
     deleteTaskEpic_1.deleteTaskEpic,
     deleteTaskGroupEpic_1.deleteTaskGroupEpic,
@@ -1118,7 +1125,10 @@ const ALL_EPICS = [
     fetchScheduleDetailsPageEpic_1.fetchScheduleDetailsPageEpic,
     fetchScheduleListEpic_1.fetchScheduleListEpic,
     fetchSchedulesAccountEpic_1.fetchSchedulesAccountEpic,
+    fetchSkillDetailEpic_1.fetchSkillDetailEpic,
     fetchSkillsEpic_1.fetchSkillsEpic,
+    fetchSkillVersionBodyEpic_1.fetchSkillVersionBodyEpic,
+    fetchSkillVersionsEpic_1.fetchSkillVersionsEpic,
     fetchSubscriptionAddOnsEpic_1.fetchSubscriptionAddOnsEpic,
     fetchSubscriptionCouponsEpic_1.fetchSubscriptionCouponsEpic,
     fetchSubscriptionCreateEstimateEpic_1.fetchSubscriptionCreateEstimateEpic,
@@ -1319,6 +1329,7 @@ const ALL_EPICS = [
     saveRemiSetupApproverViewUpdatesEpic_1.saveRemiSetupApproverViewUpdatesEpic,
     saveScheduleAccruedDetailsEpic_1.saveScheduleAccruedDetailsEpic,
     saveScheduleDetailsEpic_1.saveScheduleDetailsEpic,
+    saveSkillEpic_1.saveSkillEpic,
     saveSubscriptionNotesUpdatesEpic_1.saveSubscriptionNotesUpdatesEpic,
     saveSubscriptionUpdatesEpic_1.saveSubscriptionUpdatesEpic,
     saveTaskDetailEpic_1.saveTaskDetailEpic,
@@ -1339,6 +1350,7 @@ const ALL_EPICS = [
     sendOtpEpic_1.sendOtpEpic,
     sendReferralInviteEpic_1.sendReferralInviteEpic,
     sessionHeartbeatEpic_1.sessionHeartbeatEpic,
+    setSkillVisibilityEpic_1.setSkillVisibilityEpic,
     snoozeTaskEpic_1.snoozeTaskEpic,
     startInvoicingQboBackfillEpic_1.startInvoicingQboBackfillEpic,
     statementCloseDayEpic_1.statementCloseDayEpic,

package/lib/esm/epic.js

@@ -50,10 +50,16 @@ import { acceptMasterTOSEpic, } from './view/aiCfoView/epics/acceptMasterTOSEpic
 import { createSessionAndSubmitEpic, } from './view/aiCfoView/epics/createSessionAndSubmitEpic';
 import { createSessionEpic, } from './view/aiCfoView/epics/createSessionEpic';
 import { deleteChatSessionEpic, } from './view/aiCfoView/epics/deleteChatSessionEpic';
+import { deleteSkillEpic } from './view/aiCfoView/epics/deleteSkillEpic';
 import { fetchChatHistoryEpic, } from './view/aiCfoView/epics/fetchChatHistoryEpic';
 import { fetchChatSessionsForUserEpic, } from './view/aiCfoView/epics/fetchChatSessionsForUserEpic';
+import { fetchSkillDetailEpic } from './view/aiCfoView/epics/fetchSkillDetailEpic';
+import { fetchSkillVersionBodyEpic } from './view/aiCfoView/epics/fetchSkillVersionBodyEpic';
+import { fetchSkillVersionsEpic } from './view/aiCfoView/epics/fetchSkillVersionsEpic';
 import { fetchSkillsEpic, } from './view/aiCfoView/epics/fetchSkillsEpic';
 import { fetchSuggestedQuestionsEpic, } from './view/aiCfoView/epics/fetchSuggestedQuestionsEpic';
+import { saveSkillEpic } from './view/aiCfoView/epics/saveSkillEpic';
+import { setSkillVisibilityEpic } from './view/aiCfoView/epics/setSkillVisibilityEpic';
 import { stopSubmitEpic, } from './view/aiCfoView/epics/stopSubmitEpic';
 import { stopSubmitQuestionEpic, } from './view/aiCfoView/epics/stopSubmitQuestionEpic';
 import { submitFeedbackEpic, } from './view/aiCfoView/epics/submitFeedbackEpic';
@@ -799,6 +805,7 @@ const ALL_EPICS = [
     deleteRemiEpic,
     deleteScheduleAccruedDetailEpic,
     deleteScheduleDetailEpic,
+    deleteSkillEpic,
     deleteTagEpic,
     deleteTaskEpic,
     deleteTaskGroupEpic,
@@ -1115,7 +1122,10 @@ const ALL_EPICS = [
     fetchScheduleDetailsPageEpic,
     fetchScheduleListEpic,
     fetchSchedulesAccountEpic,
+    fetchSkillDetailEpic,
     fetchSkillsEpic,
+    fetchSkillVersionBodyEpic,
+    fetchSkillVersionsEpic,
     fetchSubscriptionAddOnsEpic,
     fetchSubscriptionCouponsEpic,
     fetchSubscriptionCreateEstimateEpic,
@@ -1316,6 +1326,7 @@ const ALL_EPICS = [
     saveRemiSetupApproverViewUpdatesEpic,
     saveScheduleAccruedDetailsEpic,
     saveScheduleDetailsEpic,
+    saveSkillEpic,
     saveSubscriptionNotesUpdatesEpic,
     saveSubscriptionUpdatesEpic,
     saveTaskDetailEpic,
@@ -1336,6 +1347,7 @@ const ALL_EPICS = [
     sendOtpEpic,
     sendReferralInviteEpic,
     sessionHeartbeatEpic,
+    setSkillVisibilityEpic,
     snoozeTaskEpic,
     startInvoicingQboBackfillEpic,
     statementCloseDayEpic,

package/lib/esm/index.js

@@ -683,8 +683,8 @@ export { updateAiAccountantCustomers, updateAiAccountantCustomer, updateAiAccoun
 export { getAiAccountantCustomers, getAiAccountantJobsByTenantId, } from './entity/aiAccountantCustomer/aiAccountantCustomerSelector';
 export { cancelAiAccountantOnboarding, fetchAiAccountantCustomers, fetchAiAccountantJobs, setSelectedTenantIdsForJobTrigger, triggerAiAccountantJob, updateAiAccountantUIState, } from './view/aiAccountantView/aiAccountantViewReducer';
 export { getAiAccountantCockpitView, } from './view/aiAccountantView/aiAccountantViewSelector';
-export { fetchChatSessionsForUser, createSession, submitQuestion, updateAiCfoViewScrollPosition, updateCurrentInput, updateCotCollapsedState, setSession, clearCurrentSessionId, clearDeleteChatSessionStatus, clearLastContextMessage, createSessionAndSubmit, stopSubmit, stopSubmitQuestion, fetchChatHistory, acceptMasterTOS, deleteChatSession, submitFeedback, updateResponseState, fetchSuggestedQuestions, fetchSuggestedQuestionsSuccess, fetchSuggestedQuestionsFailure, fetchSkills, fetchSkillsSuccess, fetchSkillsFailure, applyAiCfoSidePanelHostPageTransition, clearAiCfoSidePanelHostPageContext, } from './view/aiCfoView/aiCfoViewReducer';
-export { getAiCfoView, getSuggestedQuestionsForPageContext, getSkills, } from './view/aiCfoView/aiCfoViewSelector';
+export { fetchChatSessionsForUser, createSession, submitQuestion, updateAiCfoViewScrollPosition, updateCurrentInput, updateCotCollapsedState, setSession, clearCurrentSessionId, clearDeleteChatSessionStatus, clearLastContextMessage, createSessionAndSubmit, stopSubmit, stopSubmitQuestion, fetchChatHistory, acceptMasterTOS, deleteChatSession, submitFeedback, updateResponseState, fetchSuggestedQuestions, fetchSuggestedQuestionsSuccess, fetchSuggestedQuestionsFailure, fetchSkills, fetchSkillsSuccess, fetchSkillsFailure, fetchSkillDetail, clearSkillDetail, clearSkillMutation, deleteSkill, fetchSkillVersionBody, fetchSkillVersions, saveSkill, setSkillVisibility, openSkillsBrowser, closeSkillsBrowser, applyAiCfoSidePanelHostPageTransition, clearAiCfoSidePanelHostPageContext, } from './view/aiCfoView/aiCfoViewReducer';
+export { getAiCfoView, getSuggestedQuestionsForPageContext, getSkills, getSkillDetail, getSkillMutation, getSkillVersionBody, getSkillVersions, getSkillsEnabled, } from './view/aiCfoView/aiCfoViewSelector';
 export { setNewSession, setSessions, updateAiCfoAnswerCardPolicyWizardPlan, upsertAnswerPayload, upsertOrAddQuestionAnswerPayload, addQuestionPayload, clearSession, toAiCfoExports, toAiCfoVisualization, appendSyntheticAiCfoAnswer, updateSyntheticAiCfoAnswer, removeSyntheticAiCfoAnswer, } from './entity/aiCfo/aiCfoReducer';
 export { getAllQuestionAnswersForChatSession, getQuestionAnswerByIdForChatSession, getAllQuestionsForChatSession, getAiCfoSelectorView, getSyntheticAiCfoAnswersForChatSession, getSyntheticAiCfoAnswerByQuestionAnswerId, } from './entity/aiCfo/aiCfoSelector';
 export { ALL_AI_CFO_CHARTS_TYPES, ALL_AI_CFO_VISUALIZATION_TYPES, ALL_AI_CFO_ANSWER_STATE_TYPES, ALL_AI_CFO_ANSWER_RESPONSE_TYPES, ALL_INTERACTIVE_FORM_TYPES, toAiCfoVisualizationType, toAiCfoVisualizationTypeStrict, toAiCfoChartType, toAiCfoChartTypeStrict, toAiCfoAnswerStateType, toAiCfoAnswerStateTypeStrict, toAiCfoAnswerResponseType, toAiCfoAnswerResponseTypeStrict, toInteractiveFormType, toInteractiveFormTypeStrict, ALL_SYNTHETIC_AI_CFO_ANSWER_KINDS, toMessageSender, toMessageType, } from './entity/aiCfo/aiCfoState';

package/lib/esm/view/aiCfoView/aiCfoViewReducer.js

@@ -48,8 +48,81 @@ export const initialAiCfoViewState = {
     responseStateByChatSessionId: {},
     stopSubmitStateByChatSessionId: {},
     suggestedQuestionsByPageContext: {},
-    skills: { fetchState: 'Not-Started', error: undefined, skills: [] },
+    skillDetail: {
+        fetchState: 'Not-Started',
+        error: undefined,
+        detail: undefined,
+        macroId: undefined,
+    },
+    skillMutation: {
+        fetchState: 'Not-Started',
+        error: undefined,
+        savedMacroId: undefined,
+    },
+    skillVersionBody: {
+        fetchState: 'Not-Started',
+        error: undefined,
+        instructions: undefined,
+        macroId: undefined,
+        version: undefined,
+    },
+    skillVersions: {
+        fetchState: 'Not-Started',
+        error: undefined,
+        versions: [],
+        macroId: undefined,
+    },
+    skills: {
+        fetchState: 'Not-Started',
+        error: undefined,
+        deletedMacroIds: [],
+        skills: [],
+        skillsEnabled: false,
+        canPublish: false,
+    },
 };
+/**
+ * Forget the skill that was open — prompt AND history together.
+ *
+ * Both, always. They are two slices describing one selection, so clearing only the
+ * prompt leaves `skillVersions.macroId` matching: an in-flight history response then
+ * passes its own stale-response guard and renders the previous skill's versions under
+ * whatever the user opened next.
+ */
+function clearOpenSkillState(draft) {
+    draft.skillDetail = {
+        fetchState: 'Not-Started',
+        error: undefined,
+        detail: undefined,
+        macroId: undefined,
+    };
+    draft.skillVersions = {
+        fetchState: 'Not-Started',
+        error: undefined,
+        versions: [],
+        macroId: undefined,
+    };
+    // Part of the same selection: a pinned body left behind would seed a restore form for
+    // a skill the user has left.
+    draft.skillVersionBody = {
+        fetchState: 'Not-Started',
+        error: undefined,
+        instructions: undefined,
+        macroId: undefined,
+        version: undefined,
+    };
+}
+/**
+ * Take the Skills browser down and drop the prompt it had open.
+ *
+ * Both halves together, always. Closing without clearing leaves the next open landing on
+ * the previous skill instead of the list, and leaves a `macroId` that makes an in-flight
+ * detail response look current to the stale-response guard.
+ */
+function closeSkillsBrowserState(draft) {
+    draft.uiState.isSkillsBrowserOpen = false;
+    clearOpenSkillState(draft);
+}
 const aiCfoView = createSlice({
     name: 'aiCfoView',
     initialState: initialAiCfoViewState,
@@ -57,6 +130,16 @@ const aiCfoView = createSlice({
         submitQuestion: {
             reducer(draft, action) {
                 const { sessionId } = action.payload;
+                // A turn is being sent, so the chat body has to be what the user sees. Doing this
+                // here rather than at each call site is deliberate: the composer, the sample
+                // chips, Run, and the `/` menu all submit, and every one of them was a separate
+                // chance to forget and leave the answer hidden behind the browser.
+                //
+                // Guarded so a submit with the browser already closed — i.e. almost every submit
+                // — does not produce a new `uiState` for every subscriber to re-render on.
+                if (draft.uiState.isSkillsBrowserOpen === true) {
+                    closeSkillsBrowserState(draft);
+                }
                 draft.responseStateByChatSessionId[sessionId] = {
                     thought: { fetchState: 'In-Progress', error: undefined },
                     analysis: { fetchState: 'In-Progress', error: undefined },
@@ -183,6 +266,13 @@ const aiCfoView = createSlice({
         setSession(draft, action) {
             draft.uiState.currentChatSessionId = action.payload.sessionId;
             draft.uiState.agentId = action.payload.agentId;
+            // Picking a session means wanting to see that chat, so the browser cannot stay over
+            // it. In the reducer because there are six dispatch sites across three separate
+            // rails, and covering them one at a time missed half of them.
+            if (action.payload.isRestore !== true &&
+                draft.uiState.isSkillsBrowserOpen === true) {
+                closeSkillsBrowserState(draft);
+            }
             mirrorCurrentSessionToPageKeyMapIfActive(draft);
         },
         createSession: {
@@ -202,6 +292,12 @@ const aiCfoView = createSlice({
             draft.uiState.agentId = agentId;
             draft.createSessionState.fetchState = 'Completed';
             draft.createSessionState.error = undefined;
+            // Same invariant as submitQuestion and initializeNewSessionState: a new chat exists
+            // and is now current, so the browser cannot stay over it. This is the fifth entry
+            // point to that rule — the others are submit, new-session, New Chat and setSession.
+            if (draft.uiState.isSkillsBrowserOpen === true) {
+                closeSkillsBrowserState(draft);
+            }
             draft.uiState.currentChatSessionId = sessionId;
             draft.uiState.lastContextMessage = contextMessage;
             draft.uiState.lastContextStatus = contextStatus;
@@ -243,6 +339,10 @@ const aiCfoView = createSlice({
         },
         initializeNewSessionState(draft, action) {
             const { agentId, sessionId } = action.payload;
+            // Same invariant as submitQuestion — this is the new-session half of submitting.
+            if (draft.uiState.isSkillsBrowserOpen === true) {
+                closeSkillsBrowserState(draft);
+            }
             draft.uiState.currentChatSessionId = sessionId;
             draft.uiState.agentId = agentId;
             draft.uiState.chatHistoryNextPageToken[sessionId] = {
@@ -363,6 +463,10 @@ const aiCfoView = createSlice({
             };
             draft.allChatSessionIds = draft.allChatSessionIds.filter((id) => id !== chatSessionId);
             if (draft.uiState.currentChatSessionId === chatSessionId) {
+                // Deliberately does NOT close the Skills browser, unlike the other writers of
+                // this field. Deleting a chat from the history list is housekeeping: if the
+                // browser is open the user is not looking at that chat, and closing it would
+                // yank them out of what they were doing to reveal an empty chat area.
                 draft.uiState.currentChatSessionId = undefined;
                 draft.uiState.agentId = undefined;
             }
@@ -415,6 +519,10 @@ const aiCfoView = createSlice({
             },
         },
         clearCurrentSessionId(draft) {
+            // New Chat: the empty composer is the point, so the browser cannot stay over it.
+            if (draft.uiState.isSkillsBrowserOpen === true) {
+                closeSkillsBrowserState(draft);
+            }
             draft.uiState.currentChatSessionId = undefined;
             const key = draft.uiState.aiCfoSidePanelHostPageKey;
             if (key != null) {
@@ -431,8 +539,24 @@ const aiCfoView = createSlice({
                 draft.uiState.chatSessionIdByPageKey[prevKey] = curId;
             }
             draft.uiState.aiCfoSidePanelHostPageKey = newAiCfoSidePanelHostPageKey;
-            draft.uiState.currentChatSessionId =
-                nextCurrentChatSessionId != null ? nextCurrentChatSessionId : undefined;
+            const nextId = nextCurrentChatSessionId != null ? nextCurrentChatSessionId : undefined;
+            // The sixth entry point to the same invariant as submitQuestion,
+            // initializeNewSessionState, clearCurrentSessionId, setSession and
+            // createSessionSuccess: a different chat is now visible, so the browser cannot stay
+            // over it. Missing here, this path swapped the session underneath an open browser
+            // and left its macroId looking current to the stale-response guards.
+            //
+            // Only when the session actually CHANGES, and only when there WAS one. A host page
+            // resolving back to the same chat has not changed what is on screen. And
+            // `undefined` becoming a saved id is a hydrate, not a navigation — the same
+            // settle/restore race `setSession` exempts via `isRestore`: opening Skills on a
+            // host page whose chat has not resolved yet would otherwise close the browser
+            // under the user the moment the sessions list arrives.
+            const hadASession = draft.uiState.currentChatSessionId != null;
+            if (hadASession && nextId !== draft.uiState.currentChatSessionId) {
+                closeSkillsBrowserState(draft);
+            }
+            draft.uiState.currentChatSessionId = nextId;
             draft.uiState.currentInput = '';
             mirrorCurrentSessionToPageKeyMapIfActive(draft);
         },
@@ -477,28 +601,306 @@ const aiCfoView = createSlice({
             };
         },
         fetchSkills(draft) {
-            // Optional-chain: persisted/rehydrated state from before this slice existed
-            // may have no `skills`, so don't assume draft.skills is set.
+            // Read straight off `draft.skills` — no optional chain. This store is never
+            // persisted or rehydrated (no redux-persist anywhere in the repo, and no
+            // `preloadedState` in any consumer), and `initialAiCfoViewState` gives every skills
+            // slice a concrete value, so an undefined slice is unreachable here rather than
+            // merely unlikely. An earlier version of this comment claimed the opposite and
+            // justified a guard by a mechanism that does not exist — which then got applied to
+            // some accesses and not others, so the file read as if the slice were sometimes
+            // missing and sometimes not. The one guard that IS meaningful lives in the
+            // selectors, where `state.aiCfoViewState` may genuinely be absent for a host that
+            // composes a partial root reducer; that is a boundary, and this is not.
             draft.skills = {
                 fetchState: 'In-Progress',
                 error: undefined,
-                skills: draft.skills?.skills ?? [],
+                skills: draft.skills.skills,
+                skillsEnabled: draft.skills.skillsEnabled,
+                // Carried for the same reason as skillsEnabled: a refetch in flight is not a
+                // reason to hide the publish control the user was just looking at.
+                canPublish: draft.skills.canPublish,
+                // Carried, not cleared: the response this is about to fetch may still contain a
+                // skill deleted moments ago, which is the whole reason the list is kept.
+                deletedMacroIds: draft.skills.deletedMacroIds,
             };
         },
         fetchSkillsSuccess(draft, action) {
+            // A list response can predate a delete — a save emits fetchSkills, and a delete can
+            // land while that is still outstanding — so replacing the array wholesale would put
+            // the deleted row back and read as the delete having failed.
             draft.skills = {
                 fetchState: 'Completed',
                 error: undefined,
-                skills: action.payload.skills,
+                skills: action.payload.skills.filter((skill) => !draft.skills.deletedMacroIds.includes(skill.macroId)),
+                skillsEnabled: action.payload.skillsEnabled,
+                canPublish: action.payload.canPublish,
+                // Applied ONCE, then forgotten — the tombstone covers a race window, not a
+                // permanent state.
+                //
+                // The previous rule (keep it until a response omits that skill) was wrong in the
+                // case it was written for: nothing refetches after a delete, and a create emits
+                // fetchSkills whose response DOES contain the id, so the tombstone was kept and
+                // reapplied forever. A skill recreated under the same id stayed invisible with no
+                // way to clear it.
+                deletedMacroIds: [],
             };
         },
         fetchSkillsFailure(draft, action) {
             draft.skills = {
                 fetchState: 'Error',
                 error: action.payload,
-                skills: draft.skills?.skills ?? [],
+                skills: draft.skills.skills,
+                // Kept, not reset: a failed refresh is not evidence the feature was turned
+                // off, and dropping it to false would make the Skills entry vanish on any
+                // transient error.
+                skillsEnabled: draft.skills.skillsEnabled,
+                // Same reasoning again: a transient error must not revoke a privilege the server
+                // already granted, or the control flickers away on every failed refresh.
+                canPublish: draft.skills.canPublish,
+                // Same reasoning: a failed refresh is not confirmation the delete was undone.
+                deletedMacroIds: draft.skills.deletedMacroIds,
+            };
+        },
+        fetchSkillDetail(draft, action) {
+            // Opening a skill IS the selection change, so forget the previous one whole —
+            // history included. Resetting only the prompt left `skillVersions.macroId` pointing
+            // at the skill just navigated away from, so a history response still in flight for
+            // it passed its own guard and rendered under the new title. Close, back and delete
+            // all go through the same helper for the same reason.
+            clearOpenSkillState(draft);
+            draft.skillDetail = {
+                fetchState: 'In-Progress',
+                error: undefined,
+                // Cleared rather than kept: holding the previous skill's prompt while a new
+                // one loads would show the wrong instructions under the right title.
+                detail: undefined,
+                macroId: action.payload.macroId,
+            };
+        },
+        fetchSkillDetailSuccess(draft, action) {
+            // Ignore a response for a skill the user has already navigated away from.
+            if (draft.skillDetail.macroId !== action.payload.macroId) {
+                return;
+            }
+            draft.skillDetail = {
+                fetchState: 'Completed',
+                error: undefined,
+                detail: action.payload.detail,
+                macroId: action.payload.macroId,
+            };
+        },
+        fetchSkillDetailFailure(draft, action) {
+            if (draft.skillDetail.macroId !== action.payload.macroId) {
+                return;
+            }
+            draft.skillDetail = {
+                fetchState: 'Error',
+                error: action.payload.error,
+                detail: undefined,
+                macroId: action.payload.macroId,
             };
         },
+        clearSkillDetail(draft) {
+            // Back to the list. History belongs to the detail view, so it goes too.
+            clearOpenSkillState(draft);
+        },
+        fetchSkillVersions: {
+            reducer(draft, action) {
+                draft.skillVersions = {
+                    fetchState: 'In-Progress',
+                    error: undefined,
+                    // Cleared, not kept: showing the previous skill's history under a new title
+                    // is worse than showing none while this loads.
+                    versions: [],
+                    macroId: action.payload.macroId,
+                };
+            },
+            prepare(payload) {
+                return { payload };
+            },
+        },
+        fetchSkillVersionsSuccess(draft, action) {
+            // Same stale-response guard as skillDetail: a switchMap cancels the request but
+            // cannot un-dispatch one already in flight.
+            if (draft.skillVersions.macroId !== action.payload.macroId) {
+                return;
+            }
+            draft.skillVersions.fetchState = 'Completed';
+            draft.skillVersions.error = undefined;
+            draft.skillVersions.versions = action.payload.versions;
+        },
+        fetchSkillVersionsFailure(draft, action) {
+            if (draft.skillVersions.macroId !== action.payload.macroId) {
+                return;
+            }
+            draft.skillVersions.fetchState = 'Error';
+            draft.skillVersions.error = action.payload.error;
+        },
+        fetchSkillVersionBody: {
+            reducer(draft, action) {
+                draft.skillVersionBody = {
+                    fetchState: 'In-Progress',
+                    error: undefined,
+                    // Cleared: seeding a restore form from the PREVIOUS version's body is the same
+                    // class of mistake as showing one skill's prompt under another's title.
+                    instructions: undefined,
+                    macroId: action.payload.macroId,
+                    version: action.payload.version,
+                };
+            },
+            prepare(payload) {
+                return { payload };
+            },
+        },
+        fetchSkillVersionBodySuccess(draft, action) {
+            // Both parts of the key: a user can jump between versions of the same skill, so
+            // macroId alone would let an earlier version's body land on a later request.
+            if (draft.skillVersionBody.macroId !== action.payload.macroId ||
+                draft.skillVersionBody.version !== action.payload.version) {
+                return;
+            }
+            draft.skillVersionBody.fetchState = 'Completed';
+            draft.skillVersionBody.error = undefined;
+            draft.skillVersionBody.instructions = action.payload.instructions;
+        },
+        fetchSkillVersionBodyFailure(draft, action) {
+            if (draft.skillVersionBody.macroId !== action.payload.macroId ||
+                draft.skillVersionBody.version !== action.payload.version) {
+                return;
+            }
+            draft.skillVersionBody.fetchState = 'Error';
+            draft.skillVersionBody.error = action.payload.error;
+        },
+        saveSkill: {
+            reducer(draft) {
+                draft.skillMutation = {
+                    fetchState: 'In-Progress',
+                    error: undefined,
+                    savedMacroId: undefined,
+                };
+            },
+            prepare(payload) {
+                return { payload };
+            },
+        },
+        saveSkillSuccess(draft, action) {
+            draft.skillMutation.fetchState = 'Completed';
+            draft.skillMutation.error = undefined;
+            draft.skillMutation.savedMacroId = action.payload.macroId;
+            // ONLY when the saved skill is still the one on screen. The save is a mergeMap —
+            // deliberately not cancellable, because cancelling a write only hides its outcome —
+            // so it can land after the user opened another skill, went back to the list, closed
+            // the browser, or deleted it. Writing unconditionally would drop one skill's prompt
+            // on top of another's, which is precisely what the fetch guards exist to prevent.
+            //
+            // A create lands here with nothing open, and is deliberately NOT force-opened:
+            // `savedMacroId` is the signal for "what did I just create", and the caller decides
+            // whether to open it.
+            if (draft.skillDetail.macroId !== action.payload.macroId) {
+                return;
+            }
+            // The save response IS a detail read, so adopt it. Discarding it left an editor
+            // sitting on the same skill showing the prompt and card it had BEFORE the save —
+            // the user's own change missing from the thing they just saved.
+            draft.skillDetail = {
+                fetchState: 'Completed',
+                error: undefined,
+                detail: action.payload.detail,
+                macroId: action.payload.macroId,
+            };
+            // History is now a version behind; the epic refetches it.
+            draft.skillVersions = {
+                fetchState: 'Not-Started',
+                error: undefined,
+                versions: [],
+                macroId: undefined,
+            };
+        },
+        saveSkillFailure(draft, action) {
+            draft.skillMutation.fetchState = 'Error';
+            draft.skillMutation.error = action.payload;
+        },
+        setSkillVisibility: {
+            reducer(draft) {
+                // Shares skillMutation with save and delete. One in-flight write at a time is the
+                // existing contract here, and publishing is a write like any other.
+                draft.skillMutation = {
+                    fetchState: 'In-Progress',
+                    error: undefined,
+                    savedMacroId: undefined,
+                };
+            },
+            prepare(payload) {
+                return { payload };
+            },
+        },
+        setSkillVisibilitySuccess(draft, action) {
+            draft.skillMutation.fetchState = 'Completed';
+            draft.skillMutation.error = undefined;
+            draft.skillMutation.savedMacroId = action.payload.macroId;
+            // Same staleness guard as saveSkillSuccess, for the same reason: this is a mergeMap
+            // write, so it can land after the user opened another skill.
+            if (draft.skillDetail.macroId !== action.payload.macroId) {
+                return;
+            }
+            draft.skillDetail = {
+                fetchState: 'Completed',
+                error: undefined,
+                detail: action.payload.detail,
+                macroId: action.payload.macroId,
+            };
+            // Deliberately NOT clearing skillVersions, unlike saveSkillSuccess. No version was
+            // created, so the history the user is looking at is still correct — refetching it
+            // would flash a loading state for a list that did not change.
+        },
+        setSkillVisibilityFailure(draft, action) {
+            draft.skillMutation.fetchState = 'Error';
+            draft.skillMutation.error = action.payload;
+        },
+        deleteSkill: {
+            reducer(draft) {
+                draft.skillMutation = {
+                    fetchState: 'In-Progress',
+                    error: undefined,
+                    savedMacroId: undefined,
+                };
+            },
+            prepare(payload) {
+                return { payload };
+            },
+        },
+        deleteSkillSuccess(draft, action) {
+            draft.skillMutation.fetchState = 'Completed';
+            draft.skillMutation.error = undefined;
+            // Drop it from the list immediately rather than waiting for a refetch: the row
+            // the user just deleted staying on screen reads as the delete having failed.
+            draft.skills.skills = draft.skills.skills.filter((skill) => skill.macroId !== action.payload.macroId);
+            if (!draft.skills.deletedMacroIds.includes(action.payload.macroId)) {
+                draft.skills.deletedMacroIds.push(action.payload.macroId);
+            }
+            if (draft.skillDetail.macroId === action.payload.macroId) {
+                clearOpenSkillState(draft);
+            }
+        },
+        deleteSkillFailure(draft, action) {
+            draft.skillMutation.fetchState = 'Error';
+            draft.skillMutation.error = action.payload;
+        },
+        clearSkillMutation(draft) {
+            draft.skillMutation = {
+                fetchState: 'Not-Started',
+                error: undefined,
+                savedMacroId: undefined,
+            };
+        },
+        openSkillsBrowser(draft) {
+            draft.uiState.isSkillsBrowserOpen = true;
+        },
+        closeSkillsBrowser(draft) {
+            // Unguarded, unlike the implicit closes: an explicit close should clear a lingering
+            // detail even if the flag is already false.
+            closeSkillsBrowserState(draft);
+        },
         updateCotCollapsedState(draft, action) {
             const { isCollapsed, questionAnswerId } = action.payload;
             draft.uiState.cotCollapsedByQuestionAnswerId[questionAnswerId] =
@@ -511,5 +913,5 @@ const aiCfoView = createSlice({
         });
     },
 });
-export const { submitQuestion, submitQuestionSuccess, submitQuestionFailure, updateResponseState, updateAiCfoViewScrollPosition, updateCurrentInput, submitFeedback, submitFeedbackSuccess, submitFeedbackFailure, setSession, createSession, createSessionSuccess, createSessionFailure, createSessionAndSubmit, initializeNewSessionState, createSessionAndSubmitSuccess, createSessionAndSubmitFailure, fetchChatSessionsForUser, fetchChatSessionsForUserSuccess, fetchChatSessionsForUserFailure, fetchChatHistory, fetchChatHistorySuccess, fetchChatHistoryFailure, clearCurrentSessionId, clearLastContextMessage, stopSubmit, stopSubmitSuccess, stopSubmitFailure, stopSubmitQuestion, stopSubmitQuestionSuccess, stopSubmitQuestionFailure, acceptMasterTOS, acceptMasterTOSSuccess, acceptMasterTOSFailure, deleteChatSession, deleteChatSessionSuccess, deleteChatSessionFailure, clearDeleteChatSessionStatus, updateCotCollapsedState, fetchSuggestedQuestions, fetchSuggestedQuestionsSuccess, fetchSuggestedQuestionsFailure, fetchSkills, fetchSkillsSuccess, fetchSkillsFailure, applyAiCfoSidePanelHostPageTransition, clearAiCfoSidePanelHostPageContext, } = aiCfoView.actions;
+export const { submitQuestion, submitQuestionSuccess, submitQuestionFailure, updateResponseState, updateAiCfoViewScrollPosition, updateCurrentInput, submitFeedback, submitFeedbackSuccess, submitFeedbackFailure, setSession, createSession, createSessionSuccess, createSessionFailure, createSessionAndSubmit, initializeNewSessionState, createSessionAndSubmitSuccess, createSessionAndSubmitFailure, fetchChatSessionsForUser, fetchChatSessionsForUserSuccess, fetchChatSessionsForUserFailure, fetchChatHistory, fetchChatHistorySuccess, fetchChatHistoryFailure, clearCurrentSessionId, clearLastContextMessage, stopSubmit, stopSubmitSuccess, stopSubmitFailure, stopSubmitQuestion, stopSubmitQuestionSuccess, stopSubmitQuestionFailure, acceptMasterTOS, acceptMasterTOSSuccess, acceptMasterTOSFailure, deleteChatSession, deleteChatSessionSuccess, deleteChatSessionFailure, clearDeleteChatSessionStatus, updateCotCollapsedState, fetchSuggestedQuestions, fetchSuggestedQuestionsSuccess, fetchSuggestedQuestionsFailure, fetchSkills, fetchSkillsSuccess, fetchSkillsFailure, fetchSkillDetail, fetchSkillDetailSuccess, fetchSkillDetailFailure, clearSkillDetail, clearSkillMutation, deleteSkill, deleteSkillSuccess, deleteSkillFailure, fetchSkillVersionBody, fetchSkillVersionBodySuccess, fetchSkillVersionBodyFailure, fetchSkillVersions, fetchSkillVersionsSuccess, fetchSkillVersionsFailure, saveSkill, saveSkillSuccess, saveSkillFailure, setSkillVisibility, setSkillVisibilitySuccess, setSkillVisibilityFailure, openSkillsBrowser, closeSkillsBrowser, applyAiCfoSidePanelHostPageTransition, clearAiCfoSidePanelHostPageContext, } = aiCfoView.actions;
 export default aiCfoView.reducer;