Duplicate `operationId` in openapi.yaml

[apiarya]

openapi.yaml posted here are invalid and failing validation. Among the many issues, there is one specific error that operationIds are not unique.

e.g. operationId: drives.drive.items.driveItem.workbook.worksheets.workbookWorksheet.range.cell is used multiple times in above openapi.yaml file at line number 164872 and 167647.

grep "drives.drive.items.driveItem.workbook.worksheets.workbookWorksheet.range.cell" graph/openapi.yaml -n
164872:      operationId: drives.drive.items.driveItem.workbook.worksheets.workbookWorksheet.range.cell
167647:      operationId: drives.drive.items.driveItem.workbook.worksheets.workbookWorksheet.range.cell

Steps to reproduce

Use any openapi validator.

Expected result

An operationId needs to be unique, there can't be any duplicates in the document, you can't re-use them. Make sure the ID used for this operation is unique.

[andrueastman]

Thanks for raising this @apiarya

Any chance you can double check this from your end? From a search from the head of the master branch using the operation id you shared there exists no duplicates. Rather these two exist

• drives.drive.items.driveItem.workbook.worksheets.workbookWorksheet.range.cell-bc5e
• drives.drive.items.driveItem.workbook.worksheets.workbookWorksheet.range.cell-b0fa

We also do currently perform operation id generation on the openApi files using the redocly validator as part of the CI and we currently do not have reports of duplicate operation ids.

https://github.com/microsoftgraph/msgraph-metadata/blob/46a86c3aae83835b1c1599b89f41c79bf76bc47b/.github/workflows/metadata-parser-validation.yml#L45

[apiarya]

Yes, I don't see duplication in the latest yaml files anymore. But it was a problem. @erwinkramer too reported it here - microsoftgraph/msgraph-metadata#691

[andrueastman]

Thanks for confirming!
There was indeed an issue with the duplicate operation ids. But we have since resolved them in the following PRs

• Fix duplicate operation ids in overloads that are composable. #645
• Fixes potential duplicate operation ids of navigation property paths with overloaded composable functions #597

We'll close this one for now.