@adverant/nexus-memory-skill 2.3.1 → 2.3.3

package/hooks/auto-recall.sh

@@ -162,8 +162,14 @@ BODY=$(echo "$RESPONSE" | sed '$d')
 
 # Check for errors (silently fail - don't block conversation)
 if [[ "$HTTP_CODE" != "200" ]]; then
-  log "Failed to recall memories (HTTP $HTTP_CODE), continuing without context"
-  exit 0
+  if [[ "$VERBOSE" == "1" ]]; then
+    echo "⚠️  Auto-recall failed (HTTP $HTTP_CODE), continuing without context" >&2
+
+    if [[ "$HTTP_CODE" == "401" ]]; then
+      echo "   Authentication failed - check NEXUS_API_KEY" >&2
+    fi
+  fi
+  exit 0  # Don't block conversation on recall failure
 fi
 
 # Validate JSON response
@@ -186,6 +192,13 @@ ENTITY_COUNT=$(echo "$ENTITIES" | jq 'length' 2>/dev/null || echo "0")
 FACT_COUNT=$(echo "$FACTS" | jq 'length' 2>/dev/null || echo "0")
 EPISODIC_COUNT=$(echo "$EPISODIC" | jq 'length' 2>/dev/null || echo "0")
 
+# Log success in verbose mode
+if [[ "$VERBOSE" == "1" ]]; then
+  if [[ "$MEMORY_COUNT" != "0" ]] || [[ "$ENTITY_COUNT" != "0" ]]; then
+    echo "✅ Auto-recall: $MEMORY_COUNT memories, $ENTITY_COUNT entities injected" >&2
+  fi
+fi
+
 log "Found $MEMORY_COUNT memories, $ENTITY_COUNT entities, $FACT_COUNT facts, $EPISODIC_COUNT episodic"
 
 # Skip if nothing to show

package/hooks/recall-memory.sh

@@ -255,12 +255,22 @@ log "Response code: $HTTP_CODE"
 
 # Handle API errors
 if [[ "$HTTP_CODE" != "200" ]]; then
-  log_error "Failed to recall memories (HTTP $HTTP_CODE)"
   if [[ "$VERBOSE" == "1" ]]; then
-    log "Response: ${BODY:0:500}"
+    echo "❌ Recall failed (HTTP $HTTP_CODE)" >&2
+
+    ERROR_MSG=$(echo "$BODY" | jq -r '.error.message // .message // ""' 2>/dev/null)
+    if [[ -n "$ERROR_MSG" ]]; then
+      echo "   Error: $ERROR_MSG" >&2
+    fi
+
+    if [[ "$HTTP_CODE" == "401" ]]; then
+      echo "   Check: NEXUS_API_KEY environment variable" >&2
+    fi
   fi
+
+  # Return empty response so Claude doesn't get errors
   echo "$EMPTY_RESPONSE"
-  exit 0
+  exit 1  # Non-blocking error for logging
 fi
 
 # Validate JSON response
@@ -317,10 +327,12 @@ fi
 # Output verbose info if enabled
 if [[ "$VERBOSE" == "1" ]]; then
   MEMORY_COUNT=$(echo "$NORMALIZED" | jq '.memories | length')
-  ENTITY_COUNT=$(echo "$NORMALIZED" | jq '.entities | length')
-  FACT_COUNT=$(echo "$NORMALIZED" | jq '.facts | length')
 
-  log "Retrieved: $MEMORY_COUNT memories, $ENTITY_COUNT entities, $FACT_COUNT facts"
+  if [[ "$MEMORY_COUNT" == "0" ]]; then
+    echo "⚠️  No memories found for query" >&2
+  else
+    echo "✅ Retrieved $MEMORY_COUNT memories" >&2
+  fi
 fi
 
 echo "$NORMALIZED"

package/hooks/store-memory.sh

@@ -268,15 +268,12 @@ if [[ -z "$CONTENT" ]] || [[ "$CONTENT" == "null" ]]; then
   exit 0
 fi
 
-# Classify content to determine storage value
+# Content quality check moved to API level
+# Hook no longer filters content locally - API will return HTTP 400 if rejected
+# This ensures users see the helpful error message from the API
+# Still classify for bead detection and metadata tagging
 CONTENT_CLASS=$(classify_content "$CONTENT" "$EVENT_TYPE")
