From ca89eafb0bc3c5b80b3bd164038da5be28cff720 Mon Sep 17 00:00:00 2001
From: arcticfly <41524992+arcticfly@users.noreply.github.com>
Date: Fri, 18 Aug 2023 08:09:07 -0700
Subject: [PATCH] Create new uiId for forked variants and scenarios (#175)

* Create new uiIds for forked variants and scenarios

* Add replaceVariant.mutateAsync to onSave dependencies
---
 app/src/components/OutputsTable/VariantEditor.tsx | 2 +-
 app/src/server/api/routers/experiments.router.ts  | 2 ++
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/app/src/components/OutputsTable/VariantEditor.tsx b/app/src/components/OutputsTable/VariantEditor.tsx
index f36ebd5..c235525 100644
--- a/app/src/components/OutputsTable/VariantEditor.tsx
+++ b/app/src/components/OutputsTable/VariantEditor.tsx
@@ -110,7 +110,7 @@ export default function VariantEditor(props: { variant: PromptVariant }) {
     setIsChanged(false);
 
     await utils.promptVariants.list.invalidate();
-  }, [checkForChanges]);
+  }, [checkForChanges, replaceVariant.mutateAsync]);
 
   useEffect(() => {
     if (monaco) {
diff --git a/app/src/server/api/routers/experiments.router.ts b/app/src/server/api/routers/experiments.router.ts
index 1be0275..5820aac 100644
--- a/app/src/server/api/routers/experiments.router.ts
+++ b/app/src/server/api/routers/experiments.router.ts
@@ -178,6 +178,7 @@ export const experimentsRouter = createTRPCRouter({
         existingToNewVariantIds.set(variant.id, newVariantId);
         variantsToCreate.push({
           ...variant,
+          uiId: uuidv4(),
           id: newVariantId,
           experimentId: newExperimentId,
         });
@@ -191,6 +192,7 @@ export const experimentsRouter = createTRPCRouter({
         scenariosToCreate.push({
           ...scenario,
           id: newScenarioId,
+          uiId: uuidv4(),
           experimentId: newExperimentId,
           variableValues: scenario.variableValues as Prisma.InputJsonValue,
         });