Fix circular import by moving DEPLOY_PATTERNS to separate config module

Break circular dependency between main.py and services

Author: Taniya Mathur

scripts/sdlc/idp-cli/src/idp_cli/cli/main.py

@@ -1,13 +1,6 @@
 # Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
 # SPDX-License-Identifier: MIT-0
 
-# Global pattern configuration - modify these to control which patterns are deployed/tested
-DEPLOY_PATTERNS = {
-    "p1": "Pattern1 - Packet or Media processing with Bedrock Data Automation (BDA)",
-    # "p2": "Pattern2 - Packet processing with Textract and Bedrock",
-    # "p3": "Pattern3 - Packet processing with Textract, SageMaker(UDOP), and Bedrock"
-}
-
 import os
 import sys
 from idp_cli.service.install_service import InstallService

scripts/sdlc/idp-cli/src/idp_cli/config.py

@@ -0,0 +1,9 @@
+# Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
+# SPDX-License-Identifier: MIT-0
+
+# Global pattern configuration - modify these to control which patterns are deployed/tested
+DEPLOY_PATTERNS = {
+    "p1": "Pattern1 - Packet or Media processing with Bedrock Data Automation (BDA)",
+    # "p2": "Pattern2 - Packet processing with Textract and Bedrock",
+    # "p3": "Pattern3 - Packet processing with Textract, SageMaker(UDOP), and Bedrock"
+}

scripts/sdlc/idp-cli/src/idp_cli/service/install_service.py

@@ -363,7 +363,7 @@ def install(self, admin_email: str):
         Args:
             admin_email: Email address for the admin user
         """
-        from idp_cli.cli.main import DEPLOY_PATTERNS
+        from idp_cli.config import DEPLOY_PATTERNS
         patterns = DEPLOY_PATTERNS
 
         def deploy_pattern(suffix, pattern_name):
@@ -389,7 +389,7 @@ def deploy_pattern(suffix, pattern_name):
 
         all_patterns_succeeded = all(results.values())
         if all_patterns_succeeded:
-            logger.info("Both patterns installed successfully!")
+            logger.info("All configured patterns installed successfully!")
         else:
             logger.error(f"Some patterns failed: {results}")
 

scripts/sdlc/idp-cli/src/idp_cli/service/smoketest_service.py

@@ -16,7 +16,7 @@ def __init__(self,
                  file_path: str, 
                  verify_string: str):
 
-        from idp_cli.cli.main import DEPLOY_PATTERNS
+        from idp_cli.config import DEPLOY_PATTERNS
 
         self.stack_name_prefix = stack_name_prefix
         self.stack_names = [f"{stack_name_prefix}-{suffix}" for suffix in DEPLOY_PATTERNS.keys()]

scripts/sdlc/idp-cli/src/idp_cli/service/uninstall_service.py

@@ -14,12 +14,14 @@ class UninstallService():
     def __init__(self, stack_name_prefix: str,
                  account_id: str,
                  cfn_prefix: Optional[str] = "idp-dev"):
+        from idp_cli.config import DEPLOY_PATTERNS
+        
         self.stack_name_prefix = stack_name_prefix
         self.account_id = account_id
         self.cfn_prefix = cfn_prefix 
         self.region = os.environ.get('AWS_REGION', 'us-east-1')
         self.install_bucket_name = f"{self.cfn_prefix}-{self.account_id}-{self.region}"
-        self.stack_names = [f"{stack_name_prefix}-p1", f"{stack_name_prefix}-p2"]
+        self.stack_names = [f"{stack_name_prefix}-{suffix}" for suffix in DEPLOY_PATTERNS.keys()]
         logger.debug(f"stack_names: {self.stack_names}\naccount_id: {account_id}\ncfn_prefix: {cfn_prefix}\nregion:{self.region}")
 
     def uninstall(self):