Skip to content

build(deps): update test Commons IO to 2.22.0#788

Open
jbeckwith-oai wants to merge 1 commit into
mainfrom
codex/update-test-commons-io
Open

build(deps): update test Commons IO to 2.22.0#788
jbeckwith-oai wants to merge 1 commit into
mainfrom
codex/update-test-commons-io

Conversation

@jbeckwith-oai

Copy link
Copy Markdown
Contributor

Summary

  • constrain WireMock 2.35.2's test-only Commons IO dependency from 2.11.0 to 2.22.0 in both openai-java-core and openai-java-client-okhttp
  • remediate Dependabot alert 30 (GHSA-78wr-2p64-hpwj / CVE-2024-47554)
  • leave WireMock 2.35.2 and the Jackson 2.14.0 compatibility floor unchanged

The alert's patched floor is Commons IO 2.14.0. Commons IO 2.22.0 is the current stable release and still requires Java 8. The resolved artifact has Java 8 classfile level 52, and the focused WireMock tests pass with it under both Jackson compatibility lines.

Downstream compatibility

This is limited to testImplementation constraints. Generated Maven POMs and Gradle module metadata for both published modules contain neither Commons IO nor WireMock. There are no production source, bytecode, public API, runtime dependency, or Java baseline changes for SDK consumers.

Validation

  • dependency insight for both testRuntimeClasspath graphs and core's jacksonPublishedRuntime resolves Commons IO to 2.22.0
  • focused WireMock tests in core and okhttp pass under Jackson 2.14.0 and 2.18.9
  • all repository test tasks, ProGuard/R8 checks, and testJacksonPublished pass (external generated-API mock-server tests skipped locally)
  • ./scripts/build --no-configuration-cache
  • ./scripts/lint
  • ./scripts/detect-breaking-changes origin/main
  • generated Maven POM and Gradle module metadata inspection for both affected modules
  • Java 8 classfile and WireMock bytecode-reference inspection

@jbeckwith-oai jbeckwith-oai requested a review from a team as a code owner July 15, 2026 22:48
@openai-sdks

openai-sdks Bot commented Jul 15, 2026

Copy link
Copy Markdown

OkTest Summary

237/237 SDK tests passed in 17.011s for Java SDK PR #788.

Test results — 42 files
Test Result Time
tests/chat-completions-complex-body.test.ts ✅ Passed 350ms
tests/chat-completions-create.test.ts ✅ Passed 844ms
tests/chat-completions-stream.test.ts ✅ Passed 344ms
tests/files-content-binary.test.ts ✅ Passed 293ms
tests/files-create-multipart.test.ts ✅ Passed 377ms
tests/files-list-pagination.test.ts ✅ Passed 357ms
tests/initialize-config.test.ts ✅ Passed 269ms
tests/instance-isolation.test.ts ✅ Passed 248ms
tests/models-list.test.ts ✅ Passed 193ms
tests/responses-background-lifecycle.test.ts ✅ Passed 383ms
tests/responses-body-method-errors.test.ts ✅ Passed 499ms
tests/responses-cancel-timeout.test.ts ✅ Passed 247ms
tests/responses-cancel.test.ts ✅ Passed 372ms
tests/responses-compact-retries.test.ts ✅ Passed 341ms
tests/responses-compact.test.ts ✅ Passed 599ms
tests/responses-create-advanced-stream.test.ts ✅ Passed 240ms
tests/responses-create-advanced.test.ts ✅ Passed 1.385s
tests/responses-create-disconnect.test.ts ✅ Passed 942ms
tests/responses-create-errors.test.ts ✅ Passed 426ms
tests/responses-create-malformed-api-responses.test.ts ✅ Passed 322ms
tests/responses-create-retries.test.ts ✅ Passed 851ms
tests/responses-create-stream-failures.test.ts ✅ Passed 272ms
tests/responses-create-stream-timeout.test.ts ✅ Passed 256ms
tests/responses-create-stream-wire.test.ts ✅ Passed 6.309s
tests/responses-create-stream.test.ts ✅ Passed 419ms
tests/responses-create-terminal-states.test.ts ✅ Passed 433ms
tests/responses-create-timeout.test.ts ✅ Passed 222ms
tests/responses-create.test.ts ✅ Passed 269ms
tests/responses-delete.test.ts ✅ Passed 280ms
tests/responses-input-items-errors.test.ts ✅ Passed 312ms
tests/responses-input-items-list.test.ts ✅ Passed 333ms
tests/responses-input-items-options.test.ts ✅ Passed 412ms
tests/responses-input-tokens-count-timeout.test.ts ✅ Passed 243ms
tests/responses-input-tokens-count.test.ts ✅ Passed 426ms
tests/responses-malformed-inputs.test.ts ✅ Passed 5.174s
tests/responses-not-found-errors.test.ts ✅ Passed 390ms
tests/responses-parse.test.ts ✅ Passed 576ms
tests/responses-retrieve-retries.test.ts ✅ Passed 291ms
tests/responses-retrieve.test.ts ✅ Passed 307ms
tests/responses-stored-method-errors.test.ts ✅ Passed 1.027s
tests/retry-behavior.test.ts ✅ Passed 3.516s
tests/sdk-error-shape.test.ts ✅ Passed 400ms

View OkTest run #29462769221

SDK merge (c47ab59440b0) · head (f90bb53cc3cf) · base (845b2bc87fb0) · OkTest (fd7a761c81c1)

@jbeckwith-oai jbeckwith-oai force-pushed the codex/update-test-commons-io branch from 76147f8 to f90bb53 Compare July 16, 2026 00:55
@jbeckwith-oai jbeckwith-oai changed the title fix(deps): update test Commons IO to 2.22.0 build(deps): update test Commons IO to 2.22.0 Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant