From 71799199a9e395419f51c79c5606054f7fd4f104 Mon Sep 17 00:00:00 2001
From: Larko <59736843+Larkooo@users.noreply.github.com>
Date: Thu, 5 Dec 2024 22:16:22 +0700
Subject: [PATCH] refactor(torii-args): bool values no need for explicit true
 (#2767)

* refactor(torii-args): bool values no need for explicit true

* fmt

* fix test
---
 crates/torii/cli/src/args.rs    |  3 ---
 crates/torii/cli/src/options.rs | 14 ++++++++++----
 2 files changed, 10 insertions(+), 7 deletions(-)

diff --git a/crates/torii/cli/src/args.rs b/crates/torii/cli/src/args.rs
index 4049ff3049..36a72ad20a 100644
--- a/crates/torii/cli/src/args.rs
+++ b/crates/torii/cli/src/args.rs
@@ -218,12 +218,9 @@ mod test {
             "http://0.0.0.0:6060",
             "--db-dir",
             "/tmp/torii-test2",
-            "--events.raw",
-            "false",
             "--events.historical",
             "a-A",
             "--indexing.transactions",
-            "true",
             "--config",
             path_str.as_str(),
         ];
diff --git a/crates/torii/cli/src/options.rs b/crates/torii/cli/src/options.rs
index 48fc62915d..eaffd58b2d 100644
--- a/crates/torii/cli/src/options.rs
+++ b/crates/torii/cli/src/options.rs
@@ -2,7 +2,6 @@ use std::net::{IpAddr, Ipv4Addr};
 use std::str::FromStr;
 
 use anyhow::Context;
-use clap::ArgAction;
 use serde::ser::SerializeSeq;
 use serde::{Deserialize, Serialize};
 use starknet::core::types::Felt;
@@ -102,7 +101,11 @@ pub struct IndexingOptions {
     pub blocks_chunk_size: u64,
 
     /// Enable indexing pending blocks
-    #[arg(long = "indexing.pending", action = ArgAction::Set, default_value_t = true, help = "Whether or not to index pending blocks.")]
+    #[arg(
+        long = "indexing.pending",
+        default_value_t = true,
+        help = "Whether or not to index pending blocks."
+    )]
     #[serde(default)]
     pub pending: bool,
 
@@ -127,7 +130,6 @@ pub struct IndexingOptions {
     /// Whether or not to index world transactions
     #[arg(
         long = "indexing.transactions",
-        action = ArgAction::Set,
         default_value_t = false,
         help = "Whether or not to index world transactions and keep them in the database."
     )]
@@ -214,7 +216,11 @@ impl IndexingOptions {
 #[command(next_help_heading = "Events indexing options")]
 pub struct EventsOptions {
     /// Whether or not to index raw events
-    #[arg(long = "events.raw", action = ArgAction::Set, default_value_t = false, help = "Whether or not to index raw events.")]
+    #[arg(
+        long = "events.raw",
+        default_value_t = false,
+        help = "Whether or not to index raw events."
+    )]
     #[serde(default)]
     pub raw: bool,