Merge branch 'main' into anomaly-explanation

Author: lcawl

compiler/src/steps/validate-rest-spec.ts

@@ -68,6 +68,35 @@ export default async function validateRestSpec (model: model.Model, jsonSpec: Ma
         }
       }
 
+      // are all path parameters properly required or optional?
+      let urlPartsRequired = new Set(urlParts)
+      // A part is considered required if it is included in
+      // every path for the API endpoint.
+      for (const path of spec.url.paths) {
+        if (path.parts == null) {
+          // No parts means that all path parameters are optional!
+          urlPartsRequired = new Set()
+          break
+        }
+        urlPartsRequired = new Set([...Object.keys(path.parts)].filter((x) => urlPartsRequired.has(x)))
+      }
+
+      // transform [{name: ..., required: ...}] -> {name: {required: ...}}
+      const pathPropertyMap: Record<string, model.Property> = requestProperties.path.reduce((prev, prop) => ({ ...prev, [prop.name]: prop }), {})
+      for (const name of pathProperties) {
+        // okay to skip if it's not included since this scenario
+        // is covered above with a different error.
+        if (!urlParts.includes(name)) {
+          continue
+        }
+        // Find the mismatches between the specs
+        if (urlPartsRequired.has(name) && !pathPropertyMap[name].required) {
+          errors.addEndpointError(endpoint.name, 'request', `${endpoint.request.name}: path parameter '${name}' is required in the json spec`)
+        } else if (!urlPartsRequired.has(name) && pathPropertyMap[name].required) {
+          errors.addEndpointError(endpoint.name, 'request', `${endpoint.request.name}: path parameter '${name}' is optional in the json spec`)
+        }
+      }
+
       if (spec.params != null) {
         const params = Object.keys(spec.params)
         const queryProperties = requestProperties.query.map(property => property.name)

output/schema/schema.json

@@ -34,7 +34,7 @@ export interface Request extends RequestBase {
      * Comma-separated list of data streams, indices, and index aliases used to limit the request. Wildcard (`*`) expressions are supported.
      * To target all data streams and indices in a cluster, omit this parameter or use `_all` or `*`.
      */
-    index: Indices
+    index?: Indices
   }
   query_parameters: {
     /**

output/schema/validation-errors.json

@@ -0,0 +1,37 @@
+{
+  "semantic_search": {
+    "documentation": {
+      "url": "https://www.elastic.co/guide/en/elasticsearch/reference/current/semantic-search.html",
+      "description": "Semantic search API using dense vector similarity"
+    },
+    "stability": "experimental",
+    "visibility": "public",
+    "headers": {
+      "accept": ["application/json"],
+      "content_type": ["application/json"]
+    },
+    "url": {
+      "paths": [
+        {
+          "path": "/{index}/_semantic_search",
+          "methods": ["GET", "POST"],
+          "parts": {
+            "index": {
+              "type": "list",
+              "description": "A comma-separated list of index names to search; use `_all` to perform the operation on all indices"
+            }
+          }
+        }
+      ]
+    },
+    "params": {
+      "routing": {
+        "type": "list",
+        "description": "A comma-separated list of specific routing values"
+      }
+    },
+    "body": {
+      "description": "The search definition"
+    }
+  }
+}

output/typescript/types.ts

@@ -79,7 +79,6 @@ import {
 } from '@_types/mapping/geo'
 
 export class PropertyBase {
-  local_metadata?: Metadata
   /**
    * Metadata about the field.
    * @doc_id mapping-meta-field

specification/_global/rank_eval/RankEvalRequest.ts

@@ -43,7 +43,7 @@ export interface Request extends RequestBase {
     /**
      * A single target to search. If the target is an index alias, it must resolve to a single index.
      */
-    index: IndexName | IndexAlias
+    index?: IndexName | IndexAlias
   }
   query_parameters: {
     /**

specification/_json_spec/semantic_search.json

@@ -37,6 +37,6 @@ export interface Request extends RequestBase {
     /**
      * The unique identifier of the trained model.
      */
-    model_id?: Id
+    model_id: Id
   }
 }