appmap 0.99.1 → 0.99.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ff070f04eeeb2d79a08d4f93462a84e277dfad345bffd1d6b87bd063a63b8f00
-  data.tar.gz: 27e47f4e5a85fce938cbd259bf76cce952490851cfee98315fddf6d6b1c798d3
+  metadata.gz: c0d8aa81209b5f8d33fc480c3ff3b7f6cd61027dc51fad74c9ad4f19ba6c7faf
+  data.tar.gz: 5d8ac162d8cf5eacfc045ff81a16c2b473df79ae0619ddaface5b4bab8d422cb
 SHA512:
-  metadata.gz: 97db1ff69373241c3e2ca4633fcca4c800d72b60734bde3b32aae2d3765e86f484e490a4148b98c9e90c40b3fa68cc9b734e56ce9175d1e26ec9bfc767b9707e
-  data.tar.gz: 074b036daecfc348fbe13510ab7c6cca46fe8d2d630bd81e28808ef4f05e15b47dedda31e9d3a2e423f426ba0980ea04e50439b3533b8b7690c9047be199a1a9
+  metadata.gz: 34c7b6b0fd6e45c33158776e3c3f1a8725031913ad49856f24aa5af50cbc41efd2e047face5c3c1f02a9c20670af08be43492e5bd665a17323578c735b07900b
+  data.tar.gz: 0ea3f5be3436ea8eec1d8e14cd959b67e6e4547d03de6401c6e21250b3ba91b521b57f665298e1e4ec3be9616a1fbcf0a2f11b8ff09a5c53cd0d60c2d4a358f6

data/CHANGELOG.md

@@ -1,3 +1,10 @@
+## [0.99.2](https://github.com/getappmap/appmap-ruby/compare/v0.99.1...v0.99.2) (2023-05-10)
+
+
+### Bug Fixes
+
+* Ensure that signature hash key consists of strings ([acb5db9](https://github.com/getappmap/appmap-ruby/commit/acb5db9ecb0a6cdf40de83bf506f45f5283f641b))
+
 ## [0.99.1](https://github.com/getappmap/appmap-ruby/compare/v0.99.0...v0.99.1) (2023-04-24)
 
 

data/lib/appmap/config.rb

@@ -499,6 +499,11 @@ module AppMap
     end
 
     def never_hook?(cls, method)
+      unless method
+        HookLog.log "method is nil" if HookLog.enabled?
+        return true
+      end
+
       _, separator, = ::AppMap::Hook.qualify_method_name(method)
       if exclude.member?(cls.name) || exclude.member?([ cls.name, separator, method.name ].join)
         HookLog.log "Hooking of #{method} disabled by configuration" if HookLog.enabled?

data/lib/appmap/hook/method.rb

@@ -4,11 +4,23 @@ require 'appmap/util'
 
 module AppMap
   class Hook
+    class << self
+      def method_hash_key(cls, method)
+        [ cls, method.name ].hash
+      rescue TypeError => e
+        warn "Error building hash key for #{cls}, #{method}: #{e}"
+      end
+    end
+    
+
     SIGNATURES = {}
     LOOKUP_SIGNATURE = lambda do |id|
       method = super(id)
 
-      signature = SIGNATURES[[ method.owner, method.name ]]
+      hash_key = Hook.method_hash_key(method.owner, method)
+      return method unless hash_key
+
+      signature = SIGNATURES[hash_key]
       if signature
         method.singleton_class.module_eval do
           define_method(:parameters) do
@@ -48,7 +60,8 @@ module AppMap
         end
 
         hook_method_parameters = hook_method.parameters.dup.freeze
-        SIGNATURES[[ hook_class, hook_method.name ]] = hook_method_parameters
+        hash_key = Hook.method_hash_key(hook_class, hook_method)
+        SIGNATURES[hash_key] = hook_method_parameters if hash_key
 
         # irb(main):001:0> Kernel.public_instance_method(:system)
         # (irb):1:in `public_instance_method': method `system' for module `Kernel' is  private (NameError)

data/lib/appmap/version.rb

@@ -3,7 +3,7 @@
 module AppMap
   URL = 'https://github.com/applandinc/appmap-ruby'
 
-  VERSION = '0.99.1'
+  VERSION = '0.99.2'
 
   APPMAP_FORMAT_VERSION = '1.12.0'
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: appmap
 version: !ruby/object:Gem::Version
-  version: 0.99.1
+  version: 0.99.2
 platform: ruby
 authors:
 - Kevin Gilpin
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2023-04-24 00:00:00.000000000 Z
+date: 2023-05-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: method_source