data-refinery-cli 0.5.0__tar.gz → 0.5.1__tar.gz

{data_refinery_cli-0.5.0 → data_refinery_cli-0.5.1}/.github/workflows/tests.yml

@@ -31,11 +31,23 @@ jobs:
 
       - run: uv run pytest -n auto --cov=data_refinery --cov-report=xml:coverage.xml --cov-report=term -v
 
+      # Stamp each analysis with the package version so SonarCloud's "Previous
+      # version" New Code period has a real boundary to diff against (the version
+      # is bumped every PR; see version-check below + the version-bump skill).
+      - name: Resolve project version
+        id: ver
+        run: |
+          VERSION=$(uv run python -c 'import tomllib; print(tomllib.load(open("pyproject.toml","rb"))["project"]["version"])')
+          echo "version=$VERSION" >> "$GITHUB_OUTPUT"
+
       - name: SonarCloud Scan
         if: env.SONAR_TOKEN != ''
         uses: SonarSource/sonarqube-scan-action@fd88b7d7ccbaefd23d8f36f73b59db7a3d246602 # v6
         env:
           SONAR_HOST_URL: https://sonarcloud.io
+        with:
+          args: >
+            -Dsonar.projectVersion=${{ steps.ver.outputs.version }}
 
   lint:
     runs-on: ubuntu-latest

{data_refinery_cli-0.5.0 → data_refinery_cli-0.5.1}/CHANGELOG.md

@@ -5,6 +5,16 @@ All notable changes to this project will be documented in this file.
 Format follows [Keep a Changelog](https://keepachangelog.com/). This project
 adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.5.1] - 2026-06-21
+
+### Changed
+
+- CI stamps each SonarCloud analysis with `sonar.projectVersion` read from `pyproject.toml`, so the "Previous version" New Code period has a real per-release boundary to diff against (`.github/workflows/tests.yml`)
+
+### Fixed
+
+- Cleared 4 pre-existing SonarCloud smells in `data_refinery/cli/_commands/stack.py` (wave-1), behaviour-preserving: merged the implicitly concatenated `_DOCKER_HINT` literal (S5799), extracted `_load_ps_rows` to drop `_parse_ps` cognitive complexity (S3776), single-exit `cmd_stack_status` via a `_render_status_text` helper (S3516), and an `_add_json_flag` helper for the repeated `"Emit structured JSON."` literal (S1192)
+
 ## [0.5.0] - 2026-06-20
 
 ### Added

