@zhushanwen/pi-rename-session 0.9.3 → 0.9.5

package/README.md

@@ -115,7 +115,7 @@ rename 是 best-effort 副作用，任何失败静默跳过、绝不阻断 agent
 | 11 | `skip: empty prompt` | message_end handler（`firstPrompt skip: empty prompt`） | user message 载荷无文本（理论不发生） |
 | 12 | `tool renamed to "<title>"` | rename_session 工具 execute | agent-tool 路径落库成功（`setSessionName` 之后打出，无防覆盖前缀） |
 
-另有两条**非 debug 常开**日志（无条件经 `logger.warn` 落 appendEntry entry）：`rename LLM call failed`（`logger.warn(msg, { error })` 形态，error 详情在结构化 data 字段；超时时 llm-shared callLLM 内部的 extractText 将空错误文本归一为 `unknown error`——extension 侧 `result.error ?? "unknown error"` 只兜 null/undefined，空串兜底发生在 llm-shared 层）、`model not available, skipping`（选模失败）。handler 侧日志 message 带 `t=<ISO时间>` 与 `turnIndex`；llm 侧带 `t=<ISO时间>`、无 turnIndex。
+另有三条**非 debug 常开**日志（无条件经 `logger.warn` 落 appendEntry entry，任意环境可 `grep "rename-session:log" <session.jsonl>` 排障）：`rename LLM call failed`（`logger.warn(msg, { error })` 形态，error 详情在结构化 data 字段；超时时 llm-shared callLLM 内部的 extractText 将空错误文本归一为 `unknown error`——extension 侧 `result.error ?? "unknown error"` 只兜 null/undefined，空串兜底发生在 llm-shared 层）、`model not available, skipping`（选模失败）、`title empty after clean, skipping rename`（调用成功但标题清洗后为空，结构化 data 含 `model` / `stopReason` / `raw`(≤100 码点标题候选预览)——`stopReason=length` 表示输出预算被 thinking 吃尽，是上调 `maxTokens` 常量的数据信号）。handler 侧日志 message 带 `t=<ISO时间>` 与 `turnIndex`；llm 侧带 `t=<ISO时间>`、无 turnIndex。文件日志通道（`<agentDir>/logs/`）需 `TAIJI_AGENT_DEBUG=1`，独立 pi 用户排障优先 grep session 文件（无需复跑）。
 
 ## E2E 验收
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zhushanwen/pi-rename-session",
-  "version": "0.9.3",
+  "version": "0.9.5",
   "type": "module",
   "main": "index.ts",
   "taiji": {
@@ -35,7 +35,7 @@
   "dependencies": {
     "@zhushanwen/pi-ext-guards": "0.4.1",
     "@zhushanwen/pi-extension-logger": "0.6.1",
-    "@zhushanwen/pi-llm-shared": "0.8.1"
+    "@zhushanwen/pi-llm-shared": "0.10.0"
   },
   "devDependencies": {
     "@vitest/coverage-v8": "^4.1.9",

package/src/__tests__/index.test.ts

@@ -673,14 +673,18 @@ describe("renameSessionExtension", () => {
 		for (const lit of debugLiterals) {
 			expect(warnText(warnSpy)).not.toContain(lit);
 		}
-		// 「零调用」的精确口径：剔除既有 A1 日志（model not available / call failed，
-		// 非本轮 debug 契约、本用例数据 ok:true 不触发这两类）后，其余 warn 调用必须为 0。
+		// 「零调用」的精确口径：剔除常开 A1 日志（model not available / call failed /
+		// title empty——三者为设计内的无条件 warn，非 debug 契约；本用例组 2 空内容
+		// 触发 title empty 属预期留痕）后，其余 warn 调用必须为 0。
 		// 成功路径的 rename with model 已改为 debug-only，debug 关闭时同样不得出现。
 		const nonA1Calls = warnLines(warnSpy).filter(
 			(l) =>
 				!l.includes("model not available") &&
-				!l.includes("rename LLM call failed"),
+				!l.includes("rename LLM call failed") &&
+				!l.includes("title empty after clean"),
 		);
+		// 组 2 的空内容路径恰好触发一次 title empty warn（常开留痕的存在性锚点）
+		expect(warnLines(warnSpy).filter((l) => l.includes("title empty after clean"))).toHaveLength(1);
 		expect(nonA1Calls).toHaveLength(0);
 	});
 });

package/src/__tests__/llm.test.ts

@@ -444,14 +444,30 @@ describe("callRenameLLM", () => {
 		expect(result).toBe("修复登录bug");
 	});
 
