Skip to content

Add @client on interface AddOptionalParam in resiliency srv-driven specs#4058

Open
Copilot wants to merge 2 commits intomainfrom
copilot/add-client-to-interface
Open

Add @client on interface AddOptionalParam in resiliency srv-driven specs#4058
Copilot wants to merge 2 commits intomainfrom
copilot/add-client-to-interface

Conversation

Copy link
Contributor

Copilot AI commented Mar 17, 2026

The AddOptionalParam interface in the srv-driven resiliency specs was missing the @client decorator, preventing it from being recognized as an operation group client.

Changes

  • specs/resiliency/srv-driven/main.tsp — added @client on interface AddOptionalParam
  • specs/resiliency/srv-driven/old.tsp — added @client on interface AddOptionalParam
@route("/add-optional-param")
@client
interface AddOptionalParam { ... }

Copilot AI and others added 2 commits March 17, 2026 11:04
…ngelog

Co-authored-by: weidongxu-microsoft <53292327+weidongxu-microsoft@users.noreply.github.com>
@microsoft-github-policy-service microsoft-github-policy-service bot added the lib:azure-http-specs For issues/prs related to the @azure-tools/typespec-azure-http-specs package label Mar 17, 2026
@weidongxu-microsoft
Copy link
Member

weidongxu-microsoft commented Mar 17, 2026

Generated code would be different (the op in interface would now be in a subclient).

Thought about keep generated code same, but that would change the scenario name
#4057

Remove the @client on namespace also work, but that changes the root client name. Hence still keep that.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 17, 2026

Open in StackBlitz

npm i https://pkg.pr.new/@azure-tools/azure-http-specs@4058

commit: dd640bd

@azure-sdk
Copy link
Collaborator

All changed packages have been documented.

  • @azure-tools/azure-http-specs
Show changes

@azure-tools/azure-http-specs - fix ✏️

Add @client on interface AddOptionalParam in resiliency srv-driven specs.

@azure-sdk
Copy link
Collaborator

You can try these changes here

🛝 Playground 🌐 Website

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lib:azure-http-specs For issues/prs related to the @azure-tools/typespec-azure-http-specs package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants