[PR #7200] [MERGED] [AI] Publish loot-core (@actual-app/core) nightly first in workflow #41435

Closed
opened 2026-04-23 14:13:45 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/7200
Author: @MatissJanis
Created: 3/14/2026
Status: Merged
Merged: 3/15/2026
Merged by: @MatissJanis

Base: masterHead: matiss/publish-loot-core-nightly-first


📝 Commits (8)

  • 2a1794b [AI] Publish loot-core (@actual-app/core) nightly first in workflow
  • 369a3ae [autofix.ci] apply automated fixes
  • a372262 Refactor imports and update configuration
  • d2d23be Merge branch 'matiss/publish-loot-core-nightly-first' of github.com:actualbudget/actual into matiss/publish-loot-core-nightly-first
  • 214306f Merge branch 'master' into matiss/publish-loot-core-nightly-first
  • ca44f16 Add workflow steps to pack and publish the core package nightly
  • 29f1744 Remove nightly tag from npm publish command in workflow for core package
  • ba48b11 Update post-build script comment to reflect correct workspace command for loot-core declarations

📊 Changes

33 files changed (+164 additions, -136 deletions)

View changed files

📝 .github/workflows/publish-nightly-npm-packages.yml (+13 -0)
📝 .github/workflows/publish-npm-packages.yml (+11 -0)
📝 .oxfmtrc.json (+2 -2)
📝 AGENTS.md (+2 -2)
📝 bin/package-browser (+1 -1)
📝 bin/package-electron (+4 -4)
📝 package.json (+4 -4)
📝 packages/api/index.ts (+2 -2)
📝 packages/api/methods.test.ts (+1 -1)
📝 packages/api/methods.ts (+6 -6)
📝 packages/api/package.json (+3 -3)
📝 packages/api/scripts/inline-loot-core-types.mjs (+2 -2)
📝 packages/api/utils.ts (+1 -1)
📝 packages/desktop-client/package.json (+1 -1)
📝 packages/desktop-client/src/globals.ts (+1 -1)
📝 packages/desktop-client/tsconfig.json (+2 -0)
📝 packages/desktop-client/vite.config.mts (+2 -0)
📝 packages/desktop-electron/index.ts (+1 -1)
📝 packages/desktop-electron/package.json (+1 -1)
📝 packages/docs/docs/contributing/development-setup.md (+2 -2)

...and 13 more files

📄 Description

Description

Publish loot-core to the registry. Fixing API package resolution issues.

N/A

Testing

Build and test, etc. should all succeed

Checklist

  • Release notes added (see link above)
  • No obvious regressions in affected areas
  • Self-review has been performed - I understand what each change in the code does and why it is needed

Bundle Stats

Bundle Files count Total bundle size % Changed
desktop-client 27 14.94 MB 0%
loot-core 1 5.76 MB → 5.76 MB (-8 B) -0.00%
api 3 4.7 MB → 4.7 MB (-8 B) -0.00%
View detailed bundle stats

desktop-client

Total

Files count Total bundle size % Changed
27 14.94 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
static/js/index.js 9.8 MB 0%
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 187.85 kB 0%
static/js/da.js 106.13 kB 0%
static/js/de.js 179.82 kB 0%
static/js/en-GB.js 7.18 kB 0%
static/js/en.js 171.21 kB 0%
static/js/es.js 174.29 kB 0%
static/js/fr.js 179.34 kB 0%
static/js/it.js 170.91 kB 0%
static/js/nb-NO.js 156.74 kB 0%
static/js/nl.js 113.03 kB 0%
static/js/pl.js 89.61 kB 0%
static/js/pt-BR.js 182.82 kB 0%
static/js/th.js 181.54 kB 0%
static/js/uk.js 215.25 kB 0%
static/js/resize-observer.js 18.37 kB 0%
static/js/BackgroundImage.js 120.54 kB 0%
static/js/ReportRouter.js 1.16 MB 0%
static/js/narrow.js 423.11 kB 0%
static/js/TransactionList.js 106.52 kB 0%
static/js/wide.js 164.37 kB 0%
static/js/AppliedFilters.js 9.99 kB 0%
static/js/usePayeeRuleCounts.js 11.57 kB 0%
static/js/useTransactionBatchActions.js 13.24 kB 0%
static/js/FormulaEditor.js 1.04 MB 0%

loot-core

Total

Files count Total bundle size % Changed
1 5.76 MB → 5.76 MB (-8 B) -0.00%
Changeset
File Δ Size
home/runner/work/actual/actual/packages/loot-core/src/server/payees/app.ts 📉 -8 B (-0.11%) 7.08 kB → 7.07 kB
View detailed bundle breakdown

Added

Asset File Size % Changed
kcab.worker.d3q7AChe.js 0 B → 5.76 MB (+5.76 MB) -

Removed

Asset File Size % Changed
kcab.worker.DLyaij02.js 5.76 MB → 0 B (-5.76 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
3 4.7 MB → 4.7 MB (-8 B) -0.00%
Changeset
File Δ Size
home/runner/work/actual/actual/packages/loot-core/src/server/payees/app.ts 📉 -8 B (-0.12%) 6.41 kB → 6.4 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
index.js 4.41 MB → 4.41 MB (-8 B) -0.00%

Unchanged

Asset File Size % Changed
index-BKGP2w5F.js 285.56 kB 0%
multipart-parser-DwddZ4BH.js 9.33 kB 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/7200 **Author:** [@MatissJanis](https://github.com/MatissJanis) **Created:** 3/14/2026 **Status:** ✅ Merged **Merged:** 3/15/2026 **Merged by:** [@MatissJanis](https://github.com/MatissJanis) **Base:** `master` ← **Head:** `matiss/publish-loot-core-nightly-first` --- ### 📝 Commits (8) - [`2a1794b`](https://github.com/actualbudget/actual/commit/2a1794b655fe58e06a6a7fce2d76e46b81a98477) [AI] Publish loot-core (@actual-app/core) nightly first in workflow - [`369a3ae`](https://github.com/actualbudget/actual/commit/369a3aee5e96b4f20b7e568e5e678ea0265874a8) [autofix.ci] apply automated fixes - [`a372262`](https://github.com/actualbudget/actual/commit/a372262187ab13c5333e8890e6069c55795847f7) Refactor imports and update configuration - [`d2d23be`](https://github.com/actualbudget/actual/commit/d2d23bed3e17e402ad5864d0aa676e4318721585) Merge branch 'matiss/publish-loot-core-nightly-first' of github.com:actualbudget/actual into matiss/publish-loot-core-nightly-first - [`214306f`](https://github.com/actualbudget/actual/commit/214306fdcbe50394d0524c399dd6837c65f3e165) Merge branch 'master' into matiss/publish-loot-core-nightly-first - [`ca44f16`](https://github.com/actualbudget/actual/commit/ca44f165f6a92c40a4f4e4538fac1bf3add49ba5) Add workflow steps to pack and publish the core package nightly - [`29f1744`](https://github.com/actualbudget/actual/commit/29f1744ed81069d039b2b1bf23d87ed29c189b7b) Remove nightly tag from npm publish command in workflow for core package - [`ba48b11`](https://github.com/actualbudget/actual/commit/ba48b118924e65d6ba466e630226123fd3cbeec1) Update post-build script comment to reflect correct workspace command for loot-core declarations ### 📊 Changes **33 files changed** (+164 additions, -136 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/publish-nightly-npm-packages.yml` (+13 -0) 📝 `.github/workflows/publish-npm-packages.yml` (+11 -0) 📝 `.oxfmtrc.json` (+2 -2) 📝 `AGENTS.md` (+2 -2) 📝 `bin/package-browser` (+1 -1) 📝 `bin/package-electron` (+4 -4) 📝 `package.json` (+4 -4) 📝 `packages/api/index.ts` (+2 -2) 📝 `packages/api/methods.test.ts` (+1 -1) 📝 `packages/api/methods.ts` (+6 -6) 📝 `packages/api/package.json` (+3 -3) 📝 `packages/api/scripts/inline-loot-core-types.mjs` (+2 -2) 📝 `packages/api/utils.ts` (+1 -1) 📝 `packages/desktop-client/package.json` (+1 -1) 📝 `packages/desktop-client/src/globals.ts` (+1 -1) 📝 `packages/desktop-client/tsconfig.json` (+2 -0) 📝 `packages/desktop-client/vite.config.mts` (+2 -0) 📝 `packages/desktop-electron/index.ts` (+1 -1) 📝 `packages/desktop-electron/package.json` (+1 -1) 📝 `packages/docs/docs/contributing/development-setup.md` (+2 -2) _...and 13 more files_ </details> ### 📄 Description <!-- Thank you for submitting a pull request! Make sure to follow the instructions to write release notes for your PR — it should only take a minute or two: https://github.com/actualbudget/docs#writing-good-release-notes. Try running yarn generate:release-notes *before* pushing your PR for an interactive experience. --> ## Description <!-- What does this PR do? Why is it needed? Please give context on the "why?": why do we need this change? What problem is it solving for you?--> Publish loot-core to the registry. Fixing API package resolution issues. ## Related issue(s) <!-- e.g. Fixes #123, Relates to #456 --> N/A ## Testing <!-- What did you test? How can we reproduce the issue you are fixing or how can we test the feature you built? --> Build and test, etc. should all succeed ## Checklist - [x] Release notes added (see link above) - [x] No obvious regressions in affected areas - [x] Self-review has been performed - I understand what each change in the code does and why it is needed <!--- actual-bot-sections ---> <!--- bundlestats-action-comment key:combined start ---> ### Bundle Stats Bundle | Files count | Total bundle size | % Changed ------ | ----------- | ----------------- | --------- desktop-client | 27 | 14.94 MB | 0% loot-core | 1 | 5.76 MB → 5.76 MB (-8 B) | -0.00% api | 3 | 4.7 MB → 4.7 MB (-8 B) | -0.00% <details> <summary>View detailed bundle stats</summary> #### desktop-client **Total** Files count | Total bundle size | % Changed ----------- | ----------------- | --------- 27 | 14.94 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 ----- | --------- | --------- static/js/index.js | 9.8 MB | 0% 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 | 187.85 kB | 0% static/js/da.js | 106.13 kB | 0% static/js/de.js | 179.82 kB | 0% static/js/en-GB.js | 7.18 kB | 0% static/js/en.js | 171.21 kB | 0% static/js/es.js | 174.29 kB | 0% static/js/fr.js | 179.34 kB | 0% static/js/it.js | 170.91 kB | 0% static/js/nb-NO.js | 156.74 kB | 0% static/js/nl.js | 113.03 kB | 0% static/js/pl.js | 89.61 kB | 0% static/js/pt-BR.js | 182.82 kB | 0% static/js/th.js | 181.54 kB | 0% static/js/uk.js | 215.25 kB | 0% static/js/resize-observer.js | 18.37 kB | 0% static/js/BackgroundImage.js | 120.54 kB | 0% static/js/ReportRouter.js | 1.16 MB | 0% static/js/narrow.js | 423.11 kB | 0% static/js/TransactionList.js | 106.52 kB | 0% static/js/wide.js | 164.37 kB | 0% static/js/AppliedFilters.js | 9.99 kB | 0% static/js/usePayeeRuleCounts.js | 11.57 kB | 0% static/js/useTransactionBatchActions.js | 13.24 kB | 0% static/js/FormulaEditor.js | 1.04 MB | 0% </div> </details> --- #### loot-core **Total** Files count | Total bundle size | % Changed ----------- | ----------------- | --------- 1 | 5.76 MB → 5.76 MB (-8 B) | -0.00% <details> <summary>Changeset</summary> File | Δ | Size ---- | - | ---- `home/runner/work/actual/actual/packages/loot-core/src/server/payees/app.ts` | 📉 -8 B (-0.11%) | 7.08 kB → 7.07 kB </details> <details> <summary>View detailed bundle breakdown</summary> <div> **Added** Asset | File Size | % Changed ----- | --------- | --------- kcab.worker.d3q7AChe.js | 0 B → 5.76 MB (+5.76 MB) | - **Removed** Asset | File Size | % Changed ----- | --------- | --------- kcab.worker.DLyaij02.js | 5.76 MB → 0 B (-5.76 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 ----------- | ----------------- | --------- 3 | 4.7 MB → 4.7 MB (-8 B) | -0.00% <details> <summary>Changeset</summary> File | Δ | Size ---- | - | ---- `home/runner/work/actual/actual/packages/loot-core/src/server/payees/app.ts` | 📉 -8 B (-0.12%) | 6.41 kB → 6.4 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 ----- | --------- | --------- index.js | 4.41 MB → 4.41 MB (-8 B) | -0.00% **Unchanged** Asset | File Size | % Changed ----- | --------- | --------- index-BKGP2w5F.js | 285.56 kB | 0% multipart-parser-DwddZ4BH.js | 9.33 kB | 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-04-23 14:13:45 -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#41435