kk-git 0.2.3 → 0.2.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 476a578e1aeea922d2d1c7835f42066e625a9cc976f5d19d713def1f8f11aa6c
-  data.tar.gz: dac594a319b48a7a089b58f45b5739fc9070cedb74ff113e64306c1c1faa202c
+  metadata.gz: 4b432c2d8b24c8a15fe3185d0f8d9c26bc07816a788a051933b7eaeb7b2a5409
+  data.tar.gz: f7666db73c822b09c1918e656437fc165f88ef13a6f23e1fd32fd0d3eb53c0f4
 SHA512:
-  metadata.gz: 44f2c4ddbff4478b973096815179d39a28711497e3f9bc2bd3b4461e6a7c44283565eb52e93b25588aca78a6ca9a95f57b170f0af0b5fafe6b50e6ae13e283cc
-  data.tar.gz: 54679b9c73d3e181db53ba4fccab282f3ae3878d07215663af441458e9215b6d935af1e41b735d9821e78baf74f4aad0e31db6f6d4209eb4c21f7b38f430c214
+  metadata.gz: a08f91f62c4252dee11fc5be7a414f75c7f1c96de5151b83c25a7c78bce8d33419623c35f9768c02f5e16c81485d8622e645f1dbf8af4298cc8bd80d2b93ccd7
+  data.tar.gz: 2167ef28d0b789fda42ac855834340e968ccaf973d0858281b7dbb8d8059128624bbb2b997100ee8d4ac6720c9356dc7d76b8989feb7011c36380d8046619944

data/exe/kk-git

@@ -169,6 +169,8 @@ def main_help
       KK_GIT_DEFAULT_TYPE  default type for code edits (default: chore)
       KK_GIT_PUSH_URL       push to explicit URL (bypass SSH remote)
       KK_GIT_PULL_URL       pull from explicit URL (bypass SSH remote)
+      KK_GIT_SKIP_FORMAT=1  skip pre-commit auto-format
+      KK_GIT_FORMAT_ALL=1   format all tracked files (default: changed only)
 
     Run `kk-git <command> --help` for command-specific options.
   HELP

data/lib/kk/git/auto_format.rb

