docs: address PR 119 review — Google-style sections and doc accuracy

Author: clean6378-max-it

api/composers.py

@@ -126,6 +126,8 @@ def list_composers() -> tuple[Response, int] | Response:
     except Exception:
         _logger.exception("Failed to get composers")
         return json_response({"error": "Failed to get composers"}, 500)
+
+
 @bp.route("/api/composers/<composer_id>")
 def get_composer(composer_id: str) -> tuple[Response, int] | Response:
     """Fetch one composer by ID (GET /api/composers/<composer_id>).

api/config_api.py

@@ -106,9 +106,10 @@ def validate_path() -> tuple[Response, int] | Response:
 def set_workspace() -> tuple[Response, int] | Response:
     """Persist a validated workspace storage path (POST /api/set-workspace).
 
-    Body: ``{"path": "<workspaceStorage root>"}``. Path is canonicalized via
-    :func:`utils.path_validation.validate_workspace_path` before storing the
-    thread-safe module override.
+    Args:
+        path: Workspace storage root from JSON body ``{"path": "..."}``.
+            Canonicalized via :func:`utils.path_validation.validate_workspace_path`
+            before storing the thread-safe module override.
 
     Returns:
         ``{"success": true, "path": "..."}`` on success. 400 for invalid path or

api/pdf.py

@@ -49,8 +49,9 @@ def _safe_text(text: str) -> str:
 def generate_pdf() -> tuple[Response, int] | Response:
     """Render markdown chat content as a PDF download (POST /api/generate-pdf).
 
-    Body (optional): ``{"markdown": "...", "title": "..."}``. Missing keys
-    default to ``""`` and ``"Chat"`` respectively.
+    Args:
+        markdown: Markdown source text (optional; defaults to ``""``).
+        title: Document title (optional; defaults to ``"Chat"``).
 
     Returns:
         ``application/pdf`` attachment on success. 500 JSON error on failure.

api/search.py

@@ -27,11 +27,13 @@
 def search() -> tuple[Response, int] | Response:
     """Search chats, composers, and CLI sessions across Cursor storage.
 
-    Query params: ``q`` (required), ``type`` (``all`` | ``chat`` | ``composer``).
+    Args:
+        q: Search query string (required; 400 when empty).
+        type: Filter scope — ``all`` (default), ``chat``, or ``composer``.
 
     Returns:
         JSON ``{"results": [...]}`` with optional ``warnings``. 400 when ``q`` is
-        empty; 500 on unexpected failure.
+        empty; 500 with ``{"error": ..., "results": []}`` on unexpected failure.
     """
     try:
         query = request.args.get("q", "").strip()

api/workspaces.py

@@ -24,10 +24,7 @@
     warn_workspace_json_read,
 )
 from utils.workspace_descriptor import read_json_file
-from services.workspace_resolver import (
-    infer_workspace_name_from_context,
-    lookup_workspace_display_name,
-)
+from services.workspace_resolver import infer_workspace_name_from_context
 from services.cli_tabs import get_cli_workspace_tabs
 from services.workspace_listing import list_workspace_projects
 from services.workspace_tabs import (
@@ -59,7 +56,8 @@ def _request_nocache() -> bool:
 def list_workspaces() -> tuple[Response, int] | Response:
     """List workspace projects for the sidebar (GET /api/workspaces).
 
-    Honors ``?nocache=1`` to bypass the summary disk cache.
+    Args:
+        nocache: When ``1`` or ``true``, bypass the summary disk cache.
 
     Returns:
         JSON with ``projects`` and optional ``warnings``. 500 on failure.
@@ -171,9 +169,8 @@ def get_workspace_tabs(workspace_id: str) -> tuple[Response, int] | Response:
     Args:
         workspace_id: Storage folder name, ``global`` for unassigned chats, or
             ``cli:<project_id>``.
-
-    Query params: ``summary=1`` for lightweight tab headers only; ``nocache=1`` to
-    bypass cache on summary requests.
+        summary: When ``1`` or ``true``, return lightweight tab headers only.
+        nocache: When ``1`` or ``true``, bypass cache on summary requests.
 
     Returns:
         Tabs payload from :func:`services.workspace_tabs` helpers (typically

models/cli_session.py

@@ -27,7 +27,7 @@ def from_dict(cls, raw: dict[str, Any]) -> "CliSessionMeta":
 
         Raises:
             SchemaError: When ``latestRootBlobId`` is missing, falsey (e.g.
-            empty string or ``None``), or not a string.
+                empty string or ``None``), or not a string.
         """
         raw = require_dict(raw, model="CliSessionMeta", field="meta")
         latest = require_truthy(

services/summary_cache.py

@@ -32,7 +32,8 @@ def nocache_enabled(*, request_nocache: bool = False) -> bool:
         request_nocache: True when the HTTP request included ``?nocache=1``.
 
     Returns:
-        True when bypass is requested or ``CURSOR_CHAT_BROWSER_NOCACHE`` is set.
+        True when bypass is requested or ``CURSOR_CHAT_BROWSER_NOCACHE`` is set
+        to ``"1"``, ``"true"``, or ``"yes"`` (case-insensitive).
     """
     if request_nocache:
         return True