@suifei/dsh-ai-team 0.3.30 → 0.3.32

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## 0.3.32 — 2026-11-10
+
+- 四象限「需要你」与收件箱使用同一过滤：纯进度汇报不再计入 Q1；
+- 看板「需要你」筛选无 Task 但有真实收件箱决策时，显示「需要你处理 N 条」并一键打开收件箱，不再显示空任务；
+- Client 侧同步过滤旧 bootstrap 的 Q1，无需重启宿主也能消除截图里的 3 条假待办。
+
+## 0.3.31 — 2026-11-10
+
+- 信息性 Thread 汇报（“推进记录/备案/Human 决策收到”）不再生成 Human 决策收件箱项，避免“打开 Task 后无事可做”；
+- 收件箱只保留真正的问题：`askHuman`、报告六问、失败/需要决定；
+- 旧账本里的同类 `thread_mention` 项在 bootstrap/client 中隐藏，不再计入“待你处理”；
+- 真正决策提交后仍自动回写 Thread 并唤醒 owner，不需要 Human 再发一句话。
+
 ## 0.3.30 — 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

@@ -331,6 +331,9 @@ window.__ModuleLoader__.load({
 			filterLabel: '筛选：{name}',
 			filterClear: '清除筛选',
 			filterInboxExtra: '另有 {n} 条收件箱决策项',
+			filterInboxTitle: '需要你处理 {n} 条',
+			filterInboxBody: '这些是收件箱决策，不是 Task；去收件箱逐条处理。',
+			filterInboxAction: '打开收件箱',
 			filteredEmptyTitle: '这个筛选下没有任务',
 			filteredEmptyBody: '清除筛选后可看到全部任务。',
 			itemCount: '{n} 项',
@@ -653,6 +656,9 @@ window.__ModuleLoader__.load({
 			filterLabel: 'Filter: {name}',
 			filterClear: 'Clear filter',
 			filterInboxExtra: '{n} inbox decision item(s) not on the board',
+			filterInboxTitle: '{n} item(s) need you',
+			filterInboxBody: 'These are inbox decisions, not Tasks; open the inbox to handle them.',
+			filterInboxAction: 'Open inbox',
 			filteredEmptyTitle: 'No tasks under this filter',
 			filteredEmptyBody: 'Clear the filter to see all tasks.',
 			itemCount: '{n} items',
@@ -847,10 +853,20 @@ window.__ModuleLoader__.load({
 			};
 		};
 
+		function isInformationalInboxItem(item) {
+			return item?.ref?.category === 'thread_mention'
+				&& !(Array.isArray(item.ref?.questions) && item.ref.questions.length > 0);
+		}
+
 		/** Derive the read-only quadrants projection from the bootstrap DTO (spike fallback). */
 		function deriveQuadrants(data) {
 			const direct = data?.quadrants;
-			if (direct && direct.q1 && direct.q2 && direct.q3 && direct.q4) return direct;
+			if (direct && direct.q1 && direct.q2 && direct.q3 && direct.q4) {
+				const inboxById = new Map((data?.inbox ?? []).map((item) => [item.id, item]));
+				const q1Items = (direct.q1.items ?? []).filter((item) =>
+					item.kind !== 'inbox' || !isInformationalInboxItem(inboxById.get(item.id)));
+				return { ...direct, q1: { ...direct.q1, count: q1Items.length, items: q1Items } };
+			}
 			const tasks = data?.tasks ?? [];
 			const inbox = data?.inbox ?? [];
 			const q1 = [];
@@ -1520,7 +1536,7 @@ window.__ModuleLoader__.load({
 				})));
 		}
 
-		function Board({ t, tasks, selected, follows, doneOpen, onToggleDone, onOpen, onToggleFollow, filter, onClearFilter, q1Extra, onBackMain }) {
+		function Board({ t, tasks, selected, follows, doneOpen, onToggleDone, onOpen, onToggleFollow, filter, onClearFilter, q1Extra, onBackMain, onOpenInbox }) {
 			const filtered = (filter === 'all' ? tasks : tasks.filter((task) => matchesFilter(task, filter)))
 				.slice()
 				.sort((left, right) => (right.number ?? 0) - (left.number ?? 0));
@@ -1540,13 +1556,21 @@ window.__ModuleLoader__.load({
 							body: t('emptyBoardBody'),
 							action: { label: t('emptyBoardAction'), onClick: onBackMain },
 						})
-						: h(EmptyState, {
-							t,
-							icon: 'board',
-							title: t('filteredEmptyTitle'),
-							body: t('filteredEmptyBody'),
-							action: { label: t('filterClear'), onClick: onClearFilter },
-						}));
+						: filter === 'q1' && q1Extra > 0
+							? h(EmptyState, {
+								t,
+								icon: 'inbox',
+								title: tf(t, 'filterInboxTitle', { n: q1Extra }),
+								body: t('filterInboxBody'),
+								action: { label: t('filterInboxAction'), onClick: onOpenInbox },
+							})
+							: h(EmptyState, {
+								t,
+								icon: 'board',
+								title: t('filteredEmptyTitle'),
+								body: t('filteredEmptyBody'),
+								action: { label: t('filterClear'), onClick: onClearFilter },
+							}));
 			}
 			return h('div', { className: 'at-boardwrap', role: 'tabpanel', id: 'at-panel-board', 'aria-labelledby': 'at-tab-board' },
 				bar,
@@ -2374,9 +2398,8 @@ 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) => !isInformationalInboxItem(item));
+			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));
@@ -3108,8 +3131,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'
@@ -3172,6 +3195,7 @@ window.__ModuleLoader__.load({
 						filter: boardFilter,
 						onClearFilter: () => setBoardFilter('all'),
 						q1Extra,
+						onOpenInbox: switchInbox,
 						onBackMain: () => setTab('main'),
 					})
 					: tab === 'situation'

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

