opencode-acp 1.14.16-pr.302.9 → 1.14.16-pr.303.11

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
@@ -8944,7 +8947,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.16-pr.302.9" : "dev";
+var LOG_VERSION = true ? "1.14.16-pr.303.11" : "dev";
 var Logger = class {
   logDir;
   enabled;
@@ -9272,7 +9275,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: [