fix: restore evidence_filter logic for non-history facet filtering

Coding-Dev-Tools · Coding-Dev-Tools · commit dcb731827f7b · 2026-08-10T19:23:53.000-04:00
The previous commit introduced a dependency on prune_entities in the
evidence_filter assignment that broke live-only evidence filtering
when memory_types/time facets were requested without include_history.
Restore the original 'not include_history' logic; the prune_entities
override for history mode is handled separately below.

Fixes test_graph_scene_filters_supporting_memory_type_and_time_window.
diff --git a/engraphis/service.py b/engraphis/service.py
@@ -7797,7 +7797,7 @@ def _graph_scene_rows_unlocked(self, *, workspace: str, repo: Optional[str] = No
         prune_entities = bool(
             clean_memory_types or lower_time is not None or upper_time is not None
         )
-        evidence_filter = not include_history and not prune_entities
+        evidence_filter = not include_history
         allow_supportless = not (
             clean_memory_types or lower_time is not None or upper_time is not None
         )

Original file line number	Diff line number	Diff line change
`@@ -7797,7 +7797,7 @@ def _graph_scene_rows_unlocked(self, *, workspace: str, repo: Optional[str] = No`
`7797`	`7797`	`prune_entities = bool(`
`7798`	`7798`	`clean_memory_types or lower_time is not None or upper_time is not None`
`7799`	`7799`	`)`
`7800`		`- evidence_filter = not include_history and not prune_entities`
	`7800`	`+ evidence_filter = not include_history`
`7801`	`7801`	`allow_supportless = not (`
`7802`	`7802`	`clean_memory_types or lower_time is not None or upper_time is not None`
`7803`	`7803`	`)`