@suifei/dsh-ai-team 0.3.29 → 0.3.31

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## 0.3.31 — 2026-11-10
+
+- 信息性 Thread 汇报（“推进记录/备案/Human 决策收到”）不再生成 Human 决策收件箱项，避免“打开 Task 后无事可做”；
+- 收件箱只保留真正的问题：`askHuman`、报告六问、失败/需要决定；
+- 旧账本里的同类 `thread_mention` 项在 bootstrap/client 中隐藏，不再计入“待你处理”；
+- 真正决策提交后仍自动回写 Thread 并唤醒 owner，不需要 Human 再发一句话。
+
+## 0.3.30 — 2026-11-10
+
+- 已提交的决策项增加「重新决策」：复用原六问/选项/建议重开一份未读项，可直接采纳建议后修改；
+- 看板已完成默认展开、按 Task 号倒序；
+- 决策建议逐条/全部采纳。
+
 ## 0.3.29 — 2026-11-10
 
 - 看板「已完成」默认展开，不再先折叠成「展开全部」；

package/README.md

@@ -87,7 +87,7 @@ dsh --profile ai-team --no-open --port 33271
 - **主对话**：Human 的一句话进入 Main stream，Project owner 拆 Task；顶部「团队动态」直接显示正在推进的 Task 和运行中的成员，点击 Task 可打开 Thread。
 - **看板**：四列；进行中的卡片显示 accountable owner 与执行者；数量只统计 active Task。
 - **态势**：四象限——需要你、团队在做、等待受阻、最近完成。点击条目打开对应 Task 的 Thread。
-- **收件箱**：跨 Project 聚合；顶部「待你处理 · N」一键进入；决策项直接展开全部问题与选项，可当场选择/填写并提交，不用再翻 Thread/报告。
+- **收件箱**：跨 Project 聚合；顶部「待你处理 · N」一键进入；只放真正需要 Human 的决策/失败，Agent 的进度汇报留在 Thread/动态，不制造“打开 Task 后无事可做”。
 - **直接对话**：点击成员名字直接打开该 Agent 的 DSH 原生 Session；没有 session 会先激活；打不开时落到成员资料，资料里有「直接对话」按钮。
 - **成员任务**：成员资料里点「他的任务」，查看他 owner/assignee 的 Task。
 - **成员资料**：点「⋯」改名、职责、personality、provider/model、运行配方、notes、skills、re-distill、离队/人才库。

package/client.js

