captive-release 0.12.0 → 0.13.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6a8fd7809824a87b08805de31f9ddc40af157b6ba41e0b76e19f90edd4c9ade2
-  data.tar.gz: 33dff63edd19f1889f5fa014271eff8640b6cba8effc7937ab9332fe2bf14316
+  metadata.gz: f164e66eb747a40700e3fb75708097cc3284dbaf52cd40719bcf72689d6c11df
+  data.tar.gz: 464925ca718160f9968ee03d1e4611c7a5ccc3fa5f3b076a7f29de96507696fd
 SHA512:
-  metadata.gz: 9cfbc9751d8b2607e05512a7120f3773d3553526786e57d47a45d557e97bda793e26f09be493f015c3caaf982329f5363576a61e14399ecef1fa4157d913b352
-  data.tar.gz: 6a114be8ba3248b560619b8e345e1ec8e9a9d9b009b73d7723be59ba743bae25ba519f1f9baf533e772e16608741d0a77fdc1ab42007a359a6a38b49b266ffee
+  metadata.gz: 7ebed4eca5ded03e043ec51adc60da07141435c8aa368ae2caea3185605b9a5f86becb1e3bf1ccec66a6df42505da10d20f89d029a85217d60526b41fbc543cd
+  data.tar.gz: 788821f98efa0e1a3afbcf97a8307370fda1b6131c8769250bd5a157b8b929bb4cec13d3b6b3c19a8820209ff719ffcd0a080befb0820df0d0ac22b47c95aae2

data/lib/captive_release/build_dispatch_run.rb

@@ -21,6 +21,9 @@ module CaptiveRelease
 
       @terminal.done Messages.t(:build_dispatched)
       @terminal.say "     #{url}\n"
+    rescue WorkflowIntrouvable => e
+      @terminal.say @terminal.error(e.message)
+      exit 1
     end
 
     private

data/lib/captive_release/git_commands.rb

@@ -41,9 +41,17 @@ module CaptiveRelease
     # Le tag part avant HEAD : au moment où le webhook du push sur main arrive, le tag existe
     # déjà. Dans l'autre sens, demander « ce sha porte-t-il un tag v* ? » répond non presque
     # toujours, et la même release se construit deux fois.
+    #
+    # Le refus de HEAD se rend plutôt qu'il ne lève : une branche protégée refuse le push
+    # direct, et la release se replie alors sur une pull request.
     def self.push(version)
       Shell.run! "git push origin v#{version}"
-      Shell.run! "git push origin HEAD"
+      Shell.run? "git push origin HEAD"
+    end
+
+    # Pousse HEAD sous un autre nom sans quitter la branche courante.
+    def self.push_branch(nom)
+      Shell.run! "git push origin HEAD:refs/heads/#{Shell.escape(nom)}"
     end
   end
 end

data/lib/captive_release/git_release.rb

@@ -7,8 +7,9 @@ module CaptiveRelease
   # change. Chaque étape est annoncée au fur et à mesure : une release qui s'arrête doit
   # laisser voir jusqu'où elle est allée.
   class GitRelease
-    def initialize(terminal:)
-      @terminal = terminal
+    def initialize(terminal:, token_provider:)
+      @terminal       = terminal
+      @token_provider = token_provider
     end
 
     def call(version, paths)
@@ -17,8 +18,27 @@ module CaptiveRelease
       @terminal.done %(git commit "🔖 Set version v#{version}")
       GitCommands.tag(version)
       @terminal.done "git tag v#{version}"
-      GitCommands.push(version)
-      @terminal.done "git push + tag"
+      return @terminal.done("git push + tag") if GitCommands.push(version)
+
+      pull_request(version)
+    end
+
+    private
+
+    # Le tag est déjà poussé et reste sur le commit de version : la release GitHub peut
+    # suivre, et un merge commit le ramène sur la branche de base.
+    def pull_request(version)
+      @terminal.say "  #{@terminal.warn(Messages.t(:push_refused))}"
+      base    = GitCommands.current_branch
+      branche = "release/v#{version}"
+      GitCommands.push_branch(branche)
+      @terminal.done "git push origin #{branche}"
+
+      jeton = @token_provider.call
+      raise Messages.t(:pull_request_needs_token, branch: branche) unless jeton
+
+      @terminal.done ReleasePullRequest.create(version: version, base: base, head: branche,
+                                               remote_url: GitCommands.remote_url, token: jeton)
     end
   end
 end

