chore: enable stricter typescript checks

[43081j]

Enables noUnusedLocals and noUnusedParameters.

assuming my similar change lands in pathe, it'd be good to enable them here too. if you'd rather keep them off, let me know and we can close this

[pkg-pr-new]

Open in Stackblitz

npm i https://pkg.pr.new/unjs/unplugin@459

commit: 5b67b42

[sxzz]

In @antfu/eslint-config, we're using an ESLint plugin for checking unused vars and params

[43081j]

we are, and arguably don't need to once every project uses typescript (or at least scope it to *.js)

i'd still enable these for strictness and consistency, then update the eslint config to be more specific

the reason the TSESLint no-unused-vars rule exists is mostly because you might want to configure it, or you want to use eslint-ignore comments. though they do recommend it instead of typescript's flags because of that

if we don't need to ever configure or ignore these though, i would've just used the compiler. let me know, i don't mind either way as long as we get agreement

[antfu]

I am up to having noUnusedLocals, but not very sure about noUnusedParameters - as it doesn't bring much value in my opinion, force to add an understand prefix doesn't change we still need to declare the arg

[43081j]

updated to remove noUnusedParameters