Repository navigation
feat(ai): add API Route graph-query provider - #1774
Conversation
Signed-off-by: dennyho0917 <dennyho0917@gmail.com>
|
@DennyHo0917 is attempting to deploy a commit to the shashankss1205's projects Team on Vercel. A member of the Team first needs to authorize it. |
Shashankss1205
left a comment
There was a problem hiding this comment.
Reviewed the complete diff and ran all 55 API tests under MCP 2.3; they pass. This is an optional key-selected provider, preserves existing provider precedence, reuses the same read-only query guard/request path, supports endpoint/model overrides, and documents what is sent. No live credential was needed for review. CI checks have not been created for this external head yet; approval is for the reviewed source and local coverage, and merge will wait for the repository's full checks.
|
All 55 API tests pass locally and the source review is approved. The fork's workflow runs are action_required. Published maintainer integration #1776 with the identical reviewed tree and your original commit retained as a parent, to run the complete repository checks. No workflow/permission changes; merge will wait for those checks. |
|
Merged via #1776 after all maintainer-head checks passed, including Python 3.12–3.14 unit/integration jobs, platform E2E, strict parity and packaging/security. Your original contributor commit is preserved in main. Local API validation: 55 passed. Thank you for the focused contribution and affiliation disclosure. |
Summary
Add API Route as an optional OpenAI-compatible provider for the visualizer's natural-language graph queries, following the Cheaper Inference integration in #1756.
API_ROUTE_API_KEYin both the query endpoint's credential check andcall_llm().https://global.api-route.com/v1andgpt-6.1-sol; allow overrides withAPI_ROUTE_API_BASEandAPI_ROUTE_MODEL.Usage
Use only the API Route key in that process to select this provider. Questions, schema and query results are sent to the selected provider, as documented in the guide.
Validation
python -m pytest tests/unit/api/ -q: 55 passed, including the existing AI-query tests and 10 new cases.gpt-6.1-sol. The actualviz.server.call_llm()path using the default endpoint/model returnedOK.git diff --checkpasses.tests/unit/ tests/integration/, with UTF-8 enabled on Windows): 389 passed, then stopped attest_startup_failure_for_one_path_does_not_disable_another_path. The same case fails with the parent source because the embedded FalkorDB driver explicitly does not support Windows. The complete fast suite is therefore not claimed green.Validation environment: Windows, Python 3.12. No frontend or database-driver changes.
Disclosure: I’m affiliated with API Route. This contribution was prepared with AI assistance.
Remote checks
GitHub Actions for this external contribution await maintainer approval. Vercel also requires project-team authorization for the preview deployment; it has not run the deployment.