flog 4.9.2 → 4.9.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 79aa339ba2ed65cdd83437fad88ae503bff7a4392255d9dd9920e2ca400d164d
-  data.tar.gz: 6b69169958fec009c2b6e106d98089d4a581821bf26f6568a488d91bb0043abf
+  metadata.gz: 9047c8b22acece9cd9e5c1dbb2e44d486fd903c40bd80967d1e4c257fa8669a9
+  data.tar.gz: 69a903441dcc41ab504399e173f8d231a242869ffbc5b6aebfa74e758822c4af
 SHA512:
-  metadata.gz: 21d6a21c74691972cb98cec49cbe1289b5c870d6874d1606b4c2451431e3ec63d261f22a235927b80ffeea15273bb1d31a6aa18530fdbb346ca5e74e882ec94f
-  data.tar.gz: e713d2e30e361ac522f889d77efd0f291a6f38c47fb3c84c777c8879f770fc928e7e919318e816eca9c8347e110b1591edb17b4b3a5863ad17bc9f43279557f5
+  metadata.gz: 88c867edf9e72b5b1f392697f3de483dfd18234a620028eeb6a60527b5dbdc21e994afb103870fa2d79fd4d88a91301a35fc37c0d095b330d1469be72595b879
+  data.tar.gz: f57466932cb71074ae085276bb9dbc9f47e684f4da599ce52a33d2a94198d2e9d86b22a41094bf2f65f6a8afa3a3c4be369ded47f970bcaad21fe82d9e334dbd

data/History.rdoc

@@ -1,3 +1,16 @@
+=== 4.9.4 / 2026-01-06
+
+* 2 bug fixes:
+
+  * Namespace NotRubyParser to avoid problems with bundler's horrid eager loading. (nateberkopec)
+  * Patch prism to recognize `it` as a local. (faisal)
+
+=== 4.9.3 / 2026-01-05
+
+* 1 bug fix:
+
+  * Fixed bug caused by last release duplicating (frozen) options hash. (nateberkopec)
+
 === 4.9.2 / 2026-01-04
 
 * 2 bug fixes:

data/Rakefile

@@ -38,7 +38,7 @@ task :debug do
   details = ENV["D"]
   continue = ENV["C"]
 
-  flog = FlogCLI.new :parser => RubyParser
+  flog = FlogCLI.new
 
   flog.option[:details] = true if details
   flog.option[:continue] = true if continue

data/lib/flog.rb

@@ -4,7 +4,14 @@ require "prism/translation/ruby_parser"
 require "timeout"
 require "racc" # for Racc::ParseError
 
-NotRubyParser = Class.new Prism::Translation::RubyParser
+class Prism::Translation::RubyParser
+  module TweakIt
+    def visit_it_local_variable_read_node(node) # TODO: upstream
+      s(node, :lvar, :it)
+    end
+  end
+  Compiler.prepend TweakIt
+end
 
 ##
 # Flog is a SexpProcessor that calculates a ABC (assignments,
@@ -16,15 +23,15 @@ NotRubyParser = Class.new Prism::Translation::RubyParser
 # thoroughly test.
 
 class Flog < MethodBasedSexpProcessor
-  VERSION = "4.9.2" # :nodoc:
+  VERSION = "4.9.4" # :nodoc:
+
+  NotRubyParser = Class.new Prism::Translation::RubyParser # compatibility layer :nodoc:
 
   ##
   # Cut off point where the report should stop unless --all given.
 
   DEFAULT_THRESHOLD = 0.60
 
-  THRESHOLD = DEFAULT_THRESHOLD # :nodoc:
-
   ##
   # The scoring system hash. Maps node type to score.
 
@@ -230,7 +237,7 @@ class Flog < MethodBasedSexpProcessor
 
   def initialize option = {}
     super()
-    @option              = option.dup
+    @option              = option = option.dup
     @mass                = {}
     @parser              = nil
     @threshold           = option[:threshold] || DEFAULT_THRESHOLD

data/test/test_flog.rb

@@ -19,7 +19,7 @@ end
 
 class TestFlog < FlogTest
   def setup
-    @flog = Flog.new parser: NotRubyParser
+    @flog = Flog.new
   end
 
   def test_add_to_score
@@ -344,6 +344,17 @@ class TestFlog < FlogTest
     assert_process sexp, 2.326, :loop => 1.0, :branch => 1.1, :block_call => 1
   end
 
+  def test_process_iter__it
+    ruby = "blah { puts it }"
+    sexp = s(:iter,
+             s(:call, nil, :blah), 0,
+             s(:call, nil, :puts, s(:lvar, :it)))
+
+    assert_parse sexp, ruby
+
+    assert_process sexp, 2.326, blah: 1, block_call: 1, puts: 1.1
+  end
+
   def test_process_iter_dsl
     # task :blah do
     #   something
@@ -465,7 +476,7 @@ class TestFlog < FlogTest
   end
 
   def test_process_lit_complex
-    sexp = s(:lit, (0+1i))
+    sexp = eval 's(:lit, 0+1i)' # stupid emacs is borking on the complex
     assert_process sexp, 0.25
   end
 
@@ -571,6 +582,10 @@ class TestFlog < FlogTest
     end
   end
 
+  def assert_parse sexp, ruby
+    assert_equal sexp, Flog::NotRubyParser.new.process(ruby)
+  end
+
   def assert_process sexp, score = -1, hash = {}
     setup
     @flog.process sexp
@@ -697,3 +712,15 @@ class TestFlog < FlogTest
     @flog.add_to_score "blah", 42
   end
 end
