Python: added operationid validation in openapi spec (#12551)

Description
This PR enforces the presence of an explicit `operationId` for every
operation in the OpenAPI specification. Previously, if `operationId` was
missing, a default value was generated using the path and HTTP method.
Now, the parser will raise a PluginInitializationError if operationId is
not provided, ensuring that all operations are uniquely and explicitly
identified as required by the OpenAPI standard.

Key changes:

- Added validation to check for the existence of operationId in each
operation.
- If operationId is missing, a clear error is raised, preventing
fallback to a default value.
-
[python/tests/unit/connectors/openapi_plugin/no-operationid-openapi.yaml](https://github.com/microsoft/semantic-kernel/compare/main...sshandilya1991:semantic-kernel:sshandilya/12443_add_operation_id_validation?expand=1#diff-3d272039b3afa97f89805f0ee0258a98b613f0f373c603467b336a8912d5651a)
serves as a test case.
- Closes #12443

Edit:
- Added more validation for operationId to be unique & throw in case
duplicate operationId found while parsing.

Author: sshandilya1991

python/semantic_kernel/connectors/openapi_plugin/openapi_parser.py

@@ -206,6 +206,7 @@ def create_rest_api_operations(
 
         paths = parsed_document.get("paths", {})
         request_objects = {}
+        unique_operation_ids_registered: set[str] = set()
 
         servers = parsed_document.get("servers", [])
 
@@ -230,7 +231,15 @@ def create_rest_api_operations(
         for path, methods in paths.items():
             for method, details in methods.items():
                 request_method = method.lower()
-                operationId = details.get("operationId", path + "_" + request_method)
+                # Validate that operationId exists
+                if "operationId" not in details:
+                    raise PluginInitializationError(f"operationId missing, path: '{path}', method: '{method}'")
+                operationId = details["operationId"]
+                if operationId in unique_operation_ids_registered:
+                    raise PluginInitializationError(
+                        f"Duplicate operationId: '{operationId}', path: '{path}', method: '{method}'"
+                    )
+                unique_operation_ids_registered.add(operationId)
 
                 summary = details.get("summary", None)
                 description = details.get("description", None)

python/tests/unit/connectors/openapi_plugin/duplicate-operationid-openapi.yaml

@@ -0,0 +1,32 @@
+openapi: 3.0.3
+info:
+  title: Simple HTTPBin API
+  version: 1.0.0
+
+servers:
+  - url: https://httpbin.org
+
+paths:
+  /get:
+    get:
+      operationId: duplicateId
+      summary: Simple GET request to httpbin.org
+      responses:
+        '200':
+          description: Successful response from httpbin
+          content:
+            application/json:
+              schema:
+                type: object
+
+  /ip:
+    get:
+      operationId: duplicateId
+      summary: Get client IP address from httpbin
+      responses:
+        '200':
+          description: Successful response with IP
+          content:
+            application/json:
+              schema:
+                type: object

python/tests/unit/connectors/openapi_plugin/no-operationid-openapi.yaml

@@ -0,0 +1,17 @@
+openapi: 3.0.3
+info:
+  title: Simple HTTPBin API
+  version: 1.0.0
+servers:
+  - url: https://httpbin.org
+paths:
+  /get:
+    get:
+      summary: Simple GET request to httpbin.org
+      responses:
+        '200':
+          description: Successful response from httpbin
+          content:
+            application/json:
+              schema:
+                type: object

python/tests/unit/connectors/openapi_plugin/test_openapi_parser.py

@@ -178,3 +178,14 @@ def test_it_adds_security_metadata_to_operation(document_file_name, security_typ
                 if found:
                     break
             assert found, f"Security type '{security_type}' not found in operation '{operation.id}'"
+
+
+def test_no_operationid_raises_error():
+    """Test that OpenAPI spec without operationId raises PluginInitializationError."""
+    no_op_path = os.path.join(current_dir, "no-operationid-openapi.yaml")
+    with pytest.raises(PluginInitializationError, match="operationId missing"):
+        create_functions_from_openapi(
+            plugin_name="noOpTest",
+            openapi_document_path=no_op_path,
+            execution_settings=None,
+        )