node_mutation 1.24.3 → 1.24.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a1838915f4699a2c9bc566d05d38ed23f892b31a0a44d3812f4fe54b760a213f
-  data.tar.gz: 0fc0b9aac1857fc18675578da657a663786eb5ffcb43b2b2f4c27a013487eb78
+  metadata.gz: 5fbc8f86a01844e6cefbdc699124dfaef0fa8da5581a4375a4f99132efdc0bbf
+  data.tar.gz: 8868c51bced5f0ec6a4dfb52951989ba7dc1e5fad389c9be2d3d96303346e490
 SHA512:
-  metadata.gz: 699e09b43684f17ad6d0ab0b3902cc2910eb5ba53a870a5a2dfb6bb1c19b5df770a747c8d8c4e22fb8414cfac7f25acb1e3498465f0f8279abc458544eb4d8b5
-  data.tar.gz: 3702e772e0379b37d1adea9b7024ebeafe2ea0e77a8b5263a1d9a32b62d581be58e64d1a735d6ac0b7b6bbc3440cfbd477744d03e68da58f5b426fd5209c470b
+  metadata.gz: 808c1a596e88980cdbdaab4c8c6fda0e7d21431f4bbb2529a79c1cddaa4fe51f09eadba2ed5a90e6f8f8c8d3c4fd1396a906a4f0cc8398bbf497b35068204390
+  data.tar.gz: 6999f0fb1490422a929d9faa2de9daec7ada538fd17c27b6f6f5c19655068a08a5ed87c07a8aec49ab5ef8440353c914d4fc38136087af4cb1601384df4745ff

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # NodeMutation
 
+## 1.24.4 (2024-04-16)
+
+* Use Prism `character_offset` instead byte `offset`
+
 ## 1.24.3 (2024-04-07)
 
 * Update `prism_ext` to 0.3.0

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    node_mutation (1.24.3)
+    node_mutation (1.24.4)
 
 GEM
   remote: https://rubygems.org/

data/README.md

