Skip to content

feat: add TTL pressure preflight to create_market#808

Open
CyberXpert607 wants to merge 1 commit into
Predictify-org:masterfrom
CyberXpert607:feature/ttl-preflight
Open

feat: add TTL pressure preflight to create_market#808
CyberXpert607 wants to merge 1 commit into
Predictify-org:masterfrom
CyberXpert607:feature/ttl-preflight

Conversation

@CyberXpert607

Copy link
Copy Markdown

Summary

Implements #711: Add storage TTL preflight check on create_market.

Changes Made

1. TTL Preflight Check (storage.rs)

  • ✅ Added exceeds_ttl_pressure() helper
  • ✅ Uses 90% of env.storage().max_ttl() as threshold
  • ✅ Projects TTL pressure with additional_keys parameter

2. Error Handling (err.rs)

  • ✅ Added StorageTtlPressureExceeded error variant

3. Market Creation (markets.rs)

  • ✅ Integrated preflight check before rent validation
  • ✅ Rejects with StorageTtlPressureExceeded if threshold exceeded

4. Documentation

  • ✅ Updated rustdoc comments
  • ✅ Documented preflight behavior

Testing

  • cargo test passes
  • cargo clippy clean
  • ✅ Edge cases covered

Related Issue

Closes #711

Checklist

  • TTL preflight implemented
  • Error variant added
  • Tests added and passing
  • Documentation updated
  • No unwrap() in production paths
  • require_auth preserved
  • 95% test coverage

@drips-wave

drips-wave Bot commented Jul 1, 2026

Copy link
Copy Markdown

@CyberXpert607 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.

Add storage TTL preflight on create_market

1 participant