Derive tag on fork from .hermesversion in "react-native"

[kraenhansen]

Merging this PR will:

• Derive the tag used when cloning the fork from the contents of the .hermesversion in "react-native" brought by the app: https://github.com/facebook/react-native/blob/v0.79.2/packages/react-native/sdks/.hermesversion

With this change, we can start pushing multiple tags to the React Native fork, to enable multiple React Native versions, ultimately fixing #157.

[changeset-bot]

🦋 Changeset detected

Latest commit: 2b82046

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	Minor
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]

Turns out this didn't result in a failure.

[Copilot]

Pull Request Overview

This PR replaces the hardcoded Hermes git tag with a version derived from the .hermesversion file in the React Native package and adjusts the clone logic and error handling accordingly.

• Remove static HERMES_GIT_TAG and derive patchedTag from .hermesversion
• Read and log the Hermes version from React Native’s SDK folder
• Update the git clone command to use the dynamic tag and add better error handling
• Add a changeset entry for the new behavior

Reviewed Changes

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

File	Description
packages/host/src/node/cli/hermes.ts	Read .hermesversion, compute patchedTag, update clone command, and refine error handling
.changeset/upset-paws-follow.md	Add a changeset to document deriving the fork tag from .hermesversion

Comments suppressed due to low confidence (1)

packages/host/src/node/cli/hermes.ts:63

• [nitpick] Add unit tests covering the new logic that reads .hermesversion and constructs patchedTag, ensuring the correct tag string is produced for different version inputs.

        const patchedTag = `node-api-${hermesVersion}`;

[Copilot]

Wrap the .hermesversion read in a try/catch to surface a clear error if the file is missing or unreadable, e.g., ‘Failed to read Hermes version from .hermesversion: ’.

Suggested change

	const hermesVersion = fs.readFileSync(hermesVersionPath, "utf8").trim();
	let hermesVersion: string;
	try {
	hermesVersion = fs.readFileSync(hermesVersionPath, "utf8").trim();
	} catch (error) {
	throw new Error(`Failed to read Hermes version from ${hermesVersionPath}: ${error.message}`);
	}

[kraenhansen]

This is very unlikely to happen. If I were to do this, I would call a function instead of using a mutable var.

[kraenhansen]

Ideally, we would match on the output of the command, but that isn't yet supported by bufout (see kraenhansen/bufout#2)