feat!: remove telemetry feature

[shajz]

Platforms affected

n/a

Motivation and Context

Closes #625.

Telemetry relies on the insight package, which is unmaintained and relies on the request package which has a CVE GHSA-p8p7-x288-28g6

This is basically the same PR as #614 but it's now following this discussion with the backing of @breautek : #631 (comment)

Description

Remove the telemetry feature entirely

Testing

Ran npm run test successfully

Checklist

• I've run the tests to see all new and existing tests pass
• I added automated test coverage as appropriate for this change
• Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))
• If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct keyword to close issues using keywords)
• I've updated the documentation if necessary

[shajz]

Haha @breautek, which PR do we keep? #632

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.53%. Comparing base (bf1f561) to head (46f9941).

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #633      +/-   ##
==========================================
- Coverage   70.68%   65.53%   -5.16%     
==========================================
  Files           4        3       -1     
  Lines         307      235      -72     
==========================================
- Hits          217      154      -63     
+ Misses         90       81       -9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[dpogue]

Hi @shajz, thanks for the PR! This one seems slightly more complete than #632 in that it also cleans up the documentation.

I'd really like to get this merged, if you have a chance to rebase it to resolve the conflicts on package.json. If you're busy, I can try to cherry-pick it onto a new branch next week and open a new PR retaining credit to you for the commit.

[breautek]

All test passes.

[shajz]

Hi @dpogue, I've rebased on master and fixed the conflicts on the package.json and package-lock.json files :)

I'll also mark the PR as ready for review, thanks for the interest in this PR!

[dpogue]

Thank you @shajz! 🎉
Apologies for letting this sit open for so long.