Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: apple/swift-openapi-generator
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 1.10.2
Choose a base ref
...
head repository: apple/swift-openapi-generator
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 1.10.3
Choose a head ref
  • 5 commits
  • 8 files changed
  • 5 contributors

Commits on Aug 4, 2025

  1. Update generator version in manual cli example (#802)

    ### Motivation
    
    This solves the missing import issue described in
    #755
    
    ### Modifications
    
    Update the version of generator in`Makefile`
    Updated runtime versions to latest released version
    
    ### Result
    
    Solved the `is not available due to missing import of defining module
    'Foundation'` issue
    
    ```text
     13 | //===----------------------------------------------------------------------===//
     14 | import OpenAPIKit
     15 |
        | `- note: add import of module 'Foundation'
     16 | /// Represents a server variable and the function of generation that should be applied.
     17 | protocol ServerVariableGenerator {
        :
    166 |             self.key = key
    167 |             swiftSafeKey = context.safeNameGenerator.swiftMemberName(for: key)
    168 |             enumName = context.safeNameGenerator.swiftTypeName(for: key.localizedCapitalized)
        |                                                                         `- error: property 'localizedCapitalized' is not available due to missing import of defining module 'Foundation'
    169 |             self.variable = variable
    170 |             self.enumValues = enumValues
    /Volumes/w/src/github.com/apple/swift-openapi-generator/Examples/manual-generation-generator-cli-example/.swift-openapi-generator/Sources/_OpenAPIGeneratorCore/Translator/TypesTranslator/translateServersVariables.swift:168:73: error: property 'localizedCapitalized' is not available due to missing import of defining module 'Foundation'
     13 | //===----------------------------------------------------------------------===//
     14 | import OpenAPIKit
    ```
    
    ### Test Plan
    
    Passed locally, `make generate` and `swift run`.
    at15 authored Aug 4, 2025
    Configuration menu
    Copy the full SHA
    340ee95 View commit details
    Browse the repository at this point in the history

Commits on Sep 16, 2025

  1. Fix PR CI - formatting (#824)

    At some point recently (I only noticed it in
    https://github.com/apple/swift-openapi-generator/actions/runs/17734822822/job/50426205189?pr=816)
    we started getting failures in PR CI due to formatting issues in a test
    and in example projects.
    
    This PR fixes them up.
    czechboy0 authored Sep 16, 2025
    Configuration menu
    Copy the full SHA
    0e6faec View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    0437e7e View commit details
    Browse the repository at this point in the history
  3. Fix typos (#820)

    ### Motivation
    
    Fix typos in the doc.
    ebariaux authored Sep 16, 2025
    Configuration menu
    Copy the full SHA
    3a95e87 View commit details
    Browse the repository at this point in the history

Commits on Sep 17, 2025

  1. Allow recursive schemas in multipart-based paths (#813)

    ### Motivation
    
    Fixes #769
    Currently, as seen in the issue mentioned above, if a recursive schema
    gets referenced from a multipart-based path, a reference cycle error
    gets thrown. Since we support recursive type generation already, this
    shouldn't happen.
    
    ### Modifications
    
    This PR removes the last `dereferenced(in:)` method call, which would
    throw if a reference cycle was found, replacing it with a simple lookup
    in the OpenAPI components tree. It also replaces
    `DerereferencedJSONSchema` with `JSONSchema` accordingly.
    
    ### Result
    
    Multipart-based paths can now reference recursive types.
    
    ### Test Plan
    
    This also adds a test with such a case. We could probably also add a
    test with a multipart path referencing an _array_ of references instead
    of just a reference to be thorough.
    ptoffy authored Sep 17, 2025
    Configuration menu
    Copy the full SHA
    d74223c View commit details
    Browse the repository at this point in the history
Loading