Skip to content

Revert the Fly.io deploy (#179) - #189

Closed
hanson0291 wants to merge 1 commit into
AgentSystemLabs:mainfrom
hanson0291:revert/fly-io
Closed

hanson0291 wants to merge 1 commit into
AgentSystemLabs:mainfrom
hanson0291:revert/fly-io

Conversation

@hanson0291

Copy link
Copy Markdown
Contributor

Reverts #179.

  • Deletes deploy/fly.sh and docs/fly.md, and the README's Deploy to Fly.io section.
  • Drops every Fly.io mention from the README, docs/railway.md, the Dockerfile, provision.sh, .dockerignore and the team comments (src/client/ui/team.ts, src/server/team.ts, src/shared/protocol.ts).
  • start.sh goes back to sshd on port 22: AGENT_OFFICE_SSHD_PORT only existed for Fly.io, and nothing else sets it.

The Dokploy deploy (#180) was merged on top of #179, so a plain revert conflicts. I resolved the conflicts by keeping all the Dokploy text and removing only the Fly.io parts.

🤖 Generated with Claude Code

…on a volume (AgentSystemLabs#179)"

This reverts commit 108a346.

Removes deploy/fly.sh, docs/fly.md, the README section and every Fly.io
mention, and start.sh's AGENT_OFFICE_SSHD_PORT (only fly.sh set it).
The Dokploy deploy (AgentSystemLabs#180), which landed on top, is kept as is.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@webdevcody

Copy link
Copy Markdown
Contributor

why?

@webdevcody

Copy link
Copy Markdown
Contributor

Thanks for putting this together, and for untangling the conflicts with the Dokploy deploy so carefully!

I'm going to close this one though. The Fly.io deploy (#179) was added on purpose and we smoke-tested it before merging. The PR doesn't mention a bug or problem with it, and AGENT_OFFICE_SSHD_PORT being set only by fly.sh is intended: Fly's own SSH proxy takes port 22, so sshd has to move.

If you hit an actual issue with the Fly deploy (a broken script, a doc error, something in start.sh misbehaving on another host), please open an issue or a targeted fix PR and we'll gladly take a look. Thanks again for contributing, and the command palette in #188 looks great 🙂

@webdevcody webdevcody closed this Sep 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants