False positives when readOnly property using referenced enum

Abierto

#286 abierto el 17/11/2023 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad: 4/5
Tiempo estimado: 3-5 días
Aptitud para principiantes: 42/100

Tipo de issue: Error
Claridad: Bastante claro
Estado de actividad: Estancado
Stack tecnológico: csharp, openapi
Área: cli

Línea de trabajo

Comienza inspeccionando la comparación descrita en Azure/azure-rest-api-specs-pr#15631, usando el modelo Trial y el enum TrialStatus referenciado como caso de reproducción. Traza dónde el diff compara el esquema enum inline anterior con el nuevo esquema $ref y el valor readOnly. La tarea estará terminada cuando este cambio equivalente ya no se notifique como un cambio breaking de readOnly.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Describe the bug

I'm working on https://github.com/Azure/azure-rest-api-specs-pr/pull/15631 and there are a lot of false positives with a message of `The read only property has changed from 'true' to 'false'.

To Reproduce
As an example from that PR:

@doc("Subscription trial availability")
model Trial {
  @doc("Trial status")
  @visibility("read")
  status?: TrialStatus;

It does produce readOnly.

        "status": {
          "$ref": "#/definitions/TrialStatus",
          "description": "Trial status",
          "readOnly": true
        },

But before it was not a $ref, it was inline:

        "status": {
          "description": "Trial status",
          "type": "string",
          "enum": [
            "TrialAvailable",
            "TrialUsed",
            "TrialDisabled"
          ],
          "readOnly": true,
          "x-ms-enum": {
            "name": "TrialStatus",
            "modelAsString": true
          }
        },

Expected behavior
It is not marked as a breaking change.

Lenguaje dominante: C#
Estrellas: 290
Forks: 50
Merge medio: 2 d 21 h
PR fusionados (30 d): 10

Guía de contribución

Abrir la guía de contribución

Primeros pasos

Lee el issue completo y luego la guía de contribución del proyecto.
Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
Haz un fork del repositorio y trabaja en una rama.
Abre un pull request que haga referencia al número del issue.

Más de Azure/openapi-diff

Crash if inherited property is structurally equal but not reference identical Abierto

Azure/openapi-diff#482 · 1 asignado · Abierto el 5/5/2026
Crash if duplicate model names (across files) used in allOf Abierto

Azure/openapi-diff#479 · 2 comentarios · 1 asignado · Abierto el 5/5/2026
Remove dependency `autorest` Abierto

Azure/openapi-diff#456 · 1 asignado · Abierto el 3/3/2026
Error: Invalid reference token: /Tables(\'{tableName}\') Abierto

Dificultad 4/5 3-5 días Aptitud para principiantes 35/100

Azure/openapi-diff#445 · Abierto el 27/1/2026
OpenAPI-diff should flag changes in the tags for an operation Abierto

Azure/openapi-diff#371 · 7 comentarios · 1 asignado · Abierto el 1/9/2025

Todos los issues de Azure/openapi-diff

Issues similares

Sticky banners hide under the fixed mobile header while scrolling Abierto

bug

Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100

nightscout/nocturne#1425 · Abierto el 23/9/2026
Loading of namespaces states "no records found" Abierto

enhancement

Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100

miracum/vfps#357 · Abierto el 23/9/2026
Login Screen always ask for password Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100

jellyfin/jellyfin#18175 · Abierto el 23/9/2026
Add Cake DSL examples to BuildSystem and CI provider APIs Abierto

Documentation

Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100

cake-build/cake#5024 · Abierto el 23/9/2026
Gå gjennom ESLint-suppressions Abierto

Frontend status/draft TechnicalDebt

Dificultad 2/5 1-2 días Aptitud para principiantes 75/100

Altinn/altinn-auth#4143 · Abierto el 23/9/2026

Más issues de C#