release 1.6.9

Author: mylee04

README.md

@@ -13,19 +13,19 @@ Desktop notifications for AI coding tools - get alerts when tasks complete or in
   <img src="assets/multi-tools-support-02.png" width="48%" alt="All tools enabled"/>
 </p>
 
-[![Version](https://img.shields.io/badge/version-1.6.8-blue.svg)](https://github.com/mylee04/code-notify/releases)
+[![Version](https://img.shields.io/badge/version-1.6.9-blue.svg)](https://github.com/mylee04/code-notify/releases)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
 [![macOS](https://img.shields.io/badge/macOS-supported-green.svg)](https://www.apple.com/macos)
 [![Linux](https://img.shields.io/badge/Linux-supported-green.svg)](https://www.linux.org/)
 [![Windows](https://img.shields.io/badge/Windows-supported-green.svg)](https://www.microsoft.com/windows)
 
 ---
 
-## What's New in v1.6.8
+## What's New in v1.6.9
 
-- **npm is now a first-class install path**: `npm install -g code-notify` now provides `code-notify`, `cn`, and `cnp` on macOS, Linux, and Windows
-- **Repeated idle/input alerts are deduped**: repeated `idle_prompt`-style notification events are suppressed so sound playback does not keep firing while Claude stays idle
-- **`cn update` understands npm installs**: npm users now get the correct `npm install -g code-notify@latest` update path, including on Windows
+- **Legacy Claude hooks are repaired during supported upgrades**: stale `claude-notify`-style Claude configs are migrated to the current `code-notify` hook format when users update through the supported install paths
+- **Claude idle dedupe now survives upgrades from older installs**: users who were still on the old blank-matcher Claude hooks no longer bypass the repeated `idle_prompt` suppression after updating
+- **Codex notify integration now reads Codex payload JSON directly**: completion notifications use Codex's `notify` payload format, and the docs/status output now clearly call out Codex's current completion-focused behavior
 
 ---
 
@@ -60,6 +60,8 @@ cn update
 code-notify version
 ```
 
+If you were using the older `claude-notify` hook layout, supported upgrades now repair those Claude hooks automatically.
+
 **Linux / WSL**
 
 ```bash
@@ -129,6 +131,9 @@ Code-Notify uses the hook systems built into AI coding tools:
 - **Codex**: `~/.codex/config.toml`
 - **Gemini CLI**: `~/.gemini/settings.json`
 
+For Codex, Code-Notify configures `notify = ["/absolute/path/to/notifier.sh", "codex"]` and reads the JSON payload Codex appends on completion.
+Codex currently exposes completion events through `notify`; approval and `request_permissions` prompts do not currently arrive through this hook.
+
 When enabled, it adds hooks that call the notification script when tasks complete:
 
 ```json
@@ -172,6 +177,8 @@ cn alerts reset                    # Back to default (idle_prompt only)
 | `auth_success`       | Authentication success                 |
 | `elicitation_dialog` | MCP tool input needed                  |
 
+Alert-type matching currently applies to Claude Code and Gemini CLI notification hooks. Codex currently uses completion events from `notify`, so `permission_prompt` and `idle_prompt` settings do not change Codex behavior.
+
 ## Troubleshooting
 
 **Command not found?**

bin/code-notify

@@ -6,7 +6,7 @@
 set -e
 
 # Version
-VERSION="1.6.8"
+VERSION="1.6.9"
 
 # Determine the directory where the script is located (resolve symlinks)
 SCRIPT_PATH="${BASH_SOURCE[0]}"
@@ -52,6 +52,10 @@ case "$COMMAND_NAME" in
                 source "$LIB_DIR/commands/global.sh"
                 handle_global_command "$@"
                 ;;
+            "repair-hooks")
+                shift
+                repair_legacy_hooks_command "${1:-}"
+                ;;
             *)
                 error "Unknown command: $1"
                 echo "Try 'cn help' for usage"
@@ -73,6 +77,10 @@ case "$COMMAND_NAME" in
                 source "$LIB_DIR/commands/global.sh"
                 handle_global_command "$@"
                 ;;
+            "repair-hooks")
+                shift
+                repair_legacy_hooks_command "${1:-}"
+                ;;
             "project")
                 shift
                 source "$LIB_DIR/commands/project.sh"

docs/installation.md

@@ -4,12 +4,16 @@
 
 ## Quick Overview
 
-Code-Notify adds desktop notifications to Claude Code. You'll get alerts when:
+Code-Notify adds desktop notifications to Claude Code, Codex, and Gemini CLI. You'll get alerts when:
 
 - Tasks complete
-- Claude needs your input
+- Claude or Gemini needs your input
 - Voice announcements (macOS)
 
+Codex currently exposes completion notifications through its `notify` hook. Approval and `request_permissions` prompts do not currently trigger Code-Notify through Codex.
+
+Supported upgrade paths also repair stale `claude-notify`-style Claude hooks automatically when they are detected.
+
 ## For LLM Agents
 
 Execute these commands to install code-notify:

lib/code-notify/commands/global.sh

@@ -387,6 +387,7 @@ enable_notifications_global() {
 enable_single_tool() {
     local tool="$1"
     local quiet="${2:-}"
+    local needs_repair=1
 
     # Check if tool is installed
     if ! is_tool_installed "$tool"; then
@@ -396,8 +397,12 @@ enable_single_tool() {
         return 1
     fi
 
+    if [[ "$tool" == "claude" ]] && claude_global_hooks_need_repair; then
+        needs_repair=0
+    fi
+
     # Check if already enabled
-    if is_tool_enabled "$tool"; then
+    if [[ $needs_repair -ne 0 ]] && is_tool_enabled "$tool"; then
         if [[ "$quiet" != "quiet" ]]; then
             warning "$tool notifications already enabled"
         fi
@@ -406,7 +411,11 @@ enable_single_tool() {
 
     # Enable the tool
     if [[ "$quiet" != "quiet" ]]; then
-        info "Enabling $tool notifications..."
+        if [[ "$tool" == "claude" ]] && [[ $needs_repair -eq 0 ]]; then
+            info "Repairing existing $tool notification hooks..."
+        else
+            info "Enabling $tool notifications..."
+        fi
     fi
 
     if ! enable_tool "$tool"; then
@@ -519,7 +528,12 @@ show_status() {
 
     # Claude Code
     if is_tool_installed "claude"; then
-        if is_tool_enabled "claude"; then
+        if claude_global_hooks_need_repair; then
+            echo "  ${WARNING} Claude Code: ${YELLOW}REPAIR NEEDED${RESET}"
+            echo "     Config: $GLOBAL_SETTINGS_FILE"
+            echo "     Current hooks still point to an older claude-notify-style configuration"
+            echo "     Run: ${CYAN}cn on claude${RESET}"
+        elif is_tool_enabled "claude"; then
             echo "  ${CHECK_MARK} Claude Code: ${GREEN}ENABLED${RESET}"
             echo "     Config: $GLOBAL_SETTINGS_FILE"
         else
@@ -534,6 +548,7 @@ show_status() {
         if is_tool_enabled "codex"; then
             echo "  ${CHECK_MARK} Codex: ${GREEN}ENABLED${RESET}"
             echo "     Config: $CODEX_CONFIG_FILE"
+            echo "     Events: completion-only via Codex notify payloads"
         else
             echo "  ${MUTE} Codex: ${DIM}DISABLED${RESET}"
         fi
@@ -984,6 +999,9 @@ show_alerts_status() {
     echo "  ${CYAN}cn alerts remove permission_prompt${RESET} # Stop permission notifications"
     echo "  ${CYAN}cn alerts reset${RESET}                   # Back to idle_prompt only"
     echo ""
+    dim "Alert-type matching currently applies to Claude Code and Gemini CLI hooks."
+    dim "Codex currently exposes completion events through notify, so permission_prompt/idle_prompt settings do not change Codex behavior."
+    echo ""
     dim "After changing, run 'cn on' to apply the new settings."
 }
 

lib/code-notify/core/config.sh

@@ -239,6 +239,115 @@ has_claude_notify_hooks() {
     json_has "$file" '(.hooks.Notification != null) or (.hooks.Stop != null)' '"(Notification|Stop)"'
 }
 
+get_global_claude_notify_command() {
+    printf '%s notification claude\n' "$(get_notify_script)"
+}
+
+get_global_claude_stop_command() {
+    printf '%s stop claude\n' "$(get_notify_script)"
+}
+
+has_current_global_claude_hooks() {
+    local file="$1"
+    local matcher notify_cmd stop_cmd
+
+    if [[ ! -f "$file" ]]; then
+        return 1
+    fi
+
+    matcher=$(get_notify_matcher)
+    notify_cmd=$(get_global_claude_notify_command)
+    stop_cmd=$(get_global_claude_stop_command)
+
+    if has_jq; then
+        jq -e \
+            --arg matcher "$matcher" \
+            --arg notify "$notify_cmd" \
+            --arg stop "$stop_cmd" \
+            '
+            (.hooks.Notification | type == "array" and length > 0) and
+            (.hooks.Stop | type == "array" and length > 0) and
+            .hooks.Notification[0].matcher == $matcher and
+            .hooks.Notification[0].hooks[0].type == "command" and
+            .hooks.Notification[0].hooks[0].command == $notify and
+            .hooks.Stop[0].matcher == "" and
+            .hooks.Stop[0].hooks[0].type == "command" and
+            .hooks.Stop[0].hooks[0].command == $stop
+            ' "$file" >/dev/null 2>&1
+        return $?
+    fi
+
+    if has_python3; then
+        python3 - "$file" "$matcher" "$notify_cmd" "$stop_cmd" << 'PYTHON'
+import json
+import sys
+
+file_path, matcher, notify_cmd, stop_cmd = sys.argv[1:5]
+
+with open(file_path, "r") as fh:
+    settings = json.load(fh)
+
+notification = settings.get("hooks", {}).get("Notification", [])
+stop = settings.get("hooks", {}).get("Stop", [])
+
+assert isinstance(notification, list) and notification
+assert isinstance(stop, list) and stop
+assert notification[0].get("matcher", "") == matcher
+assert notification[0].get("hooks", [{}])[0].get("type") == "command"
+assert notification[0].get("hooks", [{}])[0].get("command") == notify_cmd
+assert stop[0].get("matcher", "") == ""
+assert stop[0].get("hooks", [{}])[0].get("type") == "command"
+assert stop[0].get("hooks", [{}])[0].get("command") == stop_cmd
+PYTHON
+        return $?
+    fi
+
+    grep -qF "\"matcher\": \"$matcher\"" "$file" &&
+        grep -qF "\"command\": \"$notify_cmd\"" "$file" &&
+        grep -qF "\"command\": \"$stop_cmd\"" "$file"
+}
+
+has_legacy_global_claude_hooks() {
+    local file="${1:-$GLOBAL_SETTINGS_FILE}"
+
+    if [[ ! -f "$file" ]]; then
+        return 1
+    fi
+
+    grep -q 'claude-notify' "$file" ||
+        grep -qE 'notifier\.sh (notification|stop)"' "$file" ||
+        grep -q 'notifier.sh PreToolUse' "$file"
+}
+
+claude_global_hooks_need_repair() {
+    has_legacy_global_claude_hooks "$GLOBAL_SETTINGS_FILE"
+}
+
+repair_legacy_hooks_command() {
+    local quiet="${1:-}"
+    local repaired=0
+
+    if claude_global_hooks_need_repair; then
+        if ! enable_hooks_in_settings; then
+            if [[ "$quiet" != "--quiet" ]]; then
+                echo "Failed to repair legacy Claude hooks" >&2
+            fi
+            return 1
+        fi
+        repaired=1
+
+        if [[ "$quiet" != "--quiet" ]]; then
+            echo "Repaired legacy Claude hooks in $GLOBAL_SETTINGS_FILE"
+        fi
+    fi
+
+    if [[ $repaired -eq 0 ]] && [[ "$quiet" != "--quiet" ]]; then
+        echo "No legacy hooks required repair"
+    fi
+
+    return 0
+}
+
 # Check if file has any hooks
 has_any_hooks() {
     local file="$1"
@@ -686,7 +795,7 @@ enable_codex_hooks() {
     mkdir -p "$CODEX_HOME"
 
     escaped_notify_script=$(toml_escape_string "$notify_script")
-    notify_line="notify = [\"bash\", \"-c\", \"$escaped_notify_script stop codex\"]"
+    notify_line="notify = [\"$escaped_notify_script\", \"codex\"]"
 
     # Check if config.toml exists
     if [[ -f "$CODEX_CONFIG_FILE" ]]; then
@@ -701,7 +810,7 @@ enable_codex_hooks() {
 # https://developers.openai.com/codex/config-reference/
 
 # Code-Notify: Desktop notifications
-notify = ["bash", "-c", "$escaped_notify_script stop codex"]
+notify = ["$escaped_notify_script", "codex"]
 EOF
     fi
 }