Skip to content

Conversation

@swift1337
Copy link
Collaborator

@swift1337 swift1337 commented Dec 23, 2025

This PR adds various "quality of life" improvements to the codebase

  • Rename loadtest bin to catalyst
  • Improve Makefile
    • make install
  • Consolidate linters & fixers
  • Explicit args & env parsing
  • Optimize docker builds
  • Propagate logger and env to ctx
  • Fetch wallet nonces concurrently
  • Fetch tx receipts concurrently (opt-in via env var)

Closes STACK-2018


Note

CLI and build/CI updates

  • Rename loadtest binary to catalyst with new entrypoint at cmd/catalyst; remove old cmd/loadtest
  • New config package for flag/env parsing (e.g., CATALYST_CONCURRENT_RECEIPTS, DEV_LOGGING) and context propagation
  • Centralized structured logging with file output under /tmp/catalyst; logger/env plumbed via context
  • Streamlined Dockerfile (module download cache) and Makefile (build, install, fmt, lint); updated README and .dockerignore
  • CI: switch to golangci-lint GitHub Action with a project .golangci.yml

Runtime and performance improvements

  • Ethereum: optional concurrent tx receipt collection in metrics; concurrent wallet nonce initialization; improved block/tx logging; tests added for metrics, runner, and tx factory
  • Cosmos: inject logger into metrics collector; runner and client minor refactors and logging; small fixes (nonce handling, formatting)

Result handling

  • Standardize saving results/logs under /tmp/catalyst with safer file perms

Written by Cursor Bugbot for commit 98f5677. This will update automatically on new commits. Configure here.

@swift1337 swift1337 self-assigned this Dec 23, 2025
@swift1337 swift1337 marked this pull request as ready for review December 23, 2025 11:21
@linear
Copy link

linear bot commented Dec 23, 2025

@aljo242
Copy link
Collaborator

aljo242 commented Dec 23, 2025

are there downstream changes that need to be made after we change the binary name? (infra, etc?)

@swift1337
Copy link
Collaborator Author

are there downstream changes that need to be made after we change the binary name? (infra, etc?)

No, we use docker, I updated Dockerfile

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the final PR Bugbot will review for you during this billing cycle

Your free Bugbot reviews will reset on January 16

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@swift1337 swift1337 merged commit 44a659a into main Jan 6, 2026
4 checks passed
@swift1337 swift1337 deleted the feat/improvements branch January 6, 2026 18:41
@swift1337 swift1337 restored the feat/improvements branch January 14, 2026 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants