fix: handle PermissionError for directory input on Windows CLI (#3149)

joaquinhuigomez · joaquinhui · claude · web-flow · commit a39317a14785 · 2026-03-19T14:06:17.000+01:00
* fix: handle PermissionError for directory input on Windows CLI On Windows, Path.read_bytes() on a directory raises PermissionError instead of IsADirectoryError (which is raised on Linux/macOS). This caused the CLI to crash when a directory was passed as input source. Catch PermissionError alongside IsADirectoryError so directory inputs are handled correctly on all platforms. Fixes #3138 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * DCO Remediation Commit for Joaquin Hui Gomez <joaquinhui1995@gmail.com> I, Joaquin Hui Gomez <joaquinhui1995@gmail.com>, hereby add my Signed-off-by to this commit: 90c738c Signed-off-by: Joaquin Hui Gomez <joaquinhui1995@gmail.com> --------- Signed-off-by: Joaquin Hui Gomez <joaquinhui1995@gmail.com> Co-authored-by: Joaquin Hui Gomez <joaquinhui1995@gmail.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
diff --git a/docling/cli/main.py b/docling/cli/main.py
@@ -654,7 +654,7 @@ def convert(  # noqa: C901
                     f"[red]Error: The input file {src} does not exist.[/red]"
                 )
                 raise typer.Abort()
-            except IsADirectoryError:
+            except (IsADirectoryError, PermissionError):
                 # if the input matches to a file or a folder
                 try:
                     local_path = TypeAdapter(Path).validate_python(src)

Original file line number	Diff line number	Diff line change
`@@ -654,7 +654,7 @@ def convert( # noqa: C901`
`654`	`654`	`f"[red]Error: The input file {src} does not exist.[/red]"`
`655`	`655`	`)`
`656`	`656`	`raise typer.Abort()`
`657`		`- except IsADirectoryError:`
	`657`	`+ except (IsADirectoryError, PermissionError):`
`658`	`658`	`# if the input matches to a file or a folder`
`659`	`659`	`try:`
`660`	`660`	`local_path = TypeAdapter(Path).validate_python(src)`