Skip to content

Conversation

@ashenfad
Copy link
Owner

@ashenfad ashenfad commented Jan 2, 2026

No description provided.

@ashenfad ashenfad requested a review from Copilot January 2, 2026 04:31
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 removes intermediate snapshot functionality from both synchronous and asynchronous agent loops, deferring snapshots to only occur upon task completion. The change eliminates unsaved state warnings that were previously generated during loop iteration and at final processing.

Key Changes:

  • Removed intermediate snapshot logic and associated warning generation from loop iterations
  • Eliminated final snapshot checks after loop completion
  • Cleaned up unused imports (create_unsaved_warning, is_live_root)

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
agex/agent/loop/sync_loop.py Removed finally block with intermediate snapshots and final snapshot logic; cleaned up unused imports
agex/agent/loop/async_loop.py Removed async-compatible finally block with intermediate snapshots and final snapshot logic; cleaned up unused imports

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

@ashenfad ashenfad merged commit 4f9f06f into main Jan 2, 2026
7 checks passed
@ashenfad ashenfad deleted the nxt branch January 2, 2026 04:34
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.

2 participants