Merge branch 'main' into pd/do-not-return-error-string-on-precompile-error

Author: pdrobnjak

.github/workflows/go-test.yml

@@ -28,6 +28,8 @@ jobs:
             tags: ledger test_ledger_mock
           - name: sei-db
             path: ./sei-db
+          - name: sei-ibc-go
+            path: ./sei-ibc-go
           - name: sei-tendermint
             path: ./sei-tendermint
           - name: sei-wasmd

.github/workflows/integration-test.yml

@@ -143,7 +143,7 @@ jobs:
           go-version: 1.24
 
       - name: Start 4 node docker cluster
-        run: make clean && INVARIANT_CHECK_INTERVAL=10 ${{matrix.test.env}} make docker-cluster-start &
+        run: make clean && DOCKER_DETACH=true INVARIANT_CHECK_INTERVAL=10 ${{matrix.test.env}} make docker-cluster-start
 
       - name: Wait for docker cluster to start
         run: |
@@ -170,6 +170,36 @@ jobs:
           done
           unset IFS  # revert the internal field separator back to default
 
+      - name: Prepare log artifact name
+        if: ${{ always() }}
+        id: log_artifact_meta
+        run: |
+          raw="${{ matrix.test.name }}"
+          safe=$(echo "$raw" | tr '[:upper:]' '[:lower:]' | sed -E 's/[^a-z0-9]+/-/g' | sed -E 's/^-+|-+$//g')
+          if [ -z "$safe" ]; then
+            safe="logs"
+          fi
+          echo "artifact_name=$safe" >> $GITHUB_OUTPUT
+
+      - name: Collect logs directory
+        if: ${{ always() }}
+        run: |
+          LOG_ROOT="artifacts/sei-${{ steps.log_artifact_meta.outputs.artifact_name }}"
+          mkdir -p "$LOG_ROOT"
+          if [ -d build/generated/logs ]; then
+            cp -r build/generated/logs "$LOG_ROOT/"
+          else
+            echo "No logs directory found"
+          fi
+
+      - name: Upload logs directory
+        if: ${{ always() }}
+        uses: actions/upload-artifact@v4
+        with:
+          name: integration-logs-${{ steps.log_artifact_meta.outputs.artifact_name }}
+          path: artifacts/sei-${{ steps.log_artifact_meta.outputs.artifact_name }}
+          if-no-files-found: warn
+
   integration-test-check:
     name: Integration Test Check
     runs-on: ubuntu-latest

Dockerfile

@@ -12,6 +12,7 @@ COPY sei-wasmvm/go.mod sei-wasmvm/go.sum ./sei-wasmvm/
 COPY sei-wasmd/go.mod sei-wasmd/go.sum ./sei-wasmd/
 COPY sei-cosmos/go.mod sei-cosmos/go.sum ./sei-cosmos/
 COPY sei-tendermint/go.mod sei-tendermint/go.sum ./sei-tendermint/
+COPY sei-ibc-go/go.mod sei-ibc-go/go.sum ./sei-ibc-go/
 COPY sei-db/go.mod sei-db/go.sum ./sei-db/
 RUN go mod download
 

Makefile

@@ -67,6 +67,7 @@ ldflags := $(strip $(ldflags))
 # BUILD_FLAGS := -tags "$(build_tags)" -ldflags '$(ldflags)' -race
 BUILD_FLAGS := -tags "$(build_tags)" -ldflags '$(ldflags)'
 BUILD_FLAGS_MOCK_BALANCES := -tags "$(build_tags) mock_balances" -ldflags '$(ldflags)'
+BUILD_FLAGS_BENCHMARK := -tags "$(build_tags) benchmark mock_balances" -ldflags '$(ldflags)'
 
 #### Command List ####
 
@@ -78,6 +79,9 @@ install: go.sum
 install-mock-balances: go.sum
 		go install $(BUILD_FLAGS_MOCK_BALANCES) ./cmd/seid
 
