mirror of
https://github.com/actualbudget/actual.git
synced 2026-05-06 07:01:45 -05:00
[PR #7451] [AI] Formula Feature Improvements: FORMATNUMBER, FORMATCURRENCY, and Tooltip Fixes #33100
Open
opened 2026-04-18 09:02:00 -05:00 by GiteaMirror
·
0 comments
No Branch/Tag Specified
master
claude/hide-default-categories-1cwBZ
matiss/crdt-source-loading
youngcw/unlock-duplicates
matiss/crdt-protobuf
release/26.5.0
claude/update-issue-template-ykMNn
claude/fix-issue-7667-DPXi3
cursor/formula-feedback-improvements-4223
cursor/resolve-pr-7449-ee11
claude/fix-typescript-build-error-JPtZ5
jfdoming/api-tokens-part-3
jfdoming/api-tokens-part-2
jfdoming/api-tokens-part-1
claude/speed-up-vrt-workflow-ZAyI5
claude/crdt-version-auto-publish-Ph1BH
copilot/add-repository-configs-to-packages
worktree-compressed-drifting-ritchie
worktree-mellow-strolling-dawn
matiss/browser-api
claude/api-consumer-verification-kfz1K
feature/enable-banking
cursor/transaction-table-rewrite-f077
pr-7454
claude/fix-issue-7410-LLLQ4
release/v100.0.0
revert-7350-trim-deps
revert-7220-sankey-report
revert-7242-fix/split-parent-update-corruption
revert-7281-generate-icons
claude/electron-to-tauri-migration-LjBN8
worktree-remotion
release/vv26.4.0-pre
claude/browser-compatible-api-QbhHh
claude/improve-cli-transactions-waTUY
claude/publish-react-native-ios-j8qoT
js-proxy
claude/fix-flaky-ci-job-5gDdz
react-query-rules
react-query-useSchedules
claude/nightly-theme-validation-scan-DzOGD
claude/debug-simplefin-error-ZuKzB
matiss/desktop-client-subpath-imports
claude/fix-simplefin-ssrf-T31gX
claude/release-notes-validation-X7rvR
add-claude-github-actions-1772738270730
cursor/sync-performance-notification-9899
react-query-prefs
matiss/chunked-sync-and-progress-ux
v26.2.1
copilot/sub-pr-6880
fix-react-query-clear-on-close-budget
copilot/sub-pr-6140
feat/auto-note
feat/scoped-bank-sync
cursor/desktop-transactions-react-table-1d0c
fix-exhaustive-deps-App
copilot/fix-find-replace-bug
release/v26.2.0-pre
matiss/browser-tests
mobile-fix-drag-and-drop-across-groups
budget-table-v2
PayeeAutocomplete2
pglite
bugfix/plugins/fix-plugins-sw
feat/plugins/plugins-core-package
prerelease
matiss/unicode-minus-fix
cursor/fix-actual-github-issue-6206-gemini-3-pro-preview-9c37
TransactionFormPage
cursor/implement-mortgage-and-loan-account-type-78ca
tests-update-fill-with-pressSequentially
mobile/link-modal
deps/25.11
cursor/fix-update-vrt-apply-ci-job-dispatch-b324
sync-server-plugins
cursor/propose-patch-for-github-issue-5680-2a18
fix/compiler-preserve-inner-dollar-escapes
cursor/analyze-actual-budget-issue-and-propose-fix-5b70
coderabbitai/docstrings/0c070e5
cursor/add-wip-prefix-and-comment-to-prs-d78d
jfdoming/08-21-auto-focus-on-navigate-in-all-browsers
show-totals-on-mobile-budget-banners
allow-child-transactions-make-transfer
mobile-calculator-keyboard
payee-geolocation
enhance/restore_scroll_position
dm-fix-second-click-on-mobile-new-transaction-2
scrollToLocationBudget
alert-autofix-38
tsconfig-composite
mobile-fix-uncategorized-transactions-on-tracking-budgets
server-budget-handlers
fix-sql-injection-in-cleanup-template
non-chrome-draggable-workaround
mobile-budget-page-swipe-navigation
ts-db-all
stable
dark-theme-with-brand-colors
fix-mobile-delete-group
ts-db-select
UnderKoen/reconcile-context-menu
master-before-server-merge
v25.2.1
ts-runQuery
rename-redux-hooks
UnderKoen/3557-persist-state-in-history
remove-redux-CLOSE_BUDGET
fix-exhaustive-deps-errors-FinancesApp
redux-toolkit-createSlice-backup
accounts-function-component
ts-useSplitsExpanded
loot-core-server-package
useTransactios-in-TransactionEdit
react-aria-input
move-redux-to-desktop-client
QueryState-type
fix-themes-applied-late
mobile-vrts
revert-3295-spendingCardFix
react-aria-button-4
split-payee-on-mobile
twk3/pin-apis-crdt
notes-tag-autocomplete
ts-LoadBackup
dnd-kit
package-upgrades
v26.5.0
v26.4.0
v26.3.0
v26.2.1
v26.2.0
v26.1.0
v25.12.0
v25.11.0
v25.10.0
v25.9.0
v25.8.0
v25.7.1
v25.7.0
v25.6.1
v25.6.0
v25.5.0
v25.4.0
v25.3.1
v25.3.0
v25.2.1
v25.2.0
v25.1.0
v24.12.0
v24.11.0
v24.10.1
v24.10.0
v24.9.0
v24.8.0
v24.7.0
v24.6.0
v24.5.0
v24.4.0
v24.3.0
v24.2.0
v24.1.0
v23.12.0
v23.11.0
v23.10.0
v23.9.0
v23.8.1
v23.8.0
v23.7.2
v23.7.1
v23.7.0
v23.6.0
v23.5.0
v23.4.2
v23.4.1
v23.4.0
v23.3.2
v23.3.0
v23.2.9
v23.2.5
v23.1.12
v22.12.9
Labels
Clear labels
AI generated
API
bank sync
budgeting
bug
can’t replicate
dependencies
docker
documentation
electron
experimental feature
feature
feedback
goal templates
good first issue
help wanted
importers
maintenance
needs info
needs testing
needs triage
needs votes
openid
payees
pull-request
regression
reports
responsive
rules
schedules
server
✨ merged
split transactions
tech debt
theme
transaction import
transaction reconciliation
transactions
translations
upstream
user interface
✅ approved
wontfix
Mirrored from GitHub Pull Request
No Label
pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/actual#33100
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/actualbudget/actual/pull/7451
Author: @lelemm
Created: 4/9/2026
Status: 🔄 Open
Base:
master← Head:cursor/formula-feedback-improvements-4223📝 Commits (6)
6d2171c[AI] Add FORMATNUMBER and FORMATCURRENCY functions to address TEXT() formatting limitations3bb11c1[AI] Add support for line breaks in formula card results506f939[AI] Remove unusable formula functions that require cell ranges2e051ee[AI] Add comprehensive integration tests for formula rules and formula cards2fed7e2Potential fix for pull request finding 'Unused variable, import, function or class'c241c0b[AI] Make formatting functions respect app settings with locale-based fallbacks📊 Changes
9 files changed (+2433 additions, -249 deletions)
View changed files
📝
packages/desktop-client/src/components/formula/codeMirror-excelLanguage.tsx(+21 -40)📝
packages/desktop-client/src/components/formula/queryModeFunctions.ts(+63 -198)📝
packages/desktop-client/src/components/formula/transactionModeFunctions.ts(+47 -5)📝
packages/desktop-client/src/components/reports/FormulaResult.tsx(+16 -5)➕
packages/loot-core/src/server/reports/formula-card-integration.test.ts(+723 -0)📝
packages/loot-core/src/server/rules/action.ts(+5 -0)📝
packages/loot-core/src/server/rules/customFunctions.ts(+309 -1)➕
packages/loot-core/src/server/rules/formula-action-integration.test.ts(+1168 -0)📝
packages/loot-core/src/server/rules/formula-action.test.ts(+81 -0)📄 Description
Summary
This PR addresses several user feedback items from issue #5949 regarding the formula card and formula rules feature.
Changes Made
1. FORMATNUMBER() Function (Addresses TEXT() formatting limitations)
Feedback: Users reported that
TEXT(QUERY("NetWorth"), "$#,##0.00")doesn't properly format numbers with thousands separators. The TEXT function in HyperFormula has limited format code support compared to Excel.Solution: Added a new
FORMATNUMBER()function that properly formats numbers with thousands separators and respects app settings.Usage:
Settings Integration:
2. FORMATCURRENCY() Function (Proper currency formatting)
Feedback: Users need proper currency formatting for formula cards with symbol, thousands separators, and decimal places.
Solution: Added a new
FORMATCURRENCY()function that formats numbers as currency and respects app currency settings.Usage:
Settings Integration:
Locale-based Defaults:
en-GB→ GBP (£)de/fr/es/it/nl→ EUR (€)ja→ JPY (¥)en-IN/hi→ INR (₹)en-CA→ CAD ($)en-AU→ AUD ($)Number Format Patterns:
comma-dot: 1,000.00 (US/UK/Canada)dot-comma: 1.000,00 (Europe)space-comma: 1 000,00 (France)apostrophe-dot: 1'000.00 (Switzerland)comma-dot-in: 1,00,000.00 (India)3. Line Break Support in Formula Cards ✨
Feedback: User @Juulz reported that line breaks don't work in formula cards, making multi-line displays ugly. Requested ability to use CHAR(10) or CHAR(13) in CONCATENATE.
Solution: Updated FormulaResult component to support multi-line text with proper rendering and font size calculation.
Usage:
Result:
Implementation:
whiteSpace: 'pre-wrap'to preserve line breakstextAlign: 'center'to keep multi-line text centered4. Formula Function Cleanup 🧹
Feedback: User requested review of all formula functions to remove ones that don't make sense in Actual's context.
Problem: Many Excel functions require cell ranges (like
A1:A10) which don't exist in Actual. Actual uses named expressions likeQUERY("myQuery")and transaction field variables likeamountordate.Solution: Removed 27 unusable functions from autocomplete suggestions:
Removed Functions:
AVERAGEA,COUNT,COUNTA,COUNTBLANK,COUNTIF,COUNTIFSQUERY_COUNT()insteadMAXA,MINA,SUMIF,SUMIFS,SUMPRODUCT,SUMSQQUERY()with filters insteadMEDIAN,MODE,STDEV,STDEVP,VAR,VARP,PERCENTILE,QUARTILE,RANKVLOOKUP,HLOOKUP,INDEX,MATCH,LOOKUPISREFKept Useful Functions:
SUM,AVERAGE,MAX,MIN,ROUND, etc.CONCATENATE,UPPER,LEFT,RIGHT, etc.TODAY,YEAR,MONTH,EOMONTH, etc.IF,AND,OR,IFERROR, etc.QUERY,QUERY_COUNT,BUDGET_QUERY, etc.CHOOSE(works with index and individual values)Impact: Autocomplete suggestions are now more relevant and less confusing. Users won't see functions they can't actually use.
5. Tooltip Quote Consistency Fix
Feedback: Tooltips used backticks around formula examples while actual usage used double quotes, causing confusion.
Solution: Removed backticks from all formula examples in tooltips for consistency. Examples now show formulas as plain text without wrapping backticks.
Before:
`=UPPER(notes)`After:
=UPPER(notes)6. Updated Documentation
queryModeFunctions.ts(for formula cards)transactionModeFunctions.ts(for formula rules)7. Comprehensive Integration Tests 🧪
Added 50 integration tests that validate formulas with real database operations:
Formula Rule Integration Tests (36 tests)
Tests validate formula execution with actual transaction data:
Formula Card Integration Tests (14 tests)
Tests validate formulas with real query results from database:
Test Quality:
Technical Implementation
Settings Architecture
Problem: HyperFormula custom functions are synchronous, but preferences are stored in async database.
Solution:
loadUserPreferencesForFormulas()async function that loads preferences onceSettings Priority:
This approach avoids unnecessary reactors while properly respecting user preferences!
Feedbacks Not Addressed (with reasoning)
BALANCEONDAY() Function
Feedback: Request for a function to get account balance on a specific date for credit card statement calculations.
Reason not implemented: This would require database access to query transaction history, which is not available in the formula execution context (both for formula cards and formula rules). The formula execution is designed to be stateless and work with the data provided in the context. Implementing this would require architectural changes to allow async database queries during formula execution.
Alternative approach: Users can currently achieve this by creating a query with appropriate date filters and using
QUERY()to get the sum of transactions up to that date.LASTSYNCEDBALANCE Variable
Feedback: Request for access to
lastSyncedBalancein custom pills/formulas.Reason not implemented: Similar to BALANCEONDAY, this would require access to account metadata that isn't available in the current formula execution context. The
balance_currentfield is stored in the accounts table and would need to be passed as a variable to the formula execution context.Alternative approach: This could be added in a future enhancement by extending the variable context passed to formula execution to include account metadata when the formula is executed in an account-specific context.
Dynamic "Last Month" Dropdown
Feedback: Request for dynamic time selection like "last month" in the hamburger menu for time selections.
Reason not in this PR: This is a UI/UX enhancement for the query time frame selector, not related to formula functions. This would be better addressed in a separate PR focused on the query configuration UI.
Testing
Code Comments
Throughout the code, I've added comments explaining which feedback triggered each change:
This makes it easy to trace each change back to the original user feedback.
Related Issue
Addresses feedback items from #5949
Bundle Stats
View detailed bundle stats
desktop-client
Total
Changeset (largest 100 files by percent change)
node_modules/@jlongster/sql.js/dist/sql-wasm.jsnode_modules/google-protobuf/google-protobuf.jshome/runner/work/actual/actual/packages/loot-core/src/server/aql/compiler.tshome/runner/work/actual/actual/packages/loot-core/src/server/db/index.tsnode_modules/adm-zip/adm-zip.jshome/runner/work/actual/actual/packages/crdt/src/proto/sync_pb.jshome/runner/work/actual/actual/packages/loot-core/src/server/sync/index.tsnode_modules/adm-zip/zipEntry.jshome/runner/work/actual/actual/packages/loot-core/src/server/aql/schema/index.tshome/runner/work/actual/actual/packages/loot-core/src/server/budget/envelope.tsnode_modules/adm-zip/zipFile.jsnode_modules/adm-zip/headers/entryHeader.jsnode_modules/adm-zip/util/utils.jshome/runner/work/actual/actual/packages/loot-core/src/server/budget/report.tshome/runner/work/actual/actual/packages/loot-core/src/server/budget/base.tshome/runner/work/actual/actual/packages/loot-core/src/server/aql/schema/executors.tshome/runner/work/actual/actual/packages/crdt/src/crdt/timestamp.tshome/runner/work/actual/actual/packages/loot-core/src/server/aql/schema-helpers.tsnode_modules/adm-zip/methods/zipcrypto.jshome/runner/work/actual/actual/packages/loot-core/src/server/db/sort.tsnode_modules/murmurhash/murmurhash.jshome/runner/work/actual/actual/packages/loot-core/src/platform/server/sqlite/index.tsnode_modules/adm-zip/headers/mainHeader.jshome/runner/work/actual/actual/packages/loot-core/src/server/sync/encoder.tshome/runner/work/actual/actual/packages/loot-core/src/server/models.tsnode_modules/adm-zip/util/errors.jshome/runner/work/actual/actual/packages/loot-core/src/platform/server/fs/path-join.tsnode_modules/adm-zip/util/constants.jshome/runner/work/actual/actual/packages/crdt/src/crdt/merkle.tshome/runner/work/actual/actual/packages/loot-core/src/server/aql/exec.tshome/runner/work/actual/actual/packages/loot-core/src/server/sheet.tshome/runner/work/actual/actual/packages/loot-core/src/platform/server/fs/index.tsnode_modules/adm-zip/util/fattr.jshome/runner/work/actual/actual/packages/loot-core/src/server/app.tshome/runner/work/actual/actual/packages/loot-core/src/server/encryption/encryption-internals.tsnode_modules/adm-zip/methods/inflater.jshome/runner/work/actual/actual/packages/loot-core/src/platform/server/sqlite/unicodeLike.tshome/runner/work/actual/actual/packages/loot-core/src/server/post.tsnode_modules/adm-zip/methods/deflater.jshome/runner/work/actual/actual/packages/loot-core/src/server/undo.tshome/runner/work/actual/actual/packages/loot-core/src/server/prefs.tshome/runner/work/actual/actual/packages/loot-core/src/server/server-config.tshome/runner/work/actual/actual/packages/loot-core/src/server/mutators.tsnode_modules/mitt/dist/mitt.mjshome/runner/work/actual/actual/packages/loot-core/src/platform/server/connection/index.tshome/runner/work/actual/actual/packages/loot-core/src/platform/server/fs/shared.tshome/runner/work/actual/actual/packages/loot-core/src/server/budget/util.tshome/runner/work/actual/actual/packages/loot-core/src/server/encryption/index.tshome/runner/work/actual/actual/packages/loot-core/src/platform/server/fetch/index.tshome/runner/work/actual/actual/packages/crdt/src/index.tsnode_modules/adm-zip/util/index.jshome/runner/work/actual/actual/packages/loot-core/src/server/sync/repair.tsnode_modules/adm-zip/util/decoder.jsnode_modules/adm-zip/methods/index.jshome/runner/work/actual/actual/packages/loot-core/src/server/spreadsheet/globals.tshome/runner/work/actual/actual/packages/loot-core/src/server/aql/index.tsnode_modules/adm-zip/headers/index.jshome/runner/work/actual/actual/packages/loot-core/src/server/db/util.tshome/runner/work/actual/actual/packages/loot-core/src/platform/server/sqlite/normalise.ts__vite-browser-externalhome/runner/work/actual/actual/packages/loot-core/src/server/main-app.tshome/runner/work/actual/actual/packages/loot-core/src/server/spreadsheet/util.tshome/runner/work/actual/actual/packages/loot-core/src/server/cloud-storage.tsnode_modules/absurd-sql/dist/indexeddb-backend.jshome/runner/work/actual/actual/packages/loot-core/src/server/rules/customFunctions.tshome/runner/work/actual/actual/packages/loot-core/src/shared/async.tshome/runner/work/actual/actual/packages/loot-core/src/server/errors.tshome/runner/work/actual/actual/packages/loot-core/src/platform/server/asyncStorage/index.tshome/runner/work/actual/actual/packages/loot-core/src/platform/server/indexeddb/index.tshome/runner/work/actual/actual/packages/loot-core/src/server/rules/action.tssrc/components/formula/transactionModeFunctions.tssrc/components/reports/FormulaResult.tsxhome/runner/work/actual/actual/packages/loot-core/src/shared/util.tssrc/queries/aqlQuery.tssrc/undo/index.tssrc/tags/queries.tsnode_modules/chevrotain/lib_esm/src/parse/grammar/first.jssrc/hooks/useMetadataPref.tssrc/budget/mutations.tssrc/reports/queries.tshome/runner/work/actual/actual/packages/loot-core/src/server/rules/handlebars-helpers.tssrc/hooks/useTransactionBatchActions.tssrc/transactions/queries.tsnode_modules/handlebars/dist/cjs/handlebars/decorators/inline.jsnode_modules/moment/dist/moment.jssrc/components/ServerContext.tsxsrc/budget/queries.tsnode_modules/handlebars/dist/cjs/handlebars/helpers/block-helper-missing.jssrc/hooks/useSpreadsheet.tsxnode_modules/handlebars/dist/cjs/handlebars/logger.jshome/runner/work/actual/actual/packages/loot-core/src/shared/schedules.tsnode_modules/handlebars/dist/cjs/handlebars/helpers/with.jsnode_modules/handlebars/dist/cjs/handlebars/helpers/if.jsnode_modules/source-map/lib/mapping-list.jsnode_modules/handlebars/dist/cjs/handlebars.runtime.jssrc/components/transactions/TransactionList.tsxnode_modules/source-map/lib/array-set.jsnode_modules/chevrotain/lib_esm/src/parse/grammar/follow.jsnode_modules/handlebars/dist/cjs/handlebars/helpers/each.jsnode_modules/handlebars/dist/cjs/handlebars/utils.jsView detailed bundle breakdown
Added
No assets were added
Removed
No assets were removed
Bigger
Smaller
Unchanged
loot-core
Total
Changeset
home/runner/work/actual/actual/packages/loot-core/src/server/rules/customFunctions.tshome/runner/work/actual/actual/packages/loot-core/src/server/rules/action.tsView detailed bundle breakdown
Added
Removed
Bigger
No assets were bigger
Smaller
No assets were smaller
Unchanged
No assets were unchanged
api
Total
Changeset
home/runner/work/actual/actual/packages/loot-core/src/server/rules/customFunctions.tshome/runner/work/actual/actual/packages/loot-core/src/server/rules/action.tsView detailed bundle breakdown
Added
No assets were added
Removed
No assets were removed
Bigger
Smaller
No assets were smaller
Unchanged
No assets were unchanged
cli
Total
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
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.