auto-review 的模型调用未传 sessionId，导致每次工具审查的费用无法归属到任何会话

[Chichele]

auto-review 的模型调用未传 sessionId，导致每次工具审查的费用无法归属到任何会话

环境

项	值
包	@deepseek-ai/dsh-experimental-auto-review
版本	0.2.1-rc.2
宿主	DSH Web profile（dsh web）
OS	Linux
Node	v24.11.1
相关包	dsh-cost-meter 1.9.11（费用统计插件，走 llm/stream 瀑布）

现象

开启 Auto 权限预设（/permission auto）后，当天费用总计与「各会话费用之和」出现巨大差额：

日期	日级总计 calls	会话行 calls 之和	缺口	缺口费用占比
2026-10-19	623	359	264	75.6%（¥2.03 / ¥2.69）
2026-10-20	860	490	370	75.5%（¥2.98 / ¥3.94）

日级总计与中转站后台的真实扣费完全一致，说明这些调用真实发生、真实计费；但会话列表里完全看不到这笔开销。

根因

classifyRisk() 通过 ctx.llm.stream() 发起审查请求，options 里带了 provider / model / system / messages / temperature / signal，唯独没有 sessionId：

// packages/experimental/auto-review/src/index.ts（对应 lib/index.js:398-416）
async function classifyRisk(ctx, agent, exec, signal) {
	const snapshot = snapshotAutoReview(agent, exec, await ctx.workingDirectory.ensure(agent, signal));
	const options = deepFreeze({
		provider: snapshot.provider,
		model: snapshot.model,
		system: REVIEW_POLICY,
		messages: [{
			role: "user",
			content: [{ type: "text", text: reviewUserText(snapshot) }]
		}],
		temperature: 0,
		signal
	});                     // ← 没有 sessionId
	return readDecision(ctx.llm.stream(options));
}

任何依赖 llm/stream 的 options.sessionId 做归属的观察者，都会收到 undefined：

// dsh-cost-meter: billing-stream.js:89
account(usage, options?.model, options?.sessionId, startedAtMs, options?.provider)

于是这些用量只能进入「日/月总计」，无法进入任何会话行。这不是 cost-meter 的统计错误——它统计的每一次调用都是真实的，只是上游没有给出归属依据。

classifyRisk 已经拿到 agent，而 agent.session.id 就在手边（同一个函数的调用点 lib/index.js:464 才刚用过 agent.session）。

定量证据

我重建了每个会话的 permission/preset 事件时间线，只统计处于 auto 窗口内的 tool/call 事件，与账本缺口逐一对照：

日期	auto 窗口内 tool/call	账本缺口 calls
2026-10-19	264	264
2026-10-20	370	370
合计	634	634

逐日精确相等，一个不差。 每次工具调用前审查一次（tools/pre-execute 钩子），审查次数与账本缺口完全一致。

token 特征佐证

指标	实测缺口	auto-review 的预期形态
平均 input / 次	27,577	REVIEW_POLICY 20,470 字符 ≈ 5–7k token，叠加 FILTERED_HISTORY 完整历史快照
平均 output / 次	118	一个短 JSON：{"risk":"low","decision":"allow"}
缓存命中率	3.2%	每次历史不同，前缀无法复用

时间线佐证

10-19 10:28  session-8fb408f3 → auto
10-19 13:01  session-6b6d261f → auto
10-19 14:13  两者 → danger-full-access        （auto 窗口结束）

10-20 10:57  session-8cbf299f → auto
10-20 11:06  session-9216082c → auto
10-20 11:25  session-34253e6e → auto
10-20 13:00~13:34  三者陆续退出 auto

缺口严格对应以上 auto 窗口，窗口外的工具调用（10-19 的 155 次、10-20 的 233 次）没有产生任何缺口。

反向验证

把权限从 auto 切到 workspace-write（approval: ask）后，对同一账本连续采样：

时刻	day.calls	sessions.calls	缺口
13:35	867	497	370
13:40	881	511	370

两者各增长 14，缺口恒定不再扩大——审查停止后，未归属用量即刻停止产生。

影响

1. 费用归属失真：开启 Auto 时，实际有 75% 的当日费用不出现在任何会话里。用户按会话排查开销时会产生严重误判（我最初就因此误以为是统计 bug）。
2. 用量统计不可核对：日级与会话行长期不一致，任何按会话聚合的分析（预算、成本分摊、按项目核算）都会系统性偏低。
3. 影响范围不限于 cost-meter：所有通过 llm/stream 的 options.sessionId 做归属的观察者（计费、审计、限流、遥测）都会受影响。

建议修复

在 classifyRisk 的 options 中补上 sessionId：

 const options = deepFreeze({
 	provider: snapshot.provider,
 	model: snapshot.model,
+	sessionId: agent.session.id,
 	system: REVIEW_POLICY,
 	messages: [{
 		role: "user",
 		content: [{ type: "text", text: reviewUserText(snapshot) }]
 	}],
 	temperature: 0,
 	signal
 });

agent 已是 classifyRisk 的入参，agent.session.id 在同文件第 464 行已被使用，无需新增依赖。

若上游认为「审查调用不应算进被审查会话的账」，建议同时在 llm/stream 的 options 中提供一个显式的归属或标记字段（例如 attribution: 'auto-review'），让下游能把这类后台消耗单独归类，而不是变成无主的日级残差。关键是不能留 undefined——静默丢失归属比显式标注更糟。

补充说明

• 该问题在 approval: never + preset: auto 的组合下才会触发（lib/index.js:464 的 permissionPresets.current(agent.session) !== AUTO_PRESET 直接 return next()），因此关闭 Auto 即可规避。
• 这不是 dsh-cost-meter 的缺陷。它的日级数字与中转站账单一致，会话行也只统计了有归属依据的调用，两边都没算错；缺失的是上游的归属信息。

[PerryLink]

这个字段是有的，只是这条调用没传。

packages/llm/llm/src/types.ts:561-564 里 GenerateOptions 明确带 session 标记：

/**
 * Session identity stamped by the loop for request routing. Replay uses it
 * ...
 */
sessionId?: Branded<'SessionId'>

而 auto-review 构造请求时只给了 provider、model、system、messages、temperature、signal：

// packages/experimental/auto-review/src/index.ts:624-635
const options: GenerateOptions = deepFreeze({
  provider: snapshot.provider,
  model: snapshot.model,
  system: REVIEW_POLICY,
  messages: [{ role: 'user', content: [{ type: 'text', text: reviewUserText(snapshot) }] }],
  temperature: 0,
  signal,
})

:623 的注释也把这件事写明了：这段 review prompt 只走 ctx.llm.stream，从不进入 Session log。两件事合起来就是你看到的结果——请求发出去了、token 花了，但既没有 sessionId 也没有日志，费用在账单上无法归属到任何会话。

所以修法很小：在 :624 把 sessionId 补上（classifyRisk 手上已经有 agent，:617），让路由和计费有归属。另外还有一处：types.ts:570 的 purpose 目前只允许 'compaction' | 'session-title'，如果希望 review 的消耗在账单/统计里可区分，得把这一档也扩进去，否则即使补了 sessionId，它仍然会被算作普通会话请求。

一个前提需要确认，我没有核到：计费侧是不是真的只认 GenerateOptions.sessionId。如果计费是走另一条扣费标记（比如 provider 层的 key），补这个字段不会自动修好归属。但"这条调用没带 sessionId、也不进 session log"这两点是可以从上面两处直接读出来的。