@yancyyu/openhermit 1.6.42 → 1.6.43

package/src/renderer/components/team/dialogs/CreateTeamDialog.tsx

@@ -25,6 +25,7 @@ import { useCreateTeamDraft } from '@renderer/hooks/useCreateTeamDraft';
 import { useTheme } from '@renderer/hooks/useTheme';
 import { cn } from '@renderer/lib/utils';
 import { normalizePath } from '@renderer/utils/pathNormalize';
+import { generateBindProject, isValidBindProject } from '@renderer/utils/bindProjectSlug';
 import { isEphemeralProjectPath } from '@shared/utils/ephemeralProjectPath';
 import { AlertTriangle, CheckCircle2, X } from 'lucide-react';
 
@@ -62,45 +63,7 @@ export interface TeamCopyData {
   templateDirectoryId?: string;
 }
 
-/**
- * Sanitize team name: keep Unicode letters and digits (Chinese, Latin, etc.),
- * replace other sequences with `-`, then lowercase Latin chars.
- */
-function sanitizeTeamName(name: string): string {
-  const trimmed = name.trim();
-  if (!trimmed) return '';
-  let result = trimmed
-    .replace(/[^\p{L}\p{N}]+/gu, '-')
-    .replace(/^-+|-+$/g, '')
-    .toLowerCase();
-  return result;
-}
-
-/**
- * Generate a unique ASCII project identifier from a display name.
- * For Chinese names: produces "team-xxxx" (4-char random suffix).
- * For ASCII names: produces a slugified version.
- */
-function generateBindProject(displayName: string): string {
-  const trimmed = displayName.trim();
-  if (!trimmed) return '';
-  // Try to extract ASCII parts from the name
-  const asciiParts = trimmed
-    .toLowerCase()
-    .normalize('NFKD')
-    .replace(/[̀-ͯ]/g, '')
-    .replace(/[^a-z0-9]+/g, '-')
-    .replace(/-+/g, '-')
-    .replace(/^-|-$/g, '');
-  const base = asciiParts || 'team';
-  const suffix = Math.random().toString(36).slice(2, 6);
-  return `${base}-${suffix}`;
-}
-
-/** Validate bindProject: ASCII lowercase alphanumeric, hyphens, underscores. */
-function isValidBindProject(value: string): boolean {
-  return /^[a-z0-9][a-z0-9_-]{1,}$/.test(value);
-}
+// bindProject slug rules live in @renderer/utils/bindProjectSlug (unit-tested).
 
 // ---------------------------------------------------------------------------
 // Wizard step types
