scripts: check in library generation script

Also update build_twilio_library.py to use subprocess instead of
shelling out to os.system.

Author: kevinburkesegment

scripts/build_twilio_library.py

@@ -2,10 +2,13 @@
 import json
 import os
 import re
+import shlex
 import shutil
+import subprocess
 from pathlib import Path
 from typing import List, Tuple
 
+
 from clean_unused_imports import remove_unused_imports, remove_duplicate_imports
 from process_orgs_api import preprocess_orgs_spec
 
@@ -88,18 +91,34 @@ def generate_domain_for_language(spec_file: str, config_path: str, spec_folder:
     with open(full_config_path, 'w') as f:
         f.write(json.dumps(config))
 
-
 def run_openapi_generator(parent_dir: Path, language: str) -> None:
-    properties = '-DapiTests=false'
-    if language in {'node', 'python'}:
-        properties += ' -DskipFormModel=false'
-
-    command = f'cd {parent_dir} && java {properties} ' \
-              f'-cp target/twilio-openapi-generator.jar ' \
-              f'org.openapitools.codegen.OpenAPIGenerator batch {CONFIG_FOLDER}/{language}/*'
-
-    if os.system(command + '> /dev/null') != 0:  # Suppress stdout
-        raise RuntimeError()
+    properties = "-DapiTests=false"
+    if language in {"node", "python"}:
+        properties += " -DskipFormModel=false"
+
+    command = [
+        "java",
+        *properties.split(),  # Splits e.g. "-DapiTests=false -DskipFormModel=false" into separate arguments
+        "-cp",
+        "target/twilio-openapi-generator.jar",
+        "org.openapitools.codegen.OpenAPIGenerator",
+        "batch",
+        f"{CONFIG_FOLDER}/{language}/*"
+    ]
+
+    printable_cmd = " ".join(shlex.quote(arg) for arg in command)
+    print(f"Running command: {printable_cmd}")
+
+    try:
+        subprocess.run(
+            command,
+            cwd=parent_dir,              # Change working directory to parent_dir
+            check=True,                  # Raise CalledProcessError on non-zero exit
+            stdout=subprocess.DEVNULL,   # Suppress standard output
+        )
+    except subprocess.CalledProcessError as exc:
+        # Wrap the original exception for more informative error handling
+        raise RuntimeError("OpenAPI generation failed.") from exc
 
 
 def get_domain_info(oai_spec_location: str, domain: str, is_file: bool = False) -> Tuple[str, str, str]:

scripts/generate_libraries/main.py

@@ -0,0 +1,94 @@
+#!/usr/bin/env python3
+
+import argparse
+import glob
+import os
+import scripts.build_twilio_library as build_library
+
+
+def parse_args():
+    parser = argparse.ArgumentParser(
+        description="Generate files for Twilio library in different languages."
+    )
+    parser.add_argument(
+        "--language",
+        choices=["csharp", "java", "node", "php", "go", "python", "ruby", "terraform"],
+        required=True,
+        help="Specify the target language for code generation."
+    )
+    parser.add_argument(
+        "--base",
+        default=".",
+        help="Base directory for code generation. Defaults to current directory."
+    )
+    parser.add_argument(
+        "--input",
+        default=None,
+        help="Path to the OpenAPI spec JSON files. If not specified, defaults to 'spec/json' inside the base directory."
+    )
+    return parser.parse_args()
+
+
+def get_output_dir(base, language):
+    """
+    Constructs the output directory for each language based on the given base path.
+    Update these paths as needed for your environment or repository layout.
+    """
+    if language == "csharp":
+        return os.path.join(base, "twilio-csharp/src/Twilio")
+    elif language == "java":
+        return os.path.join(base, "twilio-java/src/main/java/com/twilio")
+    elif language == "node":
+        return os.path.join(base, "twilio-node/src")
+    elif language == "php":
+        return os.path.join(base, "twilio-php/src/Twilio")
+    elif language == "go":
+        return os.path.join(base, "twilio-go")
+    elif language == "python":
+        return os.path.join(base, "twilio-python/twilio")
+    elif language == "ruby":
+        return os.path.join(base, "twilio-ruby/lib/twilio-ruby")
+    elif language == "terraform":
+        return os.path.join(base, "terraform-provider-twilio")
+    else:
+        return None
+
+
+def generate_files(input_dir, output_dir, language):
+    # If needed, you can iterate over the input spec files here using glob:
+    # for filepath in glob.glob(os.path.join(input_dir, "*.json")):
+    #     print(filepath)
+    build_library.build(input_dir, output_dir, language)
+
+
+def main():
+    args = parse_args()
+
+    # Determine input directory
+    input_dir = args.input if args.input else os.path.join(args.base, "spec/json")
+
+    # Determine output directory for the chosen language
+    output_dir = get_output_dir(args.base, args.language)
+
+    if not output_dir:
+        print("Unsupported language or path couldn't be resolved.")
+        return
+
+    # Ensure the output directory exists
+    os.makedirs(output_dir, exist_ok=True)
+
+    # Run a make install (if needed for your environment)
+    os.system("make install")
+
+    # Generate files
+    generate_files(input_dir, output_dir, args.language)
+
+    # Optional language-specific post-processing
+    if args.language == "go":
+        os.chdir(output_dir)
+        os.system("go fmt ./...")
+        os.system("goimports -w .")
+
+
+if __name__ == "__main__":
+    main()