israeli-banks-actual-budget-importer 1.7.1 → 1.7.3

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## [1.7.3](https://github.com/tomerh2001/israeli-banks-actual-budget-importer/compare/v1.7.2...v1.7.3) (2025-12-04)
+
+
+### Bug Fixes
+
+* streamline reconciliation transaction handling and improve logging ([46fc2b0](https://github.com/tomerh2001/israeli-banks-actual-budget-importer/commit/46fc2b082e4e7b69666bb338960e0b7e6152f892))
+
+## [1.7.2](https://github.com/tomerh2001/israeli-banks-actual-budget-importer/compare/v1.7.1...v1.7.2) (2025-12-04)
+
+
+### Bug Fixes
+
+* improve reconciliation logic to log transaction removal when balances are in sync ([c2f3fba](https://github.com/tomerh2001/israeli-banks-actual-budget-importer/commit/c2f3fba73588fb1591ddfcc92cb0e2b0fe9a54f5))
+
 ## [1.7.1](https://github.com/tomerh2001/israeli-banks-actual-budget-importer/compare/v1.7.0...v1.7.1) (2025-12-04)
 
 

package/package.json

@@ -1,5 +1,5 @@
 {
-  "version": "1.7.1",
+  "version": "1.7.3",
   "name": "israeli-banks-actual-budget-importer",
   "module": "index.ts",
   "type": "module",

package/src/utils.ts

@@ -91,24 +91,22 @@ export async function scrapeAndImportTransactions({companyId, bank}: ScrapeTrans
 
 		const existingReconciliation = allAccountTxns.find(txn => txn.imported_id === reconciliationImportedId);
 
-		// If balances are already in sync, remove any existing reconciliation and exit.
-		if (balanceDiff === 0) {
-			if (existingReconciliation) {
-				stdout.mute();
-				await actual.deleteTransaction(existingReconciliation.id);
-				stdout.unmute();
-				log('RECONCILIATION_REMOVED');
-			}
-
-			return;
-		}
-
 		log('RECONCILIATION', {
 			from: currentBalance,
 			to: accountBalance,
 			diff: balanceDiff,
 		});
 
+		// If balances are already in sync, remove any existing reconciliation and exit.
+		if (existingReconciliation && balanceDiff === 0) {
+			stdout.mute();
+			await actual.deleteTransaction(existingReconciliation.id);
+			stdout.unmute();
+			log(`RECONCILIATION_REMOVED, since ${currentBalance.toLocaleString()} === ${accountBalance.toLocaleString()}`, {transactionId: existingReconciliation.id});
+
+			return;
+		}
+
 		const reconciliationTxn = {
 			account: bank.actualAccountId,
 			date: moment().format('YYYY-MM-DD'),
@@ -122,22 +120,25 @@ export async function scrapeAndImportTransactions({companyId, bank}: ScrapeTrans
 		stdout.mute();
 		if (existingReconciliation) {
 			// Update the single reconciliation transaction
+			stdout.mute();
 			await actual.updateTransaction(existingReconciliation.id, reconciliationTxn);
 			stdout.unmute();
 			log('RECONCILIATION_UPDATED', {transactionId: existingReconciliation.id});
-		} else {
-			// Create the reconciliation transaction for the first time
-			const reconciliationResult = await actual.importTransactions(
-				bank.actualAccountId,
-				[reconciliationTxn],
-			);
-			stdout.unmute();
 
-			if (!reconciliationResult || _.isEmpty(reconciliationResult.added)) {
-				console.error('Reconciliation errors', reconciliationResult?.errors);
-			} else {
-				log('RECONCILIATION_ADDED', {transactions: reconciliationResult.added.length});
-			}
+			return;
+		}
+
+		// Create the reconciliation transaction for the first time
+		const reconciliationResult = await actual.importTransactions(
+			bank.actualAccountId,
+			[reconciliationTxn],
+		);
+		stdout.unmute();
+
+		if (!reconciliationResult || _.isEmpty(reconciliationResult.added)) {
+			console.error('Reconciliation errors', reconciliationResult?.errors);
+		} else {
+			log('RECONCILIATION_ADDED', {transactions: reconciliationResult.added.length});
 		}
 	} catch (error) {
 		console.error('Error', companyId, error);