feat: Increase compatibility of the require patch

The previous patch did not declare the overwritten require method as
module_function. This led to subtle changes when calling
respond_to?(:require) on object later, which could return true when it
should not.

This commit changes the code to use module_function. It further moves
the loader into a module that is prepended, to avoid the addition of
method aliases.

CHANGELOG.md

@@ -6,6 +6,10 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 
 ## [Unreleased]
 
+### Added
+
+- Increase compatibility of Kernel#require patch
+
 ## [2.0.0] - 2024-02-29
 
 ### Removed

lib/mnemosyne/probes.rb

@@ -21,20 +21,18 @@ def registry
         @registry ||= ::Mnemosyne::Registry.new
       end
     end
-  end
-end
 
-module Kernel
-  alias require_without_mn require
+    module Loader
+      module_function
 
-  def require(name)
-    ret = require_without_mn(name)
-
-    begin
-      ::Mnemosyne::Probes.required(name)
-    rescue Exception # rubocop:disable Lint/RescueException,Lint/SuppressedException
+      def require(name)
+        super(name).tap do
+          ::Mnemosyne::Probes.required(name)
+        rescue Exception # rubocop:disable Lint/RescueException,Lint/SuppressedException
+        end
+      end
     end
 
-    ret
+    ::Kernel.prepend(Loader)
   end
 end

spec/mnemosyne/probes_spec.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe Mnemosyne::Probes do
+  it 'injects the loader into Kernel#require' do
+    expect(Mnemosyne::Probes).to receive(:required).with('mnemosyne/probes')
+    require 'mnemosyne/probes'
+  end
+
+  it 'does not make objects respond to require' do
+    expect(Object.new).not_to respond_to(:require)
+  end
+end