Skip to content

feat: improve model-aware parameter filtering - #3493

Merged
jterapin merged 1 commit into
version-3from
feat/model-aware-parameter-filtering
Nov 6, 2026
Merged

jterapin merged 1 commit into
version-3from
feat/model-aware-parameter-filtering

Conversation

@jterapin

@jterapin jterapin commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Context

SDK parameter filtering uses generated sensitivity metadata when formatting request logs and structures.

Description

  • Carry the operation's input model through nested structure, list, and map filtering.
  • Follow nested collection references when generating Ruby and RBS sensitivity metadata.
  • Preserve the existing parameter-name fallback and custom filtering options.

Service types need regeneration to pick up the collection metadata changes.

Testing

  • Core specs: 2,509 examples, 0 failures, 36 pending.
  • Generator specs: 2,766 examples, 0 failures, 374 pending.

🤖 This PR description was drafted with AI assistance for @jterapin. ✨

@jterapin
jterapin requested review from a team as code owners November 6, 2026 16:09
@jterapin
jterapin requested review from boblodgett, normj, richardwang1124 and sichanyoo and removed request for normj and sichanyoo November 6, 2026 16:09
@jterapin

jterapin commented Nov 6, 2026

Copy link
Copy Markdown
Contributor Author

The Build SDK dry run passed on the current PR revision. All 34 PR checks are also passing.

Ready for fresh approvals so we can merge for today's release.

🤖 This comment was drafted with AI assistance and reviewed by @jterapin. ✨

@jterapin
jterapin merged commit 7d54fc5 into version-3 Nov 6, 2026
36 of 37 checks passed
@jterapin
jterapin deleted the feat/model-aware-parameter-filtering branch November 6, 2026 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants