@shipfox/client-projects 4.0.0 → 6.0.1

package/src/routes/inputs.ts

@@ -0,0 +1,15 @@
+export interface ProjectRouteParams {
+  wid: string;
+  pid: string;
+}
+
+export function projectRouteParams(input: Record<string, unknown>): ProjectRouteParams {
+  const wid = stringParam(input.wid);
+  const pid = stringParam(input.pid);
+  if (!wid || !pid) throw new Error('Project route is missing required path parameters.');
+  return {wid, pid};
+}
+
+function stringParam(value: unknown): string | undefined {
+  return typeof value === 'string' && value.length > 0 ? value : undefined;
+}

package/src/routes/search.ts

@@ -0,0 +1,8 @@
+export interface ProjectsSearch {
+  search?: string;
+}
+
+export function validateProjectsSearch(input: Record<string, unknown>): ProjectsSearch {
+  const search = typeof input.search === 'string' ? input.search.trim() : '';
+  return search ? {search} : {};
+}

package/test/pages.tsx

@@ -15,7 +15,6 @@ import {type RenderResult, render} from '@testing-library/react';
 import {createStore, Provider as JotaiProvider} from 'jotai';
 import type {ReactElement} from 'react';
 import {CreateProjectPage} from '#pages/create-project-page.js';
-import {ProjectWorkflowsPage} from '#pages/project-workflows-page.js';
 import {ProjectsHubPage} from '#pages/projects-hub-page.js';
 
 // All test renders that exercise pages requiring `useActiveWorkspace()` mount
@@ -29,11 +28,7 @@ const authState: AuthState = {
   user: {
     id: '22222222-2222-4222-8222-222222222222',
     email: 'user@example.com',
-    name: null,
-    email_verified_at: new Date().toISOString(),
-    status: 'active',
-    created_at: new Date().toISOString(),
-    updated_at: new Date().toISOString(),
+    emailVerifiedAt: new Date().toISOString(),
   },
   workspaces: [{id: PROJECT_TEST_WID, name: 'Acme', membershipId: 'm-1'}],
 };
@@ -46,16 +41,6 @@ export function jsonResponse(body: unknown, init: ResponseInit = {}) {
   });
 }
 
-// The param-carrying routes below declare `$pid`/`$runId` in their path, so the value is
-// always present when their fallback page renders. A missing param means the harness
-// wired a route wrong — fail loudly instead of rendering a page with a fabricated id.
-function requireParam(value: string | undefined, name: string): string {
-  if (value === undefined) {
-    throw new Error(`Test router: route param "${name}" is missing`);
-  }
-  return value;
-}
-
 function createTestRouter(path: string, element: ReactElement) {
   const initialPath = path.split('?')[0] ?? path;
   const rootRoute = createRootRoute({component: Outlet});
@@ -114,7 +99,7 @@ function createTestRouter(path: string, element: ReactElement) {
         return element;
       }
 
-      return <ProjectWorkflowsPage projectId={requireParam(params.pid, 'pid')} />;
+      return <h1>Workflows</h1>;
     },
   });