quicktype-core 23.0.2 → 23.0.3

package/dist/language/Python.d.ts

@@ -46,7 +46,7 @@ export declare class PythonRenderer extends ConvenienceRenderer {
     protected withImport(module: string, name: string): Sourcelike;
     protected withTyping(name: string): Sourcelike;
     protected namedType(t: Type): Sourcelike;
-    protected pythonType(t: Type): Sourcelike;
+    protected pythonType(t: Type, _isRootTypeDef?: boolean): Sourcelike;
     protected declarationLine(t: Type): Sourcelike;
     protected declareType<T extends Type>(t: T, emitter: () => void): void;
     protected emitClassMembers(t: ClassType): void;

package/dist/language/Python.js

@@ -240,15 +240,19 @@ class PythonRenderer extends ConvenienceRenderer_1.ConvenienceRenderer {
             return name;
         return ["'", name, "'"];
     }
-    pythonType(t) {
+    pythonType(t, _isRootTypeDef = false) {
         const actualType = (0, Transformers_1.followTargetType)(t);
         return (0, TypeUtils_1.matchType)(actualType, _anyType => this.withTyping("Any"), _nullType => "None", _boolType => "bool", _integerType => "int", _doubletype => "float", _stringType => "str", arrayType => [this.withTyping("List"), "[", this.pythonType(arrayType.items), "]"], classType => this.namedType(classType), mapType => [this.withTyping("Dict"), "[str, ", this.pythonType(mapType.values), "]"], enumType => this.namedType(enumType), unionType => {
             const [hasNull, nonNulls] = (0, TypeUtils_1.removeNullFromUnion)(unionType);
             const memberTypes = Array.from(nonNulls).map(m => this.pythonType(m));
             if (hasNull !== null) {
                 let rest = [];
-                if (!this.getAlphabetizeProperties() && this.pyOptions.features.dataClasses)
+                if (!this.getAlphabetizeProperties() && this.pyOptions.features.dataClasses && _isRootTypeDef) {
+                    // Only push "= None" if this is a root level type def
+                    //   otherwise we may get type defs like List[Optional[int] = None]
+                    //   which are invalid
                     rest.push(" = None");
+                }
                 if (nonNulls.size > 1) {
                     this.withImport("typing", "Union");
                     return [
@@ -344,7 +348,7 @@ class PythonRenderer extends ConvenienceRenderer_1.ConvenienceRenderer {
                 else {
                     this.forEachClassProperty(t, "none", (name, jsonName, cp) => {
                         this.emitDescription(this.descriptionForClassProperty(t, jsonName));
-                        this.emitLine(name, this.typeHint(": ", this.pythonType(cp.type)));
+                        this.emitLine(name, this.typeHint(": ", this.pythonType(cp.type, true)));
                     });
                 }
                 this.ensureBlankLine();

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "quicktype-core",
-    "version": "23.0.2",
+    "version": "23.0.3",
     "description": "The quicktype engine as a library",
     "license": "Apache-2.0",
     "main": "dist/index.js",