@skillstate/opencode 2.2.1 → 3.0.0

package/dist/plugin.d.ts

@@ -1,77 +1,87 @@
-import { mergePatch, resolveHostStateForCwd } from '@skillstate/core';
-import type { OpenCodeMessage, SkillStatePlugin } from './plugin-types.js';
-export * from './plugin-types.js';
 /**
- * Resolve the per-project state file for a session working directory
- * (`cwd` of the current opencode session) — the core single source of
- * truth (`resolveHostStateForCwd`): `<cwd>/.skillstate/skillstate.json`,
- * or the global bucket `<home>/.skillstate/global/skillstate.json` when
- * cwd equals home. A non-empty `agentId` scopes the file under
- * `<bucket>/agents/<agentId>/skillstate.json`. Pure path arithmetic via
- * `path.resolve`, no filesystem access.
- */
-export { resolveHostStateForCwd as resolveStatePathForCwd };
-export { mergePatch };
-/** Agent scope used by the plugin when no session id is available. */
-export declare const PLUGIN_DEFAULT_AGENT_ID = "default";
-/** Options for {@link createSkillStatePlugin}. */
-export interface SkillStatePluginOptions {
-    /** Non-system messages kept in the prompt (default 3). */
-    maxHistoryMessages?: number;
-}
-/**
- * Read the state file. Missing or corrupt files yield `{}` (best-effort).
- * The on-disk envelope is `{ version: 1, state }` (migrations-compatible);
- * a bare object is tolerated and treated as the state itself. Thin fs
- * adapter over the core hook-runtime {@link readStateEnvelope}.
- */
-export declare function readSkillState(statePath: string): Record<string, unknown>;
-/**
- * Persist the state file (best-effort: read-only environments are ignored).
- * Creates the parent directory when missing (the per-project resolver may
- * target a fresh `<cwd>/.skillstate/agents/<id>/`). Writes the
- * `{ version: 1, state }` envelope so `migrate()`/runtime resume read the
- * same file — via the core hook-runtime {@link saveStateEnvelope} — under
- * the cross-process sync lock {@link lockStateWrite} (2-3 parallel agent
- * processes never interleave state writes).
- */
-export declare function saveSkillState(statePath: string, state: Record<string, unknown>): void;
-/**
- * Atomic READ-MERGE-WRITE of one `state_patch` (paper ⊕: null deletes):
- * the whole critical section runs inside {@link lockStateWrite}, so two
- * concurrent writers apply BOTH patches instead of racing between the
- * read and the write. Best-effort: lock contention or unwritable state
- * files are swallowed — the tool flow never breaks.
- */
-export declare function mergeSkillState(statePath: string, patch: Record<string, unknown>): Record<string, unknown>;
-/**
- * Extract the `state_patch` object from an LLM response's fenced ```json
- * block; `null` when there is no block, it is malformed, or it carries no
- * object-shaped `state_patch`. Thin adapter over the core hook-runtime
- * {@link findFencedPatch} (the invalid/truncated outcomes collapse to
- * `null`, preserving the legacy boolean contract).
- */
-export declare function extractPatch(response: string): Record<string, unknown> | null;
-/**
- * Agent id for an opencode hook call: the 8-char session prefix from
- * `input.sessionID` when the hook carries it, else the first non-synthetic
- * message's `info.sessionID` (the transform hook input is empty — the
- * session lives on the message envelopes), else `'default'` (the plugin
- * trims the history of one session context).
+ * `@skillstate/opencode` — the OpenCode **v2** plugin.
+ *
+ * ── What this replaces ───────────────────────────────────────────────────
+ *
+ * The v1 integration rewrote the conversation on every model request. It
+ * kept the system messages and the last three non-system messages, dropped
+ * everything else from `output.messages`, and appended a synthetic
+ * `role: "user"` message containing the raw state JSON. The reported
+ * failure was that the agent stopped doing the user's task and started
+ * emitting state JSON instead.
+ *
+ * Both halves of that were destructive, and neither was a model quirk:
+ *
+ * 1. The injected message landed LAST, so for the model it was the current
+ *    instruction — it displaced the user's actual request.
+ * 2. `slice(-3)` deleted the task statement, the tool results and the
+ *    errors the agent had just been handed. It was reasoning about work it
+ *    could no longer see.
+ *
+ * The MCP server made it worse: `spec.get` returned a procedural spec whose
+ * default was `INTERCODE_CTF_SPEC`, whose instructions read "You are an
+ * autonomous CTF agent ... hidden flag somewhere on its filesystem". A
+ * model told to look for a flag looks for a flag. (Fixed: the default is now
+ * the neutral `GENERIC_PROCEDURE_SPEC`, and its instructions describe the
+ * storage format instead of prescribing a way of working.)
+ *
+ * ── The v2 design ────────────────────────────────────────────────────────
+ *
+ * Three rules, each enforced by a test:
+ *
+ * - **Never mutate `event.messages`.** The plugin contributes one additive
+ *   fragment to `event.system` and leaves the transcript alone. See
+ *   `tests/opencode/context-integrity.test.ts`.
+ * - **Never inject behavioural instructions.** The system fragment
+ *   describes what the notes are and when to use them; it contains no
+ *   "you must", no "always", and no output format. See
+ *   `system-hint.ts`.
+ * - **Inert until used.** A project with no state file gets no system
+ *   fragment at all and behaves exactly like vanilla OpenCode. No files are
+ *   created by loading the plugin.
+ *
+ * ── Native tools AND the MCP server, on purpose ──────────────────────────
+ *
+ * This package does not replace `@skillstate/mcp`; it sits beside it.
+ *
+ * - The native tools ({@link registerTools}) are the fast path inside
+ *   opencode: a typed schema, structured output, no JSON-RPC round-trip and
+ *   no untyped text result.
+ * - The MCP server is the portable path. It is what every other
+ *   MCP-capable host reads, and the only way to reach this state from a
+ *   client that is not opencode.
+ *
+ * Both address the same `<project>/.skillstate/skillstate.json`, so they
+ * cannot disagree about what is saved. `skillstate init` registers both.
+ *
+ * The reason v1 needed the MCP server is gone: an opencode v1 plugin could
+ * not contribute first-class tools at all.
+ *
+ * Load it from `opencode.json(c)`:
+ *
+ * ```json
+ * { "plugins": ["@skillstate/opencode"] }
+ * ```
  */
