Skip to content

itdove/devaiflow#663: bug: Handle ls-remote timeouts when reopening imported sessions - #664

Merged
itdove merged 2 commits into
mainfrom
663
Nov 4, 2026
Merged

itdove merged 2 commits into
mainfrom
663

Conversation

@itdove

@itdove itdove commented Nov 3, 2026

Copy link
Copy Markdown
Owner

Description

#663: bug: Handle ls-remote timeouts when reopening imported sessions

Changes in this branch:

  • fix: handle remote branch query timeouts

Related issue (if applicable)

Closes #663 (issue)

Type of change

  • Bug fix
  • New feature
  • Security fix or detection change
  • Documentation or workflow change
  • Refactoring or maintenance

Security impact

  • This change has no security impact.
  • This change affects security behavior; describe the threat, protection, and user-visible behavior below.

Security behavior is implicated by the session context; review the threat and user-visible impact before merging.

Testing

  • Tests added or updated where needed.
  • Related tests pass locally. (test results not captured in session context)
  • Manual verification completed, if applicable. (manual verification not captured in session context)

Evidence from session context:

  • Test files changed: tests/test_branch_import_sync.py
  • Test command results were not captured in the session metadata.

Commands run

No command evidence was captured in the session context.

Checklist

  • The change is limited to this PR's purpose.
  • Documentation is updated where needed. (not required for this change)
  • CHANGELOG.md is updated for notable changes, or no update is needed. (no changelog update identified)
  • Backward compatibility was considered for the public SDK and configuration. (not independently verified in session context)
  • New dependencies have compatible licenses, if applicable. (no new dependencies identified)
  • UX contract tests are included when user-visible security behavior changes.

- Continue with existing local branches when remote checks time out
- Avoid treating unknown remote state as a missing branch
- Abort checkout safely when synchronization cannot be confirmed
- Add regression tests for timeout scenarios

🤖 Generated with [OpenCode](https://opencode.ai)

Co-Authored-By: OpenCode <noreply@opencode.ai>
- Configure `_sync_branch_for_import` to return success
- Preserve initialization test isolation
- Keep unrelated helpers mocked as before

🤖 Generated with [OpenCode](https://opencode.ai)

Co-Authored-By: OpenCode <noreply@opencode.ai>
@itdove
itdove marked this pull request as ready for review November 4, 2026 02:04
@itdove
itdove merged commit 5146137 into main Nov 4, 2026
11 checks passed
@itdove
itdove deleted the 663 branch November 4, 2026 02:05
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.

bug: Handle ls-remote timeouts when reopening imported sessions

1 participant