Pass original tap to formula when loaded from the API via TapLoader

Library/Homebrew/formulary.rb

@@ -667,8 +667,8 @@ def klass(flags:, ignore_errors:)
 
   # Load formulae from the API.
   class FormulaAPILoader < FormulaLoader
-    def initialize(name)
-      super name, Formulary.core_path(name)
+    def initialize(name, tap: CoreTap.instance)
+      super name, Formulary.core_path(name), tap: tap
     end
 
     def klass(flags:, ignore_errors:)
@@ -926,9 +926,9 @@ def self.tap_formula_name_path(tapped_name, warn:)
   def self.tap_loader_for(tapped_name, warn:)
     name, path, tap = Formulary.tap_formula_name_path(tapped_name, warn: warn)
 
-    if name.exclude?("/") && !Homebrew::EnvConfig.no_install_from_api? &&
+    if Tap.from_path(path).core_tap? && !Homebrew::EnvConfig.no_install_from_api? &&
        Homebrew::API::Formula.all_formulae.key?(name)
-      FormulaAPILoader.new(name)
+      FormulaAPILoader.new(name, tap: tap)
     else
       TapLoader.new(name, path, tap: tap)
     end

Library/Homebrew/test/formulary_spec.rb

@@ -186,6 +186,48 @@ class Wrong#{described_class.class_s(formula_name)} < Formula
       end
     end
 
+    context "when migrating from a Tap" do
+      let(:tap) { Tap.new("homebrew", "foo") }
+      let(:another_tap) { Tap.new("homebrew", "bar") }
+      let(:tap_migrations_path) { tap.path/"tap_migrations.json" }
+      let(:another_tap_formula_path) { another_tap.path/"Formula/#{formula_name}.rb" }
+
+      before do
+        tap.path.mkpath
+        another_tap_formula_path.dirname.mkpath
+        another_tap_formula_path.write formula_content
+      end
+
+      after do
+        FileUtils.rm_rf tap.path
+        FileUtils.rm_rf another_tap.path
+      end
+
+      it "returns a Formula that has gone through a tap migration into homebrew/core" do
+        tap_migrations_path.write <<~EOS
+          {
+            "#{formula_name}": "homebrew/core"
+          }
+        EOS
+        formula = described_class.factory("#{tap}/#{formula_name}")
+        expect(formula).to be_a(Formula)
+        expect(formula.tap).to eq(tap)
+        expect(formula.path).to eq(formula_path)
+      end
+
+      it "returns a Formula that has gone through a tap migration into another tap" do
+        tap_migrations_path.write <<~EOS
+          {
+            "#{formula_name}": "#{another_tap}"
+          }
+        EOS
+        formula = described_class.factory("#{tap}/#{formula_name}")
+        expect(formula).to be_a(Formula)
+        expect(formula.tap).to eq(tap)
+        expect(formula.path).to eq(another_tap_formula_path)
+      end
+    end
+
     context "when loading from Tap" do
       let(:tap) { Tap.new("homebrew", "foo") }
       let(:another_tap) { Tap.new("homebrew", "bar") }