Skip to content

Feature/keeper concept retrieval#29

Merged
rkboyce merged 6 commits intomainfrom
feature/keeper-concept-retrieval
Apr 14, 2026
Merged

Feature/keeper concept retrieval#29
rkboyce merged 6 commits intomainfrom
feature/keeper-concept-retrieval

Conversation

@rkboyce
Copy link
Copy Markdown
Collaborator

@rkboyce rkboyce commented Apr 14, 2026

No description provided.

rkboyce added 6 commits April 13, 2026 08:05
I added file-backed prompt/config assets under [mcp_server/prompts/keeper_concept_sets](/ai-agent/HadesProject/OHDSI-Study-Agent/mcp_server/prompts/keeper_concept_sets) and a new MCP module at [keeper_concept_sets.py](/ai-agent/HadesProject/OHDSI-Study-Agent/mcp_server/study_agent_mcp/tools/keeper_concept_sets.py). That module registers:
- `keeper_concept_set_bundle`
- `vocab_search_standard`
- `phoebe_related_concepts`
- `vocab_filter_standard_concepts`
- `vocab_remove_descendants`
- `vocab_add_nonchildren`
- `vocab_fetch_concepts`

For this slice, the deterministic helpers are implemented and testable now. Provider-backed search/Phoebe calls are intentionally explicit stubs: they return `*_provider_unconfigured` or `*_provider_not_implemented` until we wire real Hecate / DB / generic search backends.

I also updated MCP registration and added tests in [test_keeper_concept_sets_tools.py](/ai-agent/HadesProject/OHDSI-Study-Agent/tests/test_keeper_concept_sets_tools.py). Verification passed with:
```bash
uv run pytest tests/test_keeper_concept_sets_tools.py tests/test_mcp_tools_registry.py tests/test_keeper_interface_specs.py tests/test_mcp_prompt_bundle.py
```
… an ACP flow for keeper_concept_sets_generate`
…r work to Keeper 2.0 but using services. A full test run demonstrates this.
@rkboyce rkboyce merged commit ea36d29 into main Apr 14, 2026
2 checks passed
@rkboyce rkboyce deleted the feature/keeper-concept-retrieval branch April 14, 2026 15:34
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