notesctl 1.0.2__tar.gz → 1.0.3__tar.gz

{notesctl-1.0.2 → notesctl-1.0.3}/CHANGELOG.md

@@ -7,6 +7,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.0.3] - 2026-06-08
+
+### Fixed
+
+- `export` now exits with a non-zero status when notes are skipped due to errors, so backup and automation workflows no longer treat an incomplete export as successful
+- `list-notes --limit` rejects values below 1, preventing negative limits from hiding notes and printing a misleading "Showing -1 of N notes" count
+
 ## [1.0.2] - 2026-01-17
 
 ### Fixed
@@ -50,7 +57,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Original Notes database is never modified
 - All queries validated to be SELECT statements only
 
-[Unreleased]: https://github.com/jwmoss/notesctl/compare/v1.0.2...HEAD
+[Unreleased]: https://github.com/jwmoss/notesctl/compare/v1.0.3...HEAD
+[1.0.3]: https://github.com/jwmoss/notesctl/compare/v1.0.2...v1.0.3
 [1.0.2]: https://github.com/jwmoss/notesctl/compare/v1.0.1...v1.0.2
 [1.0.1]: https://github.com/jwmoss/notesctl/compare/v1.0.0...v1.0.1
 [1.0.0]: https://github.com/jwmoss/notesctl/releases/tag/v1.0.0

{notesctl-1.0.2 → notesctl-1.0.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: notesctl
-Version: 1.0.2
+Version: 1.0.3
 Summary: Export Apple iCloud Notes to Markdown files safely
 Project-URL: Homepage, https://github.com/jwmoss/notesctl
 Project-URL: Repository, https://github.com/jwmoss/notesctl

{notesctl-1.0.2 → notesctl-1.0.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "notesctl"
-version = "1.0.2"
+version = "1.0.3"
 description = "Export Apple iCloud Notes to Markdown files safely"
 readme = "README.md"
 requires-python = ">=3.11"

{notesctl-1.0.2 → notesctl-1.0.3}/src/notesctl/cli.py

@@ -129,6 +129,9 @@ def export(
         for error in result.errors[:10]:
             console.print(f"  {error}")
 
+    if not dry_run and (result.skipped_errors or result.errors):
+        raise typer.Exit(1)
+
 
 @app.command("list-notes")
 def list_notes(
@@ -153,6 +156,7 @@ def list_notes(
             "--limit",
             "-l",
             help="Maximum number of notes to show",
+            min=1,
         ),
     ] = 50,
     db_path: Annotated[

notesctl-1.0.3/tests/test_cli.py

@@ -0,0 +1,49 @@
+"""Tests for the command-line interface."""
+
+from pathlib import Path
+from typing import Any
+
+from typer.testing import CliRunner
+
+from notesctl.cli import app
+from notesctl.exporter import ExportResult
+
+runner = CliRunner()
+
+
+def test_export_exits_nonzero_when_notes_fail(monkeypatch):
+    """Export should fail automation when notes are skipped due to errors."""
+
+    class StubExporter:
+        def __init__(self, db_path: Path | None = None) -> None:
+            self.db_path = db_path
+
+        def export(self, *_args: Any) -> ExportResult:
+            return ExportResult(
+                total_notes=2,
+                exported_notes=1,
+                skipped_encrypted=0,
+                skipped_errors=1,
+                attachments_copied=0,
+                output_path=Path("/tmp/notes-export"),
+                exported_files=[],
+                errors=["boom"],
+            )
+
+    monkeypatch.setattr("notesctl.cli.NotesExporter", StubExporter)
+
+    result = runner.invoke(app, ["export"])
+
+    assert result.exit_code == 1
+    assert "Failed to export 1 notes" in result.output
+    assert "boom" in result.output
+
+
+def test_list_notes_rejects_negative_limit():
+    """Negative limits should fail CLI validation before rendering notes."""
+    result = runner.invoke(app, ["list-notes", "--limit", "-1"])
+
+    assert result.exit_code != 0
+    assert "Invalid value" in result.output
+    assert "Showing -1" not in result.output
+    assert "Notes (" not in result.output