Fix: Definitively resolve UnboundLocalError for timestamp tracking

- Addresses a persistent `UnboundLocalError` related to `latest_overall_review_activity_dt`.
- While the variable was initialized at the top of `main()`, to ensure robust scoping within the loop that processes overall reviews, a temporary variable (`temp_latest_overall_review_dt`) is now used for accumulating the latest timestamp within that specific loop.
- The main `latest_overall_review_activity_dt` is then updated from this temporary variable if any overall review timestamps were processed.
- This change ensures the variable is always defined before use in the conditional check that was causing the error.

(This commit also includes prior changes for removing the absl-py dependency and refactoring logging, which were part of the same active development session.)

Author: google-labs-jules[bot]

scripts/print_github_reviews.py

@@ -430,22 +430,25 @@ def parse_repo_url(url_string):
     # Output overall reviews if any exist after filtering
     if filtered_overall_reviews:
         print("# Code Reviews\n\n") # Changed heading
+        # Explicitly re-initialize before this loop to be absolutely sure for the update logic within it.
+        # The main initialization at the top of main() handles the case where this block is skipped.
+        temp_latest_overall_review_dt = None # Use a temporary variable for this loop's accumulation
         for review in filtered_overall_reviews:
             user = review.get("user", {}).get("login", "Unknown user")
             submitted_at_str = review.get("submitted_at", "N/A") # Keep original string for printing
             state = review.get("state", "N/A")
             body = review.get("body", "").strip()
 
-            # Track latest overall review timestamp
+            # Track latest overall review timestamp within this block
             if submitted_at_str and submitted_at_str != "N/A":
                 try:
                     if sys.version_info < (3, 11):
                         dt_str_submitted = submitted_at_str.replace("Z", "+00:00")
                     else:
                         dt_str_submitted = submitted_at_str
                     current_review_submitted_dt = datetime.datetime.fromisoformat(dt_str_submitted)
-                    if latest_overall_review_activity_dt is None or current_review_submitted_dt > latest_overall_review_activity_dt:
-                        latest_overall_review_activity_dt = current_review_submitted_dt
+                    if temp_latest_overall_review_dt is None or current_review_submitted_dt > temp_latest_overall_review_dt:
+                        temp_latest_overall_review_dt = current_review_submitted_dt
                 except ValueError:
                     sys.stderr.write(f"Warning: Could not parse overall review submitted_at for --since suggestion: {submitted_at_str}\n")
 
@@ -462,6 +465,11 @@ def parse_repo_url(url_string):
                 print("\n### Summary Comment:")
                 print(body)
             print("\n---")
+
+        # After processing all overall reviews in this block, update the main variable
+        if temp_latest_overall_review_dt:
+            latest_overall_review_activity_dt = temp_latest_overall_review_dt
+
         # Add an extra newline to separate from line comments if any overall reviews were printed
         print("\n")