Skip to content

[codex] Allow CLI queries of concept aliases#205

Merged
yibeichan merged 1 commit into
mainfrom
codex/issue-201-concept-aliases-cli-query
Jun 11, 2026
Merged

[codex] Allow CLI queries of concept aliases#205
yibeichan merged 1 commit into
mainfrom
codex/issue-201-concept-aliases-cli-query

Conversation

@yibeichan

Copy link
Copy Markdown
Owner

Summary

  • add concept_aliases to the CLI raw-query table allowlist
  • document that the allowlist should stay aligned with schema.sql queryable tables
  • add a regression test that selects alias rows through raw_query

Root cause

The MCP layer can write concept_aliases, but the CLI raw query validator only allowed docs, concepts, edges, docs_fts, and paper_metadata. Legitimate diagnostic queries against concept_aliases were rejected before reaching SQLite.

Validation

  • cd cli && uv run --with pytest --with . python -m pytest tests/test_sqlite_query.py
  • cd cli && uv run --with pytest --with . python -m pytest tests/

Closes #201

@yibeichan yibeichan marked this pull request as ready for review June 11, 2026 20:07
@yibeichan yibeichan merged commit ef5610d into main Jun 11, 2026
6 checks passed
@yibeichan yibeichan deleted the codex/issue-201-concept-aliases-cli-query branch June 11, 2026 23:02
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.

bug: concept_aliases missing from ALLOWED_TABLES in CLI — table is unqueryable via raw_query

1 participant