Fix build state recognition

Author: Anyrainel

src/presets/artifact-builds/[GGArtifact] 全角色配装 AllCharacterBuilds.json

@@ -1,6 +1,6 @@
 {
   "version": 5,
-  "id": "export-1772547607436",
+  "id": "export-1772547805080",
   "author": "GGArtifact",
   "description": "全角色配装 AllCharacterBuilds",
   "builds": {
@@ -3445,6 +3445,39 @@
       "roles": ["support"],
       "source": "preset"
     },
+    "37JY26m": {
+      "id": "37JY26m",
+      "name": "",
+      "visible": false,
+      "composition": "4pc",
+      "sands": ["em"],
+      "goblet": ["em"],
+      "circlet": ["em", "cr"],
+      "substats": [
+        {
+          "stat": "em",
+          "weight": 100
+        },
+        {
+          "stat": "cr",
+          "weight": 100
+        },
+        {
+          "stat": "cd",
+          "weight": 100
+        },
+        {
+          "stat": "er",
+          "weight": 50
+        }
+      ],
+      "artifactSet": "viridescent_venerer",
+      "characterId": "kaedehara_kazuha",
+      "styles": ["on-field"],
+      "roles": ["dps"],
+      "source": "preset",
+      "minCons": 6
+    },
     "xcrqdu": {
       "id": "xcrqdu",
       "name": "",
@@ -4830,7 +4863,7 @@
     "yun_jin": ["e-eIXm", "dnM9om", "e-gCcG"],
     "yoimiya": ["2FRItMu", "2FRCOKO"],
     "kamisato_ayaka": ["4aRtNc0", "4aRxcbm"],
-    "kaedehara_kazuha": ["37JY4fG"],
+    "kaedehara_kazuha": ["37JY4fG", "37JY26m"],
     "eula": ["xcrqdu"],
     "diona": ["6KCVIaG"],
     "yanfei": ["3PxdDeO"],

src/stores/useBuildsStore.ts

@@ -252,28 +252,13 @@ export const useBuildsStore = create<BuildsState>()(
       setBuild: (buildId: string, patch: Partial<Build>, baseBuild?: Build) => {
         set((state) => {
           let targetBuild = state.builds[buildId];
+          let isNew = false;
 
           if (!targetBuild) {
             if (baseBuild) {
               // Copy-on-Write: Initialize with baseBuild + patch
               targetBuild = { ...baseBuild, ...patch };
-
-              // Ensure it's tracked in the character list (Union survival)
-              const charId = targetBuild.characterId;
-              if (!state.characterToBuildIds[charId]) {
-                // Initialize from preset ordering so other builds aren't lost
-                const preset = getCachedPreset(state.activePresetId);
-                const presetIds = preset?.characterBuilds?.[charId];
-                state.characterToBuildIds[charId] = presetIds
-                  ? [...presetIds]
-                  : [];
-              }
-              if (!state.characterToBuildIds[charId].includes(buildId)) {
-                state.characterToBuildIds[charId].push(buildId);
-              }
-
-              // Proceed to register it
-              state.builds[buildId] = targetBuild;
+              isNew = true;
             } else {
               console.warn(
                 `Build ${buildId} not found and no baseBuild provided`
@@ -298,6 +283,40 @@ export const useBuildsStore = create<BuildsState>()(
           // Ensure id and characterId cannot be changed
           targetBuild.id = buildId; // Enforce ID consistency
 
+          // Check if the result still matches the preset version (no-op guard).
+          // This prevents marking a build as "modified" when no actual data changed
+          // (e.g. clicking the name input without typing, or editing back to original).
+          const preset = getCachedPreset(state.activePresetId);
+          const presetBuild = preset?.builds[buildId];
+          if (presetBuild && areBuildsEqual(targetBuild, presetBuild)) {
+            if (isNew) {
+              // No actual change from preset — skip copy-on-write entirely
+              return;
+            }
+            // User reverted all changes — auto-revert by removing local override
+            delete state.builds[buildId];
+            delete state.validationErrors[buildId];
+            return;
+          }
+
+          if (isNew) {
+            // Ensure it's tracked in the character list (Union survival)
+            const charId = targetBuild.characterId;
+            if (!state.characterToBuildIds[charId]) {
+              // Initialize from preset ordering so other builds aren't lost
+              const presetIds = preset?.characterBuilds?.[charId];
+              state.characterToBuildIds[charId] = presetIds
+                ? [...presetIds]
+                : [];
+            }
+            if (!state.characterToBuildIds[charId].includes(buildId)) {
+              state.characterToBuildIds[charId].push(buildId);
+            }
+
+            // Register the new local override
+            state.builds[buildId] = targetBuild;
+          }
+
           // Re-validate
           state.validationErrors[buildId] =
             getBuildValidationErrors(targetBuild);