trcc-linux 8.2.3__tar.gz → 8.2.4__tar.gz

{trcc_linux-8.2.3 → trcc_linux-8.2.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: trcc-linux
-Version: 8.2.3
+Version: 8.2.4
 Summary: Linux implementation of Thermalright LCD Control Center
 Project-URL: Homepage, https://github.com/Lexonight1/thermalright-trcc-linux
 Project-URL: Documentation, https://github.com/Lexonight1/thermalright-trcc-linux#readme
@@ -328,6 +328,7 @@ curl -X POST http://localhost:9876/led/color \
 |----------|-------------|
 | [Install Guide](doc/INSTALL_GUIDE.md) | Installation for all major distros |
 | [CLI Reference](doc/CLI_REFERENCE.md) | All CLI commands with options and examples |
+| [API Reference](doc/API_REFERENCE.md) | All 43 REST API endpoints with request/response models |
 | [Troubleshooting](doc/TROUBLESHOOTING.md) | Common issues and fixes |
 | [New to Linux](doc/NEW_TO_LINUX.md) | Guide for Linux beginners |
 | [Changelog](doc/CHANGELOG.md) | Version history |
@@ -413,7 +414,7 @@ A big thanks to everyone who has contributed invaluable reports to this project:
 
 Thanks to everyone who took a moment to star this project — it means the world.
 
-**[alessa-lara](https://github.com/alessa-lara)** · **[ArcaneCoder404](https://github.com/ArcaneCoder404)** · **[betolink](https://github.com/betolink)** · **[bive242](https://github.com/bive242)** · **[BrunoLeguizamon05](https://github.com/BrunoLeguizamon05)** · **[cancos1](https://github.com/cancos1)** · **[codeflitting](https://github.com/codeflitting)** · **[dabombUSA](https://github.com/dabombUSA)** · **[damachine](https://github.com/damachine)** · **[emaspa](https://github.com/emaspa)** · **[honjow](https://github.com/honjow)** · **[jezzaw007](https://github.com/jezzaw007)** · **[jhlasnik](https://github.com/jhlasnik)** · **[jmo808](https://github.com/jmo808)** · **[mgaruccio](https://github.com/mgaruccio)** · **[michael-spinelli](https://github.com/michael-spinelli)** · **[mkogut](https://github.com/mkogut)** · **[nathanielhernandez](https://github.com/nathanielhernandez)** · **[oddajpierscien](https://github.com/oddajpierscien)** · **[Pewful2021](https://github.com/Pewful2021)** · **[Pikarz](https://github.com/Pikarz)** · **[Rehaell](https://github.com/Rehaell)** · **[rslater](https://github.com/rslater)** · **[Smokemic](https://github.com/Smokemic)** · **[spiritofjon](https://github.com/spiritofjon)** · **[Vydon](https://github.com/Vydon)** · **[Xentrino](https://github.com/Xentrino)** · **[Ziusz](https://github.com/Ziusz)**
+**[alessa-lara](https://github.com/alessa-lara)** · **[ArcaneCoder404](https://github.com/ArcaneCoder404)** · **[betolink](https://github.com/betolink)** · **[bive242](https://github.com/bive242)** · **[BrunoLeguizamon05](https://github.com/BrunoLeguizamon05)** · **[cancos1](https://github.com/cancos1)** · **[codeflitting](https://github.com/codeflitting)** · **[dabombUSA](https://github.com/dabombUSA)** · **[damachine](https://github.com/damachine)** · **[emaspa](https://github.com/emaspa)** · **[honjow](https://github.com/honjow)** · **[jezzaw007](https://github.com/jezzaw007)** · **[jhlasnik](https://github.com/jhlasnik)** · **[jmo808](https://github.com/jmo808)** · **[knappstar](https://github.com/knappstar)** · **[mgaruccio](https://github.com/mgaruccio)** · **[michael-spinelli](https://github.com/michael-spinelli)** · **[mkogut](https://github.com/mkogut)** · **[nathanielhernandez](https://github.com/nathanielhernandez)** · **[oddajpierscien](https://github.com/oddajpierscien)** · **[Pewful2021](https://github.com/Pewful2021)** · **[Pikarz](https://github.com/Pikarz)** · **[Rehaell](https://github.com/Rehaell)** · **[rslater](https://github.com/rslater)** · **[Smokemic](https://github.com/Smokemic)** · **[spiritofjon](https://github.com/spiritofjon)** · **[Vydon](https://github.com/Vydon)** · **[Xentrino](https://github.com/Xentrino)** · **[Ziusz](https://github.com/Ziusz)**
 
 ## Faulkers
 

{trcc_linux-8.2.3 → trcc_linux-8.2.4}/README.md

@@ -269,6 +269,7 @@ curl -X POST http://localhost:9876/led/color \
 |----------|-------------|
 | [Install Guide](doc/INSTALL_GUIDE.md) | Installation for all major distros |
 | [CLI Reference](doc/CLI_REFERENCE.md) | All CLI commands with options and examples |
+| [API Reference](doc/API_REFERENCE.md) | All 43 REST API endpoints with request/response models |
 | [Troubleshooting](doc/TROUBLESHOOTING.md) | Common issues and fixes |
 | [New to Linux](doc/NEW_TO_LINUX.md) | Guide for Linux beginners |
 | [Changelog](doc/CHANGELOG.md) | Version history |
@@ -354,7 +355,7 @@ A big thanks to everyone who has contributed invaluable reports to this project:
 
 Thanks to everyone who took a moment to star this project — it means the world.
 
-**[alessa-lara](https://github.com/alessa-lara)** · **[ArcaneCoder404](https://github.com/ArcaneCoder404)** · **[betolink](https://github.com/betolink)** · **[bive242](https://github.com/bive242)** · **[BrunoLeguizamon05](https://github.com/BrunoLeguizamon05)** · **[cancos1](https://github.com/cancos1)** · **[codeflitting](https://github.com/codeflitting)** · **[dabombUSA](https://github.com/dabombUSA)** · **[damachine](https://github.com/damachine)** · **[emaspa](https://github.com/emaspa)** · **[honjow](https://github.com/honjow)** · **[jezzaw007](https://github.com/jezzaw007)** · **[jhlasnik](https://github.com/jhlasnik)** · **[jmo808](https://github.com/jmo808)** · **[mgaruccio](https://github.com/mgaruccio)** · **[michael-spinelli](https://github.com/michael-spinelli)** · **[mkogut](https://github.com/mkogut)** · **[nathanielhernandez](https://github.com/nathanielhernandez)** · **[oddajpierscien](https://github.com/oddajpierscien)** · **[Pewful2021](https://github.com/Pewful2021)** · **[Pikarz](https://github.com/Pikarz)** · **[Rehaell](https://github.com/Rehaell)** · **[rslater](https://github.com/rslater)** · **[Smokemic](https://github.com/Smokemic)** · **[spiritofjon](https://github.com/spiritofjon)** · **[Vydon](https://github.com/Vydon)** · **[Xentrino](https://github.com/Xentrino)** · **[Ziusz](https://github.com/Ziusz)**
+**[alessa-lara](https://github.com/alessa-lara)** · **[ArcaneCoder404](https://github.com/ArcaneCoder404)** · **[betolink](https://github.com/betolink)** · **[bive242](https://github.com/bive242)** · **[BrunoLeguizamon05](https://github.com/BrunoLeguizamon05)** · **[cancos1](https://github.com/cancos1)** · **[codeflitting](https://github.com/codeflitting)** · **[dabombUSA](https://github.com/dabombUSA)** · **[damachine](https://github.com/damachine)** · **[emaspa](https://github.com/emaspa)** · **[honjow](https://github.com/honjow)** · **[jezzaw007](https://github.com/jezzaw007)** · **[jhlasnik](https://github.com/jhlasnik)** · **[jmo808](https://github.com/jmo808)** · **[knappstar](https://github.com/knappstar)** · **[mgaruccio](https://github.com/mgaruccio)** · **[michael-spinelli](https://github.com/michael-spinelli)** · **[mkogut](https://github.com/mkogut)** · **[nathanielhernandez](https://github.com/nathanielhernandez)** · **[oddajpierscien](https://github.com/oddajpierscien)** · **[Pewful2021](https://github.com/Pewful2021)** · **[Pikarz](https://github.com/Pikarz)** · **[Rehaell](https://github.com/Rehaell)** · **[rslater](https://github.com/rslater)** · **[Smokemic](https://github.com/Smokemic)** · **[spiritofjon](https://github.com/spiritofjon)** · **[Vydon](https://github.com/Vydon)** · **[Xentrino](https://github.com/Xentrino)** · **[Ziusz](https://github.com/Ziusz)**
 
 ## Faulkers
 

{trcc_linux-8.2.3 → trcc_linux-8.2.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "trcc-linux"
-version = "8.2.3"
+version = "8.2.4"
 description = "Linux implementation of Thermalright LCD Control Center"
 readme = "README.md"
 license = "GPL-3.0-or-later"

{trcc_linux-8.2.3 → trcc_linux-8.2.4}/src/trcc/__version__.py

@@ -1,4 +1,4 @@
 """TRCC Linux version information. History in doc/CHANGELOG.md."""
 
-__version__ = "8.2.3"
+__version__ = "8.2.4"
 __version_info__ = tuple(int(x) for x in __version__.split("."))

{trcc_linux-8.2.3 → trcc_linux-8.2.4}/src/trcc/cli/_system.py

@@ -3,6 +3,7 @@ from __future__ import annotations
 
 import os
 import shlex
+import shutil
 import site
 import subprocess
 import sys
@@ -274,7 +275,6 @@ def setup_selinux():
     Required on SELinux-enforcing systems (Bazzite, Silverblue) where
     detach_kernel_driver() is silently blocked.
     """
-    import shutil
     import tempfile
 
     # Must be root
@@ -331,12 +331,11 @@ def setup_selinux():
     # Compile and install in temp directory
     try:
         with tempfile.TemporaryDirectory() as tmp:
-            import shutil as sh
             te_path = os.path.join(tmp, 'trcc_usb.te')
             mod_path = os.path.join(tmp, 'trcc_usb.mod')
             pp_path = os.path.join(tmp, 'trcc_usb.pp')
 
-            sh.copy2(te_src, te_path)
+            shutil.copy2(te_src, te_path)
 
             # checkmodule -M -m -o trcc_usb.mod trcc_usb.te
             r = subprocess.run(
@@ -379,8 +378,6 @@ def install_desktop():
     Reads the shipped .desktop file from the package assets directory.
     Works from both pip install and git clone.
     """
-    import shutil
-
     home = Path.home()
     app_dir = home / ".local" / "share" / "applications"
 
@@ -437,8 +434,6 @@ def setup_polkit():
     active desktop sessions can run dmidecode and smartctl without a
     password prompt. Requires root.
     """
-    import shutil
-
     if os.geteuid() != 0:
         return _sudo_reexec("setup-polkit")
 
@@ -494,9 +489,37 @@ def setup_polkit():
     return 0
 
 
+def _detect_install_method() -> str:
+    """Detect how trcc-linux was installed.
+
+    Returns 'pipx', 'pip', 'pacman', 'dnf', or 'apt'.
+    """
+    if 'pipx' in sys.prefix:
+        return 'pipx'
+    try:
+        from importlib.metadata import distribution
+        dist = distribution('trcc-linux')
+        installer = (dist.read_text('INSTALLER') or '').strip()
+        if installer == 'pip':
+            return 'pip'
+    except Exception:
+        pass
+    for mgr in ('pacman', 'dnf', 'apt'):
+        if shutil.which(mgr):
+            return mgr
+    return 'pip'
+
+
+def _is_externally_managed() -> bool:
+    """Check if the Python environment has PEP 668 EXTERNALLY-MANAGED marker."""
+    # The marker file lives next to the stdlib, e.g.
+    # /usr/lib/python3.12/EXTERNALLY-MANAGED
+    stdlib = Path(os.__file__).parent
+    return (stdlib / "EXTERNALLY-MANAGED").exists()
+
+
 def uninstall(*, yes: bool = False):
     """Remove all TRCC config, udev rules, autostart, and desktop files."""
-    import shutil
 
     from trcc.conf import Settings
 
@@ -560,12 +583,37 @@ def uninstall(*, yes: bool = False):
         subprocess.run(["udevadm", "control", "--reload-rules"], check=False)
         subprocess.run(["udevadm", "trigger"], check=False)
 
-    # Uninstall the pip package itself
-    print("\nUninstalling trcc-linux pip package...")
-    pip_cmd = [sys.executable, "-m", "pip", "uninstall", "trcc-linux"]
-    if yes:
-        pip_cmd.append("--yes")
-    subprocess.run(pip_cmd, check=False)
+    # Detect install method and uninstall the package accordingly
+    install_info = Settings.get_install_info()
+    method = install_info.get('method', _detect_install_method())
+
+    if method in ('pacman', 'dnf', 'apt'):
+        # System package — tell user to remove via their package manager
+        pkg_cmds = {
+            'pacman': 'sudo pacman -R trcc-linux',
+            'dnf': 'sudo dnf remove trcc-linux',
+            'apt': 'sudo apt remove trcc-linux',
+        }
+        print(f"\nInstalled via {method} — remove with:")
+        print(f"  {pkg_cmds[method]}")
+    elif method == 'pipx':
+        print("\nUninstalling trcc-linux via pipx...")
+        subprocess.run(["pipx", "uninstall", "trcc-linux"], check=False)
+    else:
+        # pip install — may need --break-system-packages on PEP 668 distros
+        print("\nUninstalling trcc-linux pip package...")
+        pip_cmd = [sys.executable, "-m", "pip", "uninstall", "trcc-linux"]
+        if yes:
+            pip_cmd.append("--yes")
+        if _is_externally_managed():
+            pip_cmd.append("--break-system-packages")
+        subprocess.run(pip_cmd, check=False)
+
+    # Clean stale shadow binary from old pip/pipx installs
+    stale_bin = Path.home() / ".local" / "bin" / "trcc"
+    if stale_bin.exists():
+        stale_bin.unlink()
+        print(f"Removed stale binary: {stale_bin}")
 
     return 0
 

{trcc_linux-8.2.3 → trcc_linux-8.2.4}/tests/README.md

@@ -1,6 +1,6 @@
 # TRCC Linux — Test Suite
 
-4021 tests across 56 files, organized to mirror `src/trcc/` hexagonal layers.
+4112 tests across 57 files, organized to mirror `src/trcc/` hexagonal layers.
 
 ## Running Tests
 
@@ -68,7 +68,8 @@ tests/
 │   ├── test_system.py
 │   └── test_theme.py
 ├── api/                         # REST API adapter
-│   └── test_api.py
+│   ├── test_api.py
+│   └── test_api_security.py
 ├── qt_components/               # GUI adapter
 │   ├── test_base.py
 │   ├── test_base_panel.py

{trcc_linux-8.2.3 → trcc_linux-8.2.4}/tests/cli/test_system.py

@@ -1110,7 +1110,9 @@ class TestUninstall:
              patch("os.geteuid", return_value=0), \
              patch("os.path.exists", return_value=False), \
              patch("trcc.cli._system.subprocess.run", return_value=_completed(0)), \
-             patch("trcc.conf.Settings.clear_installed_resolutions"):
+             patch("trcc.conf.Settings.clear_installed_resolutions"), \
+             patch("trcc.conf.Settings.get_install_info", return_value={'method': 'pip'}), \
+             patch("trcc.cli._system._is_externally_managed", return_value=False):
             rc = uninstall(yes=True)
         assert rc == 0
 
@@ -1123,7 +1125,9 @@ class TestUninstall:
              patch("os.path.exists", return_value=False), \
              patch("trcc.cli._system.subprocess.run",
                    side_effect=lambda cmd, **kw: calls.append(cmd) or _completed(0)), \
-             patch("trcc.conf.Settings.clear_installed_resolutions"):
+             patch("trcc.conf.Settings.clear_installed_resolutions"), \
+             patch("trcc.conf.Settings.get_install_info", return_value={'method': 'pip'}), \
+             patch("trcc.cli._system._is_externally_managed", return_value=False):
             uninstall(yes=True)
 
         pip_calls = [c for c in calls if "pip" in c and "uninstall" in c]
@@ -1138,7 +1142,9 @@ class TestUninstall:
              patch("os.path.exists", return_value=False), \
              patch("trcc.cli._system.subprocess.run",
                    side_effect=lambda cmd, **kw: calls.append(cmd) or _completed(0)), \
-             patch("trcc.conf.Settings.clear_installed_resolutions"):
+             patch("trcc.conf.Settings.clear_installed_resolutions"), \
+             patch("trcc.conf.Settings.get_install_info", return_value={'method': 'pip'}), \
+             patch("trcc.cli._system._is_externally_managed", return_value=False):
             uninstall(yes=True)
 
         pip_calls = [c for c in calls if "pip" in c and "uninstall" in c]
@@ -1153,7 +1159,9 @@ class TestUninstall:
              patch("os.path.exists", return_value=False), \
              patch("trcc.cli._system.subprocess.run",
                    side_effect=lambda cmd, **kw: calls.append(cmd) or _completed(0)), \
-             patch("trcc.conf.Settings.clear_installed_resolutions"):
+             patch("trcc.conf.Settings.clear_installed_resolutions"), \
+             patch("trcc.conf.Settings.get_install_info", return_value={'method': 'pip'}), \
+             patch("trcc.cli._system._is_externally_managed", return_value=False):
             uninstall(yes=False)
 
         pip_calls = [c for c in calls if "pip" in c and "uninstall" in c]
@@ -1171,7 +1179,9 @@ class TestUninstall:
              patch("os.geteuid", return_value=1000), \
              patch("os.path.exists", side_effect=lambda p: "/etc/udev" in str(p)), \
              patch("trcc.cli._system.subprocess.run", side_effect=fake_run), \
-             patch("trcc.conf.Settings.clear_installed_resolutions"):
+             patch("trcc.conf.Settings.clear_installed_resolutions"), \
+             patch("trcc.conf.Settings.get_install_info", return_value={'method': 'pip'}), \
+             patch("trcc.cli._system._is_externally_managed", return_value=False):
             uninstall(yes=True)
 
         sudo_rm_calls = [c for c in calls if "sudo" in c and "rm" in c]
@@ -1187,7 +1197,9 @@ class TestUninstall:
              patch("os.path.exists", side_effect=lambda p: "/etc/udev" in str(p)), \
              patch("os.remove", side_effect=lambda p: removed_paths.append(p)), \
              patch("trcc.cli._system.subprocess.run", return_value=_completed(0)), \
-             patch("trcc.conf.Settings.clear_installed_resolutions"):
+             patch("trcc.conf.Settings.clear_installed_resolutions"), \
+             patch("trcc.conf.Settings.get_install_info", return_value={'method': 'pip'}), \
+             patch("trcc.cli._system._is_externally_managed", return_value=False):
             uninstall(yes=True)
 
         assert any("udev" in str(p) for p in removed_paths)
@@ -1214,7 +1226,9 @@ class TestUninstall:
              patch("os.path.exists", side_effect=selective_exists), \
              patch("shutil.rmtree", side_effect=lambda p, **kw: removed.append(str(p))), \
              patch("trcc.cli._system.subprocess.run", return_value=_completed(0)), \
-             patch("trcc.conf.Settings.clear_installed_resolutions"):
+             patch("trcc.conf.Settings.clear_installed_resolutions"), \
+             patch("trcc.conf.Settings.get_install_info", return_value={'method': 'pip'}), \
+             patch("trcc.cli._system._is_externally_managed", return_value=False):
             uninstall(yes=True)
 
         assert any("trcc" in r for r in removed)
@@ -1226,7 +1240,9 @@ class TestUninstall:
              patch("os.geteuid", return_value=0), \
              patch("os.path.exists", return_value=False), \
              patch("trcc.cli._system.subprocess.run", return_value=_completed(0)), \
-             patch("trcc.conf.Settings.clear_installed_resolutions"):
+             patch("trcc.conf.Settings.clear_installed_resolutions"), \
+             patch("trcc.conf.Settings.get_install_info", return_value={'method': 'pip'}), \
+             patch("trcc.cli._system._is_externally_managed", return_value=False):
             uninstall(yes=True)
 
         out = capsys.readouterr().out
@@ -1246,7 +1262,9 @@ class TestUninstall:
              patch.object(Path, "exists", return_value=False), \
              patch("trcc.cli._system.subprocess.run",
                    side_effect=lambda cmd, **kw: calls.append(cmd) or _completed(0)), \
-             patch("trcc.conf.Settings.clear_installed_resolutions"):
+             patch("trcc.conf.Settings.clear_installed_resolutions"), \
+             patch("trcc.conf.Settings.get_install_info", return_value={'method': 'pip'}), \
+             patch("trcc.cli._system._is_externally_managed", return_value=False):
             uninstall(yes=True)
 
         udevadm_calls = [c for c in calls if "udevadm" in c]
@@ -1259,10 +1277,123 @@ class TestUninstall:
              patch("os.geteuid", return_value=0), \
              patch("os.path.exists", return_value=False), \
              patch("trcc.cli._system.subprocess.run", return_value=_completed(0)), \
-             patch("trcc.conf.Settings.clear_installed_resolutions") as mock_clear:
+             patch("trcc.conf.Settings.clear_installed_resolutions") as mock_clear, \
+             patch("trcc.conf.Settings.get_install_info", return_value={'method': 'pip'}), \
+             patch("trcc.cli._system._is_externally_managed", return_value=False):
             uninstall(yes=True)
         mock_clear.assert_called_once()
 
+    # --- Install method detection tests ---
+
+    def test_pacman_install_prints_instructions(self, tmp_path, capsys):
+        """System package installs print package manager command, not pip."""
+        home = self._base_patches(tmp_path)
+
+        with patch("trcc.cli._system.Path.home", return_value=home), \
+             patch("os.geteuid", return_value=0), \
+             patch("os.path.exists", return_value=False), \
+             patch("trcc.cli._system.subprocess.run", return_value=_completed(0)), \
+             patch("trcc.conf.Settings.clear_installed_resolutions"), \
+             patch("trcc.conf.Settings.get_install_info",
+                   return_value={'method': 'pacman'}):
+            uninstall(yes=True)
+
+        out = capsys.readouterr().out
+        assert "sudo pacman -R trcc-linux" in out
+
+    def test_dnf_install_prints_instructions(self, tmp_path, capsys):
+        home = self._base_patches(tmp_path)
+
+        with patch("trcc.cli._system.Path.home", return_value=home), \
+             patch("os.geteuid", return_value=0), \
+             patch("os.path.exists", return_value=False), \
+             patch("trcc.cli._system.subprocess.run", return_value=_completed(0)), \
+             patch("trcc.conf.Settings.clear_installed_resolutions"), \
+             patch("trcc.conf.Settings.get_install_info",
+                   return_value={'method': 'dnf'}):
+            uninstall(yes=True)
+
+        out = capsys.readouterr().out
+        assert "sudo dnf remove trcc-linux" in out
+
+    def test_pipx_install_uses_pipx_uninstall(self, tmp_path):
+        home = self._base_patches(tmp_path)
+        calls = []
+
+        with patch("trcc.cli._system.Path.home", return_value=home), \
+             patch("os.geteuid", return_value=0), \
+             patch("os.path.exists", return_value=False), \
+             patch("trcc.cli._system.subprocess.run",
+                   side_effect=lambda cmd, **kw: calls.append(cmd) or _completed(0)), \
+             patch("trcc.conf.Settings.clear_installed_resolutions"), \
+             patch("trcc.conf.Settings.get_install_info",
+                   return_value={'method': 'pipx'}):
+            uninstall(yes=True)
+
+        pipx_calls = [c for c in calls if "pipx" in c]
+        assert len(pipx_calls) >= 1
+
+    def test_pip_adds_break_system_packages_on_pep668(self, tmp_path):
+        """PEP 668 distros get --break-system-packages flag."""
+        home = self._base_patches(tmp_path)
+        calls = []
+
+        with patch("trcc.cli._system.Path.home", return_value=home), \
+             patch("os.geteuid", return_value=0), \
+             patch("os.path.exists", return_value=False), \
+             patch("trcc.cli._system.subprocess.run",
+                   side_effect=lambda cmd, **kw: calls.append(cmd) or _completed(0)), \
+             patch("trcc.conf.Settings.clear_installed_resolutions"), \
+             patch("trcc.conf.Settings.get_install_info", return_value={'method': 'pip'}), \
+             patch("trcc.cli._system._is_externally_managed", return_value=True):
+            uninstall(yes=True)
+
+        pip_calls = [c for c in calls if "pip" in c and "uninstall" in c]
+        assert any("--break-system-packages" in c for c in pip_calls)
+
+    def test_pip_no_break_system_packages_without_pep668(self, tmp_path):
+        home = self._base_patches(tmp_path)
+        calls = []
+
+        with patch("trcc.cli._system.Path.home", return_value=home), \
+             patch("os.geteuid", return_value=0), \
+             patch("os.path.exists", return_value=False), \
+             patch("trcc.cli._system.subprocess.run",
+                   side_effect=lambda cmd, **kw: calls.append(cmd) or _completed(0)), \
+             patch("trcc.conf.Settings.clear_installed_resolutions"), \
+             patch("trcc.conf.Settings.get_install_info", return_value={'method': 'pip'}), \
+             patch("trcc.cli._system._is_externally_managed", return_value=False):
+            uninstall(yes=True)
+
+        pip_calls = [c for c in calls if "pip" in c and "uninstall" in c]
+        assert all("--break-system-packages" not in c for c in pip_calls)
+
+    def test_stale_shadow_binary_removed(self, tmp_path):
+        """Old ~/.local/bin/trcc from pip/pipx gets cleaned up."""
+        home = self._base_patches(tmp_path)
+        stale = home / ".local" / "bin" / "trcc"
+        stale.parent.mkdir(parents=True)
+        stale.touch()
+
+        import os as _os
+        real_os_path_exists = _os.path.exists
+
+        def selective_exists(p):
+            if str(p).startswith("/etc") or str(p).startswith("/usr"):
+                return False
+            return real_os_path_exists(p)
+
+        with patch("trcc.cli._system.Path.home", return_value=home), \
+             patch("os.geteuid", return_value=0), \
+             patch("os.path.exists", side_effect=selective_exists), \
+             patch("trcc.cli._system.subprocess.run", return_value=_completed(0)), \
+             patch("trcc.conf.Settings.clear_installed_resolutions"), \
+             patch("trcc.conf.Settings.get_install_info",
+                   return_value={'method': 'pacman'}):
+            uninstall(yes=True)
+
+        assert not stale.exists()
+
 
 # ===========================================================================
 # TestReport