Skip to content

feat(openapi): add compute_field support#8188

Merged
leandrodamascena merged 2 commits intodevelopfrom
feat/8169
May 1, 2026
Merged

feat(openapi): add compute_field support#8188
leandrodamascena merged 2 commits intodevelopfrom
feat/8169

Conversation

@leandrodamascena
Copy link
Copy Markdown
Contributor

Issue number: closes #8169

Summary

Pydantic's @computed_field properties were excluded from generated OpenAPI schemas because Powertools always used mode="validation" when generating JSON schemas via Pydantic.

Changes

Please provide a summary of what's being changed

User experience

Please share what the user experience looks like before and after this change


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

@leandrodamascena leandrodamascena requested a review from a team as a code owner May 1, 2026 15:32
@leandrodamascena leandrodamascena requested a review from hjgraca May 1, 2026 15:32
@powertools-for-aws-oss-automation powertools-for-aws-oss-automation Bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label May 1, 2026
@leandrodamascena leandrodamascena requested a review from svozza May 1, 2026 15:37
@codecov
Copy link
Copy Markdown

codecov Bot commented May 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.72%. Comparing base (08c9921) to head (94c202e).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #8188   +/-   ##
========================================
  Coverage    96.72%   96.72%           
========================================
  Files          286      286           
  Lines        14341    14341           
  Branches      1200     1200           
========================================
  Hits         13871    13871           
  Misses         341      341           
  Partials       129      129           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@powertools-for-aws-oss-automation powertools-for-aws-oss-automation Bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels May 1, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 1, 2026

@leandrodamascena leandrodamascena merged commit 10ffef1 into develop May 1, 2026
17 checks passed
@leandrodamascena leandrodamascena deleted the feat/8169 branch May 1, 2026 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

event_handlers size/M Denotes a PR that changes 30-99 lines, ignoring generated files. tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: @computed_field properties are missing in generated OpenAPI schemas

2 participants