Skip to content

feat(admin): close GAP btcli.sudo.senate-vote#12

Draft
unarbos wants to merge 134 commits into
mainfrom
orch/agcli-parity/fix-p1-senate-vote
Draft

feat(admin): close GAP btcli.sudo.senate-vote#12
unarbos wants to merge 134 commits into
mainfrom
orch/agcli-parity/fix-p1-senate-vote

Conversation

@unarbos

@unarbos unarbos commented May 29, 2026

Copy link
Copy Markdown
Owner

Summary

  • added a governance-specific admin raw --call senate-vote alias that maps to SubtensorModule.vote
  • implemented proposal hash + vote parsing (yes/no, aye/nay, true/false, 1/0) and proposal index lookup from Triumvirate.Voting
  • added new chain helpers for Triumvirate vote data/proposals lookup and senate vote submission
  • added parity test wiring under tests/parity/ and updated admin command docs + parity matrix row

Parity mapping

  • row: btcli.sudo.senate-vote
  • status: GAP -> COVERED_E2E
  • agcli path: agcli admin raw --call senate-vote --args '["0x<proposal_hash>","yes"]' --sudo-key //Alice --yes --output json

Verification

  • SKIP_METADATA_FETCH=1 cargo build --release --bin agcli
  • SKIP_METADATA_FETCH=1 cargo test --features e2e parse_senate_vote_args -- --nocapture
  • SKIP_METADATA_FETCH=1 cargo test --features e2e --test parity -- --nocapture ✅ (test runs and skips on runtimes without Triumvirate pallet)
  • SKIP_METADATA_FETCH=1 cargo test --features e2e -- --nocapture ⚠️ baseline failure unrelated to this change: events::tests::pretty_subtensor_variants_are_exhaustive

Notes

  • the localnet runtime in this VM reports Triumvirate missing, so the new parity test intentionally exits early with a skip message in that environment while still validating the executable path and preserving the write-path assertion logic for governance-enabled runtimes.
Open in Web Open in Cursor 

cursoragent and others added 30 commits May 28, 2026 15:11
… mission

Co-authored-by: Arbos <unarbos@users.noreply.github.com>
…rBase=main

Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
…ngRef to root branch

Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
…variants; preserve failure handoff

Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
…gh-fast (others reject at API)

Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
unconst and others added 30 commits May 29, 2026 08:17
Co-authored-by: Arbos <unarbos@users.noreply.github.com>
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