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

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## [1.7.5](https://github.com/tomerh2001/israeli-banks-actual-budget-importer/compare/v1.7.4...v1.7.5) (2025-12-04)
+
+
+### Bug Fixes
+
+* optimize reconciliation logic to skip unnecessary updates when balances are in sync ([5153911](https://github.com/tomerh2001/israeli-banks-actual-budget-importer/commit/515391177d37e5ef0577ab5a306317e21bf990e4))
+
+## [1.7.4](https://github.com/tomerh2001/israeli-banks-actual-budget-importer/compare/v1.7.3...v1.7.4) (2025-12-04)
+
+
+### Bug Fixes
+
+* remove unnecessary reconciliation removal logic when balances are in sync ([0fcc569](https://github.com/tomerh2001/israeli-banks-actual-budget-importer/commit/0fcc56977394265502dd871bc0973a05e614fffe))
+
 ## [1.7.3](https://github.com/tomerh2001/israeli-banks-actual-budget-importer/compare/v1.7.2...v1.7.3) (2025-12-04)
 
 

package/package.json

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

package/src/utils.ts

@@ -91,19 +91,9 @@ export async function scrapeAndImportTransactions({companyId, bank}: ScrapeTrans
 
 		const existingReconciliation = allAccountTxns.find(txn => txn.imported_id === reconciliationImportedId);
 
-		log('RECONCILIATION', {
-			from: currentBalance,
-			to: accountBalance,
-			diff: balanceDiff,
-		});
-
-		// If balances are already in sync, remove any existing reconciliation and exit.
+		// If balances are already in sync, no need to create/update reconciliation.
 		if (existingReconciliation && balanceDiff === 0) {
-			stdout.mute();
-			await actual.deleteTransaction(existingReconciliation.id);
-			stdout.unmute();
-			log(`RECONCILIATION_REMOVED, since ${currentBalance.toLocaleString()} === ${accountBalance.toLocaleString()}`, {transactionId: existingReconciliation.id});
-
+			log('RECONCILIATION_NOT_NEEDED');
 			return;
 		}
 
@@ -117,28 +107,24 @@ export async function scrapeAndImportTransactions({companyId, bank}: ScrapeTrans
 			imported_id: reconciliationImportedId,
 		};
 
-		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});
 
+			log('RECONCILIATION_UPDATED', {from: currentBalance, to: accountBalance, diff: balanceDiff});
 			return;
 		}
 
 		// Create the reconciliation transaction for the first time
-		const reconciliationResult = await actual.importTransactions(
-			bank.actualAccountId,
-			[reconciliationTxn],
-		);
+		stdout.mute();
+		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});
+			log('RECONCILIATION_ADDED', {from: currentBalance, to: accountBalance, diff: balanceDiff});
 		}
 	} catch (error) {
 		console.error('Error', companyId, error);