refactor: consolidate infra management into justfile + add mobile-dev#797
Merged
Conversation
wesbillman
approved these changes
May 29, 2026
`just relay` previously assumed Docker services were already running, requiring manual `docker compose up` first. The desktop and mobile workflows had the same gap — multiple manual steps across terminals. Adds `_ensure-services` and `_ensure-migrations` private recipes that idempotently start Docker services and wait for health. Wires `relay` to depend on `_ensure-migrations` so it works from a cold start. Adds `mobile-dev` that handles the full stack: Docker, relay (background), iOS simulator, and `flutter run` — all in one command.
wpfleger96
force-pushed
the
wpfleger/justfile-mobile-dev
branch
from
c706139 to
d0d01e9
Compare
Three places duplicated Docker health-check and migration logic: justfile, dev-setup.sh, and run-tests.sh. Make the justfile's _ensure-services and _ensure-migrations recipes the single source of truth and have scripts delegate to them via hermit-managed just. Also wires all relay/e2e/migrate recipes to auto-start infra, removes dead recipes (web-install, web-install-ci, desktop-app), and adds a mobile-dev recipe for one-command iOS development.
wpfleger96
changed the title
Several docs referenced removed recipes (desktop-app), manual docker compose steps that are now auto-handled by just relay, or script internals that now delegate to justfile recipes.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Docker infrastructure startup logic (health checks, service startup, migration application) was duplicated across
justfile,scripts/dev-setup.sh, andscripts/run-tests.sh. This makes the justfile's_ensure-servicesand_ensure-migrationsrecipes the single source of truth and refactors the scripts to delegate to them.Adds a
just mobile-devrecipe that handles the full mobile development stack in one command (Docker, relay, simulator, Flutter), and wires all relay/e2e/migrate recipes to auto-start infrastructure from a cold start._ensure-services/_ensure-migrationsare the canonical infra guard — idempotent, sub-second when services are already runningdev-setup.shdelegates Docker health-wait tojust _ensure-servicesinstead of inlining its ownwait_healthy()loop (~45 lines removed)run-tests.shreplaces inlineservices_healthy()+ensure_services()+ensure_migrations()with a singleensure_infra()that callsjust _ensure-migrations(~50 lines removed)relay-web,relay-release,desktop-e2e-seed,desktop-e2e-integrationwired to_ensure-migrations;migratewired to_ensure-servicesmobile-devpolls for relay readiness instead of a fixed sleep, cleans up the relay process on exit via trapweb-install,web-install-ci(duplicates ofdesktop-install),desktop-app(alias fordev)docker compose upinstructions, referencejust relayauto-start behavior, documentjust mobile-dev, and fix references to removed recipes