opencode-muse 0.1.1 → 0.1.2

package/README.md

@@ -16,7 +16,7 @@ Add the published package to your project-level `opencode.jsonc` (or
 ```jsonc
 {
   "$schema": "https://opencode.ai/config.json",
-  "plugins": ["opencode-muse@0.1.1"],
+  "plugins": ["opencode-muse@0.1.2"],
 }
 ```
 
@@ -104,6 +104,16 @@ setup placeholder rather than advertising a model that will fail on use.
   tested. The plugin also allows only one active writer per workspace, so a
   new turn may be reported as busy while another is still finishing.
 
+## Troubleshooting
+
+- **"Muse Spark 1.3 Contributor" never connects after a model switch:** that
+  entry is a retired OpenCode-gateway model (`opencode/muse-spark-1.3-contributor-free`),
+  not this plugin's model. It no longer resolves anywhere, so OpenCode keeps
+  retrying and the session appears stuck on "connecting". Pick the model under
+  provider **Muse Code** — `muse/muse-profile-tbh-model-muse-spark-1.3` —
+  instead of any historical "contributor-free" entry in recent-model lists. See
+  [#26](https://github.com/pgquiles/opencode-muse/issues/26).
+
 ## Credits and references
 
 - [opencode-claude](https://github.com/openchamber/opencode-claude) — a

package/dist/model/language-model.js

@@ -1,5 +1,6 @@
 import { createHash } from "node:crypto";
 import { MspCommandRejectedError, MuseBridgeError, MuseTranscriptDivergedError, MuseUnsupportedInputError, MuseUnsupportedRequestError, MspProtocolError, MspTransportError } from "../errors.js";
+import { notificationCursorKey } from "../msp/notification.js";
 import { MUSE_CONTEXT_HEADERS, readRequestContext } from "./context.js";
 import { classifyMetaRequest, makeCompactionDigest, makeLocalTitle } from "./meta.js";
 import { promptContainsCompactionFingerprint } from "../session/fingerprint.js";
@@ -93,7 +94,7 @@ function eventTurnId(notification, itemTurnIds) {
 async function* pageGapNotifications(host, sessionId, gap) {
     let cursor = gap.after;
     const requestedCursors = new Set([gap.after]);
-    const eventCursors = new Set();
+    const eventKeys = new Set();
     for (let pageNumber = 0; pageNumber < 100; pageNumber += 1) {
         const requestCursor = cursor;
         const page = await host.request("view/page", {
@@ -104,6 +105,7 @@ async function* pageGapNotifications(host, sessionId, gap) {
         });
         if (!Array.isArray(page.events))
             throw new MspProtocolError("view/page");
+        let boundaryReached = false;
         for (const event of page.events) {
             if (typeof event !== "object" || event === null)
                 throw new MspProtocolError("view/page");
@@ -112,14 +114,20 @@ async function* pageGapNotifications(host, sessionId, gap) {
             if (typeof record.method !== "string" || !params || params.sessionId !== sessionId || typeof params.viewCursor !== "string") {
                 throw new MspProtocolError("view/page");
             }
-            if (params.viewCursor === gap.next)
-                return;
-            if (eventCursors.has(params.viewCursor))
-                throw new MspProtocolError("view/page");
-            eventCursors.add(params.viewCursor);
+            const atBoundary = params.viewCursor === gap.next;
+            if (atBoundary)
+                boundaryReached = true;
             cursor = params.viewCursor;
-            yield { jsonrpc: "2.0", method: record.method, params };
+            const notification = { jsonrpc: "2.0", method: record.method, params };
+            const key = notificationCursorKey(notification);
+            if (key && eventKeys.has(key))
+                throw new MspProtocolError("view/page");
+            if (key)
+                eventKeys.add(key);
+            yield notification;
         }
+        if (boundaryReached)
+            return;
         if (page.nextCursor === null || typeof page.nextCursor !== "string"
             || page.nextCursor === requestCursor || requestedCursors.has(page.nextCursor)) {
             throw new MspProtocolError("view/page");
@@ -196,7 +204,7 @@ export async function runMuseTurn(context, input, options, runtime, transcriptFi
         const events = {
             [Symbol.asyncIterator]: async function* () {
                 const itemTurnIds = new Map();
-                const seenCursors = new Set();
+                const seenNotifications = new Set();
                 const processNotification = async function* (notification) {
                     const params = notification.params ?? {};
                     if (params.sessionId !== session.sessionId)
@@ -227,11 +235,11 @@ export async function runMuseTurn(context, input, options, runtime, transcriptFi
                             if (typeof params.after !== "string" || typeof params.next !== "string")
                                 throw new MspProtocolError("view/gap");
                             for await (const replay of pageGapNotifications(session.host, session.sessionId, { after: params.after, next: params.next })) {
-                                const replayCursor = replay.params?.viewCursor;
-                                if (typeof replayCursor === "string" && seenCursors.has(replayCursor))
+                                const replayKey = notificationCursorKey(replay);
+                                if (replayKey && seenNotifications.has(replayKey))
                                     continue;
-                                if (typeof replayCursor === "string")
-                                    seenCursors.add(replayCursor);
+                                if (replayKey)
+                                    seenNotifications.add(replayKey);
                                 for await (const output of processNotification(replay))
                                     yield output;
                                 if (terminalSeen)
@@ -239,13 +247,13 @@ export async function runMuseTurn(context, input, options, runtime, transcriptFi
                             }
                         }
                         else {
-                            const viewCursor = params.viewCursor;
-                            if (typeof viewCursor === "string" && seenCursors.has(viewCursor)) {
+                            const notificationKey = notificationCursorKey(notification);
+                            if (notificationKey && seenNotifications.has(notificationKey)) {
                                 next = await subscription.next();
                                 continue;
                             }
-                            if (typeof viewCursor === "string")
-                                seenCursors.add(viewCursor);
+                            if (notificationKey)
+                                seenNotifications.add(notificationKey);
                             for await (const output of processNotification(notification))
                                 yield output;
                             if (terminalSeen)

package/dist/model/stream.d.ts

@@ -14,7 +14,7 @@ export declare class MuseStreamMapper {
     private readonly reasoningStarted;
     private readonly itemDeltaCounts;
     private readonly itemText;
-    private readonly seenUsageCursors;
+    private readonly seenUsageNotifications;
     private readonly textParts;
     private terminalUsage;
     private callUsage;

package/dist/model/stream.js

@@ -1,4 +1,5 @@
 import { MspProtocolError, MuseTurnFailedError, MuseTurnUnqueuedError, } from "../errors.js";
+import { notificationCursorKey } from "../msp/notification.js";
 function isNumber(value) {
     return typeof value === "number" && Number.isFinite(value) && value >= 0;
 }
@@ -17,7 +18,7 @@ export class MuseStreamMapper {
     reasoningStarted = new Set();
     itemDeltaCounts = new Map();
     itemText = new Map();
-    seenUsageCursors = new Set();
+    seenUsageNotifications = new Set();
     textParts = [];
     terminalUsage;
     callUsage = { inputTokens: 0, outputTokens: 0, totalTokens: 0, reasoningTokens: 0 };
@@ -168,9 +169,10 @@ export class MuseStreamMapper {
     tokenUsageEvent(params) {
         if (params.turnId !== this.turnId || typeof params.viewCursor !== "string")
             return;
-        if (this.seenUsageCursors.has(params.viewCursor))
+        const key = notificationCursorKey({ method: "session/tokenUsage", params });
+        if (!key || this.seenUsageNotifications.has(key))
             return;
-        this.seenUsageCursors.add(params.viewCursor);
+        this.seenUsageNotifications.add(key);
         const raw = params.usage;
         if (!raw)
             return;

package/dist/msp/notification.d.ts

@@ -0,0 +1,9 @@
+/**
+ * Stable identity for exact notification replay suppression.
+ * A view cursor may be shared by multiple distinct notification methods or
+ * payloads, so the cursor alone is not an event identity.
+ */
+export declare function notificationCursorKey(notification: {
+    method: string;
+    params?: Record<string, unknown>;
+}): string | undefined;

package/dist/msp/notification.js

@@ -0,0 +1,26 @@
+import { createHash } from "node:crypto";
+function stableJson(value) {
+    if (value === null || typeof value !== "object")
+        return JSON.stringify(value) ?? "null";
+    if (Array.isArray(value))
+        return `[${value.map(stableJson).join(",")}]`;
+    const entries = Object.entries(value)
+        .filter(([, child]) => child !== undefined)
+        .sort(([left], [right]) => left < right ? -1 : left > right ? 1 : 0);
+    return `{${entries.map(([key, child]) => `${JSON.stringify(key)}:${stableJson(child)}`).join(",")}}`;
+}
+/**
+ * Stable identity for exact notification replay suppression.
+ * A view cursor may be shared by multiple distinct notification methods or
+ * payloads, so the cursor alone is not an event identity.
+ */
+export function notificationCursorKey(notification) {
+    const params = notification.params ?? {};
+    const cursor = params.viewCursor;
+    if (typeof cursor !== "string")
+        return undefined;
+    const digest = createHash("sha256")
+        .update(stableJson({ method: notification.method, params }))
+        .digest("hex");
+    return `${cursor}:${digest}`;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-muse",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "OpenCode plugin backed by the official Muse Code CLI and MSP",
   "license": "MIT",
   "type": "module",