Skip to content

Comments

feat: expose rpc listener#1709

Open
juan518munoz wants to merge 2 commits intonextfrom
jmunoz-expose-rpclistener
Open

feat: expose rpc listener#1709
juan518munoz wants to merge 2 commits intonextfrom
jmunoz-expose-rpclistener

Conversation

@juan518munoz
Copy link
Collaborator

After #1688 the miden-client lost the capability of using the testing prover due to it's previously used structures/methods becoming private or removed. This PR re-adds this functionality.

- Improved tracing span fields ([#1650](https://github.com/0xMiden/miden-node/pull/1650))
- Replaced NTX Builder's in-memory state management with SQLite-backed persistence; account states, notes, and transaction effects are now stored in the database and inflight state is purged on startup ([#1662](https://github.com/0xMiden/miden-node/pull/1662)).
- [BREAKING] Reworked `miden-remote-prover`, removing the `worker`/`proxy` distinction and simplifying to a `worker` with a request queue ([#1688](https://github.com/0xMiden/miden-node/pull/1688)).
- Fixed `TransactionHeader` serialization for row insertion on database & fixed transaction cursor on retrievals ([#1701](https://github.com/0xMiden/miden-node/issues/1701)).
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Moved this previous entry here as it is better suited here.

@juan518munoz juan518munoz marked this pull request as ready for review February 25, 2026 14:37
Copy link
Collaborator

@Mirko-von-Leipzig Mirko-von-Leipzig Feb 25, 2026

Choose a reason for hiding this comment

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

I'm going to need some more context in how this is being used? If its just for testing why not just implement a very basic proving server?

As a general rule I'm not very comfortable having "libraries" exposed from the node since we're a binary application. Changes here would not be considered breaking.

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.

2 participants