Revert @identifiers decorator and deprecate x-ms-identifiers generation

[qiaozha]

Summary

This PR reverts the changes introduced in PR #1983 which added the @Identifiers decorator and automatic x-ms-identifiers generation from @key decorator.

Changes

• ✅ Removed @Identifiers decorator from typespec-azure-resource-manager
• ✅ Removed automatic x-ms-identifiers generation from @key decorator
• ✅ Removed x-ms-identifiers generation logic from typespec-autorest emitter
• ✅ Reverted to using @OpenAPI.extension("x-ms-identifiers", ...) for explicit specification
• ✅ Removed related tests for @Identifiers decorator
• ✅ Updated documentation to reflect the changes
• ✅ Added deprecation changelog entry

Related Issues

Reverts: #1983

Breaking Changes

• The @Identifiers decorator is no longer available
• x-ms-identifiers is no longer automatically generated from @key decorators
• Users must use @OpenAPI.extension("x-ms-identifiers", ...) to specify identifiers explicitly when needed

[azure-sdk]

All changed packages have been documented.

• ✅ @azure-tools/typespec-autorest
• ✅ @azure-tools/typespec-azure-resource-manager

Show changes

@azure-tools/typespec-autorest - deprecation ✏️

Deprecated automatic x-ms-identifiers generation from @key decorator and removed the @Identifiers decorator. Use @OpenAPI.extension("x-ms-identifiers", ...) to specify identifiers explicitly when needed.

@azure-tools/typespec-azure-resource-manager - deprecation ✏️

Deprecated automatic x-ms-identifiers generation from @key decorator and removed the @Identifiers decorator. Use @OpenAPI.extension("x-ms-identifiers", ...) to specify identifiers explicitly when needed.

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/Azure/typespec-azure/@azure-tools/typespec-autorest@3779

npm i https://pkg.pr.new/Azure/typespec-azure/@azure-tools/typespec-azure-core@3779

npm i https://pkg.pr.new/Azure/typespec-azure/@azure-tools/typespec-azure-resource-manager@3779

commit: 7f940fc

[azure-sdk]

You can try these changes here

🛝 Playground	🌐 Website

[timotheeguerin]

@qiaozha before this happens we need full confirmation from ARM team that this is uncesssary and this change cannot be done in such a breaking manner. It needs to be rolled out following the process we've established.

1. deprecate only
2. migrate everything in spec repo
3. remove 1-2 sprint later

[microsoft-github-policy-service]

Hi @@qiaozha. Your PR has had no update for 30 days and it is marked as a stale PR. If it is not updated within 30 days, the PR will automatically be closed. If you want to refresh the PR, please remove the stale label.