fix: fail open when not running on a PR (#196)

Author: JonZeolla

code_review.py

@@ -3,22 +3,24 @@
 salacious-code-reviews script entrypoint
 """
 
+import ast
+import json
 import os
 import re
-import json
-import ast
+
 import openai
-from github import Github, PullRequest, GithubException
+from github import Github, GithubException, PullRequest
 from openai import OpenAI
+
 from code_reviews import config, constants
 
 
 def get_github_session() -> Github:
     log.info("Creating GitHub Session...")
 
     if not os.getenv("GITHUB_TOKEN"):
-        log.error("Please provide a valid GITHUB_TOKEN environment variable!")
-        raise SystemExit(1)
+        log.error("No GITHUB_TOKEN environment variable was detected")
+        raise SystemExit(0)
 
     return Github(os.getenv("GITHUB_TOKEN"))
 
@@ -27,8 +29,8 @@ def get_openai_session() -> OpenAI:
     log.info("Setting up OpenAI Session...")
 
     if not os.getenv("OPENAI_API_KEY"):
-        log.error("Please provide a valid OPENAI_API_KEY environment variable!")
-        raise SystemExit(1)
+        log.error("No OPENAI_API_KEY environment variable was detected")
+        raise SystemExit(0)
 
     return OpenAI(api_key=os.getenv("OPENAI_API_KEY"))
 
@@ -42,7 +44,7 @@ def get_repo_and_pr() -> dict:
     log.info("Getting repo and pull request from runner environment...")
 
     if not github_ref:
-        log.error("Cannot find pull request, exiting!")
+        log.error("Failed to find the GITHUB_REF environment variable")
         raise SystemExit(1)
 
     # If the split from GITHUB_REF is not an int, exit
@@ -56,8 +58,8 @@ def get_repo_and_pr() -> dict:
             log.error(f"GITHUB_REF does not contain a valid PR number: {github_ref}")
             raise SystemExit(1)
     else:
-        log.error("GITHUB_REF does not reference a pull request")
-        raise SystemExit(1)
+        log.warn("Not running on a pull request; skipping the Salacious code review...")
+        raise SystemExit(0)
 
     if not github_repo:
         log.error("Cannot get repository name, exiting!")
@@ -209,10 +211,10 @@ def sanitize_json_string(json_string):
                 "ChatCompletion object does not contain expected 'choices' or 'message' structure"
             )
 
-    except openai.APIError as err:
-        log.error(f"Salacious failed due to API error: {err}")
     except openai.RateLimitError as err:
         log.error(f"Salacious failed due to an exceeded rate limit: {err}")
+    except openai.APIError as err:
+        log.error(f"Salacious failed due to API error: {err}")
     except Exception as e:
         log.error(f"Salacious failed due to unexpected error during API call: {str(e)}")