Remove /login route in favour of / landing page

apps/dashboard/src/components/layout/account-menu.tsx

@@ -162,7 +162,7 @@ export function AccountMenu({ user }: { user: AuthSessionUser }) {
           onClick={async () => {
             setOpen(false);
             await api.logout();
-            location.href = '/login';
+            location.href = '/';
           }}
         >
           <LogOut

apps/dashboard/src/lib/api.ts

@@ -55,8 +55,8 @@ async function request<T>(input: string, init?: RequestInit) {
   });
 
   if (response.status === 401) {
-    if (location.pathname !== '/login') {
-      location.href = '/login';
+    if (location.pathname !== '/') {
+      location.href = '/';
     }
     throw new Error('Unauthorized');
   }
@@ -92,8 +92,8 @@ async function requestWithMeta<T>(input: string, init?: RequestInit) {
   });
 
   if (response.status === 401) {
-    if (location.pathname !== '/login') {
-      location.href = '/login';
+    if (location.pathname !== '/') {
+      location.href = '/';
     }
     throw new Error('Unauthorized');
   }

apps/dashboard/src/routes.tsx

@@ -8,7 +8,6 @@ import type { NavItem } from '@client/nav';
 
 const LandingPage = React.lazy(() => import('./pages/landing').then(m => ({ default: m.LandingPage })));
 const DashboardPage = React.lazy(() => import('./pages/dashboard').then(m => ({ default: m.DashboardPage })));
-const LoginPage = React.lazy(() => import('./pages/login').then(m => ({ default: m.LoginPage })));
 const JobsPage = React.lazy(() => import('./pages/jobs').then(m => ({ default: m.JobsPage })));
 const JobDetailPage = React.lazy(() => import('./pages/job-detail').then(m => ({ default: m.JobDetailPage })));
 const JobLogsPage = React.lazy(() => import('./pages/job-logs').then(m => ({ default: m.JobLogsPage })));
@@ -31,11 +30,6 @@ export const publicRoutes: RouteObject[] = [
     element: withSuspense(LandingPage, true),
     errorElement: <RouteErrorBoundary />,
   },
-  {
-    path: '/login',
-    element: withSuspense(LoginPage, true),
-    errorElement: <RouteErrorBoundary />,
-  },
 ];
 
 // A boundary per child too, not just on the layout: React Router replaces the whole matched branch, so a branch-only boundary would take the sidebar and header down with a single page.

apps/worker/wrangler.jsonc

@@ -76,7 +76,6 @@
     "not_found_handling": "single-page-application",
     "run_worker_first": [
       "/",
-      "/login",
       "/dashboard*",
       "/jobs*",
       "/repos*",

test/e2e/dashboard.spec.tsx

@@ -3,7 +3,7 @@
  */
 import { expect, it, describe, vi, beforeEach } from 'vitest';
 import { render, screen } from '@testing-library/react';
-import { LoginPage } from '@client/pages/login';
+import { LandingPage } from '@client/pages/landing';
 import { DashboardPage } from '@client/pages/dashboard';
 import { MemoryRouter } from 'react-router-dom';
 import { api } from '@client/lib/api';
@@ -34,12 +34,12 @@ describe('Frontend UI Flows (JSDOM)', () => {
     render(
       <ThemeProvider>
         <MemoryRouter>
-          <LoginPage />
+          <LandingPage />
         </MemoryRouter>
       </ThemeProvider>
     );
 
-    const signInLink = screen.getByRole('link', { name: 'Sign in with GitHub' });
+    const signInLink = screen.getByRole('link', { name: /continue with github/i });
     expect(signInLink.getAttribute('href')).toBe('/auth/github');
   });
 

test/e2e/router-extensions.spec.tsx

@@ -41,7 +41,7 @@ describe('Dashboard route and nav registries (JSDOM)', () => {
   });
 
   it('exposes the built-in routes as composable arrays', () => {
-    expect(publicRoutes.map((r) => r.path)).toEqual(['/', '/login']);
+    expect(publicRoutes.map((r) => r.path)).toEqual(['/']);
     expect(shellRoutes.map((r) => r.path)).toContain('dashboard');
     expect([...publicRoutes, ...shellRoutes].map((r) => r.path)).not.toContain('*');
   });