Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(scheduler-utils): wrap all side effects in schemas. Store ttl in byOwner cache #623

Merged
merged 1 commit into from
Apr 17, 2024

Conversation

TillaTheHun0
Copy link
Member

@TillaTheHun0 TillaTheHun0 commented Apr 17, 2024

We were running into issues as a result of nothing enforcing side-effect contracts. So this PR enforces them using zod similar to other ao libs (aoconnect) and ao units. The schemas are invoked by BL and used in side-effects tests, so that we can be confident that all layers are adhering to the data contract.

This also adds ttl to the byOwner cache, so that it can be used on the schedulerHint branch in locate.

Finally, this remaps { ..., owner } -> { ..., address } in data contracts, for ease of mapping.

@TillaTheHun0 TillaTheHun0 merged commit add734a into main Apr 17, 2024
2 checks passed
@TillaTheHun0 TillaTheHun0 deleted the tillathehun0/scheduler-utils-schema branch April 17, 2024 13:38
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