Kotlinsdk #316
Open
Kotlinsdk #316
+37,847
−0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
contextablemark
requested review from
mme,
ranst91,
ataibarkai,
maxkorp,
tylerslaton and
NathanTarbert
as code owners
|
Hey @contextablemark, thanks for doing the work to add a Kotlin SDK! |
|
Thanks, @NathanTarbert ! Docs still need a bit more work to get them into the .MDX format, but I wanted to get the draft PR submitted in case anyone wants to take an advance look. |
|
@NathanTarbert Moving this back to draft as I just noticed that it seems to have the adk-middleware files intermingled. Will resubmit once I have a clean PR (unless the adk-middleware PR gets merged first 😄) |
Introduces a comprehensive Kotlin Multiplatform SDK for the Agent User Interaction Protocol (AG-UI) supporting Android, iOS, and JVM platforms. ## Features - Modular architecture with 4 specialized modules (core, client, tools, agent-sdk) - Real-time streaming with SSE support - Type-safe protocol implementation with kotlinx.serialization - Extensible tool execution framework - State management with JSON Patch support - Multiple authentication methods (Bearer, API Key, Basic Auth) ## Modules - **kotlin-core**: Protocol types, events, and message definitions - **kotlin-client**: HTTP transport, SSE parsing, and state management - **kotlin-tools**: Tool execution framework with error handling - **kotlin-agent-sdk**: High-level agent implementations ## Examples - Chat application with Compose Multiplatform UI - Java Android chat app demonstrating interop - Tool executor examples ## Testing - 140+ unit tests across all modules - Platform-specific test coverage - Integration test suites 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Add required Info.plist entry for Compose Multiplatform iOS support. This key is strictly enforced since Compose Multiplatform 1.6.10 to ensure proper support for high refresh rate displays (120Hz). Without this key, the app crashes during initialization with: PlistSanityCheck.performIfNeeded throwing a Kotlin/Native exception. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Fix gitignore pattern to properly exclude gradle-wrapper.jar files using **/gradle/wrapper/gradle-wrapper.jar instead of gradle/wrapper/gradle-wrapper.jar - Add missing gradle-wrapper.jar for tools example to enable running tests - Add gradle-wrapper.jar files for all examples and library modules This allows all example projects to run their tests independently without requiring parent gradle installation. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Fix flaky testDialogCancellation test by properly handling keyboard state and UI transitions during dialog interactions. Root cause: The test was failing because the soft keyboard remained open after typing, causing view hierarchy transitions that interfered with Espresso's ability to find the layoutEmptyState view. Changes: - Close soft keyboard explicitly before clicking Cancel button - Add brief delay (100ms) to allow UI transitions to complete - Add repository verification to ensure no agents were added - Improve test robustness with initial state verification Result: All Android instrumented tests now pass (30/30 = 100% success rate) 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Fixes #58 - Remove old licensing boilerplate text from Kotlin files. - Removed MIT license headers from 67 Kotlin files across examples - All files in examples/chatapp and examples/tools affected - License boilerplate (lines 1-23) removed while preserving code 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Implements complete support for thinking events in the Kotlin SDK: - Add 5 new thinking event types: THINKING_START, THINKING_END, THINKING_TEXT_MESSAGE_START, THINKING_TEXT_MESSAGE_CONTENT, THINKING_TEXT_MESSAGE_END - Add comprehensive event validation with state machine logic - Create full test suite with 40 test cases covering valid/invalid sequences - Add JSON serialization support for all thinking events - Ensure 100% test coverage with all 98 tests passing Thinking events enable AI agents to represent internal reasoning processes in the AG-UI protocol, following the same patterns as text messages but within thinking contexts. Fixes #61 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
…SULT events - Add TEXT_MESSAGE_CHUNK and TOOL_CALL_CHUNK event types to EventType enum - Add TextMessageChunkEvent and ToolCallChunkEvent classes with proper serialization - Add missing TOOL_CALL_RESULT event type and ToolCallResultEvent class - Implement ChunkTransform that converts chunk events to structured sequences - Transform automatically starts/ends message and tool call sequences as needed - Add comprehensive test suite covering all chunk scenarios and TOOL_CALL_RESULT - Add serialization tests for all new event types - Add verifier tests to ensure proper event validation - All 112 tests passing (100% success rate) - Update event count from 23 to 24 to match TypeScript SDK Resolves #55 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
…nt files - Add complete MDX documentation structure covering client, core, and tools modules - Create 12 detailed documentation files following JS SDK structure - Fix all technical inaccuracies including ToolCall structure, missing events, and API signatures - Update GitHub references from Contextable to ag-ui-protocol organization - Clean up redundant README and OVERVIEW files (removed ~800 lines of duplicate content) - Update package names from com.contextable to com.agui for consistency - Update bundle identifiers to cleaner format (com.agui.example.chatapp) - Update developer metadata to Mark Fogle while preserving ag-ui-protocol repository references All tests passing across Android, iOS, and JVM platforms. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
contextablemark
force-pushed
the
kotlinsdk
branch
from
fdbf95c to
c0a714e
Compare
|
Ready for review after rebasing off of main. |
tylerslaton
reviewed
Sep 17, 2025
There was a problem hiding this comment.
Hey @contextablemark, just a few things for this PR.
- Can you put the SDK under the following path:
sdks/community/kotlin? - Can you add a new Github action that will execute the Kotlin tests so we can sanity check they're passing in CI here?
- For the docs, can you add a mention that this is community contributed and maintained in the overview page?
With those in I'd feel comfortable merging this. The path is the same as the one provided by the Golang SDK in #339
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Draft PR for Kotlin SDK. Addresses #275.