From 607aa69beb4afd86de91b7d91f98997321699e2e Mon Sep 17 00:00:00 2001
From: Jason Ross <jaross@express.com>
Date: Sun, 27 Sep 2026 15:37:13 -0500
Subject: [PATCH] fix(homebrew-update): escape && so the plist is valid XML

The bash -c command used bare && inside a <string>, which isn't valid
XML: plutil -lint rejects the file, and launchd only loaded it because
its parser is lenient. Escape the operator, and move PATH into
EnvironmentVariables so the command stays short enough that an XML
formatter has no reason to re-wrap it.
---
 LaunchAgents/com.user.homebrew-update.plist | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/LaunchAgents/com.user.homebrew-update.plist b/LaunchAgents/com.user.homebrew-update.plist
index 06aa9da..caf8680 100644
--- a/LaunchAgents/com.user.homebrew-update.plist
+++ b/LaunchAgents/com.user.homebrew-update.plist
@@ -8,8 +8,13 @@
     <array>
         <string>/bin/bash</string>
         <string>-c</string>
-        <string>export PATH="/opt/homebrew/bin:/usr/local/bin:$PATH" && brew update && brew upgrade</string>
+        <string>brew update &amp;&amp; brew upgrade</string>
     </array>
+    <key>EnvironmentVariables</key>
+    <dict>
+        <key>PATH</key>
+        <string>/opt/homebrew/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin</string>
+    </dict>
     <key>StartCalendarInterval</key>
     <dict>
         <key>Hour</key>