akm-opencode 0.9.21202610030351 → 0.9.21202610040615

package/README.md

@@ -17,7 +17,7 @@ Add the plugin to `opencode.json`:
 | Tool | Description |
 | --- | --- |
 | `akm_search` | Search configured bundles or registries. `source` accepts `local`, `registry`, `all`, or a configured bundle name. |
-| `akm_show` | Show a concept by `[bundle//]conceptId[#fragment]`. Reads are exact by default; for an opaque search fragment, `context: "lead"` adds bounded indexed-safe lead context. |
+| `akm_show` | Show a concept by `[bundle//]conceptId[#fragment]`. |
 | `akm_curate` | Return ranked concepts for a task or topic. Optional `pack` is a token budget that returns the selected local assets' full content in one response; registry hits are omitted from packed items. |
 | `akm_feedback` | Record positive or negative feedback for a concept. |
 | `akm_remember` | Save durable knowledge as a searchable memory. |
@@ -26,13 +26,7 @@ Add the plugin to `opencode.json`:
 
 Use the `ref` returned by search or curate directly with show or feedback. When `akm_curate.pack` is set, the response already includes packed local content, so a separate show call is only needed for omitted or registry-only hits. Concept IDs look like `skills/code-review`, `memories/release-retro`, or `team-playbook//knowledge/deploy#Rollback`.
 
-For `akm_show`, `context` accepts `exact` or `lead`. Exact is the default and
-preserves the existing selected-fragment behavior. For an opaque fragment ref
-returned by search, lead context puts the indexed-safe first fragment before
-an explicitly labelled selected match and uses a 3,200-character default hard
-bound. Friendly authored heading selectors retain their existing source-live
-behavior. Set either `max_tokens` (estimated at four characters per token) or
-`max_chars`, never both. Search and show
+Search and show
 responses preserve canonical `ref` while exposing `selectedRef`, `parentRef`,
 fragment ordinal/count and lines, previous/next refs, separate fragment/parent
 estimates, and context truncation metadata.
@@ -56,7 +50,7 @@ The plugin subscribes to OpenCode lifecycle events. Hook failures are logged thr
 
 The session observation buffer that retrospective feedback reads from survives every non-terminal event: it is bounded by `AKM_SESSION_BUFFER_MAX_ENTRIES` and dropped only on `session.deleted`. Discarding it at `session.idle` would empty it between turns, so "thanks, that worked" would credit nothing in exactly the sessions that used the most assets.
 
-Automatic feedback skips references that AKM reports as ineligible, and no `akm_*` tool call submits feedback on its own. A *successful* `akm_show` / `akm_search` / `akm_curate` only inspected a concept, which is not evidence that it helped, so on OpenCode the positive signal comes from a retrospective confirmation instead. A *failed* call (an error result, a timeout, an ambiguous ref) is not feedback on the asset either, and is only logged: its ref is not remembered as used, so a later "thanks, that worked" does not credit it and a later correction does not blame it. Same rules as the Claude plugin, which applies them to the `akm` subcommand of a Bash invocation.
+Automatic feedback skips references that AKM reports as ineligible, and no `akm_*` tool call submits feedback on its own. A *successful* `akm_show` / `akm_search` / `akm_curate` only inspected a concept, which is not evidence that it helped, so on OpenCode the positive signal comes from a retrospective confirmation instead. A *failed* call (an error result, a timeout, an ambiguous ref) is not feedback on the asset either, and is only logged, as a warning that names its ref and the error: its ref is not remembered as used, so a later "thanks, that worked" does not credit it and a later correction does not blame it. Same rules as the Claude plugin, which applies them to the `akm` subcommand of a Bash invocation.
 
 ### Automatic learning proposals
 
@@ -137,7 +131,7 @@ These three are read once, when the plugin module is imported, so they must be s
 
 1. Start with `akm_curate` for task-oriented discovery; set `pack` when you want the selected local content immediately.
 2. Use `akm_search` when you know the concept name and need its exact ID.
-3. Fetch a concept with `akm_show` before relying on it; opt into bounded lead context only when a selected fragment needs it.
+3. Fetch a concept with `akm_show` before relying on it.
 4. Record whether the concept helped with `akm_feedback`; a failed akm call is not feedback on the asset.
 5. Use `akm_remember` for durable knowledge that should be available to future sessions.
 

package/dist/index.js

