feat(converter): extract HttpStatus enum response codes from constructor pattern#72
Open
abdul-levo wants to merge 3 commits into
Open
Conversation
…esponse key
- Add _properties_from_getters() to build request body schema from
argToCalls getter methods when the DTO is absent from userDefinedTypes
- Add _extract_response_dto_key() to derive the response object key from
the actual response DTO class name instead of hard-coding 'description'
- Add _is_custom_dto() to distinguish application DTOs from stdlib types
- Add ±1 line tolerance in _infer_java_response_codes() to handle atom
recording slices at the annotation line vs the method signature line
- Fix OpenAPI title format: '{name} OpenAPI Specification'
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR consumes those entries in the converter to emit all actual response
codes per endpoint instead of only the HTTP method default (200/201/204).
Example — /invite/validate now shows 200, 400, 401, 403 instead of just 200.
Changes:
Note: Until AppThreat/atom#244 is merged, this change has no effect on
existing slice files