Plugin detection issue with prettier v3 about plugin-php HOT 7 CLOSED

Hi @mermshaus, thanks for the bug report! It seems that plugins now have to be registered explicitly, e.g. by passing --plugin=@prettier/plugin-php. See here for details and alternatives. We'll have to update our docs to reflect this.

@fisker has the implicit plugin registration been dropped?

from plugin-php.

The plugin search feature has been removed.

from plugin-php.

@coffebar Yes, it still works for me. To avoid derailing this issue, I put more info here. I hope it may be of help to you.

from plugin-php.

Hi everyone,

thanks for the quick help! Adding the plugin configuration/CLI arguments solves this issue.

There is one thing of note: If you have a global installation of prettier, plugin configuration won’t work (right now), at least not by simply specifying the package name. It’s possible to use an absolute path to a plugin’s entry point to work around this. E.g. prettier --plugin /usr/lib/node_modules/@prettier/plugin-php/src/index.js.

from plugin-php.

@mermshaus I have tried your workaround with plugin-php and I see the error message is gone, but it's not gonna work: no formatting.
Is it still work for you?

test.php

<?php
if(1 ){ echo 2;  }
?>

prettier --log-level debug test.php \
  --plugin /home/abc/.node_modules/lib/node_modules/@prettier/plugin-php/src/index.js \
  --parser php

Output:

[debug] normalized argv: {"":["test.php"],"cache":false,"color":true,"editorconfig":true,"logLevel":"debug","plugins":["/home/abc/.node_modules/lib/node_modules/@prettier/plugin-php/src/index.js"],"parser":"php","configPrecedence":"cli-override","debugRepeat":0,"ignorePath":[".gitignore",".prettierignore"],"_":["test.php"],"__raw":{"_":["test.php"],"cache":false,"color":true,"editorconfig":true,"log-level":"debug","plugin":"/home/abc/.node_modules/lib/node_modules/@prettier/plugin-php/src/index.js","parser":"php","config-precedence":"cli-override","debug-repeat":0,"ignore-path":[".gitignore",".prettierignore"]}}
[debug] resolve config from '/home/abc/test.php'
[debug] loaded options `{"parser":"php","plugins":["/home/abc/.node_modules/lib/node_modules/@prettier/plugin-php/src/index.js"],"useTabs":false,"tabWidth":2}`
[debug] applied config-precedence (cli-override): {"parser":"php","plugins":["/home/abc/.node_modules/lib/node_modules/@prettier/plugin-php/src/index.js"],"tabWidth":2,"useTabs":false}
<?php
if(1 ){ echo 2;  }
?>

from plugin-php.

@coffebar Specifying the full plugin path works for me, but I find this to be an annoying regression as this wasn't required previously with prettier v2

from plugin-php.

Downgraded to v2 since phpStorm uses global prettier & php plugin instances 👎🏼

from plugin-php.