thumbgate 1.4.1 → 1.4.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "thumbgate",
-  "version": "1.4.1",
+  "version": "1.4.2",
   "description": "ThumbGate: self-improving agent governance for engineering teams. Three-tier approval routing (block/approve/log), shared enforcement, CI gates, and audit trails. Every mistake becomes a prevention rule. PreToolUse hooks, Thompson Sampling, SQLite+FTS5 lesson DB, and LanceDB vector search.",
   "homepage": "https://thumbgate-production.up.railway.app",
   "repository": {

package/scripts/lesson-inference.js

@@ -303,7 +303,26 @@ function getStatusbarLessonData() {
   if (!recent) return { hasLesson: false, text: null, link: null };
 
   const normalizedLesson = stripLessonPrefix(recent.lesson || '');
-  const truncated = normalizedLesson.length > 48 ? normalizedLesson.slice(0, 45) + '...' : normalizedLesson;
+
+  // Distill to actionable insight: prefer structured rule action, then
+  // whatToChange, then the lesson text itself. Raw user feedback
+  // ("are you sure?", "is this working?") is not useful in a statusbar.
+  let displayText = normalizedLesson;
+  if (recent.structuredRule && recent.structuredRule.action && recent.structuredRule.action.description) {
+    displayText = recent.structuredRule.action.description;
+  } else if (recent.whatToChange || recent.what_to_change) {
+    displayText = String(recent.whatToChange || recent.what_to_change);
+  }
+
+  // Clean up: strip noise prefixes, collapse whitespace
+  displayText = displayText
+    .replace(/^CRITICAL ERROR - User frustrated:\s*/i, '')
+    .replace(/^thumbs?\s*(up|down)\s*:?\s*/i, '')
+    .replace(/\s+/g, ' ')
+    .trim();
+
+  // Truncate to 60 chars (enough to be readable, short enough for statusbar)
+  const truncated = displayText.length > 60 ? displayText.slice(0, 57) + '...' : displayText;
 
   return {
     hasLesson: true,

package/scripts/statusline.sh

@@ -162,10 +162,15 @@ DASHBOARD_LINK="$DASHBOARD_LABEL"
 LESSONS_LINK="$LESSONS_LABEL"
 LATEST_LESSON_LINK=""
 if [ -n "$LESSON_LABEL" ]; then
+  # Only include link if it's a real URL (not localhost)
+  _DISPLAY_LINK="$LESSON_LINK"
+  case "$_DISPLAY_LINK" in
+    *localhost*|*127.0.0.1*) _DISPLAY_LINK="" ;;
+  esac
   if [ -n "$LESSON_TEXT" ]; then
-    LATEST_LESSON_LINK="$(inline_link "$LESSON_LINK" "${LESSON_LABEL}: ${LESSON_TEXT}")"
+    LATEST_LESSON_LINK="$(inline_link "$_DISPLAY_LINK" "${LESSON_LABEL}: ${LESSON_TEXT}")"
   else
-    LATEST_LESSON_LINK="$(inline_link "$LESSON_LINK" "$LESSON_LABEL")"
+    LATEST_LESSON_LINK="$(inline_link "$_DISPLAY_LINK" "$LESSON_LABEL")"
   fi
 fi