docs(node): confirm v6.6 SC migration model, add version-scoping note for operators#39
Conversation
PR SummaryLow Risk Overview The A new Reviewed by Cursor Bugbot for commit 913efa6. Bugbot is set up for automated code reviews on this repo. Configure here. |
… note The SC migration model on the skills-registry branch (memiavl_only enum + sc-keys-to-migrate-per-block) is confirmed correct against the v6.6.0-rc1 tag — the earlier review finding calling it 'unreleased main' predated the rc cut and checked only v6.5.x tags and main. Changes on top of the base branch: - Mirror the exact v6.6.0-rc1 toml template comment for sc-keys-to-migrate-per-block in the auto-generated app.toml block (drain-rate tuning guidance; 'defaults to memiavl_only' line removed to match seid init output). - Add a <Note> scoping the SC keys per version: v6.5.x uses cosmos_only/dual_write/split_write + sc-read-mode + sc-enable-lattice-hash, and each binary line rejects the other's sc-write-mode values at startup (no compat mapping in app/seidb.go), so operators must update app.toml when crossing v6.6. Also flags the post-v6.6 rework already on main (#3650): drain rate becomes the NumKeysToMigratePerBlock gov param and sc-write-mode-enable-auto (default true) derives the mode automatically. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
e8995ff to
913efa6
Compare
Context
A review finding on the SC config docs claimed the
[state-commit]model on the skills-registry branch (memiavl_onlymigration enum +sc-keys-to-migrate-per-block) described "unreleased main". That finding is outdated as of this morning:v6.6.0-rc1(tagged 2026-07-02) ships exactly this model, so the branch text is correct for v6.6 and stays.Verified against sei-chain tags
v6.5.2,v6.6.0-rc1, andmain@ 9f53204:sc-write-modecosmos_only/dual_write/split_writememiavl_only→migrate_evm→ … →flatkv_onlyautosc-read-mode/sc-enable-lattice-hashsc-keys-to-migrate-per-block(app.toml, default 1024)NumKeysToMigratePerBlockgov param (default 0)sc-write-mode-enable-auto = truederives it, ignores explicit keyChanges (on top of the skills-registry branch)
<Note>in the Giga Storage section: the SC keys documented here are v6.6+; v6.5.x uses the old key set, and each binary line rejects the other'ssc-write-modevalues at startup (no compat mapping inapp/seidb.go—ParseWriteModepanics), so operators must updateapp.tomlwhen crossing v6.6. The removedsc-read-mode/sc-enable-lattice-hashkeys are merely ignored. Also flags the post-v6.6 gov-driven rework already onmain.app.tomlblock: mirror the exactv6.6.0-rc1template comment forsc-keys-to-migrate-per-block(drain-rate tuning guidance) and drop the extra "defaults to memiavl_only" line so the block matches realseid initoutput.🤖 Generated with Claude Code