@@ -0,0 +1,193 @@
+# frozen_string_literal: true
+
+require 'open3'
+
+module KKGit
+  # Best-effort format changed files before commit; skip missing tools.
+  module AutoFormat
+    Language = Struct.new(:id, :label, :extensions, :tools, keyword_init: true)
+
+    Tool = Struct.new(:name, :bin, :argv, keyword_init: true) do
+      def build(files, repo_dir:)
+        bin_path = AutoFormat.resolve_bin(bin, repo_dir: repo_dir)
+        return nil unless bin_path
+
+        args = argv.flat_map { |part| part == '%{files}' ? files : [part] }
+        [bin_path, *args]
+      end
+    end
+
+    LANGUAGES = [
+      Language.new(
+        id: :ruby,
+        label: 'Ruby',
+        extensions: %w[.rb],
+        tools: [
+          Tool.new(name: 'rubocop', bin: 'rubocop', argv: ['-A', '--force-exclusion', '%{files}'])
+        ]
+      ),
+      Language.new(
+        id: :python,
+        label: 'Python',
+        extensions: %w[.py],
+        tools: [
+          Tool.new(name: 'ruff', bin: 'ruff', argv: ['format', '%{files}']),
+          Tool.new(name: 'black', bin: 'black', argv: ['%{files}'])
+        ]
+      ),
+      Language.new(
+        id: :go,
+        label: 'Go',
+        extensions: %w[.go],
+        tools: [
+          Tool.new(name: 'gofmt', bin: 'gofmt', argv: ['-w', '%{files}'])
+        ]
+      ),
+      Language.new(
+        id: :bash,
+        label: 'Bash',
+        extensions: %w[.sh .bash],
+        tools: [
+          Tool.new(name: 'shfmt', bin: 'shfmt', argv: ['-w', '%{files}'])
+        ]
+      ),
+      Language.new(
+        id: :prettier,
+        label: 'JS/TS/Node',
+        extensions: %w[.js .jsx .mjs .cjs .ts .tsx .json .css .scss .less .html .md .yaml .yml],
+        tools: [
+          Tool.new(name: 'prettier', bin: 'prettier', argv: ['--write', '%{files}'])
+        ]
+      )
+    ].freeze
+
+    module_function
+
+    def enabled?
+      ENV['KK_GIT_SKIP_FORMAT'] != '1'
+    end
+
+    def format_all?
+      ENV['KK_GIT_FORMAT_ALL'] == '1'
+    end
+
+    def run!(repo_dir: '.', dry_run: GitOps.dry_run?)
+      return { ran: [], skipped: [], failed: [] } unless enabled?
+
+      files = target_files(repo_dir: repo_dir)
+      grouped = group_files(files)
+      return { ran: [], skipped: [], failed: [] } if grouped.empty?
+
+      ran = []
+      skipped = []
+      failed = []
+
+      grouped.each do |lang_id, paths|
+        language = LANGUAGES.find { |l| l.id == lang_id }
+        next unless language
+
+        tool = pick_tool(language, paths, repo_dir: repo_dir)
+        unless tool
+          skipped << "#{language.label}: no formatter (#{language.tools.map(&:name).join(' / ')})"
+          next
+        end
+
+        cmd = tool.build(paths, repo_dir: repo_dir)
+        label = "#{language.label} (#{tool.name})"
+        if dry_run
+          puts "[dry-run] format #{label}: #{cmd.join(' ')}"
+          ran << label
+          next
+        end
+
+        unless executable?(cmd&.first)
+          skipped << "#{label}: not found (#{tool.name})"
+          next
+        end
+
+        out, err, ok = run_command(cmd, repo_dir: repo_dir)
+        if ok
+          puts "Formatted #{label}: #{paths.length} file(s)"
+          ran << label
+        else
+          msg = err.to_s.strip.empty? ? out.to_s.strip : err.to_s.strip
+          warn "Format failed #{label} (continuing): #{msg.lines.first}"
+          failed << label
+        end
+      end
+
+      skipped.each { |line| puts "Format skip: #{line}" }
+
+      { ran: ran, skipped: skipped, failed: failed }
+    end
+
+    def target_files(repo_dir:)
+      format_all? ? all_tracked_files(repo_dir: repo_dir) : changed_files(repo_dir: repo_dir)
+    end
+
+    def changed_files(repo_dir:)
+      out, _err, ok = GitOps.run_cmd('git', 'status', '--porcelain', '-z', chdir: repo_dir)
+      return [] unless ok
+
+      out.split("\0").filter_map do |entry|
+        next if entry.nil? || entry.empty?
+
+        path = entry[3..]
+        next if path.nil? || path.empty? || path.end_with?('/')
+
+        path
+      end.uniq
+    end
+
+    def all_tracked_files(repo_dir:)
+      out, _err, ok = GitOps.run_cmd('git', 'ls-files', '-z', chdir: repo_dir)
+      return [] unless ok
+
+      out.split("\0").reject(&:empty?)
+    end
+
+    def group_files(files)
+      grouped = Hash.new { |h, k| h[k] = [] }
+      files.each do |path|
+        ext = File.extname(path)
+        language = LANGUAGES.find { |lang| lang.extensions.include?(ext) }
+        grouped[language.id] << path if language
+      end
+      grouped.transform_values { |paths| paths.uniq.sort }
+    end
+
+    def pick_tool(language, _files, repo_dir:)
+      language.tools.find do |tool|
+        resolve_bin(tool.bin, repo_dir: repo_dir)
+      end
+    end
+
+    def resolve_bin(name, repo_dir:)
+      path = which(name)
+      return path if executable?(path)
+
+      local = File.expand_path(File.join(repo_dir, 'node_modules', '.bin', name))
+      return local if executable?(local)
+
+      nil
+    end
+
+    def which(cmd)
+      out, _, ok = Open3.capture3('command', '-v', cmd)
+      return nil unless ok
+
+      path = out.strip
+      path.empty? ? nil : path
+    end
+
+    def executable?(path)
+      path && File.file?(path) && File.executable?(path)
+    end
+
+    def run_command(cmd, repo_dir:)
+      Open3.capture3(*cmd, chdir: repo_dir)
+    rescue Errno::ENOENT, Errno::EACCES => e
+      ['', e.message, false]
+    end
+  end
+end

