Add ability to specify if looking for file or directory

[brentonhouse]

Would be nice to be able to be able to specify in options if you are looking for a file or directory.

coreyfarrell earned $40.00 by resolving this issue!

• Checkout the Issuehunt explorer to discover more funded issues.
• Need some help from other developers? Add your repositories on Issuehunt to raise funds.

[sholladay]

This should be relatively easy to add and I've wanted it in the past, too. I'm +0.5 on it. Curious to see how Sindre feels about it.

In the meantime, check out my branch that allows you to use a function to inspect filepaths as we walk up the tree: PR #28

Using that branch, you can do something like:

const findUp = require('find-up');
const pathType = require('path-type');

(async () => {
    const distPath = await findUp((dir) => {
        return pathType.dir(path.join('dist')) && 'dist';
    });
})();

... and you can be sure that distPath is a dist directory, not a file named dist.

[sindresorhus]

Yes, we need this. I'm thinking: {type: 'directory'} and {type: 'file'} and defaulting to file. Unless someone has a better suggestion?

[coreyfarrell]

Wouldn't this need to be implemented as a locate-path option? Assume someone calls findUp(['name1', 'name2'], {type: 'file', cwd: process.cwd()}), and ./name1 is a directory / name2 is a file. In this case we need to tell locate-path that it should not resolve name1.

One other question, what should happen if a symbolic link matches? Resolve with fs.realpath then retest the result? This could be a problem with using path-type, it might be necessary to check isFile and isSymbolicLink, fs.stat can do this with one promise.

[sindresorhus]

Wouldn't this need to be implemented as a locate-path option? Assume someone calls findUp(['name1', 'name2'], {type: 'file', cwd: process.cwd()}), and ./name1 is a directory / name2 is a file.

Yes

One other question, what should happen if a symbolic link matches? Resolve with fs.realpath then retest the result?

Yes, with an option to turn it off (?). (followSymlinks).

[sholladay]

pathType.isFile(path) and pathType.isDirectory(path) always follow symlinks by nature. I might be misunderstanding the problem you want to solve, but I think as long as path-type considers the given path to match the correct type, then that path should be returned as-is, not its realpath.

[coreyfarrell]

@sholladay my idea is that in some cases callers might not want to ever match symlinks (regardless of what it points to). I've posted sindresorhus/locate-path#4 with a rough implementation plus a couple tests.

[sholladay]

Okay, thanks, that's helpful. Yeah, it might happen that someone needs it. 99% of the time, though, you can just treat a symlink as the thing that it points to. But I see what you're saying. 👍

[IssueHuntBot]

@IssueHunt has funded $40.00 to this issue.

---

• Submit pull request via IssueHunt to receive this reward.
• Want to contribute? Chip in to this issue via IssueHunt.
• Checkout the IssueHunt Issue Explorer to see more funded issues.
• Need help from developers? Add your repository on IssueHunt to raise funds.

[sindresorhus]

https://github.com/sindresorhus/locate-path/releases/tag/v5.0.0

[coreyfarrell]

I'll submit a PR shortly to update package.json, testing and readme here.

[IssueHuntBot]

@sindresorhus has rewarded $36.00 to @coreyfarrell. See it on IssueHunt

• 💰 Total deposit: $40.00
• 🎉 Repository reward(0%): $0.00
• 🔧 Service fee(10%): $4.00