Skip to content

Python: feat: support path item parameters and non-method keys in OpenAPI parser - #14588

Open
Aagam Shah (AAGAM17) wants to merge 1 commit into
microsoft:mainfrom
AAGAM17:feat/openapi-path-item-parameters
Open

Aagam Shah (AAGAM17) wants to merge 1 commit into
microsoft:mainfrom
AAGAM17:feat/openapi-path-item-parameters

Conversation

@AAGAM17

Copy link
Copy Markdown

Fixes #14581. OpenApiParser.create_rest_api_operations treated every key of a path item as an HTTP method, so summary, description, servers or parameters on a path item made plugin loading fail with 'operationId missing'. Only HTTP method keys are now treated as operations, and path-level parameters are merged into each operation (an operation-level parameter with the same name and location overrides the shared one). Added tests for both.

@AAGAM17
Aagam Shah (AAGAM17) requested a review from a team as a code owner October 13, 2026 18:40
Copilot AI balanced review requested due to automatic review settings October 13, 2026 18:40

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@semantic-kernel-automation semantic-kernel-automation Bot added the python Pull requests for the Python Semantic Kernel label Oct 13, 2026
@github-actions github-actions Bot changed the title feat: support path item parameters and non-method keys in OpenAPI parser Python: feat: support path item parameters and non-method keys in OpenAPI parser Oct 13, 2026
@AAGAM17

Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree

This branch was successfully deployed

1 active deployment
github-app-auth — b05c7205 Deployed Oct 13, 2026 by AAGAM17 via team_check #616
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

python Pull requests for the Python Semantic Kernel

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Python: Bug: OpenAPI plugin load fails with 'operationId missing' when a path item has parameters, summary or description

2 participants