Skip to content

fix: resolve issues #1082, #1083, #1084, #1085 - #1146

Merged
Maki-Zeninn merged 4 commits into
Maki-Zeninn:mainfrom
Mystery-CLI:feature/tasks
Aug 26, 2026
Merged

fix: resolve issues #1082, #1083, #1084, #1085#1146
Maki-Zeninn merged 4 commits into
Maki-Zeninn:mainfrom
Mystery-CLI:feature/tasks

Conversation

@Mystery-CLI

Copy link
Copy Markdown
Contributor

Summary

This PR addresses four issues across router-timelock, router-access, and router-quote contracts:

  1. test(router-timelock): get_operations_by_status has no test coverage #1085: Add test coverage for get_operations_by_status.
  2. bug(router-access): MaxGrantsPerRoleExceeded check reads RoleMemberCount after it was already incremented, off-by-one #1084: Fix off-by-one check in MaxGrantsPerRoleExceeded.
  3. bug(router-quote): get_all_configured_routes reports the flat/default fee for routes configured only via fee tiers #1083: Report tier fee in get_all_configured_routes.
  4. feat(router-timelock): queue() does not validate that dependency op_ids reference existing operations #1082: Validate that dependency op_ids reference existing operations in queue().

Changes

router-timelock

router-access

router-quote

Closes #1082
Closes #1083
Closes #1084
Closes #1085

@drips-wave

drips-wave Bot commented Aug 26, 2026

Copy link
Copy Markdown

@Mystery-CLI 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

@Maki-Zeninn
Maki-Zeninn merged commit 1527b2a into Maki-Zeninn:main Aug 26, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment