browser_review_gate 0.2.0 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7511e93bcb04cb1c8afa751bb179553eb29e70f4d0173e5c4b9fe899009f83ed
-  data.tar.gz: 638a66f37095d8421b21528f5718e9a550644112cffdca8695b0978f7f9ff646
+  metadata.gz: '09d5a330baa46a948f70191ba4537ba0d5c7097ed8c6d89e497a5f872eb6e671'
+  data.tar.gz: 48b41886a6ebbd39bb67c62af4e1f19b5c9ae52ae3dea622dfdfe8c0708336f6
 SHA512:
-  metadata.gz: 3bf5fba978609414d2a0a191bee336805ec9c8ce7923c94c1f4e44bb135037b1252d44c549a7a29961575f7a9776da2161ecbd5e23028d54ac281c4e5878425d
-  data.tar.gz: 6b4350eb758928d89cb1ba3dd0b7da3e26b0c1d192c76b306bf30a600b2a9412b17066fcee02c9ce47717bc14a2869ead36bf5338e4d300bc73f2d760cb4fd62
+  metadata.gz: 19564cbe10913982e3bf22af716e9e147ce1a38ffa5dfa47035e0083cbecb9bc63f7f78789a97fe1776a1f51065a3ab900d1cdcf854f527f40ae4327f6029a8b
+  data.tar.gz: b33fe20d59959593e0bb9a881f047065a89700f6c9f30e8a3ffe7aa6d69574180ef7b714261301e05c399b6bb43afaffc8dd9dc8761c529ad256ac2dfc19f84f

data/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## 0.2.2
+
+- Requires json 2.7 or newer. Ruby 3.2 ships json 2.6, which ignores `script_safe`, so PR text in the
+  assessment prompt was not escaped there.
+- Links to the source code, changelog and issues.
+
+## 0.2.1
+
+- A reviewer who approved or asked for changes on the latest commit while their request was paused
+  is not asked again once the browser run is published. The notice names them instead. Someone whose
+  review is on an older commit, or who only commented, is still asked again.
+
 ## 0.2.0
 
 Re-run `browser-review-gate install` after updating: the workflow and the playbook change.

data/LICENSE.txt

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) 2026 JetRockets
+Copyright (c) 2026 Islam Gagiev
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

data/lib/browser_review_gate/gate.rb

@@ -81,20 +81,39 @@ module BrowserReviewGate
       @log.puts "Took back reviewer requests until browser verification is available."
     end
 
-    # Asks the people whose requests were taken back, now that review may proceed.
+    # Asks the people whose requests were taken back, now that review may proceed. Someone who
+    # reviewed the latest commit while the request was paused is not asked again.
     def restore_requests(number, pull_request, comments)
       waiting = paused(comments)
-      reviewers = waiting["reviewers"] - [ pull_request.dig("user", "login") ]
-      return if reviewers.empty? && waiting["teams"].empty?
+      waiting["reviewers"] -= [ pull_request.dig("user", "login") ]
+      return if waiting.values.all?(&:empty?)
 
-      @github.request_reviewers(number, reviewers: reviewers, team_reviewers: waiting["teams"])
-      write_notice(number, comments, "#{author(pull_request)}Review requested again from #{names(waiting)}: browser verification no longer holds it back.",
-        "reviewers" => [], "teams" => [])
-      @log.puts "Requested review again from #{names(waiting)}."
+      reviewed = reviewed_head(number, pull_request) & waiting["reviewers"]
+      asked = { "reviewers" => waiting["reviewers"] - reviewed, "teams" => waiting["teams"] }
+      unless asked.values.all?(&:empty?)
+        @github.request_reviewers(number, reviewers: asked["reviewers"], team_reviewers: asked["teams"])
+      end
+      write_notice(number, comments, "#{author(pull_request)}#{restored_notice(asked, reviewed)}", "reviewers" => [], "teams" => [])
+      @log.puts restored_notice(asked, reviewed)
     rescue Error => error
       @log.puts "Review was not requested again: #{error.message.lines.first.to_s.strip}"
     end
 
+    # People whose last review approved or asked for changes on the head commit.
+    def reviewed_head(number, pull_request)
+      head_sha = pull_request.fetch("head").fetch("sha")
+      last = @github.reviews(number).group_by { |review| review.dig("user", "login") }.transform_values(&:last)
+      last.select { |_login, review| review["commit_id"] == head_sha && %w[APPROVED CHANGES_REQUESTED].include?(review["state"]) }.keys
+    end
+
+    def restored_notice(asked, reviewed)
+      parts = []
+      parts << "Review requested again from #{names(asked)}: browser verification no longer holds it back." unless asked.values.all?(&:empty?)
+      parts << "#{reviewed.map { |name| Markdown.escape(name) }.join(", ")} already reviewed the latest commit, so " \
+        "#{reviewed.one? ? "was" : "were"} not asked again." if reviewed.any?
+      parts.join(" ")
+    end
+
     def notice_comment(comments)
       comments.reverse.find { |comment| comment.dig("user", "login") == Assessment::AUTHOR && comment["body"].to_s.match?(PAUSED) }
     end

data/lib/browser_review_gate/github.rb

@@ -92,6 +92,11 @@ module BrowserReviewGate
         input: { reviewers: reviewers, team_reviewers: team_reviewers })
     end
 
+    # Submitted reviews, oldest first.
+    def reviews(number)
+      pages("repos/#{repository}/pulls/#{number}/reviews")
+    end
+
     def request_reviewers(number, reviewers:, team_reviewers:)
       api("repos/#{repository}/pulls/#{number}/requested_reviewers", method: "POST",
         input: { reviewers: reviewers, team_reviewers: team_reviewers })

data/lib/browser_review_gate/version.rb

@@ -1,3 +1,3 @@
 module BrowserReviewGate
-  VERSION = "0.2.0"
+  VERSION = "0.2.2"
 end

metadata

@@ -1,18 +1,34 @@
 --- !ruby/object:Gem::Specification
 name: browser_review_gate
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.2
 platform: ruby
 authors:
-- JetRockets
+- Islam Gagiev
 bindir: exe
 cert_chain: []
 date: 1980-01-02 00:00:00.000000000 Z
-dependencies: []
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: json
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '2.7'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '2.7'
 description: CI decides whether a pull request needs a browser run and takes back
   reviewer requests until an AI coding agent (Claude Code, Cursor, Codex) has exercised
   the change in a browser and published what it observed. One command sets a project
   up.
+email:
+- omniacinis@gmail.com
 executables:
 - browser-review-gate
 extensions: []
@@ -51,9 +67,16 @@ files:
 - lib/browser_review_gate/templates/workflow.yml.erb
 - lib/browser_review_gate/version.rb
 - lib/generators/browser_review_gate/install/install_generator.rb
+homepage: https://github.com/Halvanhelv/browser_review_gate
 licenses:
 - MIT
-metadata: {}
+metadata:
+  allowed_push_host: https://rubygems.org
+  homepage_uri: https://github.com/Halvanhelv/browser_review_gate
+  source_code_uri: https://github.com/Halvanhelv/browser_review_gate
+  changelog_uri: https://github.com/Halvanhelv/browser_review_gate/blob/master/CHANGELOG.md
+  bug_tracker_uri: https://github.com/Halvanhelv/browser_review_gate/issues
+  rubygems_mfa_required: 'true'
 rdoc_options: []
 require_paths:
 - lib
@@ -68,7 +91,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.1
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Keeps human review of a pull request behind a browser run of its browser-visible
   changes.