sugarjar 1.1.1 → 1.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bc16295007c8698083a182a521c68481b3cb421644def0115aa04286867dd1eb
-  data.tar.gz: c901d92e5d7d5287472f52cf56a83d29a5b1b4b0248232278b36e1706bc7117d
+  metadata.gz: 4263a42ebb922645e2e0735c3ca33ad94df880fb01f881b2cacb602d733c533e
+  data.tar.gz: 616fb99c535c491324c1dcc4906b37c8cb424566b62dc64377c61b9a5ef5fd49
 SHA512:
-  metadata.gz: 6d64d758a56feac59e5617544cdde17341c804a8831524c49222f6397e47e886ed16f42280d60584a5231d279410839bdacebda66827ae77bb115addb24f3b6d
-  data.tar.gz: e65b75d83d852906c713f00d6424a76897177f39eb1f2eaa0b5d17896b6d2fb288de3d1cf0946f5587f5674819a074782279ecdb26094c8038d221fcd7ceab0f
+  metadata.gz: 1d918daf7096d1889bf72ee0eca74ef473234ed1ed46e4275418028e9a32c9284b4412ff202a932a3cc43c777bbf55d1455ce8fd8805c965184e4d2bc740e527
+  data.tar.gz: 0657a6629d97623d40f68d0ffe7a3e0522c951d9b22dc0c7b4b918d78eb27b026ab68d347b1c2fa68661536eceef9cbf3de3ee82b4ea4381da2675f1d34b3f2d

data/bin/sj

@@ -91,6 +91,24 @@ parser = OptionParser.new do |opts|
     options['log_level'] = level
   end
 
+  opts.on(
+    '--[no-]pr-autofill',
+    'When creating a PR, auto fill the title & description from the commit ' +
+    'if there is a single commit and if we are using "gh". [default: true]',
+  ) do |autofill|
+    options['pr_autofill'] = autofill
+  end
+
+  opts.on(
+    '--[no-]pr-autostack',
+    'When creating a PR, if this is a subfeature, should we make it a ' +
+    'PR on the PR for the parent feature. If not specified, we prompt ' +
+    'when this happens, when true always do this, when false never do ' +
+    'this. Only applicable when usiing "gh".',
+  ) do |autostack|
+    options['pr_autostack'] = autostack
+  end
+
   opts.on('--[no-]use-color', 'Enable color. [default: true]') do |color|
     options['color'] = color
   end
@@ -112,10 +130,10 @@ COMMANDS:
               Same as "amend" but without changing the message. Alias for
               "git commit --amend --no-edit".
 
-  bclean
-              If safe, delete the current branch. Unlike "git branch -d",
-              bclean can handle squash-merged branches. Think of it as
-              a smarter "git branch -d".
+  bclean [<branch>]
+              If safe, delete the current branch (or the specified branch).
+              Unlike "git branch -d", bclean can handle squash-merged branches.
+              Think of it as a smarter "git branch -d".
 
   bcleanall
               Walk all branches, and try to delete them if it's safe. See
@@ -127,7 +145,7 @@ COMMANDS:
   br
               Verbose branch list. An alias for "git branch -v".
 
-  feature
+  feature, f <branch_name>
               Create a "feature" branch. It's morally equivalent to
               "git checkout -b" except it defaults to creating it based on
               some form of 'master' instead of your current branch. In order
@@ -177,11 +195,17 @@ COMMANDS:
               A smart wrapper to "git push" that runs whatever is defined in
               "on_push" in .sugarjar.yml, and only pushes if they succeed.
 
+  subfeature, sf <feature>
+              An alias for 'sj feature <feature> <current_branch>'
+
   unit
               Run any unitests configured in .sugarjar.yaml.
 
-  up
-              Rebase the current branch on upstream/master or origin/master.
+  up [<branch>]
+              Rebase the current branch (or specified branch) intelligently.
+              In most causes this will check for a main (or master) branch on
+              upstream, then origin. If a branch explicitly tracks something
+              else, then that will be used, instead.
 
   upall
               Same as "up", but for all branches.

data/lib/sugarjar/commands.rb

@@ -23,6 +23,8 @@ class SugarJar
       @repo_config = SugarJar::RepoConfig.config
       SugarJar::Log.debug("Repoconfig: #{@repo_config}")
       @color = options['color']