@@ -84,16 +84,24 @@ mutation.replace node, '{{arguments.-1.on_value}}', with: ':update'
 source # after_commit :do_index, on: :update, if: :indexable?
 ```
 
-See more in [ParserAdapter](https://synvert-hq.github.io/node-mutation-ruby/NodeMutation/ParserAdapter.html) and [SyntaxTreeAdapter](https://synvert-hq.github.io/node-mutation-ruby/NodeMutation/SyntaxTreeAdapter.html)
+See more in [PrismAdapter](https://synvert-hq.github.io/node-mutation-ruby/NodeMutation/PrismAdapter.html)
+and [ParserAdapter](https://synvert-hq.github.io/node-mutation-ruby/NodeMutation/ParserAdapter.html)
+and [SyntaxTreeAdapter](https://synvert-hq.github.io/node-mutation-ruby/NodeMutation/SyntaxTreeAdapter.html)
 
 ## Configuration
 
 ### adapter
 
-Different parsers, like parse and ripper, will generate different AST nodes, to make NodeMutation work for them all,
+Different parsers, like prism, parse and syntax_tree, will generate different AST nodes, to make NodeMutation work for them all,
 we define an [Adapter](https://github.com/synvert-hq/node-mutation-ruby/blob/main/lib/node_mutation/adapter.rb) interface,
 if you implement the Adapter interface, you can set it as NodeMutation's adapter.
 
+It provides 3 adapters
+
+1. `PrismAdapter`
+2. `ParserAdapter`
+3. `SyntaxTreeAdapter`
+
 ### strategy
 
 It provides 3 strategies to handle conflicts when processing actions:

data/lib/node_mutation/adapter/prism.rb

@@ -6,7 +6,7 @@ require 'prism_ext'
 class NodeMutation::PrismAdapter < NodeMutation::Adapter
   def get_source(node)
     if node.is_a?(Array)
-      return node.first.instance_variable_get(:@source).source[node.first.location.start_offset...node.last.location.end_offset]
+      return node.first.instance_variable_get(:@source).source[node.first.location.start_character_offset...node.last.location.end_character_offset]
     end
 
     node.to_source
@@ -124,7 +124,7 @@ class NodeMutation::PrismAdapter < NodeMutation::Adapter
               "#{direct_child_name} is not supported for #{get_source(node)}" unless child_node
         return child_node_range(child_node, nested_child_name) if nested_child_name
 
-        return NodeMutation::Struct::Range.new(child_node.location.start_offset, child_node.location.end_offset)
+        return NodeMutation::Struct::Range.new(child_node.location.start_character_offset, child_node.location.end_character_offset)
       end
 
       raise NodeMutation::MethodNotSupported,
@@ -133,16 +133,16 @@ class NodeMutation::PrismAdapter < NodeMutation::Adapter
       child_node = node.send(direct_child_name)
       return child_node_range(child_node, nested_child_name) if nested_child_name
 
-      return NodeMutation::Struct::Range.new(child_node.location.start_offset, child_node.location.end_offset)
+      return NodeMutation::Struct::Range.new(child_node.location.start_character_offset, child_node.location.end_character_offset)
     end
 
     if node.respond_to?("#{child_name}_loc")
       node_loc = node.send("#{child_name}_loc")
-      NodeMutation::Struct::Range.new(node_loc.start_offset, node_loc.end_offset) if node_loc
+      NodeMutation::Struct::Range.new(node_loc.start_character_offset, node_loc.end_character_offset) if node_loc
     elsif node.is_a?(Prism::CallNode) && child_name.to_sym == :name
-      NodeMutation::Struct::Range.new(node.message_loc.start_offset, node.message_loc.end_offset)
+      NodeMutation::Struct::Range.new(node.message_loc.start_character_offset, node.message_loc.end_character_offset)
     elsif node.is_a?(Prism::LocalVariableReadNode) && child_name.to_sym == :name
-      NodeMutation::Struct::Range.new(node.location.start_offset, node.location.end_offset)
+      NodeMutation::Struct::Range.new(node.location.start_character_offset, node.location.end_character_offset)
     else
       raise NodeMutation::MethodNotSupported,
             "#{direct_child_name} is not supported for #{get_source(node)}" unless node.respond_to?(direct_child_name)
@@ -156,34 +156,34 @@ class NodeMutation::PrismAdapter < NodeMutation::Adapter
 
       if child_node.is_a?(Prism::Node)
         return(
-          NodeMutation::Struct::Range.new(child_node.location.start_offset, child_node.location.end_offset)
+          NodeMutation::Struct::Range.new(child_node.location.start_character_offset, child_node.location.end_character_offset)
         )
       end
 
       return(
-        NodeMutation::Struct::Range.new(child_node.first.location.start_offset, child_node.last.location.end_offset)
+        NodeMutation::Struct::Range.new(child_node.first.location.start_character_offset, child_node.last.location.end_character_offset)
       )
     end
   end
 
   def get_start(node, child_name = nil)
     node = child_node_by_name(node, child_name) if child_name
-    node.location.start_offset
+    node.location.start_character_offset
   end
 
   def get_end(node, child_name = nil)
     node = child_node_by_name(node, child_name) if child_name
-    node.location.end_offset
+    node.location.end_character_offset
   end
 
   def get_start_loc(node, child_name = nil)
     node = child_node_by_name(node, child_name) if child_name
-    NodeMutation::Struct::Location.new(node.location.start_line, node.location.start_column)
+    NodeMutation::Struct::Location.new(node.location.start_line, node.location.start_character_column)
   end
 
   def get_end_loc(node, child_name = nil)
     node = child_node_by_name(node, child_name) if child_name
-    NodeMutation::Struct::Location.new(node.location.end_line, node.location.end_column)
+    NodeMutation::Struct::Location.new(node.location.end_line, node.location.end_character_column)
   end
 
   private

data/lib/node_mutation/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 class NodeMutation
-  VERSION = "1.24.3"
+  VERSION = "1.24.4"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: node_mutation
 version: !ruby/object:Gem::Version
-  version: 1.24.3
+  version: 1.24.4
 platform: ruby
 authors:
 - Richard Huang
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-04-07 00:00:00.000000000 Z
+date: 2024-04-16 00:00:00.000000000 Z
 dependencies: []
 description: ast node mutation apis
 email: