-
Notifications
You must be signed in to change notification settings - Fork 635
fix(codex): preserve routed models in desktop picker #1056
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -53,8 +53,8 @@ ocx combo show main | |
| - `daily-fast` のように裸の場合もあれば、`team/daily-fast` のように `/` を 1 つ含む場合もあります。 | ||
| - `combo` にしたり、`combo/` で始めることはできません。 | ||
| - 別のコンボ エイリアスを複製することはできません。そして | ||
| - `gpt-`、`o1-`、`o3-`、`o4-`、またはで始まる裸のネイティブ OpenAI ファミリ名は使用できません。 | ||
| `codex-`。 | ||
| - 通常、`gpt-`、`o1-`、`o3-`、`o4-`、または `codex-` で始まる bare native OpenAI | ||
| family 名は使用できません。明示的な `nativeAlias: true` Desktop 互換モードだけが例外です。 | ||
|
|
||
| エイリアスが設定されている場合でも、正規の `combo/<id>` 形式は解決されます。正規ルックアップはエイリアスの一致の前に実行されるため、エイリアスが別のコンボの正規 ID を引き継ぐことはできません。 | ||
|
|
||
|
|
@@ -175,7 +175,10 @@ ocx combo set <id> --targets provider/model[:weight],... | |
| ocx combo remove <id> --yes | ||
| ``` | ||
|
|
||
| `set` は、`--strategy`、`--sticky`、`--effort`、`--alias`、および `--rename-from` も受け入れます。そのフィールドをクリアするには、`--effort` または `--alias` の値として `-` を使用します。 `create` および `update` は、`set` のエイリアスです。 `delete` は `remove` のエイリアスです。同じサブコマンドが `ocx route combo` で使用できます。 | ||
| `set` は、`--strategy`、`--sticky`、`--effort`、`--alias`、`--native-alias`、 | ||
| `--display-name`、および `--rename-from` も受け入れます。`--native-alias` には現在 | ||
| サポートされている bare native alias と空でない display name が必要です。`create` および `update` は `set` の | ||
| エイリアス、`delete` は `remove` のエイリアスです。 | ||
|
Comment on lines
+178
to
+181
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win Restore the missing CLI semantics in both localized guides. The English guide documents that
📍 Affects 2 files
🤖 Prompt for AI AgentsSource: Path instructions |
||
|
|
||
| ### 管理 API | ||
|
|
||
|
|
@@ -212,6 +215,8 @@ ocx combo remove <id> --yes | |
| | `stickyLimit` |いいえ | `1` |ラウンドロビン選択ごとの成功したリクエストの数は 1 ~ 100 の整数です。 | | ||
| | `defaultEffort` |いいえ | `null` | `low`、`medium`、`high`、`xhigh`、`max`、または `ultra`;呼び出し元が努力を省略し、ターゲットがサポートをアドバタイズした場合にのみ適用されます。 | | ||
| | `alias` |いいえ |なし |オプションのトリミングされたパブリック モデル ID。上記のエイリアス ルールを使用します。空の値はエイリアスなしで保存されます。 | | ||
| | `nativeAlias` |いいえ | `false` | 現在サポートされている bare native alias に routing/catalog の優先権を明示的に与えます。 | | ||
| | `displayName` |いいえ |なし | catalog 表示専用ラベル。`nativeAlias` が true の場合は必須です。 | | ||
|
|
||
| ## トラブルシューティング | ||
|
|
||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
Clarify which native switches remain in the dashboard.
Line 119 says that the dashboard keeps static switches.
src/codex/catalog/metadata.tslines 196-203 remove native-alias slugs fromnativeModelRows. The switch for a shadowed bare native slug is not available.State that only unshadowed native rows remain switchable. This prevents users from looking for a switch that the Models page does not render.
🤖 Prompt for AI Agents
Source: Path instructions