feat(erc20factory): erc20factory precompile

.github/workflows/check-changelog.yml

@@ -0,0 +1,53 @@
+on:
+  pull_request:
+    types: [ opened, synchronize, reopened, ready_for_review, edited ]
+    paths: [ "**/*.go" ]
+name: Changelog Reminder
+jobs:
+  remind:
+    name: Changelog Reminder
+    runs-on: ubuntu-latest
+    if: ${{ !github.event.pull_request.draft }}
+    permissions:
+      contents: read
+      pull-requests: write
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - name: Check PR title for semantic commit type
+        run: |
+          # Get the PR title
+          PR_TITLE="${{ github.event.pull_request.title }}"
+          echo "PR Title: $PR_TITLE"
+
+          # Check if PR title starts with feat, refactor, or fix
+          if echo "$PR_TITLE" | grep -qE "^(feat|refactor|fix)(\(.+\))?!?:"; then
+            echo "✅ PR title has relevant semantic commit type (feat, refactor, or fix)"
+            echo "has_relevant_pr_title=true" >> $GITHUB_ENV
+          else
+            echo "ℹ️ PR title doesn't have relevant semantic commit type. Skipping changelog check."
+            echo "has_relevant_pr_title=false" >> $GITHUB_ENV
+          fi
+
+      - name: Check if CHANGELOG.md was modified
+        if: env.has_relevant_pr_title == 'true'
+        run: |
+          # Get the list of changed files in this PR
+          CHANGED_FILES=$(git diff --name-only origin/${{ github.base_ref }}...HEAD)
+
+          # Check if CHANGELOG.md (case insensitive) is in the changed files
+          if echo "$CHANGED_FILES" | grep -qi "changelog\.md"; then
+            echo "✅ CHANGELOG.md has been modified in this PR"
+            echo "changelog_modified=true" >> $GITHUB_ENV
+          else
+            echo "❌ CHANGELOG.md has not been modified in this PR"
+            echo "changelog_modified=false" >> $GITHUB_ENV
+          fi
+
+      - name: Fail if changelog not updated
+        if: env.has_relevant_pr_title == 'true' && env.changelog_modified == 'false'
+        run: |
+          echo "::error::CHANGELOG.md must be updated for PRs with feat, refactor, or fix commits"
+          exit 1

.github/workflows/lint.yml

@@ -52,23 +52,14 @@ jobs:
           make lint-go
 
   markdown-lint:
-    name: markdownlint (cli2, minimal, non-blocking)
+    name: markdownlint (cli2, minimal)
     runs-on: ubuntu-latest
     timeout-minutes: 10
     permissions:
-      contents: write   # allows auto-fix pushes on same-repo PRs; forks stay read-only
+      contents: read   # read-only permissions
     steps:
-      # Same-repo PRs: checkout the head branch so fixes can be pushed back
-      - name: Checkout PR head (same-repo)
-        if: github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name == github.repository
-        uses: actions/checkout@v4
-        with:
-          token: ${{ secrets.GITHUB_TOKEN }}
-          ref: ${{ github.event.pull_request.head.ref }}
-
-      # Fork PRs, merge queue, and any other case: default checkout (safe merge ref)
-      - name: Checkout (default)
-        if: github.event_name == 'merge_group' || github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name != github.repository
+      # Standard checkout for all cases
+      - name: Checkout
         uses: actions/checkout@v4
 
       # Only run if markdown or configs changed
@@ -85,32 +76,12 @@ jobs:
         with:
           node-version: "20"
 
-      # Lint (non-blocking): auto-fix on same-repo PRs, never fail the job
-      - name: markdownlint (check or fix)
+      # Lint: check only, fail on errors
+      - name: markdownlint (check only)
         id: md_lint
         if: env.GIT_DIFF
-        continue-on-error: true
         uses: DavidAnson/markdownlint-cli2-action@v16
         with:
           globs: "**/*.md"
           config: .markdownlint.yml
-          fix: ${{ github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name == github.repository }}
-
-      # If fixes applied, commit & push (same-repo PRs only, not merge queue)
-      - name: Push auto-fixes (same-repo PRs only)
-        if: env.GIT_DIFF && github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name == github.repository
-        run: |
-          if [[ -n $(git status -s) ]]; then
-            git config --local user.email "41898282+github-actions[bot]@users.noreply.github.com"
-            git config --local user.name "github-actions[bot]"
-            git add -A
-            git commit -m "Auto-fix markdown lint issues"
-            git push
-          fi
-
-      # Warn if any issues remain after auto-fix (or on forks)
-      - name: Report remaining markdown issues (non-blocking)
-        if: env.GIT_DIFF && steps.md_lint.outcome == 'failure'
-        run: |
-          echo "::warning::Markdown lint issues remain after the check/fix step. For local fixes, run:"
-          echo "  npx markdownlint-cli2 --config .markdownlint.yml --fix '**/*.md'"
+          fix: false

