Skip to content

Persist per-investor escrow storage keys#302

Open
Hillzo wants to merge 3 commits into
Liquifact:mainfrom
Hillzo:enhancement/persistent-investor-keys
Open

Persist per-investor escrow storage keys#302
Hillzo wants to merge 3 commits into
Liquifact:mainfrom
Hillzo:enhancement/persistent-investor-keys

Conversation

@Hillzo
Copy link
Copy Markdown

@Hillzo Hillzo commented May 29, 2026

closes #253

Summary

  • Move InvestorContribution, InvestorEffectiveYield, InvestorClaimNotBefore, and InvestorClaimed from instance storage to persistent storage.
  • Keep caller-facing read/write semantics unchanged, including defaults for absent keys.
  • Bump SCHEMA_VERSION to 6 and document the redeploy requirement because instance keys cannot be enumerated for an in-place migration.
  • Update storage documentation and tests for the new persistent per-investor key behavior.

Security and correctness

  • Auth boundaries remain unchanged.
  • Contribution arithmetic still uses checked operations.
  • Per-investor claim markers remain idempotent and address-scoped.
  • Per-address persistent TTL can be extended through bump_ttl when investor addresses are supplied.

Validation

  • cargo test
  • Result: 392 passed; 0 failed; 11 ignored; doc-tests passed.

@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented May 29, 2026

@Hillzo Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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.

Move per-investor contribution and yield keys from instance to persistent storage

1 participant