cleanup

Author: mjhuff

api-client/src/runs/types.ts

@@ -204,7 +204,7 @@ export interface UpdateErrorRecoveryPolicyRequest {
 }
 
 export type UpdateErrorRecoveryPolicyResponse = Record<string, never>
-export type ErrorRecoveryPolicyResponse = ErrorRecoveryPolicy
+export type ErrorRecoveryPolicyResponse = UpdateErrorRecoveryPolicyRequest
 
 /**
  * Current Run State Data

app/src/organisms/ErrorRecoveryFlows/hooks/useRecoveryCommands.ts

@@ -229,8 +229,10 @@ export function useRecoveryCommands({
 
         return updateErrorRecoveryPolicy(ignorePolicyRules, 'append')
           .then(() => Promise.resolve())
-          .catch(() =>
-            Promise.reject(new Error('Failed to update recovery policy.'))
+          .catch((e: Error) =>
+            Promise.reject(
+              new Error(`Failed to update recovery policy: ${e.message}`)
+            )
           )
       } else {
         void proceedToRouteAndStep(RECOVERY_MAP.ERROR_WHILE_RECOVERING.ROUTE)

app/src/resources/runs/useUpdateRecoveryPolicyWithStrategy.ts

@@ -38,7 +38,7 @@ export function useUpdateRecoveryPolicyWithStrategy(
     strategy: UpdateErrorRecoveryPolicyWithStrategy['strategy']
   ) =>
     getErrorRecoveryPolicy(host as HostConfig, runId).then(res => {
-      const existingPolicyRules = res.data.policyRules.map(rule => ({
+      const existingPolicyRules = res.data.data.policyRules.map(rule => ({
         commandType: rule.matchCriteria.command.commandType,
         errorType: rule.matchCriteria.command.error.errorType,
         ifMatch: rule.ifMatch,
@@ -49,7 +49,7 @@ export function useUpdateRecoveryPolicyWithStrategy(
           case 'append':
             return [...existingPolicyRules, newPolicy]
           default: {
-            console.error('Handled policy strategy, using append.')
+            console.error('Unhandled policy strategy, defaulting to append.')
             return [...existingPolicyRules, newPolicy]
           }
         }