Merge branch 'main' into alex/adr023_ha-failover_raft2

* main:
  feat(store)!: add batching for atomicity  (#2746)
  refactor(apps): rollback cmd updates (#2744)
  chore: add makefile for tools (#2743)
  chore: fix markdown lint (#2742)
  build(deps): Bump the all-go group across 5 directories with 6 updates (#2738)
  refactor(block): improve cancellation (#2741)
  chore: make the prompt go oriented  (#2739)
  perf(block): use `sync/atomic` instead of mutexes (#2735)

Author: alpe

.github/workflows/claude-code-review.yml

@@ -36,19 +36,59 @@ jobs:
         uses: anthropics/claude-code-action@v1
         with:
           claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
+          track_progress: true
+          use_sticky_comment: true
           prompt: |
-            Please review this pull request and provide feedback on:
-            - Code quality and best practices
-            - Potential bugs or issues
-            - Performance considerations
-            - Security concerns
-            - Test coverage
+            Perform a comprehensive code review with the following focus areas:
+
+            1. **Code Quality & Architecture**
+                - Verify adherence to idiomatic Go style (effective-go, golangci-lint compliance, naming conventions).
+                - Assess package structure for clarity and modularity (avoid cyclic dependencies, promote clean boundaries).
+                - Ensure interfaces are used appropriately — not over-abstracted but testable.
+                - Review concurrency patterns (goroutines, channels, sync primitives) for correctness and deadlock safety.
+                - Check configuration management (flags, environment variables, config files) for clarity and consistency.
+                - Validate that logging, error propagation, and context handling (context.Context) are consistent and properly implemented.
+
+            2. **Error Handling & Resilience**
+              - Verify consistent error wrapping and context (use of %w, errors.Join, or fmt.Errorf).
+              - Ensure external dependencies (network, file I/O, DB connections) have proper retry, timeout, and cancellation logic.
+              - Confirm panics are avoided or safely recovered from in long-running goroutines.
+              - Assess logging for meaningful messages while avoiding sensitive data exposure.
+
+            3. **Security**
+              - Review authentication/authorization logic and credential management.
+              - Check input validation and sanitization for APIs, CLI, or device message payloads.
+              - Ensure secrets are not hard-coded or logged.
+              - Confirm secure handling of TLS, tokens, and crypto functions where applicable.
+              - Validate dependencies for known vulnerabilities and verify minimal privilege access patterns.
+
+            4. **Performance & Resource Efficiency**
+              - Inspect hot paths for unnecessary allocations or blocking operations.
+              - Check for proper use of connection pools, goroutine lifecycle management, and cleanup.
+              - Review memory and CPU usage in concurrent loops, background workers, and I/O pipelines.
+              - Evaluate database or cache query efficiency and connection reuse.
 
-            Use the repository's CLAUDE.md for guidance on style and conventions. Be constructive and helpful in your feedback.
 
-            Use `gh pr comment` with your Bash tool to leave your review as a comment on the PR.
+            5. **Testing & Reliability**
+              - Verify presence and adequacy of unit, integration, and end-to-end tests.
+              - Check test naming, isolation, and reproducibility.
+              - Ensure key edge cases (timeouts, partial failures, concurrent access) are covered.
+              - Review use of mocks/fakes for external dependencies.
+              - Confirm go test and CI workflows run cleanly and consistently.
+
+
+            6. **Documentation & Developer Experience**
+              - Confirm code comments and docstrings clearly describe public APIs, exported structs, and interfaces.
+              - Review README.md, setup scripts, and examples for developer onboarding.
+              - Ensure configuration and runtime parameters are documented.
+              - Check generated API documentation (if applicable) for completeness and alignment with implementation.
+
+
+              Feedback Guidelines
+             	•	Use inline comments for file- or line-specific issues and improvements.
+             	•	Use top-level comments for architectural observations, general patterns, or praise.
+             	•	Prioritize clarity, robustness, and production readiness in all feedback.
 
           # See https://github.com/anthropics/claude-code-action/blob/main/docs/usage.md
           # or https://docs.anthropic.com/en/docs/claude-code/sdk#command-line for available options
           claude_args: '--allowed-tools "Bash(gh issue view:*),Bash(gh search:*),Bash(gh issue list:*),Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Bash(gh pr list:*)"'
-          use_sticky_comment: true

Makefile

@@ -3,6 +3,7 @@ include ./scripts/test.mk
 include ./scripts/proto.mk
 include ./scripts/utils.mk
 include ./scripts/run.mk
+include ./tools/tools.mk
 
 # Sets the default make target to `build`.
 # Requires GNU Make >= v3.81.

apps/evm/single/cmd/rollback.go

@@ -2,6 +2,7 @@ package cmd
 
 import (
 	"context"
+	"errors"
 	"fmt"
 
 	ds "github.com/ipfs/go-datastore"
@@ -18,9 +19,8 @@ import (
 // NewRollbackCmd creates a command to rollback ev-node state by one height.
 func NewRollbackCmd() *cobra.Command {
 	var (
-		height        uint64
-		skipP2PStores bool
-		syncNode      bool
+		height   uint64
+		syncNode bool
 	)
 
 	cmd := &cobra.Command{
@@ -69,10 +69,6 @@ func NewRollbackCmd() *cobra.Command {
 				return fmt.Errorf("failed to rollback ev-node state: %w", err)
 			}
 
-			if skipP2PStores {
-				return printSuccess(height)
-			}
-
 			// rollback ev-node goheader state
 			headerStore, err := goheaderstore.NewStore[*types.SignedHeader](
 				evolveDB,
@@ -102,27 +98,26 @@ func NewRollbackCmd() *cobra.Command {
 			}
 			defer dataStore.Stop(goCtx)
 
+			var errs error
 			if err := headerStore.DeleteRange(goCtx, height+1, headerStore.Height()); err != nil {
-				return fmt.Errorf("failed to rollback header sync service state: %w", err)
+				errs = errors.Join(errs, fmt.Errorf("failed to rollback header sync service state: %w", err))
 			}
 
 			if err := dataStore.DeleteRange(goCtx, height+1, dataStore.Height()); err != nil {
-				return fmt.Errorf("failed to rollback data sync service state: %w", err)
+				errs = errors.Join(errs, fmt.Errorf("failed to rollback data sync service state: %w", err))
+			}
+
+			fmt.Printf("Rolled back ev-node state to height %d\n", height)
+			if syncNode {
+				fmt.Println("Restart the node with the `--clear-cache` flag")
 			}
 
-			return printSuccess(height)
+			return errs
 		},
 	}
 
 	cmd.Flags().Uint64Var(&height, "height", 0, "rollback to a specific height")
 	cmd.Flags().BoolVar(&syncNode, "sync-node", false, "sync node (no aggregator)")
-	cmd.Flags().BoolVar(&skipP2PStores, "skip-p2p-stores", false, "skip rollback p2p stores (goheaderstore)")
 
 	return cmd
 }
-
-func printSuccess(height uint64) error {
-	fmt.Printf("Rolled back ev-node state to height %d\n", height)
-	fmt.Println("Restart the node with the `--clear-cache` flag")
-	return nil
-}

apps/evm/single/go.mod

@@ -2,7 +2,7 @@ module github.com/evstack/ev-node/apps/evm/single
 
 go 1.24.6
 
-replace github.com/celestiaorg/go-header => github.com/julienrbrt/go-header v0.0.0-20250909151551-cb11b091bf58 // TODO: to remove after https://github.com/celestiaorg/go-header/pull/347
+replace github.com/celestiaorg/go-header => github.com/julienrbrt/go-header v0.0.0-20251008134330-747c8c192fa8 // TODO: to remove after https://github.com/celestiaorg/go-header/pull/347
 
 replace (
 	github.com/evstack/ev-node => ../../../
@@ -14,7 +14,7 @@ replace (
 require (
 	github.com/celestiaorg/go-header v0.7.3
 	github.com/ethereum/go-ethereum v1.16.4
-	github.com/evstack/ev-node v1.0.0-beta.5
+	github.com/evstack/ev-node v1.0.0-beta.6
 	github.com/evstack/ev-node/core v1.0.0-beta.3
 	github.com/evstack/ev-node/da v0.0.0-20250317130407-e9e0a1b0485e
 	github.com/evstack/ev-node/execution/evm v0.0.0-00010101000000-000000000000
@@ -34,7 +34,7 @@ require (
 	github.com/bits-and-blooms/bitset v1.20.0 // indirect
 	github.com/boltdb/bolt v1.3.1 // indirect
 	github.com/celestiaorg/go-libp2p-messenger v0.2.2 // indirect
-	github.com/celestiaorg/go-square/v3 v3.0.1 // indirect
+	github.com/celestiaorg/go-square/v3 v3.0.2 // indirect
 	github.com/cespare/xxhash/v2 v2.3.0 // indirect
 	github.com/consensys/gnark-crypto v0.18.0 // indirect
 	github.com/crate-crypto/go-eth-kzg v1.4.0 // indirect
@@ -82,10 +82,10 @@ require (
 	github.com/holiman/uint256 v1.3.2 // indirect
 	github.com/huin/goupnp v1.3.0 // indirect
 	github.com/inconshreveable/mousetrap v1.1.0 // indirect
-	github.com/ipfs/boxo v0.33.1 // indirect
+	github.com/ipfs/boxo v0.35.0 // indirect
 	github.com/ipfs/go-cid v0.5.0 // indirect
 	github.com/ipfs/go-ds-badger4 v0.1.8 // indirect
-	github.com/ipfs/go-log/v2 v2.8.0 // indirect
+	github.com/ipfs/go-log/v2 v2.8.1 // indirect
 	github.com/ipld/go-ipld-prime v0.21.0 // indirect
 	github.com/jackpal/go-nat-pmp v1.0.2 // indirect
 	github.com/jbenet/go-temp-err-catcher v0.1.0 // indirect
@@ -97,8 +97,8 @@ require (
 	github.com/libp2p/go-flow-metrics v0.3.0 // indirect
 	github.com/libp2p/go-libp2p v0.43.0 // indirect
 	github.com/libp2p/go-libp2p-asn-util v0.4.1 // indirect
-	github.com/libp2p/go-libp2p-kad-dht v0.34.0 // indirect
-	github.com/libp2p/go-libp2p-kbucket v0.7.0 // indirect
+	github.com/libp2p/go-libp2p-kad-dht v0.35.1 // indirect
+	github.com/libp2p/go-libp2p-kbucket v0.8.0 // indirect
 	github.com/libp2p/go-libp2p-pubsub v0.15.0 // indirect
 	github.com/libp2p/go-libp2p-record v0.3.1 // indirect
 	github.com/libp2p/go-libp2p-routing-helpers v0.7.5 // indirect
@@ -125,7 +125,7 @@ require (
 	github.com/multiformats/go-multicodec v0.9.2 // indirect
 	github.com/multiformats/go-multihash v0.2.3 // indirect
 	github.com/multiformats/go-multistream v0.6.1 // indirect
-	github.com/multiformats/go-varint v0.0.7 // indirect
+	github.com/multiformats/go-varint v0.1.0 // indirect
 	github.com/munnerz/goautoneg v0.0.0-20191010083416-a7dc8b61c822 // indirect
 	github.com/olekukonko/tablewriter v0.0.5 // indirect
 	github.com/pbnjay/memory v0.0.0-20210728143218-7b4eea64cf58 // indirect
@@ -178,7 +178,7 @@ require (
 	github.com/whyrusleeping/go-keyspace v0.0.0-20160322163242-5b898ac5add1 // indirect
 	github.com/wlynxg/anet v0.0.5 // indirect
 	go.opencensus.io v0.24.0 // indirect
-	go.opentelemetry.io/auto/sdk v1.1.0 // indirect
+	go.opentelemetry.io/auto/sdk v1.2.1 // indirect
 	go.opentelemetry.io/otel v1.38.0 // indirect
 	go.opentelemetry.io/otel/metric v1.38.0 // indirect
 	go.opentelemetry.io/otel/trace v1.38.0 // indirect
@@ -187,20 +187,21 @@ require (
 	go.uber.org/mock v0.5.2 // indirect
 	go.uber.org/multierr v1.11.0 // indirect
 	go.uber.org/zap v1.27.0 // indirect
-	go.yaml.in/yaml/v2 v2.4.2 // indirect
+	go.yaml.in/yaml/v2 v2.4.3 // indirect
 	go.yaml.in/yaml/v3 v3.0.4 // indirect
 	golang.org/x/crypto v0.42.0 // indirect
-	golang.org/x/exp v0.0.0-20250811191247-51f88131bc50 // indirect
-	golang.org/x/mod v0.27.0 // indirect
+	golang.org/x/exp v0.0.0-20250911091902-df9299821621 // indirect
+	golang.org/x/mod v0.28.0 // indirect
 	golang.org/x/net v0.44.0 // indirect
 	golang.org/x/sync v0.17.0 // indirect
 	golang.org/x/sys v0.36.0 // indirect
+	golang.org/x/telemetry v0.0.0-20250908211612-aef8a434d053 // indirect
 	golang.org/x/text v0.29.0 // indirect
 	golang.org/x/time v0.12.0 // indirect
-	golang.org/x/tools v0.36.0 // indirect
+	golang.org/x/tools v0.37.0 // indirect
 	golang.org/x/xerrors v0.0.0-20240903120638-7835f813f4da // indirect
 	gonum.org/v1/gonum v0.16.0 // indirect
-	google.golang.org/protobuf v1.36.9 // indirect
+	google.golang.org/protobuf v1.36.10 // indirect
 	gopkg.in/yaml.v2 v2.4.0 // indirect
 	gopkg.in/yaml.v3 v3.0.1 // indirect
 	lukechampine.com/blake3 v1.4.1 // indirect