@suifei/dsh-ai-team 0.3.25 → 0.3.26

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 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

@@ -588,6 +588,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',
@@ -2513,7 +2517,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() {
@@ -2820,7 +2829,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() {

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.26';
 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.26",
   "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",