Made Slingshot optional

Signed-off-by: Tom Wright <tom@inflatablecookie.com>

CHANGELOG.md

@@ -1,3 +1,5 @@
+* Made Slingshot optional
+
 ## v0.10.25 (2024-04-29)
 * Fixed re-adding early-bound instances to container
 

README.md

@@ -100,6 +100,8 @@ namespace Some\Other\Code
 }
 ```
 
+Note, if your plugin has a constructor with parameters, you will need to add <code>decodelabs/slingshot</code> to your project to allow Veneer to instantiate it.
+
 
 ## Licensing
 Veneer is licensed under the MIT License. See [LICENSE](./LICENSE) for the full license text.

composer.json

@@ -13,16 +13,19 @@
 
         "decodelabs/exceptional": "^0.4",
         "decodelabs/glitch-support": "^0.4",
-        "decodelabs/slingshot": "^0.1.1",
 
         "psr/container": "^2"
     },
     "require-dev": {
         "composer-runtime-api": "^2.2",
 
         "decodelabs/pandora": "^0.2.11",
+        "decodelabs/slingshot": "^0.1.1",
         "decodelabs/phpstan-decodelabs": "^0.6"
     },
+    "suggest": {
+        "decodelabs/slingshot": "Complex plugin instantiation support"
+    },
     "autoload": {
         "psr-4": {
             "DecodeLabs\\": "src/"
@@ -33,7 +36,7 @@
     ],
     "extra": {
         "branch-alias": {
-            "dev-develop": "0.10.x-dev"
+            "dev-develop": "0.11.x-dev"
         }
     }
 }

src/Veneer/Binding.php

@@ -100,13 +100,27 @@ public function resolveDeferral(
         $method = $ref->getMethod('__construct');
         $method->setAccessible(true);
 
+        // Return value doc mismatch
+        // @phpstan-ignore-next-line
         if (!$closure = $method->getClosure($instance)) {
             throw Exceptional::Logic(
                 'Unable to get closure for constructor of ' . $this->providerClass
             );
         }
 
-        (new Slingshot($container))->invoke($closure);
+        if (class_exists(Slingshot::class)) {
+            // Invoke constructor with Slingshot
+            (new Slingshot($container))->invoke($closure);
+        } else {
+            if (!empty($method->getParameters())) {
+                throw Exceptional::ComponentUnavailable(
+                    'Cannot resolve plugin constructor dependencies without Slingshot'
+                );
+            }
+
+            // Invoke constructor directly
+            $closure();
+        }
 
         // Load plugins
         $this->loadPlugins();