avoid unsetting optional fields in instance update

Author: david-crespo

app/pages/project/instances/AutoRestartCard.tsx

@@ -9,6 +9,7 @@
 import { formatDistanceToNow } from 'date-fns'
 import { type ReactNode } from 'react'
 import { useForm } from 'react-hook-form'
+import * as R from 'remeda'
 import { match } from 'ts-pattern'
 
 import {
@@ -76,9 +77,7 @@ export function AutoRestartCard() {
       path: { instance: instanceSelector.instance },
       query: { project: instanceSelector.project },
       body: {
-        ncpus: instance.ncpus,
-        memory: instance.memory,
-        bootDisk: instance.bootDiskId,
+        ...R.pick(instance, ['ncpus', 'memory', 'bootDiskId', 'cpuPlatform']),
         autoRestartPolicy: match(values.autoRestartPolicy)
           .with('default', () => undefined)
           .with('never', () => 'never' as const)

app/pages/project/instances/InstancePage.tsx

@@ -9,6 +9,7 @@ import { filesize } from 'filesize'
 import { useId, useState } from 'react'
 import { useForm } from 'react-hook-form'
 import { Link, useNavigate, type LoaderFunctionArgs } from 'react-router'
+import * as R from 'remeda'
 
 import {
   apiQueryClient,
@@ -341,7 +342,12 @@ export function ResizeInstanceModal({
     instanceUpdate.mutate({
       path: { instance: instance.name },
       query: { project },
-      body: { ncpus, memory: memory * GiB, bootDisk: instance.bootDiskId },
+      body: {
+        ncpus,
+        memory: memory * GiB,
+        // these need to be passed in or they'll be unset
+        ...R.pick(instance, ['bootDiskId', 'cpuPlatform', 'autoRestartPolicy']),
+      },
     })
   })
   const formId = useId()

app/pages/project/instances/StorageTab.tsx

@@ -165,10 +165,13 @@ export default function StorageTab() {
                 path: { instance: instance.id },
                 body: {
                   bootDisk: undefined,
-                  ncpus: instance.ncpus,
-                  memory: instance.memory,
-                  // this would get unset if we left it out
-                  autoRestartPolicy: instance.autoRestartPolicy,
+                  ...R.pick(instance, [
+                    'ncpus',
+                    'memory',
+                    // these would be unset if we left them out
+                    'autoRestartPolicy',
+                    'cpuPlatform',
+                  ]),
                 },
               }),
             errorTitle: 'Could not unset boot disk',
@@ -229,10 +232,13 @@ export default function StorageTab() {
                 path: { instance: instance.id },
                 body: {
                   bootDisk: disk.id,
-                  ncpus: instance.ncpus,
-                  memory: instance.memory,
-                  // this would get unset if we left it out
-                  autoRestartPolicy: instance.autoRestartPolicy,
+                  ...R.pick(instance, [
+                    'ncpus',
+                    'memory',
+                    // these would be unset if we left them out
+                    'autoRestartPolicy',
+                    'cpuPlatform',
+                  ]),
                 },
               }),
             errorTitle: `Could not ${verb} boot disk`,