Fix "dev" script in web package

.changeset/legal-poems-wash.md

@@ -0,0 +1,5 @@
+---
+"@workflow/web-shared": patch
+---
+
+Set `"type": "module"` in package.json

.changeset/shiny-pens-knock.md

@@ -0,0 +1,5 @@
+---
+"@workflow/web": patch
+---
+
+Fix "dev" mode

packages/web-shared/package.json

@@ -11,6 +11,7 @@
     "access": "public"
   },
   "license": "Apache-2.0",
+  "type": "module",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
   "exports": {

packages/web/app/lib/workflow-api-client.ts

@@ -189,10 +189,12 @@ export function useWorkflowRuns(
   ]);
   const [maxPagesVisited, setMaxPagesVisited] = useState(1);
 
-  // Store PageResult for each page
+  // Store PageResult for each page.
+  // Initial isLoading is false so SSR and client hydration agree; after mount,
+  // the useEffect that triggers the first fetch will set it to true on the client.
   const [allPageResults, setAllPageResults] = useState<
     Map<number, PageResult<WorkflowRun>>
-  >(new Map([[0, { data: null, isLoading: true, error: null }]]));
+  >(new Map([[0, { data: null, isLoading: false, error: null }]]));
 
   // Cache for fetched pages - key is cursor (or 'initial' for first page)
   const pageCache = useRef<
@@ -412,10 +414,12 @@ export function useWorkflowHooks(
   ]);
   const [maxPagesVisited, setMaxPagesVisited] = useState(1);
 
-  // Store PageResult for each page
+  // Store PageResult for each page.
+  // Initial isLoading is false so SSR and client hydration agree; after mount
+  // the useEffect that triggers the first fetch will set it to true on the client.
   const [allPageResults, setAllPageResults] = useState<
     Map<number, PageResult<Hook>>
-  >(new Map([[0, { data: null, isLoading: true, error: null }]]));
+  >(new Map([[0, { data: null, isLoading: false, error: null }]]));
 
   // Cache for fetched pages - key is cursor (or 'initial' for first page)
   const pageCache = useRef<

packages/web/vite.config.ts

@@ -10,15 +10,22 @@ export default defineConfig(({ command, isSsrBuild }) => ({
   // can be installed and run without needing any of the UI dependencies
   // (Radix, lucide-react, etc.) at runtime. Only Node.js built-ins and
   // express (needed by server.js) remain external.
-  ssr:
-    command === 'build' && isSsrBuild
-      ? {
-          noExternal: true,
-          external: ['express'],
-        }
-      : undefined,
+  //
+  // During dev (`react-router dev`), Vite's SSR module runner evaluates
+  // modules using its ESM evaluator which cannot handle CJS packages
+  // (they fail with "module/exports is not defined"). We disable
+  // noExternal for dev so dependencies are loaded natively by Node.js.
+  ssr: {
+    noExternal: command === 'build' ? true : undefined,
+    external: ['express'],
+  },
   plugins: [tailwindcss(), reactRouter()],
   resolve: {
+    // Ensure all workspace packages resolve React from the same location
+    // to prevent duplicate React instances (which cause "Invalid hook call"
+    // errors). This is necessary during dev when noExternal is not set and
+    // linked workspace packages might resolve their own copy of React.
+    dedupe: ['react', 'react-dom'],
     alias: [{ find: '~', replacement: '/app' }],
   },
 }));