ask-tools-shell 0.5.0 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8805be0a64d1a42f4b0eae3600c81ce0e61565fa19aaf1fd835d36fbb6fa9b58
-  data.tar.gz: fe497b0c0604cc2c9a90cd02bc17c31ff8f46140ac88a67e9697b5b9cd15f7ee
+  metadata.gz: a13d88c2167e73290edfcabe00488c6e7f9592e75fe5004fe878bc4e075585dd
+  data.tar.gz: 616e5788149068c16fabc73ea10c2984dfa917f359cdf9bc247e8b9327d562be
 SHA512:
-  metadata.gz: defeb70d692dc1da34ad21300ed52dff8496df4fd28c3365b7724d9e3a6c113afbdd7816619cfa0f9017479d9af1a061fb0ef82c140110f0e46209df8829bdc6
-  data.tar.gz: e855c29b7b8ed782dab2fccb0764c3eebe4b5b5b7af039aff713fc746c31b3b70f77b1a47abc250a54b217a3bacba4642bef97aa3f409e5f52ee80043a8f0610
+  metadata.gz: fa1585022477ed69de424aa059ad0f13fc6652a01bf3b3ea2528dc944a6a1e218f247713b57828303770d7def901d2692b6d6557356cc9e8262f14727f6b99f2
+  data.tar.gz: 2da4a9a5aed750c5991a833e06a391199643e7c3b577d0d4c425120643bd9b23169e8c81ba95140d258a79e694fcfbd90bf220d45f7da53a05092c9003a61d69

data/CHANGELOG.md

@@ -1,3 +1,14 @@
+## [0.5.1] - 2026-08-10
+
+### Fixed
+- **Ledger stays consistent after full-context edits.** `Edit` and
+  `ApplyPatch` read files in full, so they now record a full read for the
+  current state after writing — previously their write left the ledger with
+  a stale partial view that could deny a subsequent `Write` even though the
+  file had been fully read in between. (`ApplyPatch` gains the same
+  post-write record `Edit` had; `Edit`'s record moved to post-write so it
+  stays valid for the current file state.)
+
 ## [0.5.0] - 2026-08-10
 
 ### Added

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

@@ -100,6 +100,10 @@ module Ask
               return Ask::Result.error(message: "Hunk does not match file content for: #{path}")
             end
             operations.write_file(path, new_content)
+            # ApplyPatch read the whole file to apply its hunks, so the model
+            # has full context — record a full read so a later Write isn't
+            # blocked by an earlier partial view.
+            Shell::FileLedger.record(path, partial: false, lines_seen: [0, new_content.count("\n") + 1])
             results << { action: "update", path: entry[:path] }
 
           when :delete

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

@@ -95,10 +95,6 @@ module Ask
         end
 
         raw = operations.read_file(path)
-        # Edit reads the whole file, so the model has seen all of it — record
-        # that before the write so a later Write isn't blocked by a stale
-        # partial view (and so the ledger reflects what was actually shown).
-        Shell::FileLedger.record(path, partial: false, lines_seen: [0, raw.count("\n") + 1])
         bom, content = Shell.strip_bom(raw)
         original_ending = Shell.detect_line_ending(content)
         content = Shell.normalize_line_endings(content)
@@ -119,6 +115,10 @@ module Ask
         content = bom + content
 
         operations.write_file(path, content)
+        # Edit read the whole file to apply the change, so the model has full
+        # context — record a full read of the current state so a later Write
+        # isn't blocked by an earlier partial view.
+        Shell::FileLedger.record(path, partial: false, lines_seen: [0, content.count("\n") + 1])
         Ask::Result.ok(data: { path: path, replacements: count },
                         metadata: { path: path, replacements: count })
       end

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

@@ -3,7 +3,7 @@
 module Ask
   module Tools
     module Shell
-      VERSION = "0.5.0"
+      VERSION = "0.5.1"
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ask-tools-shell
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.5.1
 platform: ruby
 authors:
 - Kaka Ruto