Add the missing remote

Author: anthonykim1

src/features/terminal/utils.ts

@@ -263,11 +263,16 @@ export async function setAutoActivationType(value: AutoActivationType): Promise<
 }
 
 /**
- * Determines whether activation commands should be sent to the currently focused terminal and previous terminal.
- * Checks the legacy `python.terminal.activateEnvInCurrentTerminal` setting.
+ * Determines whether activation commands should be sent to pre-existing terminals
+ * (terminals open before extension load).
  *
- * - If the user has explicitly set the value to `false` at any scope
- *   (global, workspace, or workspace folder), returns `false`.
+ * Checks the legacy `python.terminal.activateEnvInCurrentTerminal` setting using `inspect()`
+ * to distinguish between the default value and an explicitly user-set value.
+ *
+ * Priority: workspaceFolderValue > workspaceValue > globalRemoteValue > globalLocalValue > globalValue
+ * (matches the precedence used by getShellIntegrationEnabledCache and getAutoActivationType)
+ *
+ * - If the user has explicitly set the value to `false` at any scope, returns `false`.
  * - Otherwise (default or explicitly `true`), returns `true`.
  *
  * @returns `false` only when the user has explicitly set the setting to `false`; `true` otherwise.
@@ -280,14 +285,22 @@ export function shouldActivateInCurrentTerminal(): boolean {
         return true;
     }
 
-    // Check explicit user-set values at each scope (highest precedence first).
     // Only respect `false` when the user has deliberately set it.
+    // Priority: workspaceFolder > workspace > globalRemote > globalLocal > global
+    const inspectValue = inspected as Record<string, unknown>;
+
     if (inspected.workspaceFolderValue === false) {
         return false;
     }
     if (inspected.workspaceValue === false) {
         return false;
     }
+    if ('globalRemoteValue' in inspected && inspectValue.globalRemoteValue === false) {
+        return false;
+    }
+    if ('globalLocalValue' in inspected && inspectValue.globalLocalValue === false) {
+        return false;
+    }
     if (inspected.globalValue === false) {
         return false;
     }

src/test/features/terminal/utils.unit.test.ts

@@ -477,4 +477,55 @@ suite('Terminal Utils - shouldActivateInCurrentTerminal', () => {
             'workspaceFolderValue false should take precedence',
         );
     });
+
+    test('should return false when globalRemoteValue is explicitly false', () => {
+        pythonConfig.inspect.withArgs('terminal.activateEnvInCurrentTerminal').returns({
+            key: 'terminal.activateEnvInCurrentTerminal',
+            defaultValue: false,
+            globalRemoteValue: false,
+            globalValue: undefined,
+            workspaceValue: undefined,
+            workspaceFolderValue: undefined,
+        });
+
+        assert.strictEqual(
+            shouldActivateInCurrentTerminal(),
+            false,
+            'Should return false when user explicitly set globalRemoteValue to false',
+        );
+    });
+
+    test('should return false when globalLocalValue is explicitly false', () => {
+        pythonConfig.inspect.withArgs('terminal.activateEnvInCurrentTerminal').returns({
+            key: 'terminal.activateEnvInCurrentTerminal',
+            defaultValue: false,
+            globalLocalValue: false,
+            globalValue: undefined,
+            workspaceValue: undefined,
+            workspaceFolderValue: undefined,
+        });
+
+        assert.strictEqual(
+            shouldActivateInCurrentTerminal(),
+            false,
+            'Should return false when user explicitly set globalLocalValue to false',
+        );
+    });
+
+    test('workspaceValue false takes precedence over globalRemoteValue true', () => {
+        pythonConfig.inspect.withArgs('terminal.activateEnvInCurrentTerminal').returns({
+            key: 'terminal.activateEnvInCurrentTerminal',
+            defaultValue: false,
+            globalRemoteValue: true,
+            globalValue: undefined,
+            workspaceValue: false,
+            workspaceFolderValue: undefined,
+        });
+
+        assert.strictEqual(
+            shouldActivateInCurrentTerminal(),
+            false,
+            'workspaceValue false should take precedence over globalRemoteValue true',
+        );
+    });
 });