data/lib/captive_release/github_api.rb

@@ -25,6 +25,15 @@ module CaptiveRelease
 
     def get(chemin) = analyser(@http.get(adresse(chemin)), "200")
 
+    # Un 404 dit l'absence ; tout autre échec reste une erreur plutôt qu'une absence supposée.
+    def existe?(chemin)
+      reponse = @http.get(adresse(chemin))
+      return false if reponse.code == "404"
+
+      analyser(reponse, "200")
+      true
+    end
+
     def post(chemin, corps, attendu:)
       analyser(@http.post(adresse(chemin), JSON.generate(corps)), attendu)
     end

data/lib/captive_release/github_dispatch.rb

@@ -3,6 +3,10 @@
 require_relative "github_api"
 
 module CaptiveRelease
+  # Le workflow demandé n'existe pas sur la branche par défaut du dépôt, seule que GitHub
+  # consulte pour accepter un dispatch.
+  class WorkflowIntrouvable < StandardError; end
+
   # Déclenche un workflow GitHub Actions. Elle compose ce qu'elle demande ; l'API, son
   # authentification et ses échecs vivent dans GithubApi.
   class GithubDispatch
@@ -16,10 +20,21 @@ module CaptiveRelease
     end
 
     def dispatch(workflow_file:, ref:, inputs:)
+      exiger(workflow_file)
       @api.post("actions/workflows/#{workflow_file}/dispatches",
                 { "ref" => ref, "inputs" => inputs }, attendu: "204")
 
       @api.depot.actions_url
     end
+
+    private
+
+    def exiger(workflow_file)
+      return if @api.existe?("actions/workflows/#{workflow_file}")
+
+      depot = @api.depot
+      raise WorkflowIntrouvable, Messages.t(:workflow_not_found, workflow: workflow_file,
+                                                                 repo: "#{depot.owner}/#{depot.repo}")
+    end
   end
 end

data/lib/captive_release/locales/en.yml

@@ -73,6 +73,9 @@ en:
     dispatch_branch: 'Branch:   %{value}'
     dispatching: Dispatching...
     build_dispatched: Build dispatched!
+    workflow_not_found: |-
+      Workflow %{workflow} not found on %{repo}.
+      Add .github/workflows/%{workflow} to the default branch, or fix build_workflow in the configuration.
     dirty_tree: The working tree is not clean.
     dirty_tree_hint: Commit or stash your changes before releasing.
     current_version: 'Current version: %{version}'
@@ -102,6 +105,9 @@ en:
     updating_version_files: Updating version files...
     refreshing_lockfile: Refreshing Gemfile.lock...
     running_git: Running git commands...
+    push_refused: "⚠️   Direct push refused (protected branch?) - falling back to a pull request."
+    pull_request_needs_token: 'GITHUB_TOKEN required to open the pull request: branch %{branch} is pushed, open it by hand.'
+    pull_request_body: "Direct push to %{base} was refused. Merge with a **merge commit**: a squash or a rebase would change the sha, and tag v%{version} would no longer be on %{base}."
     creating_github_release: Creating GitHub release...
     no_token_skip_release: "⚠️   GITHUB_TOKEN not set - skipping GitHub release creation."
     released: Version v%{version} released successfully!

data/lib/captive_release/locales/fr.yml

@@ -73,6 +73,9 @@ fr:
     dispatch_branch: 'Branche :  %{value}'
     dispatching: Déclenchement...
     build_dispatched: Build déclenché !
+    workflow_not_found: |-
+      Workflow %{workflow} introuvable sur %{repo}.
+      Ajoutez .github/workflows/%{workflow} sur la branche par défaut, ou corrigez build_workflow dans la configuration.
     dirty_tree: L'arbre de travail n'est pas propre.
     dirty_tree_hint: Commitez ou remisez vos modifications avant de releaser.
     current_version: 'Version actuelle : %{version}'
