captive-release 0.11.0 → 0.12.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 90b817eaab2a7cab41857f499ee8cac77d460bbf073f9074016b96621a5df027
-  data.tar.gz: b66c7485162d298b516b2dfb8e6e90bc7ae13d2a88fa59ca12254dacbadea8a4
+  metadata.gz: 6a8fd7809824a87b08805de31f9ddc40af157b6ba41e0b76e19f90edd4c9ade2
+  data.tar.gz: 33dff63edd19f1889f5fa014271eff8640b6cba8effc7937ab9332fe2bf14316
 SHA512:
-  metadata.gz: 9c75bc8285c685ef3c2b2969de5259e1078ccb45f76849f503177c1891352ab38b9705125c4a9787681c931540d7fe3187dc92f1c0184a71e1876ee380659675
-  data.tar.gz: 4fb8ee0668d2b2848817ba8a4eccaf45f38dec0dd216eaf410e08ef7373edfcef20d7cd7c76563bbdef2629240ec0e1b9e54a18962bd2383307b0eb3551b100a
+  metadata.gz: 9cfbc9751d8b2607e05512a7120f3773d3553526786e57d47a45d557e97bda793e26f09be493f015c3caaf982329f5363576a61e14399ecef1fa4157d913b352
+  data.tar.gz: 6a114be8ba3248b560619b8e345e1ec8e9a9d9b009b73d7723be59ba743bae25ba519f1f9baf533e772e16608741d0a77fdc1ab42007a359a6a38b49b266ffee

data/lib/captive_release/direct_commits.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+
+module CaptiveRelease
+  # Complète les notes générées par GitHub, qui ne listent que les PR mergées : un commit
+  # poussé directement sur la branche n'y apparaît jamais, même quand il porte la feature
+  # qui justifie le bump.
+  class DirectCommits
+    COMMIT_VERSION = /\A🔖 Set version /
+
+    def initialize(api)
+      @api = api
+    end
+
+    # Sans plage, il n'y a pas de tag précédent : lister les commits reviendrait à lister
+    # l'historique entier.
+    def completer(notes)
+      plage = notes.plage or return notes
+      lignes = commits(*plage).map { ligne(_1) }
+      lignes.empty? ? notes : notes.ajouter("## Commits directs\n#{lignes.join("\n")}\n")
+    end
+
+    private
+
+    def commits(depuis, jusqu_a)
+      @api.get("compare/#{depuis}...#{jusqu_a}")["commits"].reject do |commit|
+        titre(commit).match?(COMMIT_VERSION) || !@api.get("commits/#{commit["sha"]}/pulls").empty?
+      end
+    end
+
+    def ligne(commit)
+      auteur = commit.dig("author", "login")
+      "* #{titre(commit)}#{" by @#{auteur}" if auteur} in #{commit["sha"]}"
+    end
+
+    def titre(commit) = commit.dig("commit", "message")[/.*/]
+  end
+end

data/lib/captive_release/github_api.rb

@@ -23,15 +23,22 @@ module CaptiveRelease
       @http  = http || GithubHttp.new(token: token)
     end
 
+    def get(chemin) = analyser(@http.get(adresse(chemin)), "200")
+
     def post(chemin, corps, attendu:)
-      reponse = @http.post(URI("#{HOTE}/#{depot.path(chemin)}"), JSON.generate(corps))
+      analyser(@http.post(adresse(chemin), JSON.generate(corps)), attendu)
+    end
+
+    private
+
+    def adresse(chemin) = URI("#{HOTE}/#{depot.path(chemin)}")
+
+    def analyser(reponse, attendu)
       refuser(reponse) unless reponse.code == attendu
 
       reponse.body.to_s.empty? ? nil : JSON.parse(reponse.body)
     end
 
-    private
-
     def refuser(reponse)
       raise Messages.t(:github_api_error, code: reponse.code, message: GithubErrorMessage.from(reponse.body.to_s))
     end

data/lib/captive_release/github_release.rb

@@ -1,6 +1,8 @@
 # frozen_string_literal: true
 
 require_relative "github_api"
+require_relative "direct_commits"
+require_relative "release_notes"
 
 module CaptiveRelease
   # Crée la release GitHub d'une version. Elle compose ce qu'elle demande ; l'API, son
