Simplify custom world result editing controls

src/components/CustomWorldResultView.tsx

@@ -1,5 +1,6 @@
 import { type ReactNode,useMemo, useState } from 'react';
 
+import { normalizeCustomWorldLandmarks } from '../data/customWorldSceneGraph';
 import { Character, CustomWorldProfile } from '../types';
 import { getNineSliceStyle, UI_CHROME } from '../uiAssets';
 import { CustomWorldEntityCatalog, type ResultTab } from './CustomWorldEntityCatalog';
@@ -16,11 +17,6 @@ interface CustomWorldResultViewProps {
   onEditSetting: () => void;
   onRegenerate: () => void;
   onContinueExpand?: () => void;
-  onRegeneratePlayableNpc?: (id: string) => void;
-  onRegenerateStoryNpc?: (id: string) => void;
-  onRegenerateLandmark?: (id: string) => void;
-  onRegenerateStoryExpansion?: () => void;
-  onRegenerateLandmarkNetwork?: () => void;
   onSave: () => void;
   onProfileChange: (profile: CustomWorldProfile) => void;
 }
@@ -66,6 +62,46 @@ function getCreateLabelByTab(activeTab: ResultTab) {
   return '';
 }
 
+function removeStoryNpcsFromProfile(
+  profile: CustomWorldProfile,
+  ids: string[],
+) {
+  const idSet = new Set(ids);
+  const nextStoryNpcs = profile.storyNpcs.filter((npc) => !idSet.has(npc.id));
+
+  return {
+    ...profile,
+    storyNpcs: nextStoryNpcs,
+    landmarks: normalizeCustomWorldLandmarks({
+      landmarks: profile.landmarks.map((landmark) => ({
+        ...landmark,
+        sceneNpcIds: landmark.sceneNpcIds.filter((npcId) => !idSet.has(npcId)),
+      })),
+      storyNpcs: nextStoryNpcs,
+    }),
+  } satisfies CustomWorldProfile;
+}
+
+function removeLandmarksFromProfile(profile: CustomWorldProfile, ids: string[]) {
+  const idSet = new Set(ids);
+  const nextLandmarks = profile.landmarks.filter(
+    (landmark) => !idSet.has(landmark.id),
+  );
+
+  return {
+    ...profile,
+    landmarks: normalizeCustomWorldLandmarks({
+      landmarks: nextLandmarks.map((landmark) => ({
+        ...landmark,
+        connections: landmark.connections.filter(
+          (connection) => !idSet.has(connection.targetLandmarkId),
+        ),
+      })),
+      storyNpcs: profile.storyNpcs,
+    }),
+  } satisfies CustomWorldProfile;
+}
+
 export function CustomWorldResultView({
   profile,
   previewCharacters,
@@ -77,11 +113,6 @@ export function CustomWorldResultView({
   onEditSetting,
   onRegenerate: triggerRegenerate,
   onContinueExpand,
-  onRegeneratePlayableNpc,
-  onRegenerateStoryNpc,
-  onRegenerateLandmark,
-  onRegenerateStoryExpansion,
-  onRegenerateLandmarkNetwork,
   onSave,
   onProfileChange,
 }: CustomWorldResultViewProps) {
@@ -101,6 +132,16 @@ export function CustomWorldResultView({
     triggerRegenerate();
   };
 
+  const handleDeleteStoryNpcs = (ids: string[]) => {
+    if (ids.length === 0) return;
+    onProfileChange(removeStoryNpcsFromProfile(profile, ids));
+  };
+
+  const handleDeleteLandmarks = (ids: string[]) => {
+    if (ids.length === 0) return;
+    onProfileChange(removeLandmarksFromProfile(profile, ids));
+  };
+
   return (
     <div className="flex h-full min-h-0 flex-col">
       <div className="mb-4 flex justify-start">
@@ -122,11 +163,8 @@ export function CustomWorldResultView({
           onActiveTabChange={setActiveTab}
           onEditTarget={setEditorTarget}
           onProfileChange={onProfileChange}
-          onRegeneratePlayableNpc={onRegeneratePlayableNpc}
-          onRegenerateStoryNpc={onRegenerateStoryNpc}
-          onRegenerateLandmark={onRegenerateLandmark}
-          onRegenerateStoryExpansion={onRegenerateStoryExpansion}
-          onRegenerateLandmarkNetwork={onRegenerateLandmarkNetwork}
+          onDeleteStoryNpcs={handleDeleteStoryNpcs}
+          onDeleteLandmarks={handleDeleteLandmarks}
           createActionLabel={createLabel}
           onCreateAction={createTarget ? () => setEditorTarget(createTarget) : undefined}
         />