rails-informant 0.4.7 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e1238247be3c3dcefa39e50f4907cd23f9e10282fe32cbf480c2bb4d5edd1daa
-  data.tar.gz: 51df461568d6e54a3860c52d552f5d365531c65f80005ba025facb0b3ee54e13
+  metadata.gz: ebe80aab76837ee09279a0831b344bca9072a873e48b70197868e54a7b5a6ceb
+  data.tar.gz: f637b4e6f460bbb982a2c3f51b49d87cb3fcaa8e6fd2c3547a694fc0f4fa70fc
 SHA512:
-  metadata.gz: 02c696395075fd1bfc4b895a916812c512797c64b2031f5d03110b1c636de066043ff26c562112056af1f961da0dffea0e2d6409b1bd5ace298b2f2d7d09ec9d
-  data.tar.gz: 1e1c83d7898f5f7f96c9ad4d07682feb67cba4dc915a119d4cc16cda3035b0c7cfae5e890f7479c8593116aaa09ee416d23cfb6a0e0335a972957853f8c9bebe
+  metadata.gz: '08bf946e26795a53d035667fad63f91ccf5dda9497710a09d9e187d82ee35323ed64397b4761a8af6e81896d915df47e4a6b3f94777e5150fb3250262db6c5ba'
+  data.tar.gz: 8fe9bb913213f18310a90ff232f11c1317246b231aeec3c9afba80c4e253d017dff9e41a829d90df5beee43f9dade333eb4317c24831340b1fdeabf50e068b02

data/lib/generators/rails_informant/skill/templates/informant-alerts.sh

@@ -1,5 +1,6 @@
 #!/usr/bin/env bash
-# Informant: Check for unresolved production errors on Claude Code startup.
+# Informant: On the first Claude Code prompt of a session, check for unresolved
+# production errors — unless that first prompt is the /informant command.
 # Requires: curl, jq
 # Env vars: INFORMANT_PRODUCTION_URL, INFORMANT_PRODUCTION_TOKEN
 #           INFORMANT_PRODUCTION_PATH_PREFIX (optional, default: /informant)
@@ -11,9 +12,28 @@ set -euo pipefail
 [[ -z "${INFORMANT_PRODUCTION_TOKEN:-}" ]] && exit 0
 [[ "$INFORMANT_PRODUCTION_URL" == https://* ]] || exit 0
 
-# Silent exit if jq is not installed
+# Silent exit if jq is not installed (needed to parse the hook payload)
 command -v jq >/dev/null 2>&1 || exit 0
 
+# UserPromptSubmit delivers a JSON payload on stdin carrying session_id and prompt.
+payload=$(cat)
+session_id=$(printf '%s' "$payload" | jq -r '.session_id // empty' 2>/dev/null || true)
+
+# Run at most once per session, on the first prompt: a marker keyed by session_id
+# records that this session was handled, so later prompts short-circuit here before
+# parsing the prompt or doing network work. Require a token-shaped session_id (it is
+# interpolated into the path below) and stay silent when we can't record the marker,
+# rather than re-alerting on every later prompt of the session.
+[[ "$session_id" =~ ^[A-Za-z0-9_-]+$ ]] || exit 0
+marker="${TMPDIR:-/tmp}/rails-informant-alert-${session_id}"
+[[ -e "$marker" ]] && exit 0
+: > "$marker" 2>/dev/null || exit 0
+
+# Stay quiet for the whole session when the first prompt is the /informant command —
+# the user is already triaging errors, so the startup alert would be redundant.
+prompt=$(printf '%s' "$payload" | jq -r '.prompt // empty' 2>/dev/null || true)
+[[ "$prompt" =~ ^/informant($|[[:space:]]) ]] && exit 0
+
 path_prefix="${INFORMANT_PRODUCTION_PATH_PREFIX:-/informant}"
 url="${INFORMANT_PRODUCTION_URL}${path_prefix}/api/v1/status"
 
@@ -25,8 +45,9 @@ response=$(curl -s -f \
   "$url" <<< "Authorization: Bearer ${INFORMANT_PRODUCTION_TOKEN}" \
   2>/dev/null) || exit 0
 
-# Parse unresolved count
-unresolved=$(echo "$response" | jq -r '.unresolved_count // 0') || exit 0
+# Parse unresolved count (silent exit on a missing or non-numeric value)
+unresolved=$(echo "$response" | jq -r '.unresolved_count // 0' 2>/dev/null) || exit 0
+[[ "$unresolved" =~ ^[0-9]+$ ]] || exit 0
 [[ "$unresolved" -eq 0 ]] && exit 0
 
 # Format error summary

data/lib/generators/rails_informant/skill_generator.rb

@@ -39,20 +39,20 @@ module RailsInformant
       if File.exist?(settings_path)
         existing = JSON.parse(File.read(settings_path))
         existing["hooks"] ||= {}
-        existing["hooks"]["SessionStart"] ||= []
+        existing["hooks"]["UserPromptSubmit"] ||= []
 
-        already_registered = existing["hooks"]["SessionStart"].any? do |entry|
+        already_registered = existing["hooks"]["UserPromptSubmit"].any? do |entry|
           entry["hooks"]&.any? { it["command"] == hook_command }
         end
 
         unless already_registered
-          existing["hooks"]["SessionStart"] << session_start_hook(hook_command)
+          existing["hooks"]["UserPromptSubmit"] << user_prompt_submit_hook(hook_command)
         end
 
         create_file ".claude/settings.json", JSON.pretty_generate(existing) + "\n", force: true
       else
         create_file ".claude/settings.json", JSON.pretty_generate(
-          "hooks" => { "SessionStart" => [ session_start_hook(hook_command) ] }
+          "hooks" => { "UserPromptSubmit" => [ user_prompt_submit_hook(hook_command) ] }
         ) + "\n"
       end
     rescue JSON::ParserError
@@ -66,7 +66,7 @@ module RailsInformant
       say "  Created .mcp.json"
       say "  Created .claude/skills/informant/SKILL.md"
       say "  Created .claude/hooks/informant-alerts.sh"
-      say "  Created .claude/settings.json (SessionStart hook)"
+      say "  Created .claude/settings.json (UserPromptSubmit hook)"
       say ""
       say "Next step — set env vars so the MCP server and startup alerts can reach your app.", :yellow
       say "Add to your .envrc (or export manually):"
@@ -82,9 +82,8 @@ module RailsInformant
 
     private
 
-    def session_start_hook(command)
+    def user_prompt_submit_hook(command)
       {
-        "matcher" => "startup",
         "hooks" => [
           { "type" => "command", "command" => command, "timeout" => 10 }
         ]

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails-informant
 version: !ruby/object:Gem::Version
-  version: 0.4.7
+  version: 0.5.0
 platform: ruby
 authors:
 - Daniel López Prat