Skip to content

Toolkit: Support zswap state chaining#879

Merged
jsidorenko merged 5 commits into
mainfrom
js/toolkit-support-zswap-chaining
Mar 9, 2026
Merged

Toolkit: Support zswap state chaining#879
jsidorenko merged 5 commits into
mainfrom
js/toolkit-support-zswap-chaining

Conversation

@jsidorenko

@jsidorenko jsidorenko commented Mar 6, 2026

Copy link
Copy Markdown
Contributor

Overview

Add support for zswap state file when calling a circuit.
We can automatically fetch the zswap state between the calls, but that doesn't help in cases when we need to batch multiple intents those touch zswap state.

📌 Submission Checklist

  • Changes are backward-compatible (or flagged if breaking)
  • Pull request description explains why the change is needed
  • Self-reviewed the diff
  • I have included a change file, or skipped for this reason:
  • If the changes introduce a new feature, I have bumped the node minor version
  • Update documentation (if relevant)
  • Updated AGENTS.md if build commands, architecture, or workflows changed
  • No new todos introduced

🧪 Testing Evidence

Please describe any additional testing aside from CI:

  • Additional tests are provided (if possible)

🔱 Fork Strategy

  • Node Runtime Update
  • Node Client Update
  • Other:
  • N/A

Links

Closes https://shielded.atlassian.net/browse/PM-20404

@jsidorenko jsidorenko requested a review from ozgb March 6, 2026 15:37
@jsidorenko jsidorenko requested a review from a team as a code owner March 6, 2026 15:37

@ozgb ozgb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

# Conflicts:
#	util/toolkit/src/commands/generate_intent.rs
@jsidorenko jsidorenko enabled auto-merge March 9, 2026 09:19
@github-actions

github-actions Bot commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

kics-logo

KICS version: v2.1.19

Category Results
CRITICAL CRITICAL 0
HIGH HIGH 0
MEDIUM MEDIUM 47
LOW LOW 3
INFO INFO 59
TRACE TRACE 0
TOTAL TOTAL 109
Metric Values
Files scanned placeholder 26
Files parsed placeholder 26
Files failed to scan placeholder 0
Total executed queries placeholder 73
Queries failed to execute placeholder 0
Execution time placeholder 11

@jsidorenko jsidorenko added this pull request to the merge queue Mar 9, 2026
Merged via the queue into main with commit 30aa31d Mar 9, 2026
48 of 56 checks passed
@jsidorenko jsidorenko deleted the js/toolkit-support-zswap-chaining branch March 9, 2026 18:40
gilescope pushed a commit that referenced this pull request Apr 8, 2026
@gilescope gilescope added this to the node-1.0.0 milestone Apr 10, 2026
m2ux added a commit that referenced this pull request Apr 23, 2026
Signed-off-by: Mike Clay <mike.clay@shielded.io>
m2ux added a commit that referenced this pull request Apr 23, 2026
Signed-off-by: Mike Clay <mike.clay@shielded.io>
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.

4 participants