Skip to content

Add connection-scoped query resolution - #21

Merged
fsistemas merged 1 commit into
masterfrom
linear/fra-246-connection-queries-core
Jun 23, 2026
Merged

Add connection-scoped query resolution#21
fsistemas merged 1 commit into
masterfrom
linear/fra-246-connection-queries-core

Conversation

@fsistemas

@fsistemas fsistemas commented Jun 23, 2026

Copy link
Copy Markdown
Owner

Summary

Adds core support for connection_queries config parsing and query resolution.

  • Adds validated top-level connection_queries config support.
  • Resolves named queries in the agreed order:
    1. connection_queries.<connection>.<query_name>
    2. queries.<query_name>
    3. existing raw SQL / @file.sql handling
  • Preserves existing global-only config behavior.
  • Keeps CLI JSON-on-stderr error envelope for malformed scoped-query config.

Verification

  • .venv/bin/pytest tests/test_sql2json.py::test_run_query_by_name_uses_connection_scoped_query tests/test_sql2json.py::test_run_query_by_name_falls_back_to_global_query tests/test_sql2json.py::test_run_query_by_name_scoped_query_takes_precedence_over_global tests/test_sql2json.py::test_run_query_by_name_rejects_malformed_connection_queries tests/test_sql2json.py::test_run_query_by_name_rejects_connection_queries_for_unknown_connection tests/test_cli.py::TestErrorOutput::test_malformed_connection_queries_stderr_is_json — 6 passed
  • .venv/bin/pytest — 133 passed, 12 deselected
  • .venv/bin/black --check sql2json tests — 16 files would be left unchanged
  • .venv/bin/flake8 sql2json tests — passed
  • .venv/bin/mypy sql2json — Success: no issues found in 5 source files

@fsistemas fsistemas changed the title FRA-246 add connection-scoped query resolution Add connection-scoped query resolution Jun 23, 2026
@fsistemas
fsistemas merged commit 94fc505 into master Jun 23, 2026
6 checks passed
@fsistemas
fsistemas deleted the linear/fra-246-connection-queries-core branch June 23, 2026 00:40
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