@suifei/dsh-ai-team 0.3.25 → 0.3.27

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # Changelog
 
+## 0.3.27 — 2026-11-10
+
+- 主对话「任务」模式拆成标题 + 验收标准；不再把 Human 原话直接当 resultDefinition；
+- 开启 QA 时必须逐行填写 checklist，否则不能提交；
+- 修复真实项目 #3/#4 的根因：重复创建、宽泛结果定义、QA 空清单；
+- 重复 Task 的第二条会打开已有卡，不再静默新建。
+
+## 0.3.26 — 2026-11-10
+
+- Task 创建增加重复保护：相同 title/resultDefinition/owner 的 active Task 会被拒绝并返回已有 Task；
+- 主对话「任务」模式收到重复错误时自动打开已有 Task，不再产生 #3/#4 这类重复卡；
+- `qaRequired=true` 但没有 qaChecklist 时拒绝创建，避免产生无法验收的 #4；
+- 成员有 `pending_acceptance` Task 时不能再 claim 下一张，阻止 T2/T3 在 T1 验收前提前开工；
+- `team() accept` 现在透传 checks，QA Agent 不再需要走 Human API。
+
 ## 0.3.25 — 2026-11-10
 
 - 收件箱未读数按 `state=unread` 统计，决策提交后「待你处理」立即消失；

package/client.js