-log "Content classification: $CONTENT_CLASS"
-
-# Skip low-value content (routine commands, noise)
-if [[ "$CONTENT_CLASS" == "routine" ]] || [[ "$CONTENT_CLASS" == "noise" ]]; then
-  log "Skipping $CONTENT_CLASS content (low value)"
-  exit 0
-fi
+log "Content classification: $CONTENT_CLASS (advisory only, not filtering)"
 
 # Truncate very long content (max 10000 chars)
 if [[ ${#CONTENT} -gt 10000 ]]; then
@@ -348,7 +345,9 @@ log "Storing memory to unified endpoint: $NEXUS_API_URL/api/memory"
 # Headers: X-Company-ID, X-App-ID, X-User-ID (required for tenant context)
 # Run async (background) with timeout
 if [[ "$VERBOSE" == "1" ]]; then
-  # Sync mode with output for debugging
+  # VERBOSE MODE: Synchronous with detailed output
+  log "Storing memory (sync mode)..."
+
   RESPONSE=$(curl -s -w "\n%{http_code}" -X POST "$NEXUS_API_URL/api/memory" \
     -H "Content-Type: application/json" \
     -H "Authorization: Bearer $NEXUS_API_KEY" \
@@ -356,29 +355,56 @@ if [[ "$VERBOSE" == "1" ]]; then
     -H "X-App-ID: $APP_ID" \
     -H "X-User-ID: ${USER:-unknown}" \
     -d "$PAYLOAD" \
-    --max-time 5 2>&1)
+    --max-time 10 2>&1)
 
   HTTP_CODE=$(echo "$RESPONSE" | tail -n1)
   BODY=$(echo "$RESPONSE" | sed '$d')
 
-  log "Response code: $HTTP_CODE"
-  log "Response body: $BODY"
-
   if [[ "$HTTP_CODE" == "200" ]] || [[ "$HTTP_CODE" == "201" ]]; then
-    # Extract and save memory ID for causal chaining
     MEMORY_ID=$(echo "$BODY" | jq -r '.memoryId // .data.memoryId // empty' 2>/dev/null)
+
     if [[ -n "$MEMORY_ID" ]]; then
+      echo "✅ Memory stored: $MEMORY_ID" >&2
       save_memory_id "$MEMORY_ID"
-      log "Saved memory ID for causal chain: $MEMORY_ID"
+    else
+      echo "⚠️  Memory stored but no ID returned" >&2
     fi
 
-    # Log entity extraction results if available
-    ENTITIES=$(echo "$BODY" | jq -r '.entities // .data.entities_extracted // empty' 2>/dev/null)
-    if [[ -n "$ENTITIES" ]] && [[ "$ENTITIES" != "null" ]]; then
-      log "Entities extracted: $ENTITIES"
+    # Show triage decision if available
+    TRIAGE=$(echo "$BODY" | jq -r '.triageDecision.contentType // ""' 2>/dev/null)
+    if [[ -n "$TRIAGE" ]] && [[ "$TRIAGE" != "null" ]]; then
+      echo "   Content type: $TRIAGE" >&2
     fi
+
+  elif [[ "$HTTP_CODE" == "400" ]]; then
+    # QUALITY FILTER REJECTION - show API's error message
+    echo "❌ Storage rejected (HTTP 400)" >&2
+    ERROR_MSG=$(echo "$BODY" | jq -r '.error.message // .message // ""' 2>/dev/null)
+    REASON=$(echo "$BODY" | jq -r '.error.reason // ""' 2>/dev/null)
+
+    if [[ -n "$ERROR_MSG" ]]; then
+      echo "   Error: $ERROR_MSG" >&2
+    fi
+    if [[ -n "$REASON" ]] && [[ "$REASON" != "null" ]]; then
+      echo "   Reason: $REASON" >&2
+    fi
+    exit 1  # Non-blocking error
+
+  elif [[ "$HTTP_CODE" == "401" ]]; then
+    # AUTHENTICATION FAILURE - clear actionable message
+    echo "❌ Authentication failed (HTTP 401)" >&2
+    echo "   Check NEXUS_API_KEY environment variable" >&2
+    echo "   Current key prefix: ${NEXUS_API_KEY:0:10}..." >&2
+    exit 1  # Non-blocking error
+
   else
-    log_error "Failed to store memory (HTTP $HTTP_CODE)"
+    # OTHER ERROR - show details
+    echo "❌ Storage failed (HTTP $HTTP_CODE)" >&2
+    ERROR_MSG=$(echo "$BODY" | jq -r '.error.message // .message // ""' 2>/dev/null)
+    if [[ -n "$ERROR_MSG" ]]; then
+      echo "   Error: $ERROR_MSG" >&2
+    fi
+    exit 1  # Non-blocking error
   fi
 else
   # Async mode with memory ID tracking for normal operation

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adverant/nexus-memory-skill",
-  "version": "2.3.1",
+  "version": "2.3.3",
   "description": "Claude Code skill for persistent memory via Nexus GraphRAG - store and recall memories across all sessions and projects",
   "main": "SKILL.md",
   "type": "module",