Skip to content

chore: remove dead Devbox Cloud code - #3050

Merged
mikeland73 merged 1 commit into
mainfrom
claude/remove-devbox-cloud-code-adec9f
Nov 9, 2026
Merged

mikeland73 merged 1 commit into
mainfrom
claude/remove-devbox-cloud-code-adec9f

Conversation

@mikeland73

Copy link
Copy Markdown
Collaborator

Fixes #3047.

Devbox Cloud is shut down. These code paths only ran when DEVBOX_REGION was set, which never happens outside Devbox Cloud, so this is dead code.

Changes

  • envir: remove IsDevboxCloud(), DevboxRegion, and SSHTTY (only used by the cloud shell-access check)
  • telemetry: remove InCloud / CloudRegion metadata (Sentry context) and the Segment shell_access property. Outside cloud it was always "local", so the property carried no information.
  • vercheck: remove the cloud early return and its skip_if_devbox_cloud test case
  • services: delete internal/services/status.go entirely, not just the cloud branch. ListenToChanges and updateServiceStatusOnRemote had no callers; they watched the mutagen-synced .devbox.cloud dir. fsnotify moves to an indirect dependency in go.mod.
  • examples: drop .devbox.cloud from examples/stacks/django/.gitignore

Not in this PR

  • vscode-extension/src/openinvscode.ts still mentions devbox-vms; it's being removed separately in the change that moves VS Code install and update to GitHub releases.
  • envir.IsInBrowser() (START_WEB_TERMINAL) looks like it was also for the cloud web terminal, but the issue doesn't cover it, so I left it alone.

Testing

  • go build ./..., go test ./..., golangci-lint run (0 issues), gofumpt
  • git grep -n "IsDevboxCloud\|DevboxRegion\|DEVBOX_REGION\|devbox-vms" matches only openinvscode.ts

🤖 Generated with Claude Code

Devbox Cloud is shut down. These code paths only ran when DEVBOX_REGION
was set, which never happens outside Devbox Cloud.

- envir: drop IsDevboxCloud, DevboxRegion and the now-unused SSHTTY
- telemetry: drop InCloud/CloudRegion metadata and the shell_access
  property (always "local" outside cloud)
- vercheck: drop the cloud early return and its test
- services: delete status.go; the .devbox.cloud status watcher had no
  callers (fsnotify becomes an indirect dependency)
- examples: drop .devbox.cloud from the django .gitignore

Fixes #3047

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@claude

claude Bot commented Nov 9, 2026

Copy link
Copy Markdown

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@greptile-apps

greptile-apps Bot commented Nov 9, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The PR appears safe to merge.

Summary

Removes unused Devbox Cloud code.

  • Version checks no longer skip runs marked as Devbox Cloud.
  • Command telemetry no longer carries Devbox Cloud labels.
  • The unused service-status watcher and its cloud files are removed.

No actionable issues found. Build and tests were not run during this review.

Reviews (1) · Last reviewed commit: "chore: remove dead Devbox Cloud code" · Reviewed by Greptile

@mikeland73
mikeland73 merged commit 196f7d0 into main Nov 9, 2026
31 checks passed
@mikeland73
mikeland73 deleted the claude/remove-devbox-cloud-code-adec9f branch November 9, 2026 23:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Remove dead Devbox Cloud code

1 participant