@zeniai/client-epic-state 5.0.83-betaRR2 → 5.0.83-betaRR3

package/lib/esm/view/spendManagement/chargeCards/issueChargeCard/applyAiCardCreationFormDraftUpdate.js

@@ -1,12 +1,23 @@
 import { deriveAiCardRowsFromTeams } from './deriveAiCardRowsFromTeams';
 // Single source of truth for "what the next form draft should look like
-// after a watchHandler flush". `create` mode + missing context keep the
-// payload as-is; `review` mode re-derives step-2 cardRows from the latest
-// teamRows, preserving in-place edits via prevCardRows.
-export const applyAiCardCreationFormDraftUpdate = ({ next, context, }) => {
+// after a watchHandler flush".
+//
+// `create` mode + missing context: keep the payload as-is.
+//
+// `review` mode: re-derive step-2 `cardRows` from the latest `teamRows`
+// ONLY when the step-1 selection changed (a team was toggled on/off).
+// Step-2-only edits (dropdown swap, owner change, limit edit) must skip
+// the re-derive because `deriveAiCardRowsFromTeams` matches previous rows
+// by `cardRow.id`, which the dropdown swap has just mutated — running it
+// on a step-2-only diff reverts the user's edit back to the source team's
+// original id/label.
+export const applyAiCardCreationFormDraftUpdate = ({ next, prev, context, }) => {
     if (context == null || context.mode === 'create') {
         return next;
     }
+    if (!haveCheckedTeamRowsChanged(prev?.teamRows, next.teamRows)) {
+        return next;
+    }
     return {
         teamRows: next.teamRows,
         cardRows: deriveAiCardRowsFromTeams({
@@ -18,3 +29,15 @@ export const applyAiCardCreationFormDraftUpdate = ({ next, context, }) => {
         }),
     };
 };
+const haveCheckedTeamRowsChanged = (prevTeamRows, nextTeamRows) => {
+    const prevCheckedIds = (prevTeamRows ?? [])
+        .filter((row) => row.isChecked)
+        .map((row) => row.id);
+    const nextCheckedIds = nextTeamRows
+        .filter((row) => row.isChecked)
+        .map((row) => row.id);
+    if (prevCheckedIds.length !== nextCheckedIds.length) {
+        return true;
+    }
+    return prevCheckedIds.some((id, idx) => id !== nextCheckedIds[idx]);
+};

package/lib/esm/view/spendManagement/chargeCards/issueChargeCard/issueChargeCardReducer.js

@@ -160,6 +160,7 @@ const issueChargeCard = createSlice({
          */
         updateAiCardCreationFormDraft(draft, action) {
             draft.aiCardCreationFormDraft = applyAiCardCreationFormDraftUpdate({
+                prev: draft.aiCardCreationFormDraft,
                 next: action.payload,
                 context: draft.aiCardCreationFormDraftContext,
             });

package/lib/view/spendManagement/chargeCards/issueChargeCard/applyAiCardCreationFormDraftUpdate.d.ts

@@ -2,5 +2,6 @@ import { AiCardCreationFormDraftSeedContext, CardsFormDraft } from './aiCardCrea
 export interface ApplyAiCardCreationFormDraftUpdateArgs {
     context: AiCardCreationFormDraftSeedContext | undefined;
     next: CardsFormDraft;
+    prev: CardsFormDraft | undefined;
 }
-export declare const applyAiCardCreationFormDraftUpdate: ({ next, context, }: ApplyAiCardCreationFormDraftUpdateArgs) => CardsFormDraft;
+export declare const applyAiCardCreationFormDraftUpdate: ({ next, prev, context, }: ApplyAiCardCreationFormDraftUpdateArgs) => CardsFormDraft;

package/lib/view/spendManagement/chargeCards/issueChargeCard/applyAiCardCreationFormDraftUpdate.js

@@ -3,13 +3,24 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.applyAiCardCreationFormDraftUpdate = void 0;
 const deriveAiCardRowsFromTeams_1 = require("./deriveAiCardRowsFromTeams");
 // Single source of truth for "what the next form draft should look like
-// after a watchHandler flush". `create` mode + missing context keep the
-// payload as-is; `review` mode re-derives step-2 cardRows from the latest
-// teamRows, preserving in-place edits via prevCardRows.
-const applyAiCardCreationFormDraftUpdate = ({ next, context, }) => {
+// after a watchHandler flush".
+//
+// `create` mode + missing context: keep the payload as-is.
+//
+// `review` mode: re-derive step-2 `cardRows` from the latest `teamRows`
+// ONLY when the step-1 selection changed (a team was toggled on/off).
+// Step-2-only edits (dropdown swap, owner change, limit edit) must skip
+// the re-derive because `deriveAiCardRowsFromTeams` matches previous rows
+// by `cardRow.id`, which the dropdown swap has just mutated — running it
+// on a step-2-only diff reverts the user's edit back to the source team's
+// original id/label.
+const applyAiCardCreationFormDraftUpdate = ({ next, prev, context, }) => {
     if (context == null || context.mode === 'create') {
         return next;
     }
+    if (!haveCheckedTeamRowsChanged(prev?.teamRows, next.teamRows)) {
+        return next;
+    }
     return {
         teamRows: next.teamRows,
         cardRows: (0, deriveAiCardRowsFromTeams_1.deriveAiCardRowsFromTeams)({
@@ -22,3 +33,15 @@ const applyAiCardCreationFormDraftUpdate = ({ next, context, }) => {
     };
 };
 exports.applyAiCardCreationFormDraftUpdate = applyAiCardCreationFormDraftUpdate;
+const haveCheckedTeamRowsChanged = (prevTeamRows, nextTeamRows) => {
+    const prevCheckedIds = (prevTeamRows ?? [])
+        .filter((row) => row.isChecked)
+        .map((row) => row.id);
+    const nextCheckedIds = nextTeamRows
+        .filter((row) => row.isChecked)
+        .map((row) => row.id);
+    if (prevCheckedIds.length !== nextCheckedIds.length) {
+        return true;
+    }
+    return prevCheckedIds.some((id, idx) => id !== nextCheckedIds[idx]);
+};

package/lib/view/spendManagement/chargeCards/issueChargeCard/issueChargeCardReducer.js

@@ -164,6 +164,7 @@ const issueChargeCard = (0, toolkit_1.createSlice)({
          */
         updateAiCardCreationFormDraft(draft, action) {
             draft.aiCardCreationFormDraft = (0, applyAiCardCreationFormDraftUpdate_1.applyAiCardCreationFormDraftUpdate)({
+                prev: draft.aiCardCreationFormDraft,
                 next: action.payload,
                 context: draft.aiCardCreationFormDraftContext,
             });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zeniai/client-epic-state",
-  "version": "5.0.83-betaRR2",
+  "version": "5.0.83-betaRR3",
   "description": "Shared module between Web & Mobile containing required abstractions for state management, async network communication. ",
   "main": "lib/index.js",
   "module": "lib/esm/index.js",