Update the sub-schema for poetry. (#4670)

* Update the sub-schema for poetry.

`"type": "string"` was missing from a few arrays.

`"additionalProperties": false,` was missing from a few objects.

Replaced all "oneOf" with "anyOf". For most cases in this schema, both are identical,
but "anyOf" is simpler. Only for "poetry-script-table" there is a minor difference
and "anyOf" is the correct option.

For the 'source' property, I replaced the "if", "then" and "else" with equivalent
logic that avoids these constructs.

These issues showed up while converting the poetry JSON schema to a
[TOML schema](https://github.com/udifuchs/toml-schema)
that I am experimenting with.

One feature of the JSON to TOML conversion tool is that it complains when
the JSON schema is different from what the tool expects.

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Author: udifuchs

src/schemas/json/partial-poetry.json

@@ -44,7 +44,7 @@
       "description": "A Python packaging format."
     },
     "poetry-package-formats": {
-      "oneOf": [
+      "anyOf": [
         {
           "$ref": "#/definitions/poetry-package-format"
         },
@@ -58,7 +58,7 @@
       "description": "The format(s) for which the package must be included."
     },
     "poetry-dependency-any": {
-      "oneOf": [
+      "anyOf": [
         {
           "$ref": "#/definitions/poetry-dependency"
         },
@@ -140,13 +140,14 @@
       "additionalProperties": false,
       "properties": {
         "git": {
-          "type": "string",
           "description": "The url of the git repository.",
           "anyOf": [
             {
+              "type": "string",
               "format": "uri"
             },
             {
+              "type": "string",
               "pattern": "^([A-Za-z0-9\\-]+@|https://|http://)[A-Za-z][A-Za-z0-9+.-]*(:|/)[A-Za-z0-9\\-\\.]+(/[A-Za-z0-9\\-_\\.]+)+\\.git$"
             }
           ]
@@ -314,7 +315,7 @@
       "type": "array",
       "minItems": 1,
       "items": {
-        "oneOf": [
+        "anyOf": [
           {
             "$ref": "#/definitions/poetry-dependency"
           },
@@ -337,8 +338,7 @@
       }
     },
     "poetry-script-table": {
-      "type": "object",
-      "oneOf": [
+      "anyOf": [
         {
           "$ref": "#/definitions/poetry-extra-script-legacy"
         },
@@ -413,14 +413,17 @@
       }
     },
     "poetry-build-section": {
-      "oneOf": [
+      "anyOf": [
         {
           "$ref": "#/definitions/poetry-build-script"
         },
         {
           "$ref": "#/definitions/poetry-build-config"
         }
       ]
+    },
+    "poetry-priority": {
+      "enum": ["default", "primary", "secondary", "supplemental", "explicit"]
     }
   },
   "type": "object",
@@ -489,6 +492,9 @@
     },
     "classifiers": {
       "type": "array",
+      "items": {
+        "type": "string"
+      },
       "description": "A list of trove classifiers."
     },
     "packages": {
@@ -543,6 +549,9 @@
     },
     "exclude": {
       "type": "array",
+      "items": {
+        "type": "string"
+      },
       "description": "A list of files and folders to exclude."
     },
     "dependencies": {
@@ -551,7 +560,6 @@
       "properties": {
         "python": {
           "$ref": "#/definitions/poetry-dependency",
-          "type": "string",
           "description": "The Python versions the package is compatible with."
         }
       },
@@ -568,6 +576,7 @@
           "$ref": "#/definitions/poetry-dependency-any"
         }
       },
+      "additionalProperties": false,
       "x-tombi-table-keys-order": "ascending"
     },
     "extras": {
@@ -580,6 +589,7 @@
           }
         }
       },
+      "additionalProperties": false,
       "x-tombi-table-keys-order": "ascending"
     },
     "group": {
@@ -610,6 +620,7 @@
           "additionalProperties": false
         }
       },
+      "additionalProperties": false,
       "x-tombi-table-keys-order": "ascending"
     },
     "build": {
@@ -620,7 +631,7 @@
       "description": "A hash of scripts to be installed.",
       "patternProperties": {
         "^[a-zA-Z-_.0-9]+$": {
-          "oneOf": [
+          "anyOf": [
             {
               "$ref": "#/definitions/poetry-script-legacy"
             },
@@ -630,6 +641,7 @@
           ]
         }
       },
+      "additionalProperties": false,
       "x-tombi-table-keys-order": "ascending"
     },
     "plugins": {
@@ -657,9 +669,11 @@
             "^[a-zA-Z-_.0-9]+$": {
               "type": "string"
             }
-          }
+          },
+          "additionalProperties": false
         }
       },
+      "additionalProperties": false,
       "x-tombi-table-keys-order": "ascending"
     },
     "urls": {
@@ -670,46 +684,59 @@
           "description": "The full url of the custom url."
         }
       },
+      "additionalProperties": false,
       "x-tombi-table-keys-order": "ascending"
     },
     "source": {
       "$comment": "The unique 'pypi' source constraint is not implemented yet.",
       "type": "array",
       "items": {
-        "type": "object",
-        "additionalProperties": false,
-        "required": ["name"],
-        "properties": {
-          "name": {
-            "type": "string",
-            "description": "The name of the source."
-          },
-          "url": {
-            "type": "string",
-            "format": "uri",
-            "description": "The url of the source."
+        "anyOf": [
+          {
+            "type": "object",
+            "additionalProperties": false,
+            "required": ["name"],
+            "properties": {
+              "name": {
+                "const": "pypi",
+                "description": "The name of the source."
+              },
+              "priority": {
+                "$ref": "#/definitions/poetry-priority",
+                "description": "The priority of the source."
+              }
+            }
           },
-          "priority": {
-            "enum": [
-              "default",
-              "primary",
-              "secondary",
-              "supplemental",
-              "explicit"
-            ],
-            "description": "The priority of the source."
+          {
+            "type": "object",
+            "additionalProperties": false,
+            "required": ["name", "url"],
+            "properties": {
+              "name": {
+                "allOf": [
+                  {
+                    "type": "string"
+                  },
+                  {
+                    "not": {
+                      "const": "pypi"
+                    }
+                  }
+                ],
+                "description": "The name of the source."
+              },
+              "url": {
+                "type": "string",
+                "format": "uri",
+                "description": "The url of the source."
+              },
+              "priority": {
+                "$ref": "#/definitions/poetry-priority",
+                "description": "The priority of the source."
+              }
+            }
           }
-        },
-        "if": {
-          "properties": { "name": { "const": "pypi" } }
-        },
-        "then": {
-          "propertyNames": { "not": { "const": "url" } }
-        },
-        "else": {
-          "required": ["url"],
-          "properties": { "url": true }
-        }
+        ]
       }
     }
   }