mac-upkeep 3.0.2__tar.gz → 3.0.3__tar.gz

mac_upkeep-3.0.3/.release-please-manifest.json

@@ -0,0 +1,3 @@
+{
+  ".": "3.0.3"
+}

{mac_upkeep-3.0.2 → mac_upkeep-3.0.3}/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [3.0.3](https://github.com/calvindotsg/mac-upkeep/compare/v3.0.2...v3.0.3) (2026-08-17)
+
+
+### Bug Fixes
+
+* report handler tasks as dry-run instead of as having executed ([#51](https://github.com/calvindotsg/mac-upkeep/issues/51)) ([1db8054](https://github.com/calvindotsg/mac-upkeep/commit/1db8054d65da313f6fe4d51f00932d126fc59c79))
+
 ## [3.0.2](https://github.com/calvindotsg/mac-upkeep/compare/v3.0.1...v3.0.2) (2026-08-16)
 
 

{mac_upkeep-3.0.2 → mac_upkeep-3.0.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: mac-upkeep
-Version: 3.0.2
+Version: 3.0.3
 Summary: Automated macOS maintenance CLI
 Project-URL: Homepage, https://github.com/calvindotsg/mac-upkeep
 Project-URL: Repository, https://github.com/calvindotsg/mac-upkeep

{mac_upkeep-3.0.2 → mac_upkeep-3.0.3}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "mac-upkeep"
-version = "3.0.2"
+version = "3.0.3"
 description = "Automated macOS maintenance CLI"
 readme = "README.md"
 license = "MIT"

{mac_upkeep-3.0.2 → mac_upkeep-3.0.3}/src/mac_upkeep/output.py

@@ -97,9 +97,9 @@ class Output:
             if t.status == "running":
                 icon = Spinner("dots", style="yellow")
                 detail = Text("running", style="yellow")
-            elif t.status == "ok" and t.reason == "dry-run":
+            elif t.status == "ok" and t.reason.startswith("dry-run"):
                 icon = Text(_DRY, style="cyan")
-                detail = Text("dry-run", style="dim")
+                detail = Text(t.reason, style="dim")
             elif t.status == "ok":
                 icon = Text(_OK, style="green")
                 detail = Text(f"{t.duration:.1f}s", style="dim") if t.duration else Text("")
@@ -138,8 +138,15 @@ class Output:
         else:
             if result.status == "skipped":
                 logger.info("SKIP: %s (%s)", result.name, result.reason)
-            elif result.status == "ok" and result.reason == "dry-run":
-                logger.info("DRY-RUN: would run %s", result.name)
+            elif result.status == "ok" and result.reason.startswith("dry-run"):
+                # Handlers return a richer reason ("dry-run: 17 repos"); command
+                # tasks return the bare "dry-run". Surface the extra detail when
+                # there is any, so a handler preview is not reduced to a stub.
+                detail = result.reason[len("dry-run") :].lstrip(": ")
+                if detail:
+                    logger.info("DRY-RUN: would run %s (%s)", result.name, detail)
+                else:
+                    logger.info("DRY-RUN: would run %s", result.name)
             elif result.status == "ok":
                 logger.info("Running %s... done", result.name)
             elif result.status == "failed":

{mac_upkeep-3.0.2 → mac_upkeep-3.0.3}/tests/test_output.py

@@ -69,6 +69,49 @@ def test_output_non_interactive_task_done_dry_run(caplog):
     assert "DRY-RUN" in caplog.text
 
 
+def test_output_non_interactive_task_done_dry_run_handler_reason(caplog):
+    """Handler tasks return 'dry-run: <detail>', not the bare 'dry-run'.
+
+    An exact-equality check fell through to the 'Running X... done' branch, so a
+    dry run reported handler tasks as if they had executed.
+    """
+    import logging
+
+    output = _non_interactive()
+    result = TaskResult("git_sync", "ok", reason="dry-run: 17 repos")
+    with caplog.at_level(logging.INFO, logger="mac_upkeep"):
+        output.task_done(result)
+    assert "DRY-RUN: would run git_sync (17 repos)" in caplog.text
+    assert "done" not in caplog.text
+
+
+def test_output_non_interactive_task_done_dry_run_bare_has_no_empty_detail(caplog):
+    """The bare 'dry-run' reason must not render an empty '()' suffix."""
+    import logging
+
+    output = _non_interactive()
+    result = TaskResult("gcloud", "ok", reason="dry-run")
+    with caplog.at_level(logging.INFO, logger="mac_upkeep"):
+        output.task_done(result)
+    assert "DRY-RUN: would run gcloud" in caplog.text
+    assert "()" not in caplog.text
+
+
+def test_output_non_interactive_ok_reason_is_not_treated_as_dry_run(caplog):
+    """A successful handler reason must still read as a real run.
+
+    Guards the prefix check against over-matching: only 'dry-run...' is a preview.
+    """
+    import logging
+
+    output = _non_interactive()
+    result = TaskResult("editor_cache", "ok", reason="nothing to clean")
+    with caplog.at_level(logging.INFO, logger="mac_upkeep"):
+        output.task_done(result)
+    assert "Running editor_cache... done" in caplog.text
+    assert "DRY-RUN" not in caplog.text
+
+
 def test_output_non_interactive_task_done_skipped(caplog):
     import logging
 

{mac_upkeep-3.0.2 → mac_upkeep-3.0.3}/uv.lock

@@ -43,7 +43,7 @@ wheels = [
 
 [[package]]
 name = "mac-upkeep"
-version = "3.0.2"
+version = "3.0.3"
 source = { editable = "." }
 dependencies = [
     { name = "typer" },

mac_upkeep-3.0.2/.release-please-manifest.json

@@ -1,3 +0,0 @@
-{
-  ".": "3.0.2"
-}