ui_guardrails 1.7.0 → 1.7.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bcb56bb52cb2fe3e2d0bd155eaf5b8563177588b53d34af57f4d125653cd6f47
-  data.tar.gz: 5803d7480e71fe72cc6964bd424333fb888709d73104f215430a520d09cc4b49
+  metadata.gz: 32e83a5c20961feae0830c519f4bf0b3363bcd4774d4644bab49053ce02c8cd5
+  data.tar.gz: 8fbaaa46298c2ea2805b2b291354d36cf521fd6f11f948290d62e2796bbafb24
 SHA512:
-  metadata.gz: 18bb8c26caf6971a74cc2a0d585acbcf57fbd27b1f703bc0fcedfa4d038724d0362623addcf06723e152f2f4cd4234829a88ba720dfcf0042a79c027067c27bb
-  data.tar.gz: b559b93cc20faa99f6af5b7a806117ea5925d6dacee2df064cac8e2fe4499d7763277ce0c6a47944f4c77dd7373d091fd01bba65e21a09b3e1dcd56193628d71
+  metadata.gz: 88fe2925e7787c6b0a63861d142f9d53e5e832395c7b22e7515400954f9a16f57dab73c2fa318b3df516ee5fa12af3e104604bd63125a08eb403eb3ed93f083d
+  data.tar.gz: 9ed0ec91a1310da006bf525c05163cec82e730c17838d22ba6e5c2ed2ab7607241541769dda5c60f9a7e2681b587bc5747d2116ddbef156e952bd6f079d8cbae

data/README.md

@@ -318,7 +318,7 @@ It reports only what the agent introduced. A template is compared with its versi
 | Cursor | `.cursor/hooks.json` | `stop` and `sessionStart` only: Cursor's after-edit output doesn't reliably reach the model, so the stop check covers every changed template. Cursor also runs `.claude/settings.json` hooks; the hook recognises Cursor's input and doesn't run twice. |
 | Gemini CLI | `.gemini/settings.json` | `AfterTool` (`write_file`, `replace`), `AfterAgent`, `SessionStart` |
 
-The hook never breaks the agent's turn: if anything goes wrong it prints `{}`, explains on stderr, and exits 0. Other gems add to the same events with `Guardrails::Hooks.register(name, contributor)`, shipped at `lib/guardrails/hook_contributors/<name>.rb` so the hook finds them without booting the app.
+The hook never breaks the agent's turn: if anything goes wrong it prints `{}`, explains on stderr, and exits 0. The installed command ends in `|| echo '{}'`, so the same holds when the hook can't start at all. That happens on a branch whose Gemfile doesn't have the gem, since the agent's config usually isn't tracked by git and outlives the branch. Running the install again replaces an earlier install's hooks instead of adding new ones beside them. Other gems add to the same events with `Guardrails::Hooks.register(name, contributor)`, shipped at `lib/guardrails/hook_contributors/<name>.rb` so the hook finds them without booting the app.
 
 ---
 

data/lib/guardrails/hooks/config_file.rb

@@ -18,7 +18,8 @@ module Guardrails
 
       # `entries` maps an event name to the list entries to add under
       # `hooks.<event>` (each a Hash in the agent's own shape). An entry
-      # counts as present when one with the same command is there.
+      # counts as present when one with the same command is there; an
+      # older form of the same Guardrails hook is replaced.
       # `extra` merges top-level keys that must be set (Cursor's
       # `version`). Returns true when the file changed.
       def add(entries, extra: {})
@@ -32,7 +33,14 @@ module Guardrails
           existing = (hooks[event] ||= [])
           raise Error, "#{@path}: `hooks.#{event}` isn't a list" unless existing.is_a?(Array)
 
-          list.each { |entry| existing << entry unless commands(existing).include?(command(entry)) }
+          list.each do |entry|
+            wanted = command(entry)
+            next if commands(existing).include?(wanted)
+
+            identity = wanted.to_s[OURS]
+            remove(existing, identity) if identity
+            existing << entry
+          end
         end
         return false if JSON.generate(config) == before
 
@@ -43,6 +51,24 @@ module Guardrails
 
       private
 
+      # An earlier install of the same hook, whatever its exact command:
+      # reinstalling replaces it rather than adding a second.
+      OURS = /guardrails-hook \w+ \w+/
+
+      # Takes out the hook with this identity and any group it leaves
+      # empty; the user's own hooks, in the same group or not, stay.
+      def remove(list, identity)
+        list.reject! do |entry|
+          next false unless entry.is_a?(Hash)
+          next true if entry["command"].to_s.include?(identity)
+          next false unless entry["hooks"].is_a?(Array)
+
+          before = entry["hooks"].size
+          entry["hooks"].reject! { |hook| hook.is_a?(Hash) && hook["command"].to_s.include?(identity) }
+          entry["hooks"].empty? && before.positive?
+        end
+      end
+
       def read
         return {} unless @path.file?
 

data/lib/guardrails/hooks/installer.rb

@@ -20,7 +20,10 @@ module Guardrails
       def call
         @adapters.map do |adapter|
           path = @root.join(adapter.config_path)
-          command = ->(event) { "#{adapter.command_prefix}#{executable} #{adapter.name} #{event}" }
+          # Falls back to `{}`, the answer every agent reads as "nothing to
+          # say", when the hook can't start at all: a branch without the
+          # gem in its Gemfile, or a missing project directory.
+          command = ->(event) { "(#{adapter.command_prefix}#{executable} #{adapter.name} #{event}) || echo '{}'" }
           changed = ConfigFile.new(path).add(adapter.config_entries(command), extra: adapter.config_extra)
           Change.new(agent: adapter.name, path: path.relative_path_from(@root).to_s, changed: changed)
         end

data/lib/guardrails/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Guardrails
-  VERSION = "1.7.0"
+  VERSION = "1.7.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ui_guardrails
 version: !ruby/object:Gem::Version
-  version: 1.7.0
+  version: 1.7.1
 platform: ruby
 authors:
 - John Athayde