Skip to content

Spec 466: fee cap only — restore 4s block budget - #3182

Merged
unarbos merged 1 commit into
mainfrom
cursor/spec-466-drop-12s-e52d
Sep 18, 2026
Merged

unarbos merged 1 commit into
mainfrom
cursor/spec-466-drop-12s-e52d

Conversation

@unarbos

@unarbos unarbos commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

Drops the 12s native block-budget bump from #3181. Keeps the four-hotkey staking scan fee cap and spec_version 466.

Native MAXIMUM_BLOCK_WEIGHT, scheduler, and weight_per_gas are restored to the exact pre-#3180 4s values. The EVM/scheduler split that existed only for the 12s bump is gone. runtime/tests/block_capacity.rs (12s / 65-batch admission pins) is removed.

Admission weight is unchanged: a 25-item transfer_stake batch still fits; a 65 does not. Clients still chunk. Fees drop.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)

What this is not

The earlier 466 train on dfdfba1 (12s) is cancelled. Inti should approve this 466 Propose, not leftover 465 or the cancelled 12s 466.

Open in Web Open in Cursor 

Keep the four-hotkey staking scan fee cap and spec 466.
Drop the 12s native block bump and the EVM/scheduler split
that existed only for it. Tests that pinned 12s are removed.

Co-authored-by: Arbos <unarbos@users.noreply.github.com>
@unarbos
unarbos merged commit cdffbe2 into main Sep 18, 2026
19 of 20 checks passed
@vercel

vercel Bot commented Sep 18, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
subtensor Building Building Preview Sep 18, 2026 12:39am UTC

Request Review

@unarbos

unarbos commented Sep 18, 2026

Copy link
Copy Markdown
Contributor Author

@IntiTechnologies spec 466 Propose is the one to approve. Fee cap only. No block-budget change.

The earlier 466 train on dfdfba1 (12s native block) is void — cancelled: https://github.com/RaoFoundation/subtensor/actions/runs/35291554030

This merge restores the exact pre-#3180 4s native budget. The 4-hotkey transfer_stake scan fee cap stays. That is the client-payout fix (PayTAO / Lium). A 65-item batch still will not admit; chunk ≤25.

Train: https://github.com/RaoFoundation/subtensor/actions/runs/35292102814
Merge: cdffbe2f7ab0c37ab07884387bfbd6443dca178d (#3182)
Live before this train: devnet 465, testnet 465, finney 464

Approve this 466 Propose, not leftover 465 (https://github.com/RaoFoundation/subtensor/actions/runs/35267938998), not leftover 464 Watch (https://github.com/RaoFoundation/subtensor/actions/runs/35262050154), and not the cancelled 12s 466.

BasketTradingEnabled stays off. Do not sudo-enable baskets. Do not merge #3135.

After you approve, signers:

btcli upgrade sign --url https://github.com/RaoFoundation/subtensor/releases/tag/v466 -w <your-wallet>

@github-actions

github-actions Bot commented Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

🛡️ AI Review — Skeptic (security review)

VERDICT: SAFE

LOW contributor risk / baseline scrutiny: write-access maintainer with substantive merged history; no listed Gittensor association. Branch targets main.

Reviewed the complete two-file diff. The native block budget returns to 4 seconds; effective scheduler and EVM gas-weight budgets remain unchanged. No dependencies or AI-review trust files changed.

Static analysis only; no PR code, builds, tests, or Cargo commands executed. git diff --check passed; git status --short was clean.

Findings

No findings.

Conclusion

No malicious behavior or introduced security vulnerability found in this rollback.


🔍 AI Review — Auditor (domain review)

VERDICT: 👍

Established contributor with write access and substantive merged history; Gittensor association UNKNOWN.

Restores the native block budget to 4 seconds while preserving effective scheduler and EVM gas-weight budgets, the staking fee cap, and spec 466. The implementation matches the described rollback.

This PR is the better candidate. Recommend closing #3180. Its fee-cap work is already integrated through #3181, while its remaining 12-second budget conflicts with the intended rollback. Other listed overlaps concern separate features.

git diff --check passed; git status --short was clean. cargo fmt --check --all could not run because rustup required a temporary file in read-only /home/runner/.rustup. No files modified; builds and tests were not needed for runtime confirmation.

Findings

No findings.

Conclusion

No substantive correctness issues found in this focused rollback.

@github-actions

Copy link
Copy Markdown
Contributor

🔄 AI review updated — Skeptic: SAFE Auditor: 👍

This branch was successfully deployed

1 active deployment
Preview — 41dd4f4c Deployed Sep 18, 2026 by vercel[bot]
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.

1 participant