chore: more route updates

Author: phil65

src/agentpool_server/opencode_server/ENDPOINTS.md

@@ -1,7 +1,7 @@
 # OpenCode API Compatibility Checklist
 
 This document tracks the implementation status of OpenCode-compatible API endpoints.
-Last audited against OpenCode source: **2026-02-24**
+Last audited against OpenCode source: **2026-03-24**
 
 ## Status Legend
 - [ ] Not implemented
@@ -20,6 +20,7 @@ Last audited against OpenCode source: **2026-02-24**
 | [x] | GET | `/global/config` | Get global configuration |
 | [x] | PATCH | `/global/config` | Update global configuration |
 | [x] | POST | `/global/dispose` | Dispose all instances |
+| [~] | POST | `/global/upgrade` | Upgrade opencode (stub - not applicable) |
 
 ---
 
@@ -29,6 +30,7 @@ Last audited against OpenCode source: **2026-02-24**
 |--------|--------|------|-------------|
 | [x] | GET | `/project` | List all projects |
 | [x] | GET | `/project/current` | Get the current project |
+| [x] | POST | `/project/git/init` | Initialize git repository |
 | [x] | PATCH | `/project/{projectID}` | Update project (name, icon, commands) |
 | [x] | GET | `/path` | Get the current path |
 | [x] | GET | `/vcs` | Get VCS info for current project |
@@ -174,7 +176,7 @@ Last audited against OpenCode source: **2026-02-24**
 
 ---
 
-## Worktrees (Experimental)
+## Worktrees & Workspaces (Experimental)
 
 | Status | Method | Path | Description |
 |--------|--------|------|-------------|
@@ -193,12 +195,20 @@ Last audited against OpenCode source: **2026-02-24**
 | Status | Method | Path | Description |
 |--------|--------|------|-------------|
 | [x] | GET | `/lsp` | Get LSP server status |
-| [x] | POST | `/lsp/start` | Start an LSP server |
-| [x] | POST | `/lsp/stop` | Stop an LSP server |
-| [x] | GET | `/lsp/servers` | List available LSP servers |
-| [x] | GET | `/lsp/diagnostics` | Get LSP diagnostics (CLI-based) |
 | [x] | GET | `/formatter` | Get formatter status (stub) |
 
+### AgentPool Extensions (commented out, not in upstream OpenCode)
+
+These routes were agentpool-specific extensions. OpenCode handles diagnostics
+internally via tool call results, not HTTP endpoints.
+
+| Status | Method | Path | Description |
+|--------|--------|------|-------------|
+| [-] | POST | `/lsp/start` | Start an LSP server (commented out) |
+| [-] | POST | `/lsp/stop` | Stop an LSP server (commented out) |
+| [-] | GET | `/lsp/servers` | List available LSP servers (commented out) |
+| [-] | GET | `/lsp/diagnostics` | Get LSP diagnostics (commented out) |
+
 ---
 
 ## MCP
@@ -416,3 +426,15 @@ _PARAM_NAME_MAP = {
     "line_hint": "lineHint",
 }
 ```
+
+---
+
+## Notes
+
+- **Diagnostics**: OpenCode does NOT expose diagnostics via HTTP routes. Diagnostics are
+  handled internally — LSP servers push them to in-process clients, and tools (`write`,
+  `edit`, `apply_patch`) include them in their return metadata after file operations.
+- **LSP extensions**: The `/lsp/start`, `/lsp/stop`, `/lsp/servers`, `/lsp/diagnostics`
+  routes are agentpool extensions commented out in `lsp_routes.py`.
+- **Upgrade**: The `/global/upgrade` route is stubbed since it's not applicable to agentpool.
+- **Git init**: The `/project/git/init` route is stubbed — could be implemented later.

src/agentpool_server/opencode_server/routes/agent_routes.py

@@ -23,9 +23,7 @@
     Agent,
     AuthInfo,
     Command,
-    FormatterStatus,
     LogRequest,
-    LspStatus,
     McpAuthorizationResponse,
     McpResource,
     MCPStatus,
@@ -484,33 +482,6 @@ async def list_tools_with_schemas(  # noqa: D417
         return result
 
 
-@router.get("/lsp")
-async def get_lsp_status(state: StateDep) -> list[LspStatus]:
-    """Get LSP server status.
-
-    Returns status of all running LSP servers.
-    """
-    return [
-        LspStatus(
-            id=server_id,
-            name=server_id,
-            status="connected" if server_state.initialized else "error",
-            root=server_state.root_uri or "",
-        )
-        for server_id, server_state in state.lsp_manager._servers.items()
-    ]
-
-
-@router.get("/formatter")
-async def get_formatter_status(state: StateDep) -> list[FormatterStatus]:
-    """Get formatter status.
-
-    Returns empty list - formatters not supported yet.
-    """
-    _ = state
-    return []
-
-
 @router.get("/provider/auth")
 async def get_provider_auth(state: StateDep) -> dict[str, list[ProviderAuthMethod]]:
     """Get provider authentication methods.

src/agentpool_server/opencode_server/routes/app_routes.py

@@ -82,6 +82,15 @@ async def get_project_current(state: StateDep) -> Project:
     return _project_data_to_response(project)
 
 
+@router.post("/project/git/init")
+async def init_git(state: StateDep) -> Project:
+    """Initialize git repository for current project."""
+    cwd = state.agent.env.cwd or state.working_dir
+    await state.agent.env.execute_command(f"git init {cwd}")
+    project = await _get_current_project(state)
+    return _project_data_to_response(project)
+
+
 @router.patch("/project/{project_id}")
 async def update_project(project_id: str, update: ProjectUpdateRequest, state: StateDep) -> Project:
     """Update project metadata (name, settings).

src/agentpool_server/opencode_server/routes/global_routes.py

@@ -114,6 +114,13 @@ async def global_dispose(state: StateDep) -> bool:
     return True
 
 
+@router.post("/global/upgrade")
+async def global_upgrade(state: StateDep) -> dict[str, object]:
+    """Upgrade opencode (stub - not applicable for agentpool)."""
+    _ = state
+    return {"success": False, "error": "Upgrade not supported in agentpool"}
+
+
 @router.post("/instance/dispose")
 async def instance_dispose(state: StateDep) -> bool:
     """Dispose the current instance."""