chore: Add missing JSpecify annotations to docling-testcontainers

[ThomasVitale]

• Add missing JSpecify annotations to the docling-testcontainers module.
• Fix bug affecting the containerEnv map being immutable when adding more variables (e.g. after calling the toBuilder() method).

[Copilot]

Pull request overview

This PR updates the docling-testcontainers module configuration API to add missing JSpecify nullability annotations and to fix a toBuilder() regression where containerEnv could become immutable and fail when adding additional environment variables.

Changes:

• Add @Nullable annotations to builder fields and to Builder.apiKey(...) to better express nullability expectations.
• Fix toBuilder()-based config cloning by copying containerEnv into a mutable HashMap before further builder mutations.
• Minor container setup call-site adjustments (this.withEnv(...), etc.) and stricter blank-check for image.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File	Description
docling-testcontainers/src/main/java/ai/docling/testcontainers/serve/DoclingServeContainer.java	Uses explicit this. qualifiers when configuring the container instance.
docling-testcontainers/src/main/java/ai/docling/testcontainers/serve/config/DoclingServeContainerConfig.java	Adds nullability annotations and fixes containerEnv mutability issues when using toBuilder().
docling-testcontainers/src/main/java/ai/docling/testcontainers/serve/config/DefaultDoclingContainerConfig.java	Tightens validation for image using isBlank().

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

[github-actions]

:java_duke: JaCoCo coverage report

Overall Project	47.59%	🔴
Files changed	100%	🟢

Module	Coverage
test-report-aggregation	47.61%	🟢
test-report-aggregation	47.61%	🟢
test-report-aggregation	47.58%	🟢
test-report-aggregation	47.58%	🟢

Files

Module	File	Coverage
test-report-aggregation	DoclingServeContainer.java	100%	🟢
	DoclingServeContainerConfig.java	95.05%	🟢
	DefaultDoclingContainerConfig.java	91.94%	🟢
test-report-aggregation	DoclingServeContainer.java	100%	🟢
	DoclingServeContainerConfig.java	95.05%	🟢
	DefaultDoclingContainerConfig.java	91.94%	🟢
test-report-aggregation	DoclingServeContainer.java	100%	🟢
	DoclingServeContainerConfig.java	95.05%	🟢
	DefaultDoclingContainerConfig.java	91.94%	🟢
test-report-aggregation	DoclingServeContainer.java	100%	🟢
	DoclingServeContainerConfig.java	95.05%	🟢
	DefaultDoclingContainerConfig.java	91.94%	🟢

[github-actions]

	Tests	Passed ✅	Skipped	Failed
Gradle Test Results (all modules & JDKs)	1792 ran	1792 passed	0 skipped	0 failed

Test	Result
No test annotations available

• View Gradle Test Results (all modules & JDKs)

[github-actions]

HTML test reports are available as workflow artifacts (zipped HTML).

• Download: Artifacts for this run

[edeandrea]

Only one comment on the image field - it can not be null (the docs even say so).

[edeandrea]

Also I think some of the copilot comments are valid

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

[github-actions]

HTML test reports are available as workflow artifacts (zipped HTML).

• Download: Artifacts for this run

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (1)

docling-testcontainers/src/main/java/ai/docling/testcontainers/serve/config/DoclingServeContainerConfig.java:199

• The Javadoc for Builder.image(String) says it throws IllegalArgumentException when the provided image is null/empty/blank, but the current implementation just assigns and defers validation until build() (via DefaultDoclingContainerConfig). Either update the Javadoc to reflect deferred validation or fail fast here so callers get the documented behavior at the point of call.

    /**
     * Sets the name of the container image to use for the configuration.
     * This value is required and must not be null, empty, or blank.
     *
     * @param image the name of the container image
     * @return the builder instance for method chaining
     * @throws IllegalArgumentException if the provided image is null, empty, or blank
     */
    public Builder image(String image) {
      this.image = image;
      return this;
    }

[github-actions]

HTML test reports are available as workflow artifacts (zipped HTML).

• Download: Artifacts for this run

[github-actions]

HTML test reports are available as workflow artifacts (zipped HTML).

• Download: Artifacts for this run