-
Notifications
You must be signed in to change notification settings - Fork 5
Tasks and messages #78
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
Conversation
5577349 to
70aaf57
Compare
1e08138 to
7a45fb5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR introduces a new task-based architecture to AVM by replacing the existing model with Messages and Tasks. The major changes include adding Messages as communication primitives that translate to resources and implementing Tasks as intermediate data structures for modular translation of message sends.
- Introduces Messages and Tasks as core abstractions for AVM message passing
- Removes Ecosystems, Functions, and Intents
- Adapts existing Apps and translation logic to work with the new Task-based system
Reviewed Changes
Copilot reviewed 38 out of 38 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| AVM/Task.lean | Implements the Task data structure with parameters, messages, and actions |
| AVM/Message.lean | Defines Message structure for object-to-object communication |
| AVM/Class/Translation.lean | Updates class translation to generate Tasks instead of direct transactions |
| Apps/*.lean | Adapts existing apps to use the new Class-based structure without Ecosystems |
| Prelude/HList.lean | Adds heterogeneous list support for Task parameter handling |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
Uh oh!
There was an error while loading. Please reload this page.