diff --git a/corvid/main.py b/corvid/main.py index 4a76446..8d95dbb 100644 --- a/corvid/main.py +++ b/corvid/main.py @@ -34,7 +34,7 @@ async def root(): @app.get("/tickets", response_class=HTMLResponse, include_in_schema=False) async def tickets_page(request: Request): return templates.TemplateResponse( - "tickets.html", {"request": request, "active_page": "tickets"} + request, "tickets.html", {"active_page": "tickets"} ) diff --git a/corvid/router.py b/corvid/router.py index 2c8df8c..f63331f 100644 --- a/corvid/router.py +++ b/corvid/router.py @@ -159,9 +159,7 @@ def make_router(api_key: str, get_db, require_user=None) -> APIRouter: return {"username": "claude", "email": "claude@internal"} if require_user is not None: return require_user(request) - if not api_key: - return {"username": "local", "email": "local@localhost"} - raise HTTPException(401, "unauthorized") + return {"username": "local", "email": "local@localhost"} # ── Environment endpoints ───────────────────────────────────────────────── diff --git a/mcp_server.py b/mcp_server.py index c2232ad..94974c7 100644 --- a/mcp_server.py +++ b/mcp_server.py @@ -229,33 +229,30 @@ def delete_ticket(ticket_id: int) -> dict: @mcp.tool() def update_ticket( - ticket_id: int, - title: str = "", - description: str = "", - status: str = "", - type: str = "", - user: str = "", - parent_id: int = -1, - effort: int = -1, - environment_id: int = -1, + ticket_id: int, + title: str = "", + description: str = "", + status: str = "", + type: str = "", + user: str = "", + parent_id: int = -1, + effort: int = -1, ) -> dict: """ Update an existing homelab ticket. Only provided (non-empty) fields are changed; omit a field to leave it unchanged. Args: - ticket_id: The integer ID of the ticket to update. - title: New title. - description: New markdown description. - status: open, ongoing, completed, abandoned. - type: feature, bug, chore, project. - user: kevin or claude. - parent_id: Set parent ticket ID; pass 0 to clear the parent. - Omit (default -1) to leave unchanged. - effort: Effort rating 1–10; pass 0 to clear it. - Omit (default -1) to leave unchanged. - environment_id: Move ticket to a different environment by ID. - Omit (default -1) to leave unchanged. + ticket_id: The integer ID of the ticket to update. + title: New title. + description: New markdown description. + status: open, ongoing, completed, abandoned. + type: feature, bug, chore, project. + user: kevin or claude. + parent_id: Set parent ticket ID; pass 0 to clear the parent. + Omit (default -1) to leave unchanged. + effort: Effort rating 1–10; pass 0 to clear it. + Omit (default -1) to leave unchanged. """ body = {} if title: body["title"] = title @@ -267,8 +264,6 @@ def update_ticket( body["parent_id"] = parent_id if parent_id > 0 else None if effort >= 0: body["effort"] = effort if effort > 0 else None - if environment_id >= 0: - body["environment_id"] = environment_id if environment_id > 0 else None if not body: raise ValueError("Provide at least one field to update") with _client() as c: