Skip to content

Commit 73d5250

Browse files
promptexecutionerrClaude Sonnet (coordinator)claude
authored
fix(ledger-core): remove conflicting From impl in ingest.rs (#93)
From<&TransactionInput> for DocumentFields conflicted with Rust's blanket TryFrom via Into. Keep only TryFrom (the fallible parse-aware version). Co-authored-by: Claude Sonnet (coordinator) <coordinator@promptexecution.com.au> Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 53debf5 commit 73d5250

1 file changed

Lines changed: 0 additions & 7 deletions

File tree

crates/ledger-core/src/ingest.rs

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -108,13 +108,6 @@ pub fn deterministic_tx_id(row: &TransactionInput) -> String {
108108
blake3::hash(canonical.as_bytes()).to_hex().to_string()
109109
}
110110

111-
impl From<&TransactionInput> for crate::pipeline::DocumentFields {
112-
fn from(row: &TransactionInput) -> Self {
113-
Self {
114-
amount: row.amount.trim().parse().ok(),
115-
..Self::default()
116-
}
117-
118111
#[derive(Debug, Clone, PartialEq, Eq)]
119112
pub struct DocumentFieldsParseError {
120113
pub field: String,

0 commit comments

Comments
 (0)