@@ -1163,54 +1163,6 @@ function shouldRecall(prompt, options) {
   return { shouldRecall: false, reason: "skip-low-signal", query: text, scopeHints };
 }
 
-// fragment-context.ts
-function positiveSafeInteger(value, field) {
-  if (value === undefined)
-    return;
-  if (typeof value !== "number" || !Number.isSafeInteger(value) || value <= 0) {
-    throw new Error(`${field} must be a positive integer`);
-  }
-  return value;
-}
-function supportsFragmentLeadContext(version) {
-  const match = /^(\d+)\.(\d+)\.(\d+)/.exec(version.trim());
-  if (!match)
-    return false;
-  const major = Number(match[1]);
-  const minor = Number(match[2]);
-  const patch = Number(match[3]);
-  return major > 0 || minor > 9 || minor === 9 && patch >= 15;
-}
-function normalizeFragmentShowInput(input, loadedAkmApiVersion) {
-  const {
-    context,
-    max_tokens: rawMaxTokens,
-    max_chars: rawMaxChars,
-    ...showInput
-  } = input;
-  const maxTokens = positiveSafeInteger(rawMaxTokens, "max_tokens");
-  const maxChars = positiveSafeInteger(rawMaxChars, "max_chars");
-  if (maxTokens !== undefined && maxChars !== undefined) {
-    throw new Error("max_tokens and max_chars are mutually exclusive");
-  }
-  const contextMode = context;
-  if (contextMode === "lead" && !supportsFragmentLeadContext(loadedAkmApiVersion)) {
-    throw new Error(`context='lead' requires akm-cli >=0.9.15; this plugin bundles ${loadedAkmApiVersion}. ` + "Install the plugin release whose exact akm-cli pin is 0.9.15 or newer.");
-  }
-  const maxContextChars = maxChars ?? (maxTokens !== undefined ? maxTokens * 4 : undefined);
-  if (maxContextChars !== undefined && !Number.isSafeInteger(maxContextChars)) {
-    throw new Error("fragment context budget is too large");
-  }
-  if (maxContextChars !== undefined && contextMode !== "lead") {
-    throw new Error("max_tokens and max_chars require context='lead'");
-  }
-  return {
-    ...showInput,
-    ...contextMode !== undefined ? { contextMode } : {},
-    ...maxContextChars !== undefined ? { maxContextChars } : {}
-  };
-}
-
 // index.ts
 var moduleDir = path4.dirname(fileURLToPath(import.meta.url));
 var AKM_AUTO_FEEDBACK = (process.env.AKM_AUTO_FEEDBACK ?? "1") !== "0";
@@ -1790,11 +1742,10 @@ async function recordRetrospectiveFeedback(client, sessionID, text) {
   }
   retrospectiveState.set(sessionID, { recentRefs });
 }
-function queueFeedback(client, ref, sentiment, note, meta, dedupe) {
-  const dedupeKey = `${ref}:${sentiment}`;
-  if (dedupe?.has(dedupeKey))
+function queueFeedback(client, ref, note, meta, dedupe) {
+  if (dedupe?.has(ref))
     return true;
-  dedupe?.add(dedupeKey);
+  dedupe?.add(ref);
   const command = resolveAkmCommand();
   if (typeof command === "object" && "ok" in command) {
     writePluginLog(client, "warn", "AKM auto-feedback skipped", {
@@ -1803,7 +1754,6 @@ function queueFeedback(client, ref, sentiment, note, meta, dedupe) {
       sessionID: meta.sessionID,
       directory: meta.directory,
       ref,
-      sentiment,
       error: command.error
     });
     return false;
@@ -1813,7 +1763,7 @@ function queueFeedback(client, ref, sentiment, note, meta, dedupe) {
       ...command.argsPrefix,
       "feedback",
       ref,
-      sentiment === "positive" ? "--positive" : "--negative",
+      "--positive",
       "--reason",
       note,
       "--format",
@@ -1831,7 +1781,6 @@ function queueFeedback(client, ref, sentiment, note, meta, dedupe) {
           sessionID: meta.sessionID,
           directory: meta.directory,
           ref,
-          sentiment,
           error: signal ? `akm feedback exited via signal ${signal}` : `akm feedback exited with code ${code}`
         });
       }
@@ -1843,7 +1792,6 @@ function queueFeedback(client, ref, sentiment, note, meta, dedupe) {
         sessionID: meta.sessionID,
         directory: meta.directory,
         ref,
-        sentiment,
         error: formatCliError(error)
       });
     });
