gitbroker 0.6.4 → 0.6.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b6c41dc04d47bc8bfbad2ccb322b3a952b58e4957c2e282c25cb9075e58e83b2
-  data.tar.gz: 6d2f1b148005c310ed3538fe6fc4349cbaa38821bd1aa0f2ef25117477f146fa
+  metadata.gz: 6ef35d4ad926d8851302c8ed0c6c5be6e0dd70f5207fac41d6fc4f77b02e46b0
+  data.tar.gz: 1c6afa6a9db5082116ecaab590dff94ce6d3ca1527d0459fc9ddf468d77c7abb
 SHA512:
-  metadata.gz: 271fb887ec812d171902d32a53ed027bd3107194c19e0d67f14a1e5d22352f143add06b8090f2ba9ddd57e5853e35c773541234ad50d4666a064d49a9683513f
-  data.tar.gz: 9a259bd5c66d1836d85b23f7e017a4011ff28722cd068f5eaeaf746eb7be8e0d2d6dbf578067cd529ed9a8858e9970fd1712d6feabc39a505afca5d9f3bd69bc
+  metadata.gz: a784b73eceee92147c09a4a28de801bc40b3b6f93f2d2323c12dcad7c2e7da04901372fd6319464f3a494f4e82c3a12adfc653dcae329e22af3b130be6b54916
+  data.tar.gz: c9aace186cb3e2a3309182e791e53ad3c1553ee167e50951cdab83b35aad0eb19c2f0f1f1bc82d847e1936e11278708d27c6cda0017d7e9854b76630e19b7344

data/lib/gitbroker/branch_publisher.rb

@@ -14,16 +14,17 @@ module Gitbroker
 
     REF = %r{\A\w[\w./-]{0,200}\z}
 
-    def initialize(dir:, head_ref:, expected_sha:, base_ref:, runner: Runner.new)
+    def initialize(dir:, head_ref:, expected_sha:, base_ref:, remote: "origin", runner: Runner.new)
       @dir = dir
       @head_ref = head_ref.to_s
       @expected = expected_sha.to_s
       @base_ref = base_ref.to_s
+      @remote = remote.to_s
       @runner = runner
     end
 
     def publish!
-      return failed("No branch name to push.") unless valid_ref?(@head_ref) && valid_ref?(@base_ref)
+      return failed("No branch name to push.") unless valid_ref?(@head_ref) && valid_ref?(@base_ref) && valid_ref?(@remote)
       return failed("No starting head to protect.") unless @expected.match?(/\A[0-9a-f]{7,40}\z/)
       return failed("A rebase or merge is still in progress in the worktree: finish it first.") if in_progress?
       return failed("The worktree has uncommitted changes.") unless git("status", "--porcelain").strip.empty?
@@ -31,16 +32,16 @@ module Gitbroker
       head = git("rev-parse", "HEAD").strip
       return Result.new(:nothing, "Nothing to push: the branch did not change.") if head.start_with?(@expected) || @expected.start_with?(head)
 
-      git("fetch", "--quiet", "origin", @base_ref)
-      unless @runner.git_success?(@dir, "merge-base", "--is-ancestor", "origin/#{@base_ref}", "HEAD")
-        return failed("The branch is not on top of origin/#{@base_ref}: nothing pushed.")
+      git("fetch", "--quiet", @remote, @base_ref)
+      unless @runner.git_success?(@dir, "merge-base", "--is-ancestor", "#{@remote}/#{@base_ref}", "HEAD")
+        return failed("The branch is not on top of #{@remote}/#{@base_ref}: nothing pushed.")
       end
-      unless git("rev-list", "--merges", "#{@expected}..HEAD").strip.empty?
+      unless git("rev-list", "--merges", "#{@remote}/#{@base_ref}..HEAD").strip.empty?
         return failed("The branch contains merge commits; it must be a clean rebase: nothing pushed.")
       end
 
