Conversation
|
The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3404 +/- ##
==========================================
- Coverage 59.24% 59.22% -0.03%
==========================================
Files 2110 2111 +1
Lines 174149 174222 +73
==========================================
Hits 103175 103175
- Misses 62041 62114 +73
Partials 8933 8933
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
825b721 to
66f51e7
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 66f51e7. Configure here.
| .DS_Store | ||
| build/ | ||
| cache/ | ||
| evm_stress |
There was a problem hiding this comment.
Gitignore pattern matches source directory too broadly
Medium Severity
The evm_stress gitignore pattern (without a leading /) matches any file or directory named evm_stress at any level in the tree, including scripts/evm_stress/. While already-tracked files like main.go remain tracked, any new files added under scripts/evm_stress/ (e.g. test files, additional Go sources) will be silently ignored by git. The pattern likely needs a leading slash (/evm_stress) to restrict the match to the root-level build artifact only.
Reviewed by Cursor Bugbot for commit 66f51e7. Configure here.
| } | ||
| time.Sleep(300 * time.Millisecond) | ||
| } | ||
| } |
There was a problem hiding this comment.
Infinite loop in waitForBalance lacks timeout
Low Severity
waitForBalance spins in an infinite for loop polling for a positive balance, with no timeout or context cancellation check. If the genesis patching fails or the node doesn't produce a block, this hangs forever with no diagnostic output. Since ctx is context.Background(), there's no external cancellation path either.
Reviewed by Cursor Bugbot for commit 66f51e7. Configure here.


Summary
./scripts/evm_stress.sh, which initializes a localseidnode, patches genesis with deterministic sender accounts, starts the node, and runs the transfer workload.scripts/evm_stress/main.go, which generates deterministic sender keys, optionally dumps their derived Sei addresses for genesis funding, and sends nonce-0 EVM transfers from unique senders to a shared recipient at the target rate.evm_stressbuild artifact.This is split out from #3376 so the investigation tooling can be reviewed independently from logging and execution optimizations.
Test Plan
go test ./scripts/evm_stressgo run github.com/securego/gosec/v2/cmd/gosec@v2.22.11 ./scripts/evm_stress/...Note
Low Risk
Low risk: adds standalone local stress-test scripts and ignores a new build artifact, without modifying chain runtime logic.
Overview
Adds a local EVM stress-testing workload:
scripts/evm_stress.shinitializes a localseid, bulk-patchesgenesis.jsonwith many pre-funded deterministic sender accounts, starts the node, tails only a couple targeted log lines, waits for the EVM RPC, then runs the workload.Introduces
scripts/evm_stress/main.go, a Go load generator that can dump sender Sei bech32 addresses for genesis funding and then sends singlenonce=0EIP-1559 transfers from 50k unique senders to one recipient at a target rate (500 TPS) using a shared rate limiter across workers.Updates
.gitignoreto exclude theevm_stresslocal build artifact.Reviewed by Cursor Bugbot for commit 66f51e7. Bugbot is set up for automated code reviews on this repo. Configure here.