code-coordinator 0.5.530__tar.gz → 0.5.531__tar.gz

{code_coordinator-0.5.530 → code_coordinator-0.5.531}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: code-coordinator
-Version: 0.5.530
+Version: 0.5.531
 Summary: Coordinate Claude Code workers across multiple machines and repos.
 Author-email: John Donaghy <johnfdonaghy@gmail.com>
 License-Expression: LicenseRef-FSL-1.1-MIT
@@ -28,10 +28,14 @@ Requires-Dist: websockets>=12.0; extra == "server"
 Requires-Dist: psutil>=5.9; extra == "server"
 Provides-Extra: postgres
 Requires-Dist: psycopg[binary]>=3.1; extra == "postgres"
+Provides-Extra: reports-xlsx
+Requires-Dist: openpyxl>=3.1; extra == "reports-xlsx"
 Provides-Extra: all
 Requires-Dist: code-coordinator[server]; extra == "all"
+Requires-Dist: code-coordinator[reports-xlsx]; extra == "all"
 Provides-Extra: dev
 Requires-Dist: code-coordinator[server]; extra == "dev"
+Requires-Dist: code-coordinator[reports-xlsx]; extra == "dev"
 Requires-Dist: pytest>=8.0; extra == "dev"
 Requires-Dist: pytest-asyncio>=0.23; extra == "dev"
 Requires-Dist: pytest-xdist>=3.5; extra == "dev"

{code_coordinator-0.5.530 → code_coordinator-0.5.531}/code_coordinator.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: code-coordinator
-Version: 0.5.530
+Version: 0.5.531
 Summary: Coordinate Claude Code workers across multiple machines and repos.
 Author-email: John Donaghy <johnfdonaghy@gmail.com>
 License-Expression: LicenseRef-FSL-1.1-MIT
@@ -28,10 +28,14 @@ Requires-Dist: websockets>=12.0; extra == "server"
 Requires-Dist: psutil>=5.9; extra == "server"
 Provides-Extra: postgres
 Requires-Dist: psycopg[binary]>=3.1; extra == "postgres"
+Provides-Extra: reports-xlsx
+Requires-Dist: openpyxl>=3.1; extra == "reports-xlsx"
 Provides-Extra: all
 Requires-Dist: code-coordinator[server]; extra == "all"
+Requires-Dist: code-coordinator[reports-xlsx]; extra == "all"
 Provides-Extra: dev
 Requires-Dist: code-coordinator[server]; extra == "dev"
+Requires-Dist: code-coordinator[reports-xlsx]; extra == "dev"
 Requires-Dist: pytest>=8.0; extra == "dev"
 Requires-Dist: pytest-asyncio>=0.23; extra == "dev"
 Requires-Dist: pytest-xdist>=3.5; extra == "dev"

{code_coordinator-0.5.530 → code_coordinator-0.5.531}/code_coordinator.egg-info/requires.txt

@@ -8,9 +8,11 @@ tzdata
 
 [all]
 code-coordinator[server]
+code-coordinator[reports-xlsx]
 
 [dev]
 code-coordinator[server]
+code-coordinator[reports-xlsx]
 pytest>=8.0
 pytest-asyncio>=0.23
 pytest-xdist>=3.5
@@ -21,6 +23,9 @@ wheel
 [postgres]
 psycopg[binary]>=3.1
 
+[reports-xlsx]
+openpyxl>=3.1
+
 [server]
 starlette>=0.37
 uvicorn>=0.30

{code_coordinator-0.5.530 → code_coordinator-0.5.531}/code_coordinator.egg-info/scm_version.json

