Skip to content

refactor(container): remove Apple Containerization support#50

Merged
jmgilman merged 4 commits intomasterfrom
refactor/remove-apple-containerization
Jan 4, 2026
Merged

refactor(container): remove Apple Containerization support#50
jmgilman merged 4 commits intomasterfrom
refactor/remove-apple-containerization

Conversation

@jmgilman
Copy link
Collaborator

@jmgilman jmgilman commented Jan 4, 2026

Summary

  • Remove Apple Containerization Framework runtime implementation and tests
  • Update configuration validation to only allow docker and podman runtimes
  • Remove Apple-specific code paths from CLI commands and instance manager
  • Update all documentation to reflect Docker/Podman-only support
  • Mark ADR-002 as "Superseded" to preserve historical context

Changes

Files Deleted

  • internal/container/apple.go - Apple runtime implementation
  • internal/container/apple_test.go - Apple runtime tests

Go Code Changes

  • internal/config/config.go - Remove "apple" from valid runtimes
  • internal/instance/manager.go - Remove RuntimeApple constant and Apple flag handling
  • internal/cmd/root.go - Remove Apple runtime selection logic
  • internal/cmd/run.go - Remove Apple-specific logs command and devcontainer warning
  • internal/instance/manager_test.go - Update tests to use Podman/Docker
  • integration/integration_test.go - Remove Apple detection and cleanup logic

Documentation Updates

  • ADR-002 marked as "Superseded" with explanation
  • All runtime references updated to show Docker/Podman only
  • Removed io.headjack.apple.flags label documentation

Test plan

  • just check passes (format, lint, test)
  • CI workflows pass

🤖 Generated with Claude Code

Current behavior:
Headjack supported three container runtimes: Docker, Podman, and Apple
Containerization Framework. Apple Containerization required macOS 26+
and used the `container` CLI binary.

New behavior:
Headjack now supports only Docker and Podman runtimes. All Apple
Containerization code, tests, and documentation references have been
removed. ADR-002 is preserved with "Superseded" status for historical
context.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jan 4, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
headjack a4b52a4 Commit Preview URL

Branch Preview URL
Jan 04 2026, 10:42 PM

jmgilman and others added 3 commits January 4, 2026 14:32
Current behavior:
Several files still contained Apple Containerization references after
the initial removal commit.

New behavior:
- justfile: Remove apple auto-detection and integration-test-apple target
- ADR-002: Update addendum to reflect runtime evolution and removal
- Design docs: Remove Apple from runtime support tables and diagrams
- Code comments: Update to show Docker/Podman only
- Dockerfile: Update iptables workaround comment

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@jmgilman jmgilman merged commit ebb321c into master Jan 4, 2026
12 checks passed
@jmgilman jmgilman deleted the refactor/remove-apple-containerization branch January 4, 2026 22:45
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.

1 participant