chore(dev): replace docker compose with mise dev for Conductor#505
Merged
hmans merged 1 commit intoMay 19, 2026
mainchattocorp/chatto:mainfrom
hmans/mise-dev-configurable-portschattocorp/chatto:hmans/mise-dev-configurable-portsCopy head branch name to clipboard
Merged
chore(dev): replace docker compose with mise dev for Conductor#505hmans merged 1 commit intomainchattocorp/chatto:mainfrom hmans/mise-dev-configurable-portschattocorp/chatto:hmans/mise-dev-configurable-portsCopy head branch name to clipboard
hmans merged 1 commit into
mainchattocorp/chatto:mainfrom
hmans/mise-dev-configurable-portschattocorp/chatto:hmans/mise-dev-configurable-portsCopy head branch name to clipboard
Conversation
- Conductor's run script now invokes `mise dev` natively, mapping $CONDUCTOR_PORT to Vite, +1 to the Go backend, +2 to embedded NATS - Add `deps-frontend` as a dep of `dev-frontend` and `dev-frontend-codegen` so fresh worktrees install node_modules before pnpm dev/codegen run - Delete compose.yml, Dockerfile.compose, Dockerfile.frontend.compose - Update README/CLAUDE.md to document the new flow
hmans
added a commit
that referenced
this pull request
May 19, 2026
The compose.yml-based dev stack (replaced by mise dev in #505) was the only reason ATProto needed a build-tag-gated URL override: orb.local hostnames couldn't satisfy the OAuth spec's "public URL or loopback" requirement. The new mise dev setup uses http://localhost:$CONDUCTOR_PORT out of the box, which the existing isLocalhostURL detection picks up natively — so the override is now dead weight (and would have been wrong, since it hardcoded port 5173 instead of $CONDUCTOR_PORT). go.mod change: indigo moved indirect → direct after mise test-cli ran go get ./...; we import the package directly, so this is correct.
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
1 participant
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.
Summary
runscript now invokesmise devnatively instead ofdocker compose up, mapping$CONDUCTOR_PORTto Vite,+1to the Go backend (CHATTO_WEBSERVER_PORT), and+2to the embedded NATS server — all override the defaults already incli/chatto.toml.deps-frontendas a dependency ofdev-frontendanddev-frontend-codegenso fresh worktrees installnode_modulesbeforepnpm dev/pnpm codegenrun.compose.yml,Dockerfile.compose, andDockerfile.frontend.compose; updated the "Development with Conductor" section ofREADME.md(andCLAUDE.mdvia the symlink) to document the new flow.Test plan
mise devand Vite/backend/NATS pick up the assigned portsmise devoutside Conductor still works against thecli/chatto.tomldefaults (5173 / 4000 / 4555)