Merge remote-tracking branch 'origin/main' into tr/auto_approve

Usage.md

@@ -100,7 +100,7 @@ python -m pr_agent.cli --pr_url=<pr_url>  /review --pr_reviewer.extra_instructio
 (2) You can print results locally, without publishing them, by setting in `configuration.toml`:
 ```
 [config]
-publish_output=true
+publish_output=false
 verbosity_level=2
 ```
 This is useful for debugging or experimenting with different tools.

pr_agent/secret_providers/__init__.py

@@ -2,15 +2,18 @@ from pr_agent.config_loader import get_settings
 
 
 def get_secret_provider():
-    try:
-        provider_id = get_settings().config.secret_provider
-    except AttributeError as e:
-        raise ValueError("secret_provider is a required attribute in the configuration file") from e
-    try:
-        if provider_id == 'google_cloud_storage':
+    if not get_settings().get("CONFIG.SECRET_PROVIDER"):
+        return None
+
+    provider_id = get_settings().config.secret_provider
+    if provider_id == 'google_cloud_storage':
+        try:
             from pr_agent.secret_providers.google_cloud_storage_secret_provider import GoogleCloudStorageSecretProvider
             return GoogleCloudStorageSecretProvider()
-        else:
-            raise ValueError(f"Unknown secret provider: {provider_id}")
-    except Exception as e:
-        raise ValueError(f"Failed to initialize secret provider {provider_id}") from e
+        except Exception as e:
+            raise ValueError(f"Failed to initialize google_cloud_storage secret provider {provider_id}") from e
+    else:
+        raise ValueError("Unknown SECRET_PROVIDER")
+
+
+

pr_agent/servers/bitbucket_app.py

@@ -26,7 +26,8 @@ from pr_agent.tools.pr_reviewer import PRReviewer
 
 setup_logger(fmt=LoggingFormat.JSON)
 router = APIRouter()
-secret_provider = get_secret_provider()
+secret_provider = get_secret_provider() if get_settings().get("CONFIG.SECRET_PROVIDER") else None
+
 
 async def get_bearer_token(shared_secret: str, client_key: str):
     try: