Skip to content

Conversation

witemple-msft
Copy link
Member

This work-in-progress PR is an attempt to fix returnTypeVisibility/parameterVisibility by changing the HTTP library to use VisibilityFilter instances internally instead of Visibility flags. The public API is extended to accept VisibilityFilter everywhere that Visibility was required, and the API converts from Visibility to VisbilityFilter more eagerly.

This (provisionally, for the purposes of being able to easily identify where the flags type is used -- we may or may not decide to go forward with this) deprecates Visibility and the old signature for getVisibilitySuffix in favor of APIs that consume VisibilityFilter instances directly.

This hasn't quite fixed the issues with returnTypeVisibility but it's a step in the right direction.

Copy link

pkg-pr-new bot commented Aug 15, 2025

Open in StackBlitz

npm i https://pkg.pr.new/microsoft/typespec/@typespec/http@8204

commit: 2af9675

Copy link
Contributor

❌ There is undocummented changes. Run chronus add to add a changeset or click here.

The following packages have changes but are not documented.

  • @typespec/http
Show changes

@azure-sdk
Copy link
Collaborator

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants