Skip to content

Conversation

@jwj019
Copy link

@jwj019 jwj019 commented Oct 7, 2025

This PR updates our fork to the latest release from https://github.com/OpenAPITools/openapi-generator

A paired PR for updating ngx-monarch-api can be found at https://github.com/niaid/ngx-monarch-api/pull/56

btpnlsl and others added 30 commits July 10, 2025 16:41
OpenAPITools#21536)

* [req] Allow models and apis list properties to span multi-lines and include white space (OpenAPITools#19628)

* Add comment for DefaultGenerator.getPropertyAsSet

* Fix some variable names in DefaultGeneratorTest.testGenerateMultiLinePropertiesIssue19628

---------

Co-authored-by: Chris Gual <[email protected]>
…script generator. (OpenAPITools#21560)

* Add optional erasable syntax configuration.

* Changes from generation clean-up.

---------

Co-authored-by: Brendan Burns <[email protected]>
…/" (OpenAPITools#21544)

* Fix image upload for content-type image

* update samples

* add test case in spec and update samples

* update bitrise stack so we get xcpretty 0.4.1

* Revert "add test case in spec and update samples"

This reverts commit 92202dd.
* feat(typescript-angular): add Angular 20 support

* chore: remove unused .openapi-generator-ignore file
* Oat++ Server Generator (C++)

* Fixed for support for newest OpenAPI version.

* ALPHA not STABLE.

* Fixed for support for newest OpenAPI version.

* Added github workflow & changed to OA3 Petstore.

* Good catch on adding the Workflow.

* Might help to update the samples.

* Set C++ Standard the CMake way.

* Would be easier if there was a .pc file.

* oatpp.lib.

* Add ws2.

* This probably doesn't work, need to take a time out.
…ng APIKeyQuery, APIKeyHeader and APIKeyCookie OpenAPITools#13474 (OpenAPITools#21551)

* fix: using APIKeyQuery and APIKeyHeader generates invalid code (OpenAPITools#13474)

* fix: include test

* fix: also fix auth generation for sttp4

* fix: maintain prev whitespaces & update generated APIs
* python: reinstate lazy imports

* python: Update samples

---------

Co-authored-by: Pascal Bachor <[email protected]>
…ionMapper (OpenAPITools#20762)

* [java][Microprofile] add config options to disable usage of ApiExceptionMapper

* Update modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/JavaClientCodegen.java

Co-authored-by: martin-mfg <[email protected]>

---------

Co-authored-by: martin-mfg <[email protected]>
…OpenAPITools#20495) (OpenAPITools#21355)

* [dart-dio] Fix compile error in enum properties with "default" values(OpenAPITools#20495)

* Add a test schema with enum properties
…OpenAPITools#21568)

* set resolveResponses=true

needed by swagger-parser>=2.1.23, see swagger-api/swagger-parser#2127

* update samples

---------

Co-authored-by: James Shaw <[email protected]>
…OpenAPITools#21598)

* [java-spring] provide a clarifying comment for the generated EnumConverterConfiguration class

* update samples

---------

Co-authored-by: Chris Gual <[email protected]>
…APITools#21564 (OpenAPITools#21579)

* [kotlin-spring] add a Spring type converter for enum values OpenAPITools#21564

* [kotlin-spring] simplify unit test for inner enum converter

* update samples

* code review feedback; move containsEnums to ModelUtils

* code review feedback; provide comment for generated EnumConverterConfiguration.kt

* update samples

---------

Co-authored-by: Chris Gual <[email protected]>
…ng. Issue OpenAPITools#21259 (OpenAPITools#21464)

* [typescript-fetch] number, string, and Date now considered in oneOf models. Issue OpenAPITools#21259

* Generated samples
wing328 and others added 25 commits September 21, 2025 16:27
…ls#22005)

* refactor scala client server workflow for testing

* fix

* migrate circleci scala tests

* add service, update host table

* java 8

* add distribution

* java 17

* 2.13.x

* update

* clean up circleci config
…APITools#20919)

* fix(java): x-discriminator-value should not produce @JsonTypeName

* fix(java): Remove unused getDiscriminatorValue()

* build at Fednot

* build at Fednot

* build at Fednot: skip sonar

* scm for release at fednot

* build fednot not 7.5.0-FEDNOT-SNAPSHOT

* build fednot not 7.5.0-FEDNOT-SNAPSHOT

* test all vars

* rollback custom pom.xml

* commit master

* commit test

* Samples for deduction

* add files  for deduction

* small improvements

* Merge changes from martin-mfg
Merge master

* Merge changes from martin-mfg
Merge master

* Merge changes from martin-mfg
Add comment to force rebuild

* Merge master

* regenerate doc

* regenerate client

* regenerate client

---------

Co-authored-by: Jean-Paul Finne <[email protected]>
Co-authored-by: martin-mfg <[email protected]>
…#22008)

* update ruby faraday to use ruby fake petstore test spec

* clean up, regenerate samples

* update test
* Only register the Swagger annotation imports for the corresponding annotationLibrary

* Fix the issue in a specific generator

* Add unit test
* [cxf] add support for Swagger2

Closes OpenAPITools#3668

* regen samples

* regen docs

* unconditionally include parameter imports

* Add swagger2 update samples

---------

Co-authored-by: Jens Kleine-Herzbruch <[email protected]>
* add test for jaxrs cxf swagger2 client

* revert
…son (OpenAPITools#21371)

* replace deprecated 'typings' field with standard 'types' in package.json

* regenerate samples after replacing 'typings' with 'types' field

---------

Co-authored-by: winiga <[email protected]>
Previous mustache template was using #isBodyParam outside of #operation
context, so it was not effective.

Even if we'd add the proper context, we'd then risk generating duplicate
imports for multiple matching parameters.

For this reason, this patch implements detection of an optional body
parameter in code, making sure the corresponding import is added just
once.

Fixes OpenAPITools#19237
* add tests with optional body for go-server

* update workfllow
* [python] fix OpenAPITools#19255 add async httpx support

* update docs

* 1. "async" parameter for templates
2. hand written tests for python-httpx
3. CI workflow updated

* fix mypy
…ension` (OpenAPITools#22032)

Separate models are located in `modelPackage` folder with `index.ts` file. ESM requires an index file with the provided extension when `importFileExtension` is defined.
* Support AnyOf, AllOf

* Update

* Fix

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Update

* Update
…odels (OpenAPITools#22029)

* fix(kotlin-spring): add missing constructor parentheses for hashmap models

This commit fixes a bug in the kotlin-spring generator where models
defined with additionalProperties would result in uncompilable code.
The generated data class was missing the constructor invocation '()'
when inheriting from a map type.

This has been corrected to only add parentheses when the parent is a map.

The existing samples have been regenerated to reflect this change.

* Trigger CI
@jwj019 jwj019 requested a review from ccampanale October 7, 2025 21:52
Copy link

@ccampanale ccampanale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's a lot here but assuming things were merged in without breaking the custom changes Orion made, we should be good. Testing downstream is really important though.

Do we have a tag or something we can easily point to for a rebuild if needed? Maybe just make sure something like that is possible before merging. But otherwise, LGTM!

Really appreciate you getting at this too!

@jwj019
Copy link
Author

jwj019 commented Oct 10, 2025

I created a tag at https://github.com/niaid/openapi-generator/releases/tag/v7.0.0. This points to the last change that Orion made before he left so we should be able to revert and/or build this tag if we need to go back

@jwj019 jwj019 merged commit 31767c0 into master Oct 30, 2025
590 of 608 checks passed
@jwj019 jwj019 deleted the sync-upstream-v7.16.0 branch October 30, 2025 16:10
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.