Add Kernel#tee, Tee alias, inline Array#remove, remove tap override

- Add Kernel#tee as block-less method chaining via Tee/Functor
- Add Tee as alias for Functor (gradual rename)
- Remove Kernel#tap override (use #tee instead)
- Inline delete_first logic in Array#remove! to avoid dependency

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: trans

HISTORY.md

@@ -15,6 +15,8 @@ Changes:
   * Add `Array#indexes` / `Array#index_all` to find all matching indexes. (PR#294)
   * Add `String#dashcase` for kebab-case conversion. (PR#297)
   * Add `Binding#caller_locations`.
+  * Add `Kernel#tee` — block-less method chaining via Tee/Functor, replaces `tap` override.
+  * Add `Tee` as alias for `Functor` (gradual rename).
 
 * Improved Features
 
@@ -32,6 +34,7 @@ Changes:
 
 * Removals
 
+  * Remove `Kernel#tap` override. Use `Kernel#tee` for the block-less Functor form.
   * Remove `Enumerable#filter` (conflicts with Ruby's built-in `filter` since 2.6).
     Use `each_with_object` instead, with block arguments reversed:
     `collection.each_with_object([]) { |el, out| out << el if cond }`.

lib/core/facets/array/remove.rb

@@ -31,7 +31,8 @@ def remove(other_ary)
   #
   def remove!(other_ary)
     other_ary.each do |el|
-      delete_first(el)
+      i = index(el)
+      delete_at(i) if i
     end
     self
   end

lib/core/facets/functor.rb

@@ -90,3 +90,9 @@ def method_missing(op, *args, &blk)
   end
 
 end
+
+# Tee is an alias for Functor. The name comes from the Unix `tee` command,
+# which forks a data stream — here it forks a method call for side effects
+# while passing the original receiver through. Functor will be gradually
+# deprecated in favor of Tee.
+Tee = Functor

lib/core/facets/kernel/tap.rb

@@ -1,36 +1,3 @@
-require 'facets/functor'
-
-module Kernel
-
-  # The tap K-Combinator. This yields self -and- returns self.
-  #
-  #   'foo.yml'.tap{ |f| YAML.load(f) }  #=> 'foo.yml'
-  #
-  # Unlike Ruby's definition, this rendition can be used as a higher
-  # order message. This form allows a single call before returning
-  # the receiver.
-  #
-  #   YAML.tap.load_file('foo.yml').load_file('bar.yml')
-  #
-  # IMPORTANT: This is one of few core overrides in Facets.
-
-  def tap #:yield:
-    if block_given?
-      yield(self)
-      self
-    else
-      Functor.new{ |op,*a,&b| self.send(op, *a, &b); self }
-    end
-  end
-
-  # Old Definition:
-  #
-  #   def tap #:yield:
-  #     if block_given?
-  #       b.arity == 1 ? yield(self) : instance_eval(&b)
-  #     end
-  #     return self
-  #   end
-  #
-
-end
+# Facets no longer overrides Kernel#tap. Use Kernel#tee instead
+# for the block-less Functor/Tee form.
+require 'facets/kernel/tee'

lib/core/facets/kernel/tee.rb

@@ -0,0 +1,18 @@
+require 'facets/functor'
+
+module Kernel
+
+  # Returns a Tee (Functor) that intercepts method calls, forwards
+  # them to self for side effects, and returns self. This is like
+  # a block-less version of #tap that allows method chaining.
+  #
+  #   YAML.tee.load_file('foo.yml').load_file('bar.yml')
+  #
+  # This is analogous to the Unix `tee` command — the data flows
+  # through while side effects are forked off.
+
+  def tee
+    Tee.new{ |op,*a,&b| self.send(op, *a, &b); self }
+  end
+
+end