{data_refinery_cli-0.5.0 → data_refinery_cli-0.5.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: data-refinery-cli
-Version: 0.5.0
+Version: 0.5.1
 Summary: Agent and CLI for data quality in storage and retrieval — validating, deduplicating, and checking the integrity and freshness of data as it is stored and fetched. Split out of eidetic-cli so eidetic keeps agent-memory; sibling to daria, the Data Refinery Intelligent Agent.
 Project-URL: Homepage, https://github.com/agentculture/data-refinery-cli
 Project-URL: Issues, https://github.com/agentculture/data-refinery-cli/issues

{data_refinery_cli-0.5.0 → data_refinery_cli-0.5.1}/data_refinery/cli/_commands/stack.py

@@ -27,13 +27,14 @@ import json
 import shutil
 import subprocess  # nosec B404 - used with a fixed argv, never shell=True
 from pathlib import Path
+from typing import Any
 
 from data_refinery.cli._errors import EXIT_ENV_ERROR, CliError
 from data_refinery.cli._output import emit_diagnostic, emit_result
 
 _COMPOSE_FILENAME = "docker-compose.yml"
 _DOCKER_HINT = (
-    "install Docker and ensure 'docker compose' works " "(https://docs.docker.com/get-docker/)"
+    "install Docker and ensure 'docker compose' works (https://docs.docker.com/get-docker/)"
 )
 _COMPOSE_HINT = (
     "run from the data-refinery-cli repo (it ships docker-compose.yml), or pull "
@@ -113,26 +114,37 @@ def _compose(compose: Path, *args: str) -> subprocess.CompletedProcess[str]:
     return proc
 
 
+def _load_ps_rows(text: str) -> list[Any]:
+    """Decode compose ps output: a top-level JSON array, else one object per line.
+
+    Split out of :func:`_parse_ps` so the two-mode decoding (with its nested
+    error handling) doesn't push that function's cognitive complexity over the
+    limit. *text* is assumed already stripped and non-empty.
+    """
+    try:
+        parsed = json.loads(text)
+        return parsed if isinstance(parsed, list) else [parsed]
+    except json.JSONDecodeError:
+        pass
+    rows: list[Any] = []
+    for line in text.splitlines():
+        line = line.strip()
+        if not line:
+            continue
+        try:
+            rows.append(json.loads(line))
+        except json.JSONDecodeError:
+            continue
+    return rows
+
+
 def _parse_ps(stdout: str) -> list[dict[str, object]]:
     """Parse ``docker compose ps --format json`` (array OR one-object-per-line)."""
     text = stdout.strip()
     if not text:
         return []
-    try:
-        parsed = json.loads(text)
-        rows = parsed if isinstance(parsed, list) else [parsed]
-    except json.JSONDecodeError:
-        rows = []
-        for line in text.splitlines():
-            line = line.strip()
-            if not line:
-                continue
-            try:
-                rows.append(json.loads(line))
-            except json.JSONDecodeError:
-                continue
     services: list[dict[str, object]] = []
-    for row in rows:
+    for row in _load_ps_rows(text):
         if not isinstance(row, dict):
             continue
         services.append(
@@ -198,6 +210,18 @@ def cmd_stack_down(args: argparse.Namespace) -> int:
     return 0
 
 
+def _render_status_text(payload: dict[str, object]) -> str:
+    """Render the human-readable ``stack status`` text from a status payload."""
+    services = payload["services"]
+    if not services:
+        return "stack status: no services running (try 'data-refinery stack up')"
+    lines = [f"compose: {payload['compose_file']}", f"healthy: {payload['healthy']}"]
+    for s in services:  # type: ignore[attr-defined]
+        health = f" ({s['health']})" if s["health"] else ""
+        lines.append(f"- {s['name']}: {s['state']}{health}")
+    return "\n".join(lines)
+
+
 def cmd_stack_status(args: argparse.Namespace) -> int:
     _require_docker()
     compose = _require_compose()
@@ -206,18 +230,8 @@ def cmd_stack_status(args: argparse.Namespace) -> int:
     payload["command"] = "status"
     if json_mode:
         emit_result(payload, json_mode=True)
-        return 0
-    services = payload["services"]
-    if not services:
-        emit_result(
-            "stack status: no services running (try 'data-refinery stack up')", json_mode=False
-        )
-        return 0
-    lines = [f"compose: {payload['compose_file']}", f"healthy: {payload['healthy']}"]
-    for s in services:
-        health = f" ({s['health']})" if s["health"] else ""
-        lines.append(f"- {s['name']}: {s['state']}{health}")
-    emit_result("\n".join(lines), json_mode=False)
+    else:
+        emit_result(_render_status_text(payload), json_mode=False)
     return 0
 
 
@@ -254,28 +268,32 @@ def _stack_overview(args: argparse.Namespace) -> int:
     return 0
 
 
+def _add_json_flag(p: argparse.ArgumentParser) -> None:
+    p.add_argument("--json", action="store_true", help="Emit structured JSON.")
+
+
 def register(sub: argparse._SubParsersAction) -> None:
     p = sub.add_parser(
         "stack",
         help="Manage the storage substrate (mongo + neo4j) via docker compose.",
     )
-    p.add_argument("--json", action="store_true", help="Emit structured JSON.")
+    _add_json_flag(p)
     p.set_defaults(func=_stack_overview, json=False)
     # Propagate the structured-error parser_class to nested verbs.
     verb = p.add_subparsers(dest="stack_command", parser_class=type(p))
 
     up = verb.add_parser("up", help="Bring the stack up (docker compose up -d).")
-    up.add_argument("--json", action="store_true", help="Emit structured JSON.")
+    _add_json_flag(up)
     up.set_defaults(func=cmd_stack_up)
 
     down = verb.add_parser("down", help="Stop the stack (docker compose down).")
-    down.add_argument("--json", action="store_true", help="Emit structured JSON.")
+    _add_json_flag(down)
     down.set_defaults(func=cmd_stack_down)
 
     status = verb.add_parser("status", help="Report per-service state + health.")
-    status.add_argument("--json", action="store_true", help="Emit structured JSON.")
+    _add_json_flag(status)
     status.set_defaults(func=cmd_stack_status)
 
     ov = verb.add_parser("overview", help="Describe the stack noun.")
-    ov.add_argument("--json", action="store_true", help="Emit structured JSON.")
+    _add_json_flag(ov)
     ov.set_defaults(func=_stack_overview)

{data_refinery_cli-0.5.0 → data_refinery_cli-0.5.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "data-refinery-cli"
-version = "0.5.0"
+version = "0.5.1"
 description = "Agent and CLI for data quality in storage and retrieval — validating, deduplicating, and checking the integrity and freshness of data as it is stored and fetched. Split out of eidetic-cli so eidetic keeps agent-memory; sibling to daria, the Data Refinery Intelligent Agent."
 readme = "README.md"
 license = "Apache-2.0"

{data_refinery_cli-0.5.0 → data_refinery_cli-0.5.1}/uv.lock

@@ -156,7 +156,7 @@ wheels = [
 
 [[package]]
 name = "data-refinery-cli"
-version = "0.5.0"
+version = "0.5.1"
 source = { editable = "." }
 
 [package.optional-dependencies]