Investigate workflow job failing on main: lint / ESLint check

[github-actions]

🚨 Failure Summary 🚨:

• 📋 Job Name: lint / ESLint check
• 🔧 Failure in Workflow: Process new code merged to main
• 🔗 Triggered by PR: PR Link
• 👤 PR Author: @mountiny
• 🤝 Merged by: @mountiny
• 🐛 Error Message:
  warning: Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/checkout@8ade135. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
  failure: Process completed with exit code 1.
  failure: Unsafe argument of type error typed assigned to a parameter of type OnyxEntry<ReportViolations>
  failure: Unsafe member access [${ONYXKEYS.COLLECTION.REPORT_VIOLATIONS}${report.reportID}] on an error typed value
  failure: Unsafe assignment of an error typed value
  failure: Unsafe argument of type error typed assigned to a parameter of type OnyxEntry<ReportViolations>
  failure: Unsafe member access [${ONYXKEYS.COLLECTION.REPORT_VIOLATIONS}${report.reportID}] on an error typed value
  failure: Unsafe assignment of an error typed value
  warning: Onyx.connect() is deprecated. Use useOnyx() hook instead and pass the data as parameters to a pure function
  warning: Onyx.connect() is deprecated. Use useOnyx() hook instead and pass the data as parameters to a pure function
  warning: Onyx.connect() is deprecated. Use useOnyx() hook instead and pass the data as parameters to a pure function
  warning: Onyx.connect() is deprecated. Use useOnyx() hook instead and pass the data as parameters to a pure function
  warning: React Hook useMemo has a missing dependency: 'shouldShowBusinessBankAccountOptions'. Either include it or remove the dependency array
  warning: React Hook useCallback has a missing dependency: 'ownerBillingGraceEndPeriod'. Either include it or remove the dependency array
  warning: React Hook useMemo has a missing dependency: 'styles.mr3'. Either include it or remove the dependency array
  warning: React Hook useMemo has a missing dependency: 'ownerBillingGraceEndPeriod'. Either include it or remove the dependency array
  warning: React Hook useCallback has a missing dependency: 'introSelected'. Either include it or remove the dependency array
  warning: The 'targetPolicyTags' conditional could make the dependencies of useCallback Hook (at line 818) change on every render. Move it inside the useCallback callback. Alternatively, wrap the initialization of 'targetPolicyTags' in its own useMemo() Hook

⚠️ Action Required ⚠️:

🛠️ A recent merge appears to have caused a failure in the job named lint / ESLint check.
🔍 This issue has been automatically created and labeled with Workflow Failure for investigation.

👀 Please look into the following:

1. Why the PR caused the job to fail?
2. Address any underlying issues.

🐛 We appreciate your help in squashing this bug!

[mountiny]

That is all working now in main as we reverted the offending pr too