Skip to content

Conversation

@twittner
Copy link
Contributor

@twittner twittner commented Jan 14, 2026

Expose a key-value interface and removes the TIMEBOOST_STAMP environment variable. The timeboost executable passes in the name of the stamp file.

Expose a key-value interface and remove the TIMEBOOST_STAMP environment
variable. The `timeboost` executable passes in the name of the stamp
file.
Copy link
Contributor

@akonring akonring left a comment

Choose a reason for hiding this comment

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

LGTM 👍


let is_recover = {
let stamp = format!("{sign_pubkey}.stamp");
let mut io = StateIo::create().await.context("state i/o")?;
Copy link
Contributor

Choose a reason for hiding this comment

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

Thought: stamp could maybe be input to StateIo::create.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I wanted to keep state-io generic in the sense that consumers could load and store arbitrary key-value pairs, not necessarily just the stamp value.

@twittner twittner merged commit 4373a81 into main Jan 14, 2026
20 of 22 checks passed
@twittner twittner deleted the tw/simplify-state-io branch January 14, 2026 12:15
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.

3 participants