feat: add all retry strategies to monitors, since they will be limited by plan instead

Author: sorccu

packages/cli/src/constructs/monitor.ts

@@ -7,16 +7,24 @@ import { CheckGroupV2 } from './check-group-v2'
 import { Frequency } from './frequency'
 import { IncidentTrigger } from './incident'
 import { PrivateLocation, PrivateLocationRef } from './private-location'
-import { NoRetriesRetryStrategy, RetryStrategyType, SingleRetryRetryStrategy } from './retry-strategy'
+import {
+  ExponentialRetryStrategy,
+  FixedRetryStrategy,
+  LinearRetryStrategy,
+  NoRetriesRetryStrategy,
+  SingleRetryRetryStrategy,
+} from './retry-strategy'
 import { Check, CheckProps } from './check'
 import { Diagnostics } from './diagnostics'
 import { validateRemovedDoubleCheck } from './internal/common-diagnostics'
-import { InvalidPropertyValueDiagnostic } from './construct-diagnostics'
 
 /**
  * Retry strategies supported by monitors.
  */
 export type MonitorRetryStrategy =
+  | LinearRetryStrategy
+  | ExponentialRetryStrategy
+  | FixedRetryStrategy
   | SingleRetryRetryStrategy
   | NoRetriesRetryStrategy
 
@@ -83,8 +91,6 @@ export interface MonitorProps extends Omit<CheckProps, 'doubleCheck'> {
    * Sets a retry policy for the monitor. Use RetryStrategyBuilder to create a
    * suitable retry strategy.
    *
-   * Note that monitors only support a single retry.
-   *
    * @example
    * ```typescript
    * // Single retry
@@ -114,29 +120,6 @@ export abstract class Monitor extends Check {
     await validateRemovedDoubleCheck(diagnostics, this)
   }
 
-  async validate (diagnostics: Diagnostics): Promise<void> {
-    await super.validate(diagnostics)
-
-    if (this.retryStrategy) {
-      const supported: RetryStrategyType[] = ['SINGLE_RETRY', 'NO_RETRIES']
-      if (!supported.includes(this.retryStrategy.type)) {
-        diagnostics.add(new InvalidPropertyValueDiagnostic(
-          'retryStrategy',
-          new Error(
-            `Monitors only support a single retry. The following options `
-            + `are available:`
-            + `\n\n`
-            + `  // Single retry\n`
-            + `  RetryStrategyBuilder.singleRetry()`
-            + `\n\n`
-            + `  // No retries\n`
-            + `  RetryStrategyBuilder.noRetries()`,
-          ),
-        ))
-      }
-    }
-  }
-
   synthesize () {
     return {
       ...super.synthesize(),