Commit History
-
/fix surface catalog foreign keys to the planner (FK-backed joins)
Rifqi Hafizuddin +1: Claude Opus 4.8The slow-path planner's CatalogSummary omitted foreign_keys, but the IR validator only allows FK-backed joins. So join-requiring questions ("which product made the most revenue") made the LLM guess joins that were rejected 3x → slow path failed. The prompt already promised FKs; the rendered catalog just never included them. - inputs.py: CatalogSummary carries foreign_keys + renders FK lines with the exact join ids (target_table_id/left_column_id/right_column_id) to copy verbatim. - planner.md: join ONLY on a listed FK; copy ids; don't invent a join (single table or say the data isn't linked). - render.py (single-query planner): FK lines now include the join ids too (were name-only) — keeps both planners consistent. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>446c575
-
feat/Catalog Retrieval System (#1)
- initial commit (8557a2045ca2c0873e07efdf6227064b83e4a1e5) - initial structure (efc0c0a8b2444014cbe74f04617c3c974d9915be) - [KM-553] initialize shared contract (f31f6731692b0199d8cb5c5d29dffa466d6d4ca0) - [KM-553] migrate retrieval layer and remove obsolete rag/tools modules (df220ea47c96eb8a1efa07c6eebef4604fc308d6) - [NOTICKET] catalog enricher + structured pipeline + on_db_registered (77ea82d29d86400c5ed30ac154be8df6a7e897eb) - Merge commit 'refs/pr/1' of https://huggingface.co/spaces/DataEyond/Agentic-Service-Data-Eyond-Catalog into pr/1 (0ae8717c7efce2cda14fe4b8caecd774459b1f6e) - [NOTICKET] SqlCompiler (Postgres) + DbExecutor (212dad360714508c1aa607995039e6818fa40f78) - Merge commit 'refs/pr/1' of https://huggingface.co/spaces/DataEyond/Agentic-Service-Data-Eyond-Catalog into pr/1 (faf8a4f6d7e06ccf1ce84c2e7f4990c79f27f843) - TabularIntrospector + on_tabular_uploaded trigger + 31 tests (ca425208969b72a20ed76f86ef0faa1a0e912339) - [KM 556] add columns field to QueryResult + update…6bff5d9
