Skip to content

Commit 340a752

Browse files
authored
fix(codex): stop forcing low text verbosity (#2501)
1 parent 85773b4 commit 340a752

1 file changed

Lines changed: 0 additions & 37 deletions

File tree

‎crates/forge_repo/src/provider/openai_responses/codex_transformer.rs‎

Lines changed: 0 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -28,13 +28,6 @@ impl Transformer for CodexTransformer {
2828
includes.push(oai::IncludeEnum::ReasoningEncryptedContent);
2929
}
3030

31-
// Force text verbosity to Low for concise codex output
32-
let text = request.text.get_or_insert(oai::ResponseTextParam {
33-
format: oai::TextResponseFormatConfiguration::Text,
34-
verbosity: None,
35-
});
36-
text.verbosity = Some(oai::Verbosity::Low);
37-
3831
if let Some(reasoning) = request.reasoning.as_mut() {
3932
reasoning.effort = Some(oai::ReasoningEffort::High);
4033
reasoning.summary = Some(oai::ReasoningSummary::Auto);
@@ -149,36 +142,6 @@ mod tests {
149142
);
150143
}
151144

152-
#[test]
153-
fn test_codex_transformer_sets_text_verbosity_low() {
154-
let fixture = fixture();
155-
let mut transformer = CodexTransformer;
156-
let actual = transformer.transform(fixture);
157-
158-
let expected = Some(oai::Verbosity::Low);
159-
assert_eq!(
160-
actual.text.as_ref().and_then(|t| t.verbosity.clone()),
161-
expected
162-
);
163-
}
164-
165-
#[test]
166-
fn test_codex_transformer_overrides_text_verbosity_to_low() {
167-
let mut fixture = fixture();
168-
fixture.text = Some(oai::ResponseTextParam {
169-
format: oai::TextResponseFormatConfiguration::Text,
170-
verbosity: Some(oai::Verbosity::High),
171-
});
172-
let mut transformer = CodexTransformer;
173-
let actual = transformer.transform(fixture);
174-
175-
let expected = Some(oai::Verbosity::Low);
176-
assert_eq!(
177-
actual.text.as_ref().and_then(|t| t.verbosity.clone()),
178-
expected
179-
);
180-
}
181-
182145
#[test]
183146
fn test_codex_transformer_no_reasoning_unchanged() {
184147
let fixture = fixture();

0 commit comments

Comments
 (0)