Remove unneeded stats.isSymbolicLink() calls

jdalton · jdalton · commit bcc7137de8a9 · 2025-09-04T08:43:28.000-04:00
diff --git a/scripts/utils/fs.js b/scripts/utils/fs.js
@@ -21,7 +21,7 @@ function findUpSync(name, options) {
       const filePath = path.join(dir, name)
       try {
         const stats = statSync(filePath, { throwIfNoEntry: false })
-        if (!onlyDirectories && (stats?.isFile() || stats?.isSymbolicLink())) {
+        if (!onlyDirectories && stats?.isFile()) {
           return filePath
         }
         if (!onlyFiles && stats?.isDirectory()) {
diff --git a/src/utils/fs.mts b/src/utils/fs.mts
@@ -35,7 +35,7 @@ export async function findUp(
       try {
         // eslint-disable-next-line no-await-in-loop
         const stats = await fs.stat(thePath)
-        if (!onlyDirectories && (stats.isFile() || stats.isSymbolicLink())) {
+        if (!onlyDirectories && stats.isFile()) {
           return thePath
         }
         if (!onlyFiles && stats.isDirectory()) {

Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ function findUpSync(name, options) {`
`21`	`21`	`const filePath = path.join(dir, name)`
`22`	`22`	`try {`
`23`	`23`	`const stats = statSync(filePath, { throwIfNoEntry: false })`
`24`		`- if (!onlyDirectories && (stats?.isFile() \|\| stats?.isSymbolicLink())) {`
	`24`	`+ if (!onlyDirectories && stats?.isFile()) {`
`25`	`25`	`return filePath`
`26`	`26`	`}`
`27`	`27`	`if (!onlyFiles && stats?.isDirectory()) {`
Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ export async function findUp(`
`35`	`35`	`try {`
`36`	`36`	`// eslint-disable-next-line no-await-in-loop`
`37`	`37`	`const stats = await fs.stat(thePath)`
`38`		`- if (!onlyDirectories && (stats.isFile() \|\| stats.isSymbolicLink())) {`
	`38`	`+ if (!onlyDirectories && stats.isFile()) {`
`39`	`39`	`return thePath`
`40`	`40`	`}`
`41`	`41`	`if (!onlyFiles && stats.isDirectory()) {`