feat(cosmosfaucet): loosen faucet check when indexer disabled

changelog.md

@@ -18,6 +18,7 @@
 - [#4613](https://github.com/ignite/cli/pull/4613) Improve and simplify prompting logic by bubbletea.
 - [#4615](https://github.com/ignite/cli/pull/4615) Fetch Ignite announcements from API.
 - [#4624](https://github.com/ignite/cli/pull/4624) Fix autocli templates for variadics.
+- [#4633](https://github.com/ignite/cli/pull/4633) Loosen faucet check when indexer disabled.
 
 ### Fixes
 

ignite/pkg/cosmosfaucet/cosmosfaucet.go

@@ -74,6 +74,9 @@ type Faucet struct {
 
 	// version holds the cosmos-sdk version.
 	version cosmosver.Version
+
+	// indexerDisabled tells whether the indexing is disabled on the node.
+	indexerDisabled bool
 }
 
 // Option configures the faucetOptions.
@@ -139,6 +142,14 @@ func Version(version cosmosver.Version) Option {
 	}
 }
 
+// IndexerDisabled tells whether the indexing is disabled on the node.
+// Without indexing, the faucet won't be able to check the limits for each account, nor verify the transaction status.
+func IndexerDisabled() Option {
+	return func(f *Faucet) {
+		f.indexerDisabled = true
+	}
+}
+
 // New creates a new faucet with ccr (to access and use blockchain's CLI) and given options.
 func New(ctx context.Context, ccr chaincmdrunner.Runner, options ...Option) (Faucet, error) {
 	f := Faucet{

ignite/pkg/cosmosfaucet/transfer.go

@@ -73,6 +73,11 @@ func (f *Faucet) Transfer(ctx context.Context, toAccountAddress string, coins sd
 	transfer := sdk.NewCoins()
 	// check for each coin, the max transferred amount hasn't been reached
 	for _, c := range coins {
+		if f.indexerDisabled { // we cannot check the transfer history if indexer is disabled
+			transfer = transfer.Add(c)
+			continue
+		}
+
 		totalSent, err := f.TotalTransferredAmount(ctx, toAccountAddress, c.Denom)
 		if err != nil {
 			return "", err
@@ -109,6 +114,10 @@ func (f *Faucet) Transfer(ctx context.Context, toAccountAddress string, coins sd
 		return "", err
 	}
 
+	if f.indexerDisabled {
+		return txHash, nil // we cannot check the tx status if indexer is disabled
+	}
+
 	// wait for send tx to be confirmed
 	return txHash, f.runner.WaitTx(ctx, txHash, time.Second, 30)
 }

ignite/services/chain/faucet.go

@@ -2,7 +2,9 @@ package chain
 
 import (
 	"context"
+	"fmt"
 	"os"
+	"strings"
 	"time"
 
 	sdkmath "cosmossdk.io/math"
@@ -14,6 +16,7 @@ import (
 	"github.com/ignite/cli/v29/ignite/pkg/cosmosfaucet"
 	"github.com/ignite/cli/v29/ignite/pkg/errors"
 	"github.com/ignite/cli/v29/ignite/pkg/xurl"
+	"github.com/ignite/cli/v29/ignite/pkg/xyaml"
 )
 
 var (
@@ -84,6 +87,12 @@ func (c *Chain) Faucet(ctx context.Context) (cosmosfaucet.Faucet, error) {
 		cosmosfaucet.Version(c.Version),
 	}
 
+	// check if indexer is enabled or not.
+	if indexerDisabled(validator.Config) {
+		faucetOptions = append(faucetOptions, cosmosfaucet.IndexerDisabled())
+		c.ev.Send("⚠️ CometBFT indexer disabled. Faucet can't check account limits or verify transaction status.")
+	}
+
 	// parse coins to pass to the faucet as coins.
 	for _, coin := range conf.Faucet.Coins {
 		parsedCoin, err := sdk.ParseCoinNormalized(coin)
@@ -130,3 +139,15 @@ func (c *Chain) Faucet(ctx context.Context) (cosmosfaucet.Faucet, error) {
 	// init the faucet with options and return.
 	return cosmosfaucet.New(ctx, commands, faucetOptions...)
 }
+
+// indexerDisabled checks if the indexer is disabled in the config.yml.
+// More specifically, it checks if a kv indexer is used (psql indexer is not supported).
+func indexerDisabled(valCfg xyaml.Map) bool {
+	const txIndexKey = "tx_index"
+	v, ok := valCfg[txIndexKey]
+	if !ok {
+		return false
+	}
+
+	return !strings.Contains(fmt.Sprintf("%v", v), "kv")
+}