Venv Telemetry and Structure Refactoring

[andreamah]

Description:

Adding telemetry to the new venv setup process merged in #218 .

EDIT: also re-structured setup code. Now, the code for setup is abstracted into a SetupService object. The telemetryAI object is now passed in within the SetupService constructor and setupEnv is called whenever the setup process needs to be triggered.

Type of change

• New feature (non-breaking change which adds functionality)

Testing:

Tested setup process (common flows)

• installing virtual environment in extension
• installing with personal virtual environment

Checklist:

• My code follows the style guidelines of this project
• My code has been formatted with npm run format and passes the checks in npm run check
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• Any dependent changes have been merged and published in downstream modules

[xnkevinnguyen]

Would be nice to have venv as a service where you would only set telemetry ai once as an attribute

[xnkevinnguyen]

Nice work! Will be nice to have telemetry

[nasadigital]

LGTM! 🏖🏖🏖