-export declare function pluginAgentId(input: {
-    sessionID?: unknown;
-}, messages?: OpenCodeMessage[]): string;
+import { Plugin } from '@opencode/plugin';
+/** Stable plugin id — scopes plugin storage and identifies it in `/api/plugin`. */
+export declare const PLUGIN_ID = "skillstate";
 /**
- * Build the OpenCode plugin function with the same behavior for every host
- * entry point (thin generated loaders, direct imports).
- *
- * State resolution is ALWAYS per-project: the state file path is computed
- * from the session cwd on EVERY hook call via
- * `resolveStatePathForCwd(process.cwd(), os.homedir(), agentId)` — each
- * project gets its own `<cwd>/.skillstate/`, each session (sub-agent) its
- * isolated `agents/<session>/` copy, and a session launched from `$HOME`
- * uses the global bucket.
+ * The plugin definition.
+ *
+ * `setup` wires three things and returns a cleanup function:
+ *
+ * - a {@link SessionRegistry}, fed by the server event stream, so a
+ *   sub-agent session is recognised and given its own state file;
+ * - a {@link ProjectStateStore} rooted at the plugin's own project
+ *   location, so two checkouts served by one OpenCode server never share
+ *   state;
+ * - native tools plus a single additive `context` hook.
+ *
+ * The event subscription is the only resource the plugin owns, so the
+ * returned cleanup aborts it. Hook and tool registrations are disposed by
+ * OpenCode when the plugin unloads.
  */
-export declare function createSkillStatePlugin(options?: SkillStatePluginOptions): SkillStatePlugin;
+export declare const SkillStatePlugin: Plugin.Plugin;
+export default SkillStatePlugin;
 //# sourceMappingURL=plugin.d.ts.map