@@ -119,6 +82,7 @@ interface CreateTeamDialogProps {
   clearProvisioningError?: (teamName?: string) => void;
   existingTeamNames: string[];
   existingBindProjects?: string[];
+  existingDisplayNames?: string[];
   provisioningTeamNames?: string[];
   activeTeams?: ActiveTeamRef[];
   initialData?: unknown;
@@ -135,6 +99,7 @@ export const CreateTeamDialog = ({
   clearProvisioningError,
   existingTeamNames,
   existingBindProjects = [],
+  existingDisplayNames = [],
   provisioningTeamNames = [],
   activeTeams,
   defaultProjectPath,
@@ -166,16 +131,13 @@ export const CreateTeamDialog = ({
   const [description, setDescription] = useState('');
 
   // ── bindProject (ASCII unique identifier) ────────────────────────────
-  const [bindProject, setBindProject] = useState('');
+  // Only the *manual* override is state. The auto value is DERIVED (below) so
+  // it is collision-free in the SAME render as `existingBindProjectSet`. A
+  // state + useEffect pair lagged one render behind the set and flickered a
+  // false "该项目标识已存在" red box whenever the teams list refreshed.
+  const [manualBindProject, setManualBindProject] = useState('');
   const [bindProjectManuallyEdited, setBindProjectManuallyEdited] = useState(false);
 
-  // Auto-generate bindProject from displayName when not manually edited
-  useEffect(() => {
-    if (bindProjectManuallyEdited) return;
-    const auto = generateBindProject(teamName);
-    setBindProject(auto);
-  }, [teamName, bindProjectManuallyEdited]);
-
   // ── Projects (for path selector) ─────────────────────────────────────
   const [projects, setProjects] = useState<Project[]>([]);
   const [projectsLoading, setProjectsLoading] = useState(false);
@@ -189,10 +151,63 @@ export const CreateTeamDialog = ({
   const [localError, setLocalError] = useState<string | null>(null);
   const [isSubmitting, setIsSubmitting] = useState(false);
   const [fieldErrors, setFieldErrors] = useState<{ teamName?: string; cwd?: string }>({});
+  // The slug ACTUALLY used at creation. The live `bindProject` re-derives once
+  // the parent's fetchTeams() refreshes existingBindProjects (the just-created
+  // slug is now taken) and its numeric-counter fallback can land on an
+  // UNRELATED existing team's slug — so the done step's "open" button must use
+  // this captured id, not the re-derived one. See the "opens the JUST-CREATED
+  // team" regression test.
+  const [createdTeamName, setCreatedTeamName] = useState<string | null>(null);
 
   // ── Name conflict detection ──────────────────────────────────────────
-  const isBindProjectTaken = existingBindProjects.includes(bindProject);
-  const isNameProvisioning = provisioningTeamNames.includes(bindProject) && !isBindProjectTaken;
+  const existingBindProjectSet = useMemo(
+    () => new Set(existingBindProjects.map((value) => value.trim().toLowerCase()).filter(Boolean)),
+    [existingBindProjects]
+  );
+  const provisioningTeamNameSet = useMemo(
+    () => new Set(provisioningTeamNames.map((value) => value.trim().toLowerCase()).filter(Boolean)),
+    [provisioningTeamNames]
+  );
+  // displayName (the human-readable name) must be unique too. A duplicate name
+  // silently produced a ghost team (the 222-2 incident: draft-restored name +
+  // auto-collision-free bindProject + a second create click), so block it up front.
+  const existingDisplayNameSet = useMemo(
+    () => new Set(existingDisplayNames.map((value) => value.trim().toLowerCase()).filter(Boolean)),
+    [existingDisplayNames]
+  );
+  const isDisplayNameTaken =
+    Boolean(teamName.trim()) && existingDisplayNameSet.has(teamName.trim().toLowerCase());
+  // The collision check is reactive to the live `teams` list. The instant a
+  // create succeeds, the parent store adds the new team and `isDisplayNameTaken`
+  // trips on the just-created name for the one frame before the dialog moves to
+  // the success step — a false "该名称已存在" red-box flicker. Validation
+  // (validateCreateFields) runs pre-submit and still uses the raw check, but the
+  // *visual* red box / error text / disabled state is suppressed while a create
+  // is in flight, since by then the name has already been accepted.
+  const showDisplayNameTaken = isDisplayNameTaken && !isSubmitting;
+
+  // bindProject is DERIVED, not state. The auto value is generated in the same
+  // render as `existingBindProjectSet`, so it is collision-free BY CONSTRUCTION
+  // and `isBindProjectTaken` can never flicker true between a set change and an
+  // effect flush. (The previous state + useEffect implementation lagged one
+  // render and produced a transient false "该项目标识已存在" red box.)
+  const autoBindProject = useMemo(
+    () => generateBindProject(teamName, existingBindProjectSet),
+    [teamName, existingBindProjectSet]
+  );
+  const bindProject = bindProjectManuallyEdited ? manualBindProject : autoBindProject;
+  const normalizedBindProject = bindProject.trim().toLowerCase();
+
+  const isBindProjectTaken = existingBindProjectSet.has(normalizedBindProject);
+  // Slug the done-step "open" + provisioning-error lookup target: the one
+  // actually used at creation (captured), falling back to the live form value
+  // before creation. Never the re-derived `bindProject` once the parent list
+  // refreshes — that drifts to a different/colliding id (createdTeamName note).
+  const openTeamSlug = createdTeamName ?? bindProject;
+  const isNameProvisioning =
+    provisioningTeamNameSet.has(normalizedBindProject) && !isBindProjectTaken;
+  const isBindProjectFormatInvalid =
+    Boolean(bindProject) && !isValidBindProject(normalizedBindProject);
 
   const effectiveCwd =
     cwdMode === 'project'
@@ -305,15 +320,24 @@ export const CreateTeamDialog = ({
     setIsSubmitting(false);
     setConflictDismissed(false);
     setSelectedProviderRef(null);
-    setBindProject('');
+    setManualBindProject('');
     setBindProjectManuallyEdited(false);
+    setCreatedTeamName(null);
     setStep('name');
   };
 
+  // Every close path (overlay click, ✕, 取消) clears the persisted draft so a
+  // stale restored name can never seed a duplicate create on reopen.
+  const handleClose = () => {
+    clearDraft();
+    resetState();
+    onClose();
+  };
+
   const buildCreateRequest = (): TeamCreateRequest => ({
-    teamName: bindProject,
-    bindProject,
-    displayName: teamName.trim() || undefined,
+    teamName: normalizedBindProject,
+    bindProject: normalizedBindProject,
+    displayName: teamName.trim(),
     description: description.trim() || undefined,
     color: teamColor || undefined,
     members: [],
@@ -330,16 +354,26 @@ export const CreateTeamDialog = ({
       setLocalError('请输入数字员工名称');
       return false;
     }
-    if (!bindProject) {
+    if (isDisplayNameTaken) {
+      setLocalError('该名称已存在，请换一个');
+      return false;
+    }
+    if (!normalizedBindProject) {
       setLocalError('请输入项目标识');
       return false;
     }
-    if (!isValidBindProject(bindProject)) {
-      setLocalError('项目标识只能包含小写英文字母、数字、连字符和下划线（至少2个字符，以字母或数字开头）');
+    if (!isValidBindProject(normalizedBindProject)) {
+      setLocalError(
+        '项目标识为必填，只能包含小写英文字母、数字、连字符和下划线，且必须以字母或数字开头'
+      );
+      return false;
+    }
+    if (isNameProvisioning) {
+      setLocalError('数字员工正在启动中');
       return false;
     }
     if (isBindProjectTaken) {
-      setLocalError(isNameProvisioning ? '数字员工正在启动中' : `项目标识"${bindProject}"已存在，请换一个`);
+      setLocalError(`项目标识"${normalizedBindProject}"已存在，请换一个`);
       return false;
     }
     if (!effectiveCwd) {
@@ -349,8 +383,8 @@ export const CreateTeamDialog = ({
     return true;
   };
 
-  const createLocalTeam = async (): Promise<TeamCreateRequest | null> => {
-    if (!validateCreateFields()) return null;
+  const createLocalTeam = async (): Promise<boolean> => {
+    if (!validateCreateFields()) return false;
 
     setFieldErrors({});
     setLocalError(null);
@@ -359,19 +393,31 @@ export const CreateTeamDialog = ({
     try {
       const request = buildCreateRequest();
       await onCreate(request);
-      return request;
+      // Capture the slug that was actually persisted. The done step opens /
+      // surfaces errors for THIS team; the live `bindProject` would drift to a
+      // different id once the parent list refreshes (see createdTeamName note).
+      setCreatedTeamName(request.teamName);
+      // Advance to the success step in the SAME synchronous block as the
+      // `finally` that releases isSubmitting. The create just added this name
+      // to the parent's existingDisplayNames; if the name step re-rendered
+      // with isSubmitting already false, `showDisplayNameTaken` would trip on
+      // the just-created name for one frame — the false "该名称已存在"
+      // red-box flicker right before the success screen. Setting 'done' here
+      // (no await between it and the finally) batches with
+      // setIsSubmitting(false), so the name step is unmounted atomically and
+      // that transition window never opens.
+      setStep('done');
+      return true;
     } catch {
       // error shown via provisioningErrorsByTeam
-      return null;
+      return false;
     } finally {
       setIsSubmitting(false);
     }
   };
 
   const handleCreate = async () => {
-    const request = await createLocalTeam();
-    if (!request) return;
-    setStep('done');
+    await createLocalTeam();
   };
 
   // ── Render ───────────────────────────────────────────────────────────
@@ -379,10 +425,7 @@ export const CreateTeamDialog = ({
     <Dialog
       open={open}
       onOpenChange={(next: boolean) => {
-        if (!next) {
-          resetState();
-          onClose();
-        }
+        if (!next) handleClose();
       }}
     >
       <DialogContent className="w-[calc(100vw-2rem)] max-w-2xl sm:w-[40rem]">
@@ -407,7 +450,7 @@ export const CreateTeamDialog = ({
               <AlertTriangle className="mt-0.5 size-4 shrink-0" />
               <div className="min-w-0 flex-1 space-y-1">
                 <p className="font-medium">
-                  该工作目录下已有数字员工"{conflictingTeam.displayName}"正在运行
+                  该工作目录下已有数字员工&quot;{conflictingTeam.displayName}&quot;正在运行
                 </p>
                 <p className="opacity-80">在同一目录同时运行两个数字员工存在风险。</p>
               </div>
@@ -426,12 +469,14 @@ export const CreateTeamDialog = ({
         {step === 'name' && (
           <div className="space-y-4 py-2">
             <div className="space-y-1.5">
-              <Label htmlFor="team-name">数字员工名称</Label>
+              <Label htmlFor="team-name">数字员工名称 *</Label>
               <Input
                 id="team-name"
+                required
                 className={cn(
                   'h-8 text-xs',
-                  fieldErrors.teamName && 'border-[var(--field-error-border)]'
+                  (fieldErrors.teamName || showDisplayNameTaken) &&
+                    'border-[var(--field-error-border)]'
                 )}
                 value={teamName}
                 onChange={(e) => setTeamName(e.target.value)}
@@ -443,23 +488,38 @@ export const CreateTeamDialog = ({
                   同名数字员工正在启动中
                 </p>
               )}
+              {showDisplayNameTaken && (
+                <p className="text-[11px] text-red-400">该名称已存在，请换一个</p>
+              )}
             </div>
 
             <div className="space-y-1.5">
-              <Label htmlFor="team-bind-project">项目标识</Label>
+              <Label htmlFor="team-bind-project">项目标识 *</Label>
               <Input
                 id="team-bind-project"
+                required
                 className={cn(
-                  'h-8 text-xs font-mono',
-                  isBindProjectTaken && 'border-[var(--field-error-border)]'
+                  'h-8 font-mono text-xs',
+                  (isBindProjectTaken || isBindProjectFormatInvalid) &&
+                    'border-[var(--field-error-border)]'
                 )}
                 value={bindProject}
                 onChange={(e) => {
-                  setBindProject(e.target.value.toLowerCase().replace(/[^a-z0-9_-]/g, ''));
+                  setManualBindProject(e.target.value.toLowerCase().replace(/[^a-z0-9_-]/g, ''));
                   setBindProjectManuallyEdited(true);
                 }}
                 placeholder="auto-generated-id"
               />
+              {isBindProjectFormatInvalid && (
+                <p className="text-[11px]" style={{ color: 'var(--field-error-text)' }}>
+                  项目标识只能包含小写英文、数字、连字符和下划线，且必须以字母或数字开头
+                </p>
+              )}
+              {isNameProvisioning && (
+                <p className="text-[11px]" style={{ color: 'var(--warning-text)' }}>
+                  该项目标识正在创建中
+                </p>
+              )}
               {isBindProjectTaken && (
                 <p className="text-[11px]" style={{ color: 'var(--field-error-text)' }}>
                   该项目标识已存在
@@ -533,8 +593,8 @@ export const CreateTeamDialog = ({
                         onClick={() => selectProviderRef(provider.name)}
                         className={`w-full rounded-lg border px-3 py-2 text-left transition-colors ${
                           checked
-                            ? 'border-indigo-400/60 bg-indigo-500/10'
-                            : 'border-[var(--color-border-subtle)] bg-black/10 hover:border-[var(--color-border)] hover:bg-white/[0.04]'
+                            ? 'shadow-[var(--color-accent-glow)]/20 border-[var(--color-accent-border)] bg-[var(--color-accent-muted)] shadow-sm'
+                            : 'border-[var(--color-border-subtle)] bg-black/10 hover:border-[var(--color-border)] hover:bg-[var(--color-accent-soft)]'
                         }`}
                       >
                         <div className="flex items-center justify-between gap-3">
@@ -549,7 +609,7 @@ export const CreateTeamDialog = ({
                           <span
                             className={`shrink-0 rounded-full px-2 py-0.5 text-[10px] ${
                               checked
-                                ? 'bg-indigo-400/20 text-indigo-200'
+                                ? 'bg-[var(--color-accent-soft)] text-[var(--color-accent)]'
                                 : 'bg-white/5 text-[var(--color-text-muted)]'
                             }`}
                           >
@@ -600,7 +660,7 @@ export const CreateTeamDialog = ({
                 {localError}
               </p>
             )}
-            {provisioningErrorsByTeam[bindProject] && (
+            {provisioningErrorsByTeam[openTeamSlug] && (
               <p
                 className="mt-1 rounded border p-2 text-xs"
                 style={{
@@ -609,7 +669,7 @@ export const CreateTeamDialog = ({
                   backgroundColor: 'var(--field-error-bg)',
                 }}
               >
-                {provisioningErrorsByTeam[bindProject]}
+                {provisioningErrorsByTeam[openTeamSlug]}
               </p>
             )}
           </div>
@@ -631,7 +691,7 @@ export const CreateTeamDialog = ({
                 <Button
                   size="sm"
                   onClick={() => {
-                    onOpenTeam(bindProject, effectiveCwd || undefined, {
+                    onOpenTeam(openTeamSlug, effectiveCwd || undefined, {
                       displayName: teamName.trim() || undefined,
                     });
                     clearDraft();
@@ -644,10 +704,23 @@ export const CreateTeamDialog = ({
               </>
             ) : (
               <>
-                <Button variant="outline" size="sm" onClick={onClose}>
+                <Button variant="outline" size="sm" onClick={handleClose}>
                   取消
                 </Button>
-                <Button size="sm" disabled={!teamName.trim() || !bindProject || !effectiveCwd || isSubmitting} onClick={handleCreate}>
+                <Button
+                  size="sm"
+                  disabled={
+                    !teamName.trim() ||
+                    !normalizedBindProject ||
+                    !effectiveCwd ||
+                    isBindProjectFormatInvalid ||
+                    isBindProjectTaken ||
+                    showDisplayNameTaken ||
+                    isNameProvisioning ||
+                    isSubmitting
+                  }
+                  onClick={handleCreate}
+                >
                   {isSubmitting ? '创建中...' : '创建数字员工'}
                 </Button>
               </>

package/src/renderer/components/team/dialogs/EditTeamDialog.tsx

@@ -1,7 +1,6 @@
 import { useEffect, useMemo, useRef, useState } from 'react';
 
 import { Button } from '@renderer/components/ui/button';
-import { Checkbox } from '@renderer/components/ui/checkbox';
 import {
   Dialog,
   DialogContent,
@@ -42,60 +41,20 @@ export const EditTeamDialog = ({
   }));
 
   // ── Derived defaults ─────────────────────────────────────────
-  const rawSettings = useMemo(
-    () => (data?.settings ?? {}) as Record<string, unknown>,
-    [data?.settings]
-  );
-
+  // Loop 动态设置（语言/管理来源/飞书权限/消息格式）已统一迁入 RuntimeConfigDialog，
+  // 这里只保留团队基础信息：名称、描述、颜色 (#21)。
   const defaults = useMemo(() => {
     const cfg = data?.config;
     return {
       name: cfg?.name ?? '',
       description: cfg?.description ?? '',
       color: cfg?.color ?? '',
-      language:
-        cfg?.language ?? (typeof rawSettings.language === 'string' ? rawSettings.language : 'zh'),
-      managedSources:
-        cfg?.managedSources ??
-        (typeof rawSettings.admin_from === 'string' ? rawSettings.admin_from : '*'),
-      platformAllowFrom:
-        cfg?.platformAllowFrom ??
-        (typeof rawSettings.platform_allow_from === 'object' &&
-        rawSettings.platform_allow_from !== null &&
-        !Array.isArray(rawSettings.platform_allow_from)
-          ? (rawSettings.platform_allow_from as Record<string, string>)
-          : {}),
-      platformAllowChat:
-        cfg?.platformAllowChat ??
-        (typeof (rawSettings as Record<string, unknown>).platform_allow_chat === 'object' &&
-        (rawSettings as Record<string, unknown>).platform_allow_chat !== null &&
-        !Array.isArray((rawSettings as Record<string, unknown>).platform_allow_chat)
-          ? ((rawSettings as Record<string, unknown>).platform_allow_chat as Record<string, string>)
-          : {}),
-      showContextIndicator:
-        cfg?.showContextIndicator ??
-        (typeof rawSettings.show_context_indicator === 'boolean'
-          ? rawSettings.show_context_indicator
-          : true),
-      replyFooter:
-        cfg?.replyFooter ??
-        (typeof rawSettings.reply_footer === 'boolean' ? rawSettings.reply_footer : true),
-      injectSender:
-        cfg?.injectSender ??
-        (typeof rawSettings.inject_sender === 'boolean' ? rawSettings.inject_sender : false),
     };
-  }, [data, rawSettings]);
+  }, [data]);
 
   // ── Local form state ─────────────────────────────────────────
   const [name, setName] = useState(defaults.name);
   const [description, setDescription] = useState(defaults.description);
-  const [language, setLanguage] = useState(defaults.language);
-  const [managedSources, setManagedSources] = useState(defaults.managedSources);
-  const [feishuAllowFrom, setFeishuAllowFrom] = useState(defaults.platformAllowFrom.feishu ?? '*');
-  const [feishuAllowChat, setFeishuAllowChat] = useState(defaults.platformAllowChat.feishu ?? '*');
-  const [showContextIndicator, setShowContextIndicator] = useState(defaults.showContextIndicator);
-  const [replyFooter, setReplyFooter] = useState(defaults.replyFooter);
-  const [injectSender, setInjectSender] = useState(defaults.injectSender);
   const [savePhase, setSavePhase] = useState<'idle' | 'saving' | 'done'>('idle');
   const [error, setError] = useState<string | null>(null);
   const saving = savePhase === 'saving';
@@ -115,13 +74,6 @@ export const EditTeamDialog = ({
     setError(null);
     setName(d.name);
     setDescription(d.description);
-    setLanguage(d.language);
-    setManagedSources(d.managedSources);
-    setFeishuAllowFrom(d.platformAllowFrom.feishu ?? '*');
-    setFeishuAllowChat(d.platformAllowChat.feishu ?? '*');
-    setShowContextIndicator(d.showContextIndicator);
-    setReplyFooter(d.replyFooter);
-    setInjectSender(d.injectSender);
   }, [open]);
 
   const handleSave = (): void => {
@@ -133,22 +85,12 @@ export const EditTeamDialog = ({
     setSavePhase('saving');
     setError(null);
 
-    const feishu = feishuAllowFrom.trim();
-    const feishuChat = feishuAllowChat.trim();
-
     void (async () => {
       try {
         await api.teams.updateConfig(teamName, {
           name: name.trim(),
           description: description.trim(),
           color: defaultsRef.current.color,
-          language: language.trim() || undefined,
-          managedSources: managedSources.trim() || undefined,
-          platformAllowFrom: feishu ? { feishu } : {},
-          platformAllowChat: feishuChat ? { feishu: feishuChat } : {},
-          showContextIndicator,
-          replyFooter,
-          injectSender,
         });
         await Promise.all([fetchTeams(), selectTeam(teamName)]);
         setSavePhase('done');
@@ -173,7 +115,7 @@ export const EditTeamDialog = ({
       <DialogContent className="max-w-lg">
         <DialogHeader>
           <DialogTitle>编辑团队</DialogTitle>
-          <DialogDescription>修改团队信息和消息设置（无需重启）</DialogDescription>
+          <DialogDescription>修改团队名称和描述</DialogDescription>
         </DialogHeader>
 
         <div className="space-y-4">
@@ -214,105 +156,15 @@ export const EditTeamDialog = ({
             />
           </div>
 
-          {/* Messaging settings */}
-          <div className="rounded-md border border-[var(--color-border)] p-3">
-            <h3 className="text-xs font-medium text-[var(--color-text)]">消息设置</h3>
-            <div className="mt-3 space-y-3">
-              <div className="grid gap-3 md:grid-cols-2">
-                <div>
-                  <label className={labelCls}>语言</label>
-                  <input
-                    type="text"
-                    value={language}
-                    onChange={(e) => {
-                      setError(null);
-                      setLanguage(e.target.value);
-                    }}
-                    className={inputCls}
-                    placeholder="zh"
-                  />
-                </div>
-                <div>
-                  <label className={labelCls}>管理来源</label>
-                  <input
-                    type="text"
-                    value={managedSources}
-                    onChange={(e) => {
-                      setError(null);
-                      setManagedSources(e.target.value);
-                    }}
-                    className={inputCls}
-                    placeholder="user1,user2 或 *"
-                  />
-                </div>
-              </div>
-              <div className="grid gap-3 md:grid-cols-2">
-                <div>
-                  <label className={labelCls}>飞书私聊权限</label>
-                  <input
-                    type="text"
-                    value={feishuAllowFrom}
-                    onChange={(e) => {
-                      setError(null);
-                      setFeishuAllowFrom(e.target.value);
-                    }}
-                    className={inputCls}
-                    placeholder="ou_xxx 或 *"
-                  />
-                </div>
-                <div>
-                  <label className={labelCls}>飞书群聊权限</label>
-                  <input
-                    type="text"
-                    value={feishuAllowChat}
-                    onChange={(e) => {
-                      setError(null);
-                      setFeishuAllowChat(e.target.value);
-                    }}
-                    className={inputCls}
-                    placeholder="oc_xxx 或 *"
-                  />
-                </div>
-              </div>
-              <div className="grid gap-2 md:grid-cols-3">
-                <label className="flex cursor-pointer items-center gap-2 rounded-md border border-[var(--color-border)] px-2 py-1.5 text-xs text-[var(--color-text-secondary)]">
-                  <Checkbox
-                    checked={showContextIndicator}
-                    onCheckedChange={(c) => {
-                      setError(null);
-                      setShowContextIndicator(c === true);
-                    }}
-                  />
-                  上下文指示
-                </label>
-                <label className="flex cursor-pointer items-center gap-2 rounded-md border border-[var(--color-border)] px-2 py-1.5 text-xs text-[var(--color-text-secondary)]">
-                  <Checkbox
-                    checked={replyFooter}
-                    onCheckedChange={(c) => {
-                      setError(null);
-                      setReplyFooter(c === true);
-                    }}
-                  />
-                  回复尾部信息
-                </label>
-                <label className="flex cursor-pointer items-center gap-2 rounded-md border border-[var(--color-border)] px-2 py-1.5 text-xs text-[var(--color-text-secondary)]">
-                  <Checkbox
-                    checked={injectSender}
-                    onCheckedChange={(c) => {
-                      setError(null);
-                      setInjectSender(c === true);
-                    }}
-                  />
-                  注入发送者
-                </label>
-              </div>
-            </div>
-          </div>
-
           {error && <p className="text-xs text-red-400">{error}</p>}
         </div>
 
         <DialogFooter>
+          {onDeleteTeam && teamName !== 'default' ? (
+            <Button variant="ghost" size="sm" onClick={onDeleteTeam} disabled={saving}>
+              删除项目
+            </Button>
+          ) : null}
           <Button variant="outline" size="sm" onClick={onClose} disabled={saving}>
             {savePhase === 'done' ? '关闭' : '取消'}
           </Button>