Skip to content
This repository was archived by the owner on Sep 23, 2025. It is now read-only.

Commit 2a13102

Browse files
committed
Clear tree view after user action
**Clean UX: Tree view clears after feedback action** **Changes:** - Added clearPR() method to SyntheticPRProvider - Call clearPR() after user selects any action button - Tree view shows 'No active pull request' after action **Flow:** 1. User clicks action button (Request Changes, Checkpoint, Close Review) 2. Action processed and feedback sent to LLM 3. Tree view immediately clears to show completion 4. User knows their action was processed **Benefits:** - **Clear visual feedback** - User sees action was processed - **Clean state** - No stale review UI after completion - **Intuitive UX** - Tree view state matches review state
1 parent 57caa77 commit 2a13102

File tree

2 files changed

+10
-0
lines changed

2 files changed

+10
-0
lines changed

extension/src/extension.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -530,6 +530,8 @@ class DaemonClient implements vscode.Disposable {
530530
});
531531
}
532532

533+
// Clear tree view after action
534+
this.syntheticPRProvider.clearPR();
533535
this.pendingFeedbackResolvers.delete(reviewId);
534536
}
535537

extension/src/syntheticPRProvider.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -457,6 +457,14 @@ export class SyntheticPRProvider implements vscode.Disposable {
457457
reply.thread.canReply = true;
458458
}
459459

460+
/**
461+
* Clear the current PR from tree view
462+
*/
463+
clearPR(): void {
464+
this.treeProvider.clearPR();
465+
this.currentPR = null;
466+
}
467+
460468
dispose(): void {
461469
this.commentController.dispose();
462470
this.treeProvider.clearPR();

0 commit comments

Comments
 (0)