chore(js-ts): Convert app/util/bytes.js to TypeScript

[devin-ai-integration]

Pull Request Description

Motivation

This pull request aims to convert the bytes.js file to TypeScript, improving type safety and maintainability of the codebase.

Context

As part of our ongoing efforts to migrate the MetaMask Mobile codebase to TypeScript, we are converting JavaScript files to TypeScript one at a time. This PR focuses on the bytes.js utility file.

Changes

• Renamed app/util/bytes.js to app/util/bytes.ts
• Added type annotations for the function parameter and return value
• Refactored the for loop to a for-of loop to address potential linting issues
• Created notes.md to track TODOs and type-related considerations

Files Changed

1. app/util/bytes.ts (renamed from bytes.js)
2. notes.md (new file)

Testing

• Ran yarn tsc to ensure successful TypeScript compilation
• Ran yarn lint to check for any linting issues

Notes

• The value parameter type is currently set to Uint8Array, but we may need to consider allowing number[] as well in the future.
• All existing functionality should remain unchanged.

Checklist

• Code follows the MetaMask Coding Standards
• Tests for the changes have been added (if applicable)
• All tests pass
• Lint and prettier checks pass
• This PR is ready for review

Link to Devin run

https://preview.devin.ai/devin/0a3389f83f8d4560bdce6a8a5a186bdd

Additional Information

This pull request was created by Devin at the request of Moritz.

If you have any feedback, you can leave comments in the PR and I'll address them in the app!

[github-actions]

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud