OpenAPITools / OpenAPITools/openapi-generator
[BUG][typescript-axios] Using allOf with a $ref and an object, along with `withSeparateModelsAndApi` generates broken imports
Open
Nobody has claimed this yet.
Client: TypeScript
Issue: Bug
- Dominant language
- Java
- Stars
- 26.8k
- Forks
- 7.7k
- PR merge metrics
- PR metrics pending
Description
Bug Report Checklist
- Have you provided a full/minimal spec to reproduce the issue?
- Have you validated the input using an OpenAPI validator (example)?
- Have you tested with the latest master to confirm the issue still exists?
- Have you searched for related issues/PRs?
- What's the actual output vs expected output?
- [Optional] Sponsorship to speed up the bug fix or feature request (example)
Description
Using allOf with a $ref and an object, along with withSeparateModelsAndApi generates broken imports
openapi-generator version
5.1.0
OpenAPI declaration file content or url
openapi: 3.0.2
info:
title: hello
version: 1.0.0
components:
schemas:
SomeSchema:
type: object
properties:
some_prop:
type: integer
paths:
/foobar:
get:
responses:
200:
description: ok
content:
application/json:
schema:
allOf:
- $ref: '#/components/schemas/SomeSchema'
- type: object
properties:
foobar:
type: string
Generation Details
openapi-generator-cli generate -i http://localhost:8000/swagger.yml -g typescript-axios -o foo -p "withSeparateModelsAndApi=true,modelPackage=models,apiPackage=api"
Steps to reproduce
- Run the above command with the provided yaml file
- Look at the generated
default-api.ts
On line 24, it creates an invalid import statement: import { SomeSchema & object } from '../models';
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Reproduce the issue with the supplied OpenAPI YAML and the typescript-axios generation command using withSeparateModelsAndApi. Inspect the generated default-api.ts import on line 24 and compare it with valid TypeScript syntax; done means the generated client has valid model imports and builds successfully.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- api, tooling
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100