+install-bench: go.sum
+		go install $(BUILD_FLAGS_BENCHMARK) ./cmd/seid
+
 install-with-race-detector: go.sum
 		go install -race $(BUILD_FLAGS) ./cmd/seid
 
@@ -262,14 +266,26 @@ docker-cluster-start: docker-cluster-stop build-docker-node
 	@rm -rf $(PROJECT_HOME)/build/generated
 	@mkdir -p $(shell go env GOPATH)/pkg/mod
 	@mkdir -p $(shell go env GOCACHE)
-	@cd docker && USERID=$(shell id -u) GROUPID=$(shell id -g) GOCACHE=$(shell go env GOCACHE) NUM_ACCOUNTS=10 INVARIANT_CHECK_INTERVAL=${INVARIANT_CHECK_INTERVAL} UPGRADE_VERSION_LIST=${UPGRADE_VERSION_LIST} MOCK_BALANCES=${MOCK_BALANCES} docker compose up
+	@cd docker && \
+		if [ "$${DOCKER_DETACH:-}" = "true" ]; then \
+			DETACH_FLAG="-d"; \
+		else \
+			DETACH_FLAG=""; \
+		fi; \
+		USERID=$(shell id -u) GROUPID=$(shell id -g) GOCACHE=$(shell go env GOCACHE) NUM_ACCOUNTS=10 INVARIANT_CHECK_INTERVAL=${INVARIANT_CHECK_INTERVAL} UPGRADE_VERSION_LIST=${UPGRADE_VERSION_LIST} MOCK_BALANCES=${MOCK_BALANCES} docker compose up $$DETACH_FLAG
 
 .PHONY: localnet-start
 
 # Use this to skip the seid build process
 docker-cluster-start-skipbuild: docker-cluster-stop build-docker-node
 	@rm -rf $(PROJECT_HOME)/build/generated
-	@cd docker && USERID=$(shell id -u) GROUPID=$(shell id -g) GOCACHE=$(shell go env GOCACHE) NUM_ACCOUNTS=10 SKIP_BUILD=true docker compose up
+	@cd docker && \
+		if [ "$${DOCKER_DETACH:-}" = "true" ]; then \
+			DETACH_FLAG="-d"; \
+		else \
+			DETACH_FLAG=""; \
+		fi; \
+		USERID=$(shell id -u) GROUPID=$(shell id -g) GOCACHE=$(shell go env GOCACHE) NUM_ACCOUNTS=10 SKIP_BUILD=true docker compose up $$DETACH_FLAG
 .PHONY: localnet-start
 
 # Stop 4-node docker containers
@@ -308,4 +324,4 @@ test-group-%:split-test-packages
 		echo "⚡ Not found, running with -parallel=4"; \
 		PARALLEL="-parallel=4"; \
 	fi; \
-	cat $(BUILDDIR)/packages.txt.$* | xargs go test $$PARALLEL -mod=readonly -timeout=10m -race -coverprofile=$*.profile.out -covermode=atomic -coverpkg=./...
+	cat $(BUILDDIR)/packages.txt.$* | xargs go test $$PARALLEL -mod=readonly -timeout=10m -race -coverprofile=$*.profile.out -covermode=atomic -coverpkg=./...

app/abci.go

