detectkit 0.8.1__tar.gz → 0.8.2__tar.gz

{detectkit-0.8.1/detectkit.egg-info → detectkit-0.8.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: detectkit
-Version: 0.8.1
+Version: 0.8.2
 Summary: Metric monitoring with automatic anomaly detection
 Author: detectkit team
 License: MIT

{detectkit-0.8.1 → detectkit-0.8.2}/detectkit/__init__.py

@@ -4,7 +4,7 @@ detectk - Anomaly Detection for Time-Series Metrics
 A Python library for data analysts and engineers to monitor metrics with automatic anomaly detection.
 """
 
-__version__ = "0.8.1"
+__version__ = "0.8.2"
 
 from detectkit.core.interval import Interval
 from detectkit.core.models import ColumnDefinition, TableModel

detectkit-0.8.2/detectkit/cli/_output.py

@@ -0,0 +1,50 @@
+"""Shared CLI output helpers so every command renders in one house style.
+
+Mirrors the load → detect → alert pipeline's tree look (``┌─ / │ / └─``) used by
+``dtk run`` so the maintenance commands (``dtk clean``, ``dtk unlock``) match it
+instead of each inventing its own formatting.
+
+House conventions:
+- A metric *with* something to report is a tree: a cyan-bold ``┌─ <name>``
+  header followed by one child line per item (``│   `` for all but the last,
+  ``└─ `` for the last).
+- A metric with *nothing* to do is a single ``•`` line.
+- A per-metric error is a red ``✗`` line (to stderr).
+- The final summary is a cyan-bold ``Done. …`` line.
+"""
+
+from __future__ import annotations
+
+import click
+
+
+def echo_tree(name: str, children: list[str], *, warnings: list[str] | None = None) -> None:
+    """Print a ``┌─ name`` header with ``│``/``└─`` child lines.
+
+    ``warnings`` (if any) are rendered as yellow ``│`` continuation lines above
+    the children. ``children`` must be non-empty (a metric with no items should
+    use :func:`echo_noop` instead).
+    """
+    click.echo(click.style(f"  ┌─ {name}", fg="cyan", bold=True))
+    for warning in warnings or []:
+        click.echo(click.style(f"  │   ⚠ {warning}", fg="yellow", bold=True))
+    last = len(children) - 1
+    for i, child in enumerate(children):
+        prefix = "  └─ " if i == last else "  │   "
+        click.echo(f"{prefix}{child}")
+
+
+def echo_noop(name: str, reason: str) -> None:
+    """A metric with nothing to do — a single ``•`` line."""
+    click.echo(f"  • {name}: {reason}")
+
+
+def echo_error(name: str, message: str) -> None:
+    """A per-metric failure — a red ``✗`` line on stderr."""
+    click.echo(click.style(f"  ✗ {name}: {message}", fg="red"), err=True)
+
+
+def echo_done(summary: str) -> None:
+    """The closing ``Done. …`` summary (cyan, bold), preceded by a blank line."""
+    click.echo()
+    click.echo(click.style(f"Done. {summary}", fg="cyan", bold=True))

{detectkit-0.8.1 → detectkit-0.8.2}/detectkit/cli/commands/clean.py

@@ -27,6 +27,7 @@ from pathlib import Path
 
 import click
 
+from detectkit.cli._output import echo_done, echo_error, echo_noop, echo_tree
 from detectkit.cli.commands.run import find_project_root, select_metrics
 from detectkit.config.metric_config import MetricConfig
 from detectkit.config.profile import ProfilesConfig
@@ -112,6 +113,8 @@ def _clean_drift(
     total_det_groups = 0
     total_alert_rows = 0
 
+    verb = "deleting" if execute else "would delete"
+
     for _, config in metrics:
         metric_name = config.name
         try:
@@ -120,7 +123,7 @@ def _clean_drift(
             db_detectors = internal_manager.list_detector_ids(metric_name)
             db_alerts = internal_manager.list_alert_config_ids(metric_name)
         except Exception as e:
-            click.echo(click.style(f"  ✗ {metric_name}: error inspecting: {e}", fg="red"), err=True)
+            echo_error(metric_name, f"error inspecting: {e}")
             continue
 
         orphan_detectors = {
@@ -129,55 +132,42 @@ def _clean_drift(
         orphan_alerts = [a for a in db_alerts if a not in valid_alerts]
 
         if not orphan_detectors and not orphan_alerts:
-            click.echo(f"  • {metric_name}: nothing stale")
+            echo_noop(metric_name, "nothing stale")
             continue
 
-        click.echo(click.style(f"  {metric_name}:", bold=True))
+        children = [
+            f"detector {det_id}: {verb} {count:,} detection row(s)"
+            for det_id, count in sorted(orphan_detectors.items())
+        ] + [
+            f"alert_config {alert_id}: {verb} stale alert state"
+            for alert_id in sorted(orphan_alerts)
+        ]
 
         # An empty valid set means EVERY stored row is "orphaned" — usually a
         # config mid-edit, not an intent to wipe the metric. Flag it loudly.
+        warnings = []
         if orphan_detectors and not valid_detectors:
-            click.echo(
-                click.style(
-                    "    ⚠ config defines no detectors — ALL detections below would be removed",
-                    fg="yellow",
-                    bold=True,
-                )
-            )
+            warnings.append("config defines no detectors — ALL detections below would be removed")
         if orphan_alerts and not valid_alerts:
-            click.echo(
-                click.style(
-                    "    ⚠ config defines no alerting — ALL alert states below would be removed",
-                    fg="yellow",
-                    bold=True,
-                )
-            )
+            warnings.append("config defines no alerting — ALL alert states below would be removed")
+
+        echo_tree(metric_name, children, warnings=warnings)
 
-        for det_id, count in sorted(orphan_detectors.items()):
-            total_det_groups += 1
-            verb = "deleting" if execute else "would delete"
-            click.echo(f"    detector {det_id}: {verb} {count:,} detection row(s)")
-            if execute:
+        if execute:
+            for det_id in orphan_detectors:
                 internal_manager.delete_detections(
                     metric_name=metric_name, detector_id=det_id, mutations_sync=True
                 )
-
-        for alert_id in sorted(orphan_alerts):
-            total_alert_rows += 1
-            verb = "deleting" if execute else "would delete"
-            click.echo(f"    alert_config {alert_id}: {verb} stale alert state")
-            if execute:
+            for alert_id in orphan_alerts:
                 internal_manager.delete_alert_state(metric_name, alert_id)
 
-    click.echo()
-    prefix = "Deleted" if execute else "Would delete"
-    click.echo(
-        click.style(
-            f"{prefix} {total_det_groups} orphaned detector group(s) "
-            f"and {total_alert_rows} orphaned alert-state row(s).",
-            fg="cyan",
-            bold=True,
-        )
+        total_det_groups += len(orphan_detectors)
+        total_alert_rows += len(orphan_alerts)
+
+    verb_done = "Removed" if execute else "Would remove"
+    echo_done(
+        f"{verb_done} {total_det_groups} detector group(s) "
+        f"and {total_alert_rows} alert-state row(s)."
     )
     if not execute and (total_det_groups or total_alert_rows):
         click.echo("Re-run with --execute to apply.")
@@ -222,15 +212,10 @@ def _clean_orphaned_metrics(
         try:
             counts = internal_manager.count_metric_rows(name)
         except Exception as e:
-            click.echo(click.style(f"  ✗ {name}: error counting rows: {e}", fg="red"), err=True)
+            echo_error(name, f"error counting rows: {e}")
             continue
-        total = sum(counts.values())
-        verb = "deleting" if execute else "would delete"
-        detail = ", ".join(f"{table}={count:,}" for table, count in counts.items() if count)
-        click.echo(
-            click.style(f"  {name}: {verb} {total:,} row(s)", bold=True)
-            + (f"  [{detail}]" if detail else "")
-        )
+        children = [f"{table}: {count:,} row(s)" for table, count in counts.items() if count]
+        echo_tree(name, children or ["(no rows)"])
 
     if not execute:
         click.echo()
@@ -264,16 +249,10 @@ def _clean_orphaned_metrics(
         try:
             internal_manager.purge_metric(name)
             purged += 1
-            click.echo(click.style(f"  ✓ {name}: purged", fg="green"))
         except Exception as e:
-            click.echo(click.style(f"  ✗ {name}: error purging: {e}", fg="red"), err=True)
+            echo_error(name, f"error purging: {e}")
 
-    click.echo()
-    click.echo(
-        click.style(
-            f"Done. Purged {purged} of {len(orphans)} orphaned metric(s).", fg="cyan", bold=True
-        )
-    )
+    echo_done(f"Purged {purged} of {len(orphans)} orphaned metric(s).")
 
 
 # ── helpers ──────────────────────────────────────────────────────────────────

{detectkit-0.8.1 → detectkit-0.8.2}/detectkit/cli/commands/unlock.py

@@ -8,6 +8,7 @@ auto-expires after its timeout, but this command clears it immediately.
 
 import click
 
+from detectkit.cli._output import echo_done, echo_error, echo_noop, echo_tree
 from detectkit.cli.commands.run import find_project_root, select_metrics
 from detectkit.config.profile import ProfilesConfig
 from detectkit.database.internal_tables import InternalTablesManager
@@ -83,23 +84,13 @@ def run_unlock(select: str, profile: str | None):
         try:
             was_locked = internal_manager.clear_lock(metric_name)
         except Exception as e:
-            click.echo(
-                click.style(f"  ✗ {metric_name}: error clearing lock: {e}", fg="red"),
-                err=True,
-            )
+            echo_error(metric_name, f"error clearing lock: {e}")
             continue
 
         if was_locked:
             cleared += 1
-            click.echo(click.style(f"  ✓ {metric_name}: lock cleared", fg="green"))
+            echo_tree(metric_name, ["lock cleared"])
         else:
-            click.echo(f"  • {metric_name}: no active lock")
+            echo_noop(metric_name, "no active lock")
 
-    click.echo()
-    click.echo(
-        click.style(
-            f"Done. Cleared {cleared} lock(s) of {len(metrics)} metric(s).",
-            fg="cyan",
-            bold=True,
-        )
-    )
+    echo_done(f"Cleared {cleared} lock(s) of {len(metrics)} metric(s).")

{detectkit-0.8.1 → detectkit-0.8.2/detectkit.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: detectkit
-Version: 0.8.1
+Version: 0.8.2
 Summary: Metric monitoring with automatic anomaly detection
 Author: detectkit team
 License: MIT

{detectkit-0.8.1 → detectkit-0.8.2}/detectkit.egg-info/SOURCES.txt

@@ -29,6 +29,7 @@ detectkit/alerting/orchestrator/_recovery.py
 detectkit/alerting/orchestrator/_types.py
 detectkit/alerting/orchestrator/orchestrator.py
 detectkit/cli/__init__.py
+detectkit/cli/_output.py
 detectkit/cli/main.py
 detectkit/cli/commands/__init__.py
 detectkit/cli/commands/clean.py