+      @pr_autofill = options['pr_autofill']
+      @pr_autostack = options['pr_autostack']
       @feature_prefix = options['feature_prefix']
       @checks = {}
       @main_branch = nil
@@ -43,19 +45,31 @@ class SugarJar
       name = fprefix(name)
       die("#{name} already exists!") if all_local_branches.include?(name)
       base ||= most_main
-      base_pieces = base.split('/')
-      git('fetch', base_pieces[0]) if base_pieces.length > 1
+      # If our base is a local branch, don't try to parse it for a remote name
+      unless all_local_branches.include?(base)
+        base_pieces = base.split('/')
+        git('fetch', base_pieces[0]) if base_pieces.length > 1
+      end
       git('checkout', '-b', name, base)
+      git('branch', '-u', base)
       SugarJar::Log.info(
         "Created feature branch #{color(name, :green)} based on " +
         color(base, :green),
       )
     end
+    alias f feature
+
+    def subfeature(name)
+      assert_in_repo
+      SugarJar::Log.debug("Subfature: #{name}")
+      feature(name, current_branch)
+    end
+    alias sf subfeature
 
     def bclean(name = nil)
       assert_in_repo
       name ||= current_branch
-      name = fprefix(name) unless all_local_branches.include?(name)
+      name = fprefix(name)
       if clean_branch(name)
         SugarJar::Log.info("#{name}: #{color('reaped', :green)}")
       else
@@ -100,7 +114,7 @@ class SugarJar
       # and then add any featureprefix, and if _that_ is a branch
       # name, replace the last arguement with that
       name = args.last
-      bname = fprefix(name) unless all_local_branches.include?(name)
+      bname = fprefix(name)
       if all_local_branches.include?(bname)
         SugarJar::Log.debug("Featurepefixing #{name} -> #{bname}")
         args[-1] = bname
@@ -133,11 +147,14 @@ class SugarJar
 
     alias sl smartlog
 
-    def up
+    def up(branch = nil)
       assert_in_repo
+      branch ||= current_branch
+      branch = fprefix(branch)
       # get a copy of our current branch, if rebase fails, we won't
       # be able to determine it without backing out
       curr = current_branch
+      git('checkout', branch)
       result = gitup
       if result['so'].error?
         backout = ''
@@ -156,6 +173,8 @@ class SugarJar
         SugarJar::Log.info(
           "#{color(current_branch, :green)} rebased on #{result['base']}",
         )
+        # go back to where we were if we rebased a different branch
+        git('checkout', curr) if branch != curr
       end
     end
 
@@ -302,6 +321,7 @@ class SugarJar
     def smartpullrequest(*args)
       assert_in_repo
       assert_common_main_branch
+
       if dirty?
         SugarJar::Log.warn(
           'Your repo is dirty, so I am not going to create a pull request. ' +
@@ -309,9 +329,38 @@ class SugarJar
         )
         exit(1)
       end
+
       if gh?
+        curr = current_branch
+        base = tracked_branch
+        if @pr_autofill
+          num_commits = git(
+            'rev-list', '--count', curr, "^#{base}"
+          ).stdout.strip.to_i
+          if num_commits > 1
+            SugarJar::Log.debug(
+              "Not using --fill because there are #{num_commits} commits",
+            )
+          else
+            SugarJar::Log.info('Autofilling in PR from commit message')
+            args.unshift('--fill')
+          end
+        end
+        if subfeature?(base)
+          # nil is prompt, true is always, false is never
+          if @pr_autostack.nil?
+            $stdout.print(
+              'It looks like this is a subfeature, would you like to base ' +
+              "this PR on #{base}? [y/n] ",
+            )
+            ans = $stdin.gets.strip
+            args += ['--base', base] if %w{Y y}.include?(ans)
+          elsif @pr_autostack
+            args += ['--base', base]
+          end
+        end
         SugarJar::Log.trace("Running: gh pr create #{args.join(' ')}")
-        system(which('gh'), 'pr', 'create', '--fill', *args)
+        system(which('gh'), 'pr', 'create', *args)
       else
         SugarJar::Log.trace("Running: hub pull-request #{args.join(' ')}")
         system(which('hub'), 'pull-request', *args)
@@ -341,7 +390,7 @@ class SugarJar
 
       src = "origin/#{current_branch}"
       fetch('origin')
-      diff = git('diff', src).stdout
+      diff = git('diff', "..#{src}").stdout
       return unless diff && !diff.empty?
 
       puts "Will merge the following suggestions:\n\n#{diff}"
