rails-informant 0.4.6 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7a778d19578d0a29c371f4b5e8363044db91487e9703db86278944208de434d6
-  data.tar.gz: 7a60618611f8b9b3d3dcf13385fa720dfd434c6ad3512ed5f70ff072184bbbb7
+  metadata.gz: ebe80aab76837ee09279a0831b344bca9072a873e48b70197868e54a7b5a6ceb
+  data.tar.gz: f637b4e6f460bbb982a2c3f51b49d87cb3fcaa8e6fd2c3547a694fc0f4fa70fc
 SHA512:
-  metadata.gz: 8c0d57e79fb46d2e67ab05e34deb0737638710fdd69acafc25b7ea0b7346225251dcf450819a9d0345afb7961c2ff77bbad66de98e2db8110480b28b41457c10
-  data.tar.gz: 12f152c83d8d3b7018dad5aaf78e053c4504fc39849dbc0b0ebca9c5bafbb0128af0db53d54f9fb4790d65530225985c8ca0353f77e988ac23587936b8322cc5
+  metadata.gz: '08bf946e26795a53d035667fad63f91ccf5dda9497710a09d9e187d82ee35323ed64397b4761a8af6e81896d915df47e4a6b3f94777e5150fb3250262db6c5ba'
+  data.tar.gz: 8fe9bb913213f18310a90ff232f11c1317246b231aeec3c9afba80c4e253d017dff9e41a829d90df5beee43f9dade333eb4317c24831340b1fdeabf50e068b02

data/README.md

@@ -119,6 +119,13 @@ vanishing silently.
 Add your own classes with `config.ignored_exceptions`; it still walks the cause chain, so
 ignoring a class also ignores exceptions that wrap it.
 
+### Interactive Console
+
+Informant never captures errors raised in an interactive `rails console` session -- nothing
+is recorded and no notifications are sent. You see exceptions live at the prompt, so
+recording and alerting on them would only be noise. This is always on and not configurable.
+Errors from web requests, background jobs, and rake tasks are captured as normal.
+
 ### Silenced Blocks
 
 ```ruby

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 }
         ]

data/lib/rails_informant/error_recorder.rb

@@ -12,6 +12,9 @@ module RailsInformant
       end
 
       def record(error, severity: "error", context: {}, source: nil, env: nil)
+        # Never capture from an interactive console — the operator sees errors
+        # live, so recording and alerting on them is noise.
+        return if RailsInformant.console_mode?
         return unless RailsInformant.initialized?
         return if self_caused_error?(error)
 

data/lib/rails_informant.rb

@@ -132,6 +132,10 @@ module RailsInformant
       error.instance_variable_set(:@__rails_informant_captured, true) unless error.frozen?
     end
 
+    def console_mode?
+      defined?(Rails::Console)
+    end
+
     def server_mode?
       defined?(Rails::Server)
     end

metadata

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