fix: Guided Tours Ephemeral State

src/components/Editor/Context/RenamePipeline.tsx

@@ -66,6 +66,7 @@ const RenamePipeline = () => {
       onSubmit={handleTitleUpdate}
       submitButtonText="Update Title"
       isSubmitDisabled={isSubmitDisabled}
+      excludeNames={title ? [title] : undefined}
       onOpenChange={(open) => {
         if (open) track("pipeline_editor.name_pipeline_dialog_impression");
       }}

src/components/Learn/FeaturedTours.tsx

@@ -1,10 +1,12 @@
+import { useQueryClient } from "@tanstack/react-query";
 import { Link, useNavigate } from "@tanstack/react-router";
 
 import { Badge } from "@/components/ui/badge";
 import { Button } from "@/components/ui/button";
 import { Icon } from "@/components/ui/icon";
 import { BlockStack, InlineStack } from "@/components/ui/layout";
 import { Heading, Paragraph, Text } from "@/components/ui/typography";
+import { resetAllTourPipelineState } from "@/providers/TourProvider/tourPipelineStorage/resetAllTourPipelineState";
 import { APP_ROUTES } from "@/routes/router";
 import { tracking } from "@/utils/tracking";
 
@@ -45,8 +47,10 @@ function buildFeaturedTours(): FeaturedTour[] {
 export function FeaturedTours() {
   const featured = buildFeaturedTours();
   const navigate = useNavigate();
+  const queryClient = useQueryClient();
 
   const startTour = (tourId: string) => {
+    resetAllTourPipelineState(queryClient);
     void navigate({ to: APP_ROUTES.TOUR_DETAIL, params: { tourId } });
   };
 

src/components/Learn/ToursLibrary.tsx

@@ -1,3 +1,4 @@
+import { useQueryClient } from "@tanstack/react-query";
 import { useNavigate } from "@tanstack/react-router";
 
 import { Badge } from "@/components/ui/badge";
@@ -12,6 +13,7 @@ import {
 import { Icon } from "@/components/ui/icon";
 import { BlockStack, InlineStack } from "@/components/ui/layout";
 import { Heading, Paragraph, Text } from "@/components/ui/typography";
+import { resetAllTourPipelineState } from "@/providers/TourProvider/tourPipelineStorage/resetAllTourPipelineState";
 import { APP_ROUTES } from "@/routes/router";
 import { tracking } from "@/utils/tracking";
 
@@ -29,8 +31,10 @@ import { getTour } from "./tours/registry";
 function TourCard({ tour }: { tour: Tour }) {
   const isAvailable = getTour(tour.id) !== undefined;
   const navigate = useNavigate();
+  const queryClient = useQueryClient();
 
   const startTour = () => {
+    resetAllTourPipelineState(queryClient);
     void navigate({
       to: APP_ROUTES.TOUR_DETAIL,
       params: { tourId: tour.id },

src/components/shared/Dialogs/PipelineNameDialog.tsx

@@ -3,6 +3,7 @@ import {
   type ChangeEvent,
   type ReactNode,
   useCallback,
+  useMemo,
   useState,
 } from "react";
 
@@ -34,6 +35,9 @@ interface PipelineNameDialogProps {
   onSubmit: (name: string) => void;
   isSubmitDisabled?: (name: string, error: string | null) => boolean;
   onOpenChange?: (open: boolean) => void;
+  // Names to skip when checking for collisions — typically the caller's own
+  // current name in a rename flow, so renaming-to-self isn't flagged.
+  excludeNames?: string[];
 }
 
 const PipelineNameDialog = ({
@@ -47,8 +51,8 @@ const PipelineNameDialog = ({
   onSubmit,
   isSubmitDisabled,
   onOpenChange,
+  excludeNames,
 }: PipelineNameDialogProps) => {
-  const [error, setError] = useState<string | null>(null);
   const [name, setName] = useState(initialName);
 
   const {
@@ -57,33 +61,29 @@ const PipelineNameDialog = ({
     refetch: refetchUserPipelines,
   } = useLoadUserPipelines();
 
-  const handleOnChange = useCallback(
-    (e: ChangeEvent<HTMLInputElement>) => {
-      const newName = e.target.value;
-      const existingPipelineNames = new Set(
-        Array.from(userPipelines.keys()).map((name) => name.toLowerCase()),
-      );
+  const error = useMemo(() => {
+    if (isLoadingUserPipelines) return null;
+    const normalized = name.trim().toLowerCase();
+    if (normalized === "") return "Name cannot be empty";
+    const excluded = new Set(
+      (excludeNames ?? []).map((n) => n.trim().toLowerCase()),
+    );
+    const existing = new Set(
+      Array.from(userPipelines.keys())
+        .map((n) => n.toLowerCase())
+        .filter((n) => !excluded.has(n)),
+    );
+    if (existing.has(normalized)) return "Name already exists";
+    return null;
+  }, [name, userPipelines, isLoadingUserPipelines, excludeNames]);
 
-      const normalizedNewName = newName.trim().toLowerCase();
-
-      if (normalizedNewName === "") {
-        setError("Name cannot be empty");
-      } else if (existingPipelineNames.has(normalizedNewName)) {
-        setError("Name already exists");
-      } else {
-        setError(null);
-      }
-
-      setName(newName);
-    },
-    [userPipelines],
-  );
+  const handleOnChange = useCallback((e: ChangeEvent<HTMLInputElement>) => {
+    setName(e.target.value);
+  }, []);
 
   const handleDialogOpenChange = useCallback(
     (open: boolean) => {
-      if (!open) {
-        setError(null);
-      } else {
+      if (open) {
         setName(initialName);
         refetchUserPipelines();
       }

src/hooks/useLoadUserPipelines.ts

@@ -7,7 +7,7 @@ import {
 import { USER_PIPELINES_LIST_NAME } from "@/utils/constants";
 
 const useLoadUserPipelines = () => {
-  const [isLoadingUserPipelines, setIsLoadingUserPipelines] = useState(false);
+  const [isLoadingUserPipelines, setIsLoadingUserPipelines] = useState(true);
   const [userPipelines, setUserPipelines] = useState<
     Map<string, ComponentFileEntry>
   >(new Map());

src/providers/TourProvider/TourModeContext.tsx

@@ -5,6 +5,7 @@ import type { TourDefinition } from "@/components/Learn/tours/registry";
 export interface TourModeValue {
   tour: TourDefinition;
   tempPipelineName: string;
+  promoteToPipeline: (newName: string, yamlContent: string) => Promise<void>;
 }
 
 const TourModeContext = createContext<TourModeValue | null>(null);

src/providers/TourProvider/tourPipelineStorage/SessionStoragePipelineDriver.ts

@@ -0,0 +1,62 @@
+import type {
+  PipelineFileDescriptor,
+  PipelineStorageDriver,
+} from "@/services/pipelineStorage/types";
+
+import { SESSION_KEY_PREFIX } from "./constants";
+
+export class SessionStoragePipelineDriver implements PipelineStorageDriver {
+  readonly type = "session-storage";
+  readonly allowsMoveIn = false;
+  readonly allowsMoveOut = false;
+
+  private key(storageKey: string): string {
+    return `${SESSION_KEY_PREFIX}${storageKey}`;
+  }
+
+  async list(): Promise<PipelineFileDescriptor[]> {
+    const descriptors: PipelineFileDescriptor[] = [];
+    for (let i = 0; i < sessionStorage.length; i++) {
+      const fullKey = sessionStorage.key(i);
+      if (fullKey?.startsWith(SESSION_KEY_PREFIX)) {
+        descriptors.push({
+          storageKey: fullKey.slice(SESSION_KEY_PREFIX.length),
+        });
+      }
+    }
+    return descriptors;
+  }
+
+  async read(storageKey: string): Promise<string> {
+    const content = sessionStorage.getItem(this.key(storageKey));
+    if (content === null) {
+      throw new Error(
+        `Tour pipeline "${storageKey}" not found in sessionStorage`,
+      );
+    }
+    return content;
+  }
+
+  async write(storageKey: string, content: string): Promise<void> {
+    sessionStorage.setItem(this.key(storageKey), content);
+  }
+
+  async delete(storageKey: string): Promise<void> {
+    sessionStorage.removeItem(this.key(storageKey));
+  }
+
+  async rename(oldStorageKey: string, newStorageKey: string): Promise<void> {
+    const content = sessionStorage.getItem(this.key(oldStorageKey));
+    if (content === null) {
+      throw new Error(
+        `Tour pipeline "${oldStorageKey}" not found in sessionStorage`,
+      );
+    }
+    sessionStorage.setItem(this.key(newStorageKey), content);
+    sessionStorage.removeItem(this.key(oldStorageKey));
+  }
+
+  async hasKey(storageKey: string): Promise<boolean> {
+    return sessionStorage.getItem(this.key(storageKey)) !== null;
+  }
+}

src/providers/TourProvider/tourPipelineStorage/TourPipelineFolder.ts

@@ -0,0 +1,49 @@
+import { PipelineFile } from "@/services/pipelineStorage/PipelineFile";
+import { PipelineFolder } from "@/services/pipelineStorage/PipelineFolder";
+
+export class TourPipelineFolder extends PipelineFolder {
+  override async listPipelines(): Promise<PipelineFile[]> {
+    const descriptors = await this.driver.list();
+    return descriptors.map(
+      (d) =>
+        new PipelineFile({
+          id: d.storageKey,
+          storageKey: d.storageKey,
+          folder: this,
+          createdAt: d.createdAt,
+          modifiedAt: d.modifiedAt,
+        }),
+    );
+  }
+
+  override async findFile(
+    storageKey: string,
+  ): Promise<PipelineFile | undefined> {
+    if (!(await this.driver.hasKey(storageKey))) return undefined;
+    return new PipelineFile({
+      id: storageKey,
+      storageKey,
+      folder: this,
+    });
+  }
+
+  override async assignFile(storageKey: string): Promise<PipelineFile> {
+    return new PipelineFile({
+      id: storageKey,
+      storageKey,
+      folder: this,
+    });
+  }
+
+  override async addFile(
+    storageKey: string,
+    content: string,
+  ): Promise<PipelineFile> {
+    await this.driver.write(storageKey, content);
+    return new PipelineFile({
+      id: storageKey,
+      storageKey,
+      folder: this,
+    });
+  }
+}

src/providers/TourProvider/tourPipelineStorage/TourPipelineStorageProvider.tsx

@@ -0,0 +1,20 @@
+import type { ReactNode } from "react";
+import { useState } from "react";
+
+import { PipelineStorageCtx } from "@/services/pipelineStorage/PipelineStorageProvider";
+
+import { TourPipelineStorageService } from "./TourPipelineStorageService";
+
+export function TourPipelineStorageProvider({
+  children,
+}: {
+  children: ReactNode;
+}) {
+  const [service] = useState(() => new TourPipelineStorageService());
+
+  return (
+    <PipelineStorageCtx.Provider value={service}>
+      {children}
+    </PipelineStorageCtx.Provider>
+  );
+}

src/providers/TourProvider/tourPipelineStorage/TourPipelineStorageService.ts

@@ -0,0 +1,48 @@
+import type { PipelineFile } from "@/services/pipelineStorage/PipelineFile";
+import { PipelineFolder } from "@/services/pipelineStorage/PipelineFolder";
+import { PipelineStorageService } from "@/services/pipelineStorage/PipelineStorageService";
+
+import { TOUR_FOLDER_ID } from "./constants";
+import { SessionStoragePipelineDriver } from "./SessionStoragePipelineDriver";
+import { TourPipelineFolder } from "./TourPipelineFolder";
+
+export class TourPipelineStorageService extends PipelineStorageService {
+  constructor() {
+    super();
+    this.rootFolder = new TourPipelineFolder({
+      id: TOUR_FOLDER_ID,
+      name: "Tour",
+      parentId: null,
+      driver: new SessionStoragePipelineDriver(),
+    });
+  }
+
+  override async findPipelineById(id: string): Promise<PipelineFile> {
+    const file = await this.rootFolder.findFile(id);
+    if (!file) {
+      throw new Error(`Tour pipeline not found: ${id}`);
+    }
+    return file;
+  }
+
+  override async resolvePipelineByName(
+    name: string,
+  ): Promise<PipelineFile | undefined> {
+    return this.rootFolder.findFile(name);
+  }
+
+  override async findFolderById(id: string): Promise<PipelineFolder> {
+    if (id === TOUR_FOLDER_ID || id === this.rootFolder.id) {
+      return this.rootFolder;
+    }
+    throw new Error(`Folder not available in tour mode: ${id}`);
+  }
+
+  override async getAllFolders(): Promise<PipelineFolder[]> {
+    return [this.rootFolder];
+  }
+
+  override async getFavoriteFolders(): Promise<PipelineFolder[]> {
+    return [];
+  }
+}

src/providers/TourProvider/tourPipelineStorage/constants.ts

@@ -0,0 +1,2 @@
+export const SESSION_KEY_PREFIX = "tour-pipeline:";
+export const TOUR_FOLDER_ID = "__tour_folder__";

src/providers/TourProvider/tourPipelineStorage/resetAllTourPipelineState.ts

@@ -0,0 +1,30 @@
+import type { QueryClient } from "@tanstack/react-query";
+
+import { TOUR_PIPELINE_PREFIX } from "@/providers/TourProvider/tourPipelineLifecycle";
+import { EDITOR_SPEC_QUERY_KEY } from "@/routes/v2/pages/Editor/hooks/useLoadSpec";
+
+import { SESSION_KEY_PREFIX } from "./constants";
+
+export function resetAllTourPipelineState(queryClient: QueryClient): void {
+  const sessionKeys: string[] = [];
+  for (let i = 0; i < sessionStorage.length; i++) {
+    const key = sessionStorage.key(i);
+    if (key?.startsWith(SESSION_KEY_PREFIX)) {
+      sessionKeys.push(key);
+    }
+  }
+  for (const key of sessionKeys) {
+    sessionStorage.removeItem(key);
+  }
+
+  queryClient.removeQueries({
+    predicate: (query) => {
+      const [head, second] = query.queryKey;
+      return (
+        head === EDITOR_SPEC_QUERY_KEY &&
+        typeof second === "string" &&
+        second.startsWith(TOUR_PIPELINE_PREFIX)
+      );
+    },
+  });
+}