@cortexkit/opencode-magic-context 0.33.0 → 0.33.1

package/src/shared/opencode-compaction-detector.test.ts

@@ -1,239 +0,0 @@
-import { afterEach, beforeEach, describe, expect, it, spyOn } from "bun:test";
-import { mkdirSync, rmSync, writeFileSync } from "node:fs";
-import { join } from "node:path";
-import { isOpenCodeAutoCompactionEnabled } from "./opencode-compaction-detector";
-import * as configDir from "./opencode-config-dir";
-
-describe("opencode-compaction-detector", () => {
-    let tmpDir: string;
-    let previousDisableAutocompact: string | undefined;
-    let restoreConfigPaths: () => void;
-
-    beforeEach(() => {
-        tmpDir = join("/tmp", `compaction-detector-test-${Date.now()}`);
-        mkdirSync(join(tmpDir, ".opencode"), { recursive: true });
-        previousDisableAutocompact = process.env.OPENCODE_DISABLE_AUTOCOMPACT;
-        delete process.env.OPENCODE_DISABLE_AUTOCOMPACT;
-        const configPathsSpy = spyOn(configDir, "getOpenCodeConfigPaths").mockReturnValue({
-            configJson: join(tmpDir, "user-config", "opencode.json"),
-            configJsonc: join(tmpDir, "user-config", "opencode.jsonc"),
-        } as ReturnType<typeof configDir.getOpenCodeConfigPaths>);
-        restoreConfigPaths = () => configPathsSpy.mockRestore();
-    });
-
-    afterEach(() => {
-        restoreConfigPaths();
-        try {
-            rmSync(tmpDir, { recursive: true, force: true, maxRetries: 10, retryDelay: 100 });
-        } catch {
-            /* Ignore EBUSY on Windows */
-        }
-        if (previousDisableAutocompact === undefined) {
-            delete process.env.OPENCODE_DISABLE_AUTOCOMPACT;
-        } else {
-            process.env.OPENCODE_DISABLE_AUTOCOMPACT = previousDisableAutocompact;
-        }
-    });
-
-    describe("#given no config exists", () => {
-        it("#then returns true (default: compaction enabled)", () => {
-            const emptyDir = join("/tmp", `compaction-empty-${Date.now()}`);
-            mkdirSync(emptyDir, { recursive: true });
-
-            const result = isOpenCodeAutoCompactionEnabled(emptyDir);
-
-            expect(result).toBe(true);
-            try {
-                rmSync(emptyDir, { recursive: true, force: true, maxRetries: 10, retryDelay: 100 });
-            } catch {
-                /* Ignore EBUSY on Windows */
-            }
-        });
-    });
-
-    describe("#given OPENCODE_DISABLE_AUTOCOMPACT env flag is set", () => {
-        it("#then returns false", () => {
-            process.env.OPENCODE_DISABLE_AUTOCOMPACT = "1";
-
-            const result = isOpenCodeAutoCompactionEnabled(tmpDir);
-
-            expect(result).toBe(false);
-        });
-    });
-
-    describe("#given project config has compaction.auto = false", () => {
-        it("#when opencode.json #then returns false", () => {
-            writeFileSync(
-                join(tmpDir, ".opencode", "opencode.json"),
-                JSON.stringify({ compaction: { auto: false } }),
-            );
-
-            const result = isOpenCodeAutoCompactionEnabled(tmpDir);
-
-            expect(result).toBe(false);
-        });
-
-        it("#when opencode.jsonc #then returns false", () => {
-            writeFileSync(
-                join(tmpDir, ".opencode", "opencode.jsonc"),
-                `{
-          // compaction disabled
-          "compaction": { "auto": false }
-        }`,
-            );
-
-            const result = isOpenCodeAutoCompactionEnabled(tmpDir);
-
-            expect(result).toBe(false);
-        });
-    });
-
-    describe("#given project config has compaction.auto = true", () => {
-        it("#then returns true", () => {
-            writeFileSync(
-                join(tmpDir, ".opencode", "opencode.json"),
-                JSON.stringify({ compaction: { auto: true } }),
-            );
-
-            const result = isOpenCodeAutoCompactionEnabled(tmpDir);
-
-            expect(result).toBe(true);
-        });
-    });
-
-    describe("#given project config has compaction.prune = true", () => {
-        it("#then returns true (conflict enabled)", () => {
-            writeFileSync(
-                join(tmpDir, ".opencode", "opencode.json"),
-                JSON.stringify({ compaction: { auto: false, prune: true } }),
-            );
-
-            const result = isOpenCodeAutoCompactionEnabled(tmpDir);
-
-            expect(result).toBe(true);
-        });
-    });
-
-    describe("#given project config has auto/prune both false", () => {
-        it("#then returns false", () => {
-            writeFileSync(
-                join(tmpDir, ".opencode", "opencode.json"),
-                JSON.stringify({ compaction: { auto: false, prune: false } }),
-            );
-
-            const result = isOpenCodeAutoCompactionEnabled(tmpDir);
-
-            expect(result).toBe(false);
-        });
-    });
-
-    describe("#given project config has only compaction.prune = false", () => {
-        it("#then returns false", () => {
-            writeFileSync(
-                join(tmpDir, ".opencode", "opencode.json"),
-                JSON.stringify({ compaction: { prune: false } }),
-            );
-
-            const result = isOpenCodeAutoCompactionEnabled(tmpDir);
-
-            expect(result).toBe(false);
-        });
-    });
-
-    describe("#given jsonc and json both exist", () => {
-        it("#then jsonc takes precedence", () => {
-            writeFileSync(
-                join(tmpDir, ".opencode", "opencode.json"),
-                JSON.stringify({ compaction: { auto: true } }),
-            );
-            writeFileSync(
-                join(tmpDir, ".opencode", "opencode.jsonc"),
-                `{ "compaction": { "auto": false } }`,
-            );
-
-            const result = isOpenCodeAutoCompactionEnabled(tmpDir);
-
-            expect(result).toBe(false);
-        });
-    });
-
-    describe("#given config exists but no compaction field", () => {
-        it("#then returns true (default)", () => {
-            writeFileSync(
-                join(tmpDir, ".opencode", "opencode.json"),
-                JSON.stringify({ model: "claude-opus-4-6" }),
-            );
-
-            const result = isOpenCodeAutoCompactionEnabled(tmpDir);
-
-            expect(result).toBe(true);
-        });
-    });
-
-    describe("#given env flag overrides config", () => {
-        it("#then env flag wins even when config has auto: true", () => {
-            process.env.OPENCODE_DISABLE_AUTOCOMPACT = "true";
-            writeFileSync(
-                join(tmpDir, ".opencode", "opencode.json"),
-                JSON.stringify({ compaction: { auto: true } }),
-            );
-
-            const result = isOpenCodeAutoCompactionEnabled(tmpDir);
-
-            expect(result).toBe(false);
-        });
-    });
-
-    describe("#given root-level project config", () => {
-        it("#when root opencode.json has compaction.auto = false #then returns false", () => {
-            writeFileSync(
-                join(tmpDir, "opencode.json"),
-                JSON.stringify({ compaction: { auto: false } }),
-            );
-
-            const result = isOpenCodeAutoCompactionEnabled(tmpDir);
-
-            expect(result).toBe(false);
-        });
-
-        it("#when root opencode.jsonc has compaction.auto = false #then returns false", () => {
-            writeFileSync(join(tmpDir, "opencode.jsonc"), `{ "compaction": { "auto": false } }`);
-
-            const result = isOpenCodeAutoCompactionEnabled(tmpDir);
-
-            expect(result).toBe(false);
-        });
-    });
-
-    describe("#given .opencode/ overrides root-level config", () => {
-        it("#when root says false but .opencode says true #then .opencode wins", () => {
-            writeFileSync(
-                join(tmpDir, "opencode.json"),
-                JSON.stringify({ compaction: { auto: false } }),
-            );
-            writeFileSync(
-                join(tmpDir, ".opencode", "opencode.json"),
-                JSON.stringify({ compaction: { auto: true } }),
-            );
-
-            const result = isOpenCodeAutoCompactionEnabled(tmpDir);
-
-            expect(result).toBe(true);
-        });
-
-        it("#when root says true but .opencode says false #then .opencode wins", () => {
-            writeFileSync(
-                join(tmpDir, "opencode.json"),
-                JSON.stringify({ compaction: { auto: true } }),
-            );
-            writeFileSync(
-                join(tmpDir, ".opencode", "opencode.json"),
-                JSON.stringify({ compaction: { auto: false } }),
-            );
-
-            const result = isOpenCodeAutoCompactionEnabled(tmpDir);
-
-            expect(result).toBe(false);
-        });
-    });
-});

