Resolve extension identifiers from manifest

[jpogran]

This pulls the extension identifiers from the package.json file using the extension context instead of hard coding them. It also moves the appInsightsKey to the package.json file.

Closes #786

[radeksimko]

Is it worth also addressing these hard-coded extension IDs?

vscode-terraform/src/test/helper.ts

Line 26 in d79aeca

const ext = vscode.extensions.getExtension('hashicorp.terraform');

vscode-terraform/src/test/workspaces.test.ts

Line 6 in d79aeca

const ext = vscode.extensions.getExtension('hashicorp.terraform');

[jpogran]

I misunderstood the original ticket @radeksimko made, and thought the requirement was removing vscode.extensions.getExtension from the tests as well as the main activate function. This would be difficult to do in general without a lot of work, and impossible in the end-to-end tests we have. Radek meant removing the hardcoded ID now, and look into improving the tests at a later date.

[radeksimko]

LGTM

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.