readability-cli 0.8.1__tar.gz → 0.8.2__tar.gz

{readability_cli-0.8.1 → readability_cli-0.8.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: readability-cli
-Version: 0.8.1
+Version: 0.8.2
 Summary: A CLI to lint, format, and type-check code with Google-style defaults, and pull Google style guides in markdown format.
 Project-URL: Homepage, https://github.com/owahltinez/readability
 Project-URL: Repository, https://github.com/owahltinez/readability
@@ -164,7 +164,10 @@ if report.findings or report.failed or not report.ran:
 
 The returned `CheckReport` records only whether findings occurred and which
 tools ran, were skipped, or failed. Detailed tool findings are still written
-as each tool runs.
+as each tool runs. Every path is validated before any tool runs, and a missing
+one raises `FileNotFoundError` rather than being misreported as a finding.
+Relative paths remain relative to the process working directory;
+`project_root` controls configuration discovery only.
 
 ### Default Configurations
 

{readability_cli-0.8.1 → readability_cli-0.8.2}/README.md

@@ -150,7 +150,10 @@ if report.findings or report.failed or not report.ran:
 
 The returned `CheckReport` records only whether findings occurred and which
 tools ran, were skipped, or failed. Detailed tool findings are still written
-as each tool runs.
+as each tool runs. Every path is validated before any tool runs, and a missing
+one raises `FileNotFoundError` rather than being misreported as a finding.
+Relative paths remain relative to the process working directory;
+`project_root` controls configuration discovery only.
 
 ### Default Configurations
 

{readability_cli-0.8.1 → readability_cli-0.8.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "readability-cli"
-version = "0.8.1"
+version = "0.8.2"
 description = "A CLI to lint, format, and type-check code with Google-style defaults, and pull Google style guides in markdown format."
 readme = "README.md"
 requires-python = ">=3.12"

{readability_cli-0.8.1 → readability_cli-0.8.2}/readability.py

@@ -1196,18 +1196,30 @@ def check_paths(
     the result.
 
     Args:
-        paths: Files or directories to check, as strings or paths.
-        project_root: Root used to discover tool configuration. Defaults to
-            the current working directory.
+        paths: Files or directories to check, as strings or paths. Relative
+            paths are interpreted from the current working directory.
+        project_root: Root used only to discover tool configuration. Defaults
+            to the current working directory; it does not rebase paths.
         fix: Whether to apply automatic fixes.
 
     Returns:
         A report aggregated across all provided paths.
+
+    Raises:
+        FileNotFoundError: If any requested path does not exist. Every path is
+            validated before any tools run.
     """
     root = Path.cwd() if project_root is None else project_root
+    requested_paths = [Path(path) for path in paths]
+    missing_path = next(
+        (path for path in requested_paths if not path.exists()), None
+    )
+    if missing_path is not None:
+        raise FileNotFoundError(f"Path does not exist: {missing_path}")
+
     report = CheckReport()
-    for path in paths:
-        report.absorb(_check_path(Path(path), root, fix=fix))
+    for path in requested_paths:
+        report.absorb(_check_path(path, root, fix=fix))
     return report
 
 

{readability_cli-0.8.1 → readability_cli-0.8.2}/test_readability.py

@@ -522,10 +522,17 @@ def test_bundled_default_configs_are_valid(tmp_path: Path) -> None:
 
 @patch("readability._check_path")
 def test_check_paths_aggregates_str_and_path_inputs(
-    mock_check_path: MagicMock, tmp_path: Path
+    mock_check_path: MagicMock,
+    tmp_path: Path,
+    monkeypatch: pytest.MonkeyPatch,
 ) -> None:
     """The public API folds each per-path result into one limited report."""
     project_root = tmp_path / "project"
+    project_root.mkdir()
+    (tmp_path / "src").mkdir()
+    (tmp_path / "src" / "example.py").touch()
+    (tmp_path / "README.md").touch()
+    monkeypatch.chdir(tmp_path)
     mock_check_path.side_effect = [
         CheckReport(ran={"ruff"}, skipped={"pyrefly"}),
         CheckReport(
@@ -561,6 +568,7 @@ def test_check_paths_defaults_project_root_to_cwd(
 ) -> None:
     """The public API discovers configuration from the caller's directory."""
     monkeypatch.chdir(tmp_path)
+    (tmp_path / "example.py").touch()
 
     check_paths([Path("example.py")])
 
@@ -581,6 +589,23 @@ def test_check_paths_with_no_paths_reports_that_nothing_ran(
     mock_check_path.assert_not_called()
 
 
+@patch("readability._check_path")
+def test_check_paths_rejects_missing_paths_before_dispatch(
+    mock_check_path: MagicMock,
+    tmp_path: Path,
+) -> None:
+    """A missing path prevents partial checks and fixes on earlier paths."""
+    existing = tmp_path / "exists.py"
+    missing = tmp_path / "missing.py"
+    existing.touch()
+
+    with pytest.raises(FileNotFoundError) as excinfo:
+        check_paths([existing, missing], project_root=tmp_path, fix=True)
+
+    assert str(missing) in str(excinfo.value)
+    mock_check_path.assert_not_called()
+
+
 @patch(
     "readability._check_path",
     return_value=CheckReport(
@@ -590,10 +615,15 @@ def test_check_paths_with_no_paths_reports_that_nothing_ran(
     ),
 )
 def test_check_paths_returns_report_without_cli_status_prose(
-    mock_check_path: MagicMock, capsys: pytest.CaptureFixture[str]
+    mock_check_path: MagicMock,
+    capsys: pytest.CaptureFixture[str],
+    tmp_path: Path,
 ) -> None:
     """Library callers receive report states without CLI policy or summaries."""
-    report = check_paths(["example.py"])
+    path = tmp_path / "example.py"
+    path.touch()
+
+    report = check_paths([path])
 
     assert report.findings is True
     assert report.skipped == {"pyrefly"}

{readability_cli-0.8.1 → readability_cli-0.8.2}/uv.lock

@@ -194,7 +194,7 @@ wheels = [
 
 [[package]]
 name = "readability-cli"
-version = "0.8.1"
+version = "0.8.2"
 source = { editable = "." }
 dependencies = [
     { name = "beautifulsoup4" },