Skip to content

Conversation

@cyc60
Copy link
Contributor

@cyc60 cyc60 commented Dec 12, 2025

No description provided.

Copilot AI review requested due to automatic review settings December 12, 2025 13:14
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the codebase to use a new graph client by upgrading the sw-utils dependency from v0.10.6 to v0.10.8 and integrating the graph client setup and teardown into the application lifecycle.

Key Changes:

  • Integrated graph client initialization and cleanup alongside the existing execution client
  • Refactored client setup into a unified setup_clients() function
  • Moved the try-finally block in main to properly wrap all client setup and teardown

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
src/main.py Updated imports and restructured main function to use the new unified client setup
src/common/clients.py Added setup_clients() function to initialize both execution and graph clients, and updated close_clients() to disconnect the graph client
pyproject.toml Upgraded sw-utils dependency from v0.10.6 to v0.10.8 to support the new graph client

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cyc60 cyc60 merged commit 923b633 into master Dec 15, 2025
5 checks passed
@cyc60 cyc60 deleted the graph-session branch December 15, 2025 09:24
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.

3 participants