.github/workflows/pr_title.yml

@@ -0,0 +1,15 @@
+name: PR Conventional Commit Validation
+
+on:
+  pull_request:
+    types: [opened, synchronize, reopened, edited]
+
+jobs:
+  validate-pr-title:
+    runs-on: ubuntu-latest
+    steps:
+      - name: PR Conventional Commit Validation
+        uses:  ytanikin/pr-conventional-commits@1.4.0
+        with:
+          task_types: '["feat","fix","docs","test","ci","refactor","perf","chore","revert","style","build"]'
+          add_label: 'false'

.github/workflows/system-test.yml

@@ -29,6 +29,9 @@ jobs:
         with:
           version: stable
       - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+          fetch-tags: true
       - uses: technote-space/get-diff-action@v6.1.2
         with:
           PATTERNS: |

CHANGELOG.md

@@ -2,7 +2,56 @@
 
 ## UNRELEASED
 
-## v0.4.0
+### DEPENDENCIES
+
+### BUG FIXES
+
+- [\#471](https://github.com/cosmos/evm/pull/471) Notify new block for mempool in time
+- [\#492](https://github.com/cosmos/evm/pull/492) Duplicate case switch to avoid empty execution block
+- [\#509](https://github.com/cosmos/evm/pull/509) Allow value with slashes when query token_pairs
+- [\#495](https://github.com/cosmos/evm/pull/495) Allow immediate SIGINT interrupt when mempool is not empty
+- [\#416](https://github.com/cosmos/evm/pull/416) Fix regression in CometBlockResultByNumber when height is 0 to use the latest block. This fixes eth_getFilterLogs RPC.
+- [\#545](https://github.com/cosmos/evm/pull/545) Check if mempool is not nil before accepting nonce gap error tx.
+- [\#585](https://github.com/cosmos/evm/pull/585) Use zero constructor to avoid nil pointer panic when BaseFee is 0d
+- [\#591](https://github.com/cosmos/evm/pull/591) CheckTxHandler should handle "invalid nonce" tx
+
+### IMPROVEMENTS
+
+- [\#538](https://github.com/cosmos/evm/pull/538) Optimize `eth_estimateGas` gRPC path: short-circuit plain transfers, add optimistic gas bound based on `MaxUsedGas`.
+- [\#513](https://github.com/cosmos/evm/pull/513) Replace `TestEncodingConfig` with production `EncodingConfig` in encoding package to remove test dependencies from production code.
+- [\#467](https://github.com/cosmos/evm/pull/467) Replace GlobalEVMMempool by passing to JSONRPC on initiate.
+- [\#352](https://github.com/cosmos/evm/pull/352) Remove the creation of a Geth EVM instance, stateDB during the AnteHandler balance check.
+- [\#496](https://github.com/cosmos/evm/pull/496) Simplify mempool instantiation by using configs instead of objects.
+- [\#512](https://github.com/cosmos/evm/pull/512) Add integration test for appside mempool.
+- [\#568](https://github.com/cosmos/evm/pull/568) Avoid unnecessary block notifications when the event bus is already set up.
+- [\#511](https://github.com/cosmos/evm/pull/511) Minor code cleanup for `AddPrecompileFn`.
+- [\#576](https://github.com/cosmos/evm/pull/576) Parse logs from the txResult.Data and avoid emitting EVM events to cosmos-sdk events.
+- [\#584](https://github.com/cosmos/evm/pull/584) Fill block hash and timestamp for json rpc.
+- [\#582](https://github.com/cosmos/evm/pull/582) Add block max-gas (from genesis.json) and new min-tip (from app.toml/flags) ingestion into mempool config
+- [\#580](https://github.com/cosmos/evm/pull/580) add appside mempool e2e test
+- [\#598](https://github.com/cosmos/evm/pull/598) Reduce number of times CreateQueryContext in mempool.
+- [\#606](https://github.com/cosmos/evm/pull/606) Regenerate mock file for bank keeper related test.
+- [\#609](https://github.com/cosmos/evm/pull/609) Make `erc20Keeper` optional in the EVM keeper
+- [\#624](https://github.com/cosmos/evm/pull/624) Cleanup unnecessary `fix-revert-gas-refund-height`.
+- [\#635](https://github.com/cosmos/evm/pull/635) Move DefaultStaticPrecompiles to /evm and allow projects to set it by default alongside the keeper.
+
+
+### FEATURES
+
+- [\#346](https://github.com/cosmos/evm/pull/346) Add eth_createAccessList method and implementation
+- [\#405](https://github.com/cosmos/evm/pull/405) Add erc20 factory precompile.
+- [\#502](https://github.com/cosmos/evm/pull/502) Add block time in derived logs.
+- [\#588](https://github.com/cosmos/evm/pull/588) go-ethereum metrics are now available in Cosmos SDK's telemetry server at host:port/geth/metrics (default localhost:1317/geth/metrics).
+
+
+### STATE BREAKING
+
+### API-BREAKING
+
+- [\#477](https://github.com/cosmos/evm/pull/477) Refactor precompile constructors to accept keeper interfaces instead of concrete implementations, breaking the existing `NewPrecompile` function signatures.
+- [\#594](https://github.com/cosmos/evm/pull/594) Remove all usage of x/params
+
+## v0.4.1
 
 ### DEPENDENCIES
 
@@ -45,6 +94,7 @@
 - [\#398](https://github.com/cosmos/evm/pull/398) Post-audit security fixes (batch 4)
 - [\#442](https://github.com/cosmos/evm/pull/442) Prevent nil pointer by checking error in gov precompile FromResponse.
 - [\#387](https://github.com/cosmos/evm/pull/387) (Experimental) EVM-compatible appside mempool
+- [\#476](https://github.com/cosmos/evm/pull/476) Add revert error e2e tests for contract and precompile calls
 
 ### FEATURES
 

CONTRIBUTING.md

@@ -21,7 +21,10 @@ We appreciate community contributions! To ensure a smooth process, please follow
    - All commits must be [signed](https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits).
    - Unsigned commits will be rejected.
 
-3. **Documentation Contributions**
+3. **Conventional Commit PR Titles**
+    - PR titles should use the [conventional commit](https://www.conventionalcommits.org/en/v1.0.0/#summary) format.
+
+4. **Documentation Contributions**
    - We only accept documentation PRs that make **substantial or impactful changes**.
    - Minor typo or style-only fixes in documentation will not be accepted.
 

Makefile

@@ -378,14 +378,90 @@ test-rpc-compat-stop:
 
 .PHONY: localnet-start localnet-stop localnet-build-env localnet-build-nodes test-rpc-compat test-rpc-compat-stop
 
-test-system: build
-	ulimit -n 1300
+test-system: build-v04 build
 	mkdir -p ./tests/systemtests/binaries/
 	cp $(BUILDDIR)/evmd ./tests/systemtests/binaries/
 	$(MAKE) -C tests/systemtests test
 
+build-v04:
+	mkdir -p ./tests/systemtests/binaries/v0.4
+	git checkout v0.4.1
+	make build
+	cp $(BUILDDIR)/evmd ./tests/systemtests/binaries/v0.4
+	git checkout -
+
 mocks:
 	@echo "--> generating mocks"
 	@go get github.com/vektra/mockery/v2
 	@go generate ./...
 	@make format-go
+
+###############################################################################
+###                              D2 Diagrams                                ###
+###############################################################################
+
+D2_THEME=300
+D2_DARK_THEME=200
+D2_LAYOUT=tala
+
+D2_ENV_VARS=D2_THEME=$(D2_THEME) \
+	    D2_DARK_THEME=$(D2_DARK_THEME) \
+	    D2_LAYOUT=$(D2_LAYOUT)
+
+.PHONY: d2check d2watch d2gen d2gen-all
+
+d2check:
+	@echo "🔍 checking if d2 is installed..."
+	@which d2 > /dev/null 2>&1 || { \
+		echo "🔴 d2 is not installed, see installation docs: https://d2lang.com/tour/install/"; \
+		exit 1; \
+	}
+	@echo "🟢 d2 is installed"
+	@echo "🔍 checking if $(D2_LAYOUT) layout is installed..."
+	@d2 layout | grep $(D2_LAYOUT) > /dev/null 2>&1 || { \
+		echo "🔴 $(D2_LAYOUT) layout is not installed, see docs: https://d2lang.com/tour/layouts/"; \
+		exit 1; \
+	}
+	@echo "🟢 $(D2_LAYOUT) layout is installed"
+
+d2watch: d2check
+	@if [ -z "$(FILE)" ]; then \
+		echo "🔴 missing required parameter FILE, the correct usage is: make d2watch FILE=path/to/file.d2"; \
+		exit 1; \
+	fi
+	@if [ ! -f "$(FILE)" ]; then \
+		echo "🔴 file $(FILE) does not exist"; \
+		exit 1; \
+	fi
+	@echo "🔄 watching $(FILE) for changes..."
+	@dir=$$(dirname "$(FILE)"); \
+	basename=$$(basename "$(FILE)" .d2); \
+	svgfile="$$dir/$$basename.svg"; \
+	printf "📊 generating $$svgfile from $(FILE)... "; \
+	$(D2_ENV_VARS) d2 --watch "$(FILE)" "$$svgfile"
+
+d2gen: d2check
+	@if [ -z "$(FILE)" ]; then \
+		echo "🔴 missing required parameter FILE, the correct usage is: make d2gen FILE=path/to/file.d2"; \
+		exit 1; \
+	fi
+	@if [ ! -f "$(FILE)" ]; then \
+		echo "🔴 file $(FILE) does not exist"; \
+		exit 1; \
+	fi
+	@dir=$$(dirname "$(FILE)"); \
+	basename=$$(basename "$(FILE)" .d2); \
+	svgfile="$$dir/$$basename.svg"; \
+	printf "📊 generating $$svgfile from $(FILE)... "; \
+	$(D2_ENV_VARS) d2 "$(FILE)" "$$svgfile" > /dev/null 2>&1 && echo "done ✅" || echo "failed ❌";
+
+d2gen-all: d2check
+	@echo "🟢 generating svg files for all d2 diagrams..."
+	@find . -name "*.d2" -type f | while read d2file; do \
+		dir=$$(dirname "$$d2file"); \
+		basename=$$(basename "$$d2file" .d2); \
+		svgfile="$$dir/$$basename.svg"; \
+		printf "📊 generating $$svgfile from $$d2file... "; \
+		$(D2_ENV_VARS) d2 "$$d2file" "$$svgfile" > /dev/null 2>&1 && echo "done ✅" || echo "failed ❌"; \
+	done
+	@echo "✅ svg files generated for all d2 diagrams"

README.md

@@ -197,3 +197,8 @@ We welcome open source contributions and discussions! For more on contributing,
 Cosmos EVM is open-source under the Apache 2.0 license, an extension of the license of the original codebase (https://github.com/evmos/OS)
 created by Tharsis and the evmOS team - who conducted the foundational work for EVM compatibility and
 interoperability in Cosmos.
+
+### Key Contributors
+
+We at ICL want to thank our key contributors at [B-Harvest](https://bharvest.io/) and 
+[Mantra](https://www.mantrachain.io/) for contributing to and helping us drive the development of Cosmos EVM.

ante/evm/05_signature_verification.go

@@ -32,11 +32,9 @@ func NewEthSigVerificationDecorator(ek anteinterfaces.EVMKeeper) EthSigVerificat
 // Failure in RecheckTx will prevent tx to be included into block, especially when CheckTx succeed, in which case user
 // won't see the error message.
 func (esvd EthSigVerificationDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, simulate bool, next sdk.AnteHandler) (newCtx sdk.Context, err error) {
-	evmParams := esvd.evmKeeper.GetParams(ctx)
 	ethCfg := evmtypes.GetEthChainConfig()
 	blockNum := big.NewInt(ctx.BlockHeight())
 	signer := ethtypes.MakeSigner(ethCfg, blockNum, uint64(ctx.BlockTime().Unix())) //#nosec G115 -- int overflow is not a concern here
-	allowUnprotectedTxs := evmParams.GetAllowUnprotectedTxs()
 
 	msgs := tx.GetMsgs()
 	if msgs == nil {
@@ -49,7 +47,7 @@ func (esvd EthSigVerificationDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, s
 			return ctx, errorsmod.Wrapf(errortypes.ErrUnknownRequest, "invalid message type %T, expected %T", msg, (*evmtypes.MsgEthereumTx)(nil))
 		}
 
-		err := SignatureVerification(msgEthTx, msgEthTx.AsTransaction(), signer, allowUnprotectedTxs)
+		err := SignatureVerification(msgEthTx, msgEthTx.AsTransaction(), signer)
 		if err != nil {
 			return ctx, err
 		}
@@ -66,23 +64,7 @@ func SignatureVerification(
 	msg *evmtypes.MsgEthereumTx,
 	ethTx *ethtypes.Transaction,
 	signer ethtypes.Signer,
-	allowUnprotectedTxs bool,
 ) error {
-	ethCfg := evmtypes.GetEthChainConfig()
-
-	if !allowUnprotectedTxs {
-		if !ethTx.Protected() {
-			return errorsmod.Wrapf(
-				errortypes.ErrNotSupported,
-				"rejected unprotected ethereum transaction; please sign your transaction according to EIP-155 to protect it against replay-attacks")
-		}
-		if ethTx.ChainId().Uint64() != ethCfg.ChainID.Uint64() {
-			return errorsmod.Wrapf(
-				errortypes.ErrInvalidChainID,
-				"rejected ethereum transaction with incorrect chain-id; expected %d, got %d", ethCfg.ChainID, ethTx.ChainId())
-		}
-	}
-
 	if err := msg.VerifySender(signer); err != nil {
 		return errorsmod.Wrapf(errortypes.ErrorInvalidSigner, "signature verification failed: %s", err.Error())
 	}