@@ -1856,7 +1804,6 @@ function queueFeedback(client, ref, sentiment, note, meta, dedupe) {
       sessionID: meta.sessionID,
       directory: meta.directory,
       ref,
-      sentiment,
       error: formatCliError(error)
     });
     return false;
@@ -2898,8 +2845,7 @@ async function runInProcess(client, operation, input, meta) {
     if (operation === "curate" && input.pack !== undefined && (typeof input.pack !== "number" || !Number.isInteger(input.pack) || input.pack <= 0)) {
       throw new Error("pack must be a positive integer token budget");
     }
-    const normalizedInput = operation === "show" ? normalizeFragmentShowInput(input, BUNDLED_AKM_API_VERSION) : input;
-    const result = operation === "search" ? await akmSearch(normalizedInput) : operation === "show" ? await akmShowUnified(normalizedInput) : await (async () => {
+    const result = operation === "search" ? await akmSearch(input) : operation === "show" ? await akmShowUnified(input) : await (async () => {
       const { pack, ...curateInput } = input;
       const curated = await akmCurate(curateInput);
       return typeof pack === "number" ? packCuratedHits(curated, pack) : curated;
@@ -3262,7 +3208,7 @@ ${hint}` : hint;
             });
             if (!shouldSubmitAutomaticFeedback(signal))
               continue;
-            queueFeedback(logClient, ref, "positive", signal.note, {
+            queueFeedback(logClient, ref, signal.note, {
               toolName: "chat.message",
               sessionID: input.sessionID,
               agent: input.agent
@@ -3347,6 +3293,7 @@ ${hint}` : hint;
             sessionID: input.sessionID,
             callID: input.callID,
             title: output.title,
+            refs: allRefs,
             error: typeof parsed.error === "string" ? parsed.error : undefined
           });
         }
@@ -3417,22 +3364,13 @@ ${hint}` : hint;
         }
       }),
       akm_show: tool({
-        description: "Show an AKM asset by [bundle//]conceptId[#fragment]. Reads stay exact by default. For an opaque fragment ref returned by search, context='lead' prepends bounded indexed-safe lead context and labels the selected match last; authored heading selectors retain their existing source-live behavior. Read an asset this way before relying on it, then record akm_feedback once you know whether it helped.",
+        description: "Show an AKM asset by [bundle//]conceptId[#fragment]. Read an asset this way before relying on it, then record akm_feedback once you know whether it helped.",
         args: {
           ref: tool.schema.string().describe("Asset ref returned by akm_curate or akm_search, optionally with a #fragment \u2014 e.g. `skills/code-review` or `local//knowledge/deploy#Rollback`."),
-          detail: tool.schema.enum(["brief", "summary", "normal", "full"]).optional().describe("Response detail level. Defaults to 'normal'."),
-          context: tool.schema.enum(["exact", "lead"]).optional().describe("Fragment context mode. Defaults to exact. For an opaque search fragment, use lead to prepend indexed-safe lead context while keeping the selected match last."),
-          max_tokens: tool.schema.number().optional().describe("Positive token budget for context='lead', estimated as four characters per token. Mutually exclusive with max_chars."),
-          max_chars: tool.schema.number().optional().describe("Positive character budget for context='lead'. Mutually exclusive with max_tokens. Lead defaults to 3200 characters.")
+          detail: tool.schema.enum(["brief", "summary", "normal", "full"]).optional().describe("Response detail level. Defaults to 'normal'.")
         },
-        async execute({ ref, detail, context, max_tokens, max_chars }, toolContext) {
-          return runInProcess(client, "show", {
-            ref,
-            detail,
-            context,
-            max_tokens,
-            max_chars
-          }, { toolName: "akm_show", sessionID: toolContext.sessionID, directory: toolContext.directory });
+        async execute({ ref, detail }, toolContext) {
+          return runInProcess(client, "show", { ref, detail }, { toolName: "akm_show", sessionID: toolContext.sessionID, directory: toolContext.directory });
         }
       }),
       akm_remember: tool({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "akm-opencode",
-  "version": "0.9.21202610030351",
+  "version": "0.9.21202610040615",
   "type": "module",
   "description": "OpenCode plugin for AKM with discovery tools, feedback, memory, and automatic learning proposals from corrections and recurring workflows.",
   "keywords": [