refactor: remove implicit multiline string concatenation

Author: hesreallyhim

dashboard_action/runtime/scripts/collect.py

@@ -80,9 +80,9 @@ def __init__(
         self.retry_at_utc = retry_at_utc
         self.retry_source = source
         message = (
-            "GitHub secondary rate limit encountered for "
-            f"{url}. Do not retry until after "
-            f"{retry_at_utc.strftime('%Y-%m-%d %H:%M:%S UTC')} "
+            "GitHub secondary rate limit encountered for " +
+            f"{url}. Do not retry until after " +
+            f"{retry_at_utc.strftime('%Y-%m-%d %H:%M:%S UTC')} " +
             f"({retry_after_seconds}s, source: {source})."
         )
         super().__init__(message, response=response)
@@ -101,8 +101,8 @@ def __init__(
         self.response = response
         self.attempts = attempts
         message = (
-            "Repository traffic endpoint remained unavailable after "
-            f"{attempts} attempt(s): {url}. The repo may have been deleted, "
+            "Repository traffic endpoint remained unavailable after " +
+            f"{attempts} attempt(s): {url}. The repo may have been deleted, " +
             "renamed, transferred, or may no longer be accessible."
         )
         super().__init__(message, response=response)
@@ -177,8 +177,8 @@ def _write_step_summary(
         ])
         for warning in _NETWORK_WARNINGS:
             lines.append(
-                "- Attempt "
-                f"{warning['attempt']} for `{warning['url']}` failed with "
+                "- Attempt " +
+                f"{warning['attempt']} for `{warning['url']}` failed with " +
                 f"`{warning['error_type']}`: {warning['message']}"
             )
 
@@ -368,7 +368,7 @@ def fetch_json(
             if attempt < MAX_RETRIES:
                 wait = _retry_delay_with_jitter(attempt)
                 print(
-                    f"  Retry {attempt}/{MAX_RETRIES} after network error: {exc} "
+                    f"  Retry {attempt}/{MAX_RETRIES} after network error: {exc} " +
                     f"(sleeping {wait:.2f}s)"
                 )
                 time.sleep(wait)
@@ -433,8 +433,8 @@ def discover_repositories(headers: Headers) -> list[RepoMetadata]:
     discovered = []
     while True:
         url = (
-            f"{REPO_DISCOVERY_URL}?affiliation=owner,collaborator,organization_member"
-            f"&sort=updated&direction=desc&per_page={REPO_DISCOVERY_PAGE_SIZE}"
+            f"{REPO_DISCOVERY_URL}?affiliation=owner,collaborator,organization_member" +
+            f"&sort=updated&direction=desc&per_page={REPO_DISCOVERY_PAGE_SIZE}" +
             f"&page={page}"
         )
         page_rows = fetch_json(url, headers)
@@ -574,8 +574,8 @@ def resolve_repositories(
             print("Error: no eligible repositories remain in 'include_only'.")
             sys.exit(1)
         print(
-            "Repository discovery: "
-            f"{len(discovered)} accessible, {len(eligible)} eligible after filters, "
+            "Repository discovery: " +
+            f"{len(discovered)} accessible, {len(eligible)} eligible after filters, " +
             f"tracking {len(resolved)} from include_only."
         )
         return resolved, manifest, {
@@ -626,9 +626,9 @@ def resolve_repositories(
         state["auto_cutoff_created_at"] = ""
 
     print(
-        "Repository discovery: "
-        f"{len(discovered)} accessible, {len(eligible)} eligible after filters, "
-        f"tracking {len(resolved)} "
+        "Repository discovery: " +
+        f"{len(discovered)} accessible, {len(eligible)} eligible after filters, " +
+        f"tracking {len(resolved)} " +
         f"({explicit_count} explicit, {auto_count} automatic)."
     )
 
@@ -785,7 +785,7 @@ def collect_repo_metrics(
 
 def _fallback_repo_detail_warning(repo: str, exc: Exception) -> str:
     return (
-        f"{repo}: repository detail request failed ({exc}); "
+        f"{repo}: repository detail request failed ({exc}); " +
         "traffic collection continued and repo metrics used discovery fallback."
     )
 
@@ -847,7 +847,7 @@ def main() -> None:
             )
 
             print(
-                f"  OK: {len(vc_rows)} day(s), {len(ref_rows)} referrer(s), "
+                f"  OK: {len(vc_rows)} day(s), {len(ref_rows)} referrer(s), " +
                 f"{len(path_rows)} path(s), {len(metric_rows)} repo metric row(s)"
             )
         except SecondaryRateLimitError as exc:

dashboard_action/runtime/scripts/load_data.py

@@ -797,8 +797,8 @@ def _growth_insight_candidates(daily_rows, metric_rows=None, growth=None):
                 visitors=visitors,
                 downstream_delta=downstream,
                 text=(
-                    f"`{repo}` drew {views:,} views and {visitors:,} visitors "
-                    f"without downstream growth in the selected window."
+                    f"`{repo}` drew {views:,} views and {visitors:,} visitors " +
+                    "without downstream growth in the selected window."
                 ),
             )
 
@@ -813,7 +813,7 @@ def _growth_insight_candidates(daily_rows, metric_rows=None, growth=None):
                 traffic=views,
                 downstream_delta=downstream,
                 text=(
-                    f"`{repo}` added {downstream:+,} downstream signals on only "
+                    f"`{repo}` added {downstream:+,} downstream signals on only " +
                     f"{views:,} views."
                 ),
             )
@@ -831,7 +831,7 @@ def _growth_insight_candidates(daily_rows, metric_rows=None, growth=None):
                 clones=clones,
                 stargazers_delta=star_delta,
                 text=(
-                    f"`{repo}` is clone-heavy but star-light "
+                    f"`{repo}` is clone-heavy but star-light " +
                     f"({clones:,} clones, {star_delta:+,} stars)."
                 ),
             )
@@ -872,8 +872,8 @@ def _growth_insight_candidates(daily_rows, metric_rows=None, growth=None):
                 delta=subscriber_delta,
                 denominator=denom,
                 text=(
-                    f"`{repo}` watchers rose {subscriber_delta:+,} "
-                    f"in the selected window."
+                    f"`{repo}` watchers rose {subscriber_delta:+,} " +
+                    "in the selected window."
                 ),
             )
 
@@ -888,8 +888,8 @@ def _growth_insight_candidates(daily_rows, metric_rows=None, growth=None):
                 traffic=views,
                 downstream_delta=downstream,
                 text=(
-                    f"`{repo}` had a traffic spike shape ({views:,} views) "
-                    f"without stars, watchers, or forks moving."
+                    f"`{repo}` had a traffic spike shape ({views:,} views) " +
+                    "without stars, watchers, or forks moving."
                 ),
             )
 
@@ -904,7 +904,7 @@ def _growth_insight_candidates(daily_rows, metric_rows=None, growth=None):
                 traffic=views,
                 downstream_delta=downstream,
                 text=(
-                    f"`{repo}` gained {downstream:+,} downstream signals without "
+                    f"`{repo}` gained {downstream:+,} downstream signals without " +
                     f"a matching traffic spike ({views:,} views)."
                 ),
             )
@@ -971,7 +971,7 @@ def _window_change_candidate(repo, metric, values, min_floor):
         "delta": delta,
         "pct": pct_value,
         "text": (
-            f"`{repo}` {metric} {pct_text} over the last {window}d "
+            f"`{repo}` {metric} {pct_text} over the last {window}d " +
             f"({prev:,} -> {curr:,}, {delta:+,})."
         ),
     }
@@ -1013,7 +1013,7 @@ def _spike_candidate(repo, metric, values):
         "baseline": median,
         "delta": delta,
         "text": (
-            f"`{repo}` {metric} {direction} versus baseline "
+            f"`{repo}` {metric} {direction} versus baseline " +
             f"(latest {latest:,} vs trailing median {median:.0f})."
         ),
     }