Skip to content

feat: unified LLM/Cypher hardening (timeouts, provider fallback, tests) - #45

Open
omakszewska wants to merge 6 commits into
Solvro:mainfrom
omakszewska:fix/llm-cypher-hardening-unified
Open

feat: unified LLM/Cypher hardening (timeouts, provider fallback, tests)#45
omakszewska wants to merge 6 commits into
Solvro:mainfrom
omakszewska:fix/llm-cypher-hardening-unified

Conversation

@omakszewska

Copy link
Copy Markdown
Contributor

Unified LLM/Cypher hardening (replaces #39 and #40).

  • Cypher: read-only validation before Neo4j (block writes, strip fences, enforce LIMIT)
  • Timeouts: configurable LLM + graph timeouts (env/yaml); wired into MCP RAG, MCP client, and ToPWR API final-answer LLM
  • Fallback: OpenAI → DeepSeek → Google via LangChain with_fallbacks on transient errors
  • Guardrails: JSON decisions (generate / end), fail-closed to end
  • Tests: unit tests for Cypher guardrails and provider fallback / exception routing
    No tenacity — resilience is provider switch, not retrying the same provider.

Closes #28

@omakszewska

Copy link
Copy Markdown
Contributor Author

@Dominik-Galus can you please review this PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

LLM & Cypher Hardening

1 participant