package/src/shared/opencode-compaction-detector.ts

@@ -1,81 +0,0 @@
-import { join } from "node:path";
-import { readJsoncFile } from "./jsonc-parser";
-import { log } from "./logger";
-import { getOpenCodeConfigPaths } from "./opencode-config-dir";
-
-interface OpenCodeConfig {
-    compaction?: {
-        auto?: boolean;
-        prune?: boolean;
-    };
-}
-
-function hasCompactionConflict(
-    compaction: OpenCodeConfig["compaction"] | undefined,
-): boolean | undefined {
-    if (!compaction) return undefined;
-    const hasExplicitSetting = compaction.auto !== undefined || compaction.prune !== undefined;
-    if (!hasExplicitSetting) return undefined;
-    return compaction.auto === true || compaction.prune === true;
-}
-
-export function isOpenCodeAutoCompactionEnabled(directory: string): boolean {
-    if (process.env.OPENCODE_DISABLE_AUTOCOMPACT) {
-        log(
-            "[compaction-detector] OPENCODE_DISABLE_AUTOCOMPACT env flag set — auto compaction disabled",
-        );
-        return false;
-    }
-
-    const projectCompaction = readProjectCompactionConfig(directory);
-    if (projectCompaction !== undefined) {
-        log("[compaction-detector] project config compaction conflict =", projectCompaction);
-        return projectCompaction;
-    }
-
-    const userCompaction = readUserCompactionConfig(directory);
-    if (userCompaction !== undefined) {
-        log("[compaction-detector] user config compaction conflict =", userCompaction);
-        return userCompaction;
-    }
-
-    log("[compaction-detector] no compaction config found — default is enabled");
-    return true;
-}
-
-function readProjectCompactionConfig(directory: string): boolean | undefined {
-    // .opencode/ dir config has higher precedence than root-level config in OpenCode's loading order.
-    // Check highest precedence first — if .opencode/ sets compaction.auto, that wins.
-    const dotOpenCodeJsonc = join(directory, ".opencode", "opencode.jsonc");
-    const dotOpenCodeJson = join(directory, ".opencode", "opencode.json");
-    const dotOpenCodeConfig =
-        readJsoncFile<OpenCodeConfig>(dotOpenCodeJsonc) ??
-        readJsoncFile<OpenCodeConfig>(dotOpenCodeJson);
-
-    const dotOpenCodeCompactionConflict = hasCompactionConflict(dotOpenCodeConfig?.compaction);
-    if (dotOpenCodeCompactionConflict !== undefined) {
-        return dotOpenCodeCompactionConflict;
-    }
-
-    // Root-level project config (lower precedence than .opencode/)
-    const rootJsonc = join(directory, "opencode.jsonc");
-    const rootJson = join(directory, "opencode.json");
-    const rootConfig =
-        readJsoncFile<OpenCodeConfig>(rootJsonc) ?? readJsoncFile<OpenCodeConfig>(rootJson);
-
-    return hasCompactionConflict(rootConfig?.compaction);
-}
-
-function readUserCompactionConfig(_directory: string): boolean | undefined {
-    try {
-        const paths = getOpenCodeConfigPaths({ binary: "opencode" });
-        const config =
-            readJsoncFile<OpenCodeConfig>(paths.configJsonc) ??
-            readJsoncFile<OpenCodeConfig>(paths.configJson);
-
-        return hasCompactionConflict(config?.compaction);
-    } catch {
-        // Intentional: config read is best-effort; missing/unreadable config is not an error
-        return undefined;
-    }
-}

