Do not mark repeated fields as optional

go.mod

@@ -1,4 +1,4 @@
-module github.com/chrusty/protoc-gen-jsonschema
+module github.com/ikstewa/protoc-gen-jsonschema
 
 go 1.14
 

internal/converter/testdata/payload_message_2.go

@@ -53,6 +53,15 @@ const PayloadMessage2 = `{
                         }
                     ],
                     "title": "Topology"
+                },
+                "array": {
+                    "items": {
+                        "type": "integer"
+                    },
+                    "type": "array"
+                },
+                "opt_int": {
+                    "type": "integer"
                 }
             },
             "additionalProperties": true,

internal/converter/testdata/proto/PayloadMessage2.proto

@@ -15,10 +15,12 @@ message PayloadMessage2 {
         ARRAY_OF_MESSAGE = 5;
     }
 
-    string name       = 1;
-    string timestamp  = 2;
-    int32 id          = 3;
-    float rating      = 4;
-    bool complete     = 5;
-    Topology topology = 6;
+    string name            = 1;
+    string timestamp       = 2;
+    int32 id               = 3;
+    float rating           = 4;
+    bool complete          = 5;
+    Topology topology      = 6;
+    repeated int32 array   = 7;
+    optional int32 opt_int = 8;
 }

internal/converter/types.go

@@ -667,7 +667,8 @@ func (c *Converter) recursiveConvertMessageType(curPkg *ProtoPackage, msgDesc *d
 
 		// Enforce all_fields_required:
 		if messageFlags.AllFieldsRequired {
-			if fieldDesc.OneofIndex == nil && !fieldDesc.GetProto3Optional() {
+			if fieldDesc.OneofIndex == nil && !fieldDesc.GetProto3Optional() && fieldDesc.GetLabel() != descriptor.FieldDescriptorProto_LABEL_REPEATED {
+
 				if c.Flags.UseJSONFieldnamesOnly {
 					jsonSchemaType.Required = append(jsonSchemaType.Required, fieldDesc.GetJsonName())
 				} else {

jsonschemas/PayloadMessage2.json

@@ -51,6 +51,15 @@
                         }
                     ],
                     "title": "Topology"
+                },
+                "array": {
+                    "items": {
+                        "type": "integer"
+                    },
+                    "type": "array"
+                },
+                "opt_int": {
+                    "type": "integer"
                 }
             },
             "additionalProperties": true,
@@ -59,4 +68,4 @@
             "description": "PayloadMessage2 contains some common types  PayloadMessage2 is used throughout the test suite and can have multi-line comments"
         }
     }
-}
+}