add min stake const

[open-junius]

Description

Add a new const for min stake.

Related Issue(s)

• Closes #[issue number]

Type of Change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation update
• Other (please describe):

Breaking Change

If this PR introduces a breaking change, please provide a detailed description of the impact and the migration path for existing applications.

Checklist

• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have run ./scripts/fix_rust.sh to ensure my code is formatted and linted correctly
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• Any dependent changes have been merged and published in downstream modules

Screenshots (if applicable)

Please include any relevant screenshots or GIFs that demonstrate the changes made.

Additional Notes

Please provide any additional information or context that may be helpful for reviewers.

[github-actions]

🛡️ AI Review — Skeptic (security review)

VERDICT: SAFE

BASELINE scrutiny: open-junius has write permission, established subtensor PR history, no Gittensor allowlist hit; add-min-stake-const -> devnet-ready.

Reviewed the prefetched PR metadata, prior Skeptic comment, contributor signals, and full patch. No prior Skeptic findings were present. The diff does not modify .github/ai-review/* or .github/copilot-instructions.md; the runtime change replaces the hardcoded default min stake with a config constant set to the same value, plus test/mock wiring and generated contract-test lock metadata.

Findings

No findings.

Conclusion

No malicious behavior or security vulnerability found in the static diff. The change does not add privileged paths, new extrinsics, runtime panic sources, build scripts, or new Cargo dependencies.

---

🔍 AI Review — Auditor (domain review)

VERDICT: 👍

LIKELY Gittensor-associated established contributor: repo writer with substantial recent subtensor PR history; no allowlist hit, no same-purpose duplicate found from prefetched open PR titles.

The Auditor proposed a replacement PR description, but the current body is non-trivial; not overwriting. Maintainers: ask the Auditor to regenerate if you want it.

No domain findings. The runtime change keeps the effective default minimum stake at 2_000_000 RAO while moving it behind a Config constant and wiring the visible runtime/mock implementations.

Validation note: git diff --check passed. A targeted cargo check -p pallet-subtensor --tests could not start because rustup attempted to write under read-only /home/runner/.rustup; the devnet spec-version query also could not resolve dev.chain.opentensor.ai from this sandbox, so no spec-version auto-fix was applied.

Findings

No findings.

Conclusion

The PR is a narrow config extraction with no new extrinsics, migrations, arithmetic, or storage iteration risk. I do not see a domain issue that should block merge.

[github-actions]

🔄 AI review updated — Skeptic: SAFE Auditor: 👍