[PR #7320] [AI] Respect user notes when rules append or prepend on new transactions (#7303) #56387

Open
opened 2026-05-01 04:13:20 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/7320
Author: @pranayseela
Created: 3/29/2026
Status: 🔄 Open

Base: masterHead: fix/7303-rule-notes-on-new-transactions


📝 Commits (5)

  • 63b3a42 [AI] Respect user notes when rules append or prepend on new transactions (#7303)
  • b55084a [AI] Add release note for PR 7320 (#7303)
  • a4148cc [AI] Await initial file write in upload-user-file sync test
  • 548c185 [AI] Tighten release note wording for PR 7320 (imperative, user-facing)
  • ab453bd [AI] Expand shouldApplyRuleDiffField tests for #7303 edge cases

📊 Changes

6 files changed (+90 additions, -16 deletions)

View changed files

📝 packages/desktop-client/src/components/mobile/transactions/TransactionEdit.tsx (+7 -8)
📝 packages/desktop-client/src/components/transactions/TransactionList.tsx (+11 -5)
📝 packages/loot-core/src/shared/util.test.ts (+25 -0)
📝 packages/loot-core/src/shared/util.ts (+36 -0)
📝 packages/sync-server/src/app-sync.test.ts (+5 -3)
upcoming-release-notes/7320.md (+6 -0)

📄 Description

Fixes #7303

Made with Cursor


Bundle Stats

Bundle Files count Total bundle size % Changed
desktop-client 27 12.09 MB → 12.09 MB (-118 B) -0.00%
loot-core 1 4.83 MB 0%
api 4 4.06 MB 0%
cli 1 7.88 MB 0%
View detailed bundle stats

desktop-client

Total

Files count Total bundle size % Changed
27 12.09 MB → 12.09 MB (-118 B) -0.00%
Changeset
File Δ Size
home/runner/work/actual/actual/packages/loot-core/src/shared/util.ts 📈 +388 B (+4.27%) 8.87 kB → 9.25 kB
src/components/mobile/transactions/TransactionEdit.tsx 📉 -76 B (-0.13%) 58.28 kB → 58.2 kB
src/components/transactions/TransactionList.tsx 📉 -42 B (-0.24%) 17.15 kB → 17.11 kB
View detailed bundle breakdown

Added
No assets were added

Removed
No assets were removed

Bigger
No assets were bigger

Smaller

Asset File Size % Changed
static/js/index.js 3.23 MB → 3.23 MB (-76 B) -0.00%
static/js/useTransactionBatchActions.js 4.29 MB → 4.29 MB (-42 B) -0.00%

Unchanged

Asset File Size % Changed
static/js/BackgroundImage.js 119.98 kB 0%
static/js/FormulaEditor.js 846.44 kB 0%
static/js/ReportRouter.js 1.02 MB 0%
static/js/TransactionList.js 81.29 kB 0%
static/js/ca.js 182.91 kB 0%
static/js/da.js 104.66 kB 0%
static/js/de.js 174.79 kB 0%
static/js/en-GB.js 7.16 kB 0%
static/js/en.js 170.76 kB 0%
static/js/es.js 182.09 kB 0%
static/js/fr.js 177.47 kB 0%
static/js/indexeddb-main-thread-worker-e59fee74.js 13.46 kB 0%
static/js/it.js 166.25 kB 0%
static/js/narrow.js 354.27 kB 0%
static/js/nb-NO.js 152.2 kB 0%
static/js/nl.js 108.93 kB 0%
static/js/pl.js 88.34 kB 0%
static/js/pt-BR.js 177.84 kB 0%
static/js/resize-observer.js 18.03 kB 0%
static/js/sv.js 80.58 kB 0%
static/js/th.js 179.94 kB 0%
static/js/theme.js 30.68 kB 0%
static/js/uk.js 213.14 kB 0%
static/js/wide.js 418 B 0%
static/js/workbox-window.prod.es5.js 7.28 kB 0%

loot-core

Total

Files count Total bundle size % Changed
1 4.83 MB 0%
View detailed bundle breakdown

Added
No assets were added

Removed
No assets were removed

Bigger
No assets were bigger

Smaller
No assets were smaller

Unchanged

Asset File Size % Changed
kcab.worker.CwpE34S5.js 4.83 MB 0%

api

Total

Files count Total bundle size % Changed
4 4.06 MB 0%
View detailed bundle breakdown

Added
No assets were added

Removed
No assets were removed

Bigger
No assets were bigger

Smaller
No assets were smaller

Unchanged

Asset File Size % Changed
index.js 3.84 MB 0%
from-Bl-Hslp4.js 167.73 kB 0%
multipart-parser-BnDysoMr.js 8.1 kB 0%
src-iMkUmuwR.js 43.64 kB 0%

cli

Total

Files count Total bundle size % Changed
1 7.88 MB 0%
View detailed bundle breakdown

Added
No assets were added

Removed
No assets were removed

Bigger
No assets were bigger

Smaller
No assets were smaller

Unchanged

Asset File Size % Changed
cli.js 7.88 MB 0%

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/actualbudget/actual/pull/7320 **Author:** [@pranayseela](https://github.com/pranayseela) **Created:** 3/29/2026 **Status:** 🔄 Open **Base:** `master` ← **Head:** `fix/7303-rule-notes-on-new-transactions` --- ### 📝 Commits (5) - [`63b3a42`](https://github.com/actualbudget/actual/commit/63b3a42efda8053e079b3efceddb9d98a624131b) [AI] Respect user notes when rules append or prepend on new transactions (#7303) - [`b55084a`](https://github.com/actualbudget/actual/commit/b55084af38e39164254d7ceaaf7efdbdc2b95964) [AI] Add release note for PR 7320 (#7303) - [`a4148cc`](https://github.com/actualbudget/actual/commit/a4148cca5e1f15d462f3e09b60cf1684f49c0dba) [AI] Await initial file write in upload-user-file sync test - [`548c185`](https://github.com/actualbudget/actual/commit/548c1857b9002f536342858891d3fad2fa3d2bba) [AI] Tighten release note wording for PR 7320 (imperative, user-facing) - [`ab453bd`](https://github.com/actualbudget/actual/commit/ab453bde5e7e40495b33f18a9854d1b9131bc6ae) [AI] Expand shouldApplyRuleDiffField tests for #7303 edge cases ### 📊 Changes **6 files changed** (+90 additions, -16 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/mobile/transactions/TransactionEdit.tsx` (+7 -8) 📝 `packages/desktop-client/src/components/transactions/TransactionList.tsx` (+11 -5) 📝 `packages/loot-core/src/shared/util.test.ts` (+25 -0) 📝 `packages/loot-core/src/shared/util.ts` (+36 -0) 📝 `packages/sync-server/src/app-sync.test.ts` (+5 -3) ➕ `upcoming-release-notes/7320.md` (+6 -0) </details> ### 📄 Description Fixes #7303 Made with [Cursor](https://cursor.com) <!--- actual-bot-sections ---> <hr /> <!--- bundlestats-action-comment key:combined start ---> ### Bundle Stats Bundle | Files count | Total bundle size | % Changed ------ | ----------- | ----------------- | --------- desktop-client | 27 | 12.09 MB → 12.09 MB (-118 B) | -0.00% loot-core | 1 | 4.83 MB | 0% api | 4 | 4.06 MB | 0% cli | 1 | 7.88 MB | 0% <details> <summary>View detailed bundle stats</summary> #### desktop-client **Total** Files count | Total bundle size | % Changed ----------- | ----------------- | --------- 27 | 12.09 MB → 12.09 MB (-118 B) | -0.00% <details> <summary>Changeset</summary> File | Δ | Size ---- | - | ---- `home/runner/work/actual/actual/packages/loot-core/src/shared/util.ts` | 📈 +388 B (+4.27%) | 8.87 kB → 9.25 kB `src/components/mobile/transactions/TransactionEdit.tsx` | 📉 -76 B (-0.13%) | 58.28 kB → 58.2 kB `src/components/transactions/TransactionList.tsx` | 📉 -42 B (-0.24%) | 17.15 kB → 17.11 kB </details> <details> <summary>View detailed bundle breakdown</summary> <div> **Added** No assets were added **Removed** No assets were removed **Bigger** No assets were bigger **Smaller** Asset | File Size | % Changed ----- | --------- | --------- static/js/index.js | 3.23 MB → 3.23 MB (-76 B) | -0.00% static/js/useTransactionBatchActions.js | 4.29 MB → 4.29 MB (-42 B) | -0.00% **Unchanged** Asset | File Size | % Changed ----- | --------- | --------- static/js/BackgroundImage.js | 119.98 kB | 0% static/js/FormulaEditor.js | 846.44 kB | 0% static/js/ReportRouter.js | 1.02 MB | 0% static/js/TransactionList.js | 81.29 kB | 0% static/js/ca.js | 182.91 kB | 0% static/js/da.js | 104.66 kB | 0% static/js/de.js | 174.79 kB | 0% static/js/en-GB.js | 7.16 kB | 0% static/js/en.js | 170.76 kB | 0% static/js/es.js | 182.09 kB | 0% static/js/fr.js | 177.47 kB | 0% static/js/indexeddb-main-thread-worker-e59fee74.js | 13.46 kB | 0% static/js/it.js | 166.25 kB | 0% static/js/narrow.js | 354.27 kB | 0% static/js/nb-NO.js | 152.2 kB | 0% static/js/nl.js | 108.93 kB | 0% static/js/pl.js | 88.34 kB | 0% static/js/pt-BR.js | 177.84 kB | 0% static/js/resize-observer.js | 18.03 kB | 0% static/js/sv.js | 80.58 kB | 0% static/js/th.js | 179.94 kB | 0% static/js/theme.js | 30.68 kB | 0% static/js/uk.js | 213.14 kB | 0% static/js/wide.js | 418 B | 0% static/js/workbox-window.prod.es5.js | 7.28 kB | 0% </div> </details> --- #### loot-core **Total** Files count | Total bundle size | % Changed ----------- | ----------------- | --------- 1 | 4.83 MB | 0% <details> <summary>View detailed bundle breakdown</summary> <div> **Added** No assets were added **Removed** No assets were removed **Bigger** No assets were bigger **Smaller** No assets were smaller **Unchanged** Asset | File Size | % Changed ----- | --------- | --------- kcab.worker.CwpE34S5.js | 4.83 MB | 0% </div> </details> --- #### api **Total** Files count | Total bundle size | % Changed ----------- | ----------------- | --------- 4 | 4.06 MB | 0% <details> <summary>View detailed bundle breakdown</summary> <div> **Added** No assets were added **Removed** No assets were removed **Bigger** No assets were bigger **Smaller** No assets were smaller **Unchanged** Asset | File Size | % Changed ----- | --------- | --------- index.js | 3.84 MB | 0% from-Bl-Hslp4.js | 167.73 kB | 0% multipart-parser-BnDysoMr.js | 8.1 kB | 0% src-iMkUmuwR.js | 43.64 kB | 0% </div> </details> --- #### cli **Total** Files count | Total bundle size | % Changed ----------- | ----------------- | --------- 1 | 7.88 MB | 0% <details> <summary>View detailed bundle breakdown</summary> <div> **Added** No assets were added **Removed** No assets were removed **Bigger** No assets were bigger **Smaller** No assets were smaller **Unchanged** Asset | File Size | % Changed ----- | --------- | --------- cli.js | 7.88 MB | 0% </div> </details> </details> <!--- bundlestats-action-comment key:combined end ---> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-05-01 04:13:20 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#56387