Fix ExtractFile Node 24 migration issue

[rajmishra1997]

Issue:
Canary Test pipeline failed after Node 24 migartion for Task ExtractFile: https://dev.azure.com/canarytest/PipelineTasks/_build/results?buildId=235044&view=results

AB#2343422
AB#2343402

Why this issue:

• As part of node 24 migration, task-lib version is changed from 4.16.0 to 5.2.2
• Issue arose from line 1123
• task.ts ls functionality changed in PR: Remove the shelljs dependency #1079, Earlier version used shell.js
• Line 1123, pathsCopy.find, which is Array.Prototype.Find should return undefined in case of no match but returned -1
• This issue was caused by a custom polyfill implementation overriding the native behavior, as shown in the image below.

Solution:

1. Added method "safeFind" in place of Array.Prototype.Find
2. Tested in DevFabric

Failure Scenario:

Successful Scenario

[rajmishra1997]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[sanjuyadav24]

shouldn't we fix this in the function signature of ls where path is defined as unknow?

[rajmishra1997]

paths is kept as unknown type to supported different method overloading, removed pathsCopy type cast

[rajmishra1997]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[raujaiswal]

@rajmishra1997 , could you please check if we miss to upgrade the version of task-lib in package.json with this changes. from 5.2.2 to 5.2.3 ?