aidevops 3.1.55 → 3.1.57

package/VERSION

@@ -1 +1 @@
-3.1.55
+3.1.57

package/aidevops.sh

@@ -3,7 +3,7 @@
 # AI DevOps Framework CLI
 # Usage: aidevops <command> [options]
 #
-# Version: 3.1.55
+# Version: 3.1.57
 
 set -euo pipefail
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "aidevops",
-  "version": "3.1.55",
+  "version": "3.1.57",
   "description": "AI DevOps Framework - AI-assisted development workflows, code quality, and deployment automation",
   "type": "module",
   "bin": {

package/setup.sh

@@ -10,7 +10,7 @@ shopt -s inherit_errexit 2>/dev/null || true
 # AI Assistant Server Access Framework Setup Script
 # Helps developers set up the framework for their infrastructure
 #
-# Version: 3.1.55
+# Version: 3.1.57
 #
 # Quick Install:
 #   npm install -g aidevops && aidevops update          (recommended)
@@ -193,6 +193,38 @@ _launchd_has_agent() {
 	return $?
 }
 
+# Install a launchd plist only if its content has changed.
+# Avoids unnecessary unload/reload which resets StartInterval timers.
+# Usage: _launchd_install_if_changed <label> <plist_path> <new_content>
+# Returns: 0 = installed or unchanged, 1 = failed to load
+_launchd_install_if_changed() {
+	local label="$1"
+	local plist_path="$2"
+	local new_content="$3"
+
+	# Compare with existing plist — skip reload if identical
+	if [[ -f "$plist_path" ]]; then
+		local existing_content
+		existing_content=$(cat "$plist_path")
+		if [[ "$existing_content" == "$new_content" ]]; then
+			# Ensure it's loaded even if content unchanged
+			if ! _launchd_has_agent "$label"; then
+				launchctl load "$plist_path" 2>/dev/null || return 1
+			fi
+			return 0
+		fi
+		# Content changed — unload before replacing
+		if _launchd_has_agent "$label"; then
+			launchctl unload "$plist_path" 2>/dev/null || true
+		fi
+	fi
+
+	# Write new plist and load
+	printf '%s\n' "$new_content" >"$plist_path"
+	launchctl load "$plist_path" 2>/dev/null || return 1
+	return 0
+}
+
 # Detect whether a scheduler is already installed via launchd or cron.
 # Optionally migrates legacy launchd labels / cron entries to launchd on macOS.
 _scheduler_detect_installed() {
@@ -1138,15 +1170,13 @@ PLIST
 		if [[ "$(uname -s)" == "Darwin" ]]; then
 			local stats_plist="$HOME/Library/LaunchAgents/${stats_label}.plist"
 
-			if _launchd_has_agent "$stats_label"; then
-				launchctl unload "$stats_plist" 2>/dev/null || true
-			fi
-
 			local _xml_stats_script _xml_stats_home _xml_stats_path
 			_xml_stats_script=$(_xml_escape "$stats_script")
 			_xml_stats_home=$(_xml_escape "$HOME")
 			_xml_stats_path=$(_xml_escape "$PATH")
-			cat >"$stats_plist" <<PLIST
+			local stats_plist_content
+			stats_plist_content=$(
+				cat <<PLIST
 <?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
 <plist version="1.0">
@@ -1178,7 +1208,8 @@ PLIST
 </dict>
 </plist>
 PLIST
-			if launchctl load "$stats_plist"; then
+			)
+			if _launchd_install_if_changed "$stats_label" "$stats_plist" "$stats_plist_content"; then
 				print_info "Stats wrapper enabled (launchd, every 15 min)"
 			else
 				print_warning "Failed to load stats wrapper LaunchAgent"
@@ -1253,11 +1284,6 @@ PLIST
 		if [[ "$(uname -s)" == "Darwin" ]]; then
 			local guard_plist="$HOME/Library/LaunchAgents/${guard_label}.plist"
 
-			# Unload old plist if upgrading
-			if _launchd_has_agent "$guard_label"; then
-				launchctl unload "$guard_plist" || true
-			fi
-
 			# XML-escape paths for safe plist embedding (prevents injection
 			# if $HOME or paths contain &, <, > characters)
 			local _xml_guard_script _xml_guard_home _xml_guard_path
@@ -1265,7 +1291,9 @@ PLIST
 			_xml_guard_home=$(_xml_escape "$HOME")
 			_xml_guard_path=$(_xml_escape "$PATH")
 
-			cat >"$guard_plist" <<GUARD_PLIST
+			local guard_plist_content
+			guard_plist_content=$(
+				cat <<GUARD_PLIST
 <?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
 <plist version="1.0">
@@ -1306,8 +1334,9 @@ PLIST
 </dict>
 </plist>
 GUARD_PLIST
+			)
 
-			if launchctl load "$guard_plist"; then
+			if _launchd_install_if_changed "$guard_label" "$guard_plist" "$guard_plist_content"; then
 				print_info "Process guard enabled (launchd, every 30s, survives reboot)"
 			else
 				print_warning "Failed to load process guard LaunchAgent"
@@ -1343,12 +1372,9 @@ GUARD_PLIST
 		if [[ "$(uname -s)" == "Darwin" ]]; then
 			local monitor_plist="$HOME/Library/LaunchAgents/${monitor_label}.plist"
 
-			# Unload old plist if upgrading
-			if _launchd_has_agent "$monitor_label"; then
-				launchctl unload "$monitor_plist" 2>/dev/null || true
-			fi
-
-			cat >"$monitor_plist" <<MONITOR_PLIST
+			local monitor_plist_content
+			monitor_plist_content=$(
+				cat <<MONITOR_PLIST
 <?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
 <plist version="1.0">
@@ -1386,8 +1412,9 @@ GUARD_PLIST
 </dict>
 </plist>
 MONITOR_PLIST
+			)
 
-			if launchctl load "$monitor_plist" 2>/dev/null; then
+			if _launchd_install_if_changed "$monitor_label" "$monitor_plist" "$monitor_plist_content"; then
 				print_info "Memory pressure monitor enabled (launchd, every 60s, survives reboot)"
 			else
 				print_warning "Failed to load memory pressure monitor LaunchAgent"
@@ -1418,17 +1445,14 @@ MONITOR_PLIST
 		if [[ "$(uname -s)" == "Darwin" ]]; then
 			local st_plist="$HOME/Library/LaunchAgents/${st_label}.plist"
 
-			# Unload old plist if upgrading
-			if _launchd_has_agent "$st_label"; then
-				launchctl unload "$st_plist" 2>/dev/null || true
-			fi
-
 			# XML-escape paths for safe plist embedding
 			local _xml_st_script _xml_st_home
 			_xml_st_script=$(_xml_escape "$st_script")
 			_xml_st_home=$(_xml_escape "$HOME")
 
-			cat >"$st_plist" <<ST_PLIST
+			local st_plist_content
+			st_plist_content=$(
+				cat <<ST_PLIST
 <?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
 <plist version="1.0">
@@ -1467,8 +1491,9 @@ MONITOR_PLIST
 </dict>
 </plist>
 ST_PLIST
+			)
 
-			if launchctl load "$st_plist" 2>/dev/null; then
+			if _launchd_install_if_changed "$st_label" "$st_plist" "$st_plist_content"; then
 				print_info "Screen time snapshot enabled (launchd, every 6h, survives reboot)"
 			else
 				print_warning "Failed to load screen time snapshot LaunchAgent"
@@ -1489,6 +1514,100 @@ ST_PLIST
 		fi
 	fi
 
+	# Contribution watch — monitors external issues/PRs for new activity (t1554).
+	# Auto-seeds on first run (discovers authored/commented issues/PRs), then installs
+	# a launchd/cron job to scan periodically. Requires gh CLI authenticated.
+	# No consent needed — this is passive monitoring (read-only notifications API),
+	# not autonomous action. Comment bodies are never processed by LLM in automated context.
+	# Respects config: aidevops config set orchestration.contribution_watch false
+	local cw_script="$HOME/.aidevops/agents/scripts/contribution-watch-helper.sh"
+	local cw_label="sh.aidevops.contribution-watch"
+	local cw_state="$HOME/.aidevops/cache/contribution-watch.json"
+	if [[ -x "$cw_script" ]] && is_feature_enabled contribution_watch 2>/dev/null && command -v gh &>/dev/null && gh auth status &>/dev/null 2>&1; then
+		mkdir -p "$HOME/.aidevops/cache" "$HOME/.aidevops/logs"
+
+		# Auto-seed on first run (populates state file with existing contributions)
+		if [[ ! -f "$cw_state" ]]; then
+			print_info "Discovering external contributions for contribution watch..."
+			if bash "$cw_script" seed >/dev/null 2>&1; then
+				print_info "Contribution watch seeded (external issues/PRs discovered)"
+			else
+				print_warning "Contribution watch seed failed (non-fatal, will retry on next run)"
+			fi
+		fi
+
+		# Install/update scheduled scanner
+		if [[ "$(uname -s)" == "Darwin" ]]; then
+			local cw_plist="$HOME/Library/LaunchAgents/${cw_label}.plist"
+
+			local _xml_cw_script _xml_cw_home
+			_xml_cw_script=$(_xml_escape "$cw_script")
+			_xml_cw_home=$(_xml_escape "$HOME")
+
+			local cw_plist_content
+			cw_plist_content=$(
+				cat <<CW_PLIST
+<?xml version="1.0" encoding="UTF-8"?>
+<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
+<plist version="1.0">
+<dict>
+	<key>Label</key>
+	<string>${cw_label}</string>
+	<key>ProgramArguments</key>
+	<array>
+		<string>/bin/bash</string>
+		<string>${_xml_cw_script}</string>
+		<string>scan</string>
+	</array>
+	<key>StartInterval</key>
+	<integer>3600</integer>
+	<key>StandardOutPath</key>
+	<string>${_xml_cw_home}/.aidevops/logs/contribution-watch.log</string>
+	<key>StandardErrorPath</key>
+	<string>${_xml_cw_home}/.aidevops/logs/contribution-watch.log</string>
+	<key>EnvironmentVariables</key>
+	<dict>
+		<key>PATH</key>
+		<string>/opt/homebrew/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin</string>
+		<key>HOME</key>
+		<string>${_xml_cw_home}</string>
+	</dict>
+	<key>RunAtLoad</key>
+	<false/>
+	<key>KeepAlive</key>
+	<false/>
+	<key>ProcessType</key>
+	<string>Background</string>
+	<key>LowPriorityBackgroundIO</key>
+	<true/>
+	<key>Nice</key>
+	<integer>10</integer>
+</dict>
+</plist>
+CW_PLIST
+			)
+
+			if _launchd_install_if_changed "$cw_label" "$cw_plist" "$cw_plist_content"; then
+				print_info "Contribution watch enabled (launchd, hourly scan)"
+			else
+				print_warning "Failed to load contribution watch LaunchAgent"
+			fi
+		else
+			# Linux: cron entry (hourly)
+			local _cron_cw_script
+			_cron_cw_script=$(_cron_escape "$cw_script")
+			(
+				crontab -l 2>/dev/null | grep -v 'aidevops: contribution-watch'
+				echo "0 * * * * /bin/bash ${_cron_cw_script} scan >> \"\$HOME/.aidevops/logs/contribution-watch.log\" 2>&1 # aidevops: contribution-watch"
+			) | crontab - 2>/dev/null || true
+			if crontab -l 2>/dev/null | grep -qF "aidevops: contribution-watch" 2>/dev/null; then
+				print_info "Contribution watch enabled (cron, hourly scan)"
+			else
+				print_warning "Failed to install contribution watch cron entry"
+			fi
+		fi
+	fi
+
 	# Profile README — auto-create repo and seed README if not already set up.
 	# Requires gh CLI authenticated. Creates username/username repo, seeds README
 	# with stat markers, registers in repos.json with priority: "profile".
@@ -1544,17 +1663,14 @@ ST_PLIST
 		if [[ "$(uname -s)" == "Darwin" ]]; then
 			local pr_plist="$HOME/Library/LaunchAgents/${pr_label}.plist"
 
-			# Unload old plist if upgrading
-			if _launchd_has_agent "$pr_label"; then
-				launchctl unload "$pr_plist" 2>/dev/null || true
-			fi
-
 			# XML-escape paths for safe plist embedding
 			local _xml_pr_script _xml_pr_home
 			_xml_pr_script=$(_xml_escape "$pr_script")
 			_xml_pr_home=$(_xml_escape "$HOME")
 
-			cat >"$pr_plist" <<PR_PLIST
+			local pr_plist_content
+			pr_plist_content=$(
+				cat <<PR_PLIST
 <?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
 <plist version="1.0">
@@ -1593,8 +1709,9 @@ ST_PLIST
 </dict>
 </plist>
 PR_PLIST
+			)
 
-			if launchctl load "$pr_plist" 2>/dev/null; then
+			if _launchd_install_if_changed "$pr_label" "$pr_plist" "$pr_plist_content"; then
 				print_info "Profile README update enabled (launchd, hourly)"
 			else
 				print_warning "Failed to load profile README update LaunchAgent"