opencode-acp 1.14.8-dev.3 → 1.14.8-dev.4

package/README.md

@@ -492,6 +492,33 @@ For the complete list with root cause analysis, see the [bug tracker](https://gi
 
 ## Changelog
 
+### v1.14.8-dev.4 — Dev Prerelease (1 PR since v1.14.8-dev.3)
+
+Dev prerelease covering PR #252. Published to `dev` npm tag for early testing.
+
+**PRs included**:
+- **#252** — Two fixes: (1) **Issue #251**: `filterRecommendedRanges` suppressed ALL recommendations when compressible content was below 5% of modelContextLimit (50K at 1M context). Rewrote to always show all ranges with `dangerous: true` on the last segment. Simplified `RangeFilterOptions`. (2) **Nudge loop fix**: `lastNudgeShownTokens` was reset to `undefined` on `nothingToCompress`, causing `growthReference` to fall back to stale session-start baseline → nudges firing every turn. Removed the reset. Also fixed growth display to use `lastNudgeShownTokens ?? lastPerMessageNudgeTokens` matching the actual decision logic. Oracle + Explore review: both APPROVE.
+
+**Install**: `opencode plugin opencode-acp@dev --global`
+
+### v1.14.8 — Stable Release (10 PRs since v1.14.7)
+
+Stable release promoted from dev prereleases v1.14.8-dev.1 through v1.14.8-dev.3. Includes all fixes tested in the dev channel.
+
+**Highlights**:
+
+- **System token visibility** (#241): `acp_status` and nudge breakdown now show system prompt token proportion (5-15% of context). Consolidated 4 duplicate estimation algorithms into 1 shared `estimateSystemPromptTokens()` using the real Anthropic tokenizer.
+- **HOW_TO_COMPRESS_RULES re-added to nudge** (#245): v1.14.7 over-removed the compression rules from all nudge locations. In long sessions (8,000+ messages), the rules in the system prompt degrade due to "lost in the middle" effect — re-added to nudge for high-attention guidance when compression triggers.
+- **Tool pair integrity** (#248): Compression ranges that split tool_use/tool_result pairs caused API rejections. New `adjustBoundariesForToolPairs` auto-extends boundaries by callID matching.
+- **Pluggable message filters** (#239, #242): `keepLastOnly` dedup mechanism + 4 OMO builtin filters for cleaning up third-party (e.g. OMO) injected messages that accumulate duplicates.
+- **Orphan message splicing** (#240): When `hideConsumedCompressCalls` leaves only structural parts (step-finish, reasoning), the message is now spliced out instead of surviving as a 500-2000 token orphan.
+- **E2E hardening** (#238): Observation recording, T2 cadence regression scenario, consumed-call hiding scenario, auxiliary call filtering. 12 E2E scenarios (was 8).
+- **acp_status consumed compress hiding** (#244): Hides consumed compress calls from the PROTECTED list.
+
+**All PRs included**: #238 (E2E hardening), #232 (remove dead turnProtection/DCP migration), #234 (re-add /acp stats), #239 (pluggable message filter), #240 (orphan splice), #241 (system tokens), #242 (keepLastOnly + OMO filters), #244 (acp_status consumed compress), #245 (HOW_TO_COMPRESS_RULES re-add), #248 (tool pair integrity).
+
+**Install**: `opencode plugin opencode-acp@stable --global`
+
 ### v1.14.8-dev.3 — Dev Prerelease (1 PR since v1.14.8-dev.2)
 
 Dev prerelease covering PR #248. Published to `dev` npm tag for early testing.

package/README.zh-CN.md

@@ -446,6 +446,15 @@ ACP 是 DCP 的直接替代品。迁移步骤：
 
 ## 更新日志
 
+### v1.14.8-dev.4 — Dev 预发布（v1.14.8-dev.3 以来 1 个 PR）
+
+Dev 预发布，涵盖 PR #252。发布到 `dev` npm tag 供早期测试。
+
+**包含的 PR**：
+- **#252** — 两个修复：(1) **Issue #251**：`filterRecommendedRanges` 在可压缩内容低于 modelContextLimit 的 5%（1M 上下文为 50K）时抑制所有推荐。重写为始终显示所有范围，最后一段标记 `dangerous: true`。简化 `RangeFilterOptions`。(2) **Nudge 循环修复**：`nothingToCompress` 时 `lastNudgeShownTokens` 被重置为 `undefined`，导致 `growthReference` 回退到 session 开始时的 stale baseline → 每轮都触发 nudge。移除该重置。同时修复增长显示使用 `lastNudgeShownTokens ?? lastPerMessageNudgeTokens`，与实际决策逻辑一致。Oracle + Explore Review：均通过。
+
+**安装**：`opencode plugin opencode-acp@dev --global`
+
 ### v1.14.8-dev.3 — Dev 预发布（v1.14.8-dev.2 以来 1 个 PR）
 
 Dev 预发布，涵盖 PR #248。发布到 `dev` npm tag 供早期测试。
@@ -480,6 +489,24 @@ Dev 预发布，涵盖 PR #238–#242。发布到 `dev` npm tag 供早期测试
 
 **安装**：`opencode plugin opencode-acp@dev --global`
 
+### v1.14.8 — 正式版（v1.14.7 以来 10 个 PR）
+
+正式版，从 dev 预发布 v1.14.8-dev.1 至 v1.14.8-dev.3 提升而来。包含所有在 dev 通道测试过的修复。
+
+**亮点**：
+
+- **系统 token 可见性**（#241）：`acp_status` 和 nudge breakdown 现在显示系统 prompt token 占比（上下文的 5-15%）。将 4 处重复的估算算法整合为 1 个共享 `estimateSystemPromptTokens()`，使用真正的 Anthropic 分词器。
+- **HOW_TO_COMPRESS_RULES 重新加入 nudge**（#245）：v1.14.7 过度删除了所有 nudge 位置的压缩规则。在长 session（8000+ 消息）中，system prompt 中的规则因"lost in the middle"效应衰减——重新加入 nudge 以在压缩触发时提供高注意力区指导。
+- **工具配对完整性**（#248）：压缩范围拆分 tool_use/tool_result 配对会导致 API 拒绝。新增 `adjustBoundariesForToolPairs` 按 callID 匹配自动扩展边界。
+- **可插拔消息过滤器**（#239, #242）：`keepLastOnly` 去重机制 + 4 个 OMO 内置过滤器，用于清理第三方注入消息的重复累积。
+- **孤儿消息清理**（#240）：`hideConsumedCompressCalls` 仅剩结构化部分时（step-finish、reasoning），消息现在被 splice 掉，不再作为 500-2000 token 孤儿存活。
+- **E2E 强化**（#238）：观察记录、T2 节奏回归场景、consumed-call 隐藏场景、辅助调用过滤。E2E 场景 8→12 个。
+- **acp_status 隐藏已消耗 compress**（#244）：从 PROTECTED 列表中隐藏已消耗的 compress 调用。
+
+**包含的所有 PR**：#238（E2E 强化）、#232（移除死代码 turnProtection/DCP 迁移）、#234（重新添加 /acp stats）、#239（可插拔消息过滤器）、#240（孤儿消息清理）、#241（系统 token）、#242（keepLastOnly + OMO 过滤器）、#244（acp_status 已消耗 compress）、#245（HOW_TO_COMPRESS_RULES 重新加入）、#248（工具配对完整性）。
+
+**安装**：`opencode plugin opencode-acp@stable --global`
+
 ### v1.14.7 — 去重 HOW_TO_COMPRESS_RULES（PR #228）
 
 **问题**：`HOW_TO_COMPRESS_RULES`（~1.2K tokens）每次 nudge 注入重复 3-4 次——系统提示词 1 次、nudge 模板 1-2 次、breakdown 块 1 次。非 maxLimit 场景下 suffix 消息单独就包含 2-3 份规则。每次 nudge 浪费 2.4-3.6K tokens。v1.14.6 让 debug 模式持久化 nudge 文本到聊天界面后，重复变得可见。

package/dist/index.js

@@ -7168,62 +7168,19 @@ function buildCompressibleRanges(messages, state, protectedTools = [], protected
   };
 }
 function filterRecommendedRanges(compressible, _protectedRanges, options) {
-  const { modelContextLimit, growthRatio, logger } = options;
+  const { logger } = options;
   const log = logger?.debug.bind(logger);
-  if (!modelContextLimit || modelContextLimit <= 0) {
-    log?.("filterRecommendedRanges: modelContextLimit unknown, passthrough", {
-      inputCount: compressible.length
-    });
-    if (compressible.length === 0) return [];
-    const passthrough = [...compressible];
-    passthrough[passthrough.length - 1] = {
-      ...passthrough[passthrough.length - 1],
-      dangerous: true
-    };
-    return passthrough;
-  }
   if (compressible.length === 0) {
     log?.("filterRecommendedRanges: no compressible ranges, returning empty");
     return [];
   }
-  const growthThreshold = modelContextLimit * growthRatio;
-  const lastSegmentFloor = growthThreshold * 2;
-  const lastIndex = compressible.length - 1;
-  const lastRange = compressible[lastIndex];
-  const lastSegmentIncluded = lastRange.tokens >= lastSegmentFloor;
-  let effectiveCompressible = 0;
-  const result = [];
-  for (let i = 0; i < compressible.length; i++) {
-    const r = compressible[i];
-    if (i === lastIndex) {
-      const excess = Math.max(0, r.tokens - lastSegmentFloor);
-      effectiveCompressible += excess;
-      if (lastSegmentIncluded) {
-        result.push({ ...r, dangerous: true });
-      }
-    } else {
-      effectiveCompressible += r.tokens;
-      result.push(r);
-    }
-  }
-  const suppressed = effectiveCompressible < growthThreshold;
-  log?.("filterRecommendedRanges: decision", {
+  const result = compressible.map(
+    (r, i) => i === compressible.length - 1 ? { ...r, dangerous: true } : r
+  );
+  log?.("filterRecommendedRanges: passthrough (last segment marked dangerous)", {
     inputRanges: compressible.length,
-    inputTokens: compressible.reduce((s, r) => s + r.tokens, 0),
-    growthThreshold,
-    lastSegmentFloor,
-    lastSegment: {
-      ref: `${lastRange.startRef}\u2013${lastRange.endRef}`,
-      tokens: lastRange.tokens,
-      included: lastSegmentIncluded
-    },
-    effectiveCompressible,
-    suppressed,
-    outputRanges: suppressed ? 0 : result.length
+    outputRanges: result.length
   });
-  if (suppressed) {
-    return [];
-  }
   return result;
 }
 function formatCompressibleRanges(ranges, protectedRanges) {
@@ -7537,37 +7494,24 @@ var injectCompressNudges = (state, config, logger, messages, prompts, compressio
   const recommendedRanges = filterRecommendedRanges(
     unprotectedCompressible,
     contextRanges.protected,
-    { modelContextLimit, growthRatio: 0.05, logger }
+    { logger }
   );
   const hasRecommendations = recommendedRanges.length > 0;
-  if (config.debug && contextRanges.compressible.length > 0 && modelContextLimit) {
-    const growthThreshold = modelContextLimit * 0.05;
-    const lastSegmentFloor = growthThreshold * 2;
+  if (config.debug && contextRanges.compressible.length > 0) {
     const compressible = contextRanges.compressible;
-    const lastRange = compressible[compressible.length - 1];
-    const lastIncluded = lastRange.tokens >= lastSegmentFloor;
-    const nonLastTokens = compressible.slice(0, -1).reduce((s, r) => s + r.tokens, 0);
-    const effective = nonLastTokens + Math.max(0, lastRange.tokens - lastSegmentFloor);
-    const suppressed = effective < growthThreshold;
     const fmt = (n) => n >= 1e3 ? `${(n / 1e3).toFixed(1)}K` : String(n);
     const lines = [
-      `[ACP Debug] Recommendation filter decision:`,
+      `[ACP Debug] Recommendation filter:`,
       `  Input: ${compressible.length} range(s), ${fmt(compressible.reduce((s, r) => s + r.tokens, 0))} tokens`,
-      `  Thresholds: growth=${fmt(growthThreshold)}, lastSegmentFloor=${fmt(lastSegmentFloor)}`,
-      `  Last segment: ${lastRange.startRef}\u2013${lastRange.endRef} ${fmt(lastRange.tokens)} tokens \u2192 ${lastIncluded ? "included (dangerous)" : "excluded (< floor)"}`,
-      `  Effective compressible: ${fmt(effective)} \u2192 ${suppressed ? "SUPPRESSED (< growth threshold)" : `${recommendedRanges.length} range(s) recommended`}`
+      `  Output: ${recommendedRanges.length} range(s) (last segment marked dangerous)`
     ];
     logger.debug(lines.join("\n"));
   }
-  const filterSuppressed = contextRanges.compressible.length > 0 && !hasRecommendations;
   const allProtected = contextRanges.compressible.length === 0 && contextRanges.protected.length > 0;
   const allInProtectedZone = protectedRefs.size > 0 && unprotectedCompressible.length === 0;
-  const nothingToCompress = filterSuppressed || allProtected || allInProtectedZone;
+  const nothingToCompress = allProtected || allInProtectedZone;
   const shouldInjectNudge = nudgeAllowed && (!nothingToCompress || emergencyOverride);
   let shouldInject = shouldInjectNudge;
-  if (nudgeAllowed && nothingToCompress && !emergencyOverride) {
-    state.nudges.lastNudgeShownTokens = void 0;
-  }
   if (shouldInjectNudge) {
     applyAnchoredNudges(state, config, messages, prompts, compressionPriorities, currentTokens, modelContextLimit, suffixMessage);
   }
@@ -7662,7 +7606,7 @@ ${rules}`;
     if (suffixMessage && composition.total > 0) {
       const fmt = (n) => n >= 1e3 ? `${(n / 1e3).toFixed(1)}K` : String(n);
       const pct2 = (n) => n > 0 ? Math.max(1, Math.round(n / composition.total * 100)) : 0;
-      const growth = currentTokens !== void 0 && state.nudges.lastPerMessageNudgeTokens !== void 0 ? currentTokens - state.nudges.lastPerMessageNudgeTokens : 0;
+      const growth = currentTokens !== void 0 && (state.nudges.lastNudgeShownTokens ?? state.nudges.lastPerMessageNudgeTokens) !== void 0 ? currentTokens - (state.nudges.lastNudgeShownTokens ?? state.nudges.lastPerMessageNudgeTokens) : 0;
       const growthStr = growth > 0 ? ` (+${fmt(growth)} since last nudge)` : "";
       const plainTextTokens = composition.textTokens;
       const efficiencyNote = effectiveTipsVariant !== "maxLimit" ? `
@@ -8892,7 +8836,7 @@ import { writeFile as writeFile2, mkdir as mkdir2 } from "fs/promises";
 import { join as join3 } from "path";
 import { existsSync as existsSync3 } from "fs";
 import { homedir as homedir3 } from "os";
-var LOG_VERSION = true ? "1.14.8-dev.3" : "dev";
+var LOG_VERSION = true ? "1.14.8-dev.4" : "dev";
 var Logger = class {
   logDir;
   enabled;