@@ -34,7 +34,12 @@ window.__ModuleLoader__.load({
 			tabSituation: '态势',
 			tabsAria: '项目视图',
 			composer: '写一条消息…',
-			taskPlaceholder: '一句话任务，可稍后补完判据',
+			taskPlaceholder: '任务标题：要做什么',
+			taskDefinitionPlaceholder: '验收标准（必填）：怎样算完成、拿什么证据',
+			taskDefinitionAria: 'Task 验收标准',
+			qaChecklistPlaceholder: 'QA 清单，每行一条（开启 QA 时必填）',
+			qaChecklistAria: 'QA 验收清单',
+			taskDefinitionRequired: '请先写验收标准，再创建 Task。',
 			discuss: '讨论',
 			taskMode: '任务',
 			send: '发送',
@@ -341,7 +346,12 @@ window.__ModuleLoader__.load({
 			tabSituation: 'Situation',
 			tabsAria: 'Project views',
 			composer: 'Write a message…',
-			taskPlaceholder: 'A one-line task; add the definition of done later',
+			taskPlaceholder: 'Task title: what to do',
+			taskDefinitionPlaceholder: 'Definition of done (required): how it is accepted, with what evidence',
+			taskDefinitionAria: 'Definition of done',
+			qaChecklistPlaceholder: 'QA checklist, one per line (required when QA is on)',
+			qaChecklistAria: 'QA checklist',
+			taskDefinitionRequired: 'Write the definition of done before creating a Task.',
 			discuss: 'Discuss',
 			taskMode: 'Task',
 			send: 'Send',
@@ -588,6 +598,10 @@ window.__ModuleLoader__.load({
 			conversationMock: 'spike: this host has no working layout.selectPanel(conversation).',
 			discussSent: 'Sent to the Project owner.',
 			taskSent: 'Task created.',
+			duplicateTask: 'An identical Task already exists (#{number}); opened it instead.',
+			qaNeedsChecklist: 'A QA Task needs at least one checklist item.',
+			duplicateTask: '相同 Task 已存在（#{number}），已为你打开它。',
+			qaNeedsChecklist: 'QA Task 至少需要一条验收清单。',
 			sendFailed: 'Send failed: {message}',
 			pendingHuman: 'Needs you',
 			inboxUnread: 'Inbox, {n} unread',
@@ -974,6 +988,9 @@ window.__ModuleLoader__.load({
 			.at-inboxactions { display: flex; align-items: center; gap: 6px; flex: 0 0 auto; }
 			.at-qacheck { display: inline-flex; align-items: center; gap: 4px; color: var(--dsw-alias-label-tertiary); font-size: 11px; white-space: nowrap; }
 			.at-qacheck input { accent-color: var(--dsw-alias-state-business-primary); }
+			.at-taskfields { display: flex; flex-direction: column; gap: 6px; padding: 0 10px 8px; }
+			.at-taskdef, .at-taskqa { width: 100%; box-sizing: border-box; resize: vertical; min-height: 42px; padding: 7px 8px; border: 1px solid var(--dsw-alias-border-l2); border-radius: var(--dsw-radius-sm); background: var(--dsw-alias-bg-layer-2); color: var(--dsw-alias-label-primary); font: inherit; font-size: 12px; line-height: 18px; }
+			.at-taskdef::placeholder, .at-taskqa::placeholder { color: var(--dsw-alias-label-caption); }
 			.at-qabadge { display: inline-flex; align-items: center; margin-left: 5px; padding: 0 5px; border: 1px solid var(--dsw-alias-state-business-primary); border-radius: 999px; color: var(--dsw-alias-state-business-primary); font-size: 9px; line-height: 15px; vertical-align: 1px; }
 			.at-talented { opacity: 0.82; }
 			.at-btn-small { min-height: 24px; padding: 0 8px; font-size: 11px; }
@@ -1915,8 +1932,12 @@ window.__ModuleLoader__.load({
 							h('button', { className: 'at-send', type: 'button', disabled: !reply.trim(), onClick: onReplySend }, t('send'))))) : null);
 		}
 
-		function Composer({ t, mode, draft, disabled, qaRequired, onQaRequired, onMode, onDraft, onSend }) {
-			const send = () => { if (draft.trim() && !disabled) onSend(); };
+		function Composer({ t, mode, draft, definition, qaChecklist, disabled, qaRequired, onQaRequired, onDefinition, onQaChecklist, onMode, onDraft, onSend }) {
+			const send = () => {
+				if (!draft.trim() || disabled) return;
+				if (mode === 'task' && (!definition.trim() || (qaRequired && !qaChecklist.trim()))) return;
+				onSend();
+			};
 			return h('div', { className: 'at-composer' },
 				h('div', { className: 'at-composerinner' },
 					h('div', { className: cx('at-box', 'at-composerbox', mode === 'task' && 'at-taskmode') },
@@ -1934,6 +1955,25 @@ window.__ModuleLoader__.load({
 								}
 							},
 						}),
+						mode === 'task' ? h('div', { className: 'at-taskfields' },
+							h('textarea', {
+								className: 'at-taskdef',
+								rows: 2,
+								value: definition,
+								disabled,
+								placeholder: t('taskDefinitionPlaceholder'),
+								'aria-label': t('taskDefinitionAria'),
+								onChange: (event) => onDefinition(event.target.value),
+							}),
+							qaRequired ? h('textarea', {
+								className: 'at-taskqa',
+								rows: 2,
+								value: qaChecklist,
+								disabled,
+								placeholder: t('qaChecklistPlaceholder'),
+								'aria-label': t('qaChecklistAria'),
+								onChange: (event) => onQaChecklist(event.target.value),
+							}) : null) : null,
 						h('div', { className: 'at-barrow' },
 							h('div', { className: 'at-mode', role: 'group', 'aria-label': t('modeAria') },
 								h('button', { type: 'button', 'aria-pressed': mode === 'discuss', onClick: () => onMode('discuss') }, t('discuss')),
@@ -1942,7 +1982,7 @@ window.__ModuleLoader__.load({
 								h('input', { type: 'checkbox', checked: qaRequired, onChange: (event) => onQaRequired(event.target.checked) }),
 								h('span', null, t('qaRequired'))) : null,
 							h('span', { className: 'at-hint' }, t('enterHint')),
-							h('button', { className: 'at-send', type: 'button', disabled: disabled || !draft.trim(), onClick: send }, mode === 'task' ? t('createTask') : t('send'))))));
+							h('button', { className: 'at-send', type: 'button', disabled: disabled || !draft.trim() || (mode === 'task' && (!definition.trim() || (qaRequired && !qaChecklist.trim()))), onClick: send }, mode === 'task' ? t('createTask') : t('send'))))));
 		}
 
 		function QuadrantCard({ t, meta, quadrant, tasks, items, count, onItem, onViewAll }) {
@@ -2115,6 +2155,8 @@ window.__ModuleLoader__.load({
 			const [threadOpen, setThreadOpen] = React.useState(false);
 			const [mode, setMode] = React.useState('discuss');
 			const [qaTask, setQaTask] = React.useState(false);
+			const [taskDefinition, setTaskDefinition] = React.useState('');
+			const [qaChecklistText, setQaChecklistText] = React.useState('');
 			const [draft, setDraft] = React.useState('');
 			const [note, setNote] = React.useState('');
 			const [rightNote, setRightNote] = React.useState('');
@@ -2513,7 +2555,12 @@ window.__ModuleLoader__.load({
 					body: JSON.stringify({ type, payload }),
 				});
 				const body = await response.json().catch(() => null);
-				if (!response.ok) throw new Error(body?.error?.message ?? ('HTTP ' + response.status));
+				if (!response.ok) {
+					const error = new Error(body?.error?.message ?? ('HTTP ' + response.status));
+					error.code = body?.error?.code;
+					error.details = body?.error?.details;
+					throw error;
+				}
 				return body;
 			}
 			function openAddAgent() {
@@ -2812,7 +2859,18 @@ window.__ModuleLoader__.load({
 				if (!project?.workspacePath) { say(t('workspaceRequired')); return; }
 				try {
 					if (mode === 'task') {
-						await runCommand('task.create', { title: text, resultDefinition: text, qaRequired: qaTask });
+						const definition = taskDefinition.trim();
+						const checklist = qaChecklistText.split(/\r?\n/).map((item) => item.trim()).filter(Boolean);
+						if (definition === '') { say(t('taskDefinitionRequired')); return; }
+						if (qaTask && checklist.length === 0) { say(t('qaNeedsChecklist')); return; }
+						await runCommand('task.create', {
+							title: text,
+							resultDefinition: definition,
+							qaRequired: qaTask,
+							...(qaTask ? { qaChecklist: checklist } : {}),
+						});
+						setTaskDefinition('');
+						setQaChecklistText('');
 					} else {
 						await runCommand('stream.post', { body: text });
 					}
@@ -2820,7 +2878,15 @@ window.__ModuleLoader__.load({
 					say(mode === 'task' ? t('taskSent') : t('discussSent'));
 					await reloadData();
 				} catch (error) {
-					say(tf(t, 'sendFailed', { message: error?.message ?? String(error) }));
+					if (error?.code === 'E_CONFLICT' && error.details?.taskId) {
+						setSelected(error.details.taskId);
+						setThreadOpen(true);
+						say(tf(t, 'duplicateTask', { number: error.details.number ?? '' }));
+					} else if (error?.message?.includes('qaChecklist')) {
+						say(t('qaNeedsChecklist'));
+					} else {
+						say(tf(t, 'sendFailed', { message: error?.message ?? String(error) }));
+					}
 				}
 			}
 			async function sendReply() {
@@ -3154,7 +3220,11 @@ window.__ModuleLoader__.load({
 							draft,
 							disabled: !project || !project.workspacePath,
 							qaRequired: qaTask,
+							definition: taskDefinition,
+							qaChecklist: qaChecklistText,
 							onQaRequired: setQaTask,
+							onDefinition: setTaskDefinition,
+							onQaChecklist: setQaChecklistText,
 							onMode: setMode,
 							onDraft: setDraft,
 							onSend: send,

package/index.js

@@ -34,7 +34,7 @@ export const name = 'ai-team';
 /** The workbench only makes sense where the Web carrier exists. */
 export const inject = ['webServer', 'permissionPresets', 'sandboxPolicy'];
 
-const VERSION = '0.3.25';
+const VERSION = '0.3.27';
 const PROJECT_ID = 'prj_main';
 const HUMAN_ID = 'mem_human';
 

package/lib/domain/policy.js

@@ -48,6 +48,12 @@ export function memberHasInProgress(tasksById, memberId) {
 	return Object.values(tasksById ?? {}).find((candidate) => candidate.assigneeId === memberId && candidate.status === 'in_progress');
 }
 
+/** A member must not start the next Task while their previous Task awaits QA. */
+export function memberHasPendingAcceptance(tasksById, memberId) {
+	if (memberId === null) return undefined;
+	return Object.values(tasksById ?? {}).find((candidate) => candidate.assigneeId === memberId && candidate.status === 'pending_acceptance');
+}
+
 /**
  * A reviewer is eligible when the task is in pending_acceptance and the
  * reviewer never claimed it (Agent) or is Human. Human self-accept is allowed

package/lib/domain/task.js

@@ -8,6 +8,7 @@ import {
 	assertRevision,
 	dependenciesDone,
 	memberHasInProgress,
+	memberHasPendingAcceptance,
 } from './policy.js';
 
 /**
@@ -103,6 +104,12 @@ export function claimTask(task, actor, tasksById, options = {}) {
 			details: { taskId: task.id, blockedBy: task.blockedBy },
 		});
 	}
+	const awaiting = memberHasPendingAcceptance(tasksById, memberId);
+	if (awaiting !== undefined) {
+		fail(ERRORS.CONFLICT, 'member has a task awaiting acceptance; wait for QA before claiming the next one', {
+			details: { memberId, taskId: awaiting.id },
+		});
+	}
 	const busy = memberHasInProgress(tasksById, memberId);
 	if (busy !== undefined) {
 		fail(ERRORS.CONFLICT, 'member already has an in_progress task', {

package/lib/host/member-tools.js

@@ -71,7 +71,8 @@ export function createTeamTool(options) {
 				ownerId: { type: 'string', description: 'Accountable owner id from members for create; defaults to you.' },
 				assigneeId: { type: 'string', description: 'Optional assignee id from members for create.' },
 				qaRequired: { type: 'boolean', description: 'Require a QA member to accept this task.' },
-				qaChecklist: { type: 'array', items: { type: 'string' }, description: 'QA checklist items; all must pass before acceptance.' },
+				qaChecklist: { type: 'array', items: { type: 'string' }, description: 'QA checklist items; all must pass before acceptance; required when qaRequired is true.' },
+				allowDuplicate: { type: 'boolean', description: 'Set true only when an identical active Task is intentionally required.' },
 				checks: {
 					type: 'array',
 					description: 'Acceptance checks for accept; include every qaChecklist item with passed=true/false.',
@@ -237,6 +238,7 @@ export function createTeamTool(options) {
 						resultDefinition: args.resultDefinition,
 						ownerId: owner.id,
 						...(assignee === null ? {} : { assigneeId: assignee.id }),
+						...(args.allowDuplicate === true ? { allowDuplicate: true } : {}),
 						...(args.qaRequired === true ? { qaRequired: true } : {}),
 						...(Array.isArray(args.qaChecklist) ? { qaChecklist: args.qaChecklist } : {}),
 					});

package/lib/host/service.js

@@ -769,17 +769,35 @@ export class TeamService {
 		if (project === undefined) fail(ERRORS.NOT_FOUND, `project ${String(projectId)} not found`);
 		const number = Math.max(0, ...Object.values(state.tasks ?? {}).map((task) => task.number ?? 0)) + 1;
 		const ownerId = payload.ownerId ?? project.ownerMemberId ?? project.ownerId;
+		const qaRequired = payload.qaRequired === true;
+		const qaChecklist = Array.isArray(payload.qaChecklist) ? payload.qaChecklist.filter((item) => typeof item === 'string' && item.trim() !== '') : [];
+		if (qaRequired && qaChecklist.length === 0) {
+			fail(ERRORS.INVALID_ARGUMENT, 'qaRequired needs at least one qaChecklist item');
+		}
+		const title = typeof payload.title === 'string' ? payload.title.trim() : '';
+		const resultDefinition = typeof payload.resultDefinition === 'string' ? payload.resultDefinition.trim() : '';
+		const duplicate = Object.values(state.tasks ?? {}).find((candidate) =>
+			candidate.tombstone !== true
+			&& ['pending', 'in_progress', 'pending_acceptance'].includes(candidate.status)
+			&& candidate.ownerId === ownerId
+			&& candidate.title === title
+			&& candidate.resultDefinition === resultDefinition);
+		if (duplicate !== undefined && payload.allowDuplicate !== true) {
+			fail(ERRORS.CONFLICT, 'an active Task with the same title, result definition and owner already exists', {
+				details: { taskId: duplicate.id, number: duplicate.number, title: duplicate.title },
+			});
+		}
 		const { task, event } = createTask({
 			number,
-			title: payload.title,
-			resultDefinition: payload.resultDefinition,
+			title,
+			resultDefinition,
 			ownerId,
 			assigneeId: payload.assigneeId,
 			blockedBy: payload.blockedBy,
 			writeScopes: payload.writeScopes,
 			irreversible: payload.irreversible,
-			qaRequired: payload.qaRequired === true,
-			qaChecklist: Array.isArray(payload.qaChecklist) ? payload.qaChecklist : [],
+			qaRequired,
+			qaChecklist,
 			originRefs: payload.originRefs,
 			at: this.now(),
 		});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@suifei/dsh-ai-team",
-  "version": "0.3.25",
+  "version": "0.3.27",
   "type": "module",
   "description": "文渊阁 (Wenyuan Pavilion) — AI Team mode for DeepSeek Harness: concurrent projects, owned tasks, threads, inbox and an append-only team ledger",
   "license": "MIT",