@typespec/http-client-python 0.33.0 → 0.34.1

package/emitter/src/node-runner.ts

@@ -16,6 +16,7 @@ import { blackExcludeDirs, PYGEN_WHEEL_FILENAME } from "./constants.js";
 import { saveCodeModelAsYaml } from "./external-process.js";
 import { PythonEmitterOptions, reportDiagnostic } from "./lib.js";
 import { runPython3 } from "./run-python3.js";
+import { quoteShellArg } from "./utils.js";
 
 export interface RunNodeEmitArgs {
   context: EmitContext<PythonEmitterOptions>;
@@ -106,7 +107,7 @@ export async function runNodeEmit({
   commandArgs["output-folder"] = outputDir;
   commandArgs["tsp-file"] = yamlPath;
   const commandFlags = Object.entries(commandArgs)
-    .map(([key, value]) => `--${key}=${value}`)
+    .map(([key, value]) => `--${key}=${quoteShellArg(String(value))}`)
     .join(" ");
   const command = `${venvPath} ${root}/eng/scripts/setup/run_tsp.py ${commandFlags}`;
   execSync(command);

package/emitter/src/types.ts

@@ -233,6 +233,7 @@ function emitProperty(
     // Python convert all the type of file part to FileType so clear these models' usage so that they won't be generated
     addDisableGenerationMap(context, property.type);
   }
+  const isNullable = !isMultipartFileInput && sourceType.kind === "nullable";
   return {
     clientName: getClientName(property),
     isExactName: property.isExactName,
@@ -242,6 +243,7 @@ function emitProperty(
         : property.serializationOptions?.json?.name) ?? property.name,
     type: getType(context, sourceType),
     optional: property.optional,
+    nullable: isNullable,
     description: property.summary ? property.summary : property.doc,
     addedOn: getAddedOn(context, property),
     apiVersions: property.apiVersions,

package/emitter/src/utils.ts

@@ -268,6 +268,19 @@ export function capitalize(name: string): string {
   return name[0].toUpperCase() + name.slice(1);
 }
 
+/**
+ * Quotes a value so it can be safely embedded in a shell command line.
+ *
+ * The value is wrapped in double quotes (handling spaces and other separators)
+ * and any embedded double quotes are escaped. This quoting must only be applied
+ * when the value is passed through a shell (e.g. `execSync`); it must never be
+ * baked into the option value itself, otherwise the quotes leak into non-shell
+ * consumers such as the Pyodide runtime and end up in generated files.
+ */
+export function quoteShellArg(value: string): string {
+  return `"${value.replace(/\\/g, "\\\\").replace(/"/g, '\\"')}"`;
+}
+
 // Library namespaces that should not be used as client namespaces
 const LIB_NAMESPACE = [
   "azure.core",