Repository navigation
chore(deps): bump sei-config to v0.0.29 across all three modules - #629
Conversation
v0.0.29 carries sei-config#54, which adds the 32 [evm] keys seid reads that the unified schema did not model, including per-IP rate limiting (rate_limiting_enabled, ip_rate_limit_rps, ip_rate_limit_burst, trusted_proxy_cidrs) and the request-admission limits around it. Until the pin moves, the sidecar rejects these as unknown override keys and drops them from the app.toml it renders, so managed nodes run on seid's compiled-in defaults with no way to tune them 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. Rendered app.toml files now carry the 32 keys explicitly at seid's defaults, so runtime behaviour on existing nodes is unchanged. Refs PLT-1553 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
PR SummaryMedium Risk Overview What the new pin enables: v0.0.29 models 32 Operational note: the next full or incremental config apply will emit those keys in Reviewed by Cursor Bugbot for commit a2d91ed. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
Bumps sei-config from v0.0.28 to v0.0.29 in all three modules, which lets the 32 seid [evm] keys be set through spec.overrides; go.mod and go.sum change consistently and the module's own go.mod hash is unchanged, so no new transitive dependencies come in. Nothing blocks: the defaults in v0.0.29 match seid's DefaultConfig and pass the new Validate() checks (burst 200 ≥ batch limit 100), configValues drift is keyed on the spec hash rather than the library version so the bump rolls no nodes, and the tree was read at the PR merge ref. The codex reading found nothing, and that matches this review, so it adds no findings.
seidroid review · decision approve · session e3f0c5c9882f45bd9d0817c6e41ae04d · turn resp_claude_6aa3d2ddd112a180895842ec706e4c80 · item 4d97360f74615253904ff01188529e07
Findings: 0 blocking | 0 non-blocking | 0 posted inline
Bumps
github.com/sei-protocol/sei-configfrom v0.0.28 to v0.0.29 in the root,sidecarandsidecarapimodules. Pins only; no source change. Part of PLT-1553.What it unlocks
v0.0.29 carries sei-protocol/sei-config#54, which adds the 32
[evm]keys seid reads that the unified schema didn't model:rate_limiting_enabled,ip_rate_limit_rps,ip_rate_limit_burst,trusted_proxy_cidrsbatch_request_limit,batch_response_max_size,max_request_body_bytes,max_concurrent_request_bytes,body_read_idle_timeout,ws_admission_timeout,max_open_connections,rpc_default_timeout,rpc_method_timeoutsmax_filters,max_log_bytes,trace_allowed_tracers,trace_bake_*and othersUntil the pin moves, the sidecar rejects these as
unknown override key, andWriteConfigToDirdrops them fromapp.toml. Managed nodes run on seid's compiled-in defaults with no way to tune them throughspec.overrides. After this bump they can be set asevm.<seid key>, e.g.evm.ip_rate_limit_rps.Effect on existing nodes
app.tomlfiles now include all 32 keys, set to seid's defaults, so seid behaves exactly as before.app.tomlrendered by v0.0.28 or earlier is missing these keys. On an incremental apply they read as defaults, not zeros, so the sidecar can't write backrate_limiting_enabled = falseormax_filters = 0.