Skip to content

[WIP][AI] Show entity names in rule filter pills instead of UUIDs (#7123)#7321

Open
pranayseela wants to merge 4 commits intoactualbudget:masterfrom
pranayseela:fix/7123-rule-filter-pill-names
Open

[WIP][AI] Show entity names in rule filter pills instead of UUIDs (#7123)#7321
pranayseela wants to merge 4 commits intoactualbudget:masterfrom
pranayseela:fix/7123-rule-filter-pill-names

Conversation

@pranayseela
Copy link
Copy Markdown
Contributor

@pranayseela pranayseela commented Mar 29, 2026

Fixes #7123

Made with Cursor


Bundle Stats

Bundle Files count Total bundle size % Changed
desktop-client 28 12.17 MB → 12.17 MB (+5.34 kB) +0.04%
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
28 12.17 MB → 12.17 MB (+5.34 kB) +0.04%
Changeset
File Δ Size
src/components/rules/Value.tsx 📈 +352 B (+8.07%) 4.26 kB → 4.6 kB
locale/zh-Hans.json 📈 +4.99 kB (+6.14%) 81.33 kB → 86.32 kB
View detailed bundle breakdown

Added
No assets were added

Removed
No assets were removed

Bigger

Asset File Size % Changed
static/js/zh-Hans.js 81.33 kB → 86.32 kB (+4.99 kB) +6.14%
static/js/useTransactionBatchActions.js 4.29 MB → 4.29 MB (+352 B) +0.01%

Smaller
No assets were smaller

Unchanged

Asset File Size % Changed
static/js/index.js 3.23 MB 0%
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.18 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.5 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%

…albudget#7123)

Resolve payee/category/account IDs when displaying rule condition values
with valueIsRaw (contains / matches / does not contain).

Made-with: Cursor
@actual-github-bot actual-github-bot bot changed the title [AI] Show entity names in rule filter pills instead of UUIDs (#7123) [WIP] [AI] Show entity names in rule filter pills instead of UUIDs (#7123) Mar 29, 2026
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 29, 2026

Deploy Preview for actualbudget ready!

Name Link
🔨 Latest commit 776e038
🔍 Latest deploy log https://app.netlify.com/projects/actualbudget/deploys/69ce45d0c16e810007b75b61
😎 Deploy Preview https://deploy-preview-7321.demo.actualbudget.org
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions
Copy link
Copy Markdown
Contributor

👋 Hello contributor!

We would love to review your PR! Before we can do that, please make sure:

  • ✅ All CI checks pass
  • ✅ The PR is moved from draft to open (if applicable)
  • ✅ The "[WIP]" prefix is removed from the PR title
  • ✅ All CodeRabbit code review comments are resolved (if you disagree with anything - reply to the bot with your reasoning so we can read through it). The bot will eventually approve the PR.

We do this to reduce the TOIL the core contributor team has to go through for each PR and to allow for speedy reviews and merges.

For more information, please see our Contributing Guide.

@pranayseela pranayseela changed the title [WIP] [AI] Show entity names in rule filter pills instead of UUIDs (#7123) [AI] Show entity names in rule filter pills instead of UUIDs (#7123) Mar 29, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 29, 2026

📝 Walkthrough

Walkthrough

Adds a UUID-detection helper and updates the Value component to, when given raw UUID-like values and matching data arrays, resolve payee/category/category_group/account/rule IDs to their human-readable descriptions; also adds a release note documenting the bugfix.

Changes

Cohort / File(s) Summary
Value Component Enhancement
packages/desktop-client/src/components/rules/Value.tsx
Added local isLikelyEntityId(value: unknown): value is string helper and updated formatValue to detect UUID-like raw values for fields payee, category, category_group, account, and rule; if data is an array, attempts to find item.id === value and return describe(item) when found, otherwise returns the raw value.
Release Notes
upcoming-release-notes/7321.md
Added a Bugfix release note clarifying that rule filter pills now show human-readable names instead of raw UUIDs for operators like contains, matches, and does not contain.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Suggested labels

size small

Poem

🐰 A UUID once hid the name so bright,
Now I hop and fetch it into light.
Pills show names, not cryptic keys,
Users smile with simpler ease. 🥕

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The code changes implement the fix for issue #7123 by adding UUID validation and entity name resolution for rule filter pills when operators use raw values.
Out of Scope Changes check ✅ Passed All changes (Value.tsx logic and release notes) are directly related to fixing the UUID display issue in rule filter pills for issue #7123.
Description check ✅ Passed The PR description references issue #7123 and mentions the changes made, linking the pull request to the underlying objective.
Title check ✅ Passed The title clearly summarizes the main change: displaying entity names instead of UUIDs in rule filter pills, which directly addresses the PR's core objective.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

coderabbitai[bot]
coderabbitai bot previously approved these changes Mar 29, 2026
@pranayseela
Copy link
Copy Markdown
Contributor Author

For reviewers

@coderabbitai coderabbitai bot removed the size small label Mar 29, 2026
@matt-fidd
Copy link
Copy Markdown
Member

This fix seems unrelated to the issue you've linked, and does not fix it.

@matt-fidd matt-fidd force-pushed the master branch 2 times, most recently from 5c7c70d to d262f7d Compare April 5, 2026 17:13
@youngcw youngcw changed the title [AI] Show entity names in rule filter pills instead of UUIDs (#7123) [WIP][AI] Show entity names in rule filter pills instead of UUIDs (#7123) Apr 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Changing Category Group from is to contains replaces budget with the raw UUID

2 participants