fix: Handle missing provider configurations with defaults
- Fixed 'Neo4j provider configuration not found' error in CI - Create default provider configurations when not explicitly provided - Both Neo4j and FalkorDB now fall back to sensible defaults 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
parent
694ea46f66
commit
1f61587279
1 changed files with 16 additions and 6 deletions
|
|
@ -254,9 +254,14 @@ class DatabaseDriverFactory:
|
||||||
|
|
||||||
match provider:
|
match provider:
|
||||||
case 'neo4j':
|
case 'neo4j':
|
||||||
if not config.providers.neo4j:
|
# Use Neo4j config if provided, otherwise use defaults
|
||||||
raise ValueError('Neo4j provider configuration not found')
|
if config.providers.neo4j:
|
||||||
neo4j_config = config.providers.neo4j
|
neo4j_config = config.providers.neo4j
|
||||||
|
else:
|
||||||
|
# Create default Neo4j configuration
|
||||||
|
from config.schema import Neo4jProviderConfig
|
||||||
|
neo4j_config = Neo4jProviderConfig()
|
||||||
|
|
||||||
return {
|
return {
|
||||||
'uri': neo4j_config.uri,
|
'uri': neo4j_config.uri,
|
||||||
'user': neo4j_config.username,
|
'user': neo4j_config.username,
|
||||||
|
|
@ -270,10 +275,15 @@ class DatabaseDriverFactory:
|
||||||
raise ValueError(
|
raise ValueError(
|
||||||
'FalkorDB driver not available in current graphiti-core version'
|
'FalkorDB driver not available in current graphiti-core version'
|
||||||
)
|
)
|
||||||
if not config.providers.falkordb:
|
|
||||||
raise ValueError('FalkorDB provider configuration not found')
|
# Use FalkorDB config if provided, otherwise use defaults
|
||||||
|
if config.providers.falkordb:
|
||||||
|
falkor_config = config.providers.falkordb
|
||||||
|
else:
|
||||||
|
# Create default FalkorDB configuration
|
||||||
|
from config.schema import FalkorDBProviderConfig
|
||||||
|
falkor_config = FalkorDBProviderConfig()
|
||||||
|
|
||||||
falkor_config = config.providers.falkordb
|
|
||||||
return {
|
return {
|
||||||
'driver': 'falkordb',
|
'driver': 'falkordb',
|
||||||
'uri': falkor_config.uri,
|
'uri': falkor_config.uri,
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue