isaacbmiller · 2026-01-23T20:45:41Z

Pyodide's FFI (Foreign Function Interface) crashes when passing strings larger than 128MB between JavaScript and Python. This PR adds filesystem-based injection for large variables to work around this limit.

Before: All variables were embedded directly in Python code:
code = f"data = {repr(large_string)}\n" + code
After: Variables over 100MB are written to Pyodide's virtual filesystem and loaded via json.loads():

# Host writes JSON to /tmp/dspy_vars/data.json
# Generated code:
data = json.loads(
    open('/tmp/dspy_vars/data.json').read()
)

The PR also extracts a _send_request helper to deduplicate JSON-RPC request/response handling, which is reused by the new inject_var RPC method.

isaacbmiller · 2026-01-23T21:12:24Z

@codex

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 15a4ee8a72

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

isaacbmiller · 2026-01-23T22:50:01Z

approved offline

isaacbmiller added 2 commits January 23, 2026 15:45

fix large variable injection

49f578b

re-inject large vars on brokenPipe and correctly write filename

15a4ee8

isaacbmiller marked this pull request as ready for review January 23, 2026 21:12

chatgpt-codex-connector Bot reviewed Jan 23, 2026

View reviewed changes

Comment thread dspy/primitives/python_interpreter.py

add json as blocked key

7ab1adb

isaacbmiller merged commit 815f92a into main Jan 23, 2026
12 checks passed

isaacbmiller deleted the isaac/rlm-async-debug branch January 23, 2026 22:51

Navigation Menu

Uh oh!

fix(RLM): large variable injection - #9233