@@ -118,6 +118,9 @@ window.__ModuleLoader__.load({
 			decisionAdopt: '采纳',
 			decisionAdoptAll: '全部采纳建议',
 			decisionAdoptHint: '建议可直接采纳，再按需修改',
+			decisionReopen: '重新决策',
+			decisionReopenSuffix: '复核',
+			decisionReopened: '已重开一份可编辑的决策项。',
 			decisionSubmit: '提交决策',
 			decisionSubmitted: '决策已提交给团队。',
 			inboxEmptyAction: '看任务看板',
@@ -433,6 +436,9 @@ window.__ModuleLoader__.load({
 			decisionAdopt: 'Adopt',
 			decisionAdoptAll: 'Adopt all suggestions',
 			decisionAdoptHint: 'Adopt a suggestion, then edit if needed',
+			decisionReopen: 'Decide again',
+			decisionReopenSuffix: 'review',
+			decisionReopened: 'A new editable decision item is ready.',
 			decisionSubmit: 'Submit decision',
 			decisionSubmitted: 'Decision sent to the team.',
 			inboxEmptyAction: 'Open the board',
@@ -2368,9 +2374,9 @@ window.__ModuleLoader__.load({
 			const selectedTask = tasks.find((task) => task.id === selected) ?? tasks[0] ?? null;
 			const thread = threadFor(data, selectedTask?.id);
 			const quadrants = deriveQuadrants(data);
-			const inboxUnread = Number.isFinite(data.inboxUnread)
-				? data.inboxUnread
-				: (data.inbox ?? []).filter((item) => !item.read).length;
+			const inboxItems = (data.inbox ?? []).filter((item) =>
+				!(item.ref?.category === 'thread_mention' && !(Array.isArray(item.ref?.questions) && item.ref.questions.length > 0)));
+			const inboxUnread = inboxItems.filter((item) => item.state === 'unread').length;
 			const q1Extra = (quadrants.q1.items ?? []).filter((item) => item.kind === 'inbox').length;
 			const threadHead = thread?.headSeq ?? thread?.entries?.length ?? 0;
 			const followingOf = (card) => (Object.prototype.hasOwnProperty.call(follows, card.id) ? follows[card.id] : Boolean(card.following));
@@ -2864,6 +2870,20 @@ window.__ModuleLoader__.load({
 					say(tf(t, 'sendFailed', { message: error?.message ?? String(error) }));
 				}
 			}
+			async function reopenDecision(item) {
+				try {
+					await runCommand('decision.request', {
+						title: (item.ref?.title || t('inboxDecisionTitle')) + ' · ' + t('decisionReopenSuffix'),
+						questions: item.ref?.questions ?? [],
+						taskId: item.ref?.taskId ?? null,
+						source: item.ref?.source ?? 'reopen',
+					}, item.projectId ?? projectId);
+					say(t('decisionReopened'));
+					await reloadData();
+				} catch (error) {
+					say(tf(t, 'sendFailed', { message: error?.message ?? String(error) }));
+				}
+			}
 			async function ackInbox(item) {
 				try {
 					await runCommand('inbox.ack', { itemId: item.id, disposition: 'read' }, item.projectId ?? projectId);
@@ -3088,8 +3108,8 @@ window.__ModuleLoader__.load({
 			const body = inInbox
 				? h('div', { className: 'at-stream', role: 'region', 'aria-label': t('inboxTitle') },
 					h('div', { className: 'at-streaminner' },
-						(data.inbox ?? []).length
-							? (data.inbox ?? []).map((item) => h('div', { key: item.id, className: cx('at-inboxitem', item.state !== 'unread' && 'at-inboxdone') },
+						inboxItems.length
+							? inboxItems.map((item) => h('div', { key: item.id, className: cx('at-inboxitem', item.state !== 'unread' && 'at-inboxdone') },
 								h('div', { className: 'at-inboxbody' },
 									h('div', { className: 'at-inboxtitle' },
 										item.type === 'decision_required'
@@ -3116,6 +3136,9 @@ window.__ModuleLoader__.load({
 									item.state !== 'unread' ? h('div', { className: 'at-inboxstate' }, item.state) : null),
 								h('div', { className: 'at-inboxactions' },
 									item.ref?.taskId ? h('button', { className: 'at-btn at-btn-small', type: 'button', onClick: () => openInboxTask(item) }, t('inboxOpenTask')) : null,
+									item.type === 'decision_required' && item.state !== 'unread' && Array.isArray(item.ref?.questions) && item.ref.questions.length
+										? h('button', { className: 'at-btn at-btn-small', type: 'button', onClick: () => reopenDecision(item) }, t('decisionReopen'))
+										: null,
 									item.state === 'unread' && !(item.type === 'decision_required' && Array.isArray(item.ref?.questions) && item.ref.questions.length)
 										? h('button', { className: 'at-btn at-btn-small at-btn-primary', type: 'button', onClick: () => ackInbox(item) }, t('inboxAck'))
 										: null)))

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.29';
+const VERSION = '0.3.31';
 const PROJECT_ID = 'prj_main';
 const HUMAN_ID = 'mem_human';
 

package/lib/host/service.js

@@ -124,6 +124,9 @@ export class TeamService {
 		const stream = project === undefined ? [] : (state.stream ?? []).filter((entry) => entry.projectId === project.id);
 		const selected = threadTaskId ?? tasks.find((task) => task.status === 'in_progress')?.id ?? tasks[0]?.id;
 		const thread = selected === undefined ? null : (state.threads?.[selected] ?? { taskId: selected, headSeq: 0, entries: [] });
+		const inbox = Object.values(state.inbox ?? {}).filter((item) =>
+			(item.memberId === state.humanMemberId || item.memberId === 'h1')
+			&& !(item.ref?.category === 'thread_mention' && !(Array.isArray(item.ref?.questions) && item.ref.questions.length > 0)));
 		return {
 			project: project ?? null,
 			projects,
@@ -132,7 +135,7 @@ export class TeamService {
 			tasks,
 			stream,
 			thread,
-			inbox: Object.values(state.inbox ?? {}).filter((item) => item.memberId === state.humanMemberId || item.memberId === 'h1'),
+			inbox,
 			board: project === undefined ? null : this.board(project.id),
 			quadrants: project === undefined ? null : this.quadrants(project.id),
 		};
@@ -149,11 +152,6 @@ export class TeamService {
 			?? null;
 		if (humanId === null) return 0;
 		const projectId = Object.keys(this.state.projects ?? {})[0];
-		const existing = new Set(
-			Object.values(this.state.inbox ?? {})
-				.filter((item) => item.memberId === humanId && item.ref?.entrySeq !== undefined)
-				.map((item) => `${item.ref?.taskId}:${item.ref?.entrySeq}`),
-		);
 		let reportPath = null;
 		for (const task of Object.values(this.state.tasks ?? {})) {
 			for (const entry of this.state.threads?.[task.id]?.entries ?? []) {
@@ -168,18 +166,17 @@ export class TeamService {
 				reportQuestions = parseDecisionQuestions(readFileSync(reportPath, 'utf8'));
 			} catch { /* report may be unavailable; keep the Thread fallback */ }
 		}
+		if (reportQuestions.length === 0) return 0;
 		const candidates = [];
 		for (const task of Object.values(this.state.tasks ?? {})) {
 			if (task.tombstone === true) continue;
 			for (const entry of this.state.threads?.[task.id]?.entries ?? []) {
 				const author = this.state.members?.[entry.authorId];
 				if (author?.kind !== 'agent' || !mentionsHuman(entry.body)) continue;
-				const key = `${task.id}:${entry.seq}`;
-				if (existing.has(key) && reportQuestions.length === 0) continue;
-				candidates.push({ task, entry, key });
+				candidates.push({ task, entry, key: `${task.id}:${entry.seq}` });
 			}
 		}
-		if (reportQuestions.length > 0 && candidates.length > 0) {
+		if (candidates.length > 0) {
 			const alreadyStructured = Object.values(this.state.inbox ?? {}).some((item) =>
 				item.memberId === humanId
 				&& item.ref?.source === reportPath
@@ -212,33 +209,7 @@ export class TeamService {
 			});
 			return 1;
 		}
-		let created = 0;
-		for (const candidate of candidates) {
-			const { task, entry } = candidate;
-			existing.add(candidate.key);
-			const item = createInboxItem({
-				memberId: humanId,
-				type: 'decision_required',
-				ref: {
-					taskId: task.id,
-					entrySeq: entry.seq,
-					memberId: entry.authorId,
-					memberName: this.state.members?.[entry.authorId]?.name ?? entry.authorId,
-					category: 'thread_mention',
-					title: String(entry.body).slice(0, 180),
-				},
-				requiresDecision: true,
-			}, { at: this.now() });
-			await this.ledger.append({
-				type: 'inbox/item-created',
-				payload: { itemId: item.id, ...item },
-				actor: { kind: 'system' },
-				projectId,
-				at: this.now(),
-			});
-			created += 1;
-		}
-		return created;
+		return 0;
 	}
 
 	/**
@@ -926,42 +897,6 @@ export class TeamService {
 			projectId: command.projectId,
 			at: this.now(),
 		});
-		// A Thread message that explicitly asks Human to decide must land in the
-		// Human inbox; otherwise the board looks busy while the Human never sees
-		// the question (the exact failure mode behind “对话面板没有反馈”).
-		const body = typeof payload.body === 'string' ? payload.body : '';
-		if (actorKind === 'agent' && mentionsHuman(body)) {
-			const humanId = this.state.humanMemberId
-				?? Object.values(this.state.members ?? {}).find((candidate) => candidate.kind === 'human')?.id
-				?? null;
-			const duplicate = humanId === null ? undefined : Object.values(this.state.inbox ?? {}).find((item) =>
-				item.memberId === humanId
-				&& item.ref?.taskId === taskId
-				&& item.ref?.entrySeq === outcome.entry.seq
-				&& !['resolved', 'dismissed', 'superseded'].includes(item.state));
-			if (humanId !== null && duplicate === undefined) {
-				const item = createInboxItem({
-					memberId: humanId,
-					type: 'decision_required',
-					ref: {
-						taskId,
-						entrySeq: outcome.entry.seq,
-						memberId: authorId,
-						memberName: this.state.members?.[authorId]?.name ?? authorId,
-						category: 'thread_mention',
-						title: String(body).slice(0, 180),
-					},
-					requiresDecision: true,
-				}, { at: this.now() });
-				await this.ledger.append({
-					type: 'inbox/item-created',
-					payload: { itemId: item.id, ...item },
-					actor: { kind: 'system' },
-					projectId: command.projectId,
-					at: this.now(),
-				});
-			}
-		}
 		return { thread: outcome.thread, entry: outcome.entry };
 	}
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@suifei/dsh-ai-team",
-  "version": "0.3.29",
+  "version": "0.3.31",
   "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",