Fix simplification of tuple about modelina

[magicmatatjahu]

Also if I write something like:

{ "type": "array", "items": { "oneOf": [ { "type": "string" }, { "type": "number" } ] } }

then this is true union, but I got this same CommonModel, like in tuple case, so user/dev cannot determine which type should be rendered for model and for that must check the originalSchema, which is not acceptable.

[jonaslagoni]

This might get fixed by an upcoming pr, there are a bug in the type simplifier.

[jonaslagoni]

The simplifier does semi-require the root schema to be an object... I'll try spend a few hours to figure out how we can handle this better.

[jonaslagoni]

With the master branch I see the output to be the following:

{
  originalSchema: {
    type: "array",
    items: [
      {
        type: "string",
      },
      {
        type: "number",
      },
    ],
  },
  type: "array",
  items: {
    type: [
      "string",
      "number",
    ],
    originalSchema: {
      type: "array",
      items: [
        {
          type: "string",
        },
        {
          type: "number",
        },
      ],
    },
  },
}

Which is as expected right?

See #82 for more details

[magicmatatjahu]

@jonaslagoni No, this is not a true, because this schema indicates that you have an array of union, not a tuple. We should have schema like this:

"items": [
  { type: "string" },
  { type: "number" },
]

[magicmatatjahu]

We concluded that this a feature request, but also a bug :)

[github-actions]

This issue has been automatically marked as stale because it has not had recent activity 😴
It will be closed in 60 days if no further activity occurs. To unstale this issue, add a comment with detailed explanation.
Thank you for your contributions ❤️