diffcontext 0.5.5__tar.gz → 0.5.6__tar.gz

{diffcontext-0.5.5 → diffcontext-0.5.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: diffcontext
-Version: 0.5.5
+Version: 0.5.6
 Summary: Static-analysis-powered repository context compiler for LLMs
 Author-email: Trakshan Mishra <trakshanmishra477@gmail.com>
 License: MIT

{diffcontext-0.5.5 → diffcontext-0.5.6}/diffcontext/__init__.py

@@ -32,7 +32,7 @@ Usage as a library:
     print(ctx.reduction_pct)    # how much code was filtered out
 """
 
-__version__ = "0.5.5"
+__version__ = "0.5.6"
 
 # Public, semver-covered API. Everything not listed here (graph_builder,
 # resolver, symbols, scanner, parser internals) is importable but carries no

{diffcontext-0.5.5 → diffcontext-0.5.6}/diffcontext/cache.py

@@ -74,18 +74,24 @@ def _resolve_cache_path(
     if cache_dir:
         try:
             os.makedirs(cache_dir, exist_ok=True)
+            return os.path.join(cache_dir, ".diffcontext_cache.db")
         except OSError:
-            pass
-        return os.path.join(cache_dir, ".diffcontext_cache.db")
+            logger.warning(
+                "cache: --cache-dir %s is not writable; falling through cascade",
+                cache_dir,
+            )
 
     # 2. Env var override
     env_cache_dir = os.environ.get("DIFFCONTEXT_CACHE_DIR")
     if env_cache_dir:
         try:
             os.makedirs(env_cache_dir, exist_ok=True)
+            return os.path.join(env_cache_dir, ".diffcontext_cache.db")
         except OSError:
-            pass
-        return os.path.join(env_cache_dir, ".diffcontext_cache.db")
+            logger.warning(
+                "cache: DIFFCONTEXT_CACHE_DIR=%s is not writable; falling through cascade",
+                env_cache_dir,
+            )
 
     # 3. In-repo path (current behaviour, keeps cache locality)
     if repo_path:

{diffcontext-0.5.5 → diffcontext-0.5.6}/diffcontext.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: diffcontext
-Version: 0.5.5
+Version: 0.5.6
 Summary: Static-analysis-powered repository context compiler for LLMs
 Author-email: Trakshan Mishra <trakshanmishra477@gmail.com>
 License: MIT

{diffcontext-0.5.5 → diffcontext-0.5.6}/tests/test_cache_fallback.py

@@ -119,3 +119,32 @@ class TestReadOnlyRepoIndexing:
             os.chmod(root, 0o755)
             for f in files:
                 os.chmod(os.path.join(root, f), 0o644)
+
+    def test_unwritable_env_var_falls_through(self, tmp_path, monkeypatch):
+        """DIFFCONTEXT_CACHE_DIR pointing to an unwritable path must NOT crash —
+        the cascade continues to tiers 3-5 instead of returning the bad path.
+
+        Previously, tiers 1/2 swallowed the OSError from makedirs but returned
+        the unusable path anyway, short-circuiting the cascade. This test pins
+        the fix: an explicit override that can't be honoured warns and degrades,
+        not crashes.
+        """
+        monkeypatch.setenv("DIFFCONTEXT_CACHE_DIR", "/nonexistent/unwritable/path")
+        monkeypatch.setenv("XDG_CACHE_HOME", "/also/nonexistent")
+
+        idx = index_repository(SIMPLE)
+        assert len(idx.symbols) > 0, (
+            "unwritable DIFFCONTEXT_CACHE_DIR must fall through to :memory:, "
+            "not crash with OperationalError"
+        )
+
+    def test_unwritable_cache_dir_flag_falls_through(self, tmp_path, monkeypatch):
+        """--cache-dir pointing to an unwritable path must NOT crash either."""
+        monkeypatch.delenv("DIFFCONTEXT_CACHE_DIR", raising=False)
+        monkeypatch.setenv("XDG_CACHE_HOME", "/also/nonexistent")
+
+        idx = index_repository(SIMPLE, cache_dir="/nonexistent/unwritable/path")
+        assert len(idx.symbols) > 0, (
+            "unwritable --cache-dir must fall through to :memory:, "
+            "not crash with OperationalError"
+        )