eager_eye 1.1.11 → 1.1.12
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- checksums.yaml +4 -4
- data/CHANGELOG.md +8 -0
- data/README.md +1 -1
- data/lib/eager_eye/detectors/custom_method_query.rb +11 -7
- data/lib/eager_eye/version.rb +1 -1
- data/lib/eager_eye.rb +1 -0
- metadata +2 -2
checksums.yaml
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
---
|
|
2
2
|
SHA256:
|
|
3
|
-
metadata.gz:
|
|
4
|
-
data.tar.gz:
|
|
3
|
+
metadata.gz: 8b9c34f34d7b2a4433772c5478a94eb28ada60b751b560900c257a9d013b9b9f
|
|
4
|
+
data.tar.gz: 41ba835f40199d7a0946b5e2f42db2a73c1beb43ef3684d32f147bf1250dc5f3
|
|
5
5
|
SHA512:
|
|
6
|
-
metadata.gz:
|
|
7
|
-
data.tar.gz:
|
|
6
|
+
metadata.gz: 6cc33125266267c58f0dff32d1e791cd7209660be223b54f57a001af08aae178528ee081f95613638109942ce169638b37e0239077df9794e8162cb2cf69f367
|
|
7
|
+
data.tar.gz: 40a0fd9a91164e0a261c8dd3119db5ce7f3795b1f50b07b99276d226c3a72cd2f3acbb98fc0ec6a9063ed846eb93f5a1968e227e5c6cdfde8c0d0691fe016b90
|
data/CHANGELOG.md
CHANGED
|
@@ -7,6 +7,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|
|
7
7
|
|
|
8
8
|
## [Unreleased]
|
|
9
9
|
|
|
10
|
+
## [1.1.12] - 2026-01-15
|
|
11
|
+
|
|
12
|
+
### Fixed
|
|
13
|
+
|
|
14
|
+
- **CustomMethodQuery Stack Overflow** - Fix infinite recursion in `collection_is_array?`
|
|
15
|
+
- Added `visited` Set to track already-visited nodes and prevent cyclic traversal
|
|
16
|
+
- Fixes `stack level too deep` error on complex method chains
|
|
17
|
+
|
|
10
18
|
## [1.1.11] - 2026-01-13
|
|
11
19
|
|
|
12
20
|
### Fixed
|
data/README.md
CHANGED
|
@@ -10,7 +10,7 @@
|
|
|
10
10
|
|
|
11
11
|
<p align="center">
|
|
12
12
|
<a href="https://github.com/hamzagedikkaya/eager_eye/actions/workflows/main.yml"><img src="https://github.com/hamzagedikkaya/eager_eye/actions/workflows/main.yml/badge.svg" alt="CI"></a>
|
|
13
|
-
<a href="https://rubygems.org/gems/eager_eye"><img src="https://img.shields.io/badge/gem-v1.1.
|
|
13
|
+
<a href="https://rubygems.org/gems/eager_eye"><img src="https://img.shields.io/badge/gem-v1.1.12-red.svg" alt="Gem Version"></a>
|
|
14
14
|
<a href="https://github.com/hamzagedikkaya/eager_eye"><img src="https://img.shields.io/badge/coverage-95%25-brightgreen.svg" alt="Coverage"></a>
|
|
15
15
|
<a href="https://www.ruby-lang.org/"><img src="https://img.shields.io/badge/ruby-%3E%3D%203.1-ruby.svg" alt="Ruby"></a>
|
|
16
16
|
<a href="https://opensource.org/licenses/MIT"><img src="https://img.shields.io/badge/License-MIT-yellow.svg" alt="License: MIT"></a>
|
|
@@ -93,28 +93,32 @@ module EagerEye
|
|
|
93
93
|
first_arg&.type == :arg ? first_arg.children[0] : nil
|
|
94
94
|
end
|
|
95
95
|
|
|
96
|
-
def collection_is_array?(node, definitions = {})
|
|
96
|
+
def collection_is_array?(node, definitions = {}, visited = Set.new)
|
|
97
97
|
return false unless node.is_a?(Parser::AST::Node)
|
|
98
|
+
return false if visited.include?(node.object_id)
|
|
99
|
+
|
|
100
|
+
visited.add(node.object_id)
|
|
101
|
+
|
|
98
102
|
return true if %i[array hash].include?(node.type)
|
|
99
|
-
return check_lvar_collection?(node, definitions) if node.type == :lvar
|
|
100
|
-
return check_send_collection?(node, definitions) if node.type == :send
|
|
103
|
+
return check_lvar_collection?(node, definitions, visited) if node.type == :lvar
|
|
104
|
+
return check_send_collection?(node, definitions, visited) if node.type == :send
|
|
101
105
|
|
|
102
106
|
false
|
|
103
107
|
end
|
|
104
108
|
|
|
105
|
-
def check_lvar_collection?(node, definitions)
|
|
109
|
+
def check_lvar_collection?(node, definitions, visited)
|
|
106
110
|
return false unless definitions
|
|
107
111
|
|
|
108
112
|
definition = definitions[node.children[0]]
|
|
109
|
-
definition ? collection_is_array?(definition, definitions) : false
|
|
113
|
+
definition ? collection_is_array?(definition, definitions, visited) : false
|
|
110
114
|
end
|
|
111
115
|
|
|
112
|
-
def check_send_collection?(node, definitions)
|
|
116
|
+
def check_send_collection?(node, definitions, visited)
|
|
113
117
|
method_name = node.children[1]
|
|
114
118
|
return true if %i[map select collect flat_map uniq compact].include?(method_name)
|
|
115
119
|
return true if SAFE_TRANSFORM_METHODS.include?(method_name)
|
|
116
120
|
|
|
117
|
-
collection_is_array?(node.children[0], definitions)
|
|
121
|
+
collection_is_array?(node.children[0], definitions, visited)
|
|
118
122
|
end
|
|
119
123
|
|
|
120
124
|
def receiver_ends_with_safe_transform_method?(node)
|
data/lib/eager_eye/version.rb
CHANGED
data/lib/eager_eye.rb
CHANGED
metadata
CHANGED
|
@@ -1,14 +1,14 @@
|
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
|
2
2
|
name: eager_eye
|
|
3
3
|
version: !ruby/object:Gem::Version
|
|
4
|
-
version: 1.1.
|
|
4
|
+
version: 1.1.12
|
|
5
5
|
platform: ruby
|
|
6
6
|
authors:
|
|
7
7
|
- hamzagedikkaya
|
|
8
8
|
autorequire:
|
|
9
9
|
bindir: exe
|
|
10
10
|
cert_chain: []
|
|
11
|
-
date: 2026-01-
|
|
11
|
+
date: 2026-01-15 00:00:00.000000000 Z
|
|
12
12
|
dependencies:
|
|
13
13
|
- !ruby/object:Gem::Dependency
|
|
14
14
|
name: ast
|