fix(cli[stop]): Exit with code 1 on SessionNotFound

command_stop caught SessionNotFound, printed the error, but returned
normally with exit code 0. This broke CI/scripting that relies on
non-zero exit codes to detect failures. Now calls sys.exit(1).

Author: tony

src/tmuxp/cli/stop.py

@@ -5,6 +5,7 @@
 import argparse
 import logging
 import os
+import sys
 import typing as t
 
 from libtmux.server import Server
@@ -122,7 +123,7 @@ def command_stop(
             raise exc.SessionNotFound(args.session_name)
     except TmuxpException as e:
         tmuxp_echo(colors.error(str(e)))
-        return
+        sys.exit(1)
 
     session_name = session.name
 

tests/cli/test_stop.py

@@ -62,13 +62,15 @@ def test_stop_nonexistent_session(
     monkeypatch: pytest.MonkeyPatch,
     capsys: pytest.CaptureFixture[str],
 ) -> None:
-    """Test stopping a session that doesn't exist shows error."""
+    """Test stopping a session that doesn't exist exits with code 1."""
     monkeypatch.delenv("TMUX", raising=False)
 
     assert server.socket_name is not None
 
-    cli.cli(["stop", "nonexistent", "-L", server.socket_name])
+    with pytest.raises(SystemExit) as exc_info:
+        cli.cli(["stop", "nonexistent", "-L", server.socket_name])
 
+    assert exc_info.value.code == 1
     captured = capsys.readouterr()
     assert "Session not found" in captured.out