mirror of
https://github.com/actualbudget/actual.git
synced 2026-03-10 04:02:38 -05:00
[Bug]: docker does not build properly when developing on Mac #2805
Closed
opened 2026-02-28 20:29:03 -06:00 by GiteaMirror
·
4 comments
No Branch/Tag Specified
master
claude/fix-simplefin-batch-sync-O8LcD
matiss/modal-and-server-refactor
claude/fix-simplefin-ssrf-T31gX
claude/release-notes-validation-X7rvR
ai/custom-theme-dual-prefs
matiss/fix-6804
add-claude-github-actions-1772738270730
react-query-rules
react-query-useSchedules
matiss/separate-lint-format
dependabot/npm_and_yarn/ajv-6.14.0
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.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
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#2805
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?
Originally created by @RMcGhee on GitHub (Jan 11, 2026).
Verified issue does not already exist?
What happened?
When developing on a Mac, setting up the development env requires yarn to install various tools for type checking, linting, etc, and if you do that, node_modules is populated at various levels (root, desktop-client, etc). That works ok.
However, when you try to run yarn vrt:docker, the docker build will fail because of the docker compose, which copies everything in the root directory to /app, including the Mac specific node_modules. The fix for this is to delete node_modules at various levels, then rebuild with no cache, but then the next time you try to yarn typecheck on the Mac side, it will fail because it finds the linux version of the node_modules. Going back and forth like this is pretty annoying.
It may be that I'm missing some part of the dev env setup that would ameliorate this issue, but I think we're simply missing a .dockerignore that would prevent the copying of node_modules. I can create the PR for this if it's not the case that I'm missing a step in env setup.
How can we reproduce the issue?
Develop on a Mac. This is likely replicable on a linux distro that's different from the docker container, and windows.
Start with a clean repo and docker env (no docker built yet)
yarn install at the repo root.
test with yarn typecheck.
cd to packages/desktop-client
Try to build and start the docker env (HTTPS=true docker compose up --build).
It should fail with a node_modules error.
Delete node_modules at the repo root.
HTTPS=true docker compose up --build at the repo root.
The docker container builds and runs correctly.
Where are you hosting Actual?
Docker
What browsers are you seeing the problem on?
No response
Operating System
Mac OSX
@MatissJanis commented on GitHub (Jan 11, 2026):
The problem is that you are doing
cd packages/someplaceto run commands.Either run the commands available in root or run
yarn workspace @actual-app/web run some:child:command@RMcGhee commented on GitHub (Jan 11, 2026):
Appreciate the reply. Following the instructions here https://actualbudget.org/docs/contributing/development-setup/ and here: https://github.com/actualbudget/actual/tree/master/packages/desktop-client#visual-regression
Started over from a clean repo to test your suggestion, without changing directories from the root repo:
Sorry if I'm missing something obvious, but following the docs as closely as possible, and with your suggestion, I still have this issue. yarn start works fine, but the vrt cannot be done on a local/dev instance as the screenshots have a -darwin suffix on Mac. Only way for me to get the vrt to run correctly is with the docker container.
@MatissJanis commented on GitHub (Jan 11, 2026):
Try
"""
HTTPS=true yarn start
yarn vrt:docker
"""
@RMcGhee commented on GitHub (Jan 11, 2026):
BAM! That's the happy path, thanks.
I was definitely confused about which docker container was being used for what, the VRT docker container is a super lightweight playwright container, and has nothing to do with the dev docker container. I prefer not developing in the dev docker container, so I shouldn't need to go back and forth, thanks!