opencode-ipynb 0.1.0 → 0.1.2

package/README.md

@@ -167,15 +167,19 @@ OpenCode may use a different Python interpreter than the one in your terminal. T
 
    ```json
    {
-     "plugin": ["opencode-ipynb"],
-     "ipynb": {
-       "pythonPath": "C:/Users/Gabriel/miniconda3/envs/data/python.exe",
-        "preferUv": true,
-        "defaultTimeoutMs": 120000,
-        "defaultMaxOutputChars": 6000,
-        "warmKernel": false,
-        "allowOutsideWorktree": false
-      }
+     "plugin": [
+       [
+         "opencode-ipynb",
+         {
+           "pythonPath": "C:/Users/Gabriel/miniconda3/envs/data/python.exe",
+           "preferUv": true,
+           "defaultTimeoutMs": 120000,
+           "defaultMaxOutputChars": 6000,
+           "warmKernel": false,
+           "allowOutsideWorktree": false
+         }
+       ]
+     ]
    }
    ```
 
@@ -230,7 +234,7 @@ expected output.
 
 ## Release
 
-The published npm tarball contains only the runtime: `dist/`, `python/ipynb_runner.py`, `python/requirements.txt`, `README.md`, and `LICENSE`. Internal developer docs (`AGENTS.md`, `TODO.md`, the helper's dev scripts, the `.github/` workflows) are excluded from the npm package via `.npmignore`, and `AGENTS.md` / `TODO.md` are also gitignored so they are not versioned in this repository. Contributors track their work through GitHub Issues and Discussions.
+The published npm tarball contains only the runtime: `dist/`, `python/ipynb_runner.py`, `python/requirements.txt`, `README.md`, and `LICENSE`. Internal developer docs, helper dev scripts, and `.github/` workflows are excluded from the npm package via `.npmignore` and are also gitignored so they are not versioned in this repository. Contributors track their work through GitHub Issues and Discussions.
 
 Cut a release by tagging the commit and pushing the tag, then publish it on GitHub — the workflow runs only on the `published` release event:
 
@@ -321,16 +325,6 @@ test/
   unit/                   # bun:test suites
 ```
 
-## Roadmap
-
-See `TODO.md` for the full plan and review-fix history (v0.1 → v1.5). Highlights:
-
-- v0.2: polished `ipynb_outputs` UI, full `ipynb_export` polish.
-- v0.3: real `nbclient`-based execution, save outputs, timeout, traceback capture.
-- v0.4: image attachments, reproducibility reports.
-- v1.0: kernel lifecycle management, cross-platform tests, npm publish.
-- v1.1–v1.5: security, Effect, path, packaging, rich-output, and cleanup review fixes.
-
 ## License
 
 MIT. See `LICENSE`.

package/dist/format/markdown.d.ts

@@ -1,7 +1,6 @@
 import type { CellRaw } from "../domain/cell.js";
 import { type OutputFormatOptions } from "./outputs.js";
 import type { ToolAttachment } from "@opencode-ai/plugin";
-import type { SavedAttachment } from "../utils/attachments.js";
 export interface MarkdownReadOptions {
     readonly includeOutputs: boolean;
     readonly includeErrors: boolean;
@@ -14,7 +13,6 @@ export interface MarkdownReadOptions {
 export interface FormattedCell {
     readonly rendered: string;
     readonly attachments: ReadonlyArray<ToolAttachment>;
-    readonly savedAttachments: ReadonlyArray<SavedAttachment>;
 }
 export declare const formatCellMarkdown: (cell: CellRaw, index: number, optsInput?: Partial<MarkdownReadOptions>) => string;
 export declare const formatCellMarkdownDetailed: (cell: CellRaw, index: number, optsInput?: Partial<MarkdownReadOptions>) => Promise<FormattedCell>;

package/dist/format/markdown.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"markdown.d.ts","sourceRoot":"","sources":["../../src/format/markdown.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,OAAO,EAA4C,MAAM,mBAAmB,CAAA;AAC1F,OAAO,EAA8D,KAAK,mBAAmB,EAAE,MAAM,cAAc,CAAA;AACnH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,qBAAqB,CAAA;AACzD,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,yBAAyB,CAAA;AAE9D,MAAM,WAAW,mBAAmB;IAClC,QAAQ,CAAC,cAAc,EAAE,OAAO,CAAA;IAChC,QAAQ,CAAC,aAAa,EAAE,OAAO,CAAA;IAC/B,QAAQ,CAAC,eAAe,EAAE,OAAO,CAAA;IACjC,QAAQ,CAAC,cAAc,EAAE,MAAM,CAAA;IAC/B,QAAQ,CAAC,oBAAoB,CAAC,EAAE,OAAO,GAAG,SAAS,CAAA;IACnD,QAAQ,CAAC,UAAU,CAAC,EAAE,OAAO,GAAG,SAAS,CAAA;IACzC,QAAQ,CAAC,MAAM,EAAE,OAAO,CAAC,mBAAmB,CAAC,CAAA;CAC9C;AAED,MAAM,WAAW,aAAa;IAC5B,QAAQ,CAAC,QAAQ,EAAE,MAAM,CAAA;IACzB,QAAQ,CAAC,WAAW,EAAE,aAAa,CAAC,cAAc,CAAC,CAAA;IACnD,QAAQ,CAAC,gBAAgB,EAAE,aAAa,CAAC,eAAe,CAAC,CAAA;CAC1D;AAqJD,eAAO,MAAM,kBAAkB,GAC7B,MAAM,OAAO,EACb,OAAO,MAAM,EACb,YAAW,OAAO,CAAC,mBAAmB,CAAM,KAC3C,MASF,CAAA;AAED,eAAO,MAAM,0BAA0B,GACrC,MAAM,OAAO,EACb,OAAO,MAAM,EACb,YAAW,OAAO,CAAC,mBAAmB,CAAM,KAC3C,OAAO,CAAC,aAAa,CAWvB,CAAA"}
+{"version":3,"file":"markdown.d.ts","sourceRoot":"","sources":["../../src/format/markdown.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,OAAO,EAA4C,MAAM,mBAAmB,CAAA;AAC1F,OAAO,EAA8D,KAAK,mBAAmB,EAAE,MAAM,cAAc,CAAA;AACnH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,qBAAqB,CAAA;AAEzD,MAAM,WAAW,mBAAmB;IAClC,QAAQ,CAAC,cAAc,EAAE,OAAO,CAAA;IAChC,QAAQ,CAAC,aAAa,EAAE,OAAO,CAAA;IAC/B,QAAQ,CAAC,eAAe,EAAE,OAAO,CAAA;IACjC,QAAQ,CAAC,cAAc,EAAE,MAAM,CAAA;IAC/B,QAAQ,CAAC,oBAAoB,CAAC,EAAE,OAAO,GAAG,SAAS,CAAA;IACnD,QAAQ,CAAC,UAAU,CAAC,EAAE,OAAO,GAAG,SAAS,CAAA;IACzC,QAAQ,CAAC,MAAM,EAAE,OAAO,CAAC,mBAAmB,CAAC,CAAA;CAC9C;AAED,MAAM,WAAW,aAAa;IAC5B,QAAQ,CAAC,QAAQ,EAAE,MAAM,CAAA;IACzB,QAAQ,CAAC,WAAW,EAAE,aAAa,CAAC,cAAc,CAAC,CAAA;CACpD;AAkJD,eAAO,MAAM,kBAAkB,GAC7B,MAAM,OAAO,EACb,OAAO,MAAM,EACb,YAAW,OAAO,CAAC,mBAAmB,CAAM,KAC3C,MASF,CAAA;AAED,eAAO,MAAM,0BAA0B,GACrC,MAAM,OAAO,EACb,OAAO,MAAM,EACb,YAAW,OAAO,CAAC,mBAAmB,CAAM,KAC3C,OAAO,CAAC,aAAa,CAWvB,CAAA"}

package/dist/format/outputs.d.ts

@@ -1,4 +1,3 @@
-import { type SavedAttachment } from "../utils/attachments.js";
 import type { CellRaw } from "../domain/cell.js";
 import type { NotebookRaw } from "../domain/notebook.js";
 import type { ToolAttachment } from "@opencode-ai/plugin";
@@ -12,7 +11,6 @@ export interface OutputFormatOptions {
 export interface FormattedOutput {
     readonly rendered: string;
     readonly attachments: ReadonlyArray<ToolAttachment>;
-    readonly savedAttachments: ReadonlyArray<SavedAttachment>;
 }
 export declare const formatOutputs: (cell: CellRaw, optsInput?: Partial<OutputFormatOptions>) => string;
 export declare const formatOutputsDetailed: (cell: CellRaw, optsInput?: Partial<OutputFormatOptions>) => Promise<FormattedOutput>;

package/dist/format/outputs.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"outputs.d.ts","sourceRoot":"","sources":["../../src/format/outputs.ts"],"names":[],"mappings":"AAGA,OAAO,EAGL,KAAK,eAAe,EACrB,MAAM,yBAAyB,CAAA;AAEhC,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,mBAAmB,CAAA;AAChD,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,uBAAuB,CAAA;AAGxD,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,qBAAqB,CAAA;AAEzD,MAAM,WAAW,mBAAmB;IAClC,QAAQ,CAAC,aAAa,EAAE,OAAO,CAAA;IAC/B,QAAQ,CAAC,UAAU,CAAC,EAAE,OAAO,GAAG,SAAS,CAAA;IACzC,QAAQ,CAAC,cAAc,EAAE,MAAM,CAAA;IAC/B,QAAQ,CAAC,iBAAiB,EAAE,MAAM,CAAA;IAClC,QAAQ,CAAC,oBAAoB,CAAC,EAAE,OAAO,GAAG,SAAS,CAAA;CACpD;AASD,MAAM,WAAW,eAAe;IAC9B,QAAQ,CAAC,QAAQ,EAAE,MAAM,CAAA;IACzB,QAAQ,CAAC,WAAW,EAAE,aAAa,CAAC,cAAc,CAAC,CAAA;IACnD,QAAQ,CAAC,gBAAgB,EAAE,aAAa,CAAC,eAAe,CAAC,CAAA;CAC1D;AA2HD,eAAO,MAAM,aAAa,GACxB,MAAM,OAAO,EACb,YAAW,OAAO,CAAC,mBAAmB,CAAM,KAC3C,MA0DF,CAAA;AAED,eAAO,MAAM,qBAAqB,GAChC,MAAM,OAAO,EACb,YAAW,OAAO,CAAC,mBAAmB,CAAM,KAC3C,OAAO,CAAC,eAAe,CA+DzB,CAAA;AAED,eAAO,MAAM,yBAAyB,GAAI,UAAU,WAAW,EAAE,UAAU,MAAM,KAAG,MA8BnF,CAAA;AAED,eAAO,MAAM,mBAAmB,GAAI,QAAQ,MAAM,EAAE,KAAK,MAAM,KAAG,MAAsC,CAAA"}
+{"version":3,"file":"outputs.d.ts","sourceRoot":"","sources":["../../src/format/outputs.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,mBAAmB,CAAA;AAChD,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,uBAAuB,CAAA;AAGxD,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,qBAAqB,CAAA;AAEzD,MAAM,WAAW,mBAAmB;IAClC,QAAQ,CAAC,aAAa,EAAE,OAAO,CAAA;IAC/B,QAAQ,CAAC,UAAU,CAAC,EAAE,OAAO,GAAG,SAAS,CAAA;IACzC,QAAQ,CAAC,cAAc,EAAE,MAAM,CAAA;IAC/B,QAAQ,CAAC,iBAAiB,EAAE,MAAM,CAAA;IAClC,QAAQ,CAAC,oBAAoB,CAAC,EAAE,OAAO,GAAG,SAAS,CAAA;CACpD;AASD,MAAM,WAAW,eAAe;IAC9B,QAAQ,CAAC,QAAQ,EAAE,MAAM,CAAA;IACzB,QAAQ,CAAC,WAAW,EAAE,aAAa,CAAC,cAAc,CAAC,CAAA;CACpD;AAwHD,eAAO,MAAM,aAAa,GACxB,MAAM,OAAO,EACb,YAAW,OAAO,CAAC,mBAAmB,CAAM,KAC3C,MA0DF,CAAA;AAED,eAAO,MAAM,qBAAqB,GAChC,MAAM,OAAO,EACb,YAAW,OAAO,CAAC,mBAAmB,CAAM,KAC3C,OAAO,CAAC,eAAe,CA8DzB,CAAA;AAED,eAAO,MAAM,yBAAyB,GAAI,UAAU,WAAW,EAAE,UAAU,MAAM,KAAG,MA8BnF,CAAA;AAED,eAAO,MAAM,mBAAmB,GAAI,QAAQ,MAAM,EAAE,KAAK,MAAM,KAAG,MAAsC,CAAA"}

package/dist/index.js

@@ -27834,11 +27834,6 @@ var CARRIAGE_RETURN_PATTERN = /\r/g;
 var stripAnsi = (text) => text.replace(ANSI_PATTERN, "").replace(CARRIAGE_RETURN_PATTERN, "");
 
 // src/utils/attachments.ts
-import * as fs2 from "fs";
-import * as os from "os";
-import * as path4 from "path";
-import { randomBytes } from "crypto";
-import { pathToFileURL } from "url";
 var EXTENSION_BY_MIME = {
   "image/png": "png",
   "image/jpeg": "jpg",
@@ -27851,22 +27846,15 @@ var EXTENSION_BY_MIME = {
 var pickExtensionForMime = (mime) => {
   return EXTENSION_BY_MIME[mime.toLowerCase()];
 };
-var saveBase64Image = async (mime, base643, prefix = "img") => {
-  const ext = pickExtensionForMime(mime) ?? "bin";
-  const dir = path4.join(os.tmpdir(), "opencode-ipynb", `${prefix}-${process.pid}-${randomBytes(4).toString("hex")}`);
-  await fs2.promises.mkdir(dir, { recursive: true });
-  const buffer = Buffer.from(base643, "base64");
-  const filename = `${prefix}.${ext}`;
-  const filePath = path4.join(dir, filename);
-  await fs2.promises.writeFile(filePath, buffer);
-  return { path: filePath, mime, bytes: buffer.length, filename };
+var formatAttachment = (mime, base643, filename) => {
+  const ext = pickExtensionForMime(mime);
+  return {
+    type: "file",
+    mime,
+    url: `data:${mime};base64,${base643}`,
+    filename: filename ?? `image${ext ? `.${ext}` : ""}`
+  };
 };
-var formatAttachment = (a) => ({
-  type: "file",
-  mime: a.mime,
-  url: pathToFileURL(a.path).toString(),
-  filename: a.filename
-});
 
 // src/domain/output.ts
 var StreamOutputSchema = exports_external.object({
@@ -27967,11 +27955,9 @@ var formatDisplayWithAttachments = async (data, opts, optsInput, collector) => {
     if (isImageMime(mime)) {
       if (typeof value === "string") {
         if (shouldSaveImage(optsInput)) {
-          const saved = await saveBase64Image(mime, value);
-          const att = formatAttachment(saved);
+          const att = formatAttachment(mime, value);
           collector.attachments.push(att);
-          collector.saved.push(saved);
-          lines.push(`- ${mime}: (saved ${humanBytes(saved.bytes)} to ${att.url})`);
+          lines.push(`- ${mime}: (attached as image, ${humanBytes(Buffer.byteLength(value, "base64"))})`);
         } else {
           lines.push(`- ${imageNotice(mime, value)}`);
         }
@@ -28081,7 +28067,7 @@ ${e}`);
 var formatOutputsDetailed = async (cell, optsInput = {}) => {
   const opts = defaultOpts(optsInput);
   if (cell.cell_type !== "code") {
-    return { rendered: "", attachments: [], savedAttachments: [] };
+    return { rendered: "", attachments: [] };
   }
   const sections = [];
   let stdout = "";
@@ -28089,7 +28075,7 @@ var formatOutputsDetailed = async (cell, optsInput = {}) => {
   const displaySections = [];
   const resultSections = [];
   const errorSections = [];
-  const collector = { attachments: [], saved: [] };
+  const collector = { attachments: [] };
   for (const out of cell.outputs) {
     const kind = detectOutputKind(out);
     if (kind === "stream") {
@@ -28134,14 +28120,13 @@ ${r}`);
 ${e}`);
   }
   if (sections.length === 0) {
-    return { rendered: "(no output)", attachments: collector.attachments, savedAttachments: collector.saved };
+    return { rendered: "(no output)", attachments: collector.attachments };
   }
   return {
     rendered: sections.join(`
 
 `),
-    attachments: collector.attachments,
-    savedAttachments: collector.saved
+    attachments: collector.attachments
   };
 };
 var formatNotebookDiagnostics = (notebook, filePath) => {
@@ -28602,7 +28587,6 @@ ${JSON.stringify(cell.metadata, null, 2)}
       lines.push(formatted.rendered);
     }
     collector.attachments.push(...formatted.attachments);
-    collector.saved.push(...formatted.savedAttachments);
   } else if (opts.includeErrors) {
     const errOutputs = cell.outputs.filter((o) => o["output_type"] === "error");
     if (errOutputs.length > 0) {
@@ -28612,7 +28596,6 @@ ${JSON.stringify(cell.metadata, null, 2)}
       lines.push("");
       lines.push(formatted.rendered);
       collector.attachments.push(...formatted.attachments);
-      collector.saved.push(...formatted.savedAttachments);
     }
   }
   return lines.join(`
@@ -28654,15 +28637,15 @@ ${JSON.stringify(cell.metadata, null, 2)}
 };
 var formatCellMarkdownDetailed = async (cell, index, optsInput = {}) => {
   const opts = defaultReadOpts(optsInput);
-  const collector = { attachments: [], saved: [] };
+  const collector = { attachments: [] };
   if (cell.cell_type === "code") {
     const rendered = await formatCodeCellDetailed(cell, index, opts, collector);
-    return { rendered, attachments: collector.attachments, savedAttachments: collector.saved };
+    return { rendered, attachments: collector.attachments };
   }
   if (cell.cell_type === "markdown") {
-    return { rendered: formatMarkdownCell(cell, index, opts), attachments: [], savedAttachments: [] };
+    return { rendered: formatMarkdownCell(cell, index, opts), attachments: [] };
   }
-  return { rendered: formatRawCell(cell, index, opts), attachments: [], savedAttachments: [] };
+  return { rendered: formatRawCell(cell, index, opts), attachments: [] };
 };
 
 // src/services/NotebookReadService.ts
@@ -29419,7 +29402,7 @@ var makeOutputImpl = (pathSvc, fileSvc, permSvc) => {
 
 // src/services/NotebookExportService.ts
 import * as fsp2 from "fs/promises";
-import * as path5 from "path";
+import * as path4 from "path";
 
 // src/format/diff.ts
 var renderCell = (cell, cellIndex, options) => {
@@ -29646,7 +29629,7 @@ ${rendered}`;
       });
       yield* exports_Effect.tryPromise({
         try: async () => {
-          await fsp2.mkdir(path5.dirname(outAbs), { recursive: true });
+          await fsp2.mkdir(path4.dirname(outAbs), { recursive: true });
           await fsp2.writeFile(outAbs, finalRendered, "utf8");
         },
         catch: (err) => new NotebookWriteError({
@@ -30086,8 +30069,8 @@ var makeExecutionImpl = (pathSvc, fileSvc, permSvc, pythonSvc, execOpts) => {
 // src/services/PythonService.ts
 import { spawn as spawn2 } from "child_process";
 import { fileURLToPath } from "url";
-import * as fs3 from "fs";
-import * as path6 from "path";
+import * as fs2 from "fs";
+import * as path5 from "path";
 import * as readline from "readline";
 class PythonService extends exports_Context.Tag("@ipynb/PythonService")() {
 }
@@ -30159,13 +30142,13 @@ var findHelperWarned = false;
 var findHelper = (rel) => {
   const candidates = [];
   if (process.cwd())
-    candidates.push(path6.resolve(process.cwd(), rel));
-  const distDir = path6.dirname(__filename2);
-  candidates.push(path6.resolve(distDir, "..", "..", rel));
-  candidates.push(path6.resolve(distDir, "..", "..", "..", rel));
-  candidates.push(path6.resolve(distDir, "..", rel));
+    candidates.push(path5.resolve(process.cwd(), rel));
+  const distDir = path5.dirname(__filename2);
+  candidates.push(path5.resolve(distDir, "..", "..", rel));
+  candidates.push(path5.resolve(distDir, "..", "..", "..", rel));
+  candidates.push(path5.resolve(distDir, "..", rel));
   for (const c of candidates) {
-    if (fs3.existsSync(c))
+    if (fs2.existsSync(c))
       return c;
   }
   if (!findHelperWarned) {

package/dist/utils/attachments.d.ts

@@ -1,11 +1,4 @@
 import type { ToolAttachment } from "@opencode-ai/plugin";
-export interface SavedAttachment {
-    readonly path: string;
-    readonly mime: string;
-    readonly bytes: number;
-    readonly filename: string;
-}
 export declare const pickExtensionForMime: (mime: string) => string | undefined;
-export declare const saveBase64Image: (mime: string, base64: string, prefix?: string) => Promise<SavedAttachment>;
-export declare const formatAttachment: (a: SavedAttachment) => ToolAttachment;
+export declare const formatAttachment: (mime: string, base64: string, filename?: string) => ToolAttachment;
 //# sourceMappingURL=attachments.d.ts.map

package/dist/utils/attachments.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"attachments.d.ts","sourceRoot":"","sources":["../../src/utils/attachments.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,qBAAqB,CAAA;AAEzD,MAAM,WAAW,eAAe;IAC9B,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAA;IACrB,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAA;IACrB,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAA;IACtB,QAAQ,CAAC,QAAQ,EAAE,MAAM,CAAA;CAC1B;AAYD,eAAO,MAAM,oBAAoB,GAAI,MAAM,MAAM,KAAG,MAAM,GAAG,SAE5D,CAAA;AAED,eAAO,MAAM,eAAe,GAC1B,MAAM,MAAM,EACZ,QAAQ,MAAM,EACd,SAAQ,MAAc,KACrB,OAAO,CAAC,eAAe,CAazB,CAAA;AAED,eAAO,MAAM,gBAAgB,GAAI,GAAG,eAAe,KAAG,cAKpD,CAAA"}
+{"version":3,"file":"attachments.d.ts","sourceRoot":"","sources":["../../src/utils/attachments.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,qBAAqB,CAAA;AAYzD,eAAO,MAAM,oBAAoB,GAAI,MAAM,MAAM,KAAG,MAAM,GAAG,SAE5D,CAAA;AAED,eAAO,MAAM,gBAAgB,GAC3B,MAAM,MAAM,EACZ,QAAQ,MAAM,EACd,WAAW,MAAM,KAChB,cAQF,CAAA"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-ipynb",
-  "version": "0.1.0",
+  "version": "0.1.2",
   "description": "OpenCode community plugin that adds robust .ipynb (Jupyter Notebook) support: inspect, read, edit, run, outputs, clean, export, reproducibility reports, and warm kernels.",
   "type": "module",
   "packageManager": "bun@1.3.13",