chore(sync/openapi): align flag types with OpenFeature spec (add float)
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- openapi
- Área
- api
Línea de trabajo
Comienza con las definiciones referenciadas en api/v0/sync.yaml, especialmente FlagDefaultValue.oneOf y ManifestFlag.type.enum, y compáralas con schema/v0/flag-manifest.json y el tipo number de OpenFeature. Se considera terminado cuando la definición de OpenAPI de sync representa los flags de coma flotante de forma coherente en ambas ubicaciones y ya no entra en conflicto con el esquema de manifiesto existente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
While working with the sync OpenAPI specification in openfeature/cli, I noticed an inconsistency regarding numeric flag types, specifically missing support for floating-point values.
Current OpenAPI definition
Both:
FlagDefaultValue.oneOf
https://github.com/open-feature/cli/blob/144eb0f0e3b2c53c9ae99e1d671259306fe474d0/api/v0/sync.yaml#L24-L28ManifestFlag.type.enum
https://github.com/open-feature/cli/blob/144eb0f0e3b2c53c9ae99e1d671259306fe474d0/api/v0/sync.yaml#L47
are missing a floating-point / number representation.
Why this is incorrect per OpenFeature spec
https://openfeature.dev/specification/sections/flag-evaluation/#conditional-requirement-1331
https://openfeature.dev/specification/types/#number
This clearly establishes that floating-point values are a first-class concept in OpenFeature.
Existing manifest schema already supports floats : the current v0 flag manifest JSON Schema already allows float flags: flag-manifest.json#L37
This creates a mismatch where:
- JSON Schema ✅ allows float flags
- Sync OpenAPI spec ❌ does not
Impact
Because floating-point is missing from the OpenAPI spec:
- Code generation for provider servers becomes incorrect or incomplete
Suggested fix
Add number (or float) to:
FlagDefaultValue.oneOfManifestFlag.type.enum
I’m happy to provide a PR once the expected direction is agreed upon. 👍
- Lenguaje dominante
- Go
- Estrellas
- 63
- Forks
- 22
- Merge medio
- 1 h 10 min
- PR fusionados (30 d)
- 24
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Leer 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 open-feature/cli
-
Add mise as a recommended installation methodPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 1/5 1-3 horas Aptitud para principiantes 70/100
open-feature/cli#239 · 1 comentario · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
open-feature/cli#247 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
open-feature/cli#210 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Add support for installing openfeature CLI via HomebrewQuizá libre de nuevo @kriscoleman la tomó hace 205 días y no hay ningún pull request abierto. Abierto
open-feature/cli#197 · 3 comentarios · 1 reacción · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
open-feature/cli#186 · 1 reacción ·
Los mantenedores suelen responder en 1 día
Todos los issues de open-feature/cli
Issues similares
-
good first issue help wanted v3
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
oras-project/oras-go#1621 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
Los mantenedores suelen responder en 4 días
-
Chained slog With calls discard previously bound attributesPosiblemente ocupada @yuefdev la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
[scanner] TestAgentNowLineStates hard-fails instead of degrading when node is off PATHPosiblemente ocupada @arteeh la tomó hoy. Abiertoagent/scanner ci help wanted kind/test
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
hivecommons/hive#12924 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Gentleman-Programming/engram#1940 ·
Los mantenedores suelen responder en 1 día