Conversation
|
@techisigu 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! 🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
solver_registry: timelocked two-step admin transfer (#393)
This PR delivers one acceptance-criteria item from #393. #385, #395 and #396 are referenced so that they close with this PR, but nothing from them is implemented here.
#393 Admin transfer, pause, and bond-token safety for
solver_registryWhat existed: the registry had no way to change its admin at all. The address set in
initializewas permanent.Done (AC1):
propose_admin(new_admin): admin only. Stores(new_admin, eta)witheta = now + ADMIN_TIMELOCK_DELAY(48 h, matchingintent_settlement) and emitsadmin_transfer_proposed(new_admin, eta). A new proposal replaces the pending one and resets the timer.accept_admin(new_admin): two-step. It must be the proposed address (Unauthorizedotherwise), runs only oncenow >= eta(AdminTimelockNotElapsed), andnew_adminmust sign, so the role can't be handed to an address nobody controls. Emitsadmin_transferred(old, new).cancel_admin_transfer(): admin only.NoPendingAdminTransferif nothing is pending. Emitsadmin_transfer_cancelled.get_pending_admin() -> Option<(Address, u64)>.AdminTimelockNotElapsed = 16andNoPendingAdminTransfer = 17. Codes 13–15 are claimed by open PRs feat(solver_registry): exactly-once settlement writes per intent (#387 #388 #390 #391) #445 and feat(solver_registry): timelocked writer rotation (#380 #384 #386 #389) #446 (and feat(solver_registry): timelocked writer rotation (#380 #384 #386 #389) #446 uses the nameTimelockNotElapsed), so both the codes and the names were chosen not to collide on merge.docs/solver-registry-interface.mdadmin table and error table updated.Not done in this PR:
pausewith a guardian role for stake/register/writer calls, with exits kept open (AC2).rescue_tokensexcluding the bond token (AC3).#385 Versioned v2 proof payload
Not done in this PR:
params_hashand a 32-bytesrc_user, v1/v2 migration in the registry, and settlementparams_hashverification.#395 Batch tier and eligibility views
Not done in this PR:
list_solvers, settlement tier caching, and resource measurements.#396 Permissionless, soulbound reputation badges
Not done in this PR:
sync_badge, removal or timelocking of admin mint/burn, and thebadge_of/tier_since/historyreads withbadge_changedevents.Verification
In
solver_registry:cargo test: 29 passed, 0 failed (23 existing + 6 new).cargo fmt --check: no findings on lines this PR adds.mainalready has fmt drift in these files, left untouched.cargo clippy --all-targets -- -D warnings: fails onmainwith current stable clippy (manual_range_containsinset_tier_threshold, pre-existing). There are no findings on lines this PR adds.Closes #385
Closes #393
Closes #395
Closes #396