Coalesce "current pipeline or redis" into the redis method itself

Noting that Proxying subclasses that (mistakenly, perhaps) talk to redis
directly will need to take care to check for pipeline presence as well.

Also remove the redis attr reader from Proxying to prevent mistaken
calling of redis without knowledge of the current MULTI state.

Author: lewispb

lib/kredis/types/proxy.rb

@@ -2,7 +2,7 @@ class Kredis::Types::Proxy
   require_relative "proxy/failsafe"
   include Failsafe
 
-  attr_accessor :redis, :key
+  attr_accessor :key
 
   thread_mattr_accessor :pipeline
 
@@ -23,12 +23,16 @@ def multi(*args, **kwargs, &block)
   def method_missing(method, *args, **kwargs)
     Kredis.instrument :proxy, **log_message(method, *args, **kwargs) do
       failsafe do
-        (pipeline || redis).public_send method, key, *args, **kwargs
+        redis.public_send method, key, *args, **kwargs
       end
     end
   end
 
   private
+    def redis
+      pipeline || @redis
+    end
+
     def log_message(method, *args, **kwargs)
       args      = args.flatten.reject(&:blank?).presence
       kwargs    = kwargs.reject { |_k, v| v.blank? }.presence

lib/kredis/types/proxying.rb

@@ -1,14 +1,14 @@
 require "active_support/core_ext/module/delegation"
 
 class Kredis::Types::Proxying
-  attr_accessor :proxy, :redis, :key
+  attr_accessor :proxy, :key
 
   def self.proxying(*commands)
     delegate *commands, to: :proxy
   end
 
   def initialize(redis, key, **options)
-    @redis, @key = redis, key
+    @key = key
     @proxy = Kredis::Types::Proxy.new(redis, key)
     options.each { |key, value| send("#{key}=", value) }
   end