treex-cli 0.2.2__tar.gz → 0.2.4__tar.gz

{treex_cli-0.2.2 → treex_cli-0.2.4}/Cargo.lock

@@ -352,9 +352,9 @@ dependencies = [
 
 [[package]]
 name = "gix-config"
-version = "0.62.1"
+version = "0.62.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "2d8de82251a43c9f845458cd64e565f7350ce446bb786348f79d80dc5089806e"
+checksum = "3e11ccf82e9630c4e7befbae07cbe90e79de61dc73a34448c2dec70a13da9ebf"
 dependencies = [
  "bstr",
  "gix-config-value",
@@ -496,9 +496,9 @@ dependencies = [
 
 [[package]]
 name = "gix-hashtable"
-version = "0.18.0"
+version = "0.18.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "916cb6b2ec42ffaca624e7867bb382a4673015cb4ffcd7330b48085d02e4c615"
+checksum = "2f66822aee11ccb28d6a2e63b587254e578fb52a138bd5806b58d8560125592b"
 dependencies = [
  "gix-hash",
  "hashbrown 0.17.1",
@@ -507,9 +507,9 @@ dependencies = [
 
 [[package]]
 name = "gix-ignore"
-version = "0.24.0"
+version = "0.24.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "cb1c93f60845d493a72fde38ae237498bc4e4464f45afa4af23e53f8295a3ba1"
+checksum = "968f12fe42896873504f46127cd5bf0f9a34fc1479a781f43cf94b5d8d760a15"
 dependencies = [
  "bstr",
  "gix-glob",
@@ -660,9 +660,9 @@ dependencies = [
 
 [[package]]
 name = "gix-parallel"
-version = "0.50.0"
+version = "0.50.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "e994cf1eec247ba4be7f9e7c0d58cc73f83eecb5fcb9feb595df5ba3d2078546"
+checksum = "58e7e10f8343e05f6ad8651f97dffa1caca1531bb274bfc8e3050e80e3ed5946"
 dependencies = [
  "crossbeam-channel",
  "gix-trace",
@@ -777,9 +777,9 @@ dependencies = [
 
 [[package]]
 name = "gix-sec"
-version = "0.16.0"
+version = "0.16.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "5640f642a1b471616d48230b1def7ca743cb30903441ff84800cafbab3c0fddf"
+checksum = "9598d1f35baec7038c2d278bfcc18f1d8951f41dcc3f51ab6175661dfe5a6137"
 dependencies = [
  "bitflags 2.13.2",
  "gix-path",
@@ -869,9 +869,9 @@ dependencies = [
 
 [[package]]
 name = "gix-utils"
-version = "0.5.1"
+version = "0.5.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "f8e770b8c34311044d1addc7b18957db0ac27183707e1140dc90e6f674a6354e"
+checksum = "11a9446060447e9ec58f5d333b98bed53389dfa10de5d8dd46fc76d01dfc2954"
 dependencies = [
  "bstr",
  "bytes",
@@ -1257,9 +1257,9 @@ checksum = "ad8553b9b26413251cbf30e620595c7a41b3887f03da04579c0e6b0d6a06b4b2"
 
 [[package]]
 name = "rustix"
-version = "1.1.5"
+version = "1.1.6"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "891efababe418670775f199f0d233d84843c227a0949a883ce15b37c78d6629d"
+checksum = "e5e76742e5294153d4dde1207dbec5bdd81eef6fabdc0d2118babadabfd858aa"
 dependencies = [
  "bitflags 2.13.2",
  "errno",
@@ -1332,9 +1332,9 @@ checksum = "e3a9fe34e3e7a50316060351f37187a3f546bce95496156754b601a5fa71b76e"
 
 [[package]]
 name = "smallvec"
-version = "1.16.3"
+version = "1.16.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "5b3dc8af474f516a851ff4bd12db780f948b9250ad37211e4eec0bccea54e01b"
+checksum = "d0d5d4b1981b8bbbb7aed204cae5726d683d5d6ac7e74ec54122ea28f315e7c8"
 
 [[package]]
 name = "stable_deref_trait"
@@ -1411,7 +1411,7 @@ checksum = "fd3ca314f692efd6c868f8408f53fe444634a845f96c028b97d35f6a1f79f0ee"
 
 [[package]]
 name = "treex-native"
-version = "0.2.2"
+version = "0.2.4"
 dependencies = [
  "gix",
  "pyo3",

{treex_cli-0.2.2 → treex_cli-0.2.4}/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "treex-native"
-version = "0.2.2"
+version = "0.2.4"
 edition = "2024"
 
 [lib]

{treex_cli-0.2.2 → treex_cli-0.2.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: treex-cli
-Version: 0.2.2
+Version: 0.2.4
 Summary: Command-line directory tree utility with metadata and Git ignore support
 Author: Corey Goldberg
 Maintainer: Corey Goldberg
@@ -110,8 +110,8 @@ Prebuilt wheels are available for common platforms. See
 ## Git ignore support
 
 `treex` uses [Gitoxide][gitoxide-home] (`gix`), a native Rust implementation of
-Git functionality, to evaluate `.gitignore` rules without invoking the Git
-executable. Git does not need to be installed for ignore matching to work.
+Git, to evaluate `.gitignore` rules without invoking the Git executable. Git
+does not need to be installed for ignore matching to work.
 
 When the specified directory is inside a Git working tree, `treex` respects
 applicable `.gitignore` rules by default. Outside a Git working tree, Git ignore

{treex_cli-0.2.2 → treex_cli-0.2.4}/README.md

@@ -87,8 +87,8 @@ Prebuilt wheels are available for common platforms. See
 ## Git ignore support
 
 `treex` uses [Gitoxide][gitoxide-home] (`gix`), a native Rust implementation of
-Git functionality, to evaluate `.gitignore` rules without invoking the Git
-executable. Git does not need to be installed for ignore matching to work.
+Git, to evaluate `.gitignore` rules without invoking the Git executable. Git
+does not need to be installed for ignore matching to work.
 
 When the specified directory is inside a Git working tree, `treex` respects
 applicable `.gitignore` rules by default. Outside a Git working tree, Git ignore

{treex_cli-0.2.2 → treex_cli-0.2.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "treex-cli"
-version = "0.2.2"
+version = "0.2.4"
 description="Command-line directory tree utility with metadata and Git ignore support"
 requires-python = ">= 3.13"
 license = "MIT"
@@ -37,13 +37,14 @@ repository = "https://github.com/cgoldberg/treex"
 
 [dependency-groups]
 build = [
-    "cibuildwheel==4.3.4",
+    "cibuildwheel==4.3.5",
+    "twine==7.0.0",
 ]
 lint = [
-    "ruff==0.17.2",
+    "ruff==0.17.3",
 ]
 test = [
-    "pytest==9.1.1",
+    "pytest==9.2.0",
 ]
 validate = [
     "validate-pyproject[all]==0.27",
@@ -64,12 +65,10 @@ py-limited-api = true
 
 [tool.cibuildwheel]
 build = "cp313-*"
-test-requires = ["pytest==9.1.1"]
-test-command = "pytest {project}/tests"
-
-[tool.cibuildwheel.linux]
 before-all = "curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y --profile minimal"
 environment = { PATH = "$HOME/.cargo/bin:$PATH" }
+test-requires = ["pytest==9.2.0"]
+test-command = "pytest {project}"
 
 [tool.pytest]
 addopts = ["-v"]

{treex_cli-0.2.2 → treex_cli-0.2.4}/src/treex/treex.py

@@ -15,6 +15,7 @@ the filesystem is scanned normally without Git filtering.
 
 import argparse
 import fnmatch
+import os
 import re
 import sys
 from datetime import datetime
@@ -29,6 +30,9 @@ except PackageNotFoundError:
     __version__ = "unknown"
 
 
+PROG = "treex"
+
+
 class GitIgnore:
     """Apply Git ignore rules using Gitoxide when inside a repository."""
 
@@ -53,26 +57,29 @@ class GitIgnore:
         """Check if Git considers a path ignored."""
         return bool(self.ignored_many([path]))
 
-    def ignored_many(self, paths):
-        """Check which paths Git considers ignored."""
+    def ignored_many(self, paths, directory_flags=None):
+        """Return the paths that Git considers ignored."""
         if not self.enabled or self.repo_root is None or self.matcher is None:
             return set()
-        relative_paths = []
+        candidates = []
         matcher_paths = []
         for path in paths:
             try:
-                relative = path.resolve().relative_to(self.repo_root)
+                relative = path.relative_to(self.repo_root)
             except (ValueError, OSError):
                 continue
-            relative_paths.append(relative.as_posix())
-            matcher_paths.append((relative.as_posix(), path.is_dir()))
+            candidates.append(path)
+            is_dir = (
+                directory_flags[path]
+                if directory_flags is not None and path in directory_flags
+                else path.is_dir()
+            )
+            matcher_paths.append((relative.as_posix(), is_dir))
         if not matcher_paths:
             return set()
         results = self.matcher.ignored_many(matcher_paths)
         return {
-            relative
-            for relative, ignored in zip(relative_paths, results, strict=True)
-            if ignored
+            path for path, ignored in zip(candidates, results, strict=True) if ignored
         }
 
 
@@ -125,7 +132,14 @@ def _count_lines(path):
 
 
 def _file_metadata(path):
-    """Get file metadata."""
+    """Collect file metadata, identifying binary files and counting text lines.
+
+    Reads files in chunks to avoid loading large files into memory. Checks
+    the initial bytes for NUL characters and invalid UTF-8 to identify binary
+    files, and counts lines for text files in a single pass. Empty files are
+    handled without opening them. Returns formatted size, line count or file
+    status, byte size, and modification time.
+    """
     try:
         stat = path.stat()
         size = stat.st_size
@@ -133,37 +147,71 @@ def _file_metadata(path):
     except OSError:
         return "?", "[unreadable]", 0, None
     size_text = _format_size(size)
-    if _is_binary_file(path):
+    if size == 0:
+        return size_text, "0 lines", size, modified
+    try:
+        f = path.open("rb")
+    except OSError:
         return size_text, "[binary]", size, modified
-    lines = _count_lines(path)
-    if lines is None:
-        return size_text, "[unreadable]", size, modified
+    with f:
+        try:
+            chunk = f.read(8192)
+        except OSError:
+            return size_text, "[binary]", size, modified
+        if chunk:
+            if b"\x00" in chunk:
+                return size_text, "[binary]", size, modified
+            try:
+                chunk.decode("utf-8")
+            except UnicodeDecodeError:
+                return size_text, "[binary]", size, modified
+        lines = chunk.count(b"\n")
+        last_byte = chunk[-1:] if chunk else b""
+        has_data = bool(chunk)
+        # A short read from a regular file normally means EOF.
+        if len(chunk) == 8192:
+            try:
+                while True:
+                    chunk = f.read(65536)
+                    if not chunk:
+                        break
+                    has_data = True
+                    lines += chunk.count(b"\n")
+                    last_byte = chunk[-1:]
+            except OSError:
+                return size_text, "[unreadable]", size, modified
+    if has_data and last_byte != b"\n":
+        lines += 1
     return size_text, f"{lines:,} lines", size, modified
 
 
 def _get_tree_entries(path, gitignore):
-    """Get entries to include in the tree."""
+    """Get entries to include in the tree with cached type information."""
+    typed_entries = []
     try:
-        entries = list(path.iterdir())
+        # Use os.scandir() instead of Path.iterdir(): DirEntry caches
+        # filesystem metadata, reducing extra system calls for type checks.
+        entries = os.scandir(path)
     except PermissionError:
         return None
-    entries = [entry for entry in entries if not entry.is_symlink()]
+    with entries:
+        for dir_entry in entries:
+            if dir_entry.is_symlink():
+                continue
+            is_dir = dir_entry.is_dir(follow_symlinks=False)
+            is_file = dir_entry.is_file(follow_symlinks=False)
+            typed_entries.append((Path(dir_entry.path), is_dir, is_file))
     if gitignore is not None:
-        entries = [entry for entry in entries if entry.name != ".git"]
+        typed_entries = [item for item in typed_entries if item[0].name != ".git"]
         if gitignore.enabled and gitignore.repo_root is not None:
-            ignored = gitignore.ignored_many(entries)
-            filtered_entries = []
-            for entry in entries:
-                try:
-                    relative = entry.resolve().relative_to(gitignore.repo_root)
-                except (ValueError, OSError):
-                    filtered_entries.append(entry)
-                    continue
-                if relative.as_posix() not in ignored:
-                    filtered_entries.append(entry)
-            entries = filtered_entries
-    entries.sort(key=lambda p: (p.is_file(), p.name.casefold()))
-    return entries
+            directory_flags = {entry: is_dir for entry, is_dir, _ in typed_entries}
+            ignored = gitignore.ignored_many(
+                [entry for entry, _, _ in typed_entries],
+                directory_flags,
+            )
+            typed_entries = [item for item in typed_entries if item[0] not in ignored]
+    typed_entries.sort(key=lambda item: (item[2], item[0].name.casefold()))
+    return typed_entries
 
 
 def _matches_pattern(path, pattern):
@@ -183,6 +231,7 @@ def _scan_tree(
     max_depth=None,
     depth=0,
     root=None,
+    relative_prefix="",
 ):
     """Scan and filter the tree, returning nodes for rendering.
 
@@ -194,8 +243,8 @@ def _scan_tree(
     Directory sizes include only files within the permitted depth
     that pass the include/exclude filters.
     """
-    path = path.resolve()
     if root is None:
+        path = path.resolve()
         root = path
     if max_depth is not None and depth >= max_depth:
         return []
@@ -214,14 +263,14 @@ def _scan_tree(
         ]
     nodes = []
     has_include = include_pattern is not None or include_regex is not None
-    for entry in entries:
-        relative_path = entry.relative_to(root).as_posix()
-        # Exclusions take precedence over inclusions.
+    for entry, is_dir, is_file in entries:
+        name = entry.name
+        relative_path = f"{relative_prefix}/{name}" if relative_prefix else name
         if _matches_pattern(relative_path, exclude_pattern):
             continue
         if exclude_regex is not None and exclude_regex.search(relative_path):
             continue
-        if entry.is_dir():
+        if is_dir:
             entry_depth = depth + 1
             if max_depth is not None and entry_depth >= max_depth:
                 children = []
@@ -238,6 +287,7 @@ def _scan_tree(
                     max_depth,
                     entry_depth,
                     root,
+                    relative_path,
                 )
             contains_match = any(child["contains_match"] for child in children)
             if has_include and not contains_match:
@@ -247,7 +297,7 @@ def _scan_tree(
             )
             nodes.append(
                 {
-                    "name": entry.name,
+                    "name": name,
                     "kind": "directory",
                     "children": children,
                     "size": size,
@@ -257,7 +307,7 @@ def _scan_tree(
                 }
             )
             continue
-        if not entry.is_file():
+        if not is_file:
             continue
         if include_pattern is not None and not _matches_pattern(
             relative_path,
@@ -272,7 +322,7 @@ def _scan_tree(
             size_text, info, size, modified = None, None, None, None
         nodes.append(
             {
-                "name": entry.name,
+                "name": name,
                 "kind": "file",
                 "children": [],
                 "size": size,
@@ -455,7 +505,7 @@ def parse_args(argv=None):
     parser = argparse.ArgumentParser(
         description="List directory contents as a tree "
         "with file and directory metadata.",
-        usage="treex [OPTIONS] [directory]",
+        usage=f"{PROG} [OPTIONS] [directory]",
         formatter_class=formatter,
     )
     parser.add_argument(
@@ -469,7 +519,7 @@ def parse_args(argv=None):
         "-V",
         "--version",
         action="version",
-        version=f"treex {__version__}",
+        version=f"{PROG} {__version__}",
         help="show version information and exit",
     )
     parser.add_argument(

{treex_cli-0.2.2 → treex_cli-0.2.4}/src/treex_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: treex-cli
-Version: 0.2.2
+Version: 0.2.4
 Summary: Command-line directory tree utility with metadata and Git ignore support
 Author: Corey Goldberg
 Maintainer: Corey Goldberg
@@ -110,8 +110,8 @@ Prebuilt wheels are available for common platforms. See
 ## Git ignore support
 
 `treex` uses [Gitoxide][gitoxide-home] (`gix`), a native Rust implementation of
-Git functionality, to evaluate `.gitignore` rules without invoking the Git
-executable. Git does not need to be installed for ignore matching to work.
+Git, to evaluate `.gitignore` rules without invoking the Git executable. Git
+does not need to be installed for ignore matching to work.
 
 When the specified directory is inside a Git working tree, `treex` respects
 applicable `.gitignore` rules by default. Outside a Git working tree, Git ignore

{treex_cli-0.2.2 → treex_cli-0.2.4}/tests/test_treex.py

@@ -223,7 +223,7 @@ class TestGitIgnore:
         ignored_dir.mkdir()
         paths = [ignored_file, normal_file, ignored_dir]
         gitignore = treex.GitIgnore(tmp_path)
-        assert gitignore.ignored_many(paths) == {"debug.log", "build"}
+        assert gitignore.ignored_many(paths) == {ignored_file, ignored_dir}
 
     def test_ignored_many_handles_spaces(self, tmp_path):
         init_git_repo(tmp_path)
@@ -234,7 +234,7 @@ class TestGitIgnore:
         ignored_dir.mkdir()
         paths = [ignored_file, ignored_dir]
         gitignore = treex.GitIgnore(tmp_path)
-        assert gitignore.ignored_many(paths) == {"debug output.log", "build output"}
+        assert gitignore.ignored_many(paths) == {ignored_file, ignored_dir}
 
     def test_missing_git_index(self, tmp_path):
         init_git_repo(tmp_path)
@@ -260,7 +260,7 @@ class TestTreeEntries:
         (tmp_path / "Docs").mkdir()
         (tmp_path / ".git").mkdir()
         entries = treex._get_tree_entries(tmp_path, gitignore=None)
-        assert [entry.name for entry in entries] == [
+        assert [entry[0].name for entry in entries] == [
             ".git",
             "Docs",
             "src",
@@ -278,7 +278,7 @@ class TestTreeEntries:
         link = tmp_path / "link.txt"
         link.symlink_to(target)
         entries = treex._get_tree_entries(tmp_path, gitignore=None)
-        assert [entry.name for entry in entries] == ["target.txt"]
+        assert [entry[0].name for entry in entries] == ["target.txt"]
 
 
 class TestCollectTree:
@@ -380,7 +380,7 @@ class TestCollectTree:
     def test_collect_tree_permission_denied(self, tmp_path):
         rows = []
         stats = {"directories": 0, "files": 0, "total_size": 0}
-        with patch.object(Path, "iterdir", side_effect=PermissionError):
+        with patch("os.scandir", side_effect=PermissionError):
             treex._collect_tree(tmp_path, "", stats, None, rows, True, False, True)
         assert rows == [("└── [permission denied]", None)]
         assert stats["directories"] == 0
@@ -562,6 +562,22 @@ class TestScanTree:
         )
         assert nodes == []
 
+    def test_relative_prefix_matches_nested_paths(self, tmp_path):
+        nested = tmp_path / "nested"
+        nested.mkdir()
+        (nested / "app.py").touch()
+        (nested / "notes.txt").touch()
+        nodes = treex._scan_tree(
+            nested,
+            None,
+            False,
+            False,
+            include_pattern="src/*.py",
+            relative_prefix="src",
+        )
+        assert [node["name"] for node in nodes] == ["app.py"]
+        assert nodes[0]["contains_match"] is True
+
 
 class TestPrintTreeStatistics:
     def test_counts_files_and_directories(self, tmp_path):
@@ -913,7 +929,7 @@ class TestPrintTree:
     def test_reports_permission_denied(self, tmp_path, capsys):
         directory = tmp_path / "restricted"
         directory.mkdir()
-        with patch.object(Path, "iterdir", side_effect=PermissionError):
+        with patch("os.scandir", side_effect=PermissionError):
             treex.print_tree(directory)
         assert "[permission denied]" in capsys.readouterr().out
 
@@ -1019,7 +1035,7 @@ class TestArgumentParsing:
             treex.parse_args([option])
         assert exc_info.value.code == 0
         output = capsys.readouterr().out
-        assert output.splitlines()[0] == "usage: treex [OPTIONS] [directory]"
+        assert output.splitlines()[0] == f"usage: {treex.PROG} [OPTIONS] [directory]"
 
     @pytest.mark.parametrize("option", ["-V", "--version"])
     def test_version(self, option, capsys):
@@ -1027,7 +1043,7 @@ class TestArgumentParsing:
             treex.parse_args([option])
         assert exc_info.value.code == 0
         output = capsys.readouterr().out
-        assert output.strip() == f"treex {treex.__version__}"
+        assert output.strip() == f"{treex.PROG} {treex.__version__}"
 
     @pytest.mark.parametrize("option", ["-a", "--all"])
     def test_all(self, option):
@@ -1164,7 +1180,7 @@ class TestArgumentParsing:
 class TestMain:
     def test_success(self, tmp_path, capsys):
         (tmp_path / "file.txt").write_text("hello", encoding="utf-8")
-        with patch.object(sys, "argv", ["treex", str(tmp_path)]):
+        with patch.object(sys, "argv", [treex.PROG, str(tmp_path)]):
             assert treex.main() == 0
         output = capsys.readouterr().out
         assert str(tmp_path) in output
@@ -1176,7 +1192,7 @@ class TestMain:
         directory = tmp_path / "dir"
         directory.mkdir()
         (tmp_path / "file.txt").write_text("hello", encoding="utf-8")
-        with patch.object(sys, "argv", ["treex", "--dirs", str(tmp_path)]):
+        with patch.object(sys, "argv", [treex.PROG, "--dirs", str(tmp_path)]):
             assert treex.main() == 0
         output = capsys.readouterr().out
         assert str(tmp_path) in output
@@ -1188,13 +1204,13 @@ class TestMain:
     def test_not_a_directory(self, tmp_path, capsys):
         path = tmp_path / "file.txt"
         path.write_text("hello", encoding="utf-8")
-        with patch.object(sys, "argv", ["treex", str(path)]):
+        with patch.object(sys, "argv", [treex.PROG, str(path)]):
             assert treex.main() == 1
         assert f"Not a directory: {path}" in capsys.readouterr().err
 
     def test_keyboard_interrupt(self, tmp_path, capsys):
         with (
-            patch.object(sys, "argv", ["treex", str(tmp_path)]),
+            patch.object(sys, "argv", [treex.PROG, str(tmp_path)]),
             patch.object(treex, "print_tree", side_effect=KeyboardInterrupt),
         ):
             assert treex.main() == 130