churn_vs_complexity 1.6.1 → 1.6.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6dfa844a5bcd2fe116001652410d39f802c64623be5ec588afcc6ed331b9aac9
-  data.tar.gz: 42c5d56c7b4c6640bd4040f7f5d8b35ea6acac0aae9285be4c6cb1aaf860f0fb
+  metadata.gz: 3d0895177148a918e4aab91d71e25978c2f0429d8f46920851d3aee7eb8a9525
+  data.tar.gz: 32df8627394b7f85cdf7aa81093ba560bef79309efa10a606941f298e176bf34
 SHA512:
-  metadata.gz: 0ad833af0897a784aa0499d277884e011e9be71142d343c441ef0a12b1185c27f18c09b0b7780a3f8493e3ef0399faff5f97b252f51fb4713f3dd98d11b666a7
-  data.tar.gz: 19cf09a9777aa06a4772fae21fee5a4043ba034efde9d2af13618461b20a602270879438c4aa626463c3ea14054bf1dc8a7b9a25cde09c9636439d9d567a30f3
+  metadata.gz: 1bdb515d824ab01dcd9f279f072d430c9dbb5a386bde4c569e5821b63895eefb256e26d22c79f1da65c59e1c0508057487db5008155de946ea54652ca1947687
+  data.tar.gz: d3c5c601f54b852d5ed47d683e1e9c8d88526fce57191caab66f553c3326d9166008f3c409a7e1178535d290908b95f1780599af763c8bb1d995ff1117e38457

data/CHANGELOG.md

@@ -1,3 +1,11 @@
+## [1.6.2] - 2026-02-19
+
+### Changed
+- Risk classification in triage, hotspots, and focus modes is now language-aware
+- Ruby thresholds raised to low=30/high=70 (Flog scores 3-7x higher than cyclomatic tools)
+- Go thresholds raised to low=15/high=40 (cognitive complexity penalises nesting)
+- Java, Python, and JavaScript/TypeScript thresholds unchanged (low=10/high=25)
+
 ## [1.6.1] - 2026-02-19
 
 ### Changed

data/lib/churn_vs_complexity/focus/checker.rb

@@ -5,11 +5,12 @@ require 'json'
 module ChurnVsComplexity
   module Focus
     class Checker
-      def initialize(engine:, subcommand:, serializer:, baseline_path:)
+      def initialize(engine:, subcommand:, serializer:, baseline_path:, language: nil)
         @engine = engine
         @subcommand = subcommand
         @serializer = serializer
         @baseline_path = baseline_path
+        @language = language
       end
 
       def check(folder:)
@@ -24,7 +25,7 @@ module ChurnVsComplexity
 
       def run_start(folder, baseline_path)
         raw_result = @engine.check(folder:)
-        entries = RiskAnnotator.annotate(raw_result[:values_by_file])
+        entries = RiskAnnotator.annotate(raw_result[:values_by_file], language: @language)
 
         baseline = {
           timestamp: Time.now.utc.iso8601,
@@ -37,7 +38,7 @@ module ChurnVsComplexity
 
       def run_end(folder, baseline_path)
         raw_result = @engine.check(folder:)
-        current_entries = RiskAnnotator.annotate(raw_result[:values_by_file])
+        current_entries = RiskAnnotator.annotate(raw_result[:values_by_file], language: @language)
 
         baseline = load_baseline(baseline_path)
         @serializer.serialize(baseline:, current: current_entries)

data/lib/churn_vs_complexity/focus/config.rb

@@ -40,6 +40,7 @@ module ChurnVsComplexity
           subcommand: @subcommand,
           serializer: focus_serializer,
           baseline_path: @baseline_path,
+          language: @language,
         )
       end
 

data/lib/churn_vs_complexity/hotspots/checker.rb

@@ -3,14 +3,15 @@
 module ChurnVsComplexity
   module Hotspots
     class Checker
-      def initialize(engine:, serializer:)
+      def initialize(engine:, serializer:, language: nil)
         @engine = engine
         @serializer = serializer
+        @language = language
       end
 
       def check(folder:)
         raw_result = @engine.check(folder:)
-        @serializer.serialize(raw_result)
+        @serializer.serialize(raw_result.merge(language: @language))
       end
     end
   end

data/lib/churn_vs_complexity/hotspots/config.rb

@@ -34,7 +34,7 @@ module ChurnVsComplexity
           since: @since,
           excluded: @excluded,
         )
-        Checker.new(engine: normal_config.checker, serializer: hotspots_serializer)
+        Checker.new(engine: normal_config.checker, serializer: hotspots_serializer, language: @language)
       end
 
       private

data/lib/churn_vs_complexity/hotspots/serializer.rb

