feat(core): expose TxIn types in public API

Export TxIn at the crate root and add TxInExt to the prelude module.

Update internal imports throughout the codebase to use the new public
path.

Author: alexanderwiederin

src/core/mod.rs

@@ -10,11 +10,11 @@ pub use block::{
 };
 pub use block_tree_entry::BlockTreeEntry;
 pub use script::{ScriptPubkey, ScriptPubkeyRef};
-pub use transaction::{Transaction, TransactionRef, TxOut, TxOutRef};
+pub use transaction::{Transaction, TransactionRef, TxIn, TxOut, TxOutRef};
 
 pub use block::{BlockHashExt, BlockSpentOutputsExt, CoinExt, TransactionSpentOutputsExt};
 pub use script::ScriptPubkeyExt;
-pub use transaction::{TransactionExt, TxOutExt};
+pub use transaction::{TransactionExt, TxInExt, TxOutExt};
 
 pub use verify::{verify, ScriptVerifyError, ScriptVerifyStatus};
 

src/lib.rs

@@ -113,7 +113,7 @@ impl std::error::Error for KernelError {
 pub use crate::core::{
     verify, Block, BlockHash, BlockSpentOutputs, BlockSpentOutputsRef, BlockTreeEntry, Coin,
     CoinRef, ScriptPubkey, ScriptPubkeyRef, ScriptVerifyError, ScriptVerifyStatus, Transaction,
-    TransactionRef, TransactionSpentOutputs, TransactionSpentOutputsRef, TxOut, TxOutRef,
+    TransactionRef, TransactionSpentOutputs, TransactionSpentOutputsRef, TxIn, TxOut, TxOutRef,
 };
 
 pub use crate::log::{disable_logging, Log, LogCategory, LogLevel, Logger};
@@ -138,7 +138,7 @@ pub use crate::core::verify_flags::{
 pub mod prelude {
     pub use crate::core::{
         BlockHashExt, BlockSpentOutputsExt, CoinExt, ScriptPubkeyExt, TransactionExt,
-        TransactionSpentOutputsExt, TxOutExt,
+        TransactionSpentOutputsExt, TxInExt, TxOutExt,
     };
     pub use crate::notifications::BlockValidationStateExt;
 }

tests/test.rs

@@ -1,13 +1,12 @@
 #[cfg(test)]
 mod tests {
     use bitcoin::consensus::deserialize;
-    use bitcoinkernel::core::transaction::TxIn;
     use bitcoinkernel::{
         prelude::*, verify, Block, BlockHash, BlockSpentOutputs, BlockTreeEntry,
         BlockValidationStateRef, ChainParams, ChainType, ChainstateManager,
         ChainstateManagerBuilder, Coin, Context, ContextBuilder, KernelError, Log, Logger,
-        ScriptPubkey, ScriptVerifyError, Transaction, TransactionSpentOutputs, TxOut, TxOutRef,
-        VERIFY_ALL_PRE_TAPROOT, VERIFY_TAPROOT, VERIFY_WITNESS,
+        ScriptPubkey, ScriptVerifyError, Transaction, TransactionSpentOutputs, TxIn, TxOut,
+        TxOutRef, VERIFY_ALL_PRE_TAPROOT, VERIFY_TAPROOT, VERIFY_WITNESS,
     };
     use std::fs::File;
     use std::io::{BufRead, BufReader};