@@ -1,7 +1,7 @@
 {
-  "tag": "0.5.530",
+  "tag": "0.5.531",
   "distance": 0,
-  "node": "g8cda204e6985ec5ea0eca258916e347e9f330342",
+  "node": "g25bb93662530723ad3a9f4d421f8aaa555f875ad",
   "dirty": false,
   "branch": "HEAD",
   "node_date": "2026-09-28"

{code_coordinator-0.5.530 → code_coordinator-0.5.531}/coord/commands/report.py

@@ -12,18 +12,24 @@ one report as a plain table plus its notes block; ``--format json`` emits
 the ``ReportResult`` verbatim — the same bytes the daemon's
 ``GET /report/{id}`` returns for the same window.
 
-``--format csv`` (#1765) writes the machine-readable form to stdout, so it
-pipes and redirects normally.  It calls :func:`coord.reports.result_to_csv`
-on the **raw wire result** — the same function, on the same input, that
-``GET /report/{id}?format=csv`` calls — rather than re-serialising the human
-table above.  That is deliberate and load-bearing twice over: it is why the
-CLI and the daemon emit identical bytes, and it is why ``started_at``
-exports as an epoch instead of the ``13h ago`` that ``_format_cell``
-renders.  ``--json`` survives as a hidden alias for ``--format json``.
+``--format csv|ndjson|md|xlsx`` (#1765, extended by #3472) write a
+machine/human-readable export to stdout, so it pipes and redirects normally.
+Each one dispatches through :data:`coord.reports.EXPORT_FORMATS` on the
+**raw wire result** — the same table, on the same input, that
+``GET /report/{id}?format=<fmt>`` dispatches through — rather than
+re-serialising the human table above.  That is deliberate and load-bearing
+twice over: it is why the CLI and the daemon emit identical bytes, and it is
+why ``started_at`` exports as an epoch (or a real Excel datetime cell for
+``xlsx``) instead of the ``13h ago`` that ``_format_cell`` renders.
+``--format xlsx`` needs the optional ``reports-xlsx`` extra
+(``openpyxl``); without it, this exits ``2`` with a message naming the
+extra rather than a crash.  ``--json`` survives as a hidden alias for
+``--format json``.
 
 Exit codes: ``2`` for a bad request (unknown report, unknown parameter, bad
-value — the message names what was allowed), ``1`` for a read/transport
-failure.  Never a traceback for either.
+value, or ``--format xlsx`` without the extra installed — the message names
+what was allowed/needed), ``1`` for a read/transport failure.  Never a
+traceback for either.
 """
 
 from __future__ import annotations
@@ -265,10 +271,14 @@ def report_list(
 )
 @click.option(
     "--format", "output_format",
-    type=click.Choice(["table", "json", "csv"]),
+    type=click.Choice(["table", "json", "csv", "ndjson", "md", "xlsx"]),
     default="table",
     show_default=True,
-    help="Output encoding: human table, raw ReportResult JSON, or CSV.",
+    help=(
+        "Output encoding: human table, raw ReportResult JSON, CSV, "
+        "newline-delimited JSON, a markdown table, or an .xlsx workbook "
+        "(#3472; needs the 'reports-xlsx' extra)."
+    ),
 )
 @click.option(
     "--json", "legacy_json", is_flag=True, default=False, hidden=True,
@@ -318,18 +328,29 @@ def report_run(
         click.echo(_json.dumps(result, indent=2, default=str))
         return
 
-    if output_format == "csv":
-        # #1765: the *server's* serializer, applied to the raw wire result —
-        # never a re-serialisation of the human table. That is what makes
-        # these bytes identical to `GET /report/{id}?format=csv`, and what
-        # keeps `started_at` an epoch instead of the `13h ago` this module
-        # renders two functions up.
-        from coord.reports import result_to_csv  # noqa: PLC0415
-
-        # nl=False: the serializer already terminates every line, and click
-        # would otherwise append a stray blank line that the daemon's bytes
-        # do not have.
-        click.echo(result_to_csv(result), nl=False)
+    if output_format in ("csv", "ndjson", "md", "xlsx"):
+        # #1765 / #3472: the *server's* serializer, applied to the raw wire
+        # result — never a re-serialisation of the human table. That is what
+        # makes these bytes identical to `GET /report/{id}?format=<fmt>`, and
+        # what keeps `started_at` an epoch instead of the `13h ago` this
+        # module renders two functions up. `EXPORT_FORMATS` is the single
+        # table every surface (CLI, daemon, dashboard) dispatches through —
+        # see `coord.reports.EXPORT_FORMATS`.
+        from coord.reports import EXPORT_FORMATS, XlsxExtraMissingError  # noqa: PLC0415
+
+        export = EXPORT_FORMATS[output_format]
+        try:
+            body = export.serialize(result)
+        except XlsxExtraMissingError as e:
+            click.echo(f"error: {e}", err=True)
+            raise SystemExit(2) from e
+        # nl=False: the serializer already terminates every line (text
+        # formats) or is a self-contained binary blob (xlsx), and click
+        # would otherwise append a stray blank line/byte that the daemon's
+        # bytes do not have. click.echo writes bytes straight to the binary
+        # stream when given bytes, so xlsx round-trips undamaged even on a
+        # platform whose stdout does newline translation.
+        click.echo(body, nl=False)
         return
 
     window = result.get("window") or [None, None]

{code_coordinator-0.5.530 → code_coordinator-0.5.531}/coord/dashboard/server.py

@@ -1740,12 +1740,20 @@ def openapi_spec() -> dict:
                         "name": "format",
                         "in": "query",
                         "required": False,
-                        "schema": {"type": "string", "enum": ["json", "csv"]},
+                        "schema": {
+                            "type": "string",
+                            "enum": ["json", "csv", "ndjson", "md", "xlsx"],
+                        },
                         "description": (
-                            "#1765: response encoding. Absent/`json` returns "
-                            "the ReportResult unchanged; `csv` returns "
-                            "text/csv (raw values, `#`-prefixed notes) with "
-                            "a Content-Disposition filename."
+                            "#1765/#3472: response encoding. Absent/`json` "
+                            "returns the ReportResult unchanged; `csv` "
+                            "returns text/csv (raw values, `#`-prefixed "
+                            "notes); `ndjson` one JSON object per line; "
+                            "`md` a markdown table plus notes; `xlsx` a "
+                            "typed-cell workbook (needs the `reports-xlsx` "
+                            "extra server-side — a 400 names it if "
+                            "missing). Every non-JSON format ships a "
+                            "Content-Disposition filename."
                         ),
                     },
                 ],
@@ -1763,10 +1771,40 @@ def openapi_spec() -> dict:
                                     "lines."
                                 ),
                             },
+                            "application/x-ndjson": {
+                                "schema": {"type": "string"},
+                                "description": (
+                                    "`?format=ndjson`. One JSON object per "
+                                    "line: a leading `_meta` line, one "
+                                    "`_note` line per note, one line per "
+                                    "data row, a trailing `_totals` line "
+                                    "when the report has one."
+                                ),
+                            },
+                            "text/markdown": {
+                                "schema": {"type": "string"},
+                                "description": (
+                                    "`?format=md`. A pipe table labelled "
+                                    "from `column_meta` plus a trailing "
+                                    "`## Notes` list."
+                                ),
+                            },
+                            "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet": {
+                                "schema": {"type": "string", "format": "binary"},
+                                "description": (
+                                    "`?format=xlsx`. A `Report` sheet (typed "
+                                    "cells, plus a totals row) and a `Notes` "
+                                    "sheet."
+                                ),
+                            },
                         },
                     },
                     "400": {
-                        "description": "Unknown parameter / bad parameter value / unknown format"
+                        "description": (
+                            "Unknown parameter / bad parameter value / "
+                            "unknown format / format=xlsx without the "
+                            "reports-xlsx extra installed"
+                        )
                     },
                     "404": {"description": "Unknown report id"},
                 },
@@ -3611,13 +3649,16 @@ def build_app(
 
         report_id = request.path_params["report_id"]
         params = dict(request.query_params)
-        # #1765: `format` is a *rendering* choice, not a report parameter —
-        # pop it before validation or `resolve_params` rejects it as an
-        # unknown parameter.
+        # #1765 / #3472: `format` is a *rendering* choice, not a report
+        # parameter — pop it before validation or `resolve_params` rejects
+        # it as an unknown parameter. `EXPORT_FORMATS` is the single table
+        # every surface (CLI, `coord/serve_app.py`, this route) dispatches
+        # through, so "what formats exist" is answered exactly once.
         fmt = (params.pop("format", "") or "json").strip().lower()
-        if fmt not in ("json", "csv"):
+        if fmt != "json" and fmt not in _reports.EXPORT_FORMATS:
+            allowed = ", ".join(["json", *sorted(_reports.EXPORT_FORMATS)])
             return JSONResponse(
-                {"error": f"unknown format {fmt!r} — allowed values: json, csv"},
+                {"error": f"unknown format {fmt!r} — allowed values: {allowed}"},
                 status_code=400,
             )
         try:
@@ -3633,15 +3674,23 @@ def build_app(
             return JSONResponse(
                 {"error": "report run failed", "detail": str(e)}, status_code=503
             )
-        if fmt == "csv":
+        if fmt != "json":
             # Same serializer the CLI calls, so `coord report run --format
-            # csv` and this route emit identical bytes for identical params.
+            # <fmt>` and this route emit identical bytes for identical
+            # params. `format=xlsx` without the optional `reports-xlsx`
+            # extra installed raises XlsxExtraMissingError (a ReportError)
+            # rather than crashing — surfaced as a 400 naming the extra.
+            export = _reports.EXPORT_FORMATS[fmt]
+            try:
+                body = await run_in_threadpool(export.serialize, result)
+            except _reports.XlsxExtraMissingError as e:
+                return JSONResponse({"error": str(e)}, status_code=400)
             return Response(
-                _reports.result_to_csv(result),
-                media_type="text/csv; charset=utf-8",
+                body,
+                media_type=export.media_type,
                 headers={
                     "Content-Disposition": (
-                        f'attachment; filename="{_reports.csv_filename(result)}"'
+                        f'attachment; filename="{export.filename(result)}"'
                     )
                 },
             )