Treat opendir failures as empty directories

[kraenhansen]

Merging this PR fixes #185 by introducing a few changes to the implementation used when looking for Node-API modules on the filesystem:

• Treat opendir failures as empty directories.
• Exclude react-native-node-api and react-native packages completely.

[changeset-bot]

🦋 Changeset detected

Latest commit: 12e9c20

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages

Name	Type
react-native-node-api	Patch
cmake-rn	Patch
ferric-cli	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[kraenhansen]

Two known packages known not to have Node-API prebuilds which we want to link in.

[Copilot]

Pull Request Overview

This PR updates the Node-API module discovery to treat directory read failures as empty directories and completely exclude the host and React Native packages.

• Wraps fs.promises.opendir in a try/catch to skip over unreadable directories.
• Removes the path-based exclusion for react-native-node-api and adds a package-name exclusion list.
• Filters out react-native-node-api and react-native in findNodeApiModulePathsByDependency and adds a test for the unreadable-directory case.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
packages/host/src/node/path-utils.ts	Wrap opendir in try/catch, remove path exclude, add package-name exclude.
packages/host/src/node/path-utils.test.ts	Adjust existing test and add new test for unreadable directories.
.changeset/stale-coins-drop.md	Add changeset entry summarizing the fix.

Comments suppressed due to low confidence (2)

packages/host/src/node/path-utils.ts:354

• [nitpick] The comment refers to 'patterns' but the constant is a list of package names. Update the doc to clearly state these are package names being excluded.

 * Default patterns to use when excluding package names from the search for Node-API modules.

packages/host/src/node/path-utils.ts:385

• There’s no test verifying that 'react-native-node-api' and 'react-native' are actually excluded. Add a test case to cover the excludePackages behavior in findNodeApiModulePathsByDependency.

      .filter(([name]) => !excludePackages.includes(name))

[Copilot]

[nitpick] Catching all errors here may mask unexpected issues (e.g., permission vs. path-not-found). Consider narrowing this to specific filesystem errors like ENOENT or EACCES to avoid hiding genuine failures.

Suggested change

	} catch {
	// Intentionally left empty: if the directory cannot be read, we skip it
	} catch (error: any) {
	if (error.code === "ENOENT" || error.code === "EACCES") {
	// Skip the directory if it does not exist or access is denied
	} else {
	throw error; // Re-throw unexpected errors
	}