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

Commit 307a8e9

Browse files
committed
Replace QuickPick with tree view action buttons
**Better UX: Tree View Actions Instead of Interrupting QuickPick** **Changes:** - Updated Actions section in tree view with proper feedback buttons: - πŸ’¬ Add Comment - βœ… Request Changes - πŸ“ Checkpoint Work - ↩️ Close Review (was 'Return to Agent') - Added command property to action items: 'dialectic.reviewAction' - Each button passes its action type as argument **Benefits:** - **No workflow interruption** - Actions stay in review context - **More discoverable** - Buttons visible in tree view - **Clearer labels** - 'Close Review' vs 'Return to Agent' - **Better UX** - No modal dialogs blocking the interface **Next:** Wire up the command handling in extension.ts
1 parent 5922a92 commit 307a8e9

File tree

1 file changed

+23
-4
lines changed

1 file changed

+23
-4
lines changed

β€Žextension/src/syntheticPRTreeProvider.tsβ€Ž

Lines changed: 23 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,13 @@ class PRTreeItem extends vscode.TreeItem {
5151
{ selection: new vscode.Range(data.line_number - 1, 0, data.line_number - 1, 0) }
5252
]
5353
};
54+
} else if (itemType === 'action') {
55+
// Add command for action buttons
56+
this.command = {
57+
command: 'dialectic.reviewAction',
58+
title: 'Review Action',
59+
arguments: [data.action]
60+
};
5461
}
5562
}
5663
}
@@ -168,19 +175,31 @@ export class SyntheticPRTreeProvider implements vscode.TreeDataProvider<PRTreeIt
168175
}
169176

170177
if (element.itemType === 'actions') {
171-
// Show approve/request changes actions
178+
// Show feedback actions
172179
return Promise.resolve([
173180
new PRTreeItem(
174-
'βœ… Approve',
181+
'πŸ’¬ Add Comment',
175182
vscode.TreeItemCollapsibleState.None,
176183
'action',
177-
{ action: 'approve' }
184+
{ action: 'comment' }
178185
),
179186
new PRTreeItem(
180-
'πŸ”„ Request Changes',
187+
'βœ… Request Changes',
181188
vscode.TreeItemCollapsibleState.None,
182189
'action',
183190
{ action: 'request_changes' }
191+
),
192+
new PRTreeItem(
193+
'πŸ“ Checkpoint Work',
194+
vscode.TreeItemCollapsibleState.None,
195+
'action',
196+
{ action: 'checkpoint' }
197+
),
198+
new PRTreeItem(
199+
'↩️ Close Review',
200+
vscode.TreeItemCollapsibleState.None,
201+
'action',
202+
{ action: 'return' }
184203
)
185204
]);
186205
}

0 commit comments

Comments
Β (0)