opencode-acp 1.0.1 → 1.1.0

package/dist/index.js

@@ -862,20 +862,7 @@ var ParseErrorCode;
   ParseErrorCode2[ParseErrorCode2["InvalidCharacter"] = 16] = "InvalidCharacter";
 })(ParseErrorCode || (ParseErrorCode = {}));
 
-// lib/config.ts
-var DEFAULT_PROTECTED_TOOLS = [
-  "task",
-  "skill",
-  "todowrite",
-  "todoread",
-  "compress",
-  "batch",
-  "plan_enter",
-  "plan_exit",
-  "write",
-  "edit"
-];
-var COMPRESS_DEFAULT_PROTECTED_TOOLS = ["task", "skill", "todowrite", "todoread"];
+// lib/config-validation.ts
 var VALID_CONFIG_KEYS = /* @__PURE__ */ new Set([
   "$schema",
   "enabled",
@@ -1267,7 +1254,7 @@ function validateConfigTypes(config) {
       }
       if (gc.majorGcThresholdPercent !== void 0) {
         const isValidNumber = typeof gc.majorGcThresholdPercent === "number";
-        const isPercentString = typeof gc.majorGcThresholdPercent === "string" && gc.majorGcThresholdPercent.endsWith("%");
+        const isPercentString = typeof gc.majorGcThresholdPercent === "string" && /^\d+(?:\.\d+)?%$/.test(gc.majorGcThresholdPercent);
         if (!isValidNumber && !isPercentString) {
           errors.push({
             key: "gc.majorGcThresholdPercent",
@@ -1279,54 +1266,97 @@ function validateConfigTypes(config) {
     }
   }
   const strategies = config.strategies;
-  if (strategies) {
-    if (strategies.deduplication?.enabled !== void 0 && typeof strategies.deduplication.enabled !== "boolean") {
+  if (strategies !== void 0) {
+    if (typeof strategies !== "object" || strategies === null || Array.isArray(strategies)) {
       errors.push({
-        key: "strategies.deduplication.enabled",
-        expected: "boolean",
-        actual: typeof strategies.deduplication.enabled
-      });
-    }
-    if (strategies.deduplication?.protectedTools !== void 0 && !Array.isArray(strategies.deduplication.protectedTools)) {
-      errors.push({
-        key: "strategies.deduplication.protectedTools",
-        expected: "string[]",
-        actual: typeof strategies.deduplication.protectedTools
+        key: "strategies",
+        expected: "object",
+        actual: typeof strategies
       });
-    }
-    if (strategies.purgeErrors) {
-      if (strategies.purgeErrors.enabled !== void 0 && typeof strategies.purgeErrors.enabled !== "boolean") {
-        errors.push({
-          key: "strategies.purgeErrors.enabled",
-          expected: "boolean",
-          actual: typeof strategies.purgeErrors.enabled
-        });
-      }
-      if (strategies.purgeErrors.turns !== void 0 && typeof strategies.purgeErrors.turns !== "number") {
-        errors.push({
-          key: "strategies.purgeErrors.turns",
-          expected: "number",
-          actual: typeof strategies.purgeErrors.turns
-        });
-      }
-      if (typeof strategies.purgeErrors.turns === "number" && strategies.purgeErrors.turns < 1) {
-        errors.push({
-          key: "strategies.purgeErrors.turns",
-          expected: "positive number (>= 1)",
-          actual: `${strategies.purgeErrors.turns} (will be clamped to 1)`
-        });
+    } else {
+      const dedup = strategies.deduplication;
+      if (dedup !== void 0) {
+        if (typeof dedup !== "object" || dedup === null || Array.isArray(dedup)) {
+          errors.push({
+            key: "strategies.deduplication",
+            expected: "object",
+            actual: typeof dedup
+          });
+        } else {
+          if (dedup.enabled !== void 0 && typeof dedup.enabled !== "boolean") {
+            errors.push({
+              key: "strategies.deduplication.enabled",
+              expected: "boolean",
+              actual: typeof dedup.enabled
+            });
+          }
+          if (dedup.protectedTools !== void 0 && !Array.isArray(dedup.protectedTools)) {
+            errors.push({
+              key: "strategies.deduplication.protectedTools",
+              expected: "string[]",
+              actual: typeof dedup.protectedTools
+            });
+          }
+        }
       }
-      if (strategies.purgeErrors.protectedTools !== void 0 && !Array.isArray(strategies.purgeErrors.protectedTools)) {
-        errors.push({
-          key: "strategies.purgeErrors.protectedTools",
-          expected: "string[]",
-          actual: typeof strategies.purgeErrors.protectedTools
-        });
+      const purge = strategies.purgeErrors;
+      if (purge !== void 0) {
+        if (typeof purge !== "object" || purge === null || Array.isArray(purge)) {
+          errors.push({
+            key: "strategies.purgeErrors",
+            expected: "object",
+            actual: typeof purge
+          });
+        } else {
+          if (purge.enabled !== void 0 && typeof purge.enabled !== "boolean") {
+            errors.push({
+              key: "strategies.purgeErrors.enabled",
+              expected: "boolean",
+              actual: typeof purge.enabled
+            });
+          }
+          if (purge.turns !== void 0 && typeof purge.turns !== "number") {
+            errors.push({
+              key: "strategies.purgeErrors.turns",
+              expected: "number",
+              actual: typeof purge.turns
+            });
+          }
+          if (typeof purge.turns === "number" && purge.turns < 1) {
+            errors.push({
+              key: "strategies.purgeErrors.turns",
+              expected: "positive number (>= 1)",
+              actual: `${purge.turns} (will be clamped to 1)`
+            });
+          }
+          if (purge.protectedTools !== void 0 && !Array.isArray(purge.protectedTools)) {
+            errors.push({
+              key: "strategies.purgeErrors.protectedTools",
+              expected: "string[]",
+              actual: typeof purge.protectedTools
+            });
+          }
+        }
       }
     }
   }
   return errors;
 }
+
+// lib/config.ts
+var DEFAULT_PROTECTED_TOOLS = [
+  "task",
+  "skill",
+  "todowrite",
+  "todoread",
+  "compress",
+  "batch",
+  "plan_enter",
+  "plan_exit",
+  "write",
+  "edit"
+];
+var COMPRESS_DEFAULT_PROTECTED_TOOLS = ["task", "skill", "todowrite", "todoread"];
 function showConfigWarnings(ctx, configPath, configData, isProject) {
   const invalidKeys = getInvalidConfigKeys(configData);
   const typeErrors = validateConfigTypes(configData);
@@ -1389,10 +1419,10 @@ var defaultConfig = {
   compress: {
     mode: "range",
     permission: "allow",
-    showCompression: false,
+    showCompression: true,
     summaryBuffer: true,
-    maxContextLimit: 1e5,
-    minContextLimit: 5e4,
+    maxContextLimit: "55%",
+    minContextLimit: "45%",
     nudgeFrequency: 5,
     iterationNudgeThreshold: 15,
     nudgeForce: "soft",
@@ -1416,7 +1446,7 @@ var defaultConfig = {
     promotionThreshold: 5,
     maxBlockAge: 15,
     maxOldGenSummaryLength: 3e3,
-    majorGcThresholdPercent: "55%"
+    majorGcThresholdPercent: "100%"
   }
 };
 var GLOBAL_CONFIG_DIR = process.env.XDG_CONFIG_HOME ? join(process.env.XDG_CONFIG_HOME, "opencode") : join(homedir(), ".config", "opencode");
@@ -1901,8 +1931,8 @@ THE FORMAT OF COMPRESS
   topic: string,           // Short label (3-5 words) - e.g., "Auth System Exploration"
   content: [               // One or more ranges to compress
     {
-      startId: string,     // Boundary ID at range start: mNNNN or bN
-      endId: string,       // Boundary ID at range end: mNNNN or bN
+      startId: string,     // Boundary ID at range start: mNNNNN or bN
+      endId: string,       // Boundary ID at range end: mNNNNN or bN
       summary: string      // Complete technical summary replacing all content in range
     }
   ]
@@ -1916,7 +1946,7 @@ THE FORMAT OF COMPRESS
   topic: string,           // Short label (3-5 words) for the overall batch
   content: [               // One or more messages to compress independently
     {
-      messageId: string,   // Raw message ID only: mNNNN (ignore metadata attributes like priority)
+      messageId: string,   // Raw message ID only: mNNNNN (ignore metadata attributes like priority)
       topic: string,       // Short label (3-5 words) for this one message summary
       summary: string      // Complete technical summary replacing that one message
     }
@@ -1925,16 +1955,16 @@ THE FORMAT OF COMPRESS
 \`\`\``;
 
 // lib/message-ids.ts
-var MESSAGE_REF_REGEX = /^m(\d{4})$/;
+var MESSAGE_REF_REGEX = /^m(\d{4,5})$/;
 var BLOCK_REF_REGEX = /^b([1-9]\d*)$/;
 var MESSAGE_ID_TAG_NAME = "dcp-message-id";
-var MESSAGE_REF_WIDTH = 4;
+var MESSAGE_REF_WIDTH = 5;
 var MESSAGE_REF_MIN_INDEX = 1;
-var MESSAGE_REF_MAX_INDEX = 9999;
+var MESSAGE_REF_MAX_INDEX = 99999;
 function formatMessageRef(index) {
   if (!Number.isInteger(index) || index < MESSAGE_REF_MIN_INDEX || index > MESSAGE_REF_MAX_INDEX) {
     throw new Error(
-      `Message ID index out of bounds: ${index}. Supported range is 0-${MESSAGE_REF_MAX_INDEX}.`
+      `Message ID index out of bounds: ${index}. Supported range is ${MESSAGE_REF_MIN_INDEX}-${MESSAGE_REF_MAX_INDEX}.`
     );
   }
   return `m${index.toString().padStart(MESSAGE_REF_WIDTH, "0")}`;
@@ -2089,10 +2119,10 @@ function resolveBoundaryIds(context, state, startId, endId) {
   const parsedStartId = parseBoundaryId(startId);
   const parsedEndId = parseBoundaryId(endId);
   if (parsedStartId === null) {
-    issues.push("startId is invalid. Use an injected message ID (mNNNN) or block ID (bN).");
+    issues.push("startId is invalid. Use an injected message ID (mNNNNN) or block ID (bN).");
   }
   if (parsedEndId === null) {
-    issues.push("endId is invalid. Use an injected message ID (mNNNN) or block ID (bN).");
+    issues.push("endId is invalid. Use an injected message ID (mNNNNN) or block ID (bN).");
   }
   if (issues.length > 0) {
     throw new Error(
@@ -2547,16 +2577,16 @@ var ISSUE_TEMPLATES = {
     "refer to protected messages and cannot be compressed."
   ],
   "invalid-format": [
-    "is invalid. Use an injected raw message ID of the form mNNNN.",
-    "are invalid. Use injected raw message IDs of the form mNNNN."
+    "is invalid. Use an injected raw message ID of the form mNNNNN.",
+    "are invalid. Use injected raw message IDs of the form mNNNNN."
   ],
   "block-id": [
-    "is invalid here. Block IDs like bN are not allowed; use an mNNNN message ID instead.",
-    "are invalid here. Block IDs like bN are not allowed; use mNNNN message IDs instead."
+    "is invalid here. Block IDs like bN are not allowed; use an mNNNNN message ID instead.",
+    "are invalid here. Block IDs like bN are not allowed; use mNNNNN message IDs instead."
   ],
   "not-in-context": [
-    "is not available in the current conversation context. Choose an injected mNNNN ID visible in context.",
-    "are not available in the current conversation context. Choose injected mNNNN IDs visible in context."
+    "is not available in the current conversation context. Choose an injected mNNNNN ID visible in context.",
+    "are not available in the current conversation context. Choose injected mNNNNN IDs visible in context."
   ],
   protected: [
     "refers to a protected message and cannot be compressed.",
@@ -2923,6 +2953,11 @@ function resetOnCompaction(state) {
     turnNudgeAnchors: /* @__PURE__ */ new Set(),
     iterationNudgeAnchors: /* @__PURE__ */ new Set()
   };
+  state.messageIds = {
+    byRawId: /* @__PURE__ */ new Map(),
+    byRef: /* @__PURE__ */ new Map(),
+    nextRef: 1
+  };
 }
 
 // lib/state/persistence.ts
@@ -3295,6 +3330,17 @@ async function ensureSessionInitialized(client, state, sessionId, logger, messag
         state.messageIds.byRef.delete(ref);
       }
     }
+    for (const [rawId, oldRef] of state.messageIds.byRawId) {
+      const parsed = parseMessageRef(oldRef);
+      if (parsed !== null) {
+        const newRef = formatMessageRef(parsed);
+        if (newRef !== oldRef) {
+          state.messageIds.byRawId.set(rawId, newRef);
+          state.messageIds.byRef.delete(oldRef);
+          state.messageIds.byRef.set(newRef, rawId);
+        }
+      }
+    }
   }
   if (persistedAny._persistedLastCompaction !== void 0) {
     state.lastCompaction = Math.max(state.lastCompaction, persistedAny._persistedLastCompaction);
@@ -4284,7 +4330,7 @@ function buildSchema() {
     ),
     content: tool.schema.array(
       tool.schema.object({
-        messageId: tool.schema.string().describe("Raw message ID to compress (e.g. m0001)"),
+        messageId: tool.schema.string().describe("Raw message ID to compress (e.g. m00001)"),
         topic: tool.schema.string().describe("Short label (3-5 words) for this one message summary"),
         summary: tool.schema.string().describe("Complete technical summary replacing that one message")
       })
@@ -4521,9 +4567,9 @@ function buildSchema2() {
     content: tool2.schema.array(
       tool2.schema.object({
         startId: tool2.schema.string().describe(
-          "Message or block ID marking the beginning of range (e.g. m0001, b2)"
+          "Message or block ID marking the beginning of range (e.g. m00001, b2)"
         ),
-        endId: tool2.schema.string().describe("Message or block ID marking the end of range (e.g. m0012, b5)"),
+        endId: tool2.schema.string().describe("Message or block ID marking the end of range (e.g. m00012, b5)"),
         summary: tool2.schema.string().describe("Complete technical summary replacing all content in range")
       })
     ).describe(
@@ -4978,7 +5024,7 @@ Compressed block sections in context are clearly marked with a header:
 
 - \`[Compressed conversation section]\`
 
-Compressed block IDs always use the \`bN\` form (never \`mNNNN\`) and are represented in the same XML metadata tag format.
+Compressed block IDs always use the \`bN\` form (never \`mNNNNN\`) and are represented in the same XML metadata tag format.
 
 Rules:
 
@@ -5001,7 +5047,7 @@ When you use compressed block placeholders, write the surrounding summary text s
 BOUNDARY IDS
 You specify boundaries by ID using the injected IDs visible in the conversation:
 
-- \`mNNNN\` IDs identify raw messages
+- \`mNNNNN\` IDs identify raw messages
 - \`bN\` IDs identify previously compressed blocks
 
 Each message has an ID inside XML metadata tags like \`<dcp-message-id>...</dcp-message-id>\`.
@@ -5036,11 +5082,11 @@ If a message contains no significant technical decisions, code changes, or user
 MESSAGE IDS
 You specify individual raw messages by ID using the injected IDs visible in the conversation:
 
-- \`mNNNN\` IDs identify raw messages
+- \`mNNNNN\` IDs identify raw messages
 
 Each message has an ID inside XML metadata tags like \`<dcp-message-id priority="high">m0007</dcp-message-id>\`.
 The same ID tag appears in every tool output of the message it belongs to \u2014 each unique ID identifies one complete message.
-Treat these tags as message metadata only, not as content to summarize. Use only the inner \`mNNNN\` value as the \`messageId\`.
+Treat these tags as message metadata only, not as content to summarize. Use only the inner \`mNNNNN\` value as the \`messageId\`.
 The \`priority\` attribute indicates relative context cost. You MUST compress high-priority messages when their full text is no longer necessary for the active task.
 If prior compress-tool results are present, always compress and summarize them minimally only as part of a broader compression pass. Do not invoke the compress tool solely to re-compress an earlier compression result.
 Messages marked as \`<dcp-message-id>BLOCKED</dcp-message-id>\` cannot be compressed.
@@ -5048,8 +5094,8 @@ Messages marked as \`<dcp-message-id>BLOCKED</dcp-message-id>\` cannot be compre
 Rules:
 
 - Pick each \`messageId\` directly from injected IDs visible in context.
-- Only use raw message IDs of the form \`mNNNN\`.
-- Ignore XML attributes such as \`priority\` when copying the ID; use only the inner \`mNNNN\` value.
+- Only use raw message IDs of the form \`mNNNNN\`.
+- Ignore XML attributes such as \`priority\` when copying the ID; use only the inner \`mNNNNN\` value.
 - Do not invent IDs. Use only IDs that are present in context.
 
 BATCHING