Skip to content

build(deps): update Dokka to 2.1.0#791

Open
jbeckwith-oai wants to merge 1 commit into
mainfrom
codex/update-dokka
Open

build(deps): update Dokka to 2.1.0#791
jbeckwith-oai wants to merge 1 commit into
mainfrom
codex/update-dokka

Conversation

@jbeckwith-oai

Copy link
Copy Markdown
Contributor

Summary

  • upgrade the build-only Dokka Gradle plugin from 2.0.0 to 2.1.0
  • retain Dokka's V1 task/aggregation model and K1 analyzer so the existing Javadoc task graph and link layout remain stable
  • move Dokka's runtime from Woodstox 6.2.4 to 6.5.1 and Jackson 2.12.7 to 2.15.3

This targets Dependabot alerts #6 (Woodstox), #43 (Jackson Core), and #93 (Jackson Databind).

Why 2.1.0 instead of 2.2.0

2.2.0 was evaluated first with both K1 and K2 analysis. Its Javadoc renderer generates member-summary links that do not match any anchor in the target page: 76 new invalid anchors with K1 and 184 with K2 in the core artifact. That would regress published documentation links.

2.1.0 resolves the same relevant vulnerable transitives while preserving every existing Javadoc file and internal link target. Moving to Dokka's V2 aggregation model and K2 analysis should be handled as a separate documentation-pipeline change.

Compatibility

  • all four published Maven POMs are byte-for-byte identical
  • Gradle module metadata is semantically identical; only regenerated artifact checksums differ
  • production and source JAR entry contents are identical before and after
  • SDK bytecode remains Java 8 (class major version 52)
  • Javadoc JAR entry sets and artifact names are identical
  • no internal Javadoc link targets were added, removed, or broken
  • Dokka changes generated boilerplate content on 1,624 of 9,682 core Javadoc entries, but preserves their paths and anchors
  • Dokka remains a build-only dependency and is not exposed to SDK consumers

Dokka 2.1.0 still resolves Jackson 2.15.3. Newer Jackson advisories affecting that build-tool classpath remain and require a separate Dokka-classpath Jackson alignment change.

Validation

  • ./scripts/lint
  • ./scripts/build
  • ./scripts/test
  • ./scripts/detect-breaking-changes origin/main
  • ./gradlew clean publish -PpublishLocal --no-configuration-cache
  • before/after comparison of all published POMs, Gradle module metadata, production/source JAR contents, Javadoc entries, and internal links
  • resolved dokkaJavadocRuntime dependency-tree review

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

openai-sdks Bot commented Jul 15, 2026

Copy link
Copy Markdown

OkTest Summary

237/237 SDK tests passed in 17.762s for Java SDK PR #791.

Test results — 42 files
Test Result Time
tests/chat-completions-complex-body.test.ts ✅ Passed 373ms
tests/chat-completions-create.test.ts ✅ Passed 864ms
tests/chat-completions-stream.test.ts ✅ Passed 376ms
tests/files-content-binary.test.ts ✅ Passed 247ms
tests/files-create-multipart.test.ts ✅ Passed 462ms
tests/files-list-pagination.test.ts ✅ Passed 337ms
tests/initialize-config.test.ts ✅ Passed 196ms
tests/instance-isolation.test.ts ✅ Passed 287ms
tests/models-list.test.ts ✅ Passed 190ms
tests/responses-background-lifecycle.test.ts ✅ Passed 300ms
tests/responses-body-method-errors.test.ts ✅ Passed 564ms
tests/responses-cancel-timeout.test.ts ✅ Passed 204ms
tests/responses-cancel.test.ts ✅ Passed 369ms
tests/responses-compact-retries.test.ts ✅ Passed 424ms
tests/responses-compact.test.ts ✅ Passed 355ms
tests/responses-create-advanced-stream.test.ts ✅ Passed 375ms
tests/responses-create-advanced.test.ts ✅ Passed 1.426s
tests/responses-create-disconnect.test.ts ✅ Passed 1.1s
tests/responses-create-errors.test.ts ✅ Passed 389ms
tests/responses-create-malformed-api-responses.test.ts ✅ Passed 367ms
tests/responses-create-retries.test.ts ✅ Passed 254ms
tests/responses-create-stream-failures.test.ts ✅ Passed 284ms
tests/responses-create-stream-timeout.test.ts ✅ Passed 241ms
tests/responses-create-stream-wire.test.ts ✅ Passed 6.642s
tests/responses-create-stream.test.ts ✅ Passed 504ms
tests/responses-create-terminal-states.test.ts ✅ Passed 542ms
tests/responses-create-timeout.test.ts ✅ Passed 221ms
tests/responses-create.test.ts ✅ Passed 280ms
tests/responses-delete.test.ts ✅ Passed 308ms
tests/responses-input-items-errors.test.ts ✅ Passed 307ms
tests/responses-input-items-list.test.ts ✅ Passed 367ms
tests/responses-input-items-options.test.ts ✅ Passed 344ms
tests/responses-input-tokens-count-timeout.test.ts ✅ Passed 212ms
tests/responses-input-tokens-count.test.ts ✅ Passed 283ms
tests/responses-malformed-inputs.test.ts ✅ Passed 5.234s
tests/responses-not-found-errors.test.ts ✅ Passed 566ms
tests/responses-parse.test.ts ✅ Passed 873ms
tests/responses-retrieve-retries.test.ts ✅ Passed 1.034s
tests/responses-retrieve.test.ts ✅ Passed 336ms
tests/responses-stored-method-errors.test.ts ✅ Passed 1.036s
tests/retry-behavior.test.ts ✅ Passed 3.764s
tests/sdk-error-shape.test.ts ✅ Passed 467ms

View OkTest run #29462909399

SDK merge (5f244aa0dcb1) · head (21f28987d365) · base (845b2bc87fb0) · OkTest (fd7a761c81c1)

@jbeckwith-oai jbeckwith-oai changed the title fix(deps): update Dokka to 2.1.0 build(deps): update Dokka to 2.1.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