From a6214e077734bc85e2872a458ea51856c0aa212d Mon Sep 17 00:00:00 2001 From: Sri Aakash Mandavilli Date: Fri, 10 Apr 2026 19:19:13 +0000 Subject: [PATCH] (bug fix): Skipping sagemaker extension activation for non sagemaker environments --- patches/sagemaker/sagemaker-extension.diff | 10 ++++++++-- patches/sagemaker/sagemaker-extensions-sync.diff | 5 ++++- 2 files changed, 12 insertions(+), 3 deletions(-) diff --git a/patches/sagemaker/sagemaker-extension.diff b/patches/sagemaker/sagemaker-extension.diff index 1c00996..1ad51fe 100644 --- a/patches/sagemaker/sagemaker-extension.diff +++ b/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 diff --git a/patches/sagemaker/sagemaker-extensions-sync.diff b/patches/sagemaker/sagemaker-extensions-sync.diff index bf0558d..4b73791 100644 --- a/patches/sagemaker/sagemaker-extensions-sync.diff +++ b/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());