Conversation
…ement runtime api
…-runtime-apis Expose proxy type filtering rules via runtime api
PR #2645 cached chain-bought alpha in SubnetProtocolAlpha instead of burning it, but its deregistration settlement lumped SubnetAlphaIn (the AMM pool reserve) together with the cached protocol alpha. Including the pool reserve as a settlement weight diluted every staker's pro-rata TAO share. Convert only the chain-bought alpha (SubnetProtocolAlpha) to TAO on dissolution, pro-rata by the same formula as every staker's alpha, and recycle that TAO back to the chain. SubnetAlphaIn is excluded from the settlement denominator. - Update dissolve_protocol_alpha_share_is_not_paid_to_users for the corrected settlement (staker share no longer diluted by alpha-in) - Add dissolve_chain_bought_alpha_is_converted_to_tao_and_recycled asserting the bought-alpha TAO is recycled (reduces total issuance) and never leaks to the owner - Bump spec_version 414 -> 415
Fix chain buys accounting
sam0x17
added
skip-cargo-audit
🛡️ AI Review — Skeptic (security review)VERDICT: SAFE Baseline scrutiny: author is an established repo admin; branch is the legitimate testnet -> main deployment/hotfix path. No Gittensor allowlist match found in the trusted local lists. No security findings. I reviewed the runtime economic change, the migration/cutover storage, the proxy filter macro replacement, dependency changes, and protected review-prompt paths. The PR does not modify The runtime proxy filter definitions appear equivalent to the removed hand-written FindingsNo findings. ConclusionI found no malicious behavior or security vulnerability in the reviewed diff. The PR is acceptable from the Skeptic perspective. # 🔍 AI Review — Auditor (domain review) has not yet run on this PR. |
|
🔄 AI review updated — Skeptic: SAFE |
3 participants
What's Changed