fix: Exclude methods without HTTP bindings from OpenAPI definitions

[iamrajiv]

References to other Issues or PRs

Fixes: #5684

Have you read the Contributing Guidelines?

Brief description of what is fixed or changed

Other comments

[johanbrandhorst]

This looks good to me, but I'm reminded that we have a feature that automatically creates bindings without annotations (generate_unbound_methods), how does this interact with that setting? Can you try if this breaks that? Do we need to add an option for openAPIv2 as well that generates unbound methods?

[iamrajiv]

Hi @johanbrandhorst, thanks for the review. I tested the generate_unbound_methods flow and it works fine. OpenAPIv2 already had this flag. When enabled it auto-creates default POST bindings for methods without HTTP annotations (descriptor/services.go:46–52). When disabled those methods just have empty Bindings arrays. My len(meth.Bindings) == 0 check correctly handles that.