Fix update-check ordering and remove unused NewConnectionRouteScreen

- Swap isDevelopment/!isPackaged guard before hasUpdateFeedConfig in
  getAutoUpdateDisabledReason so dev builds get the specific 'packaged
  production builds' message instead of the generic 'no update feed' one.
- Update test to use hasUpdateFeedConfig: false for the dev-build case,
  reflecting real-world conditions.
- Delete unused NewConnectionRouteScreen.tsx which duplicated the actual
  route screen at app/connections/new.tsx.

Author: cursoragent

apps/desktop/src/updateState.test.ts

@@ -71,7 +71,7 @@ describe("getAutoUpdateDisabledReason", () => {
         platform: "darwin",
         appImage: undefined,
         disabledByEnv: false,
-        hasUpdateFeedConfig: true,
+        hasUpdateFeedConfig: false,
       }),
     ).toContain("packaged production builds");
   });

apps/desktop/src/updateState.ts

@@ -36,12 +36,12 @@ export function getAutoUpdateDisabledReason(args: {
   disabledByEnv: boolean;
   hasUpdateFeedConfig: boolean;
 }): string | null {
-  if (!args.hasUpdateFeedConfig) {
-    return "Automatic updates are not available because no update feed is configured.";
-  }
   if (args.isDevelopment || !args.isPackaged) {
     return "Automatic updates are only available in packaged production builds.";
   }
+  if (!args.hasUpdateFeedConfig) {
+    return "Automatic updates are not available because no update feed is configured.";
+  }
   if (args.disabledByEnv) {
     return "Automatic updates are disabled by the T3CODE_DISABLE_AUTO_UPDATE setting.";
   }