smithy-lang / smithy

Smithy is a protocol-agnostic interface definition language and set of tools for generating clients, servers, and documentation for any programming language.
https://smithy.io
Apache License 2.0
1.7k stars 201 forks source link

Add operationContextParams to RulesetParameterValidator #2295

Closed alextwoods closed 4 weeks ago

alextwoods commented 1 month ago

Background

2266 added support for the new operationContextParams trait, but missed adding support for it to the RulesetParameterValidator. Rulesets that bound a parameter to the service model only through an operationContextParameter would trigger validation errors that Parameter "exists in ruleset but not in service model". In addition, type checking for inconsistent parameters and type mismatches between operationContextParams and the bound endpoint parameter were not being done.

Testing

*New and existing tests

Links


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