@@ -2,10 +2,13 @@ package app
 
 import (
 	"context"
+	"crypto/sha256"
 	"time"
 
 	"github.com/cosmos/cosmos-sdk/telemetry"
 	sdk "github.com/cosmos/cosmos-sdk/types"
+	sdkerrors "github.com/cosmos/cosmos-sdk/types/errors"
+	"github.com/cosmos/cosmos-sdk/types/legacytm"
 	"github.com/sei-protocol/sei-chain/app/legacyabci"
 	"github.com/sei-protocol/sei-chain/utils/metrics"
 	abci "github.com/tendermint/tendermint/abci/types"
@@ -50,17 +53,60 @@ func (app *App) MidBlock(ctx sdk.Context, height int64) []abci.Event {
 	return app.BaseApp.MidBlock(ctx, height)
 }
 
-func (app *App) EndBlock(ctx sdk.Context, req abci.RequestEndBlock) (res abci.ResponseEndBlock) {
+func (app *App) EndBlock(ctx sdk.Context, height int64, blockGasUsed int64) (res abci.ResponseEndBlock) {
 	spanCtx, span := app.GetBaseApp().TracingInfo.StartWithContext("EndBlock", ctx.TraceSpanContext())
 	defer span.End()
 	ctx = ctx.WithTraceSpanContext(spanCtx)
-	return app.BaseApp.EndBlock(ctx, req)
+	defer telemetry.MeasureSince(time.Now(), "abci", "end_block")
+	ctx = ctx.WithEventManager(sdk.NewEventManager())
+	defer telemetry.MeasureSince(time.Now(), "module", "total_end_block")
+	res.ValidatorUpdates = legacyabci.EndBlock(ctx, height, blockGasUsed, app.EndBlockKeepers)
+	res.Events = sdk.MarkEventsToIndex(ctx.EventManager().ABCIEvents(), app.IndexEvents)
+	if cp := app.GetConsensusParams(ctx); cp != nil {
+		res.ConsensusParamUpdates = legacytm.ABCIToLegacyConsensusParams(cp)
+	}
+	return res
 }
 
 func (app *App) CheckTx(ctx context.Context, req *abci.RequestCheckTxV2) (*abci.ResponseCheckTxV2, error) {
 	_, span := app.GetBaseApp().TracingInfo.StartWithContext("CheckTx", ctx)
 	defer span.End()
-	return app.BaseApp.CheckTx(ctx, req)
+	defer telemetry.MeasureSince(time.Now(), "abci", "check_tx")
+	sdkCtx := app.GetCheckTxContext(req.Tx, req.Type == abci.CheckTxTypeV2Recheck)
+	tx, err := app.txDecoder(req.Tx)
+	if err != nil {
+		res := sdkerrors.ResponseCheckTx(err, 0, 0, false)
+		return &abci.ResponseCheckTxV2{ResponseCheckTx: &res}, err
+	}
+	checksum := sha256.Sum256(req.Tx)
+	gInfo, result, txCtx, err := legacyabci.CheckTx(sdkCtx, tx, app.GetTxConfig(), &app.CheckTxKeepers, checksum, func(ctx sdk.Context) (sdk.Context, sdk.CacheMultiStore) {
+		return app.CacheTxContext(ctx, checksum)
+	}, app.GetCheckCtx, app.TracingInfo)
+	if err != nil {
+		res := sdkerrors.ResponseCheckTx(err, gInfo.GasWanted, gInfo.GasUsed, false)
+		return &abci.ResponseCheckTxV2{ResponseCheckTx: &res}, err
+	}
+
+	res := &abci.ResponseCheckTxV2{
+		ResponseCheckTx: &abci.ResponseCheckTx{
+			GasWanted:    int64(gInfo.GasWanted), //nolint:gosec
+			Data:         result.Data,
+			Priority:     txCtx.Priority(),
+			GasEstimated: int64(gInfo.GasEstimate), //nolint:gosec
+		},
+		ExpireTxHandler:  txCtx.ExpireTxHandler(),
+		CheckTxCallback:  txCtx.CheckTxCallback(),
+		EVMNonce:         txCtx.EVMNonce(),
+		EVMSenderAddress: txCtx.EVMSenderAddress(),
+		IsEVM:            txCtx.IsEVM(),
+		Priority:         txCtx.Priority(),
+	}
+	if txCtx.PendingTxChecker() != nil {
+		res.IsPendingTransaction = true
+		res.Checker = txCtx.PendingTxChecker()
+	}
+
+	return res, nil
 }
 
 func (app *App) DeliverTx(ctx sdk.Context, req abci.RequestDeliverTxV2, tx sdk.Tx, checksum [32]byte) abci.ResponseDeliverTx {