-	it("callLLM 返回空 content（cleanTitle 空串）→ 返回 null", async () => {
+	it("callLLM 返回空 content（cleanTitle 空串）→ 返回 null + logger.warn 留痕（stopReason/raw 结构化）", async () => {
 		vi.mocked(resolveModel).mockReturnValue(STUB_MODEL);
-		vi.mocked(callLLM).mockResolvedValue({ ok: true, content: "   " });
+		vi.mocked(callLLM).mockResolvedValue({ ok: true, content: "   ", stopReason: "length" });
+		const result = await callRenameLLM(createCtx(), BASE_CONFIG, FINAL_MESSAGE);
+		expect(result).toBeNull();
+		// A1 契约：空标题不再无痕——warn 带 model/stopReason/raw（截断=length 是上调 maxTokens 的数据信号）
+		expect(loggerMock.warn).toHaveBeenCalledWith(
+			"title empty after clean, skipping rename",
+			{ model: "stub/stub-model", stopReason: "length", raw: "   " },
+		);
+	});
+
+	it("callLLM 返回非空但清洗后为空（纯标点）→ warn 的 raw 携带 ≤100 码点原始预览", async () => {
+		vi.mocked(resolveModel).mockReturnValue(STUB_MODEL);
+		vi.mocked(callLLM).mockResolvedValue({ ok: true, content: "。。。。。", stopReason: "stop" });
 		const result = await callRenameLLM(createCtx(), BASE_CONFIG, FINAL_MESSAGE);
 		expect(result).toBeNull();
+		expect(loggerMock.warn).toHaveBeenCalledWith(
+			"title empty after clean, skipping rename",
+			{ model: "stub/stub-model", stopReason: "stop", raw: "。。。。。" },
+		);
 	});
 
-	it("传给 callLLM 的 opts：model/systemPrompt(<200)/maxTokens=64/timeoutMs=30000/signal/sessionId/无 tools，messages 三段式", async () => {
+	it("传给 callLLM 的 opts：model/systemPrompt(<200)/maxTokens=2048/timeoutMs=30000/signal/sessionId/无 tools，messages 三段式", async () => {
 		vi.mocked(resolveModel).mockReturnValue(STUB_MODEL);
 		vi.mocked(callLLM).mockResolvedValue({ ok: true, content: "标题" });
 		await callRenameLLM(createCtx(), BASE_CONFIG, FINAL_MESSAGE);
@@ -470,7 +486,8 @@ describe("callRenameLLM", () => {
 		expect(callOpts.model).toBe(STUB_MODEL);
 		expect(callOpts.systemPrompt).toBe(RENAME_SYSTEM_PROMPT);
 		expect(callOpts.systemPrompt.length).toBeLessThan(200);
-		expect(callOpts.maxTokens).toBe(64);
+		// 输出预算覆盖 thinking+标题（reasoning 模型 thinking 600-1500 tokens 计入预算，64 必截断）
+		expect(callOpts.maxTokens).toBe(2048);
 		// 固定 30s 超时（超时归一 ok:false 走静默跳过）
 		expect(callOpts.timeoutMs).toBe(30000);
 		expect(callOpts.signal).toBeInstanceOf(AbortSignal);

package/src/llm.ts

@@ -155,9 +155,12 @@ export function buildTitleMessages(
 
 // ──────────────────────── debug 证据链 + 超时 ────────────────────────
 
-/** rename LLM 超时（固定值：输入 ≤8k token + 输出 64 token，30s 宽裕；超时归一 ok:false 静默跳过）。 */
+/** rename LLM 超时（固定值：实测 2048 预算下独立调用 5-12.5s，30s 约 2.4x+ 余量；超时归一 ok:false 静默跳过）。 */
 const RENAME_TIMEOUT_MS = 30_000;
 
+/** title empty warn 的 raw 预览上限（Unicode 码点数；预览是 LLM 标题候选，非对话原文）。 */
+const TITLE_EMPTY_RAW_PREVIEW_CODE_POINTS = 100;
+
 /** debug 开关 live 读（每次调用查 process.env，非模块加载时读——vi.stubEnv 可测 + 运行时可切换）。 */
 export function isRenameDebugEnabled(): boolean {
 	return process.env.TAIJI_AGENT_DEBUG === "1";
@@ -295,8 +298,13 @@ export async function callRenameLLM(
 		model,
 		systemPrompt: RENAME_SYSTEM_PROMPT,
 		messages,
-		// 标题只需几个词，64 token 足够且省 quota
-		maxTokens: 64,
+		// 输出预算必须覆盖「thinking + 标题」两部分：thinking 计入 maxTokens 的模型
+		//（StepFun step-5-preview 实测 thinking 600-1500 tokens 且不可禁用/不可压低，
+		// reasoning_effort/thinking 参数均被无视）在 64 预算下正文必然截断为空。
+		// 2048 = 实测最大总输出 843 tokens 的 2.4x 余量；不给 4096+ 是因为最坏满额时长
+		//（4096 ÷ 67 tok/s ≈ 61s）远超 30s 超时线。截断发生时空标题 warn 带 stopReason=length 留痕，
+		// 该 warn 累积出现即上调本常量的数据信号。maxTokens 是截断上限非预扣额度，按实际用量计费。
+		maxTokens: 2048,
 		// 固定 30s 超时（网络抖动归一为 ok:false 走静默跳过，不悬挂 fire-and-forget promise）
 		timeoutMs: RENAME_TIMEOUT_MS,
 		// thinkingLevel 直接透传（含 "off"）；llm-shared 内部会把 "off" 映射为不传 reasoning（provider 默认）
@@ -327,6 +335,15 @@ export async function callRenameLLM(
 
 	const title = cleanTitle(result.content, config.maxTitleLength);
 	if (!title) {
+		// A1 契约（不可用不静默）：调用成功但标题为空是可排查的失败形态，不走 debugLog——
+		// warn 经 appendEntry 无条件写 session 文件（rename-session:log entry），任何环境可 grep。
+		// stopReason=length = 输出预算被 thinking 吃光（上调 maxTokens 的数据信号）；
+		// stopReason=stop = 模型正常返回空/纯标点文本（换标题模型）。raw 是 LLM 生成的标题候选非对话原文。
+		logger.warn("title empty after clean, skipping rename", {
+			model: `${model.provider}/${model.id}`,
+			stopReason: result.stopReason,
+			raw: truncateCodePoints(result.content, TITLE_EMPTY_RAW_PREVIEW_CODE_POINTS, "…"),
+		});
 		debugLog("skip: title empty");
 		return null;
 	}