feat(composer): add check for bin-dir

[minorOffense]

Alright I'm going to preface this with this is the first time I've ever written Go code so hopefully it's not too terrible.

But this adds a method into the launcher to check composer for the bin-dir flag in the composer file. Otherwise it'll fallback to vendor/bin

[dasginganinja]

I'm interested in bringing this one in. I think this aligns with what we're trying to do here.

Let me finish up on the --root= code we have in progress since we've already removed the flag module. Once that is done I think this should merge in nicely.

Would you be able to write some tests for this version? I might need to pitch in here, too. 😆

[minorOffense]

I could take a shot at it. Looks like i'm going to need a composer.json to store in here. Suppose I should just grab the one from the core-recommended

[dasginganinja]

We also need to make sure that situations where folks sanitize their composer.json works as well.

[minorOffense]

Sorry what do you mean by sanitize the composer.json files? Like they remove them? Or do you mean they remove values from the json structure.

[dasginganinja]

@minorOffense What I mean is some people may remove files like composer.json from production. I can tell you that I remove things like install.php, .git/ and other stuff when we push to production in as part of our internal CI process.

I'd just like to make sure this works when there is no composer.json :) That's what the tests seemed to be complaining about, at least from what I saw.

[minorOffense]

I think I fixed it... I can't run the tests locally. But I updated the main.go to fallback to vendor/bin if the composer finder doesn't find anything.