deployangel 0.1.20 → 0.1.21

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 59eed3c6629b0dc5b81cbec2636583d50f0038c2f51af8011108205a1a23eb8c
-  data.tar.gz: d10e3a967bed17b69b854a5b74deba72b66ad4e204b3c007bd1958fc81e1ca66
+  metadata.gz: 2a115a5a35d86c5edce65dad1cdf3947833416ea66f47c69da18b85de3047153
+  data.tar.gz: ec954e62605c6452d3e374896085343cb6a12910e0b821794342a2e0a6dc0426
 SHA512:
-  metadata.gz: cecef6e5f9328a9b72877c16bbd51388d67a5673b98a75a56c78e6916f5bf99fd033baaf9056f101b25cd36553d0e6033a253800f571ecfab3724655f1301901
-  data.tar.gz: cb8460812b9f04eaf761e23282a644c62f4de9aaca20fcf42e1bc81478f898e096675fd1bdfc67e70aef60ac3053bce969de7310539b9261b036c262dae70bd1
+  metadata.gz: 4c18ca61ecff8936bdfa7f4a5900fd27b96c2a787094e504593fae78f62eaee3c7d4ea11e74811b619313c1a726bce8e4644f5a6319c687450ff81629acdd2a5
+  data.tar.gz: 2754b3cc315c626df0dc57de4723721caa668f69b7b523a0ca4eaecaf4f84e2f2ec7f9f40f366be34f9a1989ff3c9b4650f1d02dc81b976218d2b3daf5e3968c

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 0.1.21 (2026-10-09)
+
+- `deployangel exercise` stops requesting a page after its first 404, 405,
+  or 410, since a route table can list pages an app doesn't serve, and sends
+  that page's share to the pages that answered.
+
 ## 0.1.20 (2026-10-09)
 
 - `deployangel exercise --url=<production URL>` sends a release's exercise

data/README.md

@@ -471,8 +471,9 @@ path parameters, spreading any request shortfall across them (at most 200
 requests, about 5 a second, as `DeployAngel-Exercise`), then records what it
 sent on the release, where the page lists it under "Exercised from your
 side". It skips and names routes that change data or need a path
-parameter. A "CLI & coding agents" token can run it. `--dry-run` shows what
-it would send.
+parameter, and stops requesting a page after its first 404, sending its
+share to the pages that answered. A "CLI & coding agents" token can run
+it. `--dry-run` shows what it would send.
 
 ### Suggested instructions for your coding agent
 

data/lib/deployangel/exerciser.rb

@@ -16,6 +16,9 @@ module DeployAngel
     TIMEOUT = 10
     # Stop when the app isn't answering, rather than send the rest into it.
     MAX_CONSECUTIVE_ERRORS = 5
+    # A route table can list pages an app doesn't serve, like the edit page
+    # of a resource that has none. One answer like this is enough to know.
+    NOT_SERVED = [ 404, 405, 410 ].freeze
     # Rails' :id and *path, Django's <int:pk>, FastAPI's {id}.
     PARAMETER = /\/[:*{<]/
 
@@ -51,27 +54,34 @@ module DeployAngel
       [ targets, skipped ]
     end
 
+    # A route whose first request says it isn't served gets no more; its
+    # share goes to the routes that answered.
     def run(exercise_plan)
       targets, skipped = plan(exercise_plan)
-      routes = []
-      errors_in_a_row = 0
-      sent = 0
+      @statuses = Hash.new { |hash, key| hash[key] = Hash.new(0) }
+      @sent = 0
+      @errors_in_a_row = 0
+      leftover = 0
+      answered = []
       targets.each do |target|
-        statuses = Hash.new(0)
-        target.count.times do
-          break if errors_in_a_row >= MAX_CONSECUTIVE_ERRORS
-
-          @sleeper.call(INTERVAL) if sent.positive?
-          status = @requester.call(url_for(target.path))
-          sent += 1
-          kind = status ? "#{status / 100}xx" : "error"
-          statuses[kind] += 1
-          errors_in_a_row = kind == "error" ? errors_in_a_row + 1 : 0
+        target.count.times do |i|
+          break if unreachable?
+
+          status = send_request(target)
+          if i.zero? && NOT_SERVED.include?(status)
+            leftover += target.count - 1
+            break
+          end
         end
-        requests = statuses.values.sum
-        routes << { "key" => target.key, "requests" => requests, "statuses" => statuses.to_h } if requests.positive?
+        answered << target if @statuses[target.key].keys.intersect?(%w[2xx 3xx])
+      end
+      leftover.times { |i| send_request(answered[i % answered.size]) unless unreachable? } if answered.any?
+
+      routes = targets.filter_map do |target|
+        statuses = @statuses.fetch(target.key, nil) or next
+        { "key" => target.key, "requests" => statuses.values.sum, "statuses" => statuses.to_h }
       end
-      Result.new(routes: routes, skipped: skipped, sent: sent, unreachable: errors_in_a_row >= MAX_CONSECUTIVE_ERRORS)
+      Result.new(routes: routes, skipped: skipped, sent: @sent, unreachable: unreachable?)
     end
 
     def url_for(path)
@@ -82,6 +92,20 @@ module DeployAngel
     end
 
     private
+      def send_request(target)
+        @sleeper.call(INTERVAL) if @sent.positive?
+        status = @requester.call(url_for(target.path))
+        @sent += 1
+        kind = status ? "#{status / 100}xx" : "error"
+        @statuses[target.key][kind] += 1
+        @errors_in_a_row = kind == "error" ? @errors_in_a_row + 1 : 0
+        status
+      end
+
+      def unreachable?
+        @errors_in_a_row >= MAX_CONSECUTIVE_ERRORS
+      end
+
       # Requests the release is short of overall, spread across the routes,
       # or sent to the home page when the plan names no route to send them to.
       def spread_shortfall(targets, requests)

data/lib/deployangel/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module DeployAngel
-  VERSION = "0.1.20"
+  VERSION = "0.1.21"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: deployangel
 version: !ruby/object:Gem::Version
-  version: 0.1.20
+  version: 0.1.21
 platform: ruby
 authors:
 - Jordan Owens