ℹ️ No critical issues — one minor operator-label inconsistency inline.

Reviewed changes — a dashboard-wide refactor that routes deploy (and many adjacent) hrefs through a new typed route-builder layer and enables Next typedRoutes.

• Add buildRoute + routes.projects.* builders — lib/navigation/routes/{shared,projects,index}.ts substitute [segment] brackets from Next's generated ParamMap, validate the pattern against the AppRoutes literal union (typo'd/removed routes fail to compile), and append queries via withQuery.
• Make withQuery generic — lib/navigation/url.ts preserves the base's template-literal type so builder outputs stay typed.
• Enable typedRoutes — next.config.js flips typedRoutes: true; package.json typecheck becomes next typegen && tsc.
• Swap ~50 call sites — sidebar/leaves, breadcrumbs, crumbs, deployment rows/dialogs/banners, OAuth + Stripe + auth redirects move from hand-built template-literal urls to routes.* calls, with as Route casts added at sinks that accept arbitrary strings (behaviorally no-ops).

Behavior is preserved across the call-site swaps I traced. The one thing worth a glance is the logs deployment-filter operator flipping from is to contains (inline) — confirmed cosmetic, not a results regression.

  ^{｜ Fix all ➔ ｜ Fix 👍s ➔ ｜ View workflow run ｜ Using Claude Opus ｜ 𝕏}

future suggestion: lib/navigation/types.ts keeps href: string even though leaves.ts now returns typed Route values. tightening this to Route (or a union) would propagate type safety into the sidebar without needing as Route casts in nav-row.tsx.

✅ No new issues found.

Reviewed changes — the delta since the prior pullfrog review (ae5d7d0): a fix for the logs deployment-filter operator the last review flagged, plus a merge of main.

• Unify the deployment-filter operator on is: — containsFilter is renamed to isFilter in lib/navigation/routes/projects.ts, so both routes.projects.logs and routes.projects.requests now emit deploymentId=is:${id}. This restores the logs link to its pre-PR is: and aligns the requests link with the requests/logs deployment-filter UI, which itself emits operator: "is". The change is functionally inert — use-sentinel-logs-query.ts:50-52 and the runtime-logs query both discard the operator and match the id exactly — so only the filter-chip label is affected, and it now matches what manual UI filtering produces.
• Drop the inert appId param from the logs table action — deployment-list-table-action.popover.constants.tsx no longer forwards appId on the runtime-logs link (the page never parsed it).
• Update projects.test.ts — the deployment-id assertion now expects is: for requests.
• Merge main — b8dbffacc pulls in an unrelated openapi_specs unique-index migration and an audit-filter button style tweak; both sit outside this PR's refactor scope.

Prior review feedback is addressed cleanly and the merge introduced no conflicts in the touched route files.

  ^{｜ View workflow run ｜ Using Claude Opus ｜ 𝕏}

✅ No new issues found.

Reviewed changes — the delta since the prior pullfrog review (2730839): the final commit 9b122e5 swaps the last batch of hand-rolled URLs to the routes.* builders and adds one UX hardening to the create-project flow.

• Swap remaining hand-rolled URLs to routes.* — delete-project.tsx, delete-app.tsx, project-actions.tsx (3 links), project-card.tsx, and the custom-domains/add.ts CONFLICT error message all move from template-literal paths to typed builders. Each maps 1:1 to the prior string.
• Harden the workspace lookup in delete dialogs — delete-project.tsx / delete-app.tsx switch from useWorkspace() + workspace?.slug to useWorkspaceNavigation() + workspace.slug; the new hook suspends/redirects on a null workspace, so this also removes a latent /undefined/... path during loading.
• Keep the create-project dialog up during navigation — create-project-dialog.tsx adds useTransition, drops the immediate onOpenChange(false), navigates inside startNavigation, and blocks dialog-close while submitting or navigating so the redirect isn't orphaned and the user isn't left on the old page.

The route swaps are behaviorally inert (or a small improvement in the delete dialogs), and the create-project dialog change is correct and intentional — the component unmounts once the new route mounts. The prior is:/contains: operator thread was already resolved and acknowledged.

  ^{｜ View workflow run ｜ Using Claude Opus ｜ 𝕏}