From 95ec42bba70a3d0f1771fd63295d4defbe1d498b Mon Sep 17 00:00:00 2001
From: Daniil Polyakov <arjentix@gmail.com>
Date: Wed, 19 Nov 2025 17:33:25 +0300
Subject: [PATCH] chore: left some comments

---
 sequencer_runner/src/main.rs | 1 +
 wallet/src/main.rs           | 5 +++++
 2 files changed, 6 insertions(+)

diff --git a/sequencer_runner/src/main.rs b/sequencer_runner/src/main.rs
index 9de8835..97f332d 100644
--- a/sequencer_runner/src/main.rs
+++ b/sequencer_runner/src/main.rs
@@ -4,6 +4,7 @@ use sequencer_runner::main_runner;
 
 pub const NUM_THREADS: usize = 4;
 
+// TODO: Why it requires config as a directory and not as a file?
 fn main() -> Result<()> {
     actix::System::with_tokio_rt(|| {
         tokio::runtime::Builder::new_multi_thread()
diff --git a/wallet/src/main.rs b/wallet/src/main.rs
index ecc50d2..59f90fd 100644
--- a/wallet/src/main.rs
+++ b/wallet/src/main.rs
@@ -5,6 +5,10 @@ use wallet::{Args, execute_continious_run, execute_subcommand};
 
 pub const NUM_THREADS: usize = 2;
 
+// TODO #169: We have sample configs for sequencer, but not for wallet
+// TODO #168: Why it requires config as a directory? Maybe better to deduce directory from config file path?
+// TODO #172: Why it requires config as env var while sequencer_runner accepts as argument?
+// TODO #171: Running pinata doesn't give output about transaction hash and etc.
 fn main() -> Result<()> {
     let runtime = Builder::new_multi_thread()
         .worker_threads(NUM_THREADS)
@@ -18,6 +22,7 @@ fn main() -> Result<()> {
 
     runtime.block_on(async move {
         if let Some(command) = args.command {
+            // TODO: It should return error, not panic
             execute_subcommand(command).await.unwrap();
         } else if args.continious_run {
             execute_continious_run().await.unwrap();