Can outdated command be forwarded as well?

[hostep]

Hi there

Thank you very much for this wonderful composer plugin, very happy with it so far!

I've noticed with the forward-command config, that only composer install and composer update are being forwarded to the bin directories.
But I sometimes like to run composer outdated as well against everything, and right now I need to do this one by one, which is a bit annoying.
Would be nice if this command can also be forwarded.

Thanks for considering this! 🙂

[theofidry]

That's a good idea. PRs are welcomed! I don't think it requires much more than updating

composer-bin-plugin/src/BamarniBinPlugin.php

Line 36 in 0f2f6f5

private const FORWARDED_COMMANDS = ['install', 'update'];

[hostep]

Ah cool, good to know. I'll see if I can find some time in the next few days/weeks to test this out and send a PR.
Thanks for the info!

[hostep]

Just gave it a little try, it's unfortunately a bit more complex. Because composer outdated is more of an alias to composer show --outdated

So adding outdated to the FORWARDED_COMMANDS const does not work. When I add show to it, it does not work with composer outdated, but does work with composer show --outdated.
And I can see that composer bin all outdated also does not work.

I haven't debugged further since I lack the time at the moment.

What do you think, should we try to make composer outdated also working somehow? It would be convenient in my opinion.

[theofidry]

I think we could adapt this condition:

composer-bin-plugin/src/BamarniBinPlugin.php

Line 134 in 0f2f6f5

&& in_array($commandName, self::FORWARDED_COMMANDS, true)

Instead of just checking the in_array we could check if it's the show command with the outdated flag for example. The check could also be extracted into a dedicated method to keep it understandable

[LastDragon-ru]

And I can see that composer bin all outdated also does not work.

Yep, for me it checks the first package only :(

[llaville]

Related to comment of @LastDragon-ru , here is my experience report

Summary

Cannot forward correctly Composer outdated command to all namespace

Context

With BOX Manifest 4.0.1 (Commit e341cf8 from master branch) : https://github.com/llaville/box-manifest

NB: Even if it's not related to this project, because I get same behaviour with other projects

I've previously installed dependencies on a PHP 8.1 platform (8.1.30)
Then I switch to a PHP 8.2 platform (8.2.25)

Description

When I run following command :

composer bin all outdated -v

I got this output

[bamarni-bin] Current working directory: /shared/backups/bartlett/box-manifest
[bamarni-bin] Configuring bin directory to /shared/backups/bartlett/box-manifest/vendor/bin.
[bamarni-bin] Checking namespace vendor-bin/phpcs
[bamarni-bin] Changed current directory to vendor-bin/phpcs.
[bamarni-bin] Running `@composer outdated -v --working-dir='.'`.

Direct dependencies required in composer.json:
Everything up to date

Transitive dependencies not required in composer.json:
Everything up to date

It list me only the first namespace of vendor-bin while I've many others (ls -l vendor-bin) :

drwxr-xr-x  7 devilbox devilbox 4096 Nov  8 05:10 ./
drwxr-xr-x 17 devilbox devilbox 4096 Nov  8 04:34 ../
drwxr-xr-x  3 devilbox devilbox 4096 Sep 24 04:26 phpcs/
drwxr-xr-x  3 devilbox devilbox 4096 Sep 24 04:26 phplint/
drwxr-xr-x  3 devilbox devilbox 4096 Sep 24 04:26 phpstan/
drwxr-xr-x  2 devilbox devilbox 4096 Nov  8 05:10 phpstant/
drwxr-xr-x  3 devilbox devilbox 4096 Nov  4 05:46 phpunit/

If I list individual namespace (i.e : phpstan here) I got expected results

composer bin phpstan outdated -v

[bamarni-bin] Current working directory: /shared/backups/bartlett/box-manifest
[bamarni-bin] Configuring bin directory to /shared/backups/bartlett/box-manifest/vendor/bin.
[bamarni-bin] Checking namespace vendor-bin/phpstan
[bamarni-bin] Changed current directory to vendor-bin/phpstan.
[bamarni-bin] Running `@composer outdated -v --working-dir='.'`.
Color legend:
- patch or minor release available - update recommended
- major release available - update possible

Direct dependencies required in composer.json:
phpstan/phpstan 1.12.7 1.12.8 PHPStan - PHP Static Analysis Tool

Transitive dependencies not required in composer.json:
Everything up to date

I'd like to see all outdated dependencies of all namespaces at once !