Skip to content

Decouple launchd from package paths - #6

Merged
nyldn merged 1 commit into
mainfrom
fix/decouple-launchd-package-paths
Aug 23, 2026
Merged

Decouple launchd from package paths#6
nyldn merged 1 commit into
mainfrom
fix/decouple-launchd-package-paths

Conversation

@nyldn

@nyldn nyldn commented Aug 23, 2026

Copy link
Copy Markdown
Owner

What changed

  • remove the configured AI Optimizer executable from the launchd plist entirely
  • generate a versioned, owner-only product launcher keyed to the absolute executable string
  • embed that exact path with POSIX-safe quoting and execute only run-maintenance
  • refuse symlinked or drifted launchers while accepting legacy v0.1.4/v0.1.5 plists
  • fix Ruby 2.6 byte comparison for non-ASCII executable paths
  • bump to v0.1.6 and document why v0.1.4/v0.1.5 were superseded

Live lifecycle evidence

  • forced Homebrew revision upgrade changed 0.1.5 to 0.1.5_1 and removed the old Cellar directory
  • plist contained no formula executable path
  • job remained loaded after the asynchronous removal window
  • launchctl kickstart completed run 1 with exit 0 and a fresh passed receipt

Verification

  • 41 tests, 162 assertions, 0 failures
  • non-ASCII/apostrophe/space executable schedules twice and executes exactly
  • system Ruby syntax and install/uninstall round trip pass
  • deterministic release archive matches across two builds
  • Gitleaks scanned full history with no findings
  • Claude Fable 5 re-review: PASS

@coderabbitai

coderabbitai Bot commented Aug 23, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 228f7073-7460-4adf-a5c7-a5701d5f72b5


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@nyldn
nyldn merged commit 7450f0f into main Aug 23, 2026
3 checks passed
@nyldn
nyldn deleted the fix/decouple-launchd-package-paths branch August 23, 2026 03:42
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