fix: v9.14.3 — CLI temporal anti-fallback second path

E2E variation testing revealed cli.py had a second fallback path at
the _has_temporal check (~line 5190). When no temporal collection
directory existed, the CLI printed "Falling back to space-only search"
and set time_range = None, silently executing a full regular semantic
search instead of returning nothing.

Fixed: replaced time_range = None fall-through with early return after
printing honest "Temporal index not available. No results returned."
message. Consistent with Messi Rule #2 (graceful failure over forced success).

Added test: test_cli_no_temporal_index_returns_immediately verifies
execute_temporal_query_with_fusion is never called when _has_temporal=False.

Also found (not fixed here): --chunk-type, --author, diff_type filters
are silently dropped in execute_temporal_query_with_fusion — pre-existing
gap not introduced by this work.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: jsbattig

CHANGELOG.md

@@ -5,11 +5,17 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## v9.14.3
+
+### Fixes
+
+- fix: Temporal CLI anti-fallback (second path) — discovered during E2E variation testing that `cli.py` had a second fallback at the `_has_temporal` check (line ~5190). When no temporal collection directory existed at all, the CLI printed "Falling back to space-only search (current code state only)" and set `time_range = None`, causing a full regular semantic search to execute silently. Fixed by replacing `time_range = None` fall-through with an early `return` after printing the honest "No results returned" message. Added test `test_cli_no_temporal_index_returns_immediately` that verifies `execute_temporal_query_with_fusion` is never called when the temporal index is absent.
+
 ## v9.14.2
 
 ### Fixes
 
-- fix: Temporal index anti-fallback — removed dishonest "Showing results from current code only" fallback messaging when a temporal index is unavailable. The old behavior silently returned a warning claiming to show current-code results; the corrected behavior returns an empty result set and an honest message stating "No results returned. Build the temporal index with 'cidx index --index-commits' to enable time-range queries." Applies to both MCP/REST (server) and CLI paths. Rule: graceful failure over forced success (Messi Rule #2).
+- fix: Temporal index anti-fallback — removed dishonest "Showing results from current code only" fallback messaging when a temporal index is unavailable in the server/MCP path (`semantic_query_manager.py`) and in the CLI warning display when `execute_temporal_query_with_fusion` returns empty results with a warning. The corrected behavior returns an empty result set and an honest message stating "No results returned. Build the temporal index with 'cidx index --index-commits' to enable time-range queries." Rule: graceful failure over forced success (Messi Rule #2).
 
 - fix: Temporal query `temporal_context` field mapping — `_execute_temporal_query` was reading legacy fields (`first_seen`, `last_seen`, `appearance_count`, `commits`) from `TemporalSearchResult.temporal_context` that no longer exist in the current diff-based implementation, causing all temporal_context fields to be `None` in query responses. Fixed by mapping the actual current fields: `commit_hash`, `commit_date`, `commit_message`, `author_name`, `commit_timestamp`, `diff_type`. Verified end-to-end with a real git repo, temporal index, and CLI query confirming all fields populated with real data.
 

src/code_indexer/__init__.py

@@ -6,5 +6,5 @@
 HNSW graph indexing (O(log N) complexity).
 """
 
-__version__ = "9.14.2"
+__version__ = "9.14.3"
 __author__ = "Seba Battig"

src/code_indexer/cli.py

@@ -5188,18 +5188,18 @@ def get_daemon_config(self):
             if d.is_dir()
         )
         if not _has_temporal:
-            console.print("[yellow]⚠️  Temporal index not found[/yellow]")
+            console.print("[yellow]⚠️  Temporal index not available[/yellow]")
             console.print()
-            console.print("Time-range filtering requires a temporal index.")
             console.print(
-                "Falling back to space-only search (current code state only)."
+                "Temporal index not available for this repository. No results returned."
             )
             console.print()
-            console.print("To enable temporal queries, build the temporal index:")
-            console.print("  [cyan]cidx index --index-commits[/cyan]")
-            console.print()
-            # Fall through to regular search without time-range filtering
-            time_range = None
+            console.print(
+                "Build the temporal index with "
+                "[cyan]cidx index --index-commits[/cyan] "
+                "to enable time-range queries."
+            )
+            return
         else:
             # Handle time_range="all" from --time-range-all flag
             if time_range == "all":

tests/unit/cli/test_temporal_warning_display.py

@@ -122,9 +122,58 @@ def _invoke_temporal_query(runner, temp_project, temporal_results):
         )
 
 
+@pytest.fixture
+def temp_project_no_temporal(tmp_path):
+    """Project directory WITHOUT any temporal collection — _has_temporal check fails."""
+    code_indexer_dir = tmp_path / ".code-indexer" / "index"
+    code_indexer_dir.mkdir(parents=True)
+    # No temporal collection directory created — simulates missing temporal index
+    return tmp_path
+
+
 class TestCLITemporalWarningDisplay:
     """Verify the CLI displays the warning from TemporalSearchResults."""
 
+    def test_cli_no_temporal_index_returns_immediately(
+        self, runner, temp_project_no_temporal
+    ):
+        """When no temporal collection directory exists, CLI returns immediately.
+
+        Must print honest "No results returned" message and must NOT invoke
+        execute_temporal_query_with_fusion at all (no fallback search).
+        """
+        config_manager_mock = _make_config_manager_mock(temp_project_no_temporal)
+
+        with patch(
+            "src.code_indexer.services.temporal.temporal_fusion_dispatch"
+            ".execute_temporal_query_with_fusion"
+        ) as mock_fusion:
+            result = runner.invoke(
+                query_command,
+                ["--time-range-all", "authentication"],
+                obj={
+                    "config_manager": config_manager_mock,
+                    "project_root": str(temp_project_no_temporal),
+                    "mode": "local",
+                    "standalone": True,
+                },
+                catch_exceptions=False,
+            )
+
+        assert result.exit_code == 0, (
+            f"Expected exit code 0, got {result.exit_code}. Output: {result.output!r}"
+        )
+        assert "No results returned" in result.output, (
+            f"CLI must display honest 'No results returned' message. Output: {result.output!r}"
+        )
+        assert "cidx index --index-commits" in result.output, (
+            f"CLI must mention the build command. Output: {result.output!r}"
+        )
+        assert not mock_fusion.called, (
+            "Fusion dispatch must NOT be called when temporal index is absent — "
+            "no fallback search allowed"
+        )
+
     def test_cli_displays_temporal_warning_when_no_index(self, runner, temp_project):
         """When fusion dispatch returns empty results with a warning, CLI must display it.