chore: remove stale code and improve logic

[snitin315]

What kind of change does this PR introduce?

Refactoring.
Removed the stale imports, unused code & improved logic.

Did you add tests for your changes?
NA
If relevant, did you update the documentation?
NA
Summary
The motivation for this change -

Does this PR introduce a breaking change?

NO
Other information

[webpack-bot]

Thank you for your pull request! The most important CI builds succeeded, we’ll review the pull request soon.

[anshumanv]

Would suggest keeping this open for a while, go through the code and remove all stale code and logic improvement at once.

[snitin315]

Would suggest keeping this open for a while, go through the code and remove all stale code and logic improvement at once.

@anshumanv There are two more warnings in /package-utils/__tests__ regarding the use of type any -

I'm not sure what should be used instead of any.

[snitin315]

Rest all code looks good to me.

[anshumanv]

@anshumanv There are two more warnings in /package-utils/tests regarding the use of type any -

@snitin315 it should be jest.Mock 👍

Let me know if that works.

[snitin315]

Let me know if that works.

It worked thanks.

[alexander-akait]

/cc @webpack/cli-team need review

[anshumanv]

Will take a look locally in a while.

[webpack-bot]

@snitin315 Thanks for your update.

I labeled the Pull Request so reviewers will review it again.

@anshumanv Please review the new changes.

[jamesgeorge007]

Thanks