Fix: Intelligent notifications now show periodic reminders

- Updated conversation-capture-user-prompt.sh notification logic
- Replaced daily suppression (sync-notified- flag) with 2-hour periodic reminders
- New system tracks last notification time in last-sync-notification file
- Shows notification every 2 hours during high activity (>50 ops)
- Updated memory-sync command to record sync timestamp
- Added step 2 in memory-sync process to update last-memory-sync file
- Fixes issue where notifications only showed once per day

Impact:
- Notifications now visible when activity > 50 ops
- Users get periodic reminders if they don't run /memory-sync
- System tracks actual sync execution via timestamp file
- Better user experience with actionable reminders

Testing:
- Verified with 162 ops showing notification correctly
- Tested periodic reminder logic (2-hour intervals)
- Confirmed notification suppression works between intervals

Author: kunwar-shah

.claude/commands/memory-sync.md

@@ -1,7 +1,7 @@
 ---
 description: Synchronize all memory bank files with current session context
 argument-hint: "[--full] [--quick]"
-allowed-tools: Read(*), Write(*), Edit(*), Bash(date:*)
+allowed-tools: Read(*), Write(*), Edit(*), Bash(date:*), Bash(echo:*)
 ---
 
 # Memory Sync — Universal Context Synchronization
@@ -50,24 +50,28 @@ The `/memory-sync` command provides intelligent synchronization of your memory b
    - Detect technical decisions made
    - Extract progress milestones
 
-2. **Update Active Context**
+2. **Record Sync Timestamp** (IMPORTANT - do this FIRST)
+   - Update `.claude/tmp/last-memory-sync` with current Unix timestamp: `date +%s > .claude/tmp/last-memory-sync`
+   - This tracks when sync was performed for notification system
+
+3. **Update Active Context**
    - Append session summary with timestamp
    - Update current focus if changed
    - Add new blockers discovered
    - Record achievements completed
 
-3. **Update Progress (if --full)**
+4. **Update Progress (if --full)**
    - Move completed items to ✅ COMPLETED section
    - Update in-progress status
    - Add new pending tasks
    - Update sprint progress percentage
 
-4. **Record Decisions (if --full)**
+5. **Record Decisions (if --full)**
    - Extract technical decisions from conversation
    - Format as ADR (Architecture Decision Record)
    - Append to decisionLog.md with timestamp
 
-5. **Update Patterns (if applicable)**
+6. **Update Patterns (if applicable)**
    - Detect new coding patterns adopted
    - Record architectural changes
    - Update technology stack if changed

.claude/hooks/conversation-capture-user-prompt.sh

@@ -92,11 +92,22 @@ if [ "$activity_count" -gt 50 ]; then
       notifications="${notifications}🔄 High activity ($activity_count ops) + ${hours_since_sync}h since last sync. Run /memory-sync --full. "
     fi
   else
-    # No sync record exists, suggest it
-    sync_notified_flag="$CLAUDE_TMP/sync-notified-$today"
-    if [ ! -f "$sync_notified_flag" ]; then
+    # No sync record exists - check when we last notified
+    last_notification_file="$CLAUDE_TMP/last-sync-notification"
+
+    if [ -f "$last_notification_file" ]; then
+      last_notification=$(cat "$last_notification_file" 2>/dev/null || echo "0")
+      hours_since_notification=$(( (current_time - last_notification) / 3600 ))
+
+      # Show reminder every 2 hours during high activity
+      if [ "$hours_since_notification" -gt 2 ]; then
+        notifications="${notifications}🔄 High activity ($activity_count ops) - no sync record. Consider /memory-sync --full. "
+        echo "$current_time" > "$last_notification_file"
+      fi
+    else
+      # First notification
       notifications="${notifications}🔄 High activity detected ($activity_count ops). Consider /memory-sync --full. "
-      touch "$sync_notified_flag"
+      echo "$current_time" > "$last_notification_file"
     fi
   fi
 fi