Promote develop to main: quality alignment and follow-up fixes

[ZappoMan]

Summary

This is the actual promotion PR from develop to main.

It promotes the ESLint/CodeQL quality-alignment work and the immediate cleanup follow-up that landed on develop.

Included in this promotion

• #33 — Expand ESLint coverage and fix CodeQL-aligned quality issues
• #35 — Follow-up cleanup for the quality alignment changes

Highlights

• Expanded lint coverage to repo areas previously outside local lint (scripts/, tests/, packages/shared-tests/, packages/test-utils/src/, apps/mobile/__tests__/).
• Updated CI PR path filters so script-only/config-only changes still run the Test workflow.
• Included follow-up quality fixes (test cleanup/order fix, simplified guard logic, env access consistency), plus the phasePrereqs JSDoc param-name alignment fix.

Validation

• npm run lint
• npm run lint:strict
• CI checks for this promotion PR

Merge strategy

This PR targets main. Use Create a merge commit (not squash merge).

[github-actions]

⚠️ Merge Strategy Reminder

This PR targets the main branch. When merging, please use:

✅ "Create a merge commit" (NOT squash merge)

Using "Squash and merge" on PRs targeting main can cause merge conflicts when merging develop → main later.

See ARCHITECTURE.md for details.

---

Updated at: April 25, 2026 at 12:25 PM PDT

[github-actions]

CI Coverage & Test Summary

Metric	Coverage	Covered / Total
Statements	91.53%	5445 / 5949
Branches	80.62%	749 / 929
Functions	75.16%	115 / 153
Lines	89.44%	5321 / 5949

Suites: 33 passed, 0 failed (33 total) · Tests: 369 passed, 0 failed (397 total)

✅ All reported test suites passed.

Coverage artifacts: coverage-summary, coverage-packages.

---

Updated at: April 25, 2026 at 12:29 PM PDT