From 5cc2450faea40fc10309741b1fd9576baa7fc389 Mon Sep 17 00:00:00 2001 From: Dominic White Date: Wed, 24 Jun 2026 07:40:49 +0200 Subject: [PATCH 1/4] Remove support for collecting error messages from posthog Prevent post hog from collecting error message strings, only exception type. --- strix/telemetry/posthog.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/strix/telemetry/posthog.py b/strix/telemetry/posthog.py index 239e134e5..d1a9b4607 100644 --- a/strix/telemetry/posthog.py +++ b/strix/telemetry/posthog.py @@ -123,8 +123,6 @@ def end(report_state: "ReportState", exit_reason: str = "completed") -> None: ) -def error(error_type: str, error_msg: str | None = None) -> None: +def error(error_type: str) -> None: props = {**base_props(), "error_type": error_type} - if error_msg: - props["error_msg"] = error_msg _send("error", props) From d49d5c67f85097e8db46151a04adc22081c56be3 Mon Sep 17 00:00:00 2001 From: Dominic White Date: Wed, 24 Jun 2026 07:42:28 +0200 Subject: [PATCH 2/4] Refactor error function to remove error_msg parameter Remove optional error message parameter from error function. --- strix/telemetry/scarf.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/strix/telemetry/scarf.py b/strix/telemetry/scarf.py index 25e38c499..037494bb3 100644 --- a/strix/telemetry/scarf.py +++ b/strix/telemetry/scarf.py @@ -129,12 +129,10 @@ def end(report_state: ReportState, exit_reason: str = "completed") -> None: ) -def error(error_type: str, error_msg: str | None = None) -> None: +def error(error_type: str) -> None: props: dict[str, Any] = { **base_props(), "session": SESSION_ID, "error_type": error_type, } - if error_msg: - props["error_msg"] = error_msg _send("error", props) From fc2c633739fba1aa4e58085b56940f6f8eed80ab Mon Sep 17 00:00:00 2001 From: Dominic White Date: Wed, 24 Jun 2026 07:43:19 +0200 Subject: [PATCH 3/4] Simplify unhandled exception logging Remove exception message logging from unhandled exceptions. --- strix/interface/main.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/strix/interface/main.py b/strix/interface/main.py index 4eae05274..932ddcd92 100644 --- a/strix/interface/main.py +++ b/strix/interface/main.py @@ -822,8 +822,8 @@ def main() -> None: exit_reason = "interrupted" except Exception as e: exit_reason = "error" - posthog.error("unhandled_exception", str(e)) - scarf.error("unhandled_exception", str(e)) + posthog.error("unhandled_exception") + scarf.error("unhandled_exception") raise finally: report_state = get_global_report_state() From a2e6df87e234a67b519c21f161049f9098dfe7ba Mon Sep 17 00:00:00 2001 From: Dominic White Date: Wed, 24 Jun 2026 07:52:52 +0200 Subject: [PATCH 4/4] Apply greptile's suggestion to remove e As it's not longer referenced. --- strix/interface/main.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/strix/interface/main.py b/strix/interface/main.py index 932ddcd92..25df61882 100644 --- a/strix/interface/main.py +++ b/strix/interface/main.py @@ -820,7 +820,7 @@ def main() -> None: asyncio.run(run_tui(args)) except KeyboardInterrupt: exit_reason = "interrupted" - except Exception as e: + except Exception: exit_reason = "error" posthog.error("unhandled_exception") scarf.error("unhandled_exception")