@@ -102,6 +105,9 @@ fr:
     updating_version_files: Mise à jour des fichiers de version...
     refreshing_lockfile: Rafraîchissement du Gemfile.lock...
     running_git: Exécution des commandes git...
+    push_refused: "⚠️   Push direct refusé (branche protégée ?) - repli sur une pull request."
+    pull_request_needs_token: 'GITHUB_TOKEN requis pour ouvrir la pull request : la branche %{branch} est poussée, ouvrez-la à la main.'
+    pull_request_body: "Le push direct sur %{base} a été refusé. Mergez avec un **merge commit** : un squash ou un rebase changerait le sha, et le tag v%{version} ne serait plus sur %{base}."
     creating_github_release: Création de la release GitHub...
     no_token_skip_release: "⚠️   GITHUB_TOKEN absent - la release GitHub n'est pas créée."
     released: Version v%{version} publiée avec succès !

data/lib/captive_release/release_command.rb

@@ -22,7 +22,7 @@ module CaptiveRelease
       ReleaseConfirmation.new(terminal: @terminal).verifier!(suggestion, courante, prochaine) if confirmation
 
       VersionFilesWriting.new(terminal: @terminal).call(prochaine, @config.version_files)
-      GitRelease.new(terminal: @terminal).call(prochaine, chemins_a_commiter)
+      GitRelease.new(terminal: @terminal, token_provider: @token_provider).call(prochaine, chemins_a_commiter)
       publication.call(prochaine)
       @terminal.say "\n🎉  #{@terminal.bold(Messages.t(:released, version: prochaine))}\n"
     end

data/lib/captive_release/release_pull_request.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+require_relative "github_api"
+
+module CaptiveRelease
+  # Ouvre la pull request d'une release que la branche protégée a refusée en push direct.
+  # Elle compose ce qu'elle demande ; l'API, son authentification et ses échecs vivent dans
+  # GithubApi.
+  class ReleasePullRequest
+    def self.create(version:, base:, head:, remote_url:, token:, api: nil)
+      api ||= GithubApi.new(remote_url: remote_url, token: token)
+      new(api).create(version, base: base, head: head)
+    end
+
+    def initialize(api)
+      @api = api
+    end
+
+    def create(version, base:, head:)
+      @api.post("pulls",
+                { "title" => "🔖 Set version v#{version}", "head" => head, "base" => base,
+                  "body" => Messages.t(:pull_request_body, base: base, version: version), },
+                attendu: "201")["html_url"]
+    end
+  end
+end

data/lib/captive_release/shell.rb

@@ -11,6 +11,9 @@ module CaptiveRelease
       system(commande) || raise(Messages.t(:command_failed, command: commande))
     end
 
+    # Pour une commande dont l'échec est une issue attendue, que l'appelant traite.
+    def self.run?(commande) = system(commande)
+
     def self.capture(commande) = `#{commande}`.strip
 
     def self.capture_lines(commande) = capture(commande).lines.map(&:strip).reject(&:empty?)

data/lib/captive_release/version.rb

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

data/lib/captive_release.rb

@@ -43,6 +43,7 @@ require_relative "captive_release/menu"
 require_relative "captive_release/version_choice"
 require_relative "captive_release/gemfile_lock_refresh"
 require_relative "captive_release/exclusion_input"
+require_relative "captive_release/release_pull_request"
 require_relative "captive_release/git_release"
 require_relative "captive_release/changes_since"
 require_relative "captive_release/release_publication"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: captive-release
 version: !ruby/object:Gem::Version
-  version: 0.12.0
+  version: 0.13.0
 platform: ruby
 authors:
 - Captive Studio
@@ -63,6 +63,7 @@ files:
 - lib/captive_release/release_overview.rb
 - lib/captive_release/release_preconditions.rb
 - lib/captive_release/release_publication.rb
+- lib/captive_release/release_pull_request.rb
 - lib/captive_release/repo_files.rb
 - lib/captive_release/ruby_gems_version.rb
 - lib/captive_release/semver.rb