8000 feat: set values (#316) by adityamukho · Pull Request #19 · project-minigraf/minigraf-java · GitHub
[go: up one dir, main page]

Skip to content

feat: set values (#316) - #19

Merged
adityamukho merged 2 commits into
v3from
feat/316-set-values
Oct 24, 2026
Merged

adityamukho merged 2 commits into
v3from
feat/316-set-values

Conversation

@adityamukho
Copy link
Copy Markdown
Contributor

Step 24 of the minigraf#316 plan (PR 4), JVM.

  • Shim synced with minigraf-binding-template#8: MiniGrafValue.Set both ways (fact records, log writer, bind values), a nested set is API-021, execute() JSON returns a set as an array, wildcard arms for core's #[non_exhaustive] Value.
  • Sugar: a Kotlin/Java Set binds a set value. Elements are the plain values or a MiniGrafValue, and a UUID element is a ref. A non-value element (e.g. TxCount) or a List is IllegalArgumentException; a nested set reaches core → API-021.
  • Pin core v3 → 7c8c5540.
  • Tests: PreparedTest (sets, typed sets, equality with a stored set, JSON + fact-record output, UUID elements, API-021, IAE). Consumer contract (Java): a set comes back and binds.

Local run: 35 cargo tests and 33 Gradle tests pass.

Depends on minigraf-binding-template#8 (same shim).

🤖 Generated with Claude Code

https://claude.ai/code/session_01TcpoiFyFeQQQELL872LfQF

adityamukho and others added 2 commits October 25, 2026 02:11
Shim synced with the binding template: MiniGrafValue.Set both ways, a
nested set is API-021, execute() returns a set as an array. A Kotlin or
Java Set binds a set value (UUID elements are refs). Pin core v3 at
7c8c5540.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TcpoiFyFeQQQELL872LfQF
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TcpoiFyFeQQQELL872LfQF
@adityamukho
adityamukho merged commit a98c816 into v3 Oct 24, 2026
23 checks passed
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