x/feeabstraction/keeper/oracle.go (1)

39-42: Clarify disable reason in logs for non-positive base price.

Line 39 now correctly handles zero/negative prices, but Line 41 still logs only a “no price” message. That can mislead debugging when the denom exists but price is 0/negative.

♻️ Suggested logging refinement

-	if !ok || !baseTokenPrice.IsPositive() {
-		// Disable fee abstraction if there is no pricing
-		k.Logger(ctx).Debug("%s has no price, feeabstraction disabled", params.NativeOracleDenom)
+	if !ok || !baseTokenPrice.IsPositive() {
+		// Disable fee abstraction when pricing is missing or invalid
+		if !ok {
+			k.Logger(ctx).Debug("fee abstraction disabled: base token price missing", "denom", params.NativeOracleDenom)
+		} else {
+			k.Logger(ctx).Debug("fee abstraction disabled: base token price non-positive", "denom", params.NativeOracleDenom, "price", baseTokenPrice.String())
+		}
 		params.Enabled = false
 		return k.Params.Set(ctx, params)
 	}

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@x/feeabstraction/keeper/oracle.go` around lines 39 - 42, The debug log
incorrectly states "no price" even when baseTokenPrice exists but is
zero/negative; update the logging in the block that checks if !ok ||
!baseTokenPrice.IsPositive() (around baseTokenPrice, params.NativeOracleDenom,
params.Enabled) to distinguish the two cases: log "no price" when ok is false,
and log a clear message including the actual baseTokenPrice value (and that it's
non-positive) when ok is true but baseTokenPrice.IsPositive() is false, then set
params.Enabled = false as before.