refactor: rename SINGLE retry strategy to SINGLE_RETRY which it now is called

Author: sorccu

packages/cli/src/constructs/check.ts

@@ -14,7 +14,7 @@ import {
   FixedRetryStrategy,
   LinearRetryStrategy,
   NoRetriesRetryStrategy,
-  SingleRetryStrategy,
+  SingleRetryRetryStrategy,
 } from './retry-strategy'
 import { AlertEscalation } from './alert-escalation-policy'
 import { IncidentTrigger } from './incident'
@@ -30,7 +30,7 @@ export type CheckRetryStrategy =
   | LinearRetryStrategy
   | ExponentialRetryStrategy
   | FixedRetryStrategy
-  | SingleRetryStrategy
+  | SingleRetryRetryStrategy
   | NoRetriesRetryStrategy
 
 /**

packages/cli/src/constructs/monitor.ts

@@ -7,7 +7,7 @@ import { CheckGroupV2 } from './check-group-v2'
 import { Frequency } from './frequency'
 import { IncidentTrigger } from './incident'
 import { PrivateLocation, PrivateLocationRef } from './private-location'
-import { NoRetriesRetryStrategy, RetryStrategyType, SingleRetryStrategy } from './retry-strategy'
+import { NoRetriesRetryStrategy, RetryStrategyType, SingleRetryRetryStrategy } from './retry-strategy'
 import { Check, CheckProps } from './check'
 import { Diagnostics } from './diagnostics'
 import { validateRemovedDoubleCheck } from './internal/common-diagnostics'
@@ -17,7 +17,7 @@ import { InvalidPropertyValueDiagnostic } from './construct-diagnostics'
  * Retry strategies supported by monitors.
  */
 export type MonitorRetryStrategy =
-  | SingleRetryStrategy
+  | SingleRetryRetryStrategy
   | NoRetriesRetryStrategy
 
 export interface MonitorProps extends Omit<CheckProps, 'doubleCheck'> {
@@ -118,7 +118,7 @@ export abstract class Monitor extends Check {
     await super.validate(diagnostics)
 
     if (this.retryStrategy) {
-      const supported: RetryStrategyType[] = ['SINGLE', 'NO_RETRIES']
+      const supported: RetryStrategyType[] = ['SINGLE_RETRY', 'NO_RETRIES']
       if (!supported.includes(this.retryStrategy.type)) {
         diagnostics.add(new InvalidPropertyValueDiagnostic(
           'retryStrategy',

packages/cli/src/constructs/retry-strategy-codegen.ts

@@ -102,7 +102,7 @@ export function valueForRetryStrategy (genfile: GeneratedFile, strategy?: RetryS
           })
         })
       })
-    case 'SINGLE':
+    case 'SINGLE_RETRY':
       return expr(ident('RetryStrategyBuilder'), builder => {
         builder.member(ident('singleRetry'))
         builder.call(builder => {

packages/cli/src/constructs/retry-strategy.ts

@@ -1,5 +1,10 @@
 /** Available retry strategy types */
-export type RetryStrategyType = 'LINEAR' | 'EXPONENTIAL' | 'FIXED' | 'SINGLE' | 'NO_RETRIES'
+export type RetryStrategyType =
+  | 'LINEAR'
+  | 'EXPONENTIAL'
+  | 'FIXED'
+  | 'SINGLE_RETRY'
+  | 'NO_RETRIES'
 
 /**
  * Conditions can be used to limit when a retry strategy applies.
@@ -18,10 +23,10 @@ export interface RetryStrategy {
   /**
    * The number of seconds to wait before the first retry attempt.
    * This value is used differently based on the retry strategy type:
-   * - FIXED: Same delay for all retries
+   * - FIXED: Same delay f'or all retries
    * - LINEAR: Base value that increases linearly (baseBackoffSeconds * attempt)
    * - EXPONENTIAL: Base value that increases exponentially (baseBackoffSeconds ^ attempt)
-   * - SINGLE: The delay for the first and only retry
+   * - SINGLE_RETRY: The delay for the first and only retry
    *
    * @defaultValue 60
    * @example
@@ -35,7 +40,7 @@ export interface RetryStrategy {
    * // Exponential: 5s, 25s, 125s
    * baseBackoffSeconds: 5
    *
-   * // Single: 10s
+   * // Single retry: 10s
    * baseBackoffSeconds: 10
    * ```
    */
@@ -120,16 +125,16 @@ export interface FixedRetryStrategy extends RetryStrategy {
 export type FixedRetryStrategyOptions = RetryStrategyOptions
 
 /**
- * Configuration for single retry strategies.
+ * Configuration for single retry retry strategies.
  */
-export interface SingleRetryStrategy extends Pick<RetryStrategy, 'type' | 'baseBackoffSeconds' | 'sameRegion'> {
-  type: 'SINGLE'
+export interface SingleRetryRetryStrategy extends Pick<RetryStrategy, 'type' | 'baseBackoffSeconds' | 'sameRegion'> {
+  type: 'SINGLE_RETRY'
 }
 
 /**
- * Options for configuring single retry strategy behavior.
+ * Options for configuring single retry retry strategy behavior.
  */
-export type SingleRetryStrategyOptions = Pick<RetryStrategyOptions, 'baseBackoffSeconds' | 'sameRegion'>
+export type SingleRetryRetryStrategyOptions = Pick<RetryStrategyOptions, 'baseBackoffSeconds' | 'sameRegion'>
 
 /**
  * Configuration for no retry retry strategies.
@@ -224,10 +229,10 @@ export class RetryStrategyBuilder {
   /**
    * A single retry will be performed.
    */
-  static singleRetry (options?: SingleRetryStrategyOptions): SingleRetryStrategy {
+  static singleRetry (options?: SingleRetryRetryStrategyOptions): SingleRetryRetryStrategy {
     const { baseBackoffSeconds, sameRegion } = RetryStrategyBuilder.defaults(options)
     return {
-      type: 'SINGLE',
+      type: 'SINGLE_RETRY',
       baseBackoffSeconds,
       sameRegion,
     }