package/dist/plugin.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"plugin.d.ts","sourceRoot":"","sources":["../src/plugin.ts"],"names":[],"mappings":"AA+BA,OAAO,EAGL,UAAU,EAGV,sBAAsB,EAEvB,MAAM,kBAAkB,CAAC;AAC1B,OAAO,KAAK,EACV,eAAe,EAEf,gBAAgB,EACjB,MAAM,mBAAmB,CAAC;AAE3B,cAAc,mBAAmB,CAAC;AAElC;;;;;;;;GAQG;AACH,OAAO,EAAE,sBAAsB,IAAI,sBAAsB,EAAE,CAAC;AAE5D,OAAO,EAAE,UAAU,EAAE,CAAC;AAEtB,sEAAsE;AACtE,eAAO,MAAM,uBAAuB,YAAY,CAAC;AAEjD,kDAAkD;AAClD,MAAM,WAAW,uBAAuB;IACtC,0DAA0D;IAC1D,kBAAkB,CAAC,EAAE,MAAM,CAAC;CAC7B;AAED;;;;;GAKG;AACH,wBAAgB,cAAc,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAEzE;AAED;;;;;;;;GAQG;AACH,wBAAgB,cAAc,CAAC,SAAS,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,IAAI,CAWtF;AAED;;;;;;GAMG;AACH,wBAAgB,eAAe,CAC7B,SAAS,EAAE,MAAM,EACjB,KAAK,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAC7B,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAYzB;AAED;;;;;;GAMG;AACH,wBAAgB,YAAY,CAAC,QAAQ,EAAE,MAAM,GAAG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,IAAI,CAG7E;AAED;;;;;;GAMG;AACH,wBAAgB,aAAa,CAC3B,KAAK,EAAE;IAAE,SAAS,CAAC,EAAE,OAAO,CAAA;CAAE,EAC9B,QAAQ,CAAC,EAAE,eAAe,EAAE,GAC3B,MAAM,CAWR;AAKD;;;;;;;;;;GAUG;AACH,wBAAgB,sBAAsB,CAAC,OAAO,GAAE,uBAA4B,GAAG,gBAAgB,CAiF9F"}
+{"version":3,"file":"plugin.d.ts","sourceRoot":"","sources":["../src/plugin.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAgEG;AAEH,OAAO,EAAE,MAAM,EAAE,MAAM,kBAAkB,CAAC;AAO1C,mFAAmF;AACnF,eAAO,MAAM,SAAS,eAAe,CAAC;AAEtC;;;;;;;;;;;;;;;GAeG;AACH,eAAO,MAAM,gBAAgB,eAyD3B,CAAC;eAEY,gBAAgB"}

package/dist/plugin.js

@@ -1,210 +1,147 @@
 /**
- * Static OpenCode plugin — the SINGLE SOURCE OF TRUTH for the skillstate
- * host integration. `OpenCodeAdapter.generatePluginCode` emits a thin loader
- * that imports `createSkillStatePlugin` from this module; the per-project
- * state resolution lives in `@skillstate/core`
- * (`resolveHostStateForCwd`, re-exported here) and the hook logic
- * (envelope read/write, ⊕ merge, patch extraction) in the core
- * hook-runtime — this module only adapts it to the OpenCode hooks.
- *
- * Hooks (opencode 1.17 contract, verified on host):
- * - `experimental.chat.messages.transform` — entries are `{ info, parts }`
- *   envelopes (role on `info.role`); the pipeline keeps the ORIGINAL array
- *   reference, so trimming mutates in place; the state is injected as a
- *   synthetic `{ info, parts }` element. Real O(1) prompt footprint.
- * - `experimental.session.compacting` — pushes the state into
- *   `output.context` so the compaction summary preserves it.
- * - `tool.execute.after` — the tool response is `output.output`; a fenced
- *   ```json `state_patch` block is merged (paper ⊕: null deletes) and saved.
- *
- * AGENT-SCOPED STATE: the opencode hook inputs carry the session id
- * (`input.sessionID`; message envelopes carry `info.sessionID`), so every
- * hook scopes the state file to `<cwd>/.skillstate/agents/<session>/` —
- * parallel opencode sessions (sub-agents) never last-writer-win over the
- * main state. When no session id is available the `'default'` agent is
- * used (the plugin trims the history of one session context). Writes go
- * through the core cross-process sync lock (`lockStateWrite`) so a
- * session state file is never interleaved between processes.
+ * `@skillstate/opencode` — the OpenCode **v2** plugin.
+ *
+ * ── What this replaces ───────────────────────────────────────────────────
+ *
+ * The v1 integration rewrote the conversation on every model request. It
+ * kept the system messages and the last three non-system messages, dropped
+ * everything else from `output.messages`, and appended a synthetic
+ * `role: "user"` message containing the raw state JSON. The reported
+ * failure was that the agent stopped doing the user's task and started
+ * emitting state JSON instead.
+ *
+ * Both halves of that were destructive, and neither was a model quirk:
+ *
+ * 1. The injected message landed LAST, so for the model it was the current
+ *    instruction — it displaced the user's actual request.
+ * 2. `slice(-3)` deleted the task statement, the tool results and the
+ *    errors the agent had just been handed. It was reasoning about work it
+ *    could no longer see.
+ *
+ * The MCP server made it worse: `spec.get` returned a procedural spec whose
+ * default was `INTERCODE_CTF_SPEC`, whose instructions read "You are an
+ * autonomous CTF agent ... hidden flag somewhere on its filesystem". A
+ * model told to look for a flag looks for a flag. (Fixed: the default is now
+ * the neutral `GENERIC_PROCEDURE_SPEC`, and its instructions describe the
+ * storage format instead of prescribing a way of working.)
+ *
+ * ── The v2 design ────────────────────────────────────────────────────────
+ *
+ * Three rules, each enforced by a test:
+ *
+ * - **Never mutate `event.messages`.** The plugin contributes one additive
+ *   fragment to `event.system` and leaves the transcript alone. See
+ *   `tests/opencode/context-integrity.test.ts`.
+ * - **Never inject behavioural instructions.** The system fragment
+ *   describes what the notes are and when to use them; it contains no
+ *   "you must", no "always", and no output format. See
+ *   `system-hint.ts`.
+ * - **Inert until used.** A project with no state file gets no system
+ *   fragment at all and behaves exactly like vanilla OpenCode. No files are
+ *   created by loading the plugin.
+ *
+ * ── Native tools AND the MCP server, on purpose ──────────────────────────
+ *
+ * This package does not replace `@skillstate/mcp`; it sits beside it.
+ *
+ * - The native tools ({@link registerTools}) are the fast path inside
+ *   opencode: a typed schema, structured output, no JSON-RPC round-trip and
+ *   no untyped text result.
+ * - The MCP server is the portable path. It is what every other
+ *   MCP-capable host reads, and the only way to reach this state from a
+ *   client that is not opencode.
+ *
+ * Both address the same `<project>/.skillstate/skillstate.json`, so they
+ * cannot disagree about what is saved. `skillstate init` registers both.
+ *
+ * The reason v1 needed the MCP server is gone: an opencode v1 plugin could
+ * not contribute first-class tools at all.
+ *
+ * Load it from `opencode.json(c)`:
+ *
+ * ```json
+ * { "plugins": ["@skillstate/opencode"] }
+ * ```
  */
-import * as fs from 'node:fs';
-import * as os from 'node:os';
+import { Plugin } from '@opencode/plugin';
 import * as path from 'node:path';
-import { findFencedPatch, lockStateWrite, mergePatch, readStateEnvelope, resolveAgentIdFromSession, resolveHostStateForCwd, saveStateEnvelope, } from '@skillstate/core';
-export * from './plugin-types.js';
-/**
- * Resolve the per-project state file for a session working directory
- * (`cwd` of the current opencode session) — the core single source of
- * truth (`resolveHostStateForCwd`): `<cwd>/.skillstate/skillstate.json`,
- * or the global bucket `<home>/.skillstate/global/skillstate.json` when
- * cwd equals home. A non-empty `agentId` scopes the file under
- * `<bucket>/agents/<agentId>/skillstate.json`. Pure path arithmetic via
- * `path.resolve`, no filesystem access.
- */
-export { resolveHostStateForCwd as resolveStatePathForCwd };
-export { mergePatch };
-/** Agent scope used by the plugin when no session id is available. */
-export const PLUGIN_DEFAULT_AGENT_ID = 'default';
+import { SessionRegistry, stateScopeFor } from './session-registry.js';
+import { ProjectStateStore } from './state-store.js';
+import { buildStateHint } from './system-hint.js';
+import { registerTools } from './tools.js';
+/** Stable plugin id — scopes plugin storage and identifies it in `/api/plugin`. */
+export const PLUGIN_ID = 'skillstate';
 /**
- * Read the state file. Missing or corrupt files yield `{}` (best-effort).
- * The on-disk envelope is `{ version: 1, state }` (migrations-compatible);
- * a bare object is tolerated and treated as the state itself. Thin fs
- * adapter over the core hook-runtime {@link readStateEnvelope}.
- */
-export function readSkillState(statePath) {
-    return readStateEnvelope(statePath, (p) => fs.readFileSync(p, 'utf-8'));
-}
-/**
- * Persist the state file (best-effort: read-only environments are ignored).
- * Creates the parent directory when missing (the per-project resolver may
- * target a fresh `<cwd>/.skillstate/agents/<id>/`). Writes the
- * `{ version: 1, state }` envelope so `migrate()`/runtime resume read the
- * same file — via the core hook-runtime {@link saveStateEnvelope} — under
- * the cross-process sync lock {@link lockStateWrite} (2-3 parallel agent
- * processes never interleave state writes).
- */
-export function saveSkillState(statePath, state) {
-    try {
-        fs.mkdirSync(path.dirname(statePath), { recursive: true });
-        lockStateWrite(statePath, fs, () => saveStateEnvelope(statePath, state, (p, data) => fs.writeFileSync(p, data)));
-    }
-    catch {
-        // Best-effort: read-only environments or permission issues.
-    }
-}
-/**
- * Atomic READ-MERGE-WRITE of one `state_patch` (paper ⊕: null deletes):
- * the whole critical section runs inside {@link lockStateWrite}, so two
- * concurrent writers apply BOTH patches instead of racing between the
- * read and the write. Best-effort: lock contention or unwritable state
- * files are swallowed — the tool flow never breaks.
+ * The plugin definition.
+ *
+ * `setup` wires three things and returns a cleanup function:
+ *
+ * - a {@link SessionRegistry}, fed by the server event stream, so a
+ *   sub-agent session is recognised and given its own state file;
+ * - a {@link ProjectStateStore} rooted at the plugin's own project
+ *   location, so two checkouts served by one OpenCode server never share
+ *   state;
+ * - native tools plus a single additive `context` hook.
+ *
+ * The event subscription is the only resource the plugin owns, so the
+ * returned cleanup aborts it. Hook and tool registrations are disposed by
+ * OpenCode when the plugin unloads.
  */
-export function mergeSkillState(statePath, patch) {
-    try {
-        fs.mkdirSync(path.dirname(statePath), { recursive: true });
-        let merged = {};
-        lockStateWrite(statePath, fs, () => {
-            merged = mergePatch(readSkillState(statePath), patch);
-            saveStateEnvelope(statePath, merged, (p, data) => fs.writeFileSync(p, data));
+export const SkillStatePlugin = Plugin.define({
+    id: PLUGIN_ID,
+    async setup(ctx) {
+        const sessions = new SessionRegistry();
+        const scopeFor = (sessionID) => stateScopeFor(sessions, sessionID);
+        // `ctx.location.project.canonical` is the canonical checkout, stable
+        // across worktrees and symlinks. The v1 plugin used `process.cwd()`,
+        // which in v2 is the server's cwd, not the session's project.
+        const store = new ProjectStateStore({
+            directory: ctx.location.project.canonical,
         });
-        return merged;
-    }
-    catch {
-        return readSkillState(statePath);
-    }
-}
-/**
- * Extract the `state_patch` object from an LLM response's fenced ```json
- * block; `null` when there is no block, it is malformed, or it carries no
- * object-shaped `state_patch`. Thin adapter over the core hook-runtime
- * {@link findFencedPatch} (the invalid/truncated outcomes collapse to
- * `null`, preserving the legacy boolean contract).
- */
-export function extractPatch(response) {
-    const result = findFencedPatch(response);
-    return 'patch' in result ? result.patch : null;
-}
-/**
- * Agent id for an opencode hook call: the 8-char session prefix from
- * `input.sessionID` when the hook carries it, else the first non-synthetic
- * message's `info.sessionID` (the transform hook input is empty — the
- * session lives on the message envelopes), else `'default'` (the plugin
- * trims the history of one session context).
- */
-export function pluginAgentId(input, messages) {
-    const direct = resolveAgentIdFromSession(input?.sessionID);
-    if (direct.length > 0)
-        return direct;
-    const fromMessages = (messages ?? []).find((m) => typeof m.info?.sessionID === 'string' &&
-        m.info.sessionID.length > 0 &&
-        m.info.sessionID !== 'skillstate');
-    const indirect = resolveAgentIdFromSession(fromMessages?.info.sessionID);
-    return indirect.length > 0 ? indirect : PLUGIN_DEFAULT_AGENT_ID;
-}
-/** Synthetic message ids for the injected state carrier. */
-const STATE_MESSAGE_ID = 'skillstate-state-inject';
-/**
- * Build the OpenCode plugin function with the same behavior for every host
- * entry point (thin generated loaders, direct imports).
- *
- * State resolution is ALWAYS per-project: the state file path is computed
- * from the session cwd on EVERY hook call via
- * `resolveStatePathForCwd(process.cwd(), os.homedir(), agentId)` — each
- * project gets its own `<cwd>/.skillstate/`, each session (sub-agent) its
- * isolated `agents/<session>/` copy, and a session launched from `$HOME`
- * uses the global bucket.
- */
-export function createSkillStatePlugin(options = {}) {
-    const resolvePath = (agentId) => resolveHostStateForCwd(process.cwd(), os.homedir(), agentId);
-    const maxHistory = options.maxHistoryMessages ?? 3;
-    return async () => {
-        return {
-            // ── O(1) history trimming ──────────────────────────────────────────
-            // Filters messages BEFORE each LLM call: keeps all system messages
-            // plus the last `maxHistory` non-system messages, then injects a
-            // synthetic state element. Old messages are DROPPED from the prompt,
-            // not just hidden.
-            'experimental.chat.messages.transform': async (input, output) => {
-                const agentId = pluginAgentId(input, output.messages);
-                const state = readSkillState(resolvePath(agentId));
-                const messages = output.messages;
-                const systemMessages = messages.filter((m) => m.info.role === 'system');
-                const trimmed = messages
-                    .filter((m) => m.info.role !== 'system')
-                    .slice(-maxHistory);
-                // Synthetic state carrier — a `{ info, parts }` envelope whose text
-                // part carries the current state JSON.
-                const stateMessage = {
-                    info: {
-                        id: STATE_MESSAGE_ID,
-                        sessionID: 'skillstate',
-                        role: 'user',
-                        time: { created: 0 },
-                        agent: 'skillstate',
-                        model: { providerID: 'skillstate', modelID: 'skillstate' },
-                    },
-                    parts: [
-                        {
-                            id: `${STATE_MESSAGE_ID}-text`,
-                            sessionID: 'skillstate',
-                            messageID: STATE_MESSAGE_ID,
-                            type: 'text',
-                            synthetic: true,
-                            text: `Current skill state (JSON): ${JSON.stringify(state)}`,
-                        },
-                    ],
-                };
-                // The pipeline holds the original array reference — mutate in place
-                // (reassigning `output.messages` would not reach the LLM call).
-                const kept = [...systemMessages, ...trimmed, stateMessage];
-                messages.length = 0;
-                messages.push(...kept);
-            },
-            // ── Compaction context injection ───────────────────────────────────
-            // Before compaction, inject the current state into the context so the
-            // compaction summary preserves state even after history is compressed.
-            'experimental.session.compacting': async (input, output) => {
-                const agentId = pluginAgentId(input);
-                const state = readSkillState(resolvePath(agentId));
-                if (!Array.isArray(output.context)) {
-                    output.context = [];
-                }
-                output.context.push(`Skillstate: ${JSON.stringify(state)}`);
-            },
-            // ── State persistence from LLM responses ───────────────────────────
-            // After tool execution, extract state_patch from the tool response
-            // (output.output), and atomically merge it into the session-scoped
-            // state (read + merge + write all inside the cross-process lock).
-            'tool.execute.after': async (input, output) => {
-                const response = output.output ?? '';
-                if (typeof response !== 'string')
-                    return;
-                const patch = extractPatch(response);
-                if (patch) {
-                    const agentId = pluginAgentId(input);
-                    mergeSkillState(resolvePath(agentId), patch);
+        await ctx.tool.transform((editor) => {
+            registerTools(editor, { store, sessions, scopeFor });
+        });
+        // ── Session tree ────────────────────────────────────────────────────
+        // Sub-agent sessions are created by OpenCode itself, so the parent edge
+        // arrives on the event stream. Until one is seen a session is treated as
+        // a root session, which is the correct default for single-session use.
+        const controller = new AbortController();
+        void (async () => {
+            try {
+                for await (const event of ctx.event.subscribe({ signal: controller.signal })) {
+                    sessions.ingestEvent(event);
                 }
-            },
+            }
+            catch {
+                // The stream ends when the plugin unloads or the server goes away.
+                // Session scoping degrades to "everyone shares the project file",
+                // which is safe; it must never surface as an unhandled rejection.
+            }
+        })();
+        // ── System fragment ─────────────────────────────────────────────────
+        // Registered on the agent loop only. `compaction`, `generate` and
+        // `title` are separate hooks in v2 and are deliberately left alone:
+        // after a compaction the next agent-loop request re-adds the fragment,
+        // so state survives without this plugin ever touching the transcript or
+        // the summariser's input.
+        await ctx.session.hook('context', (event) => {
+            const scope = scopeFor(event.sessionID);
+            if (!store.exists(scope))
+                return;
+            const state = store.read(scope);
+            const hint = buildStateHint({
+                state,
+                statePath: path.relative(store.projectDirectory, store.pathFor(scope)),
+                scope,
+            });
+            if (hint.length === 0)
+                return;
+            event.system.push({ type: 'text', text: hint });
+        });
+        return () => {
+            controller.abort();
         };
-    };
-}
+    },
+});
+export default SkillStatePlugin;
 //# sourceMappingURL=plugin.js.map

package/dist/plugin.js.map

@@ -1 +1 @@
-{"version":3,"file":"plugin.js","sourceRoot":"","sources":["../src/plugin.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;GA2BG;AACH,OAAO,KAAK,EAAE,MAAM,SAAS,CAAC;AAC9B,OAAO,KAAK,EAAE,MAAM,SAAS,CAAC;AAC9B,OAAO,KAAK,IAAI,MAAM,WAAW,CAAC;AAClC,OAAO,EACL,eAAe,EACf,cAAc,EACd,UAAU,EACV,iBAAiB,EACjB,yBAAyB,EACzB,sBAAsB,EACtB,iBAAiB,GAClB,MAAM,kBAAkB,CAAC;AAO1B,cAAc,mBAAmB,CAAC;AAElC;;;;;;;;GAQG;AACH,OAAO,EAAE,sBAAsB,IAAI,sBAAsB,EAAE,CAAC;AAE5D,OAAO,EAAE,UAAU,EAAE,CAAC;AAEtB,sEAAsE;AACtE,MAAM,CAAC,MAAM,uBAAuB,GAAG,SAAS,CAAC;AAQjD;;;;;GAKG;AACH,MAAM,UAAU,cAAc,CAAC,SAAiB;IAC9C,OAAO,iBAAiB,CAAC,SAAS,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,EAAE,CAAC,YAAY,CAAC,CAAC,EAAE,OAAO,CAAC,CAA4B,CAAC;AACrG,CAAC;AAED;;;;;;;;GAQG;AACH,MAAM,UAAU,cAAc,CAAC,SAAiB,EAAE,KAA8B;IAC9E,IAAI,CAAC;QACH,EAAE,CAAC,SAAS,CAAC,IAAI,CAAC,OAAO,CAAC,SAAS,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAC3D,cAAc,CACZ,SAAS,EACT,EAAE,EACF,GAAG,EAAE,CAAC,iBAAiB,CAAC,SAAS,EAAE,KAAK,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,EAAE,CAAC,EAAE,CAAC,aAAa,CAAC,CAAC,EAAE,IAAI,CAAC,CAAC,CAClF,CAAC;IACJ,CAAC;IAAC,MAAM,CAAC;QACP,4DAA4D;IAC9D,CAAC;AACH,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,eAAe,CAC7B,SAAiB,EACjB,KAA8B;IAE9B,IAAI,CAAC;QACH,EAAE,CAAC,SAAS,CAAC,IAAI,CAAC,OAAO,CAAC,SAAS,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAC3D,IAAI,MAAM,GAA4B,EAAE,CAAC;QACzC,cAAc,CAAC,SAAS,EAAE,EAAE,EAAE,GAAG,EAAE;YACjC,MAAM,GAAG,UAAU,CAAC,cAAc,CAAC,SAAS,CAAC,EAAE,KAAK,CAAC,CAAC;YACtD,iBAAiB,CAAC,SAAS,EAAE,MAAM,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,EAAE,CAAC,EAAE,CAAC,aAAa,CAAC,CAAC,EAAE,IAAI,CAAC,CAAC,CAAC;QAC/E,CAAC,CAAC,CAAC;QACH,OAAO,MAAM,CAAC;IAChB,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,cAAc,CAAC,SAAS,CAAC,CAAC;IACnC,CAAC;AACH,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,YAAY,CAAC,QAAgB;IAC3C,MAAM,MAAM,GAAG,eAAe,CAAC,QAAQ,CAAC,CAAC;IACzC,OAAO,OAAO,IAAI,MAAM,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,CAAC;AACjD,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,aAAa,CAC3B,KAA8B,EAC9B,QAA4B;IAE5B,MAAM,MAAM,GAAG,yBAAyB,CAAC,KAAK,EAAE,SAAS,CAAC,CAAC;IAC3D,IAAI,MAAM,CAAC,MAAM,GAAG,CAAC;QAAE,OAAO,MAAM,CAAC;IACrC,MAAM,YAAY,GAAG,CAAC,QAAQ,IAAI,EAAE,CAAC,CAAC,IAAI,CACxC,CAAC,CAAC,EAAE,EAAE,CACJ,OAAO,CAAC,CAAC,IAAI,EAAE,SAAS,KAAK,QAAQ;QACrC,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC;QAC3B,CAAC,CAAC,IAAI,CAAC,SAAS,KAAK,YAAY,CACpC,CAAC;IACF,MAAM,QAAQ,GAAG,yBAAyB,CAAC,YAAY,EAAE,IAAI,CAAC,SAAS,CAAC,CAAC;IACzE,OAAO,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,uBAAuB,CAAC;AAClE,CAAC;AAED,4DAA4D;AAC5D,MAAM,gBAAgB,GAAG,yBAAyB,CAAC;AAEnD;;;;;;;;;;GAUG;AACH,MAAM,UAAU,sBAAsB,CAAC,OAAO,GAA4B,EAAE;IAC1E,MAAM,WAAW,GAAG,CAAC,OAAe,EAAU,EAAE,CAC9C,sBAAsB,CAAC,OAAO,CAAC,GAAG,EAAE,EAAE,EAAE,CAAC,OAAO,EAAE,EAAE,OAAO,CAAC,CAAC;IAC/D,MAAM,UAAU,GAAG,OAAO,CAAC,kBAAkB,IAAI,CAAC,CAAC;IAEnD,OAAO,KAAK,IAAI,EAAE;QAChB,OAAO;YACL,sEAAsE;YACtE,mEAAmE;YACnE,iEAAiE;YACjE,qEAAqE;YACrE,mBAAmB;YACnB,sCAAsC,EAAE,KAAK,EAC3C,KAAK,EACL,MAAM,EACS,EAAE;gBACjB,MAAM,OAAO,GAAG,aAAa,CAAC,KAAgC,EAAE,MAAM,CAAC,QAAQ,CAAC,CAAC;gBACjF,MAAM,KAAK,GAAG,cAAc,CAAC,WAAW,CAAC,OAAO,CAAC,CAAC,CAAC;gBACnD,MAAM,QAAQ,GAAG,MAAM,CAAC,QAAQ,CAAC;gBACjC,MAAM,cAAc,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,KAAK,QAAQ,CAAC,CAAC;gBACxE,MAAM,OAAO,GAAG,QAAQ;qBACrB,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,KAAK,QAAQ,CAAC;qBACvC,KAAK,CAAC,CAAC,UAAU,CAAC,CAAC;gBAEtB,oEAAoE;gBACpE,uCAAuC;gBACvC,MAAM,YAAY,GAAoB;oBACpC,IAAI,EAAE;wBACJ,EAAE,EAAE,gBAAgB;wBACpB,SAAS,EAAE,YAAY;wBACvB,IAAI,EAAE,MAAM;wBACZ,IAAI,EAAE,EAAE,OAAO,EAAE,CAAC,EAAE;wBACpB,KAAK,EAAE,YAAY;wBACnB,KAAK,EAAE,EAAE,UAAU,EAAE,YAAY,EAAE,OAAO,EAAE,YAAY,EAAE;qBAC3D;oBACD,KAAK,EAAE;wBACL;4BACE,EAAE,EAAE,GAAG,gBAAgB,OAAO;4BAC9B,SAAS,EAAE,YAAY;4BACvB,SAAS,EAAE,gBAAgB;4BAC3B,IAAI,EAAE,MAAM;4BACZ,SAAS,EAAE,IAAI;4BACf,IAAI,EAAE,+BAA+B,IAAI,CAAC,SAAS,CAAC,KAAK,CAAC,EAAE;yBAC7D;qBACF;iBACF,CAAC;gBAEF,oEAAoE;gBACpE,gEAAgE;gBAChE,MAAM,IAAI,GAAG,CAAC,GAAG,cAAc,EAAE,GAAG,OAAO,EAAE,YAAY,CAAC,CAAC;gBAC3D,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAC;gBACpB,QAAQ,CAAC,IAAI,CAAC,GAAG,IAAI,CAAC,CAAC;YACzB,CAAC;YAED,sEAAsE;YACtE,sEAAsE;YACtE,uEAAuE;YACvE,iCAAiC,EAAE,KAAK,EAAE,KAAK,EAAE,MAAM,EAAiB,EAAE;gBACxE,MAAM,OAAO,GAAG,aAAa,CAAC,KAAK,CAAC,CAAC;gBACrC,MAAM,KAAK,GAAG,cAAc,CAAC,WAAW,CAAC,OAAO,CAAC,CAAC,CAAC;gBACnD,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,OAAO,CAAC,EAAE,CAAC;oBACnC,MAAM,CAAC,OAAO,GAAG,EAAE,CAAC;gBACtB,CAAC;gBACD,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,eAAe,IAAI,CAAC,SAAS,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;YAC9D,CAAC;YAED,sEAAsE;YACtE,mEAAmE;YACnE,mEAAmE;YACnE,kEAAkE;YAClE,oBAAoB,EAAE,KAAK,EAAE,KAAK,EAAE,MAAM,EAAiB,EAAE;gBAC3D,MAAM,QAAQ,GAAG,MAAM,CAAC,MAAM,IAAI,EAAE,CAAC;gBACrC,IAAI,OAAO,QAAQ,KAAK,QAAQ;oBAAE,OAAO;gBACzC,MAAM,KAAK,GAAG,YAAY,CAAC,QAAQ,CAAC,CAAC;gBACrC,IAAI,KAAK,EAAE,CAAC;oBACV,MAAM,OAAO,GAAG,aAAa,CAAC,KAAK,CAAC,CAAC;oBACrC,eAAe,CAAC,WAAW,CAAC,OAAO,CAAC,EAAE,KAAK,CAAC,CAAC;gBAC/C,CAAC;YACH,CAAC;SACwB,CAAC;IAC9B,CAAC,CAAC;AACJ,CAAC"}
+{"version":3,"file":"plugin.js","sourceRoot":"","sources":["../src/plugin.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAgEG;AAEH,OAAO,EAAE,MAAM,EAAE,MAAM,kBAAkB,CAAC;AAC1C,OAAO,KAAK,IAAI,MAAM,WAAW,CAAC;AAClC,OAAO,EAAE,eAAe,EAAE,aAAa,EAAE,MAAM,uBAAuB,CAAC;AACvE,OAAO,EAAE,iBAAiB,EAAE,MAAM,kBAAkB,CAAC;AACrD,OAAO,EAAE,cAAc,EAAE,MAAM,kBAAkB,CAAC;AAClD,OAAO,EAAE,aAAa,EAAE,MAAM,YAAY,CAAC;AAE3C,mFAAmF;AACnF,MAAM,CAAC,MAAM,SAAS,GAAG,YAAY,CAAC;AAEtC;;;;;;;;;;;;;;;GAeG;AACH,MAAM,CAAC,MAAM,gBAAgB,GAAG,MAAM,CAAC,MAAM,CAAC;IAC5C,EAAE,EAAE,SAAS;IACb,KAAK,CAAC,KAAK,CAAC,GAAG;QACb,MAAM,QAAQ,GAAG,IAAI,eAAe,EAAE,CAAC;QACvC,MAAM,QAAQ,GAAG,CAAC,SAAiB,EAAU,EAAE,CAAC,aAAa,CAAC,QAAQ,EAAE,SAAS,CAAC,CAAC;QAEnF,qEAAqE;QACrE,qEAAqE;QACrE,8DAA8D;QAC9D,MAAM,KAAK,GAAG,IAAI,iBAAiB,CAAC;YAClC,SAAS,EAAE,GAAG,CAAC,QAAQ,CAAC,OAAO,CAAC,SAAS;SAC1C,CAAC,CAAC;QAEH,MAAM,GAAG,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,MAAM,EAAE,EAAE;YAClC,aAAa,CAAC,MAAM,EAAE,EAAE,KAAK,EAAE,QAAQ,EAAE,QAAQ,EAAE,CAAC,CAAC;QACvD,CAAC,CAAC,CAAC;QAEH,uEAAuE;QACvE,wEAAwE;QACxE,yEAAyE;QACzE,uEAAuE;QACvE,MAAM,UAAU,GAAG,IAAI,eAAe,EAAE,CAAC;QACzC,KAAK,CAAC,KAAK,IAAI,EAAE;YACf,IAAI,CAAC;gBACH,IAAI,KAAK,EAAE,MAAM,KAAK,IAAI,GAAG,CAAC,KAAK,CAAC,SAAS,CAAC,EAAE,MAAM,EAAE,UAAU,CAAC,MAAM,EAAE,CAAC,EAAE,CAAC;oBAC7E,QAAQ,CAAC,WAAW,CAAC,KAAK,CAAC,CAAC;gBAC9B,CAAC;YACH,CAAC;YAAC,MAAM,CAAC;gBACP,mEAAmE;gBACnE,kEAAkE;gBAClE,kEAAkE;YACpE,CAAC;QACH,CAAC,CAAC,EAAE,CAAC;QAEL,uEAAuE;QACvE,kEAAkE;QAClE,oEAAoE;QACpE,uEAAuE;QACvE,wEAAwE;QACxE,0BAA0B;QAC1B,MAAM,GAAG,CAAC,OAAO,CAAC,IAAI,CAAC,SAAS,EAAE,CAAC,KAAK,EAAE,EAAE;YAC1C,MAAM,KAAK,GAAG,QAAQ,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC;YACxC,IAAI,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,CAAC;gBAAE,OAAO;YACjC,MAAM,KAAK,GAAG,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YAChC,MAAM,IAAI,GAAG,cAAc,CAAC;gBAC1B,KAAK;gBACL,SAAS,EAAE,IAAI,CAAC,QAAQ,CAAC,KAAK,CAAC,gBAAgB,EAAE,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC;gBACtE,KAAK;aACN,CAAC,CAAC;YACH,IAAI,IAAI,CAAC,MAAM,KAAK,CAAC;gBAAE,OAAO;YAC9B,KAAK,CAAC,MAAM,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,CAAC,CAAC;QAClD,CAAC,CAAC,CAAC;QAEH,OAAO,GAAG,EAAE;YACV,UAAU,CAAC,KAAK,EAAE,CAAC;QACrB,CAAC,CAAC;IACJ,CAAC;CACF,CAAC,CAAC;AAEH,eAAe,gBAAgB,CAAC"}