igntui 0.2.0__tar.gz → 0.3.1__tar.gz

{igntui-0.2.0 → igntui-0.3.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: igntui
-Version: 0.2.0
+Version: 0.3.1
 Summary: Interactive Terminal User Interface for GitIgnore template generation
 Keywords: gitignore,tui,terminal,git,templates,cli
 Author: Mohammad Abu Mattar

igntui-0.3.1/pyproject.toml

@@ -0,0 +1,148 @@
+[project]
+name = "igntui"
+version = "0.3.1"
+description = "Interactive Terminal User Interface for GitIgnore template generation"
+readme = "README.md"
+license = "GPL-3.0-or-later"
+license-files = ["LICENSE"]
+requires-python = ">=3.11"
+dependencies = [
+    "windows-curses>=2.4.1; sys_platform == 'win32'",
+    "pyfiglet>=1.0.2",
+    "tomli-w>=1.0.0",
+]
+keywords = [
+    "gitignore",
+    "tui",
+    "terminal",
+    "git",
+    "templates",
+    "cli",
+]
+classifiers = [
+    "Development Status :: 4 - Beta",
+    "Environment :: Console :: Curses",
+    "Intended Audience :: Developers",
+    "Operating System :: OS Independent",
+    "Programming Language :: Python :: 3",
+    "Programming Language :: Python :: 3.11",
+    "Programming Language :: Python :: 3.12",
+    "Programming Language :: Python :: 3.13",
+    "Topic :: Software Development :: Libraries :: Python Modules",
+    "Topic :: Software Development :: Version Control :: Git",
+    "Topic :: Terminals",
+    "Topic :: Utilities",
+]
+
+[[project.authors]]
+name = "Mohammad Abu Mattar"
+email = "mohammad.khaled@outlook.com"
+
+[[project.maintainers]]
+name = "Mohammad Abu Mattar"
+email = "mohammad.khaled@outlook.com"
+
+[project.urls]
+Homepage = "https://github.com/MKAbuMattar/igntui"
+Repository = "https://github.com/MKAbuMattar/igntui"
+Documentation = "https://github.com/MKAbuMattar/igntui/blob/main/README.md"
+Issues = "https://github.com/MKAbuMattar/igntui/issues"
+
+[project.scripts]
+igntui = "igntui.main:cli_main"
+gitignore-tui = "igntui.main:tui_main"
+
+[project.optional-dependencies]
+dev = [
+    "pytest>=7.0.0",
+    "pytest-cov>=4.0.0",
+    "pytest-mock>=3.10.0",
+    "ruff>=0.1.0",
+]
+test = [
+    "pytest>=7.0.0",
+    "pytest-cov>=4.0.0",
+    "pytest-mock>=3.10.0",
+    "freezegun>=1.4.0",
+]
+windows = ["windows-curses>=2.4.1; sys_platform == 'win32'"]
+
+[build-system]
+requires = ["uv_build>=0.9.5,<0.13.0"]
+build-backend = "uv_build"
+
+[tool.ruff]
+line-length = 100
+target-version = "py311"
+
+[tool.ruff.lint]
+select = [
+    "E",
+    "W",
+    "F",
+    "I",
+    "B",
+    "C4",
+    "UP",
+]
+ignore = [
+    "E501",
+    "B008",
+    "C901",
+    "E402",
+    "W291",
+    "W293",
+    "C408",
+    "C414",
+]
+
+[tool.ruff.lint.per-file-ignores]
+"__init__.py" = ["F401"]
+
+[tool.mypy]
+python_version = "3.11"
+warn_return_any = true
+warn_unused_configs = true
+disallow_untyped_defs = false
+ignore_missing_imports = true
+
+[[tool.mypy.overrides]]
+module = [
+    "curses",
+    "curses.*",
+    "windows_curses",
+    "windows_curses.*",
+]
+ignore_missing_imports = true
+ignore_errors = true
+
+[tool.ty.environment]
+python-version = "3.11"
+
+[tool.ty.src]
+include = ["src/igntui"]
+
+[tool.pytest.ini_options]
+testpaths = ["tests"]
+python_files = ["test_*.py"]
+addopts = "-v --strict-markers"
+markers = ["integration: tests that hit the network or filesystem extensively"]
+
+[tool.coverage.run]
+source = ["src/igntui"]
+omit = [
+    "*/tests/*",
+    "*/__main__.py",
+    "*/tui/curses_setup.py",
+    "*/tui/renderer.py",
+    "*/ui/*",
+]
+
+[tool.coverage.report]
+exclude_lines = [
+    "pragma: no cover",
+    "raise NotImplementedError",
+    "if __name__ == .__main__.:",
+    "if TYPE_CHECKING:",
+]
+show_missing = true

igntui-0.2.0/pyproject.toml → igntui-0.3.1/pyproject.toml.orig

@@ -1,6 +1,6 @@
 [project]
 name = "igntui"
-version = "0.2.0"
+version = "0.3.1"
 description = "Interactive Terminal User Interface for GitIgnore template generation"
 readme = "README.md"
 license = "GPL-3.0-or-later"

{igntui-0.2.0 → igntui-0.3.1}/src/igntui/__init__.py

@@ -5,7 +5,7 @@ import platform
 import sys
 from collections.abc import Callable
 
-__version__ = "0.2.0"
+__version__ = "0.3.1"
 __author__ = "Mohammad Abu Mattar"
 __email__ = "mohammad.khaled@outlook.com"
 __description__ = """A professional TUI application for generating .gitignore files from templates

{igntui-0.2.0 → igntui-0.3.1}/src/igntui/cli/commands/cache_cmd.py

@@ -17,6 +17,11 @@ class CacheCommand(CLICommand):
 
         clear_parser = subparsers.add_parser("clear", help="Clear cache")
         clear_parser.add_argument("--force", action="store_true", help="Skip confirmation prompt")
+        clear_parser.add_argument(
+            "--expired",
+            action="store_true",
+            help="Remove only entries past their TTL, keeping everything still valid",
+        )
 
         subparsers.add_parser("stats", help="Show cache statistics")
         subparsers.add_parser("info", help="Show cache information")
@@ -33,6 +38,8 @@ class CacheCommand(CLICommand):
             elif args.cache_action == "stats":
                 return self._show_stats(cache)
             elif args.cache_action == "clear":
+                if getattr(args, "expired", False):
+                    return self._clear_expired(cache)
                 return self._clear_cache(cache, args.force)
             else:
                 print(f"Unknown cache action: {args.cache_action}")
@@ -85,6 +92,21 @@ class CacheCommand(CLICommand):
 
         return 0
 
+    def _clear_expired(self, cache: "CacheManager") -> int:
+        """Sweep only what is past its TTL.
+
+        Nothing prunes expired files on its own any more: reading every entry at
+        startup to find them cost every command ~19 ms over a realistic cache.
+        Re-requesting a template set overwrites its stale entry in place, so this
+        is for sets that were cached once and never asked for again.
+        """
+        removed = cache.cleanup_expired()
+        if removed:
+            print(f"Removed {removed} expired {'entry' if removed == 1 else 'entries'}")
+        else:
+            print("No expired entries")
+        return 0
+
     def _clear_cache(self, cache: "CacheManager", force: bool = False) -> int:
         if not force:
             response = input("Clear cache? This will remove all cached data. (y/N): ")

{igntui-0.2.0 → igntui-0.3.1}/src/igntui/cli/commands/completion_cmd.py

@@ -34,7 +34,7 @@ _igntui_complete() {
         tui)       COMPREPLY=( $(compgen -W "--no-splash" -- "$cur") ); return ;;
         list)      COMPREPLY=( $(compgen -W "--filter --count" -- "$cur") ); return ;;
         generate)  COMPREPLY=( $(compgen -W "--output --append --force --dry-run --no-sidecar" -- "$cur") ); return ;;
-        cache)     COMPREPLY=( $(compgen -W "info stats clear --force" -- "$cur") ); return ;;
+        cache)     COMPREPLY=( $(compgen -W "info stats clear --force --expired" -- "$cur") ); return ;;
         test)      COMPREPLY=( $(compgen -W "--timeout" -- "$cur") ); return ;;
         completion) COMPREPLY=( $(compgen -W "bash zsh fish" -- "$cur") ); return ;;
         "")        COMPREPLY=( $(compgen -W "$subcommands $global_flags" -- "$cur") ); return ;;
@@ -73,7 +73,10 @@ _igntui() {
                     '--force[overwrite without prompt]' \\
                     '--dry-run[print without writing]' \\
                     '--no-sidecar[skip igntui.cfg.toml]' ;;
-                cache)     _values 'cache action' info stats clear ;;
+                cache)     _arguments \\
+                    '1:action:(info stats clear)' \\
+                    '--force[skip confirmation]' \\
+                    '--expired[only entries past their TTL]' ;;
                 test)      _arguments '--timeout[seconds]:seconds:' ;;
                 completion) _values 'shell' bash zsh fish ;;
             esac
@@ -110,6 +113,8 @@ complete -c igntui -n "__fish_seen_subcommand_from generate" -l dry-run -d "Prin
 complete -c igntui -n "__fish_seen_subcommand_from generate" -l no-sidecar -d "Skip sidecar"
 complete -c igntui -n "__fish_seen_subcommand_from generate" -l force -d "Overwrite without prompt"
 complete -c igntui -n "__fish_seen_subcommand_from cache" -a "info stats clear"
+complete -c igntui -n "__fish_seen_subcommand_from cache" -l force -d "Skip confirmation"
+complete -c igntui -n "__fish_seen_subcommand_from cache" -l expired -d "Only expired entries"
 complete -c igntui -n "__fish_seen_subcommand_from completion" -a "bash zsh fish"
 """
 

{igntui-0.2.0 → igntui-0.3.1}/src/igntui/cli/parser.py

@@ -117,6 +117,11 @@ def create_command_parser(
 
     cache_clear_parser = cache_subparsers.add_parser("clear", help="Clear all cache")
     cache_clear_parser.add_argument("--force", action="store_true", help="Skip confirmation prompt")
+    cache_clear_parser.add_argument(
+        "--expired",
+        action="store_true",
+        help="Remove only entries past their TTL, keeping everything still valid",
+    )
     cache_subparsers.add_parser("stats", help="Show cache statistics")
     cache_subparsers.add_parser("info", help="Show cache information")
 

{igntui-0.2.0 → igntui-0.3.1}/src/igntui/core/cache.py

@@ -4,7 +4,9 @@
 import hashlib
 import json
 import logging
+import os
 import re
+import tempfile
 import time
 from dataclasses import asdict, dataclass
 from pathlib import Path
@@ -49,8 +51,19 @@ class CacheManager:
 
         self.cache_dir.mkdir(parents=True, exist_ok=True)
         self._purge_legacy_content_keys()
-        self._load_persistent_cache()
 
+        # Deliberately not reading the cache into memory here. `get()` already
+        # falls back to disk and promotes what it finds, so an eager load bought
+        # nothing except latency — and every command pays it, because
+        # GitIgnoreAPI constructs a manager on the way to doing anything. Over a
+        # realistic cache (300 content blobs, ~3.5 MiB) it measured ~19 ms of
+        # open + json.loads before the process could start its actual work.
+        #
+        # The eager pass also deleted expired files. Losing that is nearly free:
+        # a content key is a hash of its template set, so re-requesting the same
+        # set overwrites the stale entry in place, and `get()` deletes any
+        # expired entry it touches. Only sets never asked for again linger, and
+        # `igntui cache clear --expired` sweeps those.
         logger.debug("Initialized cache manager with directory: %s", self.cache_dir)
 
     def _purge_legacy_content_keys(self) -> None:
@@ -196,25 +209,6 @@ class CacheManager:
                 **self._stats,
             }
 
-    def _load_persistent_cache(self) -> None:
-        try:
-            loaded_count = 0
-            for cache_file in self.cache_dir.glob("*.cache"):
-                key = cache_file.stem
-                entry = self._load_disk_cache(key)
-
-                if entry and not entry.is_expired():
-                    self._memory_cache[key] = entry
-                    loaded_count += 1
-                elif entry:
-                    cache_file.unlink()
-
-            if loaded_count > 0:
-                logger.info("Loaded %d cache entries from disk", loaded_count)
-
-        except Exception as e:
-            logger.warning("Failed to load persistent cache: %s", e)
-
     def _load_disk_cache(self, key: str) -> CacheEntry | None:
         cache_file = self.cache_dir / f"{key}.cache"
 
@@ -236,16 +230,38 @@ class CacheManager:
         return None
 
     def _save_disk_cache(self, key: str, entry: CacheEntry) -> None:
+        """Write an entry to disk atomically.
+
+        Writing in place meant a crash, a full disk, or two processes saving the
+        same key could leave truncated JSON behind. That was survivable — a read
+        catches the JSONDecodeError and unlinks the file — but the entry was lost
+        and the failure was silent. Writing a temp file in the same directory and
+        renaming makes the swap atomic on POSIX and Windows: a reader sees either
+        the old entry or the new one, never half of either.
+        """
         cache_file = self.cache_dir / f"{key}.cache"
+        tmp_path: str | None = None
 
         try:
-            with open(cache_file, "w", encoding="utf-8") as f:
+            # Same directory, so os.replace is a rename rather than a cross-device copy.
+            fd, tmp_path = tempfile.mkstemp(dir=self.cache_dir, suffix=".tmp")
+            with os.fdopen(fd, "w", encoding="utf-8") as f:
                 json.dump(asdict(entry), f, separators=(",", ":"))
+            os.replace(tmp_path, cache_file)
+            tmp_path = None
 
             self._stats["disk_writes"] += 1
 
         except (OSError, TypeError) as e:
             logger.warning("Failed to save cache file %s: %s", cache_file, e)
+        finally:
+            # A serialisation failure leaves the temp file behind; the previous
+            # good entry is still in place, which is the point.
+            if tmp_path is not None:
+                try:
+                    os.unlink(tmp_path)
+                except OSError:
+                    pass
 
     def _delete_disk_cache(self, key: str) -> bool:
         cache_file = self.cache_dir / f"{key}.cache"