chore(sync/openapi): align flag types with OpenFeature spec (add float)
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 45/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Ferma
- Stack tecnologico
- openapi
- Ambito
- api
Direzione di ricerca
Inizia con le definizioni referenziate in api/v0/sync.yaml, in particolare FlagDefaultValue.oneOf e ManifestFlag.type.enum, e confrontale con schema/v0/flag-manifest.json e con il tipo number di OpenFeature. Il lavoro è completato quando la definizione OpenAPI di sync rappresenta i flag in virgola mobile in modo coerente in entrambe le posizioni e non è più in conflitto con lo schema del manifest esistente.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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. 👍
- Lingua principale
- Go
- Stelle
- 63
- Fork
- 22
- Merge medio
- 1h 27m
- PR unite (30g)
- 19
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di open-feature/cli
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 70/100
open-feature/cli#239 · 1 commento · 1 reazione ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
open-feature/cli#247 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
open-feature/cli#210 · 1 commento ·
-
open-feature/cli#197 · 3 commenti · 1 reazione · 1 assegnatario ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
open-feature/cli#186 · 1 reazione ·
Tutte le issue di open-feature/cli
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
bazel-contrib/rules_go#4726 · 1 commento ·
-
area/auto-scaling area/monitoring area/ops-productivity kind/enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
Type/Improvement
Difficoltà 1/5 1-3 ore Idoneità per principianti 90/100
OpenNSW/nsw-srilanka#522 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 92/100