ask-tools-shell 0.3.0 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: eea08734ef4ad1040ba2c9f2e7b615ae9931bb5e1d83dcb893e509487e603eaf
-  data.tar.gz: 47bda8b63b21e23fef543a5142658b363fd7758e01cd37921d4d880c0804d901
+  metadata.gz: 139a762b4412fcf6dac23e7360f7f49e537f32272491865e90f3ec36ae27da44
+  data.tar.gz: d074b1a2173259a65fc9f9eab1efd3424f2ef4bfcea2e3e868a3ee281ca5d5db
 SHA512:
-  metadata.gz: 75d972a45b9ecd3310c6693e0cd958745edb2af065dfdf366446bb7c4799d3359c3ff12d45dc58831eb3b59e88015a2221f0bec1c0857da6f755c2e1f32c7a4b
-  data.tar.gz: e5e8e83cb62a01e0ca737862917c559e9bade010f5104bb130f3ac8b44e7a615f1f6618e2ff30cde382caeb8cae186ed1da0c010f8f541bf884251667f29ab60
+  metadata.gz: 8e16c060e637dc9dfa1c420fff9eaf5c357a8abc2ce7c9489fa082bd2f319b65e4b03b64677d3310df359725877e313c334c0a39a5eaa0b5bd8ff44253c2aac9
+  data.tar.gz: ab4bbada08684a89410144614778b70be6a5bc7e7e1d0dbd596aac2727df3eab80d3ec94de9cf172d4a0519d6a1494d102010087c61ea9bee6118fed549bf17f

data/CHANGELOG.md

@@ -0,0 +1,13 @@
+# Changelog
+
+## 0.3.0 (2026-06-21)
+
+- Added `EditOperations` pluggable interface (read_file, write_file, file_exist?, file?, file_size, expand_path)
+- Added `DefaultEditOperations` default implementation
+- Added BOM detection/stripping to Edit tool (`Shell.strip_bom`)
+- Added line ending detection and preservation to Edit tool (`Shell.detect_line_ending`, `Shell.normalize_line_endings`, `Shell.restore_line_endings`)
+- Added `FileMutationQueue` for atomic batch file edits with rollback
+
+## 0.2.2
+
+- Various fixes

data/lib/ask/tools/shell/apply_patch.rb

@@ -0,0 +1,208 @@
+# frozen_string_literal: true
+
+require "fileutils"
+
+module Ask
+  module Tools
+    module Shell
+      module ApplyPatchOperations
+        def read_file(path)
+          File.read(path)
+        end
+
+        def write_file(path, content)
+          File.write(path, content)
+        end
+
+        def file_exist?(path)
+          File.exist?(path)
+        end
+
+        def file?(path)
+          File.file?(path)
+        end
+
+        def delete_file(path)
+          File.delete(path)
+        end
+
+        def expand_path(path)
+          File.expand_path(path)
+        end
+
+        def mkdir_p(path)
+          FileUtils.mkdir_p(File.dirname(path))
+        end
+      end
+
+      class DefaultApplyPatchOperations
+        include ApplyPatchOperations
+      end
+    end
+
+    # Apply a patch to files using a unified diff format.
+    # Supports creating, updating, and deleting files within a
+    # "*** Begin Patch" / "*** End Patch" envelope.
+    class ApplyPatch < Ask::Tool
+      description "Edit files using a unified diff format. " \
+                   "Wrap all changes in a \"*** Begin Patch\" / \"*** End Patch\" envelope. " \
+                   "Each file section starts with a header: " \
+                   "\"*** Add File: <path>\" for new files, " \
+                   "\"*** Update File: <path>\" for changes, or " \
+                   "\"*** Delete File: <path>\" for removals. " \
+                   "Prefix new lines with +."
+
+      param :patchText, type: :string, desc: "The full patch text describing all file changes", required: true
+
+      attr_writer :operations
+
+      def operations
+        @operations ||= Shell::DefaultApplyPatchOperations.new
+      end
+
+      MAX_FILE_SIZE = 10_000_000
+
+      def execute(patchText:)
+        patches = parse_patch(patchText)
+        return Ask::Result.error(message: "No valid patch sections found") if patches.empty?
+
+        results = []
+        patches.each do |entry|
+          path = operations.expand_path(entry[:path])
+
+          case entry[:type]
+          when :add
+            if operations.file_exist?(path)
+              return Ask::Result.error(message: "File already exists: #{path}")
+            end
+            operations.mkdir_p(path)
+            content = entry[:lines].join("\n")
+            content += "\n" unless content.end_with?("\n")
+            operations.write_file(path, content)
+            results << { action: "add", path: entry[:path], lines: entry[:lines].size }
+
+          when :update
+            unless operations.file_exist?(path)
+              return Ask::Result.error(message: "File does not exist: #{path}")
+            end
+            unless operations.file?(path)
+              return Ask::Result.error(message: "Not a file: #{path}")
+            end
+
+            raw = operations.read_file(path)
+            size = raw.bytesize
+            if size > MAX_FILE_SIZE
+              return Ask::Result.error(message: "File too large (#{size} bytes)")
+            end
+
+            new_content = apply_chunks(raw, entry[:chunks])
+            operations.write_file(path, new_content)
+            results << { action: "update", path: entry[:path] }
+
+          when :delete
+            unless operations.file_exist?(path)
+              return Ask::Result.error(message: "File does not exist: #{path}")
+            end
+            operations.delete_file(path)
+            results << { action: "delete", path: entry[:path] }
+          end
+        end
+
+        summary = results.map { |r| "#{r[:action].upcase_first} #{r[:path]}" }.join("\n")
+        Ask::Result.ok(data: { summary: summary, results: results }, metadata: { results: results })
+      end
+
+      private
+
+      PATCH_START = "*** Begin Patch"
+      PATCH_END = "*** End Patch"
+      ADD_HEADER = /^\*\*\* Add File:\s+(.+)/i
+      UPDATE_HEADER = /^\*\*\* Update File:\s+(.+)/i
+      DELETE_HEADER = /^\*\*\* Delete File:\s+(.+)/i
+      HUNK_HEADER = /^@@/
+
+      def parse_patch(text)
+        text = text.dup.force_encoding("UTF-8")
+        start_idx = text.index(PATCH_START)
+        return [] unless start_idx
+
+        after_start = text[(start_idx + PATCH_START.length)..]
+        end_idx = after_start.index(PATCH_END)
+        body = end_idx ? after_start[0...end_idx] : after_start
+
+        entries = []
+        lines = body.split("\n")
+        i = 0
+        while i < lines.length
+          line = lines[i]
+
+          if (m = line.match(ADD_HEADER))
+            path = m[1].strip
+            i += 1
+            content_lines = []
+            while i < lines.length && !lines[i].start_with?("***")
+              content_lines << lines[i].sub(/^\+/, "") if lines[i].start_with?("+")
+              i += 1
+            end
+            entries << { type: :add, path: path, lines: content_lines }
+
+          elsif (m = line.match(UPDATE_HEADER))
+            path = m[1].strip
+            i += 1
+            chunks = []
+            while i < lines.length && !lines[i].start_with?("***")
+              if lines[i].match?(HUNK_HEADER)
+                i += 1
+                old_lines = []
+                new_lines = []
+                while i < lines.length && !lines[i].start_with?("***") && !lines[i].match?(HUNK_HEADER) && !lines[i].start_with?("***")
+                  if lines[i].start_with?("-")
+                    old_lines << lines[i][1..]
+                  elsif lines[i].start_with?("+")
+                    new_lines << lines[i][1..]
+                  elsif lines[i].start_with?(" ")
+                    old_lines << lines[i][1..]
+                    new_lines << lines[i][1..]
+                  end
+                  i += 1
+                end
+                chunks << { old: old_lines, new: new_lines } unless old_lines.empty? && new_lines.empty?
+              else
+                i += 1
+              end
+            end
+            entries << { type: :update, path: path, chunks: chunks }
+
+          elsif (m = line.match(DELETE_HEADER))
+            entries << { type: :delete, path: m[1].strip }
+            i += 1
+
+          else
+            i += 1
+          end
+        end
+
+        entries
+      end
+
+      def apply_chunks(content, chunks)
+        return content if chunks.empty?
+
+        lines = content.split("\n", -1)
+        chunks.each do |chunk|
+          old_text = chunk[:old].join("\n")
+          new_text = chunk[:new].join("\n")
+          file_text = lines.join("\n")
+          idx = file_text.index(old_text)
+          next unless idx
+
+          before = file_text[0...idx]
+          after = file_text[(idx + old_text.length)..]
+          file_text = before + new_text + after
+          lines = file_text.split("\n", -1)
+        end
+        lines.join("\n")
+      end
+    end
+  end
+end

