chore(sync/openapi): align flag types with OpenFeature spec (add float)
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 45/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 停滞
- 技術スタック
- openapi
- 領域
- api
調査の方向性
api/v0/sync.yaml 内の参照されている定義、特に FlagDefaultValue.oneOf と ManifestFlag.type.enum から始め、それらを schema/v0/flag-manifest.json および OpenFeature の number 型と比較します。sync の OpenAPI 定義が両方の箇所で浮動小数点フラグを一貫して表現し、既存のマニフェストスキーマと競合しなくなれば完了です。
索引モデルが issue の本文から書いたものです。
説明
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. 👍
- 主要言語
- Go
- スター
- 63
- フォーク
- 22
- 平均マージ
- 1時間 13分
- マージ済み PR(30日)
- 23
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
open-feature/cli のほかの issue
-
難易度 1/5 1〜3時間 初心者へのやさしさ 70/100
open-feature/cli#239 · コメント 1 件 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
open-feature/cli#247 ·
メンテナーはふだん 1 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
open-feature/cli#210 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Add support for installing openfeature CLI via Homebrew再び着手できるかも @kriscoleman が 189 日前に担当しましたが、オープン中のプルリクエストはありません。 オープン
open-feature/cli#197 · コメント 3 件 · リアクション 1 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
open-feature/cli#186 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
open-feature/cli の issue をすべて見る
似ている issue
-
cvss-severity:high devguard l3montree-cybersecurity/devguard/devguard pkg:golang/github.com/l3montree-dev/devguard risk:low state:open
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
l3montree-dev/devguard#3146 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
CLI
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
databricks/cli#6910 ·
メンテナーはふだん 1 日以内に返信
-
Table presenter appends a spurious ", ..." to the FIX column when all fix versions are already shownオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
canonical/microceph#900 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
vulncheck or vulndb
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
メンテナーはふだん 1 日以内に返信