Remove invalid BRC20 inscription tracking feature

Author: wanyvic

src/index.rs

@@ -125,7 +125,6 @@ pub(crate) enum Statistic {
   OkxIndexBitmap = 19,
   OkxIndexBTCDomain = 20,
   OkxSaveInscriptionReceipts = 21,
-  OkxNoTrackingInvalidBrc20Inscriptions = 22,
 }
 
 impl Statistic {
@@ -249,7 +248,6 @@ pub struct Index {
   index_bitmap: bool,
   index_btc_domain: bool,
   save_inscription_receipts: bool,
-  disable_invalid_brc20_tracking: bool,
 }
 
 impl Index {
@@ -428,12 +426,6 @@ impl Index {
               u64::from(settings.index_brc20()),
             )?;
 
-            Self::set_statistic(
-              &mut statistics,
-              Statistic::OkxNoTrackingInvalidBrc20Inscriptions,
-              u64::from(settings.disable_invalid_brc20_tracking()),
-            )?;
-
             Self::set_statistic(
               &mut statistics,
               Statistic::OkxSaveInscriptionReceipts,
@@ -517,7 +509,6 @@ impl Index {
     let index_bitmap;
     let index_btc_domain;
     let save_inscription_receipts;
-    let disable_invalid_brc20_tracking;
 
     {
       let tx = database.begin_read()?;
@@ -529,10 +520,6 @@ impl Index {
       index_transactions = Self::is_statistic_set(&statistics, Statistic::IndexTransactions)?;
 
       index_brc20 = Self::is_statistic_set(&statistics, Statistic::OkxIndexBrc20)?;
-      disable_invalid_brc20_tracking = Self::is_statistic_set(
-        &statistics,
-        Statistic::OkxNoTrackingInvalidBrc20Inscriptions,
-      )?;
 
       index_bitmap = Self::is_statistic_set(&statistics, Statistic::OkxIndexBitmap)?;
       index_btc_domain = Self::is_statistic_set(&statistics, Statistic::OkxIndexBTCDomain)?;
@@ -578,7 +565,6 @@ impl Index {
       index_bitmap,
       index_btc_domain,
       save_inscription_receipts,
-      disable_invalid_brc20_tracking,
     })
   }
   pub(crate) fn with_metrics(mut self) -> Self {

src/index/bundle_message.rs

@@ -38,26 +38,6 @@ pub struct BundleMessage {
   pub inscription_action: InscriptionAction,
 }
 
-impl BundleMessage {
-  /// Determines whether this inscription needs to be tracked.
-  /// Returns `false` when the message is a BRC20 Mint or Transfer, otherwise `true`.
-  pub fn should_track(&self, index: &Index) -> bool {
-    if !index.disable_invalid_brc20_tracking {
-      return true;
-    }
-
-    if let InscriptionAction::Created { sub_type, .. } = &self.inscription_action {
-      if let Some(SubType::BRC20(operation)) = sub_type {
-        return !matches!(
-          operation,
-          BRC20Operation::Mint { .. } | BRC20Operation::InscribeTransfer(_)
-        );
-      }
-    }
-    true
-  }
-}
-
 impl BundleMessage {
   pub(in crate::index) fn from_okx_inscription_event(
     event: OkxInscriptionEvent,

src/index/updater/inscription_updater.rs

@@ -595,8 +595,6 @@ impl InscriptionUpdater<'_, '_> {
         .or_insert(UtxoEntryBuf::empty(index))
     });
 
-    let mut should_tracking_inscription = true;
-
     // input_script_buf will only have a valid value when index_addresses is true.
     if index.index_addresses {
       let event = OkxInscriptionEvent {
@@ -632,8 +630,6 @@ impl InscriptionUpdater<'_, '_> {
       };
 
       if let Some(message) = BundleMessage::from_okx_inscription_event(event, self.height, index)? {
-        // We should decide whether to track the inscription based on the message.
-        should_tracking_inscription = message.should_track(index);
         self
           .block_bundle_messages
           .entry(message.txid)
@@ -642,10 +638,7 @@ impl InscriptionUpdater<'_, '_> {
       }
     }
 
-    // If we are tracking the inscription, we need to push the inscription into the output UTXO entry.
-    if should_tracking_inscription {
-      output_utxo_entry.push_inscription(sequence_number, satpoint.offset, index);
-    }
+    output_utxo_entry.push_inscription(sequence_number, satpoint.offset, index);
 
     Ok(())
   }

src/okx/brc20/fixed_point.rs

@@ -27,6 +27,7 @@ impl FixedPoint {
   #[cfg(test)]
   const MIN: Self = Self { value: 0, scale: 0 };
 
+  #[allow(unused)]
   pub fn new(value: u128, scale: u8) -> Result<Self, NumParseError> {
     if scale > Self::MAX_SCALE {
       return Err(NumParseError::OutOfMaxScale(i64::from(scale)));

src/okx/composite_key.rs

@@ -94,10 +94,13 @@ pub struct CompositeKey<T, U> {
 }
 
 impl<T: Clone, U> CompositeKey<T, U> {
+  
+  #[allow(unused)]
   pub fn primary_left_endpoint(&self) -> Endpoint<T> {
     Endpoint::Left(self.primary.clone())
   }
 
+  #[allow(unused)]
   pub fn primary_right_endpoint(&self) -> Endpoint<T> {
     Endpoint::Right(self.primary.clone())
   }

src/options.rs

@@ -101,9 +101,4 @@ pub struct Options {
   pub(crate) index_btc_domain: bool,
   #[arg(long, help = "Index BRC-20 token operations.")]
   pub(crate) index_brc20: bool,
-  #[arg(
-    long,
-    help = "Disable tracking of invalid BRC-20 inscriptions. [default: false]"
-  )]
-  pub(crate) disable_invalid_brc20_tracking: bool,
 }

src/settings.rs

@@ -35,7 +35,6 @@ pub struct Settings {
   index_bitmap: bool,
   index_btc_domain: bool,
   index_brc20: bool,
-  disable_invalid_brc20_tracking: bool,
 }
 
 impl Settings {
@@ -156,8 +155,6 @@ impl Settings {
       index_bitmap: self.index_bitmap || source.index_bitmap,
       index_btc_domain: self.index_btc_domain || source.index_btc_domain,
       index_brc20: self.index_brc20 || source.index_brc20,
-      disable_invalid_brc20_tracking: self.disable_invalid_brc20_tracking
-        || source.disable_invalid_brc20_tracking,
     }
   }
 
@@ -201,7 +198,6 @@ impl Settings {
       index_bitmap: options.index_bitmap,
       index_btc_domain: options.index_btc_domain,
       index_brc20: options.index_brc20,
-      disable_invalid_brc20_tracking: options.disable_invalid_brc20_tracking,
     }
   }
 
@@ -299,7 +295,6 @@ impl Settings {
       index_bitmap: get_bool("INDEX_BITMAP"),
       index_btc_domain: get_bool("INDEX_BTC_DOMAIN"),
       index_brc20: get_bool("INDEX_BRC20"),
-      disable_invalid_brc20_tracking: get_bool("DISABLE_INVALID_BRC20_TRACKING"),
     })
   }
 
@@ -337,7 +332,6 @@ impl Settings {
       index_bitmap: false,
       index_btc_domain: false,
       index_brc20: false,
-      disable_invalid_brc20_tracking: false,
     }
   }
 
@@ -419,7 +413,6 @@ impl Settings {
       index_bitmap: self.index_bitmap,
       index_btc_domain: self.index_btc_domain,
       index_brc20: self.index_brc20,
-      disable_invalid_brc20_tracking: self.disable_invalid_brc20_tracking,
     })
   }
 
@@ -654,10 +647,6 @@ impl Settings {
     self.index_brc20
   }
 
-  pub(crate) fn disable_invalid_brc20_tracking(&self) -> bool {
-    self.disable_invalid_brc20_tracking
-  }
-
   pub(crate) fn index_bitmap(&self) -> bool {
     self.index_bitmap
   }
@@ -1144,7 +1133,6 @@ mod tests {
       ("INDEX_BITMAP", "1"),
       ("INDEX_BTC_DOMAIN", "1"),
       ("INDEX_BRC20", "1"),
-      ("DISABLE_INVALID_BRC20_TRACKING", "1"),
     ]
     .into_iter()
     .map(|(key, value)| (key.into(), value.into()))
@@ -1195,7 +1183,6 @@ mod tests {
         index_bitmap: true,
         index_btc_domain: true,
         index_brc20: true,
-        disable_invalid_brc20_tracking: true,
       }
     );
   }
@@ -1234,7 +1221,6 @@ mod tests {
           "--index-bitmap",
           "--index-btc-domain",
           "--index-brc20",
-          "--disable-invalid-brc20-tracking",
         ])
         .unwrap()
       ),
@@ -1270,7 +1256,6 @@ mod tests {
         index_bitmap: true,
         index_btc_domain: true,
         index_brc20: true,
-        disable_invalid_brc20_tracking: true,
       }
     );
   }