@@ -7,7 +7,7 @@ module ChurnVsComplexity
     module Serializer
       module Json
         def self.serialize(result)
-          entries = RiskAnnotator.annotate(result[:values_by_file])
+          entries = RiskAnnotator.annotate(result[:values_by_file], language: result[:language])
           entries.sort_by! { |e| -e[:gamma_score] }
 
           JSON.generate({ generated: Time.now.utc.iso8601, files: entries,
@@ -23,7 +23,7 @@ module ChurnVsComplexity
         }.freeze
 
         def self.serialize(result)
-          entries = RiskAnnotator.annotate(result[:values_by_file])
+          entries = RiskAnnotator.annotate(result[:values_by_file], language: result[:language])
           entries.sort_by! { |e| -e[:gamma_score] }
           grouped = entries.group_by { |e| e[:risk] }
 

data/lib/churn_vs_complexity/risk_annotator.rb

@@ -2,7 +2,7 @@
 
 module ChurnVsComplexity
   module RiskAnnotator
-    def self.annotate(values_by_file, classifier: RiskClassifier.new)
+    def self.annotate(values_by_file, language: nil, classifier: RiskClassifier.new(language: language))
       values_by_file.map { |file, values| build_entry(file, values, classifier) }
     end
 

data/lib/churn_vs_complexity/risk_classifier.rb

@@ -1,19 +1,43 @@
 # frozen_string_literal: true
 
 module ChurnVsComplexity
+  # Classifies files into low/medium/high risk based on gamma score
+  # (harmonic mean of churn and complexity).
+  #
+  # IMPORTANT: Complexity scores are NOT comparable across languages.
+  # Each language uses a different tool with a different numeric scale.
+  # All tools sum per-function scores across the file, but the magnitude differs:
+  #
+  #   Language       Tool      Metric                Trivial  Complex (~100 LOC)  Real-world range
+  #   Ruby           Flog      Code pain (weighted)  0        ~286                0-88
+  #   JavaScript/TS  ESLint    Cyclomatic complexity  3        ~62                 10-50
+  #   Python         Radon     Cyclomatic complexity  1        ~44                 5-50
+  #   Java           PMD       Cyclomatic complexity  1        ~39                 1-40
+  #   Go             gocognit  Cognitive complexity   0        ~87                 0-50
+  #
+  # The DEFAULT_LOW and DEFAULT_HIGH thresholds below are rough midpoints
+  # suitable for Java/Python/JS. They are too aggressive for Ruby (Flog
+  # scores 3-7x higher) and Go (cognitive complexity penalises nesting).
+  # Use the constructor to pass language-appropriate thresholds.
   class RiskClassifier
     DEFAULT_LOW = 10
     DEFAULT_HIGH = 25
 
+    LANGUAGE_DEFAULTS = {
+      ruby: { low: 30, high: 70 },
+      go: { low: 15, high: 40 },
+    }.freeze
+
     RECOMMENDATIONS = {
       'low' => 'Safe for quick changes.',
       'medium' => 'Exercise judgement; consider tests for non-trivial changes.',
       'high' => 'Write tests before modifying. Consider multi-agent review.',
     }.freeze
 
-    def initialize(low: DEFAULT_LOW, high: DEFAULT_HIGH)
-      @low = low
-      @high = high
+    def initialize(low: DEFAULT_LOW, high: DEFAULT_HIGH, language: nil)
+      defaults = LANGUAGE_DEFAULTS.fetch(language, {})
+      @low = defaults.fetch(:low, low)
+      @high = defaults.fetch(:high, high)
     end
 
     def classify(gamma_score:)

data/lib/churn_vs_complexity/triage/checker.rb

@@ -16,7 +16,7 @@ module ChurnVsComplexity
         folder = folder || dirs.first || '.'
         engine = build_engine(files, folder)
         raw_result = engine.check(folder:)
-        @serializer.serialize(raw_result)
+        @serializer.serialize(raw_result.merge(language: @language))
       end
 
       private

data/lib/churn_vs_complexity/triage/serializer.rb

@@ -7,7 +7,7 @@ module ChurnVsComplexity
     module Serializer
       module Json
         def self.serialize(result)
-          entries = RiskAnnotator.annotate(result[:values_by_file])
+          entries = RiskAnnotator.annotate(result[:values_by_file], language: result[:language])
           JSON.generate({ files: entries, summary: RiskAnnotator.risk_summary(entries) })
         end
       end

data/lib/churn_vs_complexity/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ChurnVsComplexity
-  VERSION = '1.6.1'
+  VERSION = '1.6.2'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: churn_vs_complexity
 version: !ruby/object:Gem::Version
-  version: 1.6.1
+  version: 1.6.2
 platform: ruby
 authors:
 - Erik T. Madsen