jpmschweitzer/clide
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

add 'Deny & simplify' permission option (T-311)

Browse Source 
A fourth button on the permission card that denies the action with a
preformatted note: it's too complex for the permission system, retry in a
simpler/more granular form, and explicitly do NOT add a memory or change
permission settings (so Claude reformulates instead of fiddling with the
permission surface). A typed note is appended rather than discarded.
Addressable by number key (4 with remember, else 3); tooltip explains it.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
jpmschweitzer
2026-06-10 14:14:00 +02:00
co-authored by Claude Opus 4.8
parent 01445c4fae
commit 2c8e9f2821
6 changed files with 198 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
test/builtin/claude/prompt_card_test.dart
+43
View File
@@ -230,6 +230,33 @@ void main() {
expect((decision as DenyTool).message, 'write it under docs/ instead');
});
testWidgets('permission: Deny & simplify resolves with the preformatted retry-simpler note (T-311)', (tester) async {
ToolDecision? decision;
await tester.pumpWidget(harness(f, ToolPromptCard(prompt: permissionPrompt(), onResolve: (_, d) => decision = d)));
await tester.pump();
expect(find.text('3. Deny & simplify'), findsOneWidget); // slot 3 with no remember button
await tester.tap(find.text('3. Deny & simplify'));
await tester.pump();
final msg = (decision as DenyTool).message;
expect(msg, contains('too complex'));
// The clause that keeps Claude off the permission surface.
expect(msg, contains('do not add a memory'));
expect(msg, contains('change permission settings'));
});
testWidgets('permission: Deny & simplify appends the user note rather than discarding it (T-311)', (tester) async {
ToolDecision? decision;
await tester.pumpWidget(harness(f, ToolPromptCard(prompt: permissionPrompt(), onResolve: (_, d) => decision = d)));
await tester.pump();
await tester.enterText(find.byType(EditableText), 'this is a glob, not a path');
await tester.pump();
await tester.tap(find.text('3. Deny & simplify'));
await tester.pump();
final msg = (decision as DenyTool).message;
expect(msg, contains('too complex'));
expect(msg, contains('User note: this is a glob, not a path'));
});
testWidgets('permission: a typed note rides Allow as a follow-up note', (tester) async {
ToolDecision? decision;
await tester.pumpWidget(harness(f, ToolPromptCard(prompt: permissionPrompt(), onResolve: (_, d) => decision = d)));
@@ -520,6 +547,22 @@ void main() {
expect(d, isA<DenyTool>());
});
testWidgets('no remember: 3 = Deny & simplify (T-311)', (tester) async {
ToolDecision? d;
await pumpCard(tester, permissionPrompt(), (x) => d = x);
await tester.sendKeyEvent(LogicalKeyboardKey.digit3);
await tester.pump();
expect((d as DenyTool).message, contains('too complex'));
});
testWidgets('with a remember suggestion: 4 = Deny & simplify (T-311)', (tester) async {
ToolDecision? d;
await pumpCard(tester, permissionPrompt(suggestions: sugg), (x) => d = x);
await tester.sendKeyEvent(LogicalKeyboardKey.digit4);
await tester.pump();
expect((d as DenyTool).message, contains('too complex'));
});
testWidgets('a number key selects a question option', (tester) async {
ToolDecision? d;
await pumpCard(tester, questionPrompt(), (x) => d = x);