data/lib/ask/tools/shell/edit.rb

@@ -42,7 +42,7 @@ module Ask
 
       # Detect the dominant line ending in content.
       def self.detect_line_ending(content)
-        crlf = content.count("\r\n")
+        crlf = content.scan("\r\n").size
         lf = content.count("\n") - crlf
         crlf > lf ? "\r\n" : "\n"
       end

data/lib/ask/tools/shell/version.rb

@@ -3,7 +3,7 @@
 module Ask
   module Tools
     module Shell
-      VERSION = "0.3.0"
+      VERSION = "0.3.1"
     end
   end
 end

data/lib/ask/tools/shell.rb

@@ -9,11 +9,16 @@ require_relative "shell/edit"
 require_relative "shell/glob"
 require_relative "shell/grep"
 require_relative "shell/code"
+require_relative "shell/apply_patch"
 
 module Ask
   module Tools
     module Shell
-      TOOLS = [Bash, Read, Write, Edit, Glob, Grep, Code].freeze
+      TOOLS = [Bash, Read, Write, Edit, Glob, Grep, Code, ApplyPatch].freeze
+
+      def self.all
+        TOOLS.map(&:new)
+      end
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ask-tools-shell
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.3.1
 platform: ruby
 authors:
 - Kaka Ruto
@@ -13,28 +13,28 @@ dependencies:
   name: ask-tools
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0.1'
 - !ruby/object:Gem::Dependency
   name: ask-sandbox-providers
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0.1'
 - !ruby/object:Gem::Dependency
@@ -86,11 +86,13 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- CHANGELOG.md
 - LICENSE
 - README.md
 - lib/ask-tools-shell.rb
 - lib/ask/skills/shell.patterns/SKILL.md
 - lib/ask/tools/shell.rb
+- lib/ask/tools/shell/apply_patch.rb
 - lib/ask/tools/shell/bash.rb
 - lib/ask/tools/shell/code.rb
 - lib/ask/tools/shell/edit.rb
@@ -103,7 +105,10 @@ files:
 homepage: https://github.com/ask-rb/ask-tools-shell
 licenses:
 - MIT
-metadata: {}
+metadata:
+  homepage_uri: https://github.com/ask-rb/ask-tools-shell
+  source_code_uri: https://github.com/ask-rb/ask-tools-shell
+  changelog_uri: https://github.com/ask-rb/ask-tools-shell/blob/master/CHANGELOG.md
 rdoc_options: []
 require_paths:
 - lib