package/src/shared/transcript-opencode.ts

@@ -1,302 +0,0 @@
-/**
- * OpenCode adapter for the harness-agnostic transcript interface.
- *
- * This is a thin proxy over OpenCode's `MessageLike[]` (i.e. `{ info,
- * parts: unknown[] }[]`) — it does NOT copy data. Mutations through
- * `setText`/`setToolOutput`/`replaceWithSentinel` write directly into
- * the source `parts[]` arrays, exactly as the existing OpenCode-only
- * transform code does today. `commit()` is a no-op because OpenCode's
- * AI SDK reads `parts[]` back from the same array we mutated.
- *
- * This module is the boundary that lets the rest of the transform code
- * (which moves to use the Transcript interface in 4b.2) work both for
- * OpenCode and Pi without branching on harness type. By the end of 4b
- * the only OpenCode-aware code in the plugin is this file plus
- * `messages-transform.ts`.
- *
- * ## Mutation contract recap
- *
- * Magic Context's transform mutates message parts in three ways:
- *
- *  1. **Tag prefix injection** — prepends `§N§ ` to text parts and
- *     tool result outputs. Repeated tagging is idempotent because
- *     `prependTag` strips any existing prefix first.
- *
- *  2. **Sentinel replacement** — when a queued drop fires, the part is
- *     replaced with a `[dropped §N§]` or `[truncated §N§]` placeholder.
- *     The original tag number is preserved so the agent's mental
- *     model of "what was here" survives.
- *
- *  3. **Structural noise stripping** — `step-start`/`step-finish`
- *     wrappers and similar structural metadata are replaced with empty
- *     sentinel parts so they don't consume tag numbers or get tagged
- *     themselves.
- *
- * The OpenCode adapter implements (1) and (2) by editing `part.text` /
- * `part.state.output` in place. For (3), structural parts surface as
- * `kind: "structural"` so callers can filter them out. Adapter does NOT
- * itself perform stripping — that's the transform pipeline's job, called
- * after the adapter wraps the messages.
- *
- * Step 4b.1 ships the adapter alone. The existing OpenCode transform
- * code keeps using `MessageLike[]` directly until 4b.2 migrates the
- * tagging+drops layer to use Transcript instances.
- */
-
-import { estimateTokens } from "../hooks/magic-context/read-session-formatting";
-import { isRecord } from "./record-type-guard";
-import type {
-    Transcript,
-    TranscriptMessage,
-    TranscriptPart,
-    TranscriptPartKind,
-} from "./transcript";
-
-/**
- * The OpenCode `MessageLike` shape. Re-declared here to avoid a circular
- * import with `tag-messages.ts` (which lives in the magic-context hooks
- * tree and depends on storage). Keeping a local minimal type also makes
- * the adapter trivially unit-testable without booting OpenCode SDK
- * types.
- *
- * MUST stay structurally compatible with `tag-messages.ts MessageLike` —
- * if that file's MessageLike adds a required field, this one needs to
- * add it too. The build will fail loudly if the shapes diverge.
- */
-export interface OpenCodeMessageLike {
-    info: { id?: string; role?: string; sessionID?: string };
-    parts: unknown[];
-}
-
-/**
- * Wrap an existing `MessageLike[]` as a Transcript. Zero copies — every
- * `TranscriptPart` returned proxies the matching entry in the source
- * `parts` array, and mutations are reflected immediately.
- */
-export function createOpenCodeTranscript(messages: OpenCodeMessageLike[]): Transcript {
-    const transcriptMessages: TranscriptMessage[] = messages.map((message) => ({
-        info: {
-            id: message.info.id,
-            role: message.info.role ?? "unknown",
-            sessionId: message.info.sessionID,
-        },
-        // `parts` is a getter so newly-replaced sentinels inside the
-        // underlying array are surfaced on the next read. Cheap; allocs
-        // one wrapper per part per access. Adapter callers iterate
-        // `messages` once per pass so the cost is O(parts) per pass —
-        // same as before the adapter existed.
-        get parts(): TranscriptPart[] {
-            return message.parts.map((part, index) => createOpenCodePart(message, index, part));
-        },
-    }));
-
-    return {
-        messages: transcriptMessages,
-        harness: "opencode",
-        // OpenCode reads parts back from the same `parts[]` array we
-        // mutated, so there's nothing to flush. Adapters that buffer
-        // mutations (Pi) override this.
-        commit(): void {
-            /* no-op for OpenCode */
-        },
-    };
-}
-
-/**
- * Construct a TranscriptPart proxy over a single OpenCode part.
- *
- * Held by `parts` getter calls only; never cached because the underlying
- * `parts[]` array can be mutated in place (sentinel replacement) and
- * cached proxies would point at stale data. The constructor cost is
- * trivial — small object literal, no allocations beyond the closure.
- */
-function createOpenCodePart(
-    parent: OpenCodeMessageLike,
-    index: number,
-    rawPart: unknown,
-): TranscriptPart {
-    const kind = classifyOpenCodePart(rawPart);
-    const id = extractPartId(rawPart);
-
-    return {
-        kind,
-        id,
-        getText(): string | undefined {
-            return readOpenCodePartText(rawPart);
-        },
-        setText(newText: string): boolean {
-            return writeOpenCodePartText(rawPart, newText);
-        },
-        setToolOutput(newText: string): boolean {
-            return writeOpenCodeToolOutput(rawPart, newText);
-        },
-        getToolMetadata(): {
-            toolName: string | undefined;
-            inputByteSize: number;
-            inputTokenCount: number;
-        } {
-            return readOpenCodeToolMetadata(rawPart);
-        },
-        getToolInput(): Record<string, unknown> | null {
-            return readOpenCodeToolInput(rawPart);
-        },
-        setToolInput(input: Record<string, unknown>): boolean {
-            return writeOpenCodeToolInput(rawPart, input);
-        },
-        replaceWithSentinel(sentinelText: string): boolean {
-            // Build a synthetic text part that carries the sentinel as
-            // its content. Subsequent passes see this as a normal text
-            // part with kind="text" — but the existing tagging code is
-            // idempotent and won't double-tag a part that already has
-            // the right prefix, so re-processing is safe.
-            //
-            // We DON'T preserve the original part type. Sentinels are
-            // always text — that's the contract the existing
-            // apply-operations code expects.
-            const sentinelPart = { type: "text", text: sentinelText };
-            parent.parts[index] = sentinelPart;
-            return true;
-        },
-    };
-}
-
-/** Classify part kind based on `type` field. Falls back to "unknown". */
-function classifyOpenCodePart(part: unknown): TranscriptPartKind {
-    if (!isRecord(part)) return "unknown";
-    const type = part.type;
-    if (typeof type !== "string") return "unknown";
-    switch (type) {
-        case "text":
-            return "text";
-        case "reasoning":
-            return "thinking";
-        case "tool":
-            return "tool_use";
-        case "file":
-            return "file";
-        case "image":
-            return "image";
-        case "step-start":
-        case "step-finish":
-            return "structural";
-        default:
-            return "unknown";
-    }
-}
-
-/**
- * Extract a stable per-part identifier when present. Used by the dropped-
- * placeholder watermark to track which sentinels are already replayed
- * across passes.
- */
-function extractPartId(part: unknown): string | undefined {
-    if (!isRecord(part)) return undefined;
-    if (typeof part.id === "string" && part.id.length > 0) return part.id;
-    if (typeof part.callID === "string" && part.callID.length > 0) return part.callID;
-    return undefined;
-}
-
-function readOpenCodePartText(part: unknown): string | undefined {
-    if (!isRecord(part)) return undefined;
-    if (typeof part.text === "string") return part.text;
-    if (typeof part.thinking === "string") return part.thinking;
-    if (part.type === "tool") {
-        const state = isRecord(part.state) ? part.state : null;
-        const output = state && typeof state.output === "string" ? state.output : "";
-        return output;
-    }
-    return undefined;
-}
-
-function writeOpenCodePartText(part: unknown, newText: string): boolean {
-    if (!isRecord(part)) return false;
-    const writable = part as Record<string, unknown>;
-    if (typeof writable.text === "string") {
-        if (writable.text === newText) return false;
-        writable.text = newText;
-        return true;
-    }
-    if (typeof writable.thinking === "string") {
-        if (writable.thinking === newText) return false;
-        writable.thinking = newText;
-        return true;
-    }
-    return false;
-}
-
-function writeOpenCodeToolOutput(part: unknown, newText: string): boolean {
-    if (!isRecord(part)) return false;
-    if (part.type !== "tool") return false;
-    const state = isRecord(part.state) ? (part.state as Record<string, unknown>) : null;
-    if (!state) return false;
-    if (typeof state.output !== "string") return false;
-    if (state.output === newText) return false;
-    state.output = newText;
-    return true;
-}
-
-function readOpenCodeToolMetadata(part: unknown): {
-    toolName: string | undefined;
-    inputByteSize: number;
-    inputTokenCount: number;
-} {
-    if (!isRecord(part)) return { toolName: undefined, inputByteSize: 0, inputTokenCount: 0 };
-    if (part.type !== "tool") return { toolName: undefined, inputByteSize: 0, inputTokenCount: 0 };
-
-    // OpenCode parts use `tool` as the tool name field; some legacy
-    // shapes use `toolName` or `name`. Match all three for forward
-    // compatibility with shape evolution.
-    const toolName =
-        typeof part.tool === "string"
-            ? part.tool
-            : typeof part.toolName === "string"
-              ? part.toolName
-              : typeof part.name === "string"
-                ? part.name
-                : undefined;
-
-    const state = isRecord(part.state) ? part.state : null;
-    const input = state?.input ?? part.args ?? part.input;
-
-    let inputByteSize = 0;
-    let inputTokenCount = 0;
-    if (input !== undefined && input !== null) {
-        try {
-            const serialized = typeof input === "string" ? input : JSON.stringify(input);
-            inputByteSize = serialized.length;
-            inputTokenCount = serialized ? estimateTokens(serialized) : 0;
-        } catch {
-            inputByteSize = 0;
-            inputTokenCount = 0;
-        }
-    }
-
-    return { toolName, inputByteSize, inputTokenCount };
-}
-
-/** Non-mutating read of an OpenCode tool part's input object (or null). */
-function readOpenCodeToolInput(part: unknown): Record<string, unknown> | null {
-    if (!isRecord(part)) return null;
-    const state = isRecord(part.state) ? part.state : null;
-    const input = state?.input ?? part.args ?? part.input;
-    return isRecord(input) ? input : null;
-}
-
-/** Replace an OpenCode tool part's input object in place. Returns true if a
- *  writable input slot was found. */
-function writeOpenCodeToolInput(part: unknown, input: Record<string, unknown>): boolean {
-    if (!isRecord(part)) return false;
-    if (isRecord(part.state) && isRecord(part.state.input)) {
-        part.state.input = input;
-        return true;
-    }
-    if (isRecord(part.args)) {
-        part.args = input;
-        return true;
-    }
-    if (isRecord(part.input)) {
-        part.input = input;
-        return true;
-    }
-    return false;
-}