Allow skipping symlink patterns, to avoid raising a fault

changelog_unreleased/cli/15533.md

@@ -0,0 +1,5 @@
+#### Process files only supported by plugins (#15533 by @sanmai-NL)
+
+In #14627, Prettier was redesigned to ‘no longer follow symbolic links’. However, the implementation went further and turned into a breaking change. Rather than adhering to the exit status contract of only returning a non-zero exit status when a file is not formatted or when something is wrong with Prettier itself, #14627 made Prettier return a non-zero exit status upon any pattern turning out to be a pointing to symlink.
+
+In case `--no-error-on-unmatched-pattern` is set, Prettier no longer always has a non-zero exit status if patterns match symlinks.

src/cli/expand-patterns.js

@@ -68,9 +68,9 @@ async function* expandPatternsInternal(context) {
 
     const stat = await lstatSafe(absolutePath);
     if (stat) {
-      if (stat.isSymbolicLink()) {
+      if (stat.isSymbolicLink() && context.argv.errorOnUnmatchedPattern !== false) {
         yield {
-          error: `Explicitly specified pattern "${pattern}" is a symbolic link.`,
+          error: `Skipping pattern "${pattern}", as it points to a symbolic link.`,
         };
       } else if (stat.isFile()) {
         entries.push({

tests/integration/__tests__/patterns-dirs.js

@@ -256,7 +256,7 @@ function isSymlinkSupported() {
       status: 2,
       stdout: "",
       stderr:
-        '[error] Explicitly specified pattern "test-a/symlink-to-directory-a" is a symbolic link.',
+        '[error] Skipping pattern "test-a/symlink-to-directory-a", as it points to a symbolic link.',
     },
     base,
   );
@@ -267,7 +267,7 @@ function isSymlinkSupported() {
       status: 2,
       stdout: "",
       stderr:
-        '[error] Explicitly specified pattern "test-a/symlink-to-directory-b" is a symbolic link.',
+        '[error] Skipping pattern "test-a/symlink-to-directory-b", as it points to a symbolic link.',
     },
     base,
   );
@@ -278,7 +278,7 @@ function isSymlinkSupported() {
       status: 2,
       stdout: "",
       stderr:
-        '[error] Explicitly specified pattern "test-a/symlink-to-file-a" is a symbolic link.',
+        '[error] Skipping pattern "test-a/symlink-to-file-a", as it points to a symbolic link.',
     },
     base,
   );
@@ -289,7 +289,7 @@ function isSymlinkSupported() {
       status: 2,
       stdout: "",
       stderr:
-        '[error] Explicitly specified pattern "test-a/symlink-to-file-b" is a symbolic link.',
+        '[error] Skipping pattern "test-a/symlink-to-file-b", as it points to a symbolic link.',
     },
     base,
   );
@@ -311,7 +311,7 @@ function isSymlinkSupported() {
       status: 2,
       stdout: "test-a/a.js",
       stderr:
-        '[error] Explicitly specified pattern "test-a/symlink-to-file-b" is a symbolic link.',
+        '[error] Skipping pattern "test-a/symlink-to-file-b", as it points to a symbolic link.',
     },
     base,
   );