refactor(routes): remove legacy chat route compatibility (#143)

Author: Onur

api/main_create_owner_test.go

@@ -66,10 +66,10 @@ func TestCreateSpritzOwnerUsesIDAndOmitsEmail(t *testing.T) {
 	if _, exists := owner["email"]; exists {
 		t.Fatalf("expected owner.email to be omitted from response, got %#v", owner["email"])
 	}
-	if data["accessUrl"] != "http://tidal-ember.spritz-test.svc.cluster.local:8080/#chat/tidal-ember" {
+	if data["accessUrl"] != "http://tidal-ember.spritz-test.svc.cluster.local:8080/c/tidal-ember" {
 		t.Fatalf("expected accessUrl to prefer chat url, got %#v", data["accessUrl"])
 	}
-	if data["chatUrl"] != "http://tidal-ember.spritz-test.svc.cluster.local:8080/#chat/tidal-ember" {
+	if data["chatUrl"] != "http://tidal-ember.spritz-test.svc.cluster.local:8080/c/tidal-ember" {
 		t.Fatalf("expected chatUrl in response, got %#v", data["chatUrl"])
 	}
 	if data["instanceUrl"] != "http://tidal-ember.spritz-test.svc.cluster.local:8080" {

operator/api/v1/access_url.go

@@ -60,10 +60,10 @@ func ChatURLForSpritz(spritz *Spritz) string {
 		return parsed.String()
 	}
 
-	parsed.Path = "/"
-	parsed.RawPath = "/"
+	parsed.Path = fmt.Sprintf("/c/%s", name)
+	parsed.RawPath = parsed.Path
 	parsed.RawQuery = ""
-	parsed.Fragment = fmt.Sprintf("chat/%s", name)
+	parsed.Fragment = ""
 	return parsed.String()
 }
 

operator/api/v1/access_url_test.go

@@ -54,15 +54,15 @@ func TestAccessURLForSpritzPromotesChatURL(t *testing.T) {
 	}
 }
 
-func TestAccessURLForSpritzPreservesFragmentChatRouteWithoutIngress(t *testing.T) {
+func TestAccessURLForSpritzUsesCanonicalPathRouteWithoutIngress(t *testing.T) {
 	spritz := &Spritz{
 		ObjectMeta: metav1ObjectMeta("openclaw-tide-wind", "spritz-test"),
 		Spec: SpritzSpec{
 			Ports: []SpritzPort{{ContainerPort: 8080}},
 		},
 	}
 
-	want := "http://openclaw-tide-wind.spritz-test.svc.cluster.local:8080/#chat/openclaw-tide-wind"
+	want := "http://openclaw-tide-wind.spritz-test.svc.cluster.local:8080/c/openclaw-tide-wind"
 	if got := AccessURLForSpritz(spritz); got != want {
 		t.Fatalf("expected access url %q, got %q", want, got)
 	}

ui/src/App.test.tsx

@@ -31,7 +31,6 @@ function renderAtRoute(path: string) {
             <Route path="create" element={<div data-testid="create-page">Create Page</div>} />
             <Route path="terminal/:name" element={<div data-testid="terminal-page">Terminal Page</div>} />
             <Route path="c/:name?" element={<div data-testid="chat-page">Chat Page</div>} />
-            <Route path="chat/:name?" element={<div data-testid="chat-page">Chat Page</div>} />
           </Routes>
         </NoticeProvider>
       </ConfigProvider>

ui/src/App.tsx

@@ -1,64 +1,25 @@
-import { useEffect } from 'react';
-import { BrowserRouter, Routes, Route, useNavigate, useLocation } from 'react-router-dom';
+import { BrowserRouter, Routes, Route } from 'react-router-dom';
 import { ConfigProvider, config } from '@/lib/config';
 import { BrandingEffects } from '@/components/branding-effects';
 import { NoticeProvider } from '@/components/notice-banner';
 import { Layout } from '@/components/layout';
-import { chatConversationPath, chatPath } from '@/lib/urls';
 import { ChatPage } from '@/pages/chat';
 import { CreatePage } from '@/pages/create';
 import { TerminalPage } from '@/pages/terminal';
 
-function HashRedirect() {
-  const navigate = useNavigate();
-  const location = useLocation();
-
-  useEffect(() => {
-    const hash = window.location.hash;
-    if (!hash || hash === '#') return;
-
-    // Convert legacy hash routes to path routes.
-    // #chat/name → /c/name
-    // #create → /create
-    // #terminal/name → /terminal/name
-    const match = hash.match(/^#(chat|terminal|create)(?:\/(.+?))?(?:\/([^/]+))?$/);
-    if (match) {
-      const route = match[1];
-      const param = match[2] || '';
-      const subParam = match[3] || '';
-      const path = route === 'chat'
-        ? subParam
-          ? chatConversationPath(param, subParam)
-          : chatPath(param)
-        : subParam
-          ? `/${route}/${param}/${subParam}`
-          : param
-            ? `/${route}/${param}`
-            : `/${route}`;
-      window.history.replaceState(null, '', window.location.pathname + window.location.search);
-      navigate(path, { replace: true });
-    }
-  }, [location, navigate]);
-
-  return null;
-}
-
 export function App() {
   return (
     <BrowserRouter>
       <ConfigProvider value={config}>
         <BrandingEffects />
         <NoticeProvider>
-          <HashRedirect />
           <Routes>
             <Route element={<Layout />}>
               <Route index element={<ChatPage />} />
               <Route path="create" element={<CreatePage />} />
               <Route path="terminal/:name" element={<TerminalPage />} />
               <Route path="c/:name?" element={<ChatPage />} />
               <Route path="c/:name/:conversationId" element={<ChatPage />} />
-              <Route path="chat/:name?" element={<ChatPage />} />
-              <Route path="chat/:name/:conversationId" element={<ChatPage />} />
             </Route>
           </Routes>
         </NoticeProvider>