+
+class TestFlogDefaultOptions < Minitest::Test
+  def test_flog_with_default_options
+    # Regression test for issue #84
+    # Flog should work with default options (no explicit parser)
+    flog = Flog.new
+    flog.flog_ruby "2 + 3", "test.rb"
+    flog.calculate_total_scores
+
+    assert_in_epsilon 1.6, flog.total_score
+  end
+end

data/test/test_flog_cli.rb

@@ -9,7 +9,7 @@ end
 
 class TestFlogCLI < FlogTest
   def setup
-    @flog = FlogCLI.new parser: NotRubyParser
+    @flog = FlogCLI.new
   end
 
   def test_cls_parse_options

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: flog
 version: !ruby/object:Gem::Version
-  version: 4.9.2
+  version: 4.9.4
 platform: ruby
 authors:
 - Ryan Davis
@@ -9,9 +9,9 @@ bindir: bin
 cert_chain:
 - |
   -----BEGIN CERTIFICATE-----
-  MIIDPjCCAiagAwIBAgIBCTANBgkqhkiG9w0BAQsFADBFMRMwEQYDVQQDDApyeWFu
+  MIIDPjCCAiagAwIBAgIBCjANBgkqhkiG9w0BAQsFADBFMRMwEQYDVQQDDApyeWFu
   ZC1ydWJ5MRkwFwYKCZImiZPyLGQBGRYJemVuc3BpZGVyMRMwEQYKCZImiZPyLGQB
-  GRYDY29tMB4XDTI1MDEwNjIzMjcwMVoXDTI2MDEwNjIzMjcwMVowRTETMBEGA1UE
+  GRYDY29tMB4XDTI2MDEwNzAxMDkxNFoXDTI3MDEwNzAxMDkxNFowRTETMBEGA1UE
   AwwKcnlhbmQtcnVieTEZMBcGCgmSJomT8ixkARkWCXplbnNwaWRlcjETMBEGCgmS
   JomT8ixkARkWA2NvbTCCASIwDQYJKoZIhvcNAQEBBQADggEPADCCAQoCggEBALda
   b9DCgK+627gPJkB6XfjZ1itoOQvpqH1EXScSaba9/S2VF22VYQbXU1xQXL/WzCkx
@@ -21,12 +21,12 @@ cert_chain:
   qhtV7HJxNKuPj/JFH0D2cswvzznE/a5FOYO68g+YCuFi5L8wZuuM8zzdwjrWHqSV
   gBEfoTEGr7Zii72cx+sCAwEAAaM5MDcwCQYDVR0TBAIwADALBgNVHQ8EBAMCBLAw
   HQYDVR0OBBYEFEfFe9md/r/tj/Wmwpy+MI8d9k/hMA0GCSqGSIb3DQEBCwUAA4IB
-  AQAC0WQJcPOWPFwkojhzweilRVjTJ19UiLhiBTw3C1wJO3LVdBkWDmnnhAmKuX4D
-  r7vjQvESlABGIPdutI1Yl7mrHQzTkfLfXvNN6MT0nLChPyIYauT6SZZxubwJrUfA
-  7R0c2CJTIboZ0XaGpLsXqHEF1c29H7TV1QvVuqKAN2mCjh4N82QVn+ZKtys28AwT
-  6GfQX2fqLoi4KSc7xIzHKaNzqxeOICmJofk9w5VZ2rRN6yes8jvFYwz9HR41wdj8
-  bwfinv7Yp5fA6AysuZLhCykyfDuZVRrUp0Vb68YCKsLjJly/Theak+euNTxvHsB+
-  al9oSgPPHICMEX65qvLywitx
+  AQA/X8/PKTTc/IkYQEUL6XWtfK8fAfbuLJzmLcz6f2ZWrtBvPsYvqRuwI1bWUtil
+  2ibJEfIuSIHX6BsUTX18+hlaIussf6EWq/YkE7e2BKmgQE42vSOZMce0kCEAPbx0
+  rQtqonfWTHQ8UbQ7GqCL3gDQ0QDD2B+HUlb4uaCZ2icxqa/eOtxMvHC2tj+h0xKY
+  xL84ipM5kr0bHGf9/MRZJWcw51urueNycBXu1Xh+pEN8qBmYsFRR4SIODLClybAO
+  6cbm2PyPQgKNiqE4H+IQrDVHd9bJs1XgLElk3qoaJBWXc/5fy0J1imYb25UqmiHG
+  snGe1hrppvBRdcyEzvhfIPjI
   -----END CERTIFICATE-----
 date: 1980-01-02 00:00:00.000000000 Z
 dependencies:

metadata.gz.sig

@@ -1 +1 @@
-K���I V[U_R"�x�Ɠyj�#��0GQ�ˆPqs{��`�W�:���c�!=���`it��>d���K`�>̘w�����rY�r�9���p��7[���ם��&pt ��\���D,�C������PuJ�ԁ�}�F�͎o��-�N�Q�j@�qs������%.���'k����QzN����ŗ�7�|��T'����C������$%��f�1M�9� �`
�H�uS�{�����8s�=�ΌX�X������
+o�Z��q��˧T4}P�E#?hO������r�v��eZ�Ԡ�lÛ�BwK4���G-�~���6`�ӥ4���EB� H:,,�u�Ac���x��M�A���Ǜ�j����v����	�
k��2y<V,��$C0��-둴��s��#��;7�*���n�4RX��y;�����j�Gq�>L��z�}��|FG��r��l-���VH?9��;YQ�i��#,�}��0qP
pyR��Fv3R ��18o����h0T'