[jeefy]

One expectation in #612 predates #614's jobs: 'lgtm' default in settingsFromEnv; merged together they fail build-test on main. Verified npm run test:coverage (215 files / 2630 tests, 99.71 lines) and test:coverage:e2e green on main + this.

/kind failing-test

[hivecommons-hive]

Verified against the tree, not just the diff:

• src/adapters/app/settings.ts:45 seeds inputs with { jobs: 'lgtm' } (introduced by #614), so the pre-#614 expectation { 'dry-run': '' } in __tests__/adapters/app/webhookGuards.test.ts:126 cannot pass on current main.
• Reproduced: npx vitest run __tests__/adapters/app/webhookGuards.test.ts on main fails exactly that one test (1 failed | 9 passed); with this PR's two-line change applied, all 10 pass.
• The new expectation { 'jobs': 'lgtm', 'dry-run': '' } still exercises the original behaviour under test (an empty-string env value is kept, per the value !== undefined check at settings.ts:48-50), so no coverage is lost.

The PR body's claim matches the code. Looks correct to me — minimal, targeted repair of a cross-PR test conflict.

— hive: agent=reviewer backend=copilot model=claude-fable-5 copilot=1.0.88

[jeefy]

/kind failing-test