From 40e74810323cbd4d7b7d8259f0253dde95e7b7f7 Mon Sep 17 00:00:00 2001
From: Junyan Qin <rockchinq@gmail.com>
Date: Fri, 26 Jun 2026 19:09:35 +0800
Subject: [PATCH] fix(web): remove discard option from event target selector

Unorchestrated events are discarded by default, so an explicit discard
target is redundant. Drop the discard CommandGroup (and the now-unused
CommandSeparator import); the currentLabel() discard branch is kept so
any pre-existing discard bindings still render correctly.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
---
 .../components/bot-form/EventBindingsEditor.tsx   | 15 ---------------
 1 file changed, 15 deletions(-)

diff --git a/web/src/app/home/bots/components/bot-form/EventBindingsEditor.tsx b/web/src/app/home/bots/components/bot-form/EventBindingsEditor.tsx
index 15f43c255..fa73d62dd 100644
--- a/web/src/app/home/bots/components/bot-form/EventBindingsEditor.tsx
+++ b/web/src/app/home/bots/components/bot-form/EventBindingsEditor.tsx
@@ -30,7 +30,6 @@ import {
   CommandInput,
   CommandItem,
   CommandList,
-  CommandSeparator,
 } from '@/components/ui/command';
 import {
   Select,
@@ -293,20 +292,6 @@ function TargetCombobox({
                 ))}
               </CommandGroup>
             )}
-            <CommandSeparator />
-            <CommandGroup>
-              <CommandItem
-                value="__discard__"
-                className="text-destructive"
-                onSelect={() => select('discard')}
-              >
-                <Ban className="mr-2 size-3.5 shrink-0" />
-                {t('bots.targetDiscard')}
-                {current === 'discard' && (
-                  <Check className="ml-auto size-3.5 shrink-0" />
-                )}
-              </CommandItem>
-            </CommandGroup>
           </CommandList>
         </Command>
       </PopoverContent>