expressir 2.4.61-aarch64-linux → 2.4.62-aarch64-linux

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3648b34f0233c36412bfeea24262e3346650c2975c983a2f331e49a909bcb479
-  data.tar.gz: c7e572bf608513f0321d0981b5e70657330f9a6a65ddd82c6184bf2e8e3c5e93
+  metadata.gz: 324e4deb12881b3d886c9d9ca434195fb2528da4cfe3b8f9d3f1026fbb655dc1
+  data.tar.gz: 12c72a4b83092d211a6c3b801e6cf6c7d060f5d372d97de1796b38de0bf231ff
 SHA512:
-  metadata.gz: e181f8cb3cb207495693344ea6e4d66bcdc168aaf8f21aed562b0ea3a42d3a2734cddcacf13aceb700779307d33945911dd0d1860ee67cfe5e56f244a636c2f0
-  data.tar.gz: a8a0ddb38e20e618831825a23ca0ed78625ba87b3d605f4c42ff89821363cd118dba0e08dc6c75141541f019b62b56b05bd502af1c3944795c91a7b394b90873
+  metadata.gz: efdaa3dd0530e121d4eef30e643db63de5a5450e22e84ab8eff2a1c4237c41fdf89844248fc8351e4d7e71b540ee904f002d696b0c00b299d159171f275fb340
+  data.tar.gz: bd907900898f2e9ed50c128e3c4eb6478d29cf810cba30d75242fd3724d2032c3a666dfd576c607ee49406403f8685fb697c5ab32a5c9fd466323e17ba3c64da

data/lib/expressir/express/node_position_index.rb

@@ -69,12 +69,6 @@ module Expressir
         entries.sort_by { |n| node_order(n) }
       end
 
-      # First node (in node order) that is_a?(type), memoized per type.
-      def first_node_of_type(type)
-        @first_of_type ||= {}
-        @first_of_type[type] ||= @nodes.find { |n| n[:node].is_a?(type) }&.dig(:node)
-      end
-
       def semantic_entries
         @semantic_entries ||=
           @nodes.select { |n| n[:line] && semantic?(n[:node]) }
@@ -200,7 +194,19 @@ module Expressir
         end
 
         candidates.min_by { |n| node_order(n) }&.dig(:node) ||
-          first_node_of_type(node_type)
+          containing_node_of_type(node_type, remark_line)
+      end
+
+      # A comment on an END_XXX line whose scope's recorded end sits past
+      # the line — remark items live between declarations, and their spans
+      # push the owner's end_line beyond END_XXX — resolves by span
+      # containment. Falls back to nil, not the first node of the type:
+      # handing an unresolvable END_FUNCTION comment to the first function
+      # piled every stray tail onto one owner (#609).
+      def containing_node_of_type(type, line)
+        semantic_entries
+          .select { |n| n[:node].is_a?(type) && n[:end_line] && line.between?(n[:line], n[:end_line]) }
+          .min_by { |n| node_order(n) }&.dig(:node)
       end
 
       private

data/lib/expressir/express/remark_attacher.rb

@@ -163,7 +163,13 @@ module Expressir
 
             # Strategy 3: Create implicit item for qualified paths only
             if target.nil? && tag.include?(".")
-              if containing_scope && function_rule_procedure?(containing_scope)
+              # A tag whose first segment is a schema id is already
+              # absolute: prepending the containing scope's path would
+              # double-count it, and the implicit walk would then name
+              # the item after the schema (#609 — `(*"s.f.lv"` inside f
+              # must create `lv`, not `s`).
+              absolute_tag = get_schema_ids(@model).include?(tag.split(".").first)
+              if containing_scope && function_rule_procedure?(containing_scope) && !absolute_tag
                 scope_path = build_scope_path(containing_scope)
                 if scope_path
                   target = create_implicit_remark_item(@model, "#{scope_path}.#{tag}",

data/lib/expressir/version.rb

@@ -3,6 +3,6 @@ module Expressir
   # autoload it (Ruby autoload only works for module/class constants, not
   # for string constants like `Expressir::VERSION`).
   module Version
-    VERSION = "2.4.61".freeze
+    VERSION = "2.4.62".freeze
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: expressir
 version: !ruby/object:Gem::Version
-  version: 2.4.61
+  version: 2.4.62
 platform: aarch64-linux
 authors:
 - Ribose Inc.