-      git("push", lease_option, "origin", "HEAD:refs/heads/#{@head_ref}")
-      Result.new(:pushed, "Pushed #{@head_ref} rebased onto origin/#{@base_ref}.")
+      git("push", lease_option, @remote, "HEAD:refs/heads/#{@head_ref}")
+      Result.new(:pushed, "Pushed #{@head_ref} rebased onto #{@remote}/#{@base_ref}.")
     rescue Error => e
       failed("Push failed: #{e.message.lines.first.to_s.strip}")
     end

data/lib/gitbroker/cli.rb

@@ -319,6 +319,7 @@ module Gitbroker
         parser.on("--head-ref REF") { options[:head_ref] = _1 }
         parser.on("--expected SHA") { options[:expected_sha] = _1 }
         parser.on("--base REF") { options[:base_ref] = _1 }
+        parser.on("--remote NAME") { options[:remote] = _1 }
       end.parse!(@argv)
       result = BranchPublisher.new(**options).publish!
       @out.puts result.message

data/lib/gitbroker/git_context.rb

@@ -61,6 +61,16 @@ module Gitbroker
       end
     end
 
+    # The remote that is `full_name` on GitHub (origin first when several are): a checkout found through its `github` remote
+    # has an origin that is something else, so fetching and pushing must name the remote, never assume origin.
+    def remote_for(dir, full_name)
+      wanted = full_name.to_s.downcase
+      matches = remote_urls(dir).select { |_name, url| RepoScanner.full_name_from(url).to_s.downcase == wanted }.map(&:first)
+      (matches.include?("origin") ? "origin" : matches.find { _1.match?(REMOTE_NAME) }) || "origin"
+    end
+
+    REMOTE_NAME = /\A\w[\w.-]{0,60}\z/
+
     def repository_from(remotes)
       remotes.sort_by { |name, _url| name == "origin" ? 0 : 1 }.lazy.filter_map { |_name, url| RepoScanner.full_name_from(url) }.first
     end

data/lib/gitbroker/neutralizer.rb

@@ -17,13 +17,13 @@ module Gitbroker
       @runner = runner
     end
 
-    def neutralize!(dir, repo_path:, number:, base_ref:)
+    def neutralize!(dir, repo_path:, number:, base_ref:, remote: "origin")
       raise Error, "Invalid base branch #{base_ref.inspect}" unless Worktree.valid_ref?(base_ref)
       raise Error, "Invalid PR number #{number.inspect}" unless number.to_s.match?(/\A[1-9]\d{0,9}\z/)
 
       @dir = File.realpath(dir)
       base = "refs/gitbroker/base-#{number}"
-      @runner.git(repo_path, "fetch", "--quiet", "origin", "+refs/heads/#{base_ref}:#{base}")
+      @runner.git(repo_path, "fetch", "--quiet", remote, "+refs/heads/#{base_ref}:#{base}")
 
       removed = paths(@runner.git(@dir, "ls-files", "-z", "--", *REMOVE))
       removed.each { delete(_1) }

data/lib/gitbroker/task_runner.rb

@@ -58,7 +58,7 @@ module Gitbroker
                               cross_repository: task["cross_repository"] == true)
       # Agents report real paths (macOS: /private/var, not /var); progress paths are matched against this form.
       dir = File.realpath(dir)
-      restored = task["untrusted"] ? @neutralizer.neutralize!(dir, repo_path:, number: task["pr_number"], base_ref: base_ref(task)) : []
+      restored = task["untrusted"] ? @neutralizer.neutralize!(dir, repo_path:, number: task["pr_number"], base_ref: base_ref(task), remote: GitContext.remote_for(repo_path, task["repo"])) : []
       task = task.merge("prompt" => prompt_for(task))
       agent = Agents.for(@config, worktree: dir)
       say(id, "#{task["kind"]} on #{task["repo"]}##{task["pr_number"]}: starting (#{agent.name}, #{task["mode"] == "terminal" ? "terminal" : "autonomous"})")
@@ -143,7 +143,8 @@ module Gitbroker
     def publish_branch(task, outcome, dir)
       return outcome unless publishes_branch?(task)
 
