Make some small refactoring related to variables

[dhriaznov]

🔘 PR Type

What kind of change does this PR introduce?

• Refactoring (no functional changes, no api changes)

📜 Background

I noticed that we currently don't have the @typescript-eslint/no-unused-vars eslint rule enabled and that we still have the hardcoded the request timeout value instead of using the constant, so this PR is here to address those things.

Issue Link: #[issue_number]
Context Link (if applicable):

🔄 Changes

Does this PR introduce a breaking change?

• Yes, Incompatible API changes
• No, Adds functionality (backwards compatible)
• No, Bug fixes (backwards compatible)

Changes:

• This PR enables the @typescript-eslint/no-unused-vars eslint rule to show the warnings for the unused vars/imports
• It also changes the requests timeout value from 30000 to API_TIMEOUT_MILLI constant which has the same value

Impact:

• This PR doesn't change any logic, it only improves the codebase and developer experience

🖼 Screenshot / 📹 Video

✅ Review checklist

Please ensure the following are true before merging:

• Code Style is consistent with the project guidelines.
• Code is readable and well-commented.
• No unnecessary or debugging code has been added.
• Security considerations have been taken into account.
• The change has been manually tested and works as expected.
• Breaking changes and their impacts have been considered and documented.
• Code does not introduce new technical debt or issues.

[github-actions]

Test this PR with npm i --save-exact @secretkeylabs/[email protected]

[victorkirov]

Nice! 🙌