8000 feat: browser prepared-query limit setters (minigraf#872) by adityamukho · Pull Request #22 · project-minigraf/minigraf-wasm · GitHub
[go: up one dir, main page]

Skip to content

feat: browser prepared-query limit setters (minigraf#872) - #22

Merged
adityamukho merged 4 commits into
v3from
feat/872-binding-overrides
Nov 10, 2026
Merged

adityamukho merged 4 commits into
v3from
feat/872-binding-overrides

Conversation

@adityamukho
@adityamukho adityamukho commented Nov 10, 2026 •
Copy link
Copy Markdown
Contributor

The browser side of minigraf#872. BrowserPreparedQuery gains withMaxDerivedFacts, withQueryMemoryLimit and withMaxIterations from core (minigraf#883). BrowserDb takes no options, so the setters are the whole browser surface.

  • The generated minigraf_wasm.d.ts lists the three setters (checked from a local wasm-pack build).
  • consumer-test/browser/contract.js runs withMaxIterations(1) against the packed package, checks the receiver is unchanged, and checks a value of 2^32 or more is no limit. It passed locally in headless Chromium.
  • wasi-pkg tests pass with a WASI binary built from the core branch.
  • README documents the setters and that BrowserDb takes no open options.

Pin. Cargo.toml pins core d9755c02, the v3 merge commit of project-minigraf/minigraf#883. Moving the pin from 02243f16 needed no other fix. The macOS jobs run on macos-15 (the macos-14 runner was retired; fixed separately on v3).

Refs project-minigraf/minigraf#872.

🤖 Generated with Claude Code

https://claude.ai/code/session_01H2a7QouMzKp3tAWxgVsxRf

@adityamukho adityamukho changed the title feat: override list, open options and prepared limit setters (minigraf#872) feat: browser prepared-query limit setters (minigraf#872) Nov 10, 2026
adityamukho and others added 3 commits November 10, 2026 13:16
…minigraf#872)

BrowserPreparedQuery gains withMaxDerivedFacts, withQueryMemoryLimit and
withMaxIterations from core. The contract runs one against the packed
package and checks the receiver is unchanged; the README documents them
and that BrowserDb takes no open options.

Refs minigraf#872.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01H2a7QouMzKp3tAWxgVsxRf
Temporary: re-pinned to the v3 merge commit once minigraf#883 merges.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01H2a7QouMzKp3tAWxgVsxRf
@adityamukho
adityamukho force-pushed the feat/872-binding-overrides branch from 76acd5e to 186cd60 Compare November 10, 2026 10:19
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01H2a7QouMzKp3tAWxgVsxRf
@adityamukho
adityamukho marked this pull request as ready for review November 10, 2026 10:54
@adityamukho
adityamukho merged commit 9adb7a4 into v3 Nov 10, 2026
7 checks passed
@adityamukho
adityamukho deleted the feat/872-binding-overrides branch November 10, 2026 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

0