fix: Prevent redirect to new DevPod Pro experience if support for it …

…is not present

desktop/src/contexts/DevPodContext/DevPodProvider.tsx

@@ -9,7 +9,7 @@ import { usePollWorkspaces } from "./workspaces"
 export type TDevpodContext = Readonly<{
   providers: TQueryResult<TProviders>
 }>
-export const DevPodContext = createContext<TDevpodContext>(null!)
+export const DevPodContext = createContext<TDevpodContext | null>(null)
 
 export function DevPodProvider({ children }: Readonly<{ children?: ReactNode }>) {
   usePollWorkspaces()

desktop/src/contexts/DevPodContext/useProviders.ts

@@ -3,8 +3,8 @@ import { TProviderManager } from "../../types"
 import { DevPodContext, TDevpodContext } from "./DevPodProvider"
 import { useProviderManager } from "./useProviderManager"
 
-export function useProviders(): [TDevpodContext["providers"], TProviderManager] {
-  const providers = useContext(DevPodContext).providers
+export function useProviders(): [TDevpodContext["providers"] | [undefined], TProviderManager] {
+  const providers = useContext(DevPodContext)?.providers ?? [undefined]
   const manager = useProviderManager()
 
   return [providers, manager]

desktop/src/lib/modals/useLoginProModal.tsx

@@ -1,6 +1,6 @@
 import { BottomActionBar, BottomActionBarError, Form, useStreamingTerminal } from "@/components"
-import { useProInstanceManager, useProInstances } from "@/contexts"
-import { exists, useFormErrors } from "@/lib"
+import { useProInstanceManager, useProInstances, useProviders } from "@/contexts"
+import { canHealthCheck, exists, useFormErrors } from "@/lib"
 import { Routes } from "@/routes"
 import {
   Box,
@@ -52,6 +52,7 @@ export function useLoginProModal() {
   >({})
   const { terminal, connectStream, clear: clearTerminal } = useStreamingTerminal({ fontSize: "sm" })
   const [[proInstances], { login, disconnect }] = useProInstances()
+  const [[providers]] = useProviders()
   const { isOpen, onClose, onOpen } = useDisclosure()
   const { handleSubmit, formState, register, reset, setValue } = useForm<TFormValues>({
     mode: "onBlur",
@@ -140,13 +141,19 @@ export function useLoginProModal() {
     resetModal()
 
     const proInstanceID = proInstances?.find((pro) => pro.provider === state.providerID)?.host
-    if (!proInstanceID) return
+    if (!proInstanceID || !state.providerID) return
 
-    // workaround for layout shift after closing modal, no clue why
-    setTimeout(() => {
-      navigate(Routes.toProInstance(proInstanceID))
-    }, 0)
-  }, [completeConfigureProvider, navigate, proInstances, resetModal, state.providerID])
+    const provider = providers?.[state.providerID]
+
+    // We only redirect to the new experience if the provider supports it.
+    // Support can be determined via canHealthCheck.
+    if (provider && canHealthCheck(provider.config)) {
+      // workaround for layout shift after closing modal, no clue why
+      setTimeout(() => {
+        navigate(Routes.toProInstance(proInstanceID))
+      }, 0)
+    }
+  }, [completeConfigureProvider, navigate, providers, proInstances, resetModal, state.providerID])
 
   const modal = useMemo(() => {
     return (

hack/rebuild.sh

@@ -29,6 +29,11 @@ if [[ $BUILD_PLATFORMS == *"linux"* ]]; then
     cp test/devpod-cli-linux-amd64 desktop/src-tauri/bin/devpod-cli-x86_64-unknown-linux-gnu
     cp test/devpod-cli-linux-arm64 desktop/src-tauri/bin/devpod-cli-aarch64-unknown-linux-gnu
 fi
+
+if [[ $BUILD_PLATFORMS == *"windows"* ]]; then
+    cp test/devpod-cli-windows-amd64 desktop/src-tauri/bin/devpod-cli-x86_64-pc-windows-msvc.exe
+fi
+
 # only copy if darwin is in BUILD_PLATFORMS
 if [[ $BUILD_PLATFORMS == *"darwin"* ]]; then
     cp test/devpod-cli-darwin-amd64 desktop/src-tauri/bin/devpod-cli-x86_64-apple-darwin