Skip to content

Redistribution for AVS developers #492

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

Merged
merged 41 commits into from
Jun 5, 2025
Merged

Redistribution for AVS developers #492

merged 41 commits into from
Jun 5, 2025

Conversation

MadelineAu
Copy link
Contributor

Updated AVS developer content for redistribution. Ready for review with the following caveats:

  1. I haven't the top level concept material hasn't yet. Once that exists, will add the relevant links.
  2. Same comment for links through to repo docs for core contracts. Will add these once release cut.

MadelineAu and others added 30 commits February 5, 2025 12:42
Copy link

vercel bot commented May 29, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
eigenlayer-docs ✅ Ready (Inspect) Visit Preview Jun 3, 2025 1:57am

@MadelineAu MadelineAu changed the title [WIP] Redistribution for AVS developers Redistribution for AVS developers May 29, 2025
@@ -18,11 +18,8 @@ The EigenLayer core contracts are documented in the [eigenlayer-contracts](https
| [AllocationManager](https://github.com/Layr-Labs/eigenlayer-contracts/tree/testnet-sepolia/docs#allocationmanager) | Responsible for creating Operator Sets, and Operator registrations to Operator Sets. The Allocation Manager also tracks allocation of stake to a Operator Set, and enables AVSs to slash that stake.
| [RewardsCoordinator](https://github.com/Layr-Labs/eigenlayer-contracts/tree/testnet-sepolia/docs#allocationmanager) | Enables AVSs to distribute ERC20 tokens to Operators and Restakers who delegated assets to Operators. The RewardsCoordinator tracks the rewards and enables Operators and Restakers to claim them.
| [PermissionController](https://github.com/Layr-Labs/eigenlayer-contracts/tree/testnet-sepolia/docs#permissioncontroller) | Enables AVSs and operators to delegate the ability to call certain core contract functions to other addresses. For more information, refer to [User Access Management](../../../eigenlayer/concepts/uam/user-access-management.md). |
| [AVSDirectory](https://github.com/Layr-Labs/eigenlayer-contracts/tree/testnet-sepolia/docs#avsdirectory) | Has been replaced by AllocationManager and will be deprecated in a future release. |
| SlashEscrowFactory | Responsible for burning or redistribution of slashed funds. The SlashEscrowFactory enforces the Slash Escrow Delay, deploys the Slash Escrow clone for each slash, and releases funds from the Slash Escrow contract.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

link to contracts doc repo

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

might be worth bringing in how it is used by the Redistribution Recipient role

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

## Returned by `slashOperator`

The `slashOperator` function returns the `slashId` and number of shares slashed for each strategy. The `slashId` is
incremented for an OperatorSet each time an Operator Set is slashed.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggest "this can be used for programmatically handling slashings"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added


## Redistribution

Redistribution may enable AVSs to benefit from a theft related to slashing so additional design care must be taken to consider
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@phileigenlabs for review

Copy link
Contributor

@non-fungible-nelson non-fungible-nelson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MadelineAu MadelineAu marked this pull request as ready for review June 4, 2025 23:15
@MadelineAu MadelineAu changed the title Redistribution for AVS developers (do not merge yet) Redistribution for AVS developers Jun 4, 2025
@MadelineAu MadelineAu changed the title (do not merge yet) Redistribution for AVS developers Redistribution for AVS developers Jun 5, 2025
Copy link
Collaborator

@wesfloyd wesfloyd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@wesfloyd wesfloyd merged commit 6048cb6 into main Jun 5, 2025
1 check passed
@wesfloyd wesfloyd deleted the redistributionAVS branch June 5, 2025 17:06
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.

3 participants