From 85260ab49a7b1d196ba1037ee8c6c0998ce98d4e Mon Sep 17 00:00:00 2001
From: Gregory Schier <gschier1990@gmail.com>
Date: Tue, 17 Mar 2026 09:07:35 -0700
Subject: [PATCH 1/6] Fix duplicate request snapshotting URL as name

When duplicating an unnamed request, the resolved name (URL fallback)
was being set as the duplicate's explicit name. Now the raw name field
is preserved, so unnamed duplicates continue to dynamically show the URL.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 crates/yaak-models/guest-js/store.ts | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/crates/yaak-models/guest-js/store.ts b/crates/yaak-models/guest-js/store.ts
index 78a966939..f1967bfcb 100644
--- a/crates/yaak-models/guest-js/store.ts
+++ b/crates/yaak-models/guest-js/store.ts
@@ -144,9 +144,10 @@ export function duplicateModel<M extends AnyModel["model"], T extends ExtractMod
     throw new Error("Failed to duplicate null model");
   }
 
-  // If the model has a name, try to duplicate it with a name that doesn't conflict
-  let name = "name" in model ? resolvedModelName(model) : undefined;
-  if (name != null) {
+  // If the model has an explicit (non-empty) name, try to duplicate it with a name that doesn't conflict.
+  // When the name is empty, keep it empty so the display falls back to the URL.
+  let name: string | undefined = "name" in model ? (model as { name: string }).name : undefined;
+  if (name) {
     const existingModels = listModels(model.model);
     for (let i = 0; i < 100; i++) {
       const hasConflict = existingModels.some((m) => {

From 097939821569a8ab466c99c897625f8c30042a38 Mon Sep 17 00:00:00 2001
From: Gregory Schier <gschier1990@gmail.com>
Date: Tue, 17 Mar 2026 09:15:10 -0700
Subject: [PATCH 2/6] Refactor ternary to if-statement for readability

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 crates/yaak-models/guest-js/store.ts | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/crates/yaak-models/guest-js/store.ts b/crates/yaak-models/guest-js/store.ts
index f1967bfcb..ca0cd3c83 100644
--- a/crates/yaak-models/guest-js/store.ts
+++ b/crates/yaak-models/guest-js/store.ts
@@ -146,7 +146,10 @@ export function duplicateModel<M extends AnyModel["model"], T extends ExtractMod
 
   // If the model has an explicit (non-empty) name, try to duplicate it with a name that doesn't conflict.
   // When the name is empty, keep it empty so the display falls back to the URL.
-  let name: string | undefined = "name" in model ? (model as { name: string }).name : undefined;
+  let name: string | undefined;
+  if ("name" in model) {
+    name = (model as { name: string }).name;
+  }
   if (name) {
     const existingModels = listModels(model.model);
     for (let i = 0; i < 100; i++) {

From df089e6f00a3f695cd0fff6f207b8725b547d07b Mon Sep 17 00:00:00 2001
From: Gregory Schier <gschier1990@gmail.com>
Date: Tue, 17 Mar 2026 09:18:17 -0700
Subject: [PATCH 3/6] Use resolvedModelName for conflict detection on named
 requests

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 crates/yaak-models/guest-js/store.ts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/crates/yaak-models/guest-js/store.ts b/crates/yaak-models/guest-js/store.ts
index ca0cd3c83..ca9e55b76 100644
--- a/crates/yaak-models/guest-js/store.ts
+++ b/crates/yaak-models/guest-js/store.ts
@@ -151,6 +151,7 @@ export function duplicateModel<M extends AnyModel["model"], T extends ExtractMod
     name = (model as { name: string }).name;
   }
   if (name) {
+    name = resolvedModelName(model);
     const existingModels = listModels(model.model);
     for (let i = 0; i < 100; i++) {
       const hasConflict = existingModels.some((m) => {

From f283b02cfeb0d6bc3057971fb1cd83f96baf5402 Mon Sep 17 00:00:00 2001
From: Gregory Schier <gschier1990@gmail.com>
Date: Tue, 17 Mar 2026 09:19:42 -0700
Subject: [PATCH 4/6] Remove redundant resolvedModelName call for non-empty
 names

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 crates/yaak-models/guest-js/store.ts | 1 -
 1 file changed, 1 deletion(-)

diff --git a/crates/yaak-models/guest-js/store.ts b/crates/yaak-models/guest-js/store.ts
index ca9e55b76..ca0cd3c83 100644
--- a/crates/yaak-models/guest-js/store.ts
+++ b/crates/yaak-models/guest-js/store.ts
@@ -151,7 +151,6 @@ export function duplicateModel<M extends AnyModel["model"], T extends ExtractMod
     name = (model as { name: string }).name;
   }
   if (name) {
-    name = resolvedModelName(model);
     const existingModels = listModels(model.model);
     for (let i = 0; i < 100; i++) {
       const hasConflict = existingModels.some((m) => {

From 356b8ea0d6f41c64e16d734780003993e573e7df Mon Sep 17 00:00:00 2001
From: Gregory Schier <gschier1990@gmail.com>
Date: Tue, 17 Mar 2026 09:24:50 -0700
Subject: [PATCH 5/6] Remove unnecessary typecast for model.name

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 crates/yaak-models/guest-js/store.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/crates/yaak-models/guest-js/store.ts b/crates/yaak-models/guest-js/store.ts
index ca0cd3c83..282ee309b 100644
--- a/crates/yaak-models/guest-js/store.ts
+++ b/crates/yaak-models/guest-js/store.ts
@@ -148,7 +148,7 @@ export function duplicateModel<M extends AnyModel["model"], T extends ExtractMod
   // When the name is empty, keep it empty so the display falls back to the URL.
   let name: string | undefined;
   if ("name" in model) {
-    name = (model as { name: string }).name;
+    name = model.name;
   }
   if (name) {
     const existingModels = listModels(model.model);

From d2a3f7a6697fc1fbd221aae043499fbf4c3897a1 Mon Sep 17 00:00:00 2001
From: Gregory Schier <gschier1990@gmail.com>
Date: Tue, 17 Mar 2026 09:28:38 -0700
Subject: [PATCH 6/6] Simplify back to ternary now that typecast is gone

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 crates/yaak-models/guest-js/store.ts | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/crates/yaak-models/guest-js/store.ts b/crates/yaak-models/guest-js/store.ts
index 282ee309b..7ea868c2f 100644
--- a/crates/yaak-models/guest-js/store.ts
+++ b/crates/yaak-models/guest-js/store.ts
@@ -146,10 +146,7 @@ export function duplicateModel<M extends AnyModel["model"], T extends ExtractMod
 
   // If the model has an explicit (non-empty) name, try to duplicate it with a name that doesn't conflict.
   // When the name is empty, keep it empty so the display falls back to the URL.
-  let name: string | undefined;
-  if ("name" in model) {
-    name = model.name;
-  }
+  let name = "name" in model ? model.name : undefined;
   if (name) {
     const existingModels = listModels(model.model);
     for (let i = 0; i < 100; i++) {