Conversation
- Add persistence configuration to RuntimeConfig (enable_persistence, persistence_base_dir, session_id) - Add RuntimeBuilder methods for persistence settings (enable_persistence, session_id, persistence_dir) - Implement PersistenceWorker creation in Runtime::build() when enabled - Implement ProverWorker creation that reads from same action log file - ProverWorker now uses FileActionLog::open() to read from PersistenceWorker's output - Add proper worker shutdown handling for persistence_worker_handle - Add new RuntimeError variants: InvalidConfig, InvalidEntityId, MissingProvider - Remove unused imports from prover.rs (Tick, ActionRepository) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Critical bug fixes: - Fix ProverWorker offset tracking (was skipping entries by jumping to EOF) - Add flush after action log append to prevent race conditions - Implement real state hash using bincode serialization - Fix checkpoint serialization by switching from JSON to bincode Environment and configuration: - Add dotenvy for .env file loading - Implement all environment variables (ENABLE_PERSISTENCE, ENABLE_ZK_PROVING, GAME_SESSION_ID, SAVE_DATA_DIR, CHECKPOINT_INTERVAL) - Use directories crate for platform-specific system directories - Add checkpoint interval configuration Proof tracking improvements: - Add proof index persistence in ProverWorker - Create dedicated directories for proof_indices/ and proofs/ - Add before/after state hashes to proof generated logs - Track proof file metadata (filename, size) Logging improvements: - Implement file logging with tracing-appender - Remove stderr output to prevent TUI interference - Add session-specific log directories Code quality: - Fix all clippy warnings (unused imports, auto-deref, collapsed ifs) - Derive Default for CliConfig instead of manual implementation - Add #[allow(dead_code)] for hash_proof_data utility 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
…ilds
The RISC0_SKIP_BUILD environment variable in .cargo/config.toml was
preventing risc0_build::embed_methods() from compiling the guest program,
resulting in "Malformed ProgramBinary" errors at runtime.
Root cause: build.rs checks env::var("RISC0_SKIP_BUILD").is_ok(), which
returns true if the variable exists with ANY value (including "0").
Solution: Remove RISC0_SKIP_BUILD from cargo config entirely. Developers
can use `RISC0_SKIP_BUILD=1 cargo build` when needed for fast iteration.
Also simplified .env.example documentation.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Add convenience script to build only the RISC0 zkVM guest program without rebuilding the entire workspace. Usage: ./scripts/build-guest.sh # Debug build ./scripts/build-guest.sh --release # Release build Also updated CLAUDE.md to reflect correct ZK build workflow and clarify RISC0_SKIP_BUILD usage. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.