Skip to content

feat(sync): Close remote issues when deleting synced tasks#100

Merged
dcramer merged 1 commit intomainfrom
feat/close-remote-on-delete
Jan 30, 2026
Merged

feat(sync): Close remote issues when deleting synced tasks#100
dcramer merged 1 commit intomainfrom
feat/close-remote-on-delete

Conversation

@dcramer
Copy link
Copy Markdown
Owner

@dcramer dcramer commented Jan 30, 2026

When running dex remove on a task synced to GitHub, the corresponding
GitHub issue is now automatically closed. This maintains consistency with
how create/update operations sync to remote integrations.

Previously, deleting a synced task would only remove it locally, leaving
the GitHub issue orphaned and open. Now the local storage remains the
source of truth and deletions propagate to the remote.

Changes:

  • Add optional closeRemote method to RegisterableSyncService interface
  • Implement closeRemote in GitHubSyncService
  • Update TaskService.delete to close remote issues before local deletion
  • Close operations run in parallel; errors are logged but don't block deletion

When running `dex remove` on a task that has been synced to GitHub,
the corresponding GitHub issue is now automatically closed. This
maintains consistency with how create/update operations sync to
remote integrations.

- Add optional `closeRemote` method to `RegisterableSyncService` interface
- Implement `closeRemote` in `GitHubSyncService` to close issues
- Update `TaskService.delete` to close remote issues before deleting
- Close operations run in parallel for efficiency
- Errors closing remote issues are logged but don't block local delete

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel bot commented Jan 30, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dex Ready Ready Preview, Comment Jan 30, 2026 1:07am

Request Review

@dcramer dcramer marked this pull request as ready for review January 30, 2026 01:10
@dcramer dcramer merged commit f53a610 into main Jan 30, 2026
4 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