make my code cleaner

Author: anthonykim1

src/client/common/terminal/service.ts

@@ -25,7 +25,7 @@ import { useEnvExtension } from '../../envExt/api.internal';
 import { ensureTerminalLegacy } from '../../envExt/api.legacy';
 import { sleep } from '../utils/async';
 import { isWindows } from '../utils/platform';
-import { getActiveInterpreter } from '../../repl/replUtils';
+import { getPythonMinorVersion } from '../../repl/replUtils';
 
 @injectable()
 export class TerminalService implements ITerminalService, Disposable {
@@ -110,16 +110,12 @@ export class TerminalService implements ITerminalService, Disposable {
         const config = getConfiguration('python');
         const pythonrcSetting = config.get<boolean>('terminal.shellIntegration.enabled');
 
-        let minorVersion: number | undefined;
-        // Need to check for if Python version is >= 3.13 since we have turned off SI on python side.
-        // Because we are not sending explicit commandline info to core, it will inject ^C. (We want to avoid)
-        if (this.options && this.options.resource) {
-            const pythonVersion = await getActiveInterpreter(
-                this.options.resource,
-                this.serviceContainer.get<IInterpreterService>(IInterpreterService),
-            );
-            minorVersion = pythonVersion?.version?.minor;
-        }
+        const minorVersion = this.options?.resource
+            ? await getPythonMinorVersion(
+                  this.options.resource,
+                  this.serviceContainer.get<IInterpreterService>(IInterpreterService),
+              )
+            : undefined;
 
         if ((isPythonShell && !pythonrcSetting) || (isPythonShell && isWindows()) || (minorVersion ?? 0) >= 13) {
             // If user has explicitly disabled SI for Python, use sendText for inside Terminal REPL.

src/client/repl/replUtils.ts

@@ -118,3 +118,17 @@ export function getTabNameForUri(uri: Uri): string | undefined {
 
     return undefined;
 }
+
+/**
+ * Function that will return the minor version of current active Python interpreter.
+ */
+export async function getPythonMinorVersion(
+    uri: Uri | undefined,
+    interpreterService: IInterpreterService,
+): Promise<number | undefined> {
+    if (uri) {
+        const pythonVersion = await getActiveInterpreter(uri, interpreterService);
+        return pythonVersion?.version?.minor;
+    }
+    return undefined;
+}