Fix unreachable fallback values in resolveServerUrl and stale null cache in RepositoryIdentityResolver

- Remove dead-code fallback arguments in resolveServerUrl since
  resolvePrimaryEnvironmentBootstrapUrl() already contains a complete
  fallback chain and always returns a non-empty string.

- Skip caching null identity results in RepositoryIdentityResolver so
  that directories without a git remote are re-resolved on subsequent
  calls, allowing later-configured remotes to be picked up.

Author: cursoragent

apps/server/src/project/Layers/RepositoryIdentityResolver.ts

@@ -115,12 +115,14 @@ export const makeRepositoryIdentityResolver = Effect.gen(function* () {
     }
 
     const resolved = yield* Effect.promise(() => resolveRepositoryIdentity(cwd));
-    yield* Ref.update(cacheRef, (current) => {
-      const next = new Map(current);
-      next.set(cwd, resolved.identity);
-      next.set(resolved.cacheKey, resolved.identity);
-      return next;
-    });
+    if (resolved.identity !== null) {
+      yield* Ref.update(cacheRef, (current) => {
+        const next = new Map(current);
+        next.set(cwd, resolved.identity);
+        next.set(resolved.cacheKey, resolved.identity);
+        return next;
+      });
+    }
     return resolved.identity;
   });
 

apps/web/src/lib/utils.ts

@@ -53,12 +53,7 @@ export const resolveServerUrl = (options?: {
   pathname?: string | undefined;
   searchParams?: Record<string, string> | undefined;
 }): string => {
-  const rawUrl = firstNonEmptyString(
-    options?.url,
-    resolvePrimaryEnvironmentBootstrapUrl(),
-    import.meta.env.VITE_WS_URL,
-    window.location.origin,
-  );
+  const rawUrl = firstNonEmptyString(options?.url, resolvePrimaryEnvironmentBootstrapUrl());
 
   const parsedUrl = new URL(rawUrl);
   if (options?.protocol) {