From 53806da8c5d863ba4c506d939f790d4773104881 Mon Sep 17 00:00:00 2001 From: Chuck Date: Sat, 23 May 2026 16:33:24 -0400 Subject: [PATCH] Fix uninstall-no-record-file detection condition The previous check used a string replacement that left 'error:' in the remaining text, causing the condition to always evaluate false. Simplify to a direct substring check: if 'uninstall-no-record-file' appears in pip stderr the affected package is installed at the system level and we write the marker, suppressing the repeated warning on every restart. Co-Authored-By: Claude Sonnet 4.6 --- src/plugin_system/plugin_loader.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/plugin_system/plugin_loader.py b/src/plugin_system/plugin_loader.py index cfb51b5a..5ddc220a 100644 --- a/src/plugin_system/plugin_loader.py +++ b/src/plugin_system/plugin_loader.py @@ -176,7 +176,7 @@ class PluginLoader: # system level (e.g. installed via dnf/apt without a pip RECORD file). # pip can't replace it, but it IS installed — write the marker so we # don't retry on every restart. - if "uninstall-no-record-file" in stderr and "error" not in stderr.lower().replace("uninstall-no-record-file", ""): + if "uninstall-no-record-file" in stderr: self.logger.warning( "Dependencies for %s include system-managed packages (no pip RECORD). " "Assuming they are satisfied: %s",