@@ -369,6 +418,9 @@ class SugarJar
     def fprefix(name)
       return name unless @feature_prefix
 
+      return name if name.start_with?(@feature_prefix)
+      return name if all_local_branches.include?(name)
+
       newname = "#{@feature_prefix}#{name}"
       SugarJar::Log.debug(
         "Munging feature name: #{name} -> #{newname} due to feature prefix",
@@ -712,11 +764,15 @@ class SugarJar
     end
 
     def all_local_branches
-      branches = []
-      git('branch', '--format', '%(refname)').stdout.lines.each do |line|
-        branches << branch_from_ref(line.strip)
+      git(
+        'branch', '--format', '%(refname)'
+      ).stdout.lines.map do |line|
+        branch_from_ref(line.strip)
       end
-      branches
+    end
+
+    def all_remotes
+      git('remote').stdout.lines.map(&:strip)
     end
 
     def safe_to_clean(branch)
@@ -794,14 +850,28 @@ class SugarJar
       SugarJar::Log.debug('Fetching upstream')
       fetch_upstream
       curr = current_branch
-      base = tracked_branch
+      # this isn't a hash, it's a named param, silly rubocop
+      # rubocop:disable Style/HashSyntax
+      base = tracked_branch(fallback: false)
+      # rubocop:enable Style/HashSyntax
+      unless base
+        SugarJar::Log.info(
+          'The brach we were tracking is gone, resetting tracking to ' +
+          most_main,
+        )
+        git('branch', '-u', most_main)
+        base = most_main
+      end
+      # If this is a subfeature based on a local branch which has since
+      # been deleted, 'tracked branch' will automatically return <most_main>
+      # so we don't need any special handling for that
       if !MAIN_BRANCHES.include?(curr) && base == "origin/#{curr}"
         SugarJar::Log.warn(
           "This branch is tracking origin/#{curr}, which is probably your " +
           'downstream (where you push _to_) as opposed to your upstream ' +
           '(where you pull _from_). This means that "sj up" is probably ' +
           'rebasing on the wrong thing and doing nothing. You probably want ' +
-          'to do a "git branch -u upstream".',
+          "to do a 'git branch -u #{most_main}'.",
         )
       end
       SugarJar::Log.debug('Rebasing')
@@ -812,6 +882,12 @@ class SugarJar
       }
     end
 
+    # determine if this branch is based on another local branch (i.e. is a
+    # subfeature). Used to figure out of we should stack the PR
+    def subfeature?(base)
+      all_local_branches.reject { |x| x == most_main }.include?(base)
+    end
+
     def rebase_in_progress?
       # for rebase without -i
       rebase_file = git('rev-parse', '--git-path', 'rebase-apply').stdout.strip
@@ -821,12 +897,13 @@ class SugarJar
       File.exist?(rebase_file) || File.exist?(rebase_merge_file)
     end
 
-    def tracked_branch
+    def tracked_branch(fallback: true)
       s = git_nofail(
         'rev-parse', '--abbrev-ref', '--symbolic-full-name', '@{u}'
       )
       if s.error?
-        most_main
+        fallback ? most_main : nil
+
       else
         s.stdout.strip
       end
@@ -844,9 +921,7 @@ class SugarJar
     def upstream
       return @remote if @remote
 
-      s = git('remote')
-
-      remotes = s.stdout.lines.map(&:strip)
+      remotes = all_remotes
       SugarJar::Log.debug("remotes is #{remotes}")
       if remotes.empty?
         @remote = nil

data/lib/sugarjar/config.rb

@@ -9,6 +9,8 @@ class SugarJar
       'github_cli' => 'auto',
       'github_user' => ENV.fetch('USER'),
       'fallthru' => true,
+      'pr_autofill' => true,
+      'pr_autostack' => nil,
     }.freeze
 
     def self._find_ordered_files

data/lib/sugarjar/version.rb

@@ -1,3 +1,3 @@
 class SugarJar
-  VERSION = '1.1.1'.freeze
+  VERSION = '1.1.2'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sugarjar
 version: !ruby/object:Gem::Version
-  version: 1.1.1
+  version: 1.1.2
 platform: ruby
 authors:
 - Phil Dibowitz
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-02-13 00:00:00.000000000 Z
+date: 2024-04-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: deep_merge