Skip to content

feat: install the Apple plugin as a symlink - #54

Merged
wcgomes merged 4 commits into
mainfrom
patch/plugin-symlink-install
Sep 25, 2026
Merged

wcgomes merged 4 commits into
mainfrom
patch/plugin-symlink-install

Conversation

@wcgomes

@wcgomes wcgomes commented Sep 25, 2026

Copy link
Copy Markdown
Owner

Summary

Replace install-plugin with plugin --install and plugin --uninstall.

  • --install removes an old copied Mach-O and creates an absolute symlink.
  • Homebrew target is $(brew --prefix)/opt/adevcontainer/bin/adevcontainer, so brew upgrade adevcontainer updates the plugin without brew writing /usr/local (brew does not run as sudo).
  • Non-Homebrew installs symlink to the running executable path without realpath.
  • --uninstall removes only bin/dev and config.toml.
  • Formula post_install no longer writes Apple's install-root.
  • container dev plugin --install must not retarget bin/dev at itself.

Caveat: run sudo adevcontainer plugin --install once, and again only after an Apple container upgrade wipes the plugin dir (apple/container#1617).

Specs archived to specs/changes/archive/20260925-plugin-symlink-install/ and specs/changes/archive/20260925-install-plugin-command/. Live contract is specs/plugin.md.

Test plan

  • swift run adevcontainerTests — not run here. Package.swift requires macOS 26, so this environment cannot execute the suite.
  • CI on this PR

install-plugin is replaced by plugin --install and plugin --uninstall.
Homebrew upgrades follow the opt symlink so brew does not write /usr/local.
Both plugin changes are archived; `specs/plugin.md` holds the symlink contract; wiki links point at the archive.
Registered MiniTest cases are 1147.
Drop the opencode OCI feature and install OpenCode v2 from install-tools.sh. Wiki matches that layout.
@wcgomes
wcgomes merged commit 7c2fd9a into main Sep 25, 2026
1 check passed
@wcgomes
wcgomes deleted the patch/plugin-symlink-install branch September 25, 2026 11:27
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