audition 0.3.0 → 0.4.1

data/lib/audition/target.rb

@@ -12,6 +12,12 @@ module Audition
       vendor node_modules tmp log coverage pkg .git .bundle
     ].freeze
 
+    # Directories holding the target's tests rather than the code
+    # a production boot loads. The default; `test_dirs` in
+    # .audition.yml replaces it for a project that files them
+    # elsewhere.
+    TEST_DIRS = %w[test spec features].freeze
+
     # Cargo and Zig build output: full of .so/.dylib artifacts that
     # are not the extension `require` loads.
     BUILD_DIRS = %w[target zig-out].freeze
@@ -20,6 +26,11 @@ module Audition
     # "bundle" on macOS, "so" everywhere else, Windows included).
     COMPILED = "*.{bundle,so}"
 
+    # Generated type stubs. The extension is the convention; where
+    # the generator writes them is not, so they are found by
+    # extension anywhere in the tree.
+    STUBS = "*.rbi"
+
     # @return [Symbol] one of `:script`, `:gem`, `:rack`, `:rails`,
     #   `:directory`, `:bundle`
     attr_reader :type
@@ -34,6 +45,10 @@ module Audition
     #   shipped with the target; empty for scripts and file lists
     attr_reader :compiled_files
 
+    # @return [Array<String>] generated type stubs (.rbi) found
+    #   anywhere in the target's tree
+    attr_reader :stub_files
+
     # @return [Hash, nil] dynamic probe entry (`:mode` plus
     #   mode-specific keys), nil for static-only targets
     attr_reader :entry
@@ -75,7 +90,8 @@ module Audition
         type: :files,
         root: Dir.pwd,
         ruby_files: paths,
-        entry: nil
+        entry: nil,
+        stub_files: stubs(Dir.pwd)
       )
     end
 
@@ -96,10 +112,26 @@ module Audition
         type: :script,
         root: File.dirname(path),
         ruby_files: [path],
-        entry: {mode: :script, path: path}
+        # A file inside a Rails root cannot run standalone; the
+        # script probe would report a misleading boot failure, so
+        # such files stay static-only.
+        entry: rails_member?(path) ? nil : {mode: :script, path: path},
+        stub_files: stubs(File.dirname(path))
       )
     end
 
+    def self.rails_member?(path)
+      dir = File.dirname(File.expand_path(path))
+      until dir == (parent = File.dirname(dir))
+        return true if File.file?(
+          File.join(dir, "config", "application.rb")
+        )
+
+        dir = parent
+      end
+      false
+    end
+
     def self.from_directory(dir)
       application_rb = File.join(dir, "config", "application.rb")
       config_ru = File.join(dir, "config.ru")
@@ -117,7 +149,8 @@ module Audition
           root: dir,
           ruby_files: glob(dir),
           entry: nil,
-          compiled_files: compiled(dir)
+          compiled_files: compiled(dir),
+          stub_files: stubs(dir)
         )
       end
     end
@@ -131,16 +164,21 @@ module Audition
           glob(File.join(spec.full_gem_path, rp))
         end,
         entry: {mode: :require, feature: name,
+                load_paths: spec.full_require_paths,
                 root: spec.full_gem_path},
-        compiled_files: compiled_for(spec)
+        compiled_files: compiled_for(spec),
+        stub_files: stubs(spec.full_gem_path)
       )
     rescue Gem::MissingSpecError
       raise Error,
         "#{name} is not a file, directory, or installed gem"
     end
 
+    # An app's own Ruby is not confined to app/lib/config: local
+    # gems, engines and tests boot into the same process, so the
+    # whole root is scanned.
     def self.rails_target(dir)
-      files = %w[app lib config].flat_map { |d| glob(File.join(dir, d)) }
+      files = glob(dir)
       files << File.join(dir, "config.ru")
       new(
         type: :rails,
@@ -151,7 +189,8 @@ module Audition
           environment: File.join(dir, "config", "environment.rb"),
           root: dir
         },
-        compiled_files: compiled(dir)
+        compiled_files: compiled(dir),
+        stub_files: stubs(dir)
       )
     end
 
@@ -160,24 +199,31 @@ module Audition
         type: :rack,
         root: dir,
         ruby_files: [config_ru] + glob(dir),
-        entry: {mode: :rack, config_ru: config_ru},
-        compiled_files: compiled(dir)
+        entry: {mode: :rack, config_ru: config_ru, root: dir},
+        compiled_files: compiled(dir),
+        stub_files: stubs(dir)
       )
     end
 
+    # Reading require_paths off the gemspec would mean evaluating
+    # it, which the static pass never does; `lib` is the packaging
+    # default, and a gem that keeps its code elsewhere falls back to
+    # the whole checkout rather than scanning nothing.
     def self.gem_dir_target(dir, gemspec)
       lib = File.join(dir, "lib")
