Fix runner log path in WM by passing it from ER.

Author: Aksem

docs/wm-er-protocol.md

@@ -64,6 +64,12 @@ The protocol is LSP-shaped with a small set of custom commands.
       - `handlers_to_initialize`: map of action name → handler names (optional)
     - Result: `{}` (empty object)
 
+  - `finecodeRunner/getInfo`
+    - Arguments: none
+    - Result: `{ "logFilePath": "/abs/path/to/runner.log" | null }`
+    - Returns runtime information about the runner. Currently reports the path
+      to the runner's log file, or `null` if logging to a file is not configured.
+
   - `actions/run`
     - Arguments:
       1. `action_name` (string)

finecode_extension_runner/src/finecode_extension_runner/cli.py

@@ -63,8 +63,8 @@ def start(
         / "runner"
         / "runner.log")
 
-    logs.setup_logging(log_level="INFO" if trace is False else "TRACE", log_file_path=log_file_path)
-    
+    global_state.log_file_path = logs.setup_logging(log_level="INFO" if trace is False else "TRACE", log_file_path=log_file_path)
+
     if debug is True:
         logger.info(f"Started debugger on 127.0.0.1:{debug_port}")
 

finecode_extension_runner/src/finecode_extension_runner/global_state.py

@@ -9,3 +9,4 @@
 project_dir_path: Path | None = None
 log_level: Literal["TRACE", "INFO"] = "INFO"
 env_name: str = ""
+log_file_path: Path | None = None

finecode_extension_runner/src/finecode_extension_runner/logs.py

@@ -114,7 +114,7 @@ def reset_log_level_for_group(group: str):
         del log_level_by_group[group]
 
 
-def setup_logging(log_level: str, log_file_path: Path) -> None:
+def setup_logging(log_level: str, log_file_path: Path) -> Path:
     logger.remove()
 
     # disable logging raw messages
@@ -124,7 +124,7 @@ def setup_logging(log_level: str, log_file_path: Path) -> None:
 
     # ~~extension runner communicates with workspace manager with tcp, we can print logs
     # to stdout as well~~. See README.md
-    save_logs_to_file(
+    actual_log_file_path = save_logs_to_file(
         file_path=log_file_path,
         log_level=log_level,
         stdout=True,
@@ -162,5 +162,7 @@ def emit(self, record: logging.LogRecord) -> None:
         "finecode_extension_runner.impls.inmemory_cache", LogLevel.WARNING
     )
 
+    return actual_log_file_path
+
 
 __all__ = ["save_logs_to_file", "set_log_level_for_group", "reset_log_level_for_group", "setup_logging"]

src/finecode/wm_server/runner/_internal_client_api.py

@@ -41,6 +41,19 @@ async def notify_initialized(client: jsonrpc_client.JsonRpcClient) -> None:
     )
 
 
+async def get_runner_info(client: jsonrpc_client.JsonRpcClient) -> dict:
+    logger.debug(f"Get runner info from {client.readable_id}")
+    response = await client.send_request(
+        method=_internal_client_types.WORKSPACE_EXECUTE_COMMAND,
+        params=_internal_client_types.ExecuteCommandParams(
+            command="finecodeRunner/getInfo",
+            arguments=[],
+        ),
+        timeout=10,
+    )
+    return response.result
+
+
 async def cancel_request(
     client: jsonrpc_client.JsonRpcClient, request_id: int | str
 ) -> None:

src/finecode/wm_server/runner/runner_manager.py

@@ -415,6 +415,17 @@ async def start_runner(
         runner.initialized_event.set()
         raise exception
 
+    try:
+        runner_info = await _internal_client_api.get_runner_info(runner.client)
+        log_file_path_str = runner_info.get("logFilePath")
+        if log_file_path_str is not None:
+            runner.log_file_path = Path(log_file_path_str)
+            logger.debug(f"Runner {runner.readable_id} log file: {runner.log_file_path}")
+        else:
+            logger.debug(f"Runner {runner.readable_id} returned no log file path")
+    except Exception as e:
+        logger.warning(f"Failed to get runner info for {runner.readable_id}: {e}")
+
     if (
         project_def.dir_path not in ws_context.ws_projects_raw_configs
         or not isinstance(project_def, domain.CollectedProject)