fix(tags): persist tag selections in the case and session editors (#456)

Adding a tag in the case or session editor created the Tag but never linked it,
and selecting an existing tag while also creating one was lost too. Creating a
tag invalidates the parent queries (the case/session rows include their tags),
which refetches the case/session; a reset/re-sync effect then reverted the
user's in-progress tag selection to the saved tags before the form submitted, so
the new (or selected) tags were dropped. The run editor was unaffected because it
keeps tags in plain component state with no such re-sync.

- Case editor: the defaults-restore effect now preserves the current tags value
  (currentValues.tags) on re-run, matching how issues were already preserved.
- Session editor: selectedTags is initialized from the loaded session once
  (guarded by a ref) instead of re-syncing on every sessionData change.

Author: therealbrad

testplanit/app/[locale]/projects/repository/[projectId]/[caseId]/page.tsx

@@ -914,7 +914,11 @@ export default function TestCaseDetails() {
       folderId: testcase.folderId,
       estimate: testcase.estimate ? formatSeconds(testcase.estimate) : "",
       automated: testcase.automated,
-      tags: testcase.tags?.map((tag: any) => tag.id) || [],
+      // Preserve the in-progress tag selection across re-runs of this effect (e.g. creating a tag
+      // invalidates the Tags query and refetches the case, re-firing this reset). Without this the
+      // user's edits to tags are silently reverted to the case's saved tags, like issues below.
+      tags:
+        currentValues.tags || testcase.tags?.map((tag: any) => tag.id) || [],
       // Preserve existing issues value if it's already set (from external issues)
       issues:
         currentValues.issues ||

testplanit/app/[locale]/projects/sessions/[projectId]/[sessionId]/page.tsx

@@ -804,6 +804,10 @@ export default function SessionPage() {
   );
   const [isSubmitting, setIsSubmitting] = useState(false);
   const [selectedTags, setSelectedTags] = useState<number[]>([]);
+  // Initialize selectedTags from the loaded session ONCE. Creating a tag invalidates parent queries
+  // (the session includes its tags), refetching sessionData; without this guard the re-sync effect
+  // below would revert the user's in-progress tag edits on that refetch.
+  const hasInitializedTagsRef = useRef(false);
   const [missionContent, setMissionContent] =
     useState<JSONContent>(emptyEditorContent);
   const [noteContent, setNoteContent] =
@@ -1251,7 +1255,6 @@ export default function SessionPage() {
         keepDefaultValues: true,
       });
       setInitialValues(formValues);
-      setSelectedTags(sessionData.tags.map((tag) => tag.id));
 
       // Delay setting form as initialized
       requestAnimationFrame(() => {
@@ -1402,10 +1405,12 @@ export default function SessionPage() {
     }
   }, [sessionData, sessionData?.note, sessionData?.mission]);
 
-  // Add useEffect for initial tags
+  // Initialize the tag selection from the loaded session, once (see hasInitializedTagsRef) so a
+  // tag-create refetch of sessionData does not revert in-progress edits.
   useEffect(() => {
-    if (sessionData) {
+    if (sessionData && !hasInitializedTagsRef.current) {
       setSelectedTags(sessionData.tags.map((tag) => tag.id));
+      hasInitializedTagsRef.current = true;
     }
   }, [sessionData]);