Idea: prune flagged autoload files

[ChrisLightfootWild]

Hey 👋

I wanted to propose additional functionality to prune any composer autoload files which become redundant if the spi plugin is allowed to run. I suspect that isn't always the case, but for certain packages we could remove the (usually minor) overhead of the autoload file which would otherwise be loaded at every runtime.

A couple of examples which OpenTelemetry packages may benefit from:

• otlp exporter
• Laravel auto-instrumentation

---

I started an implementation which I was playing with as a proof-of-concept:

ChrisLightfootWild/opentelemetry-php@6125b71

However, it seems like behaviour which would make more sense to reside within the SPI package directly.

Thanks!

[Nevay]

34ea960 adds two configuration options:

• extra.spi-config.prune-autoload-files basically does the same as your proof-of-concept
• extra.spi-config.autoload-files creates a static map from ServiceLoader::register() calls within autoload.files,
  which removes the need to duplicate service definitions in extra.spi and ServiceLoader::register().

Using open-telemetry/opentelemetry-php-contrib#269 as example, should be able to use:

  "extra": {
+   "spi-config": {
+     "autoload-files": true
+   },
    "spi": {
      "OpenTelemetry\\Config\\SDK\\Configuration\\ComponentProvider": [
        "OpenTelemetry\\Contrib\\Instrumentation\\Laravel\\LaravelComponentProvider"
-     ],
-     "OpenTelemetry\\API\\Instrumentation\\AutoInstrumentation\\Instrumentation": [
-       "OpenTelemetry\\Contrib\\Instrumentation\\Laravel\\LaravelInstrumentation"
-     ],
-     "OpenTelemetry\\Contrib\\Instrumentation\\Laravel\\Hooks\\Hook": [
-       "OpenTelemetry\\Contrib\\Instrumentation\\Laravel\\Hooks\\Illuminate\\Console\\Command",
-
-       "OpenTelemetry\\Contrib\\Instrumentation\\Laravel\\Hooks\\Illuminate\\Contracts\\Console\\Kernel",
-       "OpenTelemetry\\Contrib\\Instrumentation\\Laravel\\Hooks\\Illuminate\\Contracts\\Http\\Kernel",
-       "OpenTelemetry\\Contrib\\Instrumentation\\Laravel\\Hooks\\Illuminate\\Contracts\\Queue\\Queue",
-
-       "OpenTelemetry\\Contrib\\Instrumentation\\Laravel\\Hooks\\Illuminate\\Foundation\\Console\\ServeCommand",
-       "OpenTelemetry\\Contrib\\Instrumentation\\Laravel\\Hooks\\Illuminate\\Foundation\\Application",
-
-       "OpenTelemetry\\Contrib\\Instrumentation\\Laravel\\Hooks\\Illuminate\\Queue\\Queue",
-       "OpenTelemetry\\Contrib\\Instrumentation\\Laravel\\Hooks\\Illuminate\\Queue\\SyncQueue",
-       "OpenTelemetry\\Contrib\\Instrumentation\\Laravel\\Hooks\\Illuminate\\Queue\\Worker"
      ]
    }
  },

[ChrisLightfootWild]

@Nevay amazing, thank you for doing this. I have just tried it out and it's working as desired ❤️