opencode-acp 1.14.17-pr.304.12 → 1.14.17-pr.305.14

package/README.md

@@ -492,13 +492,13 @@ For the complete list with root cause analysis, see the [bug tracker](https://gi
 
 ## Changelog
 
-### v1.14.17 — Eliminate compress retry dead-ends (#301)
+### v1.14.17 — CI release: dist/ artifact on every PR, npm version in artifact comment
 
-**Problem**: Two compress arguments hard-failed on first use and produced confusing error loops instead of actionable feedback. (1) The model learned "add `acknowledgeRisk: true` to retry" from quality-gate rejection templates, then carried the flag into *every* retry — including retries after non-quality errors (e.g. a missing topic), where no quality-gate rejection was pending, hitting `Parameter "acknowledgeRisk": true was provided, but no quality gate rejection is pending` (issue #301). (2) `content[0] needs a topic — provide content[0].topic or the top-level topic` hard-failed compress calls whose entries had no topic and no top-level fallback.
+**Problem**: `dist/` was only built during release publishing, so PR reviews couldn't verify the bundled output, and after publishing there was no quick way to confirm which npm version a merged PR actually landed in.
 
-**Fix**: Both parameters are now tolerant. `acknowledgeRisk` without a pending rejection is a no-op — quality checks still run (the gate's protection is unchanged: bypass is only armed by a real quality-gate rejection), and a successful result carries an `⚠️ acknowledgeRisk was ignored: ...` note teaching correct usage. Topics are fully optional: an entry without its own topic falls back to the top-level topic, then to one derived from the summary's first line (markdown headings stripped, capped at 80 chars), so `search_context` always has something searchable.
+**Fix**: CI now builds `dist/` and uploads it as a workflow artifact on every PR (#298), and the PR artifact comment displays the published npm version (#299). No plugin runtime behavior changes in this release — CI/workflow only.
 
-Files: `lib/compress/range.ts`, `lib/compress/range-utils.ts` (new `deriveFallbackTopic`), `lib/compress/quality-gate/rejection.ts` (removed `buildPreemptiveAcknowledgeError`), `lib/compress/types.ts`, `lib/prompts/compress-range.ts`, `lib/prompts/extensions/tool.ts`, `lib/state/types.ts`. Tests: 976 pass.
+Files: `.github/workflows/`. Tests: 976 pass.
 
 **Install**: `opencode plugin opencode-acp@latest --global`
 

package/README.zh-CN.md

@@ -446,13 +446,13 @@ ACP 是 DCP 的直接替代品。迁移步骤：
 
 ## 更新日志
 
-### v1.14.17 — 消除 compress 重试死循环（#301）
+### v1.14.17 — CI 版本：每个 PR 构建 dist/ 产物并在评论中显示 npm 版本
 
-**问题**：两个 compress 参数在首次使用时硬报错，产生令人困惑的错误循环而非可执行的反馈。(1) 模型从质量门拒绝模板学到「重试要加 `acknowledgeRisk: true`」，于是把它带进了所有重试——包括非质量错误（如缺 topic）之后的重试，此时没有 pending 的质量门拒绝，撞上 `Parameter "acknowledgeRisk": true was provided, but no quality gate rejection is pending`（issue #301）。(2) `content[0] needs a topic — provide content[0].topic or the top-level topic` 在 entry 无 topic 且无顶层 fallback 时硬报错。
+**问题**：\`dist/\` 只在发版发布时构建，PR review 无法核对打包产物；发布后也没有快速途径确认某个已合并 PR 实际落在哪个 npm 版本。
 
-**修复**：两个参数均改为宽容处理。无 pending 拒绝时的 `acknowledgeRisk` 为 no-op——质量检查照常运行（质量门保护不变：bypass 只由真实的质量门拒绝解锁），成功结果附带 `⚠️ acknowledgeRisk was ignored: ...` 提示教模型正确用法。topic 完全可选：entry 无自有 topic 时依次回退到顶层 topic、再到从 summary 首行派生的 topic（去 markdown 标题符、截断 80 字符），保证 `search_context` 始终有可检索内容。
+**修复**：CI 现在在每个 PR 上构建 \`dist/\` 并上传为 workflow artifact（#298），PR artifact 评论会显示已发布的 npm 版本（#299）。本版本**不含插件运行时行为变更**——仅 CI/workflow 改动。
 
-文件：`lib/compress/range.ts`、`lib/compress/range-utils.ts`（新增 `deriveFallbackTopic`）、`lib/compress/quality-gate/rejection.ts`（移除 `buildPreemptiveAcknowledgeError`）、`lib/compress/types.ts`、`lib/prompts/compress-range.ts`、`lib/prompts/extensions/tool.ts`、`lib/state/types.ts`。测试：976 通过。
+文件：\`.github/workflows/\`。测试：976 通过。
 
 **安装**：`opencode plugin opencode-acp@latest --global`
 

package/dist/index.js

@@ -2075,7 +2075,7 @@ THE FORMAT OF COMPRESS
   ]
 }
 \`\`\`
-Each entry MAY have a topic. Missing topics fall back to the top-level topic, then to an automatic one derived from the summary.`;
+Each entry needs a topic \u2014 either its own or the top-level fallback.`;
 
 // lib/state/persistence.ts
 import * as fs from "fs/promises";
@@ -3756,11 +3756,8 @@ function filterProtectedRecentMessages(selection, searchContext, state, compress
 
 // lib/compress/range-utils.ts
 var BLOCK_PLACEHOLDER_REGEX = /\(b(\d+)\)|\{block_(\d+)\}/gi;
-function deriveFallbackTopic(summary) {
-  const firstLine = summary.split("\n").map((line) => line.replace(/^#{1,6}\s*/, "").trim()).find((line) => line.length > 0) ?? "compressed range";
-  return firstLine.length > 80 ? `${firstLine.slice(0, 77)}...` : firstLine;
-}
 function validateArgs(args) {
+  const hasTopLevelTopic = typeof args.topic === "string" && args.topic.trim().length > 0;
   if (!Array.isArray(args.content) || args.content.length === 0) {
     throw new Error("content is required and must be a non-empty array");
   }
@@ -3776,12 +3773,18 @@ function validateArgs(args) {
     if (typeof entry?.summary !== "string" || entry.summary.trim().length === 0) {
       throw new Error(`${prefix}.summary is required and must be a non-empty string`);
     }
+    const hasEntryTopic = typeof entry?.topic === "string" && entry.topic.trim().length > 0;
+    if (!hasEntryTopic && !hasTopLevelTopic) {
+      throw new Error(
+        `${prefix} needs a topic \u2014 provide ${prefix}.topic or the top-level topic`
+      );
+    }
   }
 }
 function resolveRanges(args, searchContext, state, logger) {
   return args.content.map((entry, index) => {
     const normalizedEntry = {
-      topic: typeof entry.topic === "string" && entry.topic.trim().length > 0 ? entry.topic.trim() : typeof args.topic === "string" && args.topic.trim().length > 0 ? void 0 : deriveFallbackTopic(entry.summary),
+      topic: typeof entry.topic === "string" && entry.topic.trim().length > 0 ? entry.topic.trim() : void 0,
       startId: entry.startId.trim(),
       endId: entry.endId.trim(),
       summary: entry.summary
@@ -5669,6 +5672,11 @@ exact values, errors). Then add "acknowledgeRisk": true to the compress tool cal
 Without acknowledgeRisk: true, the compression will be rejected again.`;
   return new Error(message);
 }
+function buildPreemptiveAcknowledgeError() {
+  return new Error(
+    'Parameter "acknowledgeRisk": true was provided, but no quality gate rejection is pending. This parameter is only valid immediately after a compression was rejected by the quality gate. Remove it and try again.'
+  );
+}
 
 // lib/compress/pipeline.ts
 function snapshotCompressionState(state) {
@@ -6142,13 +6150,13 @@ function createCompressRangeTool(factoryCtx) {
       }
       const acknowledgeRisk = args.acknowledgeRisk === true;
       const qualityGateRetryPendingBefore = ctx.state.qualityGateRetryPending;
-      const bypassQuality = acknowledgeRisk && ctx.state.qualityGateRetryPending;
-      const ignoredAcknowledgeRisk = acknowledgeRisk && !ctx.state.qualityGateRetryPending;
-      if (ignoredAcknowledgeRisk) {
-        ctx.logger.warn("compress: acknowledgeRisk ignored \u2014 no quality gate rejection pending");
+      if (acknowledgeRisk && !ctx.state.qualityGateRetryPending) {
+        throw buildPreemptiveAcknowledgeError();
       }
-      ctx.state.qualityGateRetryPending = false;
-      if (!bypassQuality) {
+      if (acknowledgeRisk) {
+        ctx.state.qualityGateRetryPending = false;
+      } else {
+        ctx.state.qualityGateRetryPending = false;
         for (const plan of preparedPlans) {
           const result = evaluatePreCommitQuality(
             rawMessages,
@@ -6224,10 +6232,7 @@ function createCompressRangeTool(factoryCtx) {
       const skippedNote = phantomSkipNotice !== null ? `
 \u26A0\uFE0F ${phantomSkipNotice}
 ` : "";
-      const ackNote = ignoredAcknowledgeRisk ? `
-\u26A0\uFE0F acknowledgeRisk was ignored: no quality gate rejection was pending, so quality checks ran normally. Only pass it when retrying immediately after a quality gate rejection.
-` : "";
-      return `Compressed ${totalCompressedMessages} messages into ${COMPRESSED_BLOCK_HEADER}.${skippedNote}${ackNote}
+      return `Compressed ${totalCompressedMessages} messages into ${COMPRESSED_BLOCK_HEADER}.${skippedNote}
 IMPORTANT: This was an automatic context compression. You MUST continue your previous task exactly where you left off. Do NOT ask the user what to do next.
 \u{1F4A1} Tip: Use search_context('keyword') to find compressed content when you need it later.`;
     }
@@ -8944,7 +8949,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.17-pr.304.12" : "dev";
+var LOG_VERSION = true ? "1.14.17-pr.305.14" : "dev";
 var Logger = class {
   logDir;
   enabled;
@@ -9272,7 +9277,7 @@ Rules:
 BATCHING
 When multiple independent ranges are ready and their boundaries do not overlap, include all of them as separate entries in the \`content\` array of a single tool call. Each entry should have its own \`startId\`, \`endId\`, and \`summary\`.
 
-When the ranges cover unrelated topics, give each entry its own \`topic\` for better summary quality \u2014 do not force unrelated content under a single shared topic. The top-level \`topic\` is an optional fallback; when no topic is provided at all, one is derived from the summary's first line automatically.
+When the ranges cover unrelated topics, give each entry its own \`topic\` for better summary quality \u2014 do not force unrelated content under a single shared topic. Omit the top-level \`topic\` when every entry has its own. Use the top-level \`topic\` only as a fallback when entries don't specify one.
 
 \`\`\`
 compress({ content: [