package/lib/domain/inbox.js

@@ -19,6 +19,12 @@ export const INBOX_TYPES = new Set([
 
 export const DECISION_TYPES = new Set(['decision_required']);
 
+/** Progress reports mentioning Human are not actionable decisions. */
+export function isInformationalThreadMention(item) {
+	return item?.ref?.category === 'thread_mention'
+		&& !(Array.isArray(item.ref?.questions) && item.ref.questions.length > 0);
+}
+
 /**
  * Create one inbox item. It points at a source object; it is not a copy of the
  * message body (the reader pulls the source on demand).

package/lib/host/service.js

@@ -2,7 +2,7 @@ import { existsSync, readFileSync } from 'node:fs';
 import { ERRORS, fail } from '../domain/errors.js';
 import { findReportPath, parseDecisionQuestions } from './decision-report.js';
 import { ids } from '../domain/ids.js';
-import { ackInboxItem, createInboxItem } from '../domain/inbox.js';
+import { ackInboxItem, createInboxItem, isInformationalThreadMention } from '../domain/inbox.js';
 import {
 	acceptTask,
 	assignTask,
@@ -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')
+			&& !isInformationalThreadMention(item));
 		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/lib/host/views.js

@@ -5,6 +5,8 @@
  * new events. The Host routes and the workbench both consume the same shapes.
  */
 
+import { isInformationalThreadMention } from '../domain/inbox.js';
+
 const ACTIVE_MEMBER_STATUSES = new Set(['provisioning', 'running', 'idle']);
 
 function nameOf(state, memberId) {
@@ -146,6 +148,7 @@ export function quadrantsView(state, options = {}) {
 
 	for (const item of Object.values(state.inbox ?? {})) {
 		if (item.memberId !== humanId || item.requiresDecision !== true) continue;
+		if (isInformationalThreadMention(item)) continue;
 		if (item.state === 'resolved' || item.state === 'dismissed' || item.state === 'superseded') continue;
 		q1.push({
 			id: item.id,

package/package.json

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