Skip to content

fix(runtime): sync docs with v0x04 checkpoint, fix unmarshal errors#21

Merged
simonovic86 merged 1 commit intomainfrom
claude/awesome-hofstadter
Mar 5, 2026
Merged

fix(runtime): sync docs with v0x04 checkpoint, fix unmarshal errors#21
simonovic86 merged 1 commit intomainfrom
claude/awesome-hofstadter

Conversation

@simonovic86
Copy link
Owner

Changes

  • Updated checkpoint format documentation from v0x02 (57-byte header) to v0x04 (209-byte header)

    • Added fields: majorVersion, leaseGeneration, leaseExpiry, prevHash, agentPubKey, signature
    • Updated README, BUDGET_MODEL.md, and MIGRATION_PROTOCOL.md
  • Fixed silent unmarshal errors in agent examples

    • Added error checking after decoding in example/main.go and reconciliation/main.go
    • Panics with descriptive message on unmarshal failure
  • Fixed hardcoded agent ID in runner.go

    • Changed log statement from "local-agent" to instance.AgentID for accurate logging

Impact

Documentation now reflects current checkpoint format. Error handling prevents silent state corruption. Logging provides accurate agent identification.

…rshal and hardcoded agent ID

Update all documentation referencing the outdated 57-byte v0x02 checkpoint
header to the current 209-byte v0x04 format. Align README project phase
status with actual implementation (Phases 2–5 complete). Add d.Err()
checks to example agent Unmarshal implementations to catch corrupt
checkpoint data. Replace hardcoded "local-agent" string with
instance.AgentID in runner logging.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@simonovic86 simonovic86 merged commit 9c53dc6 into main Mar 5, 2026
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.

1 participant