Skip to content

[codex] fix graph route ticker fallback - #3

Merged
sarthakagrawal927 merged 1 commit into
mainfrom
codex/graph-route-respects-ticker
Jun 13, 2026
Merged

[codex] fix graph route ticker fallback#3
sarthakagrawal927 merged 1 commit into
mainfrom
codex/graph-route-respects-ticker

Conversation

@sarthakagrawal927

Copy link
Copy Markdown
Member

What changed

  • Fixed the graph route so a ticker filter that matches no rows returns None instead of silently falling back to unrelated entities.
  • Added a regression test for the unmatched-ticker case.

Why

  • The previous ... or rows fallback ignored the caller's ticker constraint when the filtered set was empty.
  • That could produce theme summaries for the wrong company instead of correctly backing off.

Validation

  • uv run pytest -q tests/test_graph_route.py
  • uv run ruff check src/kb/query/graph_route.py tests/test_graph_route.py

@sarthakagrawal927
sarthakagrawal927 marked this pull request as ready for review June 13, 2026 05:30
@sarthakagrawal927
sarthakagrawal927 merged commit bf7760a into main Jun 13, 2026
0 of 2 checks passed
@sarthakagrawal927
sarthakagrawal927 deleted the codex/graph-route-respects-ticker branch June 13, 2026 05:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant