Skip to content

Commit c1b4e6b

Browse files
committed
fix: address pr review follow-ups
1 parent 000cf1f commit c1b4e6b

3 files changed

Lines changed: 14 additions & 8 deletions

File tree

docs/configuration.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,8 @@ controls how much thinking the model does.
4646
| `gpt-5.1-codex-mini` | medium, high |
4747
| `gpt-5.1` | none, low, medium, high |
4848

49-
the shipped config templates include 9 base model families and 34 shipped presets overall (34 modern variants or 34 legacy explicit entries). on OpenCode `v1.0.210+`, those 34 presets intentionally appear as 9 base model entries plus `--variant` values. `gpt-5.4-pro` ships in the templates but may still be entitlement-gated at runtime, while `gpt-5.3-codex-spark` remains a manual add-on for entitled workspaces only.
50-
for context sizing, shipped templates use:
49+
The shipped config templates include 9 base model families and 34 shipped presets overall (34 modern variants or 34 legacy explicit entries). On OpenCode `v1.0.210+`, those 34 presets intentionally appear as 9 base model entries plus `--variant` values. `gpt-5.4-pro` ships in the templates but may still be entitlement-gated at runtime, while `gpt-5.3-codex-spark` remains a manual add-on for entitled workspaces only.
50+
For context sizing, shipped templates use:
5151
- `gpt-5.4` and `gpt-5.4-pro`: `context=1050000`, `output=128000`
5252
- `gpt-5.4-mini`, `gpt-5.4-nano`, `gpt-5-codex`, `gpt-5.1-codex`, `gpt-5.1-codex-max`, and `gpt-5.1-codex-mini`: `context=400000`, `output=128000`
5353
- `gpt-5.1`: `context=272000`, `output=128000`

docs/development/CONFIG_FLOW.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,7 @@ It currently ships:
7070
- 34 total variants
7171
- `gpt-5.4` and `gpt-5.4-pro` at 1,050,000 context / 128,000 output
7272
- `gpt-5.4-mini`, `gpt-5.4-nano`, and Codex families at 400,000 context / 128,000 output
73+
- `gpt-5.1` at 272,000 context / 128,000 output
7374
- `store: false` plus `include: ["reasoning.encrypted_content"]`
7475

7576
### Full installer mode
@@ -121,7 +122,7 @@ opencode run "task" --model=openai/gpt-5.4 --variant=high
121122

122123
It currently ships:
123124

124-
- 26 explicit model entries
125+
- 34 explicit model entries
125126
- separate model IDs such as `gpt-5.4-high` and `gpt-5.4-mini-xhigh`
126127
- the same OpenAI provider defaults (`store: false`, `reasoning.encrypted_content`)
127128

scripts/install-opencode-codex-auth.js

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -172,11 +172,16 @@ async function clearCache() {
172172
}
173173

174174
async function main() {
175-
if (!existsSync(modernTemplatePath)) {
176-
throw new Error(`Config template not found at ${modernTemplatePath}`);
177-
}
178-
if (!existsSync(legacyTemplatePath)) {
179-
throw new Error(`Config template not found at ${legacyTemplatePath}`);
175+
const requiredTemplatePaths = configMode === "modern"
176+
? [modernTemplatePath]
177+
: configMode === "legacy"
178+
? [legacyTemplatePath]
179+
: [modernTemplatePath, legacyTemplatePath];
180+
181+
for (const templatePath of requiredTemplatePaths) {
182+
if (!existsSync(templatePath)) {
183+
throw new Error(`Config template not found at ${templatePath}`);
184+
}
180185
}
181186

182187
const template = await loadTemplate(configMode);

0 commit comments

Comments
 (0)