Skip to content

chore: include AISdkInfo in extra-files to keep version synced#191

Merged
mattrmc1 merged 3 commits into
mainfrom
mmccarthy/chore/release-please-config
Jul 13, 2026
Merged

chore: include AISdkInfo in extra-files to keep version synced#191
mattrmc1 merged 3 commits into
mainfrom
mmccarthy/chore/release-please-config

Conversation

@mattrmc1

@mattrmc1 mattrmc1 commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds AISdkInfo.java to Release Please extra-files so AISdkInfo.VERSION stays in sync with gradle.properties on each release PR. Follows the same pattern as the core Java server SDK and .NET AI SDK.

The SDK-info event test now references the AISdkInfo constants directly so future version bumps can't leave the expectation stale. No runtime behavior change.


Note

Low Risk
Release automation and test expectations only; no production logic changes.

Overview
Release Please for lib/sdk/server-ai now bumps AISdkInfo.java alongside gradle.properties, matching the core server SDK’s Version.java pattern so AISdkInfo.VERSION stays aligned on release PRs.

constructorEmitsSdkInfoEvent in LDAIClientImplTest asserts against AISdkInfo.NAME, VERSION, and LANGUAGE instead of hardcoded literals, so version bumps won’t leave the test out of date. No runtime behavior change.

Reviewed by Cursor Bugbot for commit 07ff4bd. Bugbot is set up for automated code reviews on this repo. Configure here.

@mattrmc1 mattrmc1 marked this pull request as ready for review July 13, 2026 17:15
@mattrmc1 mattrmc1 requested a review from a team as a code owner July 13, 2026 17:15
@mattrmc1 mattrmc1 merged commit 26d01e8 into main Jul 13, 2026
24 checks passed
@mattrmc1 mattrmc1 deleted the mmccarthy/chore/release-please-config branch July 13, 2026 19:16
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.

2 participants