[IMP] Client side code cleaning and Python extension integration

server/core/odoo.py

@@ -193,7 +193,6 @@ def reload_database(ls):
             Odoo.get().reset(ls)
         FileMgr.files = {}
         ls.show_message_log("Building new database", MessageType.Log)
-        ls.show_message("Reloading Odoo database", MessageType.Info)
         ls.launch_thread(target=Odoo.initialize, args=(ls,))
 
 

vscode/build_package.sh

@@ -1,3 +1,5 @@
+#!/bin/bash
+
 PACKAGE_VERSION=$(cat package.json \
   | grep version \
   | head -1 \

vscode/client/common/constants.ts

@@ -0,0 +1,11 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
+import * as path from 'path';
+
+const folderName = path.basename(__dirname);
+export const EXTENSION_ROOT_DIR =
+    folderName === 'common' ? path.dirname(path.dirname(__dirname)) : path.dirname(__dirname);
+export const BUNDLED_PYTHON_SCRIPTS_DIR = path.join(EXTENSION_ROOT_DIR, 'server');
+export const SERVER_SCRIPT_PATH = path.join(BUNDLED_PYTHON_SCRIPTS_DIR, '__main__.py');
+export const DEBUG_SERVER_SCRIPT_PATH = path.join(BUNDLED_PYTHON_SCRIPTS_DIR, `_debug_server.py`);

vscode/client/utils/events.ts → vscode/client/common/events.ts

@@ -2,3 +2,4 @@ import {EventEmitter} from "vscode";
 
 export const selectedConfigurationChange = new EventEmitter();
 export const ConfigurationsChange = new EventEmitter();
+export const clientStopped = new EventEmitter();

vscode/client/common/python.ts

@@ -0,0 +1,87 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License.
+
+/* eslint-disable @typescript-eslint/naming-convention */
+import { commands, Disposable, Event, EventEmitter, Uri } from 'vscode';
+import { PythonExtension, ResolvedEnvironment, VersionInfo } from '@vscode/python-extension';
+
+export interface IInterpreterDetails {
+    path?: string[];
+    resource?: Uri;
+}
+
+export const onDidChangePythonInterpreterEvent = new EventEmitter<IInterpreterDetails>();
+export const onDidChangePythonInterpreter: Event<IInterpreterDetails> = onDidChangePythonInterpreterEvent.event;
+
+let _api: PythonExtension | undefined;
+async function getPythonExtensionAPI(): Promise<PythonExtension | undefined> {
+    if (_api) {
+        return _api;
+    }
+    _api = await PythonExtension.api();
+    return _api;
+}
+
+export async function initializePython(disposables: Disposable[]): Promise<void> {
+    try {
+        const api = await getPythonExtensionAPI();
+
+        if (api) {
+            disposables.push(
+                api.environments.onDidChangeActiveEnvironmentPath((e) => {
+                    onDidChangePythonInterpreterEvent.fire({ path: [e.path], resource: e.resource?.uri });
+                }),
+            );
+
+            console.log('Waiting for interpreter from python extension.');
+            onDidChangePythonInterpreterEvent.fire(await getInterpreterDetails());
+        }
+    } catch (error) {
+        console.error('Error initializing python: ', error);
+    }
+}
+
+export async function resolveInterpreter(interpreter: string[]): Promise<ResolvedEnvironment | undefined> {
+    const api = await getPythonExtensionAPI();
+    return api?.environments.resolveEnvironment(interpreter[0]);
+}
+
+export async function getInterpreterDetails(resource?: Uri): Promise<IInterpreterDetails> {
+    const api = await getPythonExtensionAPI();
+    const environment = await api?.environments.resolveEnvironment(
+        api?.environments.getActiveEnvironmentPath(resource),
+    );
+    if (environment?.executable.uri && checkVersion(environment)) {
+        return { path: [environment?.executable.uri.fsPath], resource };
+    }
+    return { path: undefined, resource };
+}
+
+export async function getDebuggerPath(): Promise<string | undefined> {
+    const api = await getPythonExtensionAPI();
+    return api?.debug.getDebuggerPackagePath();
+}
+
+export async function runPythonExtensionCommand(command: string, ...rest: any[]) {
+    await getPythonExtensionAPI();
+    return await commands.executeCommand(command, ...rest);
+}
+
+export function checkVersion(resolved: ResolvedEnvironment | undefined): boolean {
+    const version = resolved?.version;
+    if (version?.major === 3 && version?.minor >= 8) {
+        return true;
+    }
+    console.error(`Python version ${version?.major}.${version?.minor} is not supported.`);
+    console.error(`Selected python path: ${resolved?.executable.uri?.fsPath}`);
+    console.error('Supported versions are 3.8 and above.');
+    return false;
+}
+
+export async function getPythonVersion(): Promise<VersionInfo> {
+    let interpreterDetails = await getInterpreterDetails();
+    let resolved = await resolveInterpreter(interpreterDetails.path);
+    let version = resolved.version;
+
+    return version;
+}

vscode/client/utils/utils.ts → vscode/client/common/utils.ts

@@ -1,4 +1,3 @@
-import { execSync } from "child_process";
 import { ExtensionContext, Uri, Webview } from "vscode";
 
 /**
@@ -25,17 +24,9 @@ export function getNonce() {
 	return text;
 }
 
-export function getPythonVersion(context: ExtensionContext, pythonPath: String = "python3") {
-	try {
-		return execSync(`${pythonPath} --version`, {encoding: 'utf8'})
-	} catch {
-		return null;
-	}
-}
-
 // Config related utils
 
-export function getCurrentConfig(context: ExtensionContext) {
+export async function getCurrentConfig(context: ExtensionContext) {
     const configs: any = context.globalState.get("Odoo.configurations");
     const activeConfig: number = Number(context.workspaceState.get('Odoo.selectedConfiguration'));
     return (configs && activeConfig > -1 ? configs[activeConfig] : null);

vscode/client/utils/validation.ts → vscode/client/common/validation.ts

@@ -12,6 +12,5 @@ export function getConfigurationStructure(id: number = 0) {
         "name": `New Configuration ${id}`,
         "odooPath": "",
         "addons": [],
-        "pythonPath": "python3",
     }
 }