Skip to content

Dev - #84

Merged
dcloud347 merged 7 commits into
mainfrom
dev
Jul 4, 2025
Merged

Dev#84
dcloud347 merged 7 commits into
mainfrom
dev

Conversation

@dcloud347

Copy link
Copy Markdown
Contributor

This pull request introduces updates to improve code maintainability, enhance error logging, and align configurations with environment variables. Key changes include updates to environment variable names, enhancements in error handling, and refactoring of Neo4j-related methods.

Configuration Updates:

  • Updated environment variable names in README.md to align with the new naming convention, such as replacing PROMETHEUS_OPENAI_API_KEY with PROMETHEUS_OPENAI_FORMAT_API_KEY and adding PROMETHEUS_OPENAI_FORMAT_BASE_URL for LLMs.
  • Modified test_model_response in prometheus/script/test_llm_service.py to use dynamic settings for temperature and max_output_tokens instead of hardcoded values.

Error Handling Enhancements:

  • Enhanced logging in context_retrieval_subgraph_node.py to include exception type and stack trace for better debugging during context retrieval failures.
  • Added the traceback module import to support detailed error logging.

Code Refactoring:

  • Refactored the write_parent_of_edges method in knowledge_graph_handler.py to simplify the query logic and improve transaction handling by batching and using a session-based approach.
  • Removed unused imports and redundant code from knowledge_graph_handler.py to improve code cleanliness. [1] [2]

Type Annotations:

  • Updated MetadataNode in graph_types.py to allow https_url and commit_id to be None, reflecting optional values.

Functional Improvements:

  • Added a step in upload_github_repository in service_coordinator.py to clear the existing knowledge graph and repository state before cloning a new repository.

@dcloud347
dcloud347 merged commit 39d93fe into main Jul 4, 2025
2 checks passed
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