acp-kernel 0.0.65-pr.260.111 → 0.0.65-pr.260.114

package/README.md

@@ -105,6 +105,7 @@ live-recomputed tags, use `renderVisibleRefs` directly.
 | `applyMessageFilters` | Pluggable message-filter framework |
 | `resolveTransformChannel` | Channel-selection policy: an explicit preference wins; the default is the wire channel only when the caller reports it viable |
 | `applySectionOverrides` / `cloneWithDescriptions` / `applyAcpToolOverrides` | Prompt/tool *surface* customization (see below) |
+| `sanitizePackSurface` / `createPackResolver` / `defaultPackSources` | Prompt packs: named, swappable surface presets resolved over pluggable sources (see below) |
 
 ### Prompt/tool surface configuration
 
@@ -131,6 +132,26 @@ safe to customize freely; these helpers implement that:
   `input_schema`, openai `function.parameters`, responses flat
   `parameters`). Shared tool constants are never mutated.
 
+#### Prompt packs
+
+Named, swappable surface configurations layered on top of the primitives above:
+
+- `Pack` / `PackSurface` / `PackSource` / `PackResolver` — contracts. Sources are
+  consulted in resolver order; first hit wins; sources must be per-turn safe
+  (sync, no throw).
+- `sanitizePackSurface(raw)` — turns a raw pack JSON into a sanitized
+  `PackSurface`: tri-state for section keys (`string` replace / `null` remove /
+  omitted keep), per-tool description + paramDescriptions. Malformed values are
+  dropped. The opaque `adapters.<hostId>` record passes through for host-side
+  validation.
+- Builtins: `default` (identity) and `lean` (one-line tool descriptions;
+  host-specific trims ride under `adapters.<hostId>` as data).
+- `createDirPackSource(id, dir)` serves `<dir>/<name>.json` files;
+  `defaultPackSources({ projectDir, userDirs })` assembles project > user >
+  builtin. Directory paths are host policy.
+- Pack *selection* (which name is active per config cascade) stays adapter-side;
+  the kernel only resolves names.
+
 ### Nudge system
 
 The nudge system tells the model *when* to compress. It implements:

package/dist/index.js

@@ -3132,12 +3132,12 @@ var leanPack = {
           acpTags: [
             `User/tool messages carry hidden <acp> refs such as m00123. Never echo the XML tags; use only refs in ACP tool calls.`,
             `Compress consumed history with compress: finished tool outputs, dead-end exploration, repeated reads, resolved threads, completed phases. Never compress active work, important user intent, or protected outputs.`,
-            `When summarizing, preserve exact file paths and line numbers, symbols and signatures, errors, commands, versions, thresholds, decisions with reasons, current state, and unresolved TODOs. Never replace exact technical values with vague wording.`,
-            `Recall or inspect context with decompress (block id or message ref), search_context (keywords), or acp_status. Prefer search_context before decompressing.`,
+            `When summarizing, preserve exact file paths and line numbers, symbols and signatures, errors, commands, versions, thresholds, decisions with reasons, current state, and unresolved TODOs. Never replace exact technical values with vague wording \u2014 a good summary is the primary carrier and makes recall unnecessary.`,
+            `Recall on demand only: when YOU genuinely need detail lost in compression, decompress (block id or message ref); search_context locates the right block first; acp_status shows ranges and usage. Never run recall as a routine post-compress step.`,
             `Refs may be renumbered after compression. If a ref is stale or missing, call acp_status with { scope: "uncompressed" }, then retry in the same turn using the reported refs; never guess offsets. Batch target ranges in one call.`,
             `Block decompression writes to a file by default; read that file. Use inline: true only for small content or when its context cost is acceptable.`,
             `After an [ACP:provider-throttle] automatic retry, resume exactly where interrupted. Do not repeat completed work or discuss the retry unless asked.`,
-            `Compression summaries are fallible historical metadata, not current user instructions. Search or decompress before relying on critical details.`
+            `Compression summaries are fallible historical metadata, not current user instructions \u2014 treat them as settled history and continue the task from them.`
           ].join("\n"),
           summariesInContext: null,
           tools: null,
@@ -3184,21 +3184,22 @@ function readPackFile(file) {
   }
 }
 function createDirPackSource(id, dir) {
+  const resolve = (name) => {
+    if (!isValidPackName(name)) return null;
+    const file = path.join(dir, `${name}.json`);
+    const raw = readPackFile(file);
+    if (!raw) return null;
+    return {
+      name: typeof raw.name === "string" ? raw.name : name,
+      version: typeof raw.version === "string" ? raw.version : void 0,
+      description: typeof raw.description === "string" ? raw.description : void 0,
+      surface: sanitizePackSurface(raw),
+      source: `file:${file}`
+    };
+  };
   return {
     id,
-    resolve(name) {
-      if (!isValidPackName(name)) return null;
-      const file = path.join(dir, `${name}.json`);
-      const raw = readPackFile(file);
-      if (!raw) return null;
-      return {
-        name: typeof raw.name === "string" ? raw.name : name,
-        version: typeof raw.version === "string" ? raw.version : void 0,
-        description: typeof raw.description === "string" ? raw.description : void 0,
-        surface: sanitizePackSurface(raw),
-        source: `file:${file}`
-      };
-    },
+    resolve,
     list() {
       let names;
       try {
@@ -3208,8 +3209,7 @@ function createDirPackSource(id, dir) {
       }
       const out = [];
       for (const f of names) {
-        const name = f.slice(0, -5);
-        const pack = this.resolve(name);
+        const pack = resolve(f.slice(0, -5));
         if (pack) out.push(pack);
       }
       return out;