fix: array schema output order (#329)

previously the dependency graph didn't consider top-level array `items`
correctly, which meant you could have situations that resulted in errors
like:
```
 error TS2454: Variable 's_SomeArrayItems' is used before being assigned.
```

Author: mnahkies

integration-tests/typescript-express/src/generated/api.github.com.yaml/schemas.ts

@@ -21,6 +21,10 @@ describe.each(testVersions)("%s - core/dependency-graph", (version) => {
       graph.order.indexOf("s_AOrdering"),
     )
 
+    expect(graph.order.indexOf("s_AArrayOrdering")).toBeGreaterThan(
+      graph.order.indexOf("s_ZOrdering"),
+    )
+
     expect(graph.order.indexOf("s_ObjectWithRefs")).toBeGreaterThan(
       graph.order.indexOf("s_SimpleObject"),
     )

integration-tests/typescript-koa/src/generated/api.github.com.yaml/schemas.ts

@@ -31,6 +31,7 @@ const getDependenciesFromSchema = (
         ? [schema.additionalProperties]
         : [],
     )
+    .concat(schema.type === "array" ? [schema.items] : [])
 
   return allSources.reduce((acc, it) => {
     if (isRef(it)) {

packages/openapi-code-generator/src/core/dependency-graph.spec.ts

@@ -152,6 +152,11 @@ components:
         name:
           type: string
 
+    AArrayOrdering:
+      type: array
+      items:
+        $ref: '#/components/schemas/ZOrdering'
+
     AdditionalPropertiesBool:
       type: object
       additionalProperties: true

packages/openapi-code-generator/src/core/dependency-graph.ts

@@ -156,6 +156,11 @@ components:
         name:
           type: string
 
+    AArrayOrdering:
+      type: array
+      items:
+        $ref: '#/components/schemas/ZOrdering'
+
     AdditionalPropertiesBool:
       type: object
       additionalProperties: true