client: выпилен CIO — движок приходит от потребителя
ci / JVM build + tests (push) Successful in 6m14s
release / Publish KMP libraries → caffeine Nexus (release) Successful in 33s

- :client больше не создаёт HttpClient: нет зависимости на ktor-client-cio,
  нет defaultAgentikHttpClient.
- applyAgentikDefaults(token) — конфигурация agentik (JSON + Bearer) поверх клиента.
- agentikHttpClient(engineFactory, token, configure) — сборка клиента из фабрики
  движка потребителя.
- AgentikAgent(id, baseUrl, httpClient) — клиент обязателен, параметр token убран.
- :agentik-cli получил свой defaultCliHttpClient() (CIO + requestTimeout=0);
  8 команд передают клиент явно.
This commit is contained in:
Porfiry
2026-09-19 22:41:34 +03:00
parent c486c7f9ab
commit c140d0b758
14 changed files with 99 additions and 45 deletions
+2 -2
View File
@@ -22,8 +22,7 @@ kotlin {
commonMain.dependencies {
api(project(":proto"))
implementation(libs.ktor.client.core)
implementation(libs.ktor.client.cio)
api(libs.ktor.client.core)
implementation(libs.ktor.client.content.negotiation)
implementation(libs.ktor.serialization.kotlinx.json)
@@ -37,6 +36,7 @@ kotlin {
implementation(libs.ktor.server.core)
implementation(libs.ktor.server.test.host)
implementation(libs.ktor.client.content.negotiation)
implementation(libs.ktor.client.cio)
implementation(libs.ktor.server.cio)
implementation(libs.ktor.server.sse)
}
@@ -8,9 +8,11 @@ import pw.binom.agentik.proto.Agent
* (модуль `:server`).
*
* ```
* val http = HttpClient(CIO) { applyAgentikDefaults(token = "s3cret") }
* val client = AgentikAgent(
* id = "my-agent",
* baseUrl = "http://localhost:8080/agentik",
* httpClient = http,
* )
* val conv = client.createConversation(temp = false)
* conv.send(listOf(Content.Text("hi")))
@@ -21,27 +23,13 @@ import pw.binom.agentik.proto.Agent
* агента не знает, поэтому клиент должен её знать сам (или взять из
* конфига).
*
* [httpClient] по умолчанию — [defaultAgentikHttpClient] (платформо-зависимый
* движок: CIO на JVM, libcurl на desktop-native). Можно передать свой.
* Клиент приходит снаружи: `:client` не выбирает движок. Собрать [HttpClient]
* можно через [agentikHttpClient] (фабрика движка + опциональные движковые
* настройки) или вручную, применив к блоку конфигурации [applyAgentikDefaults]
* (JSON + опциональный Bearer-токен).
*/
fun AgentikAgent(
id: String,
baseUrl: String,
token: String? = null,
httpClient: HttpClient = defaultAgentikHttpClient(token),
): Agent = AgentClient(httpClient = httpClient, baseUrl = baseUrl, id = id)
/**
* Дефолтный [HttpClient] для общения с `agentikAgent`. SSE-парсер ([readSse])
* живёт в общем коде и плагина `SSEClientContent` не требует.
*
* **Платформы:**
* - JVM: движок CIO. `engine { requestTimeout = 0 }` отключает встроенный
* 15-секундный request-таймаут движка (наш кастомный SSE-ридер не маркирует
* для долгих idle-стримов). Defense-in-depth: SSE-запросы в
* `ConversationClient.events`/`AgentClient.events` уже ставят
* `HttpTimeoutCapability` = INFINITE (см. [noSseReadTimeout]).
*
* Один движок CIO работает и на JVM, и на всех desktop-native (linux/macos/mingw).
* Реализация — в [HttpClientFactory.kt].
*/
httpClient: HttpClient,
): Agent = AgentClient(httpClient = httpClient, baseUrl = baseUrl, id = id)
@@ -1,7 +1,9 @@
package pw.binom.agentik.client
import io.ktor.client.HttpClient
import io.ktor.client.engine.cio.CIO
import io.ktor.client.HttpClientConfig
import io.ktor.client.engine.HttpClientEngineConfig
import io.ktor.client.engine.HttpClientEngineFactory
import io.ktor.client.plugins.DefaultRequest
import io.ktor.client.plugins.contentnegotiation.ContentNegotiation
import io.ktor.client.request.header
@@ -9,19 +11,26 @@ import io.ktor.http.HttpHeaders
import io.ktor.serialization.kotlinx.json.json
/**
* Единый HTTP-клиент для JVM и всех 5 native-таргетов (:agentik-cli).
* CIO в ktor 3.x — KMP, поддерживает linuxX64/Arm64, macosX64/Arm64, mingwX64.
* Общая конфигурация HTTP-клиента agentik — платформо-независимая часть.
*
* `requestTimeout = 0` — defense-in-depth против read-таймаута на SSE:
* основная защита в `HttpRequestBuilder.noSseReadTimeout()` ([SseTimeout]).
* `:client` НЕ выбирает движок: его приносит потребитель. Здесь живёт только то,
* без чего клиент несовместим с `/agentik`:
* - JSON-конфиг [agentikJson] (обязан совпадать с серверным);
* - при заданном [token] — `Authorization: Bearer <token>` на ВСЕ запросы
* через [DefaultRequest] (накрывает 10 REST-вызовов и оба SSE-потока;
* заголовок живёт на клиенте, а не в отдельных запросах).
*
* При заданном [token] на ВСЕ запросы клиента навешивается
* `Authorization: Bearer <token>` через плагин [DefaultRequest]. Это накрывает
* все 10 REST-вызовов и оба SSE-потока сразу — заголовок живёт на HTTP-клиенте,
* а не в отдельных запросах.
* `null` — авторизация выключена, заголовок не отправляется.
*
* Потребитель, знающий свой движок, добавляет к этому движковые настройки, напр.:
* ```
* val http = HttpClient(CIO) {
* engine { requestTimeout = 0 } // CIO-специфика, живёт у потребителя
* applyAgentikDefaults(token)
* }
* ```
*/
fun defaultAgentikHttpClient(token: String? = null): HttpClient = HttpClient(CIO) {
engine { requestTimeout = 0 }
fun HttpClientConfig<*>.applyAgentikDefaults(token: String? = null) {
install(ContentNegotiation) { json(agentikJson) }
if (token != null) {
install(DefaultRequest) {
@@ -29,3 +38,23 @@ fun defaultAgentikHttpClient(token: String? = null): HttpClient = HttpClient(CIO
}
}
}
/**
* Создаёт [HttpClient] из фабрики движка потребителя и сразу применяет к нему
* конфигурацию agentik ([applyAgentikDefaults]).
*
* Это точка, где `:client` НЕ привязан к реализации транспорта: [engineFactory]
* выбирает потребитель (CIO, OkHttp, Darwin, …), а `:client` только конфигурирует
* созданный клиент.
*
* [configure] — опциональный последний штрих потребителя (движковые настройки:
* таймауты, прокси, логирование). Вызывается ПОСЛЕ [applyAgentikDefaults].
*/
fun <T : HttpClientEngineConfig> agentikHttpClient(
engineFactory: HttpClientEngineFactory<T>,
token: String? = null,
configure: (HttpClientConfig<T>.() -> Unit)? = null,
): HttpClient = HttpClient(engineFactory) {
applyAgentikDefaults(token)
configure?.invoke(this)
}
@@ -1,5 +1,7 @@
package pw.binom.agentik.client
import io.ktor.client.HttpClient
import io.ktor.client.engine.cio.CIO
import io.ktor.client.request.get
import io.ktor.client.statement.bodyAsText
import io.ktor.http.ContentType
@@ -19,7 +21,7 @@ import kotlin.test.Test
import kotlin.test.assertEquals
/**
* Тесты клиентской части: [defaultAgentikHttpClient] с заданным `token` прикладывает
* Тесты клиентской части: [applyAgentikDefaults] с заданным `token` прикладывает
* `Authorization: Bearer <token>` ко всем запросам через плагин `DefaultRequest`,
* без токена — заголовок не отправляется.
*
@@ -47,6 +49,9 @@ class BearerHeaderTest {
var token: String? = null
}
private fun clientWith(token: String?): HttpClient =
HttpClient(CIO) { applyAgentikDefaults(token) }
private suspend fun startServer(): Pair<EmbeddedServer<*, *>, Int> {
val server = embeddedServer(ServerCIO, port = 0) {
routing {
@@ -66,7 +71,7 @@ class BearerHeaderTest {
fun clientWithTokenAttachesBearerHeader() = runBlocking {
val (server, port) = startServer()
try {
val client = defaultAgentikHttpClient("secret")
val client = clientWith("secret")
val resp = client.get("http://127.0.0.1:$port/agentik/conversations")
assertEquals(HttpStatusCode.OK, resp.status)
assertEquals("[]", resp.bodyAsText())
@@ -79,7 +84,7 @@ class BearerHeaderTest {
fun clientWithoutTokenGets401(): Unit = runBlocking {
val (server, port) = startServer()
try {
val client = defaultAgentikHttpClient(null)
val client = clientWith(null)
val resp = client.get("http://127.0.0.1:$port/agentik/conversations")
assertEquals(HttpStatusCode.Unauthorized, resp.status)
} finally {
@@ -91,7 +96,7 @@ class BearerHeaderTest {
fun clientWithWrongTokenGets401(): Unit = runBlocking {
val (server, port) = startServer()
try {
val client = defaultAgentikHttpClient("wrong")
val client = clientWith("wrong")
val resp = client.get("http://127.0.0.1:$port/agentik/conversations")
assertEquals(HttpStatusCode.Unauthorized, resp.status)
} finally {