ship: reset iter 1 - preserve valid search route

Author: arul28

apps/desktop/src/renderer/components/prs/prsRouteState.test.ts

@@ -175,4 +175,16 @@ describe("resolvePrsActiveTab", () => {
     expect(resolved.isWorkflowRoute).toBe(true);
     expect(resolved.activeTab).toBe("integration");
   });
+
+  it("ignores invalid hash route signals instead of dropping valid search params", () => {
+    const parsed = parsePrsRouteState({
+      search: "?tab=normal&prId=pr-123",
+      hash: "#/prs?tab=bogus",
+    });
+    const resolved = resolvePrsActiveTab(parsed);
+    expect(parsed.tab).toBe("normal");
+    expect(parsed.prId).toBe("pr-123");
+    expect(resolved.isWorkflowRoute).toBe(false);
+    expect(resolved.activeTab).toBe("normal");
+  });
 });

apps/desktop/src/renderer/components/prs/prsRouteState.ts

@@ -44,7 +44,8 @@ function parseOptionalId(value: string | null): string | null {
 export function parsePrsRouteState(args: { search?: string | null; hash?: string | null }): ParsedPrsRouteState {
   const searchParams = parseSearch(args.search ?? "");
   const hashParams = parseHashParams(args.hash ?? "");
-  const hashHasRouteSignal = hashParams.has("tab") || hashParams.has("workflow");
+  const hashHasRouteSignal =
+    parseTab(hashParams.get("tab")) !== null || parseWorkflowTab(hashParams.get("workflow")) !== null;
   const routeParams = hashHasRouteSignal ? hashParams : searchParams;
 
   const pick = (key: string): string | null => parseOptionalId(routeParams.get(key));