Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(host): Improve host builder abstractions #982

Closed
clabby opened this issue Jan 30, 2025 · 2 comments · Fixed by #992
Closed

feat(host): Improve host builder abstractions #982

clabby opened this issue Jan 30, 2025 · 2 comments · Fixed by #992
Labels
A-host Area: host binary K-feature Kind: feature

Comments

@clabby
Copy link
Collaborator

clabby commented Jan 30, 2025

Overview

#915 started on a refactor to kona-host to make it more modular, and in the process stopped exporting the default CLI / host implementations. Some consumers of kona use these default implementations verbatim, such as op-succinct or kailua.

The abstractions put in place in #915 are a bit too restrictive to allow for this. We should explore ways to split up the idea of the "host SDK" from the default, canonical implementations to better support these downstream consumers that are just changing the transport for the client program.

@clabby clabby added A-host Area: host binary K-feature Kind: feature labels Jan 30, 2025
@clabby
Copy link
Collaborator Author

clabby commented Jan 30, 2025

cc @ratankaliani + @hashcashier - please leave any feedback here if you'd like some input on where this heads. Apologies if this change has caused you guys any trouble, in the middle of both Isthmus + Interop so things are getting shuffled around a bit 🙏

@ratankaliani
Copy link
Contributor

No worries! Generally, access to the SingleChainHostCli is what we need. We extend this CLI with additional arguments to add support for proving with OP Succinct & the Succinct Prover Network + Alt-DA.

Do you imagine just having another crate that uses kona-host to construct the default types and exposes them. kona-cli or kona-host-cli feel apt here. You could move the existing interop and single implementations to that crate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-host Area: host binary K-feature Kind: feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants