refactor(renterd-libs): move autopilot routes

.changeset/selfish-queens-fold.md

@@ -0,0 +1,7 @@
+---
+'@siafoundation/renterd-js': minor
+'@siafoundation/renterd-react': minor
+'@siafoundation/renterd-types': minor
+---
+
+The host checks field is now just HostAutopilotChecks.

.changeset/ten-lies-draw.md

@@ -0,0 +1,7 @@
+---
+'@siafoundation/renterd-js': minor
+'@siafoundation/renterd-react': minor
+'@siafoundation/renterd-types': minor
+---
+
+Autopilot config routes moved to the bus.

apps/renterd/contexts/hosts/dataset.ts

@@ -42,7 +42,7 @@ export function useDataset({
           blocklist: block,
           isAllowlistActive,
         }),
-        ...getAutopilotFields(host.checks?.autopilot),
+        ...getAutopilotFields(host.checks),
         location: sch?.location,
         countryCode: sch?.country_code,
         // selectable

libs/renterd-js/README.md

@@ -53,7 +53,6 @@ export async function example() {
 
   const hosts = await bus.hosts({
     data: {
-      autopilotID: 'autopilot',
       filterMode: 'allowed',
       usabilityMode: 'usable',
       addressContains: 'example.com',

libs/renterd-js/src/autopilot.ts

@@ -2,21 +2,15 @@ import {
   AutopilotConfigEvaluateParams,
   AutopilotConfigEvaluatePayload,
   AutopilotConfigEvaluateResponse,
-  AutopilotConfigParams,
-  AutopilotConfigPayload,
-  AutopilotConfigResponse,
-  AutopilotConfigUpdateParams,
-  AutopilotConfigUpdatePayload,
-  AutopilotConfigUpdateResponse,
   AutopilotStateParams,
   AutopilotStatePayload,
   AutopilotStateResponse,
   AutopilotTriggerParams,
   AutopilotTriggerPayload,
   AutopilotTriggerResponse,
-  autopilotConfigRoute,
   autopilotStateRoute,
   autopilotTriggerRoute,
+  autopilotConfigEvaluateRoute,
 } from '@siafoundation/renterd-types'
 import { buildRequestHandler, initAxios } from '@siafoundation/request'
 
@@ -35,21 +29,11 @@ export function Autopilot({
       AutopilotStatePayload,
       AutopilotStateResponse
     >(axios, 'get', autopilotStateRoute),
-    config: buildRequestHandler<
-      AutopilotConfigParams,
-      AutopilotConfigPayload,
-      AutopilotConfigResponse
-    >(axios, 'get', autopilotConfigRoute),
-    configUpdate: buildRequestHandler<
-      AutopilotConfigUpdateParams,
-      AutopilotConfigUpdatePayload,
-      AutopilotConfigUpdateResponse
-    >(axios, 'put', autopilotConfigRoute),
     configEvaluate: buildRequestHandler<
       AutopilotConfigEvaluateParams,
       AutopilotConfigEvaluatePayload,
       AutopilotConfigEvaluateResponse
-    >(axios, 'post', autopilotConfigRoute),
+    >(axios, 'post', autopilotConfigEvaluateRoute),
     trigger: buildRequestHandler<
       AutopilotTriggerParams,
       AutopilotTriggerPayload,

libs/renterd-js/src/bus.ts

@@ -248,6 +248,13 @@ import {
   HostScanPayload,
   HostScanResponse,
   busHostHostKeyScanRoute,
+  AutopilotConfigParams,
+  AutopilotConfigPayload,
+  AutopilotConfigResponse,
+  AutopilotConfigUpdateParams,
+  AutopilotConfigUpdatePayload,
+  AutopilotConfigUpdateResponse,
+  busAutopilotRoute,
 } from '@siafoundation/renterd-types'
 import { buildRequestHandler, initAxios } from '@siafoundation/request'
 
@@ -580,5 +587,15 @@ export function Bus({ api, password }: { api: string; password?: string }) {
       MultipartUploadAddPartPayload,
       MultipartUploadAddPartResponse
     >(axios, 'post', busMultipartPartRoute),
+    autopilotConfig: buildRequestHandler<
+      AutopilotConfigParams,
+      AutopilotConfigPayload,
+      AutopilotConfigResponse
+    >(axios, 'get', busAutopilotRoute),
+    autopilotConfigUpdate: buildRequestHandler<
+      AutopilotConfigUpdateParams,
+      AutopilotConfigUpdatePayload,
+      AutopilotConfigUpdateResponse
+    >(axios, 'put', busAutopilotRoute),
   }
 }

libs/renterd-js/src/example.ts

@@ -42,7 +42,6 @@ export async function example() {
 
   const hosts = await bus.hosts({
     data: {
-      autopilotID: 'autopilot-id',
       filterMode: 'allowed',
       usabilityMode: 'usable',
       addressContains: 'example.com',

libs/renterd-react/src/autopilot.ts

@@ -1,19 +1,12 @@
 import {
   useGetSwr,
   usePostSwr,
-  usePutFunc,
   HookArgsSwr,
   HookArgsCallback,
   HookArgsWithPayloadSwr,
-  delay,
   usePostFunc,
 } from '@siafoundation/react-core'
 import {
-  AutopilotConfigParams,
-  AutopilotConfigResponse,
-  AutopilotConfigUpdateParams,
-  AutopilotConfigUpdatePayload,
-  AutopilotConfigUpdateResponse,
   AutopilotConfigEvaluateParams,
   AutopilotConfigEvaluatePayload,
   AutopilotConfigEvaluateResponse,
@@ -22,9 +15,9 @@ import {
   AutopilotTriggerParams,
   AutopilotTriggerPayload,
   AutopilotTriggerResponse,
-  autopilotConfigRoute,
   autopilotStateRoute,
   autopilotTriggerRoute,
+  autopilotConfigEvaluateRoute,
 } from '@siafoundation/renterd-types'
 
 export function useAutopilotState(
@@ -36,45 +29,14 @@ export function useAutopilotState(
   })
 }
 
-export function useAutopilotConfig(
-  args?: HookArgsSwr<AutopilotConfigParams, AutopilotConfigResponse>
-) {
-  return useGetSwr({
-    ...args,
-    route: autopilotConfigRoute,
-  })
-}
-
-export function useAutopilotConfigUpdate(
-  args?: HookArgsCallback<
-    AutopilotConfigUpdateParams,
-    AutopilotConfigUpdatePayload,
-    AutopilotConfigUpdateResponse
-  >
-) {
-  return usePutFunc(
-    { ...args, route: autopilotConfigRoute },
-    async (mutate) => {
-      mutate((key) => key === autopilotConfigRoute)
-      // Might need a delay before revalidating status which returns whether
-      // or not autopilot is configured.
-      const func = async () => {
-        await delay(1000)
-        mutate((key) => key === autopilotStateRoute)
-      }
-      func()
-    }
-  )
-}
-
 export function useAutopilotConfigEvaluate(
   args: HookArgsWithPayloadSwr<
     AutopilotConfigEvaluateParams,
     AutopilotConfigEvaluatePayload,
     AutopilotConfigEvaluateResponse
   >
 ) {
-  return usePostSwr({ ...args, route: autopilotConfigRoute })
+  return usePostSwr({ ...args, route: autopilotConfigEvaluateRoute })
 }
 
 export function useAutopilotTrigger(

libs/renterd-react/src/bus.ts

@@ -21,6 +21,12 @@ import {
   AlertsDismissResponse,
   AlertsParams,
   AlertsResponse,
+  AutopilotConfigParams,
+  AutopilotConfigResponse,
+  AutopilotConfigUpdateParams,
+  AutopilotConfigUpdatePayload,
+  AutopilotConfigUpdateResponse,
+  busAutopilotRoute,
   BucketCreateParams,
   BucketCreatePayload,
   BucketCreateResponse,
@@ -241,6 +247,7 @@ import {
   HostScanResponse,
   busHostHostKeyScanRoute,
   Host,
+  autopilotStateRoute,
 } from '@siafoundation/renterd-types'
 
 // state
@@ -1007,3 +1014,33 @@ export function useMultipartUploadAddPart(
     }
   )
 }
+
+// autopilot
+
+export function useAutopilotConfig(
+  args?: HookArgsSwr<AutopilotConfigParams, AutopilotConfigResponse>
+) {
+  return useGetSwr({
+    ...args,
+    route: busAutopilotRoute,
+  })
+}
+
+export function useAutopilotConfigUpdate(
+  args?: HookArgsCallback<
+    AutopilotConfigUpdateParams,
+    AutopilotConfigUpdatePayload,
+    AutopilotConfigUpdateResponse
+  >
+) {
+  return usePutFunc({ ...args, route: busAutopilotRoute }, async (mutate) => {
+    mutate((key) => key === busAutopilotRoute)
+    // Might need a delay before revalidating status which returns whether
+    // or not autopilot is configured.
+    const func = async () => {
+      await delay(1000)
+      mutate((key) => key === autopilotStateRoute)
+    }
+    func()
+  })
+}

libs/renterd-types/src/autopilot.ts

@@ -2,8 +2,9 @@ import { BuildState } from './bus'
 import { AutopilotConfig, SettingsGouging, SettingsRedundancy } from './types'
 
 export const autopilotStateRoute = '/autopilot/state'
-export const autopilotConfigRoute = '/autopilot/config'
+export const autopilotConfigEvaluateRoute = '/autopilot/config/evaluate'
 export const autopilotTriggerRoute = '/autopilot/trigger'
+export const busAutopilotRoute = '/bus/autopilot'
 
 type AutopilotStatus = {
   configured: boolean

libs/renterd-types/src/types.ts

@@ -184,9 +184,7 @@ export type Host = {
   storedData: number
   resolvedAddresses?: string[]
   subnets?: string[]
-  checks?: {
-    autopilot: HostAutopilotChecks
-  }
+  checks?: HostAutopilotChecks
 }
 
 export type HostAutopilotChecks = {