lxw-cli 0.1.0__tar.gz → 0.1.2__tar.gz

{lxw_cli-0.1.0 → lxw_cli-0.1.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: lxw-cli
-Version: 0.1.0
+Version: 0.1.2
 Summary: Inoffizielles CLI/TUI/MCP-Tool für die Lexware Office API (vormals lexoffice)
 Project-URL: Homepage, https://github.com/OEM-Mrks/lxw-cli
 Project-URL: Documentation, https://developers.lexware.io/
@@ -71,6 +71,22 @@ Alternativ klassisch in ein Environment:
 pip install lxw-cli
 ```
 
+### Windows (cmd, ohne Vorinstallation)
+
+Für ein frisches Windows 11 ohne uv und ohne Python — die vier Zeilen laufen
+per Copy-Paste in `cmd.exe` durch (volle Pfade, damit kein Terminal-Neustart
+nötig ist; uv lädt sich ein passendes Python selbst):
+
+```cmd
+powershell -ExecutionPolicy ByPass -c "irm https://astral.sh/uv/install.ps1 | iex"
+"%USERPROFILE%\.local\bin\uv.exe" tool install lxw-cli
+"%USERPROFILE%\.local\bin\uv.exe" tool update-shell
+"%USERPROFILE%\.local\bin\lxw.exe" profile
+```
+
+Ab dem nächsten cmd-Fenster reicht dann überall einfach `lxw`. Der API-Key
+wird beim ersten Start abgefragt und unter `%APPDATA%\lexware\.env` gespeichert.
+
 ### Aus dem Quellcode (Entwicklung)
 
 ```bash
@@ -333,18 +349,22 @@ Technisch spricht die TUI ausschließlich den UI-freien Core (`lxw_cli.core`)
 
 ## Claude-Integration (MCP-Server)
 
-Das CLI bringt einen eingebauten MCP-Server mit, mit dem **Claude Code** (und Claude Desktop) deine Lexware-Daten direkt abfragen kann.
+Das CLI bringt einen eingebauten MCP-Server mit, mit dem **Claude Code**, **Claude Desktop** und **Cowork** deine Lexware-Daten direkt abfragen können.
 
 ### Setup in einem Schritt
 
 ```bash
-lxw mcp install-claude
+lxw mcp install-claude    # Claude Code
+lxw mcp install-desktop   # Claude Desktop + Cowork
 ```
 
-Das registriert den Server bei Claude Code (Scope `user`, also für alle Projekte). Der API-Key wird dabei **nicht** an Claude übergeben — der Server liest ihn selbst aus `~/.config/lexware/.env` (chmod 600); falls er dort noch fehlt, legt `install-claude` ihn einmalig ab. Danach:
+Die beiden Frontends haben getrennte Konfigurationen: `install-claude` registriert den Server bei **Claude Code** (via `claude mcp add`, Scope `user`), `install-desktop` trägt ihn in die `claude_desktop_config.json` von **Claude Desktop** ein — das deckt auch **Cowork** ab. **Wichtig:** `install-desktop` bei beendetem Claude Desktop ausführen (Quit, nicht nur Fenster schließen) — die laufende App schreibt ihre Konfiguration aus dem Arbeitsspeicher zurück und überschreibt externe Änderungen. Danach Desktop starten.
+
+Der API-Key wird dabei **nicht** an Claude übergeben — der Server liest ihn selbst aus `~/.config/lexware/.env` (chmod 600; Windows: `%APPDATA%\lexware\.env`); falls er dort noch fehlt, legen die Install-Befehle ihn einmalig ab. Prüfen:
 
 ```bash
-claude mcp list   # 'lexware' muss in der Liste erscheinen
+lxw mcp status    # zeigt Registrierung für Claude Code und Desktop
+claude mcp list   # 'lexware' muss in der Liste erscheinen (Claude Code)
 ```
 
 Öffne Claude Code in einem Projekt und frag z.B.:
@@ -375,10 +395,12 @@ Die `list_*`-Tools bieten dieselben Optionen wie die CLI:
 ### Weitere mcp-Befehle
 
 ```bash
-lxw mcp status              # zeigt aktuelle Registrierung
-lxw mcp install-claude --force   # Re-Install (z.B. nach API-Key-Wechsel)
-lxw mcp uninstall-claude    # entfernen
-lxw mcp serve               # läuft direkt — wird von Claude Code intern aufgerufen
+lxw mcp status                    # zeigt Registrierung (Claude Code + Desktop)
+lxw mcp install-claude --force    # Re-Install Claude Code (z.B. nach Pfad-Wechsel)
+lxw mcp install-desktop --force   # Re-Install Claude Desktop/Cowork
+lxw mcp uninstall-claude          # aus Claude Code entfernen
+lxw mcp uninstall-desktop         # aus Claude Desktop entfernen
+lxw mcp serve                     # läuft direkt — wird von Claude intern aufgerufen
 ```
 
 ## Out of Scope (v1)

{lxw_cli-0.1.0 → lxw_cli-0.1.2}/README.md

@@ -36,6 +36,22 @@ Alternativ klassisch in ein Environment:
 pip install lxw-cli
 ```
 
+### Windows (cmd, ohne Vorinstallation)
+
+Für ein frisches Windows 11 ohne uv und ohne Python — die vier Zeilen laufen
+per Copy-Paste in `cmd.exe` durch (volle Pfade, damit kein Terminal-Neustart
+nötig ist; uv lädt sich ein passendes Python selbst):
+
+```cmd
+powershell -ExecutionPolicy ByPass -c "irm https://astral.sh/uv/install.ps1 | iex"
+"%USERPROFILE%\.local\bin\uv.exe" tool install lxw-cli
+"%USERPROFILE%\.local\bin\uv.exe" tool update-shell
+"%USERPROFILE%\.local\bin\lxw.exe" profile
+```
+
+Ab dem nächsten cmd-Fenster reicht dann überall einfach `lxw`. Der API-Key
+wird beim ersten Start abgefragt und unter `%APPDATA%\lexware\.env` gespeichert.
+
 ### Aus dem Quellcode (Entwicklung)
 
 ```bash
@@ -298,18 +314,22 @@ Technisch spricht die TUI ausschließlich den UI-freien Core (`lxw_cli.core`)
 
 ## Claude-Integration (MCP-Server)
 
-Das CLI bringt einen eingebauten MCP-Server mit, mit dem **Claude Code** (und Claude Desktop) deine Lexware-Daten direkt abfragen kann.
+Das CLI bringt einen eingebauten MCP-Server mit, mit dem **Claude Code**, **Claude Desktop** und **Cowork** deine Lexware-Daten direkt abfragen können.
 
 ### Setup in einem Schritt
 
 ```bash
-lxw mcp install-claude
+lxw mcp install-claude    # Claude Code
+lxw mcp install-desktop   # Claude Desktop + Cowork
 ```
 
-Das registriert den Server bei Claude Code (Scope `user`, also für alle Projekte). Der API-Key wird dabei **nicht** an Claude übergeben — der Server liest ihn selbst aus `~/.config/lexware/.env` (chmod 600); falls er dort noch fehlt, legt `install-claude` ihn einmalig ab. Danach:
+Die beiden Frontends haben getrennte Konfigurationen: `install-claude` registriert den Server bei **Claude Code** (via `claude mcp add`, Scope `user`), `install-desktop` trägt ihn in die `claude_desktop_config.json` von **Claude Desktop** ein — das deckt auch **Cowork** ab. **Wichtig:** `install-desktop` bei beendetem Claude Desktop ausführen (Quit, nicht nur Fenster schließen) — die laufende App schreibt ihre Konfiguration aus dem Arbeitsspeicher zurück und überschreibt externe Änderungen. Danach Desktop starten.
+
+Der API-Key wird dabei **nicht** an Claude übergeben — der Server liest ihn selbst aus `~/.config/lexware/.env` (chmod 600; Windows: `%APPDATA%\lexware\.env`); falls er dort noch fehlt, legen die Install-Befehle ihn einmalig ab. Prüfen:
 
 ```bash
-claude mcp list   # 'lexware' muss in der Liste erscheinen
+lxw mcp status    # zeigt Registrierung für Claude Code und Desktop
+claude mcp list   # 'lexware' muss in der Liste erscheinen (Claude Code)
 ```
 
 Öffne Claude Code in einem Projekt und frag z.B.:
@@ -340,10 +360,12 @@ Die `list_*`-Tools bieten dieselben Optionen wie die CLI:
 ### Weitere mcp-Befehle
 
 ```bash
-lxw mcp status              # zeigt aktuelle Registrierung
-lxw mcp install-claude --force   # Re-Install (z.B. nach API-Key-Wechsel)
-lxw mcp uninstall-claude    # entfernen
-lxw mcp serve               # läuft direkt — wird von Claude Code intern aufgerufen
+lxw mcp status                    # zeigt Registrierung (Claude Code + Desktop)
+lxw mcp install-claude --force    # Re-Install Claude Code (z.B. nach Pfad-Wechsel)
+lxw mcp install-desktop --force   # Re-Install Claude Desktop/Cowork
+lxw mcp uninstall-claude          # aus Claude Code entfernen
+lxw mcp uninstall-desktop         # aus Claude Desktop entfernen
+lxw mcp serve                     # läuft direkt — wird von Claude intern aufgerufen
 ```
 
 ## Out of Scope (v1)

{lxw_cli-0.1.0 → lxw_cli-0.1.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "lxw-cli"
-version = "0.1.0"
+version = "0.1.2"
 description = "Inoffizielles CLI/TUI/MCP-Tool für die Lexware Office API (vormals lexoffice)"
 readme = "README.md"
 requires-python = ">=3.11"

lxw_cli-0.1.2/src/lxw_cli/__init__.py

@@ -0,0 +1,8 @@
+"""lxw-cli — die Version kommt aus den Paket-Metadaten (pyproject.toml)."""
+
+from importlib.metadata import PackageNotFoundError, version
+
+try:
+    __version__ = version("lxw-cli")
+except PackageNotFoundError:  # uninstalliert, z.B. direkt aus dem Quellbaum
+    __version__ = "0.0.0.dev0"

lxw_cli-0.1.2/src/lxw_cli/commands/mcp.py

@@ -0,0 +1,293 @@
+"""Subcommand `lxw mcp …` — manages Claude Code/Desktop integration."""
+
+from __future__ import annotations
+
+import json
+import os
+import shutil
+import subprocess
+import sys
+from pathlib import Path
+from typing import Any
+
+import typer
+from dotenv import dotenv_values
+
+from lxw_cli.config import ENV_KEY, global_env_path, load_config, store_key
+from lxw_cli.output import console, err_console
+
+app = typer.Typer(no_args_is_help=True)
+
+MCP_NAME = "lexware"
+
+
+def desktop_config_path() -> Path:
+    """Location of Claude Desktop's `claude_desktop_config.json`.
+
+    Claude Desktop (and Cowork) read MCP servers from this file — registering
+    via `claude mcp add` only covers Claude Code. `CLAUDE_DESKTOP_CONFIG`
+    overrides the path (mostly for tests).
+    """
+    override = os.environ.get("CLAUDE_DESKTOP_CONFIG")
+    if override:
+        return Path(override).expanduser()
+    if sys.platform == "darwin":
+        return Path.home() / "Library" / "Application Support" / "Claude" / (
+            "claude_desktop_config.json"
+        )
+    if sys.platform == "win32":
+        appdata = os.environ.get("APPDATA")
+        root = Path(appdata) if appdata else Path.home() / "AppData" / "Roaming"
+        return root / "Claude" / "claude_desktop_config.json"
+    return Path.home() / ".config" / "Claude" / "claude_desktop_config.json"
+
+
+def _load_desktop_config(path: Path) -> dict[str, Any]:
+    if not path.is_file():
+        return {}
+    try:
+        data = json.loads(path.read_text(encoding="utf-8"))
+    except json.JSONDecodeError as exc:
+        err_console.print(
+            f"[red]Fehler:[/red] {path} enthält kein gültiges JSON ({exc}). "
+            "Bitte die Datei prüfen — sie wird nicht angefasst."
+        )
+        raise typer.Exit(code=2) from exc
+    if not isinstance(data, dict):
+        err_console.print(f"[red]Fehler:[/red] {path} ist kein JSON-Objekt.")
+        raise typer.Exit(code=2)
+    return data
+
+
+def _check_claude() -> None:
+    if not shutil.which("claude"):
+        err_console.print(
+            "[red]Fehler:[/red] `claude` CLI nicht gefunden. "
+            "Installiere zuerst Claude Code (https://claude.com/claude-code)."
+        )
+        raise typer.Exit(code=2)
+
+
+def _mcp_command() -> list[str]:
+    """Build the command Claude will use to launch the MCP server.
+
+    Prefer the `lxw-mcp` console script when it's on PATH (works for
+    pipx-installed setups). Otherwise fall back to the current interpreter
+    plus `-m lxw_cli.mcp_server` so the venv path is captured even when
+    `.venv/bin` isn't on the system PATH.
+    """
+    on_path = shutil.which("lxw-mcp")
+    if on_path:
+        return [on_path]
+    # Look for the entry-point next to the running python (typical venv layout;
+    # on Windows console scripts get an .exe suffix)
+    exe_suffix = ".exe" if sys.platform == "win32" else ""
+    candidate = Path(sys.executable).parent / f"lxw-mcp{exe_suffix}"
+    if candidate.exists():
+        return [str(candidate)]
+    return [sys.executable, "-m", "lxw_cli.mcp_server"]
+
+
+def _ensure_global_key(api_key: str) -> None:
+    """Make sure the MCP server can resolve the key from the global config.
+
+    The key is deliberately NOT passed via `claude mcp add -e …` — that would
+    expose it in the process list and store a second plaintext copy in the
+    Claude config. The server resolves it itself from the global `.env`
+    (chmod 600), so we only have to guarantee it exists there.
+    """
+    path = global_env_path()
+    existing = (dotenv_values(path).get(ENV_KEY) or "").strip() if path.is_file() else ""
+    if not existing:
+        stored = store_key(api_key)
+        err_console.print(
+            f"[dim]API-Key in {stored} hinterlegt (wird vom MCP-Server gelesen).[/dim]"
+        )
+    elif existing != api_key:
+        err_console.print(
+            f"[yellow]Hinweis:[/yellow] {path} enthält einen anderen API-Key — "
+            "der MCP-Server verwendet den dort hinterlegten."
+        )
+
+
+@app.command("install-claude")
+def install_claude(
+    scope: str = typer.Option(
+        "user", "--scope", help="user (global), project (per repo), oder local."
+    ),
+    force: bool = typer.Option(
+        False, "--force", help="Vorher entfernen, falls bereits registriert."
+    ),
+) -> None:
+    """Registriert den Lexware-MCP-Server bei Claude Code."""
+    _check_claude()
+    config = load_config()  # validates LEXWARE_API_KEY
+    _ensure_global_key(config.api_key)
+
+    server_cmd = _mcp_command()
+
+    if force:
+        subprocess.run(
+            ["claude", "mcp", "remove", MCP_NAME, "--scope", scope],
+            check=False,
+            capture_output=True,
+        )
+
+    cmd = [
+        "claude",
+        "mcp",
+        "add",
+        MCP_NAME,
+        "--scope",
+        scope,
+        "--",
+        *server_cmd,
+    ]
+    result = subprocess.run(cmd, capture_output=True, text=True)
+    if result.returncode != 0:
+        err_console.print(f"[red]claude mcp add fehlgeschlagen:[/red]\n{result.stderr}")
+        if "already exists" in (result.stderr or "").lower():
+            err_console.print(
+                "[yellow]Hinweis:[/yellow] Mit `--force` kannst du die "
+                "bestehende Registrierung überschreiben."
+            )
+        raise typer.Exit(code=result.returncode)
+    console.print(
+        f"[green]✓[/green] Lexware MCP-Server bei Claude Code registriert "
+        f"(scope={scope})."
+    )
+    console.print(
+        "Test: Öffne Claude Code und frag z.B. "
+        "[bold]\"Wie viele offene Rechnungen habe ich?\"[/bold]"
+    )
+
+
+@app.command("uninstall-claude")
+def uninstall_claude(
+    scope: str = typer.Option("user", "--scope", help="Gleicher Scope wie beim Install."),
+) -> None:
+    """Entfernt den Lexware-MCP-Server aus Claude Code."""
+    _check_claude()
+    result = subprocess.run(
+        ["claude", "mcp", "remove", MCP_NAME, "--scope", scope],
+        capture_output=True,
+        text=True,
+    )
+    if result.returncode != 0:
+        err_console.print(f"[red]Fehler:[/red] {result.stderr}")
+        raise typer.Exit(code=result.returncode)
+    console.print(f"[green]✓[/green] Lexware MCP-Server entfernt (scope={scope}).")
+
+
+@app.command("install-desktop")
+def install_desktop(
+    force: bool = typer.Option(
+        False, "--force", help="Bestehenden 'lexware'-Eintrag überschreiben."
+    ),
+) -> None:
+    """Registriert den Lexware-MCP-Server bei Claude Desktop (und damit Cowork).
+
+    Claude Desktop liest MCP-Server aus `claude_desktop_config.json` —
+    `install-claude` deckt nur Claude Code ab. Wichtig: Claude Desktop vorher
+    komplett beenden — die laufende App schreibt die Datei aus dem
+    Arbeitsspeicher zurück und überschreibt externe Änderungen.
+    """
+    config = load_config()  # validates LEXWARE_API_KEY
+    _ensure_global_key(config.api_key)
+
+    path = desktop_config_path()
+    if not path.parent.is_dir():
+        err_console.print(
+            f"[red]Fehler:[/red] {path.parent} existiert nicht — ist Claude "
+            "Desktop installiert? (https://claude.com/download)"
+        )
+        raise typer.Exit(code=2)
+
+    data = _load_desktop_config(path)
+    servers = data.setdefault("mcpServers", {})
+    if MCP_NAME in servers and not force:
+        err_console.print(
+            f"[yellow]Hinweis:[/yellow] '{MCP_NAME}' ist bereits in {path} "
+            "eingetragen. Mit `--force` überschreiben."
+        )
+        raise typer.Exit(code=1)
+
+    # Absolute path: GUI apps don't inherit the shell PATH, so a bare
+    # 'lxw-mcp' would not resolve when Desktop spawns the server.
+    command, *args = _mcp_command()
+    entry: dict[str, Any] = {"command": command}
+    if args:
+        entry["args"] = args
+    servers[MCP_NAME] = entry
+
+    path.write_text(
+        json.dumps(data, indent=2, ensure_ascii=False) + "\n", encoding="utf-8"
+    )
+    console.print(
+        f"[green]✓[/green] Lexware MCP-Server in {path} eingetragen."
+    )
+    console.print(
+        "[bold]Wichtig:[/bold] Läuft Claude Desktop gerade, kann es diese "
+        "Änderung beim nächsten Speichern überschreiben — den Befehl am besten "
+        "bei beendeter App ausführen (Quit, nicht nur Fenster schließen) und "
+        "Desktop danach neu starten. Gilt auch für Cowork-Sessions."
+    )
+
+
+@app.command("uninstall-desktop")
+def uninstall_desktop() -> None:
+    """Entfernt den Lexware-MCP-Server aus Claude Desktop."""
+    path = desktop_config_path()
+    data = _load_desktop_config(path)
+    servers = data.get("mcpServers") or {}
+    if MCP_NAME not in servers:
+        console.print(f"[yellow]✗[/yellow] '{MCP_NAME}' ist in {path} nicht eingetragen.")
+        raise typer.Exit(code=1)
+    del servers[MCP_NAME]
+    path.write_text(
+        json.dumps(data, indent=2, ensure_ascii=False) + "\n", encoding="utf-8"
+    )
+    console.print(f"[green]✓[/green] Lexware MCP-Server aus {path} entfernt.")
+
+
+@app.command("status")
+def status() -> None:
+    """Zeigt, wo der Lexware-MCP-Server registriert ist (Claude Code + Desktop)."""
+    # Claude Code (via `claude mcp list`)
+    if shutil.which("claude"):
+        result = subprocess.run(["claude", "mcp", "list"], capture_output=True, text=True)
+        if result.returncode == 0 and MCP_NAME in result.stdout:
+            for line in result.stdout.splitlines():
+                if MCP_NAME in line:
+                    console.print(f"[green]✓[/green] Claude Code: {line.strip()}")
+        else:
+            console.print(
+                "[yellow]✗[/yellow] Claude Code: nicht registriert. "
+                "Setup: [bold]lxw mcp install-claude[/bold]"
+            )
+    else:
+        console.print("[dim]– Claude Code: `claude` CLI nicht gefunden.[/dim]")
+
+    # Claude Desktop (claude_desktop_config.json)
+    path = desktop_config_path()
+    if not path.is_file():
+        console.print(f"[dim]– Claude Desktop: keine Konfiguration ({path}).[/dim]")
+        return
+    servers = _load_desktop_config(path).get("mcpServers") or {}
+    if MCP_NAME in servers:
+        entry = servers[MCP_NAME]
+        cmd = " ".join([entry.get("command", "?"), *entry.get("args", [])])
+        console.print(f"[green]✓[/green] Claude Desktop: {cmd}")
+    else:
+        console.print(
+            "[yellow]✗[/yellow] Claude Desktop: nicht registriert. "
+            "Setup: [bold]lxw mcp install-desktop[/bold]"
+        )
+
+
+@app.command("serve")
+def serve() -> None:
+    """Startet den MCP-Server (intern von Claude Code aufgerufen — selten manuell nötig)."""
+    from lxw_cli.mcp_server import run as run_server
+
+    run_server()

{lxw_cli-0.1.0 → lxw_cli-0.1.2}/src/lxw_cli/config.py

@@ -116,7 +116,9 @@ def _apply_project_env(path: Path) -> None:
             file=sys.stderr,
         )
     for key, value in values.items():
-        if value is not None and key not in os.environ:
+        # Skip empty values: a bare `LEXWARE_API_KEY=` line (e.g. a filled-in
+        # copy of .env.example left blank) must not mask the global config.
+        if value and key not in os.environ:
             os.environ[key] = value
 
 

{lxw_cli-0.1.0 → lxw_cli-0.1.2}/src/lxw_cli/core/services.py

@@ -403,15 +403,15 @@ def article_line_item(
 ) -> dict[str, Any]:
     """Build a `custom` line item from a raw article record.
 
-    The Public API has no article-referencing line-item type, so name, unit and
-    price are copied from the article. Prices are always sent net (the article
-    record carries `netPrice` regardless of its leading price), which pairs
-    with `taxConditions.taxType = "net"` in the document body. `net_price`
-    overrides the article's list price (e.g. a manually edited price in the
-    TUI form).
+    The Public API has no article-referencing line-item type, so name,
+    description, unit and price are copied from the article. Prices are always
+    sent net (the article record carries `netPrice` regardless of its leading
+    price), which pairs with `taxConditions.taxType = "net"` in the document
+    body. `net_price` overrides the article's list price (e.g. a manually
+    edited price in the TUI form).
     """
     price = article.get("price") or {}
-    return {
+    item = {
         "type": "custom",
         "name": article.get("title") or article.get("articleNumber") or "Position",
         "quantity": quantity,
@@ -422,6 +422,10 @@ def article_line_item(
             "taxRatePercentage": price.get("taxRate", 19),
         },
     }
+    description = (article.get("description") or "").strip()
+    if description:
+        item["description"] = description
+    return item
 
 
 def build_order_confirmation_body(

{lxw_cli-0.1.0 → lxw_cli-0.1.2}/src/lxw_cli/mcp_server.py

@@ -21,7 +21,12 @@ mcp: FastMCP = FastMCP(
     instructions=(
         "Lexware Office API. Most document tools accept either a UUID or a "
         "voucher number (e.g. invoice number 'FB2600682'). PDF download tools "
-        "save the file to ~/Downloads/lexware/ by default and return the path."
+        "save the file to ~/Downloads/lexware/ by default and return the path. "
+        "When creating documents (invoice/quotation/order confirmation/delivery "
+        "note) with line items based on an article: fetch the article first and "
+        "copy its `description` into the line item's optional `description` "
+        "field — otherwise the article description is missing on the printed "
+        "document."
     ),
 )
 
@@ -314,7 +319,12 @@ def download_delivery_note_pdf(identifier: str, output_dir: str | None = None) -
 
 @mcp.tool
 def create_invoice_draft(body: dict[str, Any]) -> dict[str, Any]:
-    """Create an invoice as draft. See https://developers.lexware.io/docs/ for body schema."""
+    """Create an invoice as draft. See https://developers.lexware.io/docs/ for body schema.
+
+    Line items support an optional `description` (long text shown under the
+    position name on the PDF). When a line item is based on an article, copy
+    the article's `description` into it.
+    """
     return services.create_invoice(_client_get(), body)
 
 
@@ -342,19 +352,34 @@ def create_article(body: dict[str, Any]) -> dict[str, Any]:
 
 @mcp.tool
 def create_quotation_draft(body: dict[str, Any]) -> dict[str, Any]:
-    """Create a quotation as draft."""
+    """Create a quotation as draft.
+
+    Line items support an optional `description` (long text shown under the
+    position name on the PDF). When a line item is based on an article, copy
+    the article's `description` into it.
+    """
     return services.create_quotation(_client_get(), body)
 
 
 @mcp.tool
 def create_order_confirmation_draft(body: dict[str, Any]) -> dict[str, Any]:
-    """Create an order confirmation (Auftrag) as draft."""
+    """Create an order confirmation (Auftrag) as draft.
+
+    Line items support an optional `description` (long text shown under the
+    position name on the PDF). When a line item is based on an article, copy
+    the article's `description` into it.
+    """
     return services.create_order_confirmation(_client_get(), body)
 
 
 @mcp.tool
 def create_delivery_note_draft(body: dict[str, Any]) -> dict[str, Any]:
-    """Create a delivery note as draft."""
+    """Create a delivery note as draft.
+
+    Line items support an optional `description` (long text shown under the
+    position name on the PDF). When a line item is based on an article, copy
+    the article's `description` into it.
+    """
     return services.create_delivery_note(_client_get(), body)
 
 

{lxw_cli-0.1.0 → lxw_cli-0.1.2}/tests/test_cli.py

@@ -79,6 +79,83 @@ def test_mcp_install_claude_keeps_key_out_of_argv(
     assert "LEXWARE_API_KEY=test-key" in global_env_path().read_text(encoding="utf-8")
 
 
+@pytest.fixture
+def desktop_config(monkeypatch: pytest.MonkeyPatch, tmp_path):
+    """Point the desktop config at a tmp file pre-filled with foreign entries."""
+    import lxw_cli.commands.mcp as mcp_cmd_mod
+
+    monkeypatch.setenv("LEXWARE_CONFIG_DIR", str(tmp_path / "cfg"))
+    monkeypatch.setattr(
+        mcp_cmd_mod.shutil, "which", lambda name: f"/usr/bin/{name}"
+    )
+    path = tmp_path / "Claude" / "claude_desktop_config.json"
+    path.parent.mkdir(parents=True)
+    path.write_text(
+        json.dumps(
+            {
+                "mcpServers": {"other": {"command": "npx", "args": ["x"]}},
+                "preferences": {"keep": True},
+            }
+        ),
+        encoding="utf-8",
+    )
+    monkeypatch.setenv("CLAUDE_DESKTOP_CONFIG", str(path))
+    return path
+
+
+def test_mcp_install_desktop_writes_config(runner: CliRunner, desktop_config) -> None:
+    result = runner.invoke(app, ["mcp", "install-desktop"])
+    assert result.exit_code == 0, result.stderr
+
+    data = json.loads(desktop_config.read_text(encoding="utf-8"))
+    entry = data["mcpServers"]["lexware"]
+    # Absolute command path — Desktop doesn't inherit the shell PATH.
+    assert entry["command"] == "/usr/bin/lxw-mcp"
+    # Foreign servers and unrelated settings survive the merge untouched.
+    assert data["mcpServers"]["other"] == {"command": "npx", "args": ["x"]}
+    assert data["preferences"] == {"keep": True}
+    # The API key never lands in the desktop config …
+    assert "test-key" not in desktop_config.read_text(encoding="utf-8")
+    # … but in the global .env, where the server reads it.
+    from lxw_cli.config import global_env_path
+
+    assert "LEXWARE_API_KEY=test-key" in global_env_path().read_text(encoding="utf-8")
+
+
+def test_mcp_install_desktop_existing_needs_force(
+    runner: CliRunner, desktop_config
+) -> None:
+    assert runner.invoke(app, ["mcp", "install-desktop"]).exit_code == 0
+    # Second run without --force refuses …
+    assert runner.invoke(app, ["mcp", "install-desktop"]).exit_code == 1
+    # … with --force it overwrites.
+    result = runner.invoke(app, ["mcp", "install-desktop", "--force"])
+    assert result.exit_code == 0, result.stderr
+
+
+def test_mcp_uninstall_desktop_removes_only_lexware(
+    runner: CliRunner, desktop_config
+) -> None:
+    assert runner.invoke(app, ["mcp", "install-desktop"]).exit_code == 0
+    result = runner.invoke(app, ["mcp", "uninstall-desktop"])
+    assert result.exit_code == 0, result.stderr
+    data = json.loads(desktop_config.read_text(encoding="utf-8"))
+    assert "lexware" not in data["mcpServers"]
+    assert "other" in data["mcpServers"]
+    # Removing again is a clean no-op error, not a crash.
+    assert runner.invoke(app, ["mcp", "uninstall-desktop"]).exit_code == 1
+
+
+def test_mcp_install_desktop_leaves_broken_json_untouched(
+    runner: CliRunner, desktop_config
+) -> None:
+    desktop_config.write_text("{not json", encoding="utf-8")
+    result = runner.invoke(app, ["mcp", "install-desktop"])
+    assert result.exit_code == 2
+    # The corrupt file was not overwritten.
+    assert desktop_config.read_text(encoding="utf-8") == "{not json"
+
+
 @respx.mock
 def test_invoices_list_default_caps_at_25(runner: CliRunner) -> None:
     page0 = [{"id": f"a{i}"} for i in range(100)]