-      result = BranchPublisher.new(dir:, head_ref: task["head_ref"], expected_sha: task["head_sha"], base_ref: base_ref(task)).publish!
+      result = BranchPublisher.new(dir:, head_ref: task["head_ref"], expected_sha: task["head_sha"], base_ref: base_ref(task),
+                                     remote: GitContext.remote_for(dir, task["repo"])).publish!
       say(task["task_id"], result.message)
       return outcome unless result.failed?
 
@@ -155,7 +156,7 @@ module Gitbroker
       return unless publishes_branch?(task)
 
       [ RbConfig.ruby, ServiceInstaller.launcher, "publish-branch", "--dir", dir, "--head-ref", task["head_ref"].to_s,
-        "--expected", task["head_sha"].to_s, "--base", base_ref(task) ]
+        "--expected", task["head_sha"].to_s, "--base", base_ref(task), "--remote", GitContext.remote_for(dir, task["repo"]) ]
     end
 
     def publishes_branch?(task) = task["kind"] == "update_branch" && task["cross_repository"] != true && !task["read_only"]

data/lib/gitbroker/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Gitbroker
-  VERSION = "0.6.4"
+  VERSION = "0.6.6"
 end

data/lib/gitbroker/worktree.rb

@@ -24,7 +24,8 @@ module Gitbroker
       validate!(full_name:, number:, head_sha:, kind:)
       dir = File.join(@root, full_name.downcase.tr("/", "-"), "pr-#{number}-#{kind}")
 
-      @runner.git(repo_path, "fetch", "--quiet", "origin", "+refs/pull/#{number}/head:refs/gitbroker/pr-#{number}")
+      remote = GitContext.remote_for(repo_path, full_name)
+      @runner.git(repo_path, "fetch", "--quiet", remote, "+refs/pull/#{number}/head:refs/gitbroker/pr-#{number}")
       unless @runner.git_success?(repo_path, "cat-file", "-e", "#{head_sha}^{commit}")
         raise Error, "Head #{head_sha[0, 7]} is not in the fetched PR; the PR may have moved. Start the task again from the card."
       end
@@ -41,7 +42,7 @@ module Gitbroker
         @runner.git(repo_path, "worktree", "add", "--quiet", *target, dir, head_sha)
       end
       if branch
-        cross_repository ? drop_upstream(repo_path, local) : track_upstream(repo_path, local, head_ref)
+        cross_repository ? drop_upstream(repo_path, local) : track_upstream(repo_path, local, head_ref, remote)
       end
       dir
     end
@@ -93,11 +94,11 @@ module Gitbroker
       raise Error, "Branch #{local} has commits that are not on the PR head; push or delete it, then start the task again."
     end
 
-    def track_upstream(repo_path, local, head_ref)
+    def track_upstream(repo_path, local, head_ref, remote)
       return unless self.class.valid_ref?(head_ref)
-      return unless @runner.git_success?(repo_path, "fetch", "--quiet", "origin", "+refs/heads/#{head_ref}:refs/remotes/origin/#{head_ref}")
+      return unless @runner.git_success?(repo_path, "fetch", "--quiet", remote, "+refs/heads/#{head_ref}:refs/remotes/#{remote}/#{head_ref}")
 
-      @runner.git(repo_path, "branch", "--quiet", "--set-upstream-to=origin/#{head_ref}", local)
+      @runner.git(repo_path, "branch", "--quiet", "--set-upstream-to=#{remote}/#{head_ref}", local)
     end
 
     def drop_upstream(repo_path, local)

data/plugin/.claude-plugin/plugin.json

@@ -1,5 +1,5 @@
 {
   "name": "gitbroker",
-  "version": "0.6.4",
+  "version": "0.6.6",
   "description": "Skills for working on Git Broker agent tasks: the gitbroker MCP tools, write rules and report_task."
 }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: gitbroker
 version: !ruby/object:Gem::Version
-  version: 0.6.4
+  version: 0.6.6
 platform: ruby
 authors:
 - Yuri Sidorov (@newstler)