Roll status and get_log_paths into list (#31)

* CLAUDE.md updates

* Unify list,status,get_log_paths to just list

* Fixes, debug logging

* Probably fix busted run()

* Claude

Author: irskep

CLAUDE.md

@@ -1,8 +1,17 @@
 # Claude Development Notes
 
+## Structure
+
+Typically begin by reading README.md.
+
+Key files:
+- persistproc/cli.py
+- persistproc/tools.py
+- persistproc/process_manager.py
+
 ## Running Tests
 
-Use `uv run python -m pytest` to run tests.
+Use `uv run python -m pytest` to run tests. For development, use `uv run python -m pytest -x --maxfail=3` to stop after 3 failures.
 
 ## Linting and Type Checking
 
@@ -18,4 +27,29 @@ Instead:
 - Use the test helpers in `tests/helpers.py` for programmatic testing
 - Debug issues through test output and logging
 
-**NEVER background a process with an `&` suffix.**
+**NEVER background a process with an `&` suffix.**
+
+**ALWAYS use `git --no-pager diff` for all diffs, never `git diff`.**
+
+If you get a timeout in tests while running locally, it is NOT a "timing issue" or "race condition." It is as REAL BUG. You are running on a fast computer that is not under load. The correct response to a test timeout is to ADD DEBUG LOGGING and THINK HARD.
+
+## Workflow guidelines
+
+For EVERY programming task assigned, you are NOT FINISHED until you can produce a message in the following format:
+
+<ReportFormat>
+After-action report for (task title here)
+
+Relevant files found:
+- (list them)
+
+(1-3 paragraphs justifying why the change is both correct and comprehensive)
+
+Steps taken to verify:
+- (list them)
+
+Web links supporting my changes:
+- (list them)
+
+I solemnly swear there are no further steps I can take to verify the changes within the boundaries set for me.
+</ReportFormat>

README.md

@@ -97,12 +97,10 @@ The server exposes the following tools:
 | Tool | Description |
 | --- | --- |
 | start | Start a new long-running process. |
-| list | List all managed processes and their status. |
-| status | Get the detailed status of a specific process. |
+| list | List all managed processes and their status. Can optionally filter by pid, command, or working directory and provides log paths. |
 | stop | Stop a running process. |
 | restart | Stops a process and starts it again with the same parameters. |
 | output | Retrieve captured output from a process. |
-| get_log_paths | Get the paths to the log files for a specific process. |
 | kill_persistproc | Kill all managed processes and get the PID of the persistproc server. |
 
 ## Getting started

docs/index.md

@@ -96,12 +96,10 @@ The server exposes the following tools:
 | Tool | Description |
 | --- | --- |
 | start | Start a new long-running process. |
-| list | List all managed processes and their status. |
-| status | Get the detailed status of a specific process. |
+| list | List all managed processes and their status. Can optionally filter by pid, command, or working directory and provides log paths. |
 | stop | Stop a running process. |
 | restart | Stops a process and starts it again with the same parameters. |
 | output | Retrieve captured output from a process. |
-| get_log_paths | Get the paths to the log files for a specific process. |
 | kill_persistproc | Kill all managed processes and get the PID of the persistproc server. |
 
 ## Getting started

docs/tools.md

@@ -213,12 +213,14 @@ Add a custom label for complex commands:
 
 ## `list`
 
-List all managed processes and their status.
+List all managed processes and their status, optionally filtered by pid, command, or working directory. Provides log paths for each process.
 
 <!-- persistproc list --help -->
 ```
 usage: persistproc list [-h] [--port PORT] [--data-dir DATA_DIR] [-v] [-q]
-                        [--format {text,json}]
+                        [--format {text,json}] [--pid PID]
+                        [--command-or-label COMMAND_OR_LABEL]
+                        [--working-directory WORKING_DIRECTORY]
 
 options:
   -h, --help            show this help message and exit
@@ -231,84 +233,55 @@ options:
                         warnings and errors.
   --format {text,json}  Output format (default: text; env:
                         $PERSISTPROC_FORMAT)
+  --pid PID             Filter by process ID
+  --command-or-label COMMAND_OR_LABEL
+                        Filter by command or label
+  --working-directory WORKING_DIRECTORY
+                        Filter by working directory
 ```
 
 **Examples**
 
+List all processes:
+
 ```bash
 > persistproc list
 PID 12345: npm run dev in /Users/user/myproject (running)
 Command: npm run dev
 Working directory: /Users/user/myproject
+Stdout log: /Users/user/Library/Application Support/persistproc/logs/12345_stdout.log
+Stderr log: /Users/user/Library/Application Support/persistproc/logs/12345_stderr.log
+Combined log: /Users/user/Library/Application Support/persistproc/logs/12345_combined.log
 
 PID 67890: python -m http.server 8080 in /Users/user/docs (running)
 Command: python -m http.server 8080
 Working directory: /Users/user/docs
 ```
 
-Get more detailed output or different formats:
+Filter by specific process ID:
 
 ```bash
-> persistproc list -v
-> persistproc list --format json
-```
-
-## `status`
-
-Get the detailed status of a specific process.
-
-<!-- persistproc status --help -->
-```
-usage: persistproc status [-h] [--port PORT] [--data-dir DATA_DIR] [-v] [-q]
-                          [--format {text,json}]
-                          [--working-directory WORKING_DIRECTORY]
-                          TARGET [args ...]
-
-positional arguments:
-  TARGET                The PID, label, or command to get status for.
-  args                  Arguments to the command
-
-options:
-  -h, --help            show this help message and exit
-  --port PORT           Server port (default: 8947; env: $PERSISTPROC_PORT)
-  --data-dir DATA_DIR   Data directory (default:
-                        ~/Library/Application Support/persistproc;
-                        env: $PERSISTPROC_DATA_DIR)
-  -v, --verbose         Increase verbosity; you can use -vv for more
-  -q, --quiet           Decrease verbosity. Passing -q once will show only
-                        warnings and errors.
-  --format {text,json}  Output format (default: text; env:
-                        $PERSISTPROC_FORMAT)
-  --working-directory WORKING_DIRECTORY
-                        The working directory for the process.
+> persistproc list --pid 12345
 ```
 
-**Examples**
-
-Get status by PID, command, or label:
+Filter by command or label:
 
 ```bash
-> persistproc status 12345
-PID: 12345
-Command: npm run dev
-Working directory: /Users/user/myproject
-Status: running
-Label: npm run dev in /Users/user/myproject
-
-> persistproc status npm run dev
-> persistproc status "my-dev-server"
+> persistproc list --command "npm run dev"
+> persistproc list --command "my-dev-server"
 ```
 
-Add working directory context when matching by command:
+Filter by working directory:
 
 ```bash
-> persistproc status --working-directory /path/to/project npm run dev
+> persistproc list --working-directory /Users/user/myproject
 ```
 
-Get structured output:
+Get more detailed output or different formats:
 
 ```bash
-> persistproc status --format json 12345
+> persistproc list -v
+> persistproc list --format json
 ```
 
 ## `stop`
@@ -507,55 +480,6 @@ Specify working directory context when matching by command:
 > persistproc output --working-directory /path/to/project npm run dev
 ```
 
-## `get_log_paths`
-
-Get the paths to the log files for a specific process.
-
-<!-- persistproc get_log_paths --help -->
-```
-usage: persistproc get_log_paths [-h] [--port PORT] [--data-dir DATA_DIR] [-v]
-                                 [-q] [--format {text,json}]
-                                 [--working-directory WORKING_DIRECTORY]
-                                 TARGET [args ...]
-
-positional arguments:
-  TARGET                The PID, label, or command to get log paths for.
-  args                  Arguments to the command
-
-options:
-  -h, --help            show this help message and exit
-  --port PORT           Server port (default: 8947; env: $PERSISTPROC_PORT)
-  --data-dir DATA_DIR   Data directory (default:
-                        ~/Library/Application Support/persistproc;
-                        env: $PERSISTPROC_DATA_DIR)
-  -v, --verbose         Increase verbosity; you can use -vv for more
-  -q, --quiet           Decrease verbosity. Passing -q once will show only
-                        warnings and errors.
-  --format {text,json}  Output format (default: text; env:
-                        $PERSISTPROC_FORMAT)
-  --working-directory WORKING_DIRECTORY
-                        The working directory for the process.
-```
-
-**Examples**
-
-Get log file paths by PID, command, or label:
-
-```bash
-> persistproc get_log_paths 12345
-Stdout log: /Users/user/Library/Application Support/persistproc/logs/12345_stdout.log
-Stderr log: /Users/user/Library/Application Support/persistproc/logs/12345_stderr.log
-
-> persistproc get_log_paths npm run dev
-> persistproc get_log_paths "my-dev-server"
-```
-
-Add working directory context when matching by command:
-
-```bash
-> persistproc get_log_paths --working-directory /path/to/project npm run dev
-```
-
 ## `kill_persistproc`
 
 Kill all managed processes and get the PID of the persistproc server.

persistproc/cli.py

@@ -1,4 +1,5 @@
 import argparse
+import logging
 import os
 import shlex
 import sys
@@ -384,6 +385,11 @@ def cli() -> None:
     """Main CLI entry point."""
     try:
         action, metadata = parse_cli(sys.argv[1:])
+
+        logger = logging.getLogger(__name__)
+        logger.debug("action: %s", repr(action))
+        logger.debug("metadata: %s", repr(metadata))
+
         handle_cli_action(action, metadata)
     except SystemExit as e:
         if e.code != 0:

persistproc/mcp_client_utils.py

@@ -11,9 +11,7 @@
 from .process_types import (
     KillPersistprocResult,
     ListProcessesResult,
-    ProcessLogPathsResult,
     ProcessOutputResult,
-    ProcessStatusResult,
     RestartProcessResult,
     StartProcessResult,
     StopProcessResult,
@@ -69,10 +67,8 @@ def _create_result_object(tool_name: str, result_data: dict) -> object | None:
     result_type_map = {
         "start": StartProcessResult,
         "stop": StopProcessResult,
-        "status": ProcessStatusResult,
         "list": ListProcessesResult,
         "output": ProcessOutputResult,
-        "get_log_paths": ProcessLogPathsResult,
         "restart": RestartProcessResult,
         "kill_persistproc": KillPersistprocResult,
     }