chore(deps): bump sei-config to v0.0.28 across all three modules - #509
Conversation
v0.0.28 carries storage.state_commit.write_mode_enable_auto (sei-config#51) and the pointer-resolution fix that shipped with it (#52). Until the pin moves, the sidecar's override resolution rejects the new key as unknown, so a node cannot be pinned to an explicit state-commit write mode through spec.overrides. All three modules move together so the schema stays consistent across the controller, the sidecar that resolves overrides, and the shared API types. Pins only; no source change. Verified that both keys the release unlocks now resolve through ValidateIntent and ResolveIntent, and that WriteModeEnableAuto defaults to nil so an unset field renders no key — which is what keeps this bump behaviour-neutral for every existing node. The pointer fix also makes consensus.unsafe_bypass_commit_timeout_override settable for the first time; no existing node can carry it, because setting a pointer field failed before.
PR SummaryLow Risk Overview The bump keeps the override/config schema aligned everywhere the controller, sidecar, and shared API types resolve Reviewed by Cursor Bugbot for commit e7b3c81. Bugbot is set up for automated code reviews on this repo. Configure here. |
Bumps
sei-configv0.0.27 → v0.0.28 in all three Go modules. Pins only; no source change.Why
v0.0.28 carries
storage.state_commit.write_mode_enable_auto(sei-config#51) and the pointer-resolution fix that shipped with it (sei-config#52). Until the pin moves, the sidecar's override resolution rejects the new key as unknown, so a node cannot be pinned to an explicit state-commit write mode throughspec.overrides.All three modules move together so the schema stays consistent across the controller, the sidecar that resolves overrides, and the shared API types.
What the bump unlocks, verified
Both keys the release makes reachable now resolve through the controller's own dependency:
That last line is what keeps this behaviour-neutral: the field defaults to nil, so an unset field renders no key and every existing node keeps the binary's own default. The legacy tag is
sc-write-mode-enable-auto,omitempty, so nil genuinely omits.consensus.unsafe_bypass_commit_timeout_overridebecomes settable for the first time — no existing node can carry it, because setting a pointer field failed before sei-config#51.Testing
go build ./...in all three modulesgo test ./...in all three modulesmake tidy-check(go mod tidy -diffper module)make verify-generatedgolangci-lintv2.12.1 with--new-from-merge-base, all three modulesThe lint run matches CI's invocation and version. I ran it before pushing this time — the same check caught a real
goconstfinding on the last controller PR that my other verifiers missed.Rollout note
The harbor rollout for the previous bump has settled: all five nodes are on
bd9a558withNodeUpdateInProgress=FalseandSidecarReady=True, and the controller lease is held by abd9a558pod.sei-protocol/platform#1584 (the frozen RPC node) is deliberately paused so this change can catch up. Landing it now would mean rolling all five harbor nodes twice, and a frozen node's pod replacement is not free — it re-enters block sync and has to reach its freeze height again. One roll, with the node created once on the final image.
🤖 Generated with Claude Code