fix(shell test): resolve config file dynamically for smoke tests

[TalZaccai]

Problem

The smoke-tests workflow has been failing on every PR (and on main itself) since #2342 merged on 2026-05-14. Symptom in CI:

• Worker teardown timeout of 600000ms exceeded on simple.spec.ts:simple
• Failed to close instance ... Operation timed out and Unable to find chat view...timeout (10 retry attempts) on the remaining tests

Root cause

PR #2342 changed tools/scripts/getKeys.mjs to default to YAML output (config.local.yaml); the legacy .env is only written when --dotenv is explicitly passed. The smoke workflow (.github/workflows/smoke-tests.yml) calls:

node tools/scripts/getKeys.mjs --vault build-pipeline-kv --commit

— no --dotenv, so it now writes ts/config.local.yaml and no .env is produced in CI.

But ts/packages/shell/test/testHelper.ts hard-coded the env path:

const args = [appPath, ""--test"", ""--env"", path.resolve(appPath, ""../../.env"")];

In --test mode, initializeKeys calls loadKeysFromEnvFile(envFile) which throws Env file <path> not found. Shell startup fails before the chat view ever loads → every Playwright test fails the same way.

I confirmed the chain by inspecting #2342's own pre-merge smoke run (25881071116) — it shows the exact same failure signature and was merged with the failure visible.

Fix

Make getLaunchArgs prefer config.local.yaml when present and fall back to .env so smoke works in both CI (YAML-only) and local developer environments that still have a .env. shell/keys.ts's parseConfigFileContent already routes by file extension, so the same code path handles both formats.

Verification

• pnpm --filter agent-shell build ✓
• Local npx playwright test simple.spec.ts --workers 1: 5/5 passed in 3.2m

Awaiting CI confirmation.

[Copilot]

Pull request overview

This PR fixes the Shell Playwright smoke tests by making the test launcher dynamically choose the correct key/config file format after the repo-wide move from legacy .env to config.local.yaml.

Changes:

• Add logic in the Shell Playwright test helper to prefer config.local.yaml when present and fall back to .env.
• Update Electron launch arguments to pass the resolved config path via --env, preventing Shell startup failures in --test mode.

---

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