Crash if inherited property is structurally equal but not reference identical
Mantenedores costumam responder em até 1 dia
@mikeharder já está trabalhando nisso.
Desde 6/5/2026.
Avaliação
Esta issue ainda não foi avaliada.
Descrição
If a definition defines a property, and inherits from another definition that also defines the property:
- If the properties are reference identical, it's allowed
- If the properties are equivalent (but not reference identical), the tool crashes
Organic PRs hitting problem
Test Cases
Pass - Reference Identical
{
"swagger": "2.0",
"info": {
"title": "structural-equality",
"version": "1.0"
},
"paths": {},
"definitions": {
"Foo": {
"type": "object",
"properties": {
"bar": {
"$ref": "#/definitions/MyObject"
}
},
"allOf": [
{
"$ref": "#/definitions/Foo2"
}
]
},
"Foo2": {
"type": "object",
"properties": {
"bar": {
"$ref": "#/definitions/MyObject"
}
}
},
"MyObject": {
"type": "object",
"additionalProperties": {
"type": "string"
}
}
}
}
Fail - Structurally Equal
{
"swagger": "2.0",
"info": {
"title": "structural-equality",
"version": "1.0"
},
"paths": {},
"definitions": {
"Foo": {
"type": "object",
"properties": {
"bar": {
"type": "object",
"additionalProperties": {
"type": "string"
}
}
},
"allOf": [
{
"$ref": "#/definitions/Foo2"
}
]
},
"Foo2": {
"type": "object",
"properties": {
"bar": {
"type": "object",
"additionalProperties": {
"type": "string"
}
}
}
}
}
}
Error
incompatible properties : bar
definitions/Foo/properties/bar
at file:///home/mharder/openapi-diff-mh/incompat-prop/src/test/specs/incompatible-properties/structural-equality-fail.json#L12:8
definitions/Foo2/properties/bar
at file:///home/mharder/openapi-diff-mh/incompat-prop/src/test/specs/incompatible-properties/structural-equality-fail.json#L28:8
- Linguagem predominante
- C#
- Estrelas
- 290
- Forks
- 50
- Merge médio
- 7h 8min
- PRs com merge (30d)
- 11
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de Azure/openapi-diff
-
Crash if duplicate model names (across files) used in allOfTalvez livre de novo @mikeharder assumiu há 159 dias e não há nenhum pull request aberto. Aberta
Azure/openapi-diff#479 · 2 comentários · 1 responsável ·
Mantenedores costumam responder em até 1 dia
-
Remove dependency `autorest`Talvez livre de novo @mikeharder assumiu há 222 dias e não há nenhum pull request aberto. Aberta
Azure/openapi-diff#456 · 1 responsável ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 35/100
Azure/openapi-diff#445 ·
Mantenedores costumam responder em até 1 dia
-
OpenAPI-diff should flag changes in the tags for an operationTalvez livre de novo @mikeharder assumiu há 405 dias e não há nenhum pull request aberto. Aberta
Azure/openapi-diff#371 · 7 comentários · 1 responsável ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 25/100
Azure/openapi-diff#364 · 4 comentários ·
Mantenedores costumam responder em até 1 dia
Todas as issues de Azure/openapi-diff
Issues semelhantes
-
Bug: Hidden loading rings keep the compositor running, so Files uses 7-94% of a CPU core while idleAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
files-community/Files#19026 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
BHoM/Civil3D_Toolkit#118 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
rjmurillo/moq.analyzers#1468 ·
Mantenedores costumam responder em até 1 dia
-
`:empty-columns` is not updated when a column's `IsVisible` changesTalvez já em andamento Um pull request vinculado a esta issue está aberto ou já foi mesclado. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 83/100
-
test(e2e): the signed-in CSP spec times out behind the match-setup dialog and asserts nothingAbertaarea:frontend FE P2
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
klasolsson81/jobbliggaren#2139 ·
Mantenedores costumam responder em até 1 dia