(bug fix): Skipping sagemaker extension activation for non sagemaker environments (#189)

Co-authored-by: Sri Aakash Mandavilli <srmanda@amazon.com>

Author: aakashmandavilli96

patches/sagemaker/sagemaker-extension.diff

@@ -8,7 +8,7 @@ Index: code-editor-src/extensions/sagemaker-extension/src/extension.ts
 ===================================================================
 --- /dev/null
 +++ code-editor-src/extensions/sagemaker-extension/src/extension.ts
-@@ -0,0 +1,172 @@
+@@ -0,0 +1,178 @@
 +import * as vscode from 'vscode';
 +import * as fs from 'fs';
 +import { SessionWarning } from "./sessionWarning";
@@ -166,7 +166,13 @@ Index: code-editor-src/extensions/sagemaker-extension/src/extension.ts
 +
 +export function activate(context: vscode.ExtensionContext) {
 +
-+    // TODO: log activation of extension
++    // this extension will only activate within a sagemaker app
++    const isSageMakerApp = !!process.env?.SAGEMAKER_APP_TYPE_LOWERCASE;
++    if (!isSageMakerApp) {
++        console.log('Skipping activation of Sagemaker Extension...');
++        return;
++    }
++
 +    console.log('Activating Sagemaker Extension...');
 +
 +    // execute the get cookie command and save the data to cookies

patches/sagemaker/sagemaker-extensions-sync.diff

@@ -147,7 +147,7 @@ Index: code-editor-src/extensions/sagemaker-extensions-sync/src/extension.ts
 ===================================================================
 --- /dev/null
 +++ code-editor-src/extensions/sagemaker-extensions-sync/src/extension.ts
-@@ -0,0 +1,100 @@
+@@ -0,0 +1,103 @@
 +import * as process from "process";
 +import * as vscode from 'vscode';
 +
@@ -169,9 +169,12 @@ Index: code-editor-src/extensions/sagemaker-extensions-sync/src/extension.ts
 +	// this extension will only activate within a sagemaker app
 +	const isSageMakerApp = !!process.env?.SAGEMAKER_APP_TYPE_LOWERCASE;
 +	if (!isSageMakerApp) {
++		console.log('Skipping activation of Sagemaker Extension Sync...');
 +		return;
 +	}
 +
++	console.log('Activating Sagemaker Extension Sync...');
++
 +	// get installed extensions. this could be different from pvExtensions b/c vscode sometimes doesn't delete the assets
 +	// for an old extension when uninstalling or changing versions
 +	const installedExtensions = new Set(await getInstalledExtensions());