From 552bfb4c0e1d9fa7aaac7762f67260180a49ff59 Mon Sep 17 00:00:00 2001 From: Sri Aakash Mandavilli Date: Fri, 13 Mar 2026 05:30:10 +0000 Subject: [PATCH 1/2] (bug fix): Skipping sagemaker extension activation for non sagemaker environments --- patches/sagemaker/sagemaker-extension.diff | 9 ++++++++- patches/sagemaker/sagemaker-extensions-sync.diff | 5 ++++- 2 files changed, 12 insertions(+), 2 deletions(-) diff --git a/patches/sagemaker/sagemaker-extension.diff b/patches/sagemaker/sagemaker-extension.diff index f979780..8206ca3 100644 --- a/patches/sagemaker/sagemaker-extension.diff +++ b/patches/sagemaker/sagemaker-extension.diff @@ -2,7 +2,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,205 @@ +@@ -0,0 +1,212 @@ +import * as vscode from 'vscode'; +import * as fs from 'fs'; +import { SessionWarning } from "./sessionWarning"; @@ -179,6 +179,13 @@ Index: code-editor-src/extensions/sagemaker-extension/src/extension.ts + +export function activate(context: vscode.ExtensionContext) { + ++ // 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; ++ } ++ + // TODO: log activation of extension + console.log('Activating Sagemaker Extension...'); + diff --git a/patches/sagemaker/sagemaker-extensions-sync.diff b/patches/sagemaker/sagemaker-extensions-sync.diff index 73cc469..781a8da 100644 --- a/patches/sagemaker/sagemaker-extensions-sync.diff +++ b/patches/sagemaker/sagemaker-extensions-sync.diff @@ -141,7 +141,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'; + @@ -163,9 +163,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()); From f1825ba2c2d2394722358721dd4174344b03ae67 Mon Sep 17 00:00:00 2001 From: Sri Aakash Mandavilli Date: Fri, 13 Mar 2026 06:02:27 +0000 Subject: [PATCH 2/2] fix: Remove TODO comment from sagemaker extension activation --- patches/sagemaker/sagemaker-extension.diff | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/patches/sagemaker/sagemaker-extension.diff b/patches/sagemaker/sagemaker-extension.diff index 8206ca3..f24e718 100644 --- a/patches/sagemaker/sagemaker-extension.diff +++ b/patches/sagemaker/sagemaker-extension.diff @@ -2,7 +2,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,212 @@ +@@ -0,0 +1,211 @@ +import * as vscode from 'vscode'; +import * as fs from 'fs'; +import { SessionWarning } from "./sessionWarning"; @@ -186,7 +186,6 @@ Index: code-editor-src/extensions/sagemaker-extension/src/extension.ts + return; + } + -+ // TODO: log activation of extension + console.log('Activating Sagemaker Extension...'); + + // execute the get cookie command and save the data to cookies