@azure-tools/typespec-python 0.29.0 → 0.31.0

package/generator/pygen/codegen/templates/test.py.jinja2

@@ -29,9 +29,9 @@ class {{ test.test_class_name }}({{ test.base_test_class_name }}):
     {% endif %}
     @recorded_by_proxy{{ async_suffix }}
     {% if code_model.options["azure_arm"] %}
-    {{ async }}def test_{{ testcase.operation.name }}(self, resource_group):
+    {{ async }}def test_{{ testcase.name }}(self, resource_group):
     {% else %}
-    {{ async }}def test_{{ testcase.operation.name }}(self, {{ prefix_lower }}_endpoint):
+    {{ async }}def test_{{ testcase.name }}(self, {{ prefix_lower }}_endpoint):
         {{ client_var }} = self.{{ test.create_client_name }}(endpoint={{ prefix_lower }}_endpoint)
     {% endif %}
         {{testcase.response }}{{ client_var }}{{ testcase.operation_group_prefix }}.{{ testcase.operation.name }}(

package/generator/pygen/m2r.py

@@ -27,7 +27,7 @@ class GeneratorRenderer(m2r2.RestRenderer):
         return f":code:`{html}`"
 
 
-class M2R(YamlUpdatePlugin):  # pylint: disable=abstract-method
+class M2R(YamlUpdatePlugin):
     """A plugin to convert any description and summary from MD to RST."""
 
     def update_yaml(self, yaml_data: Dict[str, Any]) -> None:

package/generator/pygen/postprocess/__init__.py

@@ -3,7 +3,7 @@
 # Licensed under the MIT License. See License.txt in the project root for
 # license information.
 # --------------------------------------------------------------------------
-from typing import Tuple, Any, Dict
+from typing import Tuple, Any
 from pathlib import Path
 import os
 import shutil
@@ -28,7 +28,7 @@ def format_file(file: Path, file_content: str) -> str:
     return file_content
 
 
-class PostProcessPlugin(Plugin):  # pylint: disable=abstract-method
+class PostProcessPlugin(Plugin):
     def __init__(self, **kwargs: Any):
         super().__init__(**kwargs)
         output_folder_uri = self.options["outputFolderUri"]

package/generator/pygen/postprocess/venvtools.py

@@ -52,9 +52,7 @@ def create(
     return builder.context
 
 
-def python_run(  # pylint: disable=inconsistent-return-statements
-    venv_context, module, command, directory=_ROOT_DIR
-) -> Optional[str]:
+def python_run(venv_context, module, command, directory=_ROOT_DIR) -> Optional[str]:
     try:
         cmd_line = [
             venv_context.env_exe,

package/generator/pygen/preprocess/__init__.py

@@ -163,7 +163,7 @@ def has_multi_part_content_type(yaml_data: Dict[str, Any]) -> bool:
     return any(ct for ct in yaml_data.get("contentTypes", []) if ct == "multipart/form-data")
 
 
-class PreProcessPlugin(YamlUpdatePlugin):  # pylint: disable=abstract-method
+class PreProcessPlugin(YamlUpdatePlugin):
     """Add Python naming information."""
 
     @property

package/generator/pygen/utils.py

@@ -82,9 +82,7 @@ def parse_args(
         return value
 
     unknown_args_ret = {
-        ua.strip("--").split("=", maxsplit=1)[0]: _get_value(  # pylint: disable=bad-str-strip-call
-            ua.strip("--").split("=", maxsplit=1)[1]  # pylint: disable=bad-str-strip-call
-        )
+        ua.strip("--").split("=", maxsplit=1)[0]: _get_value(ua.strip("--").split("=", maxsplit=1)[1])
         for ua in unknown_args
     }
     return args, unknown_args_ret
@@ -107,6 +105,7 @@ KNOWN_TYPES: Dict[str, Dict[str, Any]] = {
 }
 
 JSON_REGEXP = re.compile(r"^(application|text)/(.+\+)?json$")
+XML_REGEXP = re.compile(r"^(application|text)/(.+\+)?xml$")
 
 
 def build_policies(
@@ -151,3 +150,11 @@ def build_policies(
 
 def extract_original_name(name: str) -> str:
     return name[1 : -len("_initial")]
+
+
+def json_serializable(content_type: str) -> bool:
+    return bool(JSON_REGEXP.match(content_type.split(";")[0].strip().lower()))
+
+
+def xml_serializable(content_type: str) -> bool:
+    return bool(XML_REGEXP.match(content_type.split(";")[0].strip().lower()))

package/generator/setup.py

@@ -15,7 +15,7 @@ from setuptools import setup, find_packages
 
 # Version extraction inspired from 'requests'
 with open(os.path.join("pygen", "_version.py"), "r") as fd:
-    version = re.search(r'^VERSION\s*=\s*[\'"]([^\'"]*)[\'"]', fd.read(), re.MULTILINE).group(1)
+    version = re.search(r'^VERSION\s*=\s*[\'"]([^\'"]*)[\'"]', fd.read(), re.MULTILINE).group(1)  # type: ignore
 
 if not version:
     raise RuntimeError("Cannot find version information")

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@azure-tools/typespec-python",
-  "version": "0.29.0",
+  "version": "0.31.0",
   "author": "Microsoft Corporation",
   "description": "TypeSpec emitter for Python SDKs",
   "homepage": "https://github.com/Azure/autorest.python",
@@ -36,7 +36,7 @@
     "@azure-tools/typespec-azure-core": ">=0.45.0 <1.0.0",
     "@azure-tools/typespec-azure-resource-manager": ">=0.45.0 <1.0.0",
     "@azure-tools/typespec-autorest": ">=0.45.0 <1.0.0",
-    "@azure-tools/typespec-client-generator-core": ">=0.45.1 <1.0.0",
+    "@azure-tools/typespec-client-generator-core": ">=0.45.4 <1.0.0",
     "@azure-tools/typespec-azure-rulesets": ">=0.45.0 <3.0.0",
     "@typespec/compiler": ">=0.59.1 <1.0.0",
     "@typespec/http": ">=0.59.0 <1.0.0",
@@ -53,8 +53,8 @@
   "devDependencies": {
     "@azure-tools/typespec-azure-resource-manager": "~0.45.0",
     "@azure-tools/typespec-autorest": "~0.45.0",
-    "@azure-tools/cadl-ranch-expect": "~0.15.1",
-    "@azure-tools/cadl-ranch-specs": "~0.35.4",
+    "@azure-tools/cadl-ranch-expect": "~0.15.3",
+    "@azure-tools/cadl-ranch-specs": "~0.37.1",
     "@types/js-yaml": "~4.0.5",
     "@types/node": "^18.16.3",
     "@types/yargs": "17.0.32",
@@ -62,25 +62,28 @@
     "@typespec/eslint-config-typespec": "~0.55.0",
     "@typespec/openapi": "~0.59.0",
     "c8": "~7.13.0",
+    "vitest": "^2.0.4",
     "rimraf": "~5.0.0",
-    "typescript": "~5.1.3",
+    "typescript": "~5.5.4",
     "@azure-tools/typespec-azure-core": "~0.45.0",
-    "@azure-tools/typespec-client-generator-core": "0.45.1",
+    "@azure-tools/typespec-client-generator-core": "0.45.4",
     "@typespec/compiler": "~0.59.1",
     "@typespec/http": "~0.59.0",
     "@typespec/rest": "~0.59.0",
     "@typespec/versioning": "~0.59.0",
     "@azure-tools/typespec-azure-rulesets": "0.45.0",
-    "yargs": "~17.2.1"
+    "yargs": "~17.2.1",
+    "chalk": "5.3.0"
   },
   "scripts": {
     "clean": "rimraf ./dist ./temp ./venv ./node_modules",
     "build": "tsc -p .",
     "watch": "tsc -p . --watch",
-    "lint": "eslint . --ext .ts --max-warnings=0",
+    "lint": "tsx ./scripts/eng/lint.ts",
     "lint:fix": "eslint . --fix --ext .ts",
+    "format": "npx prettier **/*.ts --write && tsx ./scripts/eng/format.ts",
     "install": "tsx ./scripts/run-python3.ts ./scripts/install.py",
-    "regenerate": "tsx ./scripts/regenerate.ts",
-    "test": "tsx ./scripts/run-tests.ts"
+    "regenerate": "tsx ./scripts/eng/regenerate.ts",
+    "test": "npx vitest run ./test && tsx ./scripts/eng/run-tests.ts"
   }
 }

package/scripts/eng/format.ts

@@ -0,0 +1,5 @@
+/* eslint-disable no-console */
+
+import { runCommand } from "./utils.js";
+
+runCommand("black .", "black");

package/scripts/eng/lint.ts

@@ -0,0 +1,75 @@
+/* eslint-disable no-console */
+import yargs from "yargs";
+import { hideBin } from "yargs/helpers";
+import { runCommand, executeCommand } from "./utils.js";
+
+interface Arguments {
+    folderName: string;
+    command?: "pylint" | "mypy" | "pyright" | "eslint";
+    skipWarning?: boolean;
+    skipEslint?: boolean;
+}
+
+const validCommands = ["pylint", "mypy", "pyright", "eslint"];
+
+// PARSE INPUT ARGUMENTS
+const argv = yargs(hideBin(process.argv))
+    .option("folderName", {
+        type: "string",
+        choices: ["generator", "autorest"],
+        description: "Specify the flavor",
+        default: "generator",
+    })
+    .option("command", {
+        alias: "c",
+        type: "string",
+        choices: validCommands,
+        description: "Specify the command to run",
+    })
+    .option("skipWarning", {
+        alias: "s",
+        type: "boolean",
+        description: "Skip to check warnings",
+    })
+    .option("skipEslint", {
+        alias: "e",
+        type: "boolean",
+        description: "Skip to check eslint",
+    }).argv as Arguments;
+
+export function pylint() {
+    runCommand(`pylint ${argv.folderName}/ --rcfile ./scripts/eng/pylintrc`, "pylint");
+}
+
+export function mypy() {
+    runCommand(`mypy ${argv.folderName}/ --config-file ./scripts/eng/mypy.ini`, "mypy");
+}
+
+export function pyright() {
+    runCommand(`pyright ${argv.folderName}/ -p ./scripts/eng/pyrightconfig.json`, "pyright");
+}
+
+export function eslint() {
+    // const checkWarning = argv.skipWarning ? "" : "--max-warnings=0";
+    const checkWarning = "";
+    executeCommand(`npx eslint . --ext .ts ${checkWarning} `, "eslint");
+}
+
+if (argv.command === "pylint") {
+    pylint();
+} else if (argv.command === "mypy") {
+    mypy();
+} else if (argv.command === "pyright") {
+    pyright();
+} else if (argv.command === "eslint") {
+    if (!argv.skipEslint) {
+        eslint();
+    }
+} else {
+    pylint();
+    mypy();
+    pyright();
+    if (!argv.skipEslint) {
+        eslint();
+    }
+}

package/scripts/eng/mypy.ini

@@ -0,0 +1,38 @@
+# global configurations
+[mypy]
+python_version = 3.8
+
+
+# module level configuratiohns
+[mypy-jsonrpc.*]
+ignore_missing_imports = True
+
+[mypy-ptvsd.*]
+ignore_missing_imports = True
+
+[mypy-debugpy.*]
+ignore_missing_imports = True
+
+[mypy-m2r2.*]
+ignore_missing_imports = True
+
+[mypy-autorest.common.utils.*]
+ignore_missing_imports = True
+
+[mypy-autorest.common.python_mappings.*]
+ignore_missing_imports = True
+
+[mypy-pygen.codegen.models.*]
+ignore_missing_imports = True
+
+[mypy-setuptools]
+ignore_missing_imports = True
+
+[mypy-*._patch]
+ignore_missing_imports = True
+
+[mypy-pygen.*]
+ignore_missing_imports = True
+
+[mypy-yaml.*]
+ignore_missing_imports = True

package/scripts/eng/pylintrc

@@ -0,0 +1,58 @@
+[MASTER]
+py-version=3.8
+ignore-patterns=test_*,conftest,setup
+reports=no
+
+# PYLINT DIRECTORY BLACKLIST.
+ignore=_generated,samples,examples,test,tests,doc,.tox,generated_samples
+
+[MESSAGES CONTROL]
+
+# Add enable for useless disables
+enable=useless-suppression
+
+# For all codes, run 'pylint --list-msgs' or go to 'https://pylint.readthedocs.io/en/latest/reference_guide/features.html'
+# locally-disabled: Warning locally suppressed using disable-msg
+# cyclic-import: because of https://github.com/PyCQA/pylint/issues/850
+# too-many-arguments: Due to the nature of the CLI many commands have large arguments set which reflect in large arguments set in corresponding methods.
+# Let's black deal with bad-continuation
+disable=useless-object-inheritance,missing-docstring,locally-disabled,fixme,cyclic-import,too-many-arguments,invalid-name,duplicate-code,too-few-public-methods,consider-using-f-string,super-with-arguments,redefined-builtin,import-outside-toplevel,client-suffix-needed,unnecessary-dunder-call,unnecessary-ellipsis,disallowed-name,consider-using-max-builtin
+
+[FORMAT]
+max-line-length=120
+
+[VARIABLES]
+# Tells whether we should check for unused import in __init__ files.
+init-import=yes
+
+[DESIGN]
+# Maximum number of locals for function / method body
+max-locals=25
+# Maximum number of branch for function / method body
+max-branches=20
+# Maximum number of instance attributes for class
+max-attributes=10
+# Maximum number of ancestors
+max-parents=15
+
+[SIMILARITIES]
+min-similarity-lines=10
+
+[BASIC]
+# Naming hints based on PEP 8 (https://www.python.org/dev/peps/pep-0008/#naming-conventions).
+# Consider these guidelines and not hard rules. Read PEP 8 for more details.
+
+# The invalid-name checker must be **enabled** for these hints to be used.
+include-naming-hint=yes
+
+# keep short; underscores are discouraged
+module-naming-style=snake_case
+const-naming-style=UPPER_CASE
+class-naming-style=PascalCase
+class-attribute-naming-style=snake_case
+attr-naming-style=snake_case
+method-naming-style=snake_case
+function-naming-style=snake_case
+argument-naming-style=snake_case
+variable-naming-style=snake_case
+inlinevar-naming-style=snake_case

package/scripts/eng/pyrightconfig.json

@@ -0,0 +1,6 @@
+{
+    "reportUnnecessaryCast": "warning",
+    "reportTypeCommentUsage": true,
+    "reportMissingImports": false,
+    "pythonVersion": "3.8"
+}

package/scripts/{regenerate.ts → eng/regenerate.ts}

@@ -4,15 +4,19 @@ import { promisify } from "util";
 import yargs from "yargs";
 import { hideBin } from "yargs/helpers";
 import { dirname, join, relative, resolve } from "path";
-import { promises } from "fs";
+import { promises, rmSync } from "fs";
 import { fileURLToPath } from "url";
 
 // Promisify the exec function
 const exec = promisify(execCallback);
 
 // Get the directory of the current file
-const PLUGIN_DIR = resolve(fileURLToPath(import.meta.url), "../../");
+const PLUGIN_DIR = resolve(fileURLToPath(import.meta.url), "../../../");
 const CADL_RANCH_DIR = resolve(PLUGIN_DIR, "node_modules/@azure-tools/cadl-ranch-specs/http");
+interface TspCommand {
+    outputDir: string;
+    command: string;
+}
 
 const EMITTER_OPTIONS: Record<string, Record<string, string> | Record<string, string>[]> = {
     "resiliency/srv-driven/old.tsp": {
@@ -46,9 +50,6 @@ const EMITTER_OPTIONS: Record<string, Record<string, string> | Record<string, st
     "type/model/empty": {
         "package-name": "typetest-model-empty",
     },
-    "type/model/flatten": {
-        "package-name": "typetest-model-flatten",
-    },
     "type/model/inheritance/enum-discriminator": {
         "package-name": "typetest-model-enumdiscriminator",
     },
@@ -117,9 +118,15 @@ function getEmitterOption(spec: string): Record<string, string>[] {
 }
 
 // Function to execute CLI commands asynchronously
-async function executeCommand(command: string): Promise<void> {
+async function executeCommand(tspCommand: TspCommand): Promise<void> {
+    try {
+        rmSync(tspCommand.outputDir, { recursive: true, force: true });
+    } catch (error) {
+        console.error(`rm error: ${error}`);
+    }
     try {
-        const { stdout, stderr } = await exec(command);
+        console.log(`exec: ${tspCommand.command}`);
+        const { stdout, stderr } = await exec(tspCommand.command);
         if (stdout) console.log(`stdout: ${stdout}`);
         if (stderr) console.error(`stderr: ${stderr}`);
     } catch (error) {
@@ -162,9 +169,6 @@ async function getSubdirectories(baseDir: string, flags: RegenerateFlags): Promi
                 const mainTspRelativePath = toPosix(relative(baseDir, mainTspPath));
                 if (flags.flavor === "unbranded" && mainTspRelativePath.includes("azure")) return;
 
-                // after xml support, remove this check
-                if (mainTspRelativePath.includes("xml")) return;
-
                 // after fix test generation for nested operation group, remove this check
                 if (mainTspRelativePath.includes("client-operation-group")) return;
 
@@ -206,8 +210,13 @@ function defaultPackageName(spec: string): string {
         .toLowerCase();
 }
 
-function addOptions(spec: string, generatedFolder: string, flags: RegenerateFlags): string[] {
-    const emitterConfigs: string[] = [];
+interface EmitterConfig {
+    optionsStr: string;
+    outputDir: string;
+}
+
+function addOptions(spec: string, generatedFolder: string, flags: RegenerateFlags): EmitterConfig[] {
+    const emitterConfigs: EmitterConfig[] = [];
     for (const config of getEmitterOption(spec)) {
         const options: Record<string, string> = { ...config };
         options["flavor"] = flags.flavor;
@@ -224,41 +233,39 @@ function addOptions(spec: string, generatedFolder: string, flags: RegenerateFlag
         if (flags.flavor === "unbranded") {
             options["company-name"] = "Unbranded";
         }
-        options["examples-directory"] = toPosix(join(dirname(spec), "examples"));
+        options["examples-dir"] = toPosix(join(dirname(spec), "examples"));
         const configs = Object.entries(options).flatMap(([k, v]) => {
             return `--option @azure-tools/typespec-python.${k}=${v}`;
         });
-        emitterConfigs.push(configs.join(" "));
+        emitterConfigs.push({
+            optionsStr: configs.join(" "),
+            outputDir: options["emitter-output-dir"],
+        });
     }
     return emitterConfigs;
 }
-
-async function _regenerateSingle(spec: string, flags: RegenerateFlags): Promise<void> {
-    // Perform some asynchronous operation here
-    const options = addOptions(spec, PLUGIN_DIR, flags);
-    const commandPromises = options.map((option) => {
-        const command = `tsp compile ${spec} --emit=${toPosix(PLUGIN_DIR)} ${option}`;
-        console.log(command);
-        return executeCommand(command);
+function _getCmdList(spec: string, flags: RegenerateFlags): TspCommand[] {
+    return addOptions(spec, PLUGIN_DIR, flags).map((option) => {
+        return {
+            outputDir: option.outputDir,
+            command: `tsp compile ${spec} --emit=${toPosix(PLUGIN_DIR)} ${option.optionsStr}`,
+        };
     });
-    await Promise.all(commandPromises);
 }
 
-async function regenerate(flags: RegenerateFlagsInput): Promise<boolean> {
+async function regenerate(flags: RegenerateFlagsInput): Promise<void> {
     if (flags.flavor === undefined) {
-        const azureGeneration = await regenerate({ ...flags, flavor: "azure" });
-        const unbrandedGeneration = await regenerate({ ...flags, flavor: "unbranded" });
-        return azureGeneration && unbrandedGeneration;
+        await regenerate({ ...flags, flavor: "azure" });
+        await regenerate({ ...flags, flavor: "unbranded" });
     } else {
         const flagsResolved = { debug: false, flavor: flags.flavor, ...flags };
         const CADL_RANCH_DIR = resolve(PLUGIN_DIR, "node_modules/@azure-tools/cadl-ranch-specs/http");
         const subdirectories = await getSubdirectories(CADL_RANCH_DIR, flagsResolved);
-        const promises = subdirectories.map(async (subdirectory) => {
-            // Perform additional asynchronous operations on each subdirectory here
-            await _regenerateSingle(subdirectory, flagsResolved);
-        });
-        await Promise.all(promises);
-        return true;
+        const cmdList: TspCommand[] = subdirectories.flatMap((subdirectory) =>
+            _getCmdList(subdirectory, flagsResolved),
+        );
+        const PromiseCommands = cmdList.map((tspCommand) => executeCommand(tspCommand));
+        await Promise.all(PromiseCommands);
     }
 }
 

package/scripts/eng/run-tests.ts

@@ -0,0 +1,90 @@
+/* eslint-disable no-console */
+import { execSync } from "child_process";
+import { readFileSync } from "fs";
+import { join } from "path";
+import yargs from "yargs";
+import { hideBin } from "yargs/helpers";
+import { fileURLToPath } from "url";
+
+interface Arguments {
+    validFolders: string[];
+    folder?: string;
+    command?: string;
+    name?: string;
+}
+
+const validCommands = ["ci", "lint", "mypy", "pyright", "apiview"];
+
+// Parse command-line arguments using yargs
+const argv = yargs(hideBin(process.argv))
+    .option("validFolders", {
+        alias: "vf",
+        describe: "Specify the valid folders",
+        type: "array",
+        default: ["azure", "unbranded"],
+    })
+    .option("folder", {
+        alias: "f",
+        describe: "Specify the folder to use",
+        type: "string",
+    })
+    .option("command", {
+        alias: "c",
+        describe: "Specify the command to run",
+        choices: validCommands,
+        type: "string",
+    })
+    .option("name", {
+        alias: "n",
+        describe: "Specify the name of the test",
+        type: "string",
+    }).argv as Arguments;
+
+const foldersToProcess = argv.folder ? [argv.folder] : argv.validFolders;
+
+const commandToRun = argv.command || "all";
+
+function getCommand(command: string, folder: string, name?: string): string {
+    if (!validCommands.includes(command)) throw new Error(`Unknown command '${command}'.`);
+    const retval = `FOLDER=${folder} tox -c ./test/${folder}/tox.ini -e ${command}`;
+    if (name) {
+        return `${retval} -- -f ${name}`;
+    }
+    return retval;
+}
+
+function sectionExistsInToxIni(command: string, folder: string): boolean {
+    const toxIniPath = join(fileURLToPath(import.meta.url), `../../../test/${folder}/tox.ini`);
+    const toxIniContent = readFileSync(toxIniPath, "utf-8");
+    const sectionHeader = `[testenv:${command}]`;
+    return toxIniContent.includes(sectionHeader);
+}
+
+function myExecSync(command: string, folder: string, name?: string): void {
+    if (!sectionExistsInToxIni(command, folder)) {
+        console.log(`No section for ${command} in tox.ini for folder ${folder}. Skipping...`);
+        return;
+    }
+    execSync(getCommand(command, folder, name), { stdio: "inherit" });
+}
+
+foldersToProcess.forEach((folder) => {
+    try {
+        if (commandToRun === "all") {
+            for (const key of validCommands) {
+                console.log(`Running ${key} for folder ${folder}...`);
+                myExecSync(key, folder, argv.name);
+            }
+        } else if (getCommand(commandToRun, folder, argv.name)) {
+            console.log(`Running ${commandToRun} for folder ${folder}...`);
+            myExecSync(commandToRun, folder, argv.name);
+        } else {
+            console.error(`Error: Unknown command '${commandToRun}'.`);
+            process.exit(1);
+        }
+    } catch (error) {
+        console.error((error as Error).message);
+        console.error(`Error executing command for folder ${folder}: ${(error as Error).message}`);
+        process.exit(1);
+    }
+});

package/scripts/eng/utils.ts

@@ -0,0 +1,38 @@
+/* eslint-disable no-console */
+import { exec } from "child_process";
+import process from "process";
+import { existsSync } from "fs";
+import { dirname, join } from "path";
+import { fileURLToPath } from "url";
+import chalk from "chalk";
+
+// execute the command
+export function executeCommand(command: string, prettyName: string) {
+    exec(command, (error, stdout, stderr) => {
+        if (error) {
+            console.error(chalk.red(`Error executing ${command}(stdout): ${stdout}`));
+            console.error(chalk.red(`Error executing ${command}{stderr}: ${stderr}`));
+            process.exit(1);
+        }
+        if (stderr) {
+            // Process stderr output
+            console.log(chalk.yellow(`${command}:\n${stderr}`));
+            return;
+        }
+        console.log(chalk.green(`${prettyName} passed`));
+    });
+}
+
+// Function to run a command and log the output
+export function runCommand(command: string, prettyName: string) {
+    let pythonPath = join(dirname(fileURLToPath(import.meta.url)), "..", "..", "venv/");
+    if (existsSync(join(pythonPath, "bin"))) {
+        pythonPath = join(pythonPath, "bin", "python");
+    } else if (existsSync(join(pythonPath, "Scripts"))) {
+        pythonPath = join(pythonPath, "Scripts", "python");
+    } else {
+        throw new Error(pythonPath);
+    }
+    command = `${pythonPath} -m ${command}`;
+    executeCommand(command, prettyName);
+}

package/scripts/run_tsp.py

@@ -7,7 +7,8 @@ import sys
 import venv
 import logging
 from pathlib import Path
-from venvtools import python_run
+from pygen import m2r, preprocess, codegen, black
+from pygen.utils import parse_args
 
 _ROOT_DIR = Path(__file__).parent.parent
 
@@ -34,7 +35,8 @@ if __name__ == "__main__":
         breakpoint()  # pylint: disable=undefined-variable
 
     # run m2r
-    python_run(venv_context, "generator.pygen.m2r", command=sys.argv[1:])
-    python_run(venv_context, "generator.pygen.preprocess.__init__", command=sys.argv[1:])
-    python_run(venv_context, "generator.pygen.codegen.__init__", command=sys.argv[1:])
-    python_run(venv_context, "generator.pygen.black", command=sys.argv[1:])
+    args, unknown_args = parse_args()
+    m2r.M2R(output_folder=args.output_folder, cadl_file=args.cadl_file, **unknown_args).process()
+    preprocess.PreProcessPlugin(output_folder=args.output_folder, cadl_file=args.cadl_file, **unknown_args).process()
+    codegen.CodeGenerator(output_folder=args.output_folder, cadl_file=args.cadl_file, **unknown_args).process()
+    black.BlackScriptPlugin(output_folder=args.output_folder, **unknown_args).process()