data/lib/kk/git/git_ops.rb

@@ -1,8 +1,10 @@
 # frozen_string_literal: true
 
+require 'open3'
 require 'tempfile'
 require_relative 'git_runner'
 require_relative 'network_hints'
+require_relative 'auto_format'
 
 module KKGit
   # Git 仓库操作：供 Rake task 与 CLI 复用。
@@ -10,12 +12,12 @@ module KKGit
     class Error < StandardError; end
 
     SENSITIVE_PATH_PATTERNS = [
-      %r{\A\.env(\.|$)}i,
+      /\A\.env(\.|$)/i,
       /credentials/i,
-      %r{\.pem\z}i,
+      /\.pem\z/i,
       /id_rsa/i,
-      %r{\.key\z}i,
-      %r{\Asecrets?\.}i
+      /\.key\z/i,
+      /\Asecrets?\./i
     ].freeze
 
     # 仓库同步状态快照
@@ -58,10 +60,14 @@ module KKGit
         ENV.fetch('KK_GIT_REMOTE', 'origin')
       end
 
-      def branch(explicit: ENV['KK_GIT_BRANCH'], repo_dir: '.')
+      def branch(explicit: ENV.fetch('KK_GIT_BRANCH', nil), repo_dir: '.')
         explicit.to_s.strip.empty? ? current_branch(repo_dir: repo_dir) : explicit.to_s.strip
       end
 
+      def skip_format?
+        ENV['KK_GIT_SKIP_FORMAT'] == '1'
+      end
+
       def add_paths
         ENV.fetch('KK_GIT_ADD_PATHS', '.').split(/\s+/).reject(&:empty?)
       end
@@ -113,7 +119,7 @@ module KKGit
       def ensure_ok!(ok, title, stdout: nil, stderr: nil, repo_dir: '.')
         return if ok
 
-        msg = +"#{title} failed"
+        msg = "#{title} failed"
         msg << "\n#{stderr}" unless stderr.to_s.strip.empty?
         msg << "\n#{stdout}" unless stdout.to_s.strip.empty?
 
@@ -318,6 +324,8 @@ module KKGit
           return :noop
         end
 
+        AutoFormat.run!(repo_dir: repo_dir) unless skip_format?
+
         add_all!(repo_dir: repo_dir)
         ensure_no_sensitive_staged!(repo_dir: repo_dir)
 

data/lib/kk/git/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module KKGit
-  VERSION = '0.2.3'
+  VERSION = '0.2.5'
 end

data/lib/kk/git.rb

@@ -5,8 +5,8 @@ require_relative 'git/ruby_warnings'
 require_relative 'git/git_runner'
 require_relative 'git/commit_message'
 require_relative 'git/git_ops'
+require_relative 'git/auto_format'
 require_relative 'git/release'
 
 module KKGit
 end
-

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: kk-git
 version: !ruby/object:Gem::Version
-  version: 0.2.3
+  version: 0.2.5
 platform: ruby
 authors:
 - kk
@@ -21,6 +21,7 @@ extra_rdoc_files: []
 files:
 - exe/kk-git
 - lib/kk/git.rb
+- lib/kk/git/auto_format.rb
 - lib/kk/git/commit_message.rb
 - lib/kk/git/git_ops.rb
 - lib/kk/git/git_runner.rb