@@ -9,7 +11,6 @@ module CaptiveRelease
     # GitHub refuse (422) un corps de release plus long. Sans tag précédent, les notes
     # générées couvrent tout l'historique et dépassent vite cette limite.
     CORPS_MAX = 125_000
-    COUPURE   = "\n\n…"
 
     def self.create(version:, remote_url:, token:, api: nil)
       api ||= GithubApi.new(remote_url: remote_url, token: token)
@@ -35,21 +36,7 @@ module CaptiveRelease
 
     def notes(tag)
       corps = @api.post("releases/generate-notes", { "tag_name" => tag }, attendu: "200")["body"].to_s
-      corps.length > CORPS_MAX ? tronquer(corps) : corps
-    end
-
-    # Coupe à une fin de ligne et garde le lien Full Changelog final, qui mène à la liste complète.
-    def tronquer(corps)
-      tete, pied = separer_pied(corps)
-      place = CORPS_MAX - COUPURE.length - pied.length
-      coupe = tete[0, place]
-      coupe = coupe[0, coupe.rindex("\n")] if coupe.include?("\n")
-      "#{coupe}#{COUPURE}#{pied}"
-    end
-
-    def separer_pied(corps)
-      index = corps.rindex("\n**Full Changelog**")
-      index ? [ corps[0, index], "\n#{corps[index..]}" ] : [ corps, "" ]
+      DirectCommits.new(@api).completer(ReleaseNotes.new(corps)).tronquer(CORPS_MAX).to_s
     end
   end
 end

data/lib/captive_release/release_notes.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+module CaptiveRelease
+  # Le texte des notes d'une release, découpé autour de son pied : le lien Full Changelog,
+  # qui mène à la liste complète et doit survivre à tout ajout ou toute coupe.
+  class ReleaseNotes
+    PIED        = /(?:\n|\A)\*\*Full Changelog\*\*/
+    COMPARAISON = %r{/compare/(?<depuis>\S+?)\.\.\.(?<jusqu_a>\S+)}
+    COUPURE     = "\n\n…"
+
+    def initialize(corps)
+      index = corps.rindex(PIED)
+      @tete, @pied = index ? [ corps[0, index], corps[index..] ] : [ corps, "" ]
+    end
+
+    # Les tags qui bornent les notes ; nil sans tag précédent.
+    def plage = @pied.match(COMPARAISON)&.captures
+
+    def ajouter(section) = ReleaseNotes.new("#{@tete.rstrip}\n\n#{section}#{@pied}")
+
+    # Coupe à une fin de ligne, pour ne pas laisser un lien ou un émoji à moitié.
+    def tronquer(max)
+      return self if to_s.length <= max
+
+      coupe = @tete[0, max - COUPURE.length - @pied.length - 1]
+      coupe = coupe[0, coupe.rindex("\n")] if coupe.include?("\n")
+      ReleaseNotes.new("#{coupe}#{COUPURE}\n#{@pied}")
+    end
+
+    def to_s = "#{@tete}#{@pied}"
+  end
+end

data/lib/captive_release/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module CaptiveRelease
-  VERSION = "0.11.0"
+  VERSION = "0.12.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: captive-release
 version: !ruby/object:Gem::Version
-  version: 0.11.0
+  version: 0.12.0
 platform: ruby
 authors:
 - Captive Studio
@@ -33,6 +33,7 @@ files:
 - lib/captive_release/config_file.rb
 - lib/captive_release/config_loader.rb
 - lib/captive_release/config_writer.rb
+- lib/captive_release/direct_commits.rb
 - lib/captive_release/exclusion_input.rb
 - lib/captive_release/framework_detector.rb
 - lib/captive_release/gemfile_lock_refresh.rb
@@ -58,6 +59,7 @@ files:
 - lib/captive_release/prerelease_iteration.rb
 - lib/captive_release/release_command.rb
 - lib/captive_release/release_confirmation.rb
+- lib/captive_release/release_notes.rb
 - lib/captive_release/release_overview.rb
 - lib/captive_release/release_preconditions.rb
 - lib/captive_release/release_publication.rb