fix: pull the chromium version from the binary, not the monorepo when testing the packaged app COMPASS-9421

[lerouxb]

No description provided.

[Copilot]

Pull Request Overview

This PR introduces a new CLI flag --versions to print the full process.versions JSON (Node, Electron, Chromium) and updates the end-to-end test setup to dynamically derive the Chromium version from the target Electron binary (packaged or monorepo) to avoid mismatches.

• Add --versions flag in the Compass main process to output all process versions.
• Extend preferences schema to include a versions boolean flag.
• Rename test-runner constants to MONOREPO_ELECTRON_* and implement getChromiumVersionFromBinary for packaged-app tests.
• Update E2E helpers (compass.ts and compass-web-sandbox.ts) to use the correct Chromium version depending on context.

Reviewed Changes

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

Show a summary per file

File	Description
packages/compass/src/main/index.ts	Added --versions CLI check to dump process.versions.
packages/compass-preferences-model/src/preferences-schema.tsx	Added versions preference and schema entry for the new flag.
packages/compass-e2e-tests/helpers/test-runner-paths.ts	Renamed Electron version exports to distinguish monorepo.
packages/compass-e2e-tests/helpers/compass.ts	Imported new monorepo constant, added getChromiumVersionFromBinary, and conditional version logic.
packages/compass-e2e-tests/helpers/compass-web-sandbox.ts	Switched to use MONOREPO_ELECTRON_CHROMIUM_VERSION import.

Comments suppressed due to low confidence (2)

packages/compass/src/main/index.ts:60

• The new --versions flag isn't mentioned in the CLI help output; consider updating getHelpText() or the usage instructions to document this option.

  if (preferences.versions) {

packages/compass-e2e-tests/helpers/compass.ts:478

• The new getChromiumVersionFromBinary function isn’t covered by any tests; adding unit or integration tests would ensure it correctly parses and converts the Electron version.

async function getChromiumVersionFromBinary(path: string) {

[Copilot]

[nitpick] The parameter name path shadows the imported path module; consider renaming it (e.g., to binaryPath) to avoid confusion.

Suggested change

	async function getChromiumVersionFromBinary(path: string) {
	const { stdout } = await execFileIgnoreError(path, ['--versions'], {});
	async function getChromiumVersionFromBinary(binaryPath: string) {
	const { stdout } = await execFileIgnoreError(binaryPath, ['--versions'], {});

[gribnoysup]

One question / suggestion (that I don't think is that important), otherwise looks good to me, hopefully will help with the smokes!