Gate E2E tests behind APPLESCRIPT_E2E flag; auto-skip in CI

[Copilot]

E2E tests in test/integration/e2e-apps.test.ts spawn real macOS apps and require Automation permissions — running them unconditionally in CI (where CI=true) causes failures or hangs on GitHub Actions runners.

Changes

• Skip condition: Added a skipReason evaluated at module load time; the entire suite skips when process.env.CI is set or process.env.APPLESCRIPT_E2E is unset
• Skip messaging: Uses Node's native { skip: reason } option on describe for clean, informative output with zero test infrastructure overhead

const skipReason: string | undefined = process.env.CI
  ? "Skipped in CI environment (set APPLESCRIPT_E2E=1 and unset CI to run)"
  : !process.env.APPLESCRIPT_E2E
    ? "Set APPLESCRIPT_E2E=1 to run E2E tests locally"
    : undefined;

describe("E2E app tests", { skip: skipReason }, () => { … });

To run locally: APPLESCRIPT_E2E=1 pnpm test:integration

---

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

[Copilot]

Pull request overview

Updates the macOS AppleScript E2E integration suite to avoid running on CI runners where Automation permissions and real apps are unavailable, preventing CI hangs/failures while keeping local opt-in execution.

Changes:

• Adds a module-level skipReason that skips the whole suite when CI is set or APPLESCRIPT_E2E is not set.
• Uses node:test’s { skip: reason } option on describe() to provide a clear skip message.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

skipReason currently doesn't enforce the documented "macOS required" constraint. If someone sets APPLESCRIPT_E2E=1 on Linux/Windows (or runs on a non-darwin CI runner), the suite will still execute and likely fail/hang. Consider adding a process.platform !== "darwin" skip reason (with a clear message) so the suite can never run off-macOS.

Suggested change

	const skipReason: string | undefined = process.env.CI
	? "Skipped in CI environment (set APPLESCRIPT_E2E=1 and unset CI to run)"
	: !process.env.APPLESCRIPT_E2E
	? "Set APPLESCRIPT_E2E=1 to run E2E tests locally"
	: undefined;
	const skipReason: string | undefined =
	process.platform !== "darwin"
	? "AppleScript E2E tests require macOS (process.platform !== 'darwin')"
	: process.env.CI
	? "Skipped in CI environment (set APPLESCRIPT_E2E=1 and unset CI to run)"
	: !process.env.APPLESCRIPT_E2E
	? "Set APPLESCRIPT_E2E=1 to run E2E tests locally"
	: undefined;

[Copilot]

The gate is documented as APPLESCRIPT_E2E=1, but the check uses truthiness (!process.env.APPLESCRIPT_E2E). Values like APPLESCRIPT_E2E=0 are truthy and would incorrectly enable these tests. Consider checking explicitly for the expected value (e.g., process.env.APPLESCRIPT_E2E === "1") so the flag behaves as documented.

Suggested change

	: !process.env.APPLESCRIPT_E2E
	: process.env.APPLESCRIPT_E2E !== "1"