[PR #6179] [MERGED] Add ESM loader support and update sync-server modules #68742

Closed
opened 2026-05-14 03:21:45 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/6179
Author: @MatissJanis
Created: 11/17/2025
Status: Merged
Merged: 12/6/2025
Merged by: @MatissJanis

Base: masterHead: matiss/sync-server-loader-changes


📝 Commits (10+)

  • fe2d3e6 Add ESM loader support and update sync-server modules
  • 7b9e567 Update TypeScript configuration and fix bank file import filter in sync-server
  • f2421ef Merge branch 'master' into matiss/sync-server-loader-changes
  • 140aee1 Remove deprecated loader and register files, update TypeScript configuration to use ES2021, and add a new script for automatically adding import extensions to JavaScript files.
  • 63258d3 Update test script in package.json to include a custom loader and clean up import extensions script by removing unused 'stat' import.
  • 1f7aa2f Merge branch 'master' into matiss/sync-server-loader-changes
  • 1e60ac0 feat: Add warning for unresolved imports
  • 7ad6c63 [autofix.ci] apply automated fixes
  • 9452297 Remove unused 'import/extensions' rule from ESLint configuration
  • ea738c4 Merge branch 'matiss/sync-server-loader-changes' of github.com:actualbudget/actual into matiss/sync-server-loader-changes

📊 Changes

129 files changed (+464 additions, -333 deletions)

View changed files

📝 eslint.config.mjs (+0 -1)
📝 packages/sync-server/app.ts (+1 -1)
📝 packages/sync-server/bin/actual-server.js (+1 -0)
packages/sync-server/bin/add-import-extensions.mjs (+102 -0)
packages/sync-server/loader.mjs (+26 -0)
📝 packages/sync-server/migrations/1694360000000-create-folders.js (+1 -1)
📝 packages/sync-server/migrations/1694360479680-create-account-db.js (+1 -1)
📝 packages/sync-server/migrations/1694362247011-create-secret-table.js (+1 -1)
📝 packages/sync-server/migrations/1702667624000-rename-nordigen-secrets.js (+1 -1)
📝 packages/sync-server/migrations/1718889148000-openid.js (+1 -1)
📝 packages/sync-server/migrations/1719409568000-multiuser.js (+1 -1)
📝 packages/sync-server/package.json (+4 -3)
packages/sync-server/register-loader.mjs (+9 -0)
📝 packages/sync-server/src/account-db.js (+4 -4)
📝 packages/sync-server/src/accounts/openid.js (+5 -5)
📝 packages/sync-server/src/accounts/password.js (+3 -3)
📝 packages/sync-server/src/app-account.js (+5 -8)
📝 packages/sync-server/src/app-admin.js (+4 -4)
📝 packages/sync-server/src/app-admin.test.js (+2 -2)
📝 packages/sync-server/src/app-cors-proxy.js (+3 -3)

...and 80 more files

📄 Description

Alternative to https://github.com/actualbudget/actual/pull/6168#issuecomment-3543750315

Should do the trick, but extensive testing recommended


🔄 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/6179 **Author:** [@MatissJanis](https://github.com/MatissJanis) **Created:** 11/17/2025 **Status:** ✅ Merged **Merged:** 12/6/2025 **Merged by:** [@MatissJanis](https://github.com/MatissJanis) **Base:** `master` ← **Head:** `matiss/sync-server-loader-changes` --- ### 📝 Commits (10+) - [`fe2d3e6`](https://github.com/actualbudget/actual/commit/fe2d3e69aa316a9821c8ad1f928334ba07e08aef) Add ESM loader support and update sync-server modules - [`7b9e567`](https://github.com/actualbudget/actual/commit/7b9e56764f5e0e43142c7af529e6c37b9c5d315c) Update TypeScript configuration and fix bank file import filter in sync-server - [`f2421ef`](https://github.com/actualbudget/actual/commit/f2421ef4d57475ab5b10cfc076954073e5e75171) Merge branch 'master' into matiss/sync-server-loader-changes - [`140aee1`](https://github.com/actualbudget/actual/commit/140aee169406dd488a625223553d2666e6badede) Remove deprecated loader and register files, update TypeScript configuration to use ES2021, and add a new script for automatically adding import extensions to JavaScript files. - [`63258d3`](https://github.com/actualbudget/actual/commit/63258d3f88f60ac4fb651e6ece89fc9e25066522) Update test script in package.json to include a custom loader and clean up import extensions script by removing unused 'stat' import. - [`1f7aa2f`](https://github.com/actualbudget/actual/commit/1f7aa2f394ce7abc30908178a145153e672dd91e) Merge branch 'master' into matiss/sync-server-loader-changes - [`1e60ac0`](https://github.com/actualbudget/actual/commit/1e60ac0e0514db3e6a38fe0e40ca37b46c509df8) feat: Add warning for unresolved imports - [`7ad6c63`](https://github.com/actualbudget/actual/commit/7ad6c63c17b782d62ce329c4391623b7f8c9062b) [autofix.ci] apply automated fixes - [`9452297`](https://github.com/actualbudget/actual/commit/9452297ccb81007fa7d0fe8fa16ef36c61ef81fd) Remove unused 'import/extensions' rule from ESLint configuration - [`ea738c4`](https://github.com/actualbudget/actual/commit/ea738c4cf7ea1ea4f7915cbd678afc4be400ef6b) Merge branch 'matiss/sync-server-loader-changes' of github.com:actualbudget/actual into matiss/sync-server-loader-changes ### 📊 Changes **129 files changed** (+464 additions, -333 deletions) <details> <summary>View changed files</summary> 📝 `eslint.config.mjs` (+0 -1) 📝 `packages/sync-server/app.ts` (+1 -1) 📝 `packages/sync-server/bin/actual-server.js` (+1 -0) ➕ `packages/sync-server/bin/add-import-extensions.mjs` (+102 -0) ➕ `packages/sync-server/loader.mjs` (+26 -0) 📝 `packages/sync-server/migrations/1694360000000-create-folders.js` (+1 -1) 📝 `packages/sync-server/migrations/1694360479680-create-account-db.js` (+1 -1) 📝 `packages/sync-server/migrations/1694362247011-create-secret-table.js` (+1 -1) 📝 `packages/sync-server/migrations/1702667624000-rename-nordigen-secrets.js` (+1 -1) 📝 `packages/sync-server/migrations/1718889148000-openid.js` (+1 -1) 📝 `packages/sync-server/migrations/1719409568000-multiuser.js` (+1 -1) 📝 `packages/sync-server/package.json` (+4 -3) ➕ `packages/sync-server/register-loader.mjs` (+9 -0) 📝 `packages/sync-server/src/account-db.js` (+4 -4) 📝 `packages/sync-server/src/accounts/openid.js` (+5 -5) 📝 `packages/sync-server/src/accounts/password.js` (+3 -3) 📝 `packages/sync-server/src/app-account.js` (+5 -8) 📝 `packages/sync-server/src/app-admin.js` (+4 -4) 📝 `packages/sync-server/src/app-admin.test.js` (+2 -2) 📝 `packages/sync-server/src/app-cors-proxy.js` (+3 -3) _...and 80 more files_ </details> ### 📄 Description Alternative to https://github.com/actualbudget/actual/pull/6168#issuecomment-3543750315 Should do the trick, but extensive testing recommended --- <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-14 03:21: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#68742