+      paths = File.directory?(lib) ? [lib] : [dir]
       new(
         type: :gem,
         root: dir,
-        ruby_files: glob(lib),
+        ruby_files: paths.flat_map { |path| glob(path) },
         entry: {
           mode: :require,
           feature: File.basename(gemspec, ".gemspec"),
-          load_paths: [lib],
+          load_paths: paths,
           root: dir
         },
-        compiled_files: compiled(dir)
+        compiled_files: compiled(dir),
+        stub_files: stubs(dir)
       )
     end
 
@@ -199,6 +245,14 @@ module Audition
       end.sort
     end
 
+    # Generated type stubs anywhere under a directory.
+    #
+    # @param dir [String]
+    # @return [Array<String>]
+    def self.stubs(dir)
+      glob(dir, STUBS)
+    end
+
     # macOS debug-symbol bundles (x.bundle.dSYM/...) carry a file
     # with the extension's name that nothing ever loads.
     def self.compiled(dir)
@@ -222,15 +276,30 @@ module Audition
 
     private_class_method :from_file, :from_directory, :from_gem_name,
       :rails_target, :rack_target, :gem_dir_target,
-      :glob, :compiled, :normalize
+      :rails_member?, :glob, :compiled, :normalize
 
     def initialize(type:, root:, ruby_files:, entry:,
-      compiled_files: [])
+      compiled_files: [], stub_files: [])
       @type = type
       @root = root
       @ruby_files = ruby_files
       @entry = entry
       @compiled_files = compiled_files
+      @stub_files = stub_files
+    end
+
+    # Whether a path is test/spec code rather than code the
+    # production boot loads.
+    #
+    # @param path [String] absolute or root-relative
+    # @param dirs [Array<String>] directory names that hold tests
+    # @return [Boolean]
+    def test_file?(path, dirs: TEST_DIRS)
+      relative = File.expand_path(path, root)
+        .delete_prefix("#{root}/")
+      parts = relative.split("/")
+      parts[0..-2].any? { |p| dirs.include?(p) } ||
+        parts.last.to_s.end_with?("_test.rb", "_spec.rb")
     end
   end
 end

data/lib/audition/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Audition
-  VERSION = "0.3.0"
+  VERSION = "0.4.1"
 end

data/lib/audition.rb

@@ -1,15 +1,18 @@
 # frozen_string_literal: true
 
 require_relative "audition/version"
+require_relative "audition/progress"
 require_relative "audition/finding"
 require_relative "audition/target"
 require_relative "audition/static/source_file"
 require_relative "audition/static/literal_classifier"
 require_relative "audition/static/checks"
 require_relative "audition/static/graph_audit"
+require_relative "audition/static/gem_calls"
 require_relative "audition/static/native_extensions"
 require_relative "audition/static/analyzer"
 require_relative "audition/dynamic/prober"
+require_relative "audition/reconciliation"
 require_relative "audition/report"
 require_relative "audition/config"
 require_relative "audition/baseline"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: audition
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.1
 platform: ruby
 authors:
 - Yaroslav Markin
@@ -43,14 +43,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0.2'
+        version: '0.4'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0.2'
+        version: '0.4'
 - !ruby/object:Gem::Dependency
   name: table_tennis
   requirement: !ruby/object:Gem::Requirement
@@ -102,24 +102,32 @@ files:
 - lib/audition/dynamic/prober.rb
 - lib/audition/finding.rb
 - lib/audition/fixer.rb
+- lib/audition/progress.rb
+- lib/audition/reconciliation.rb
 - lib/audition/report.rb
 - lib/audition/report/github.rb
 - lib/audition/report/json.rb
 - lib/audition/report/style.rb
+- lib/audition/report/sweep.rb
 - lib/audition/report/text.rb
 - lib/audition/rewriters.rb
 - lib/audition/static/analyzer.rb
 - lib/audition/static/checks.rb
 - lib/audition/static/checks/base.rb
+- lib/audition/static/checks/dependency_class_state.rb
 - lib/audition/static/checks/global_variables.rb
+- lib/audition/static/checks/instance_memoization.rb
 - lib/audition/static/checks/mutable_constants.rb
 - lib/audition/static/checks/ractor_isolation.rb
 - lib/audition/static/checks/runtime_require.rb
 - lib/audition/static/checks/unsafe_calls.rb
+- lib/audition/static/checks/unshareable_reads.rb
+- lib/audition/static/gem_calls.rb
 - lib/audition/static/graph_audit.rb
 - lib/audition/static/literal_classifier.rb
 - lib/audition/static/native_extensions.rb
 - lib/audition/static/source_file.rb
+- lib/audition/static/work_split.rb
 - lib/audition/target.rb
 - lib/audition/version.rb
 homepage: https://github.com/yaroslav/audition
@@ -142,7 +150,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.16
+rubygems_version: 4.0.20
 specification_version: 4
 summary: Probe Ruby code for Ractor-readiness
 test_files: []