[PR #6690] [MERGED] Implement missing logic for limit template type #55976

Closed
opened 2026-05-01 03:26:03 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/6690
Author: @jfdoming
Created: 1/18/2026
Status: Merged
Merged: 2/14/2026
Merged by: @jfdoming

Base: masterHead: jfdoming/core-limit-refill


📝 Commits (10+)

📊 Changes

8 files changed (+185 additions, -6 deletions)

View changed files

📝 packages/desktop-client/src/components/budget/goals/reducer.ts (+2 -0)
📝 packages/loot-core/src/server/budget/category-template-context.test.ts (+99 -0)
📝 packages/loot-core/src/server/budget/category-template-context.ts (+14 -1)
📝 packages/loot-core/src/server/budget/goal-template.pegjs (+1 -1)
📝 packages/loot-core/src/server/budget/template-notes.test.ts (+37 -0)
📝 packages/loot-core/src/server/budget/template-notes.ts (+21 -4)
📝 packages/loot-core/src/types/models/templates.ts (+5 -0)
upcoming-release-notes/6690.md (+6 -0)

📄 Description

Few things missing:

  1. Support for refill mode.
  2. As a necessary extension of above, support for priorities.
  3. Unparsing to template text.

Bundle Stats

Bundle Files count Total bundle size % Changed
desktop-client 28 14.78 MB → 14.78 MB (+73 B) +0.00%
loot-core 1 5.86 MB → 5.86 MB (+994 B) +0.02%
api 1 4.4 MB → 4.4 MB (+902 B) +0.02%
View detailed bundle stats

desktop-client

Total

Files count Total bundle size % Changed
28 14.78 MB → 14.78 MB (+73 B) +0.00%
Changeset
File Δ Size
src/components/budget/goals/reducer.ts 📈 +73 B (+1.53%) 4.65 kB → 4.72 kB
View detailed bundle breakdown

Added
No assets were added

Removed
No assets were removed

Bigger

Asset File Size % Changed
static/js/index.js 9.51 MB → 9.52 MB (+73 B) +0.00%

Smaller
No assets were smaller

Unchanged

Asset File Size % Changed
static/js/indexeddb-main-thread-worker-e59fee74.js 12.94 kB 0%
static/js/workbox-window.prod.es5.js 5.64 kB 0%
static/js/ca.js 96.92 kB 0%
static/js/da.js 106.62 kB 0%
static/js/de.js 180.44 kB 0%
static/js/en-GB.js 7.18 kB 0%
static/js/en.js 165.58 kB 0%
static/js/es.js 173.83 kB 0%
static/js/fr.js 179.97 kB 0%
static/js/it.js 171.44 kB 0%
static/js/nb-NO.js 157.23 kB 0%
static/js/nl.js 106.65 kB 0%
static/js/pl.js 88.64 kB 0%
static/js/pt-BR.js 154.57 kB 0%
static/js/sv.js 78.2 kB 0%
static/js/th.js 182.35 kB 0%
static/js/uk.js 215.11 kB 0%
static/js/resize-observer.js 18.37 kB 0%
static/js/BackgroundImage.js 120.54 kB 0%
static/js/ReportRouter.js 1.13 MB 0%
static/js/narrow.js 638.75 kB 0%
static/js/TransactionList.js 106.13 kB 0%
static/js/wide.js 165.25 kB 0%
static/js/AppliedFilters.js 9.71 kB 0%
static/js/usePayeeRuleCounts.js 10.05 kB 0%
static/js/useTransactionBatchActions.js 13.23 kB 0%
static/js/FormulaEditor.js 1.04 MB 0%

loot-core

Total

Files count Total bundle size % Changed
1 5.86 MB → 5.86 MB (+994 B) +0.02%
Changeset
File Δ Size
home/runner/work/actual/actual/packages/loot-core/src/server/budget/template-notes.ts 📈 +578 B (+7.38%) 7.65 kB → 8.22 kB
home/runner/work/actual/actual/packages/loot-core/src/shared/months.ts 📈 +97 B (+2.44%) 3.88 kB → 3.97 kB
home/runner/work/actual/actual/packages/loot-core/src/server/budget/category-template-context.ts 📈 +319 B (+1.34%) 23.18 kB → 23.49 kB
View detailed bundle breakdown

Added

Asset File Size % Changed
kcab.worker.92NpqIpU.js 0 B → 5.86 MB (+5.86 MB) -

Removed

Asset File Size % Changed
kcab.worker.B2ZGg82M.js 5.86 MB → 0 B (-5.86 MB) -100%

Bigger
No assets were bigger

Smaller
No assets were smaller

Unchanged
No assets were unchanged


api

Total

Files count Total bundle size % Changed
1 4.4 MB → 4.4 MB (+902 B) +0.02%
Changeset
File Δ Size
src/server/budget/template-notes.ts 📈 +526 B (+7.53%) 6.82 kB → 7.34 kB
src/shared/months.ts 📈 +85 B (+2.45%) 3.39 kB → 3.47 kB
src/server/budget/category-template-context.ts 📈 +291 B (+1.37%) 20.71 kB → 20.99 kB
View detailed bundle breakdown

Added
No assets were added

Removed
No assets were removed

Bigger

Asset File Size % Changed
bundle.api.js 4.4 MB → 4.4 MB (+902 B) +0.02%

Smaller
No assets were smaller

Unchanged
No assets were unchanged


🔄 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/6690 **Author:** [@jfdoming](https://github.com/jfdoming) **Created:** 1/18/2026 **Status:** ✅ Merged **Merged:** 2/14/2026 **Merged by:** [@jfdoming](https://github.com/jfdoming) **Base:** `master` ← **Head:** `jfdoming/core-limit-refill` --- ### 📝 Commits (10+) - [`af5a065`](https://github.com/actualbudget/actual/commit/af5a065826b1dc57bffbccffe09308fbc63d88e8) core: support limit refill templates - [`882cb09`](https://github.com/actualbudget/actual/commit/882cb096fccf6cdb5a027e159b6ebaf0af9e71d1) notes: refill templates - [`5bf11fd`](https://github.com/actualbudget/actual/commit/5bf11fd1b404f4671150e90c8031d5302db252c5) core: apply refill limits during runs - [`487fe20`](https://github.com/actualbudget/actual/commit/487fe200b2c4461cbf679e8456f312dc9cecd236) core: prioritize refill limits - [`74299da`](https://github.com/actualbudget/actual/commit/74299da99c41e4fe20b7ee96cce7fa60a496cd71) Patch - [`5611438`](https://github.com/actualbudget/actual/commit/5611438ad0576ce343764a3036c8b680b66b52a2) Update release note - [`b5cc902`](https://github.com/actualbudget/actual/commit/b5cc90212a72f5d4be2fec9d905264b2f1d48b5a) Fix typecheck - [`25405d7`](https://github.com/actualbudget/actual/commit/25405d746ccca41e4a75927356f18df5d7a52535) rework. Tests and template notes still need reworked - [`be4f7ea`](https://github.com/actualbudget/actual/commit/be4f7ea0156a6546a23cdc691c61ec40d2e59442) fix parser syntax - [`cf3d673`](https://github.com/actualbudget/actual/commit/cf3d673553f1be5b41212405a65a7329af2121d4) Fix type issue ### 📊 Changes **8 files changed** (+185 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/budget/goals/reducer.ts` (+2 -0) 📝 `packages/loot-core/src/server/budget/category-template-context.test.ts` (+99 -0) 📝 `packages/loot-core/src/server/budget/category-template-context.ts` (+14 -1) 📝 `packages/loot-core/src/server/budget/goal-template.pegjs` (+1 -1) 📝 `packages/loot-core/src/server/budget/template-notes.test.ts` (+37 -0) 📝 `packages/loot-core/src/server/budget/template-notes.ts` (+21 -4) 📝 `packages/loot-core/src/types/models/templates.ts` (+5 -0) ➕ `upcoming-release-notes/6690.md` (+6 -0) </details> ### 📄 Description Few things missing: 1. Support for refill mode. 2. As a necessary extension of above, support for priorities. 3. Unparsing to template text. <!--- actual-bot-sections ---> <hr /> <!--- bundlestats-action-comment key:combined start ---> ### Bundle Stats Bundle | Files count | Total bundle size | % Changed ------ | ----------- | ----------------- | --------- desktop-client | 28 | 14.78 MB → 14.78 MB (+73 B) | +0.00% loot-core | 1 | 5.86 MB → 5.86 MB (+994 B) | +0.02% api | 1 | 4.4 MB → 4.4 MB (+902 B) | +0.02% <details> <summary>View detailed bundle stats</summary> #### desktop-client **Total** Files count | Total bundle size | % Changed ----------- | ----------------- | --------- 28 | 14.78 MB → 14.78 MB (+73 B) | +0.00% <details> <summary>Changeset</summary> File | Δ | Size ---- | - | ---- `src/components/budget/goals/reducer.ts` | 📈 +73 B (+1.53%) | 4.65 kB → 4.72 kB </details> <details> <summary>View detailed bundle breakdown</summary> <div> **Added** No assets were added **Removed** No assets were removed **Bigger** Asset | File Size | % Changed ----- | --------- | --------- static/js/index.js | 9.51 MB → 9.52 MB (+73 B) | +0.00% **Smaller** No assets were smaller **Unchanged** Asset | File Size | % Changed ----- | --------- | --------- static/js/indexeddb-main-thread-worker-e59fee74.js | 12.94 kB | 0% static/js/workbox-window.prod.es5.js | 5.64 kB | 0% static/js/ca.js | 96.92 kB | 0% static/js/da.js | 106.62 kB | 0% static/js/de.js | 180.44 kB | 0% static/js/en-GB.js | 7.18 kB | 0% static/js/en.js | 165.58 kB | 0% static/js/es.js | 173.83 kB | 0% static/js/fr.js | 179.97 kB | 0% static/js/it.js | 171.44 kB | 0% static/js/nb-NO.js | 157.23 kB | 0% static/js/nl.js | 106.65 kB | 0% static/js/pl.js | 88.64 kB | 0% static/js/pt-BR.js | 154.57 kB | 0% static/js/sv.js | 78.2 kB | 0% static/js/th.js | 182.35 kB | 0% static/js/uk.js | 215.11 kB | 0% static/js/resize-observer.js | 18.37 kB | 0% static/js/BackgroundImage.js | 120.54 kB | 0% static/js/ReportRouter.js | 1.13 MB | 0% static/js/narrow.js | 638.75 kB | 0% static/js/TransactionList.js | 106.13 kB | 0% static/js/wide.js | 165.25 kB | 0% static/js/AppliedFilters.js | 9.71 kB | 0% static/js/usePayeeRuleCounts.js | 10.05 kB | 0% static/js/useTransactionBatchActions.js | 13.23 kB | 0% static/js/FormulaEditor.js | 1.04 MB | 0% </div> </details> --- #### loot-core **Total** Files count | Total bundle size | % Changed ----------- | ----------------- | --------- 1 | 5.86 MB → 5.86 MB (+994 B) | +0.02% <details> <summary>Changeset</summary> File | Δ | Size ---- | - | ---- `home/runner/work/actual/actual/packages/loot-core/src/server/budget/template-notes.ts` | 📈 +578 B (+7.38%) | 7.65 kB → 8.22 kB `home/runner/work/actual/actual/packages/loot-core/src/shared/months.ts` | 📈 +97 B (+2.44%) | 3.88 kB → 3.97 kB `home/runner/work/actual/actual/packages/loot-core/src/server/budget/category-template-context.ts` | 📈 +319 B (+1.34%) | 23.18 kB → 23.49 kB </details> <details> <summary>View detailed bundle breakdown</summary> <div> **Added** Asset | File Size | % Changed ----- | --------- | --------- kcab.worker.92NpqIpU.js | 0 B → 5.86 MB (+5.86 MB) | - **Removed** Asset | File Size | % Changed ----- | --------- | --------- kcab.worker.B2ZGg82M.js | 5.86 MB → 0 B (-5.86 MB) | -100% **Bigger** No assets were bigger **Smaller** No assets were smaller **Unchanged** No assets were unchanged </div> </details> --- #### api **Total** Files count | Total bundle size | % Changed ----------- | ----------------- | --------- 1 | 4.4 MB → 4.4 MB (+902 B) | +0.02% <details> <summary>Changeset</summary> File | Δ | Size ---- | - | ---- `src/server/budget/template-notes.ts` | 📈 +526 B (+7.53%) | 6.82 kB → 7.34 kB `src/shared/months.ts` | 📈 +85 B (+2.45%) | 3.39 kB → 3.47 kB `src/server/budget/category-template-context.ts` | 📈 +291 B (+1.37%) | 20.71 kB → 20.99 kB </details> <details> <summary>View detailed bundle breakdown</summary> <div> **Added** No assets were added **Removed** No assets were removed **Bigger** Asset | File Size | % Changed ----- | --------- | --------- bundle.api.js | 4.4 MB → 4.4 MB (+902 B) | +0.02% **Smaller** No assets were smaller **Unchanged** No assets were unchanged </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 03:26:03 -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#55976