git.iko.soy/opencode-android
on master help sign in
swap
ab6a5da18 ahead, 0 behind
12 files changed+1380−132
since splitsnapshots
unifiedsplit
app/src/main/java/soy/iko/opencode/CoreViewModel.kt+18 app/src/main/java/soy/iko/opencode/MainActivity.kt+8 app/src/main/java/soy/iko/opencode/core/HttpClient.kt+1 app/src/main/java/soy/iko/opencode/core/OpencodeCore.kt+208−41 app/src/main/java/soy/iko/opencode/core/SseClient.kt+1 app/src/main/java/soy/iko/opencode/ui/OpencodeApp.kt+71−7 app/src/main/java/soy/iko/opencode/ui/components/MessageBubble.kt+296−27 app/src/main/java/soy/iko/opencode/ui/screens/ChatScreen.kt+400−33 app/src/main/java/soy/iko/opencode/ui/screens/ConnectScreen.kt+100−4 app/src/main/java/soy/iko/opencode/ui/screens/SessionsScreen.kt+240−18 app/src/main/java/soy/iko/opencode/ui/theme/Theme.kt+23−2 shared/src/app.rs+14
app/src/main/java/soy/iko/opencode/CoreViewModel.kt+18
@@ -23,6 +23,22 @@ class CoreViewModel(app: Application) : AndroidViewModel(app) {
2323 init {
2424 prefs.getString(KEY_URL, null)?.let { core.dispatch(Event.ServerUrlChanged(it)) }
2525 prefs.getString(KEY_USER, null)?.let { core.dispatch(Event.UsernameChanged(it)) }
26+ // The password is stored in plain text in SharedPreferences — a deliberate
27+ // convenience trade-off for a local dev tool so reconnecting is one tap.
28+ prefs.getString(KEY_PASSWORD, null)?.let { core.dispatch(Event.PasswordChanged(it)) }
29+
30+ // On a fresh process after a configuration change, resume the SSE stream if
31+ // the core is already connected (no-op otherwise — it guards on `connected`).
32+ core.resumeStreamingIfConnected()
33+
34+ // Returning users shouldn't have to tap Connect: if a URL was restored and we
35+ // aren't already connected, kick off the connection attempt on launch. The core
36+ // handles an empty/invalid URL, so this only fires when there's something to try.
37+ core.view.value.let { restored ->
38+ if (restored.serverUrl.isNotEmpty() && !restored.connected) {
39+ core.dispatch(Event.Connect)
40+ }
41+ }
2642
2743 // Persist connection details once a connection succeeds.
2844 viewModelScope.launch {
@@ -31,6 +47,7 @@ class CoreViewModel(app: Application) : AndroidViewModel(app) {
3147 prefs.edit {
3248 putString(KEY_URL, state.serverUrl)
3349 putString(KEY_USER, state.username)
50+ putString(KEY_PASSWORD, state.password)
3451 }
3552 }
3653 }
@@ -44,5 +61,6 @@ class CoreViewModel(app: Application) : AndroidViewModel(app) {
4461 private companion object {
4562 const val KEY_URL = "serverUrl"
4663 const val KEY_USER = "username"
64+ const val KEY_PASSWORD = "password"
4765 }
4866}
app/src/main/java/soy/iko/opencode/MainActivity.kt+8
@@ -2,7 +2,9 @@ package soy.iko.opencode
22
33import android.os.Bundle
44import androidx.activity.ComponentActivity
5+import androidx.activity.SystemBarStyle
56import androidx.activity.compose.setContent
7+import androidx.activity.enableEdgeToEdge
68import androidx.compose.runtime.getValue
79import androidx.lifecycle.compose.collectAsStateWithLifecycle
810import androidx.lifecycle.viewmodel.compose.viewModel
@@ -11,6 +13,12 @@ import soy.iko.opencode.ui.theme.OpencodeTheme
1113
1214class MainActivity : ComponentActivity() {
1315 override fun onCreate(savedInstanceState: Bundle?) {
16+ enableEdgeToEdge(
17+ statusBarStyle = SystemBarStyle.auto(
18+ lightScrim = 0xFF0F1115.toInt(),
19+ darkScrim = 0xFF0F1115.toInt(),
20+ ),
21+ )
1422 super.onCreate(savedInstanceState)
1523 setContent {
1624 OpencodeTheme {
app/src/main/java/soy/iko/opencode/core/HttpClient.kt+1
@@ -51,6 +51,7 @@ class HttpClient(
5151 ): Result<HttpResp> = withContext(Dispatchers.IO) {
5252 runCatching {
5353 val builder = Request.Builder().url(url).method()
54+ builder.header("User-Agent", "opencode-android/1.0")
5455 if (auth != null) builder.header("Authorization", auth)
5556 await(okhttp.newCall(builder.build()))
5657 }
app/src/main/java/soy/iko/opencode/core/OpencodeCore.kt+208−41
@@ -1,6 +1,7 @@
11package soy.iko.opencode.core
22
33import kotlinx.coroutines.CoroutineScope
4+import kotlinx.coroutines.Job
45import kotlinx.coroutines.delay
56import kotlinx.coroutines.flow.MutableStateFlow
67import kotlinx.coroutines.flow.StateFlow
@@ -48,6 +49,31 @@ class OpencodeCore(
4849 private var sse: SseClient? = null
4950 private var sseClosedIntentionally = false
5051 private var reconnectAttempts = 0
52+ private var reconnectJob: Job? = null
53+
54+ // Guards against overlapping session refreshes: rapid taps launch multiple
55+ // loadSessions coroutines and the slowest-to-return would otherwise win.
56+ private var loadSessionsJob: Job? = null
57+ // Tracks the in-flight createSession coroutine so navigating away can cancel it
58+ // before it dispatches SelectSession into a screen the user already left.
59+ private var createSessionJob: Job? = null
60+ // Tracks the in-flight loadMessages coroutine. Rapidly switching sessions launches
61+ // several; cancel the prior one so only the newest fetch runs (saves needless requests).
62+ private var loadMessagesJob: Job? = null
63+ // Tracks the in-flight probeHealth coroutine. Rapid double-taps on Connect would
64+ // otherwise launch two concurrent health probes; cancel the prior one first.
65+ private var connectJob: Job? = null
66+ // Tracks the in-flight DeleteSession coroutine so navigating away can cancel it
67+ // before its response handler mutates the sessions list / screen the user already left.
68+ private var deleteSessionJob: Job? = null
69+ // Tracks the in-flight sendMessage coroutine so navigating away or switching sessions
70+ // can cancel it before its response handler touches stale message state.
71+ private var sendMessageJob: Job? = null
72+ // Tracks the in-flight abort coroutine (CancelGeneration) so rapid cancel taps don't
73+ // fire multiple abort requests, and navigating away can cancel it.
74+ private var abortJob: Job? = null
75+ // Message IDs removed via SSE; a late MessageUpdated for one must not resurrect it.
76+ private val removedMessageIds = mutableSetOf<String>()
5177
5278 // ── Event entry point ──────────────────────────────────────────────────────
5379 fun dispatch(event: Event) {
@@ -55,38 +81,69 @@ class OpencodeCore(
5581 // Store the field verbatim while typing; normalizing here would fight
5682 // the user's edits (e.g. eat the "://" as they type it). The URL is
5783 // normalized once, on Connect.
58− is Event.ServerUrlChanged -> { serverUrl = event.url; emit() }
59− is Event.UsernameChanged -> { username = event.value; emit() }
60− is Event.PasswordChanged -> { password = event.value; emit() }
84+ // Clear any prior error as soon as the user edits a field, so a stale
85+ // message (e.g. "Invalid credentials") doesn't linger while they correct it.
86+ is Event.ServerUrlChanged -> { serverUrl = event.url; if (error != null) error = null; emit() }
87+ is Event.UsernameChanged -> { username = event.value; if (error != null) error = null; emit() }
88+ is Event.PasswordChanged -> { password = event.value; if (error != null) error = null; emit() }
6189
6290 Event.Connect -> {
6391 serverUrl = normalizeUrl(serverUrl)
6492 if (serverUrl.isEmpty()) { error = "Enter a server URL"; emit(); return }
65− loading = true; error = null; authRequired = false; emit()
66− scope.launch { probeHealth() }
93+ // The auth fields are showing but empty — sending the request now
94+ // would just 401 again in a loop. Prompt for credentials instead.
95+ if (authRequired && !hasCredentials()) { error = "Enter username and password"; emit(); return }
96+ // Leave authRequired as-is; probeHealth sets it from the response (true on
97+ // 401, false on success). Resetting it here just flickers the auth fields.
98+ loading = true; error = null; emit()
99+ connectJob?.cancel()
100+ connectJob = scope.launch { probeHealth() }
67101 }
68102
69− Event.LoadSessions -> { loading = true; emit(); scope.launch { loadSessions() } }
103+ Event.LoadSessions -> {
104+ if (!connected) { error = "Not connected"; emit(); return }
105+ loading = true; emit()
106+ loadSessionsJob?.cancel()
107+ loadSessionsJob = scope.launch { loadSessions() }
108+ }
70109
71110 is Event.SelectSession -> {
72− currentSessionTitle = sessions.firstOrNull { it.id == event.id }?.title.orEmpty()
111+ // The session may have been deleted (e.g. via SSE) while the list was on
112+ // screen; opening it would only 404 on loadMessages. Bail with an error.
113+ val session = sessions.firstOrNull { it.id == event.id }
114+ if (session == null) { error = "Session no longer exists"; emit(); return }
115+ currentSessionTitle = session.title
73116 currentSessionId = event.id
74117 messages.clear()
118+ removedMessageIds.clear()
75119 screen = Screen.Chat
76120 loading = true
121+ generating = false
77122 error = null
78123 emit()
79− scope.launch { loadMessages(event.id) }
124+ connectJob?.cancel()
125+ abortJob?.cancel()
126+ sendMessageJob?.cancel()
127+ deleteSessionJob?.cancel()
128+ loadMessagesJob?.cancel()
129+ loadMessagesJob = scope.launch { loadMessages(event.id) }
80130 }
81131
82− Event.CreateSession -> { loading = true; emit(); scope.launch { createSession() } }
132+ Event.CreateSession -> {
133+ if (!connected) { error = "Not connected"; emit(); return }
134+ loading = true; error = null; emit()
135+ createSessionJob?.cancel()
136+ createSessionJob = scope.launch { createSession() }
137+ }
83138
84139 is Event.DeleteSession -> {
140+ if (!connected) { error = "Not connected"; emit(); return }
85141 val id = event.id
86142 // Delete on the server first; only drop it from the UI once that
87143 // succeeds, so a failed request leaves the session recoverable.
88− loading = true; emit()
89− scope.launch {
144+ loading = true; error = null; emit()
145+ deleteSessionJob?.cancel()
146+ deleteSessionJob = scope.launch {
90147 http.delete("$serverUrl/session/$id", authHeader()).fold(
91148 onSuccess = { resp ->
92149 if (resp.code in 200..299) {
@@ -115,28 +172,57 @@ class OpencodeCore(
115172 is Event.SendMessage -> sendMessage(event.text)
116173
117174 Event.CancelGeneration -> {
118− generating = false; emit()
175+ generating = false
176+ messages.forEach { it.streaming = false }
177+ emit()
119178 val id = currentSessionId ?: return
120− scope.launch { http.postJson("$serverUrl/session/$id/abort", "{}", authHeader()) }
179+ // Debounce rapid cancel taps: cancel any in-flight abort before firing
180+ // another so we don't stack duplicate abort requests.
181+ abortJob?.cancel()
182+ abortJob = scope.launch { http.postJson("$serverUrl/session/$id/abort", "{}", authHeader()) }
121183 }
122184
123185 Event.NavigateToSessions -> {
124186 screen = Screen.Sessions
125187 currentSessionId = null
126188 messages.clear()
189+ removedMessageIds.clear()
190+ draft = ""
127191 generating = false
192+ // Clear any error from the chat (e.g. a failed send) so it doesn't
193+ // linger on the sessions list after the user navigates back.
194+ error = null
128195 loading = true
129196 emit()
130− scope.launch { loadSessions() }
197+ connectJob?.cancel()
198+ abortJob?.cancel()
199+ createSessionJob?.cancel()
200+ deleteSessionJob?.cancel()
201+ sendMessageJob?.cancel()
202+ loadMessagesJob?.cancel()
203+ loadSessionsJob?.cancel()
204+ loadSessionsJob = scope.launch { loadSessions() }
131205 }
132206
133207 Event.NavigateToConnect -> {
134208 screen = Screen.Connect
135209 connected = false
136210 loading = false
211+ generating = false
137212 sessions.clear()
138213 messages.clear()
214+ removedMessageIds.clear()
139215 currentSessionId = null
216+ draft = ""
217+ // Reset to the default so a late-firing close callback from the old
218+ // SSE connection can't flash the reconnecting banner on the next connect.
219+ sseConnected = true
220+ connectJob?.cancel()
221+ abortJob?.cancel()
222+ createSessionJob?.cancel()
223+ deleteSessionJob?.cancel()
224+ sendMessageJob?.cancel()
225+ loadMessagesJob?.cancel()
140226 closeSse()
141227 emit()
142228 }
@@ -150,26 +236,44 @@ class OpencodeCore(
150236 if (connected && sse == null) startSse()
151237 }
152238
153− fun shutdown() = closeSse()
239+ fun shutdown() {
240+ // Cancel every tracked coroutine before tearing down the stream so none can
241+ // emit into a cancelled scope after the ViewModel is cleared.
242+ connectJob?.cancel()
243+ loadSessionsJob?.cancel()
244+ createSessionJob?.cancel()
245+ loadMessagesJob?.cancel()
246+ deleteSessionJob?.cancel()
247+ sendMessageJob?.cancel()
248+ abortJob?.cancel()
249+ closeSse()
250+ }
154251
155252 // ── Effects ────────────────────────────────────────────────────────────────
156253 private fun sendMessage(raw: String) {
157254 val text = raw.trim()
158255 val session = currentSessionId ?: return
159256 if (text.isEmpty()) return
257+ // Debounce rapid double-taps on send: a send is already in flight, so a second
258+ // tap would create a duplicate user message and fire a duplicate request.
259+ if (generating) return
160260 localSeq += 1
161261 val user = MsgState("local-$localSeq", "user", System.currentTimeMillis()).apply {
162262 status = MessageStatus.Pending
163263 textParts["local"] = text
164264 }
165265 messages.add(user)
266+ // Keep the draft text so we can restore it if the send fails; otherwise the
267+ // user's message is lost and they'd have to retype it.
268+ val savedDraft = draft
166269 draft = ""
167270 generating = true
168271 error = null
169272 emit()
170273
171274 val body = Protocol.json.encodeToString(PromptBody.serializer(), PromptBody(listOf(TextInput(text = text))))
172− scope.launch {
275+ sendMessageJob?.cancel()
276+ sendMessageJob = scope.launch {
173277 val result = http.postJson("$serverUrl/session/$session/prompt_async", body, authHeader())
174278 result.fold(
175279 onSuccess = { resp ->
@@ -178,12 +282,16 @@ class OpencodeCore(
178282 } else {
179283 messages.firstOrNull { it.id == user.id }?.status = MessageStatus.Failed
180284 generating = false
285+ messages.forEach { it.streaming = false }
286+ draft = savedDraft
181287 error = "Server returned status ${resp.code}"
182288 }
183289 },
184290 onFailure = { e ->
185291 messages.firstOrNull { it.id == user.id }?.status = MessageStatus.Failed
186292 generating = false
293+ messages.forEach { it.streaming = false }
294+ draft = savedDraft
187295 error = "Request failed: ${e.message}"
188296 },
189297 )
@@ -198,7 +306,14 @@ class OpencodeCore(
198306 when {
199307 resp.code == 401 -> {
200308 loading = false
201− if (hasCreds) error = "Invalid credentials" else authRequired = true
309+ // A prior connection may have succeeded (connected = true); a
310+ // re-probe returning 401 (e.g. expired credentials) means we're
311+ // no longer authenticated, so drop the connected flag.
312+ connected = false
313+ // Always reveal the auth fields so the user can see/edit
314+ // their credentials — even when the ones supplied were wrong.
315+ authRequired = true
316+ if (hasCreds) error = "Invalid credentials"
202317 emit()
203318 }
204319 resp.code in 200..299 -> {
@@ -209,10 +324,14 @@ class OpencodeCore(
209324 loading = true
210325 emit()
211326 startSse()
212− loadSessions()
327+ // Route through loadSessionsJob so a prior in-flight loadSessions
328+ // (e.g. from a rapid double-connect) is cancelled first.
329+ loadSessionsJob?.cancel()
330+ loadSessionsJob = scope.launch { loadSessions() }
213331 }
214332 else -> {
215333 loading = false
334+ connected = false
216335 error = "Server returned status ${resp.code}"
217336 emit()
218337 }
@@ -220,6 +339,7 @@ class OpencodeCore(
220339 },
221340 onFailure = { e ->
222341 loading = false
342+ connected = false
223343 error = "Connection failed: ${e.message}"
224344 emit()
225345 },
@@ -232,6 +352,9 @@ class OpencodeCore(
232352 if (resp.code in 200..299) {
233353 sessions.clear()
234354 sessions.addAll(Protocol.parseSessions(resp.body).map { SessionView(it.id, displayTitle(it.title)) })
355+ // Impose a stable alphabetical order so the list doesn't shuffle with
356+ // whatever order the server happens to return sessions in.
357+ sessions.sortBy { it.title.lowercase() }
235358 } else {
236359 error = "Server returned status ${resp.code}"
237360 }
@@ -249,6 +372,10 @@ class OpencodeCore(
249372 if (session != null) {
250373 if (sessions.none { it.id == session.id }) {
251374 sessions.add(0, SessionView(session.id, displayTitle(session.title)))
375+ // Restore the alphabetical order immediately (matching loadSessions /
376+ // SessionUpserted) so the new session doesn't sit out-of-order at the
377+ // top until the SSE SessionUpserted event arrives to re-sort.
378+ sessions.sortBy { it.title.lowercase() }
252379 }
253380 emit()
254381 // SelectSession takes over the loading state (loads messages).
@@ -266,10 +393,11 @@ class OpencodeCore(
266393 private suspend fun loadMessages(sessionId: String) {
267394 http.get("$serverUrl/session/$sessionId/message", authHeader()).fold(
268395 onSuccess = { resp ->
269− if (!isCurrent(sessionId)) return // user navigated away while loading
396+ if (!isCurrent(sessionId)) { loading = false; return } // user navigated away while loading
270397 loading = false
271398 if (resp.code in 200..299) {
272399 messages.clear()
400+ removedMessageIds.clear()
273401 messages.addAll(Protocol.parseMessages(resp.body).map(::messageFromWire))
274402 generating = messages.any { it.streaming }
275403 } else {
@@ -278,8 +406,9 @@ class OpencodeCore(
278406 emit()
279407 },
280408 onFailure = { e ->
281− if (!isCurrent(sessionId)) return
409+ if (!isCurrent(sessionId)) { loading = false; return }
282410 loading = false
411+ generating = false
283412 error = "Request failed: ${e.message}"
284413 emit()
285414 },
@@ -308,6 +437,8 @@ class OpencodeCore(
308437
309438 private fun closeSse() {
310439 sseClosedIntentionally = true
440+ reconnectJob?.cancel()
441+ reconnectJob = null
311442 sse?.close()
312443 sse = null
313444 }
@@ -318,7 +449,10 @@ class OpencodeCore(
318449 // stream reconnects (onStateChange(true)) so a healthy server starts fresh.
319450 val delayMs = (2000L * (1L shl reconnectAttempts)).coerceAtMost(30_000L)
320451 if (delayMs < 30_000L) reconnectAttempts += 1
321− scope.launch {
452+ // Cancel any pending reconnect so rapid network flapping can't stack multiple
453+ // coroutines that each open a new SSE connection.
454+ reconnectJob?.cancel()
455+ reconnectJob = scope.launch {
322456 delay(delayMs)
323457 if (!sseClosedIntentionally && connected && screen != Screen.Connect) startSse()
324458 }
@@ -326,15 +460,20 @@ class OpencodeCore(
326460
327461 private fun handleServerEvent(json: String) {
328462 val event = Protocol.parseEvent(json) ?: return
329− applyEvent(event)
330− emit()
463+ // Only re-render if the event actually changed our state; applyEvent returns
464+ // false for events that don't apply (other session, unknown message, …).
465+ if (applyEvent(event)) emit()
331466 }
332467
333− private fun applyEvent(event: Protocol.Event) {
468+ /** Apply a server event to the model. Returns true iff any state actually changed. */
469+ private fun applyEvent(event: Protocol.Event): Boolean {
334470 when (event) {
335471 is Protocol.Event.MessageUpdated -> {
336472 val info = event.info
337− if (info.role != "assistant" || !isCurrent(info.sessionId)) return
473+ if (info.role != "assistant" || !isCurrent(info.sessionId)) return false
474+ // A MessageRemoved may have arrived before this (late/out-of-order) update;
475+ // don't re-create a message the server has already deleted.
476+ if (info.id in removedMessageIds) return false
338477 val streaming = info.time.completed == null && info.error == null
339478 val existing = messages.firstOrNull { it.id == info.id }
340479 if (existing != null) {
@@ -343,38 +482,59 @@ class OpencodeCore(
343482 } else {
344483 messages.add(MsgState(info.id, "assistant", info.time.created).apply { this.streaming = streaming })
345484 }
346− info.error?.let { error = it.message(); generating = false }
485+ info.error?.let { error = it.message(); messages.forEach { m -> m.streaming = false }; generating = false }
486+ return true
347487 }
348488
349489 is Protocol.Event.PartUpdated -> {
350490 val part = event.part
351− val msg = messages.firstOrNull { it.id == part.messageId } ?: return
352− when (part) {
353− is Protocol.Part.Text -> if (!part.synthetic) {
491+ val msg = messages.firstOrNull { it.id == part.messageId } ?: return false
492+ // Synthetic text parts are intentionally skipped; report no change so we
493+ // don't trigger a needless re-render/emit for them.
494+ return when (part) {
495+ is Protocol.Part.Text -> if (part.synthetic) {
496+ false
497+ } else {
354498 msg.textParts[part.id] = pickText(part.text, event.delta, msg.textParts[part.id])
499+ true
355500 }
356− is Protocol.Part.Reasoning ->
501+ is Protocol.Part.Reasoning -> {
357502 msg.reasoningParts[part.id] = pickText(part.text, event.delta, msg.reasoningParts[part.id])
358− is Protocol.Part.Tool ->
503+ true
504+ }
505+ is Protocol.Part.Tool -> {
359506 msg.toolParts[part.id] = ToolView(part.tool, part.status, part.title)
507+ true
508+ }
360509 }
361510 }
362511
363− is Protocol.Event.PartRemoved -> messages.firstOrNull { it.id == event.messageId }?.remove(event.partId)
512+ is Protocol.Event.PartRemoved -> {
513+ val msg = messages.firstOrNull { it.id == event.messageId } ?: return false
514+ msg.remove(event.partId)
515+ return true
516+ }
364517
365− is Protocol.Event.MessageRemoved -> messages.removeAll { it.id == event.messageId }
518+ is Protocol.Event.MessageRemoved -> {
519+ // Remember the ID so a late MessageUpdated can't resurrect the message.
520+ removedMessageIds.add(event.messageId)
521+ return messages.removeAll { it.id == event.messageId }
522+ }
366523
367− is Protocol.Event.SessionIdle -> if (isCurrent(event.sessionId)) {
524+ is Protocol.Event.SessionIdle -> {
525+ if (!isCurrent(event.sessionId)) return false
368526 generating = false
369527 messages.forEach { it.streaming = false }
528+ return true
370529 }
371530
372531 is Protocol.Event.SessionError -> {
373532 val scoped = event.sessionId == null || isCurrent(event.sessionId)
374− if (scoped) {
375− event.error?.let { error = it.message() }
376− generating = false
377− }
533+ if (!scoped) return false
534+ event.error?.let { error = it.message() }
535+ generating = false
536+ messages.forEach { it.streaming = false }
537+ return true
378538 }
379539
380540 is Protocol.Event.SessionUpserted -> {
@@ -385,19 +545,26 @@ class OpencodeCore(
385545 } else {
386546 sessions.add(0, SessionView(event.info.id, title))
387547 }
548+ // Keep the list alphabetically sorted (matching loadSessions) so an
549+ // upserted session doesn't jump to the top and break the ordering.
550+ sessions.sortBy { it.title.lowercase() }
388551 if (isCurrent(event.info.id)) currentSessionTitle = title
552+ return true
389553 }
390554
391− is Protocol.Event.SessionDeleted -> event.sessionId?.let { id ->
392− sessions.removeAll { it.id == id }
555+ is Protocol.Event.SessionDeleted -> {
556+ val id = event.sessionId ?: return false
557+ val removed = sessions.removeAll { it.id == id }
393558 if (isCurrent(id)) {
394559 currentSessionId = null
395560 messages.clear()
396561 screen = Screen.Sessions
562+ return true
397563 }
564+ return removed
398565 }
399566
400− Protocol.Event.Other -> Unit
567+ Protocol.Event.Other -> return false
401568 }
402569 }
403570
app/src/main/java/soy/iko/opencode/core/SseClient.kt+1
@@ -33,6 +33,7 @@ class SseClient(
3333
3434 fun connect() {
3535 val builder = Request.Builder().url(url).header("Accept", "text/event-stream")
36+ builder.header("User-Agent", "opencode-android/1.0")
3637 if (auth != null) builder.header("Authorization", auth)
3738 source = EventSources.createFactory(client).newEventSource(builder.build(), listener)
3839 }
app/src/main/java/soy/iko/opencode/ui/OpencodeApp.kt+71−7
@@ -2,18 +2,24 @@ package soy.iko.opencode.ui
22
33import androidx.activity.compose.BackHandler
44import androidx.compose.animation.AnimatedContent
5−import androidx.compose.animation.SizeTransform
6−import androidx.compose.animation.fadeIn
7−import androidx.compose.animation.fadeOut
5+import androidx.compose.animation.slideInHorizontally
6+import androidx.compose.animation.slideOutHorizontally
87import androidx.compose.animation.togetherWith
98import androidx.compose.foundation.layout.fillMaxSize
109import androidx.compose.foundation.layout.padding
10+import androidx.compose.material3.AlertDialog
1111import androidx.compose.material3.Scaffold
12+import androidx.compose.material3.SnackbarDuration
1213import androidx.compose.material3.SnackbarHost
1314import androidx.compose.material3.SnackbarHostState
15+import androidx.compose.material3.Text
16+import androidx.compose.material3.TextButton
1417import androidx.compose.runtime.Composable
1518import androidx.compose.runtime.LaunchedEffect
19+import androidx.compose.runtime.getValue
20+import androidx.compose.runtime.mutableStateOf
1621import androidx.compose.runtime.remember
22+import androidx.compose.runtime.setValue
1723import androidx.compose.ui.Modifier
1824import soy.iko.opencode.core.Event
1925import soy.iko.opencode.core.Screen
@@ -25,28 +31,86 @@ import soy.iko.opencode.ui.screens.SessionsScreen
2531@Composable
2632fun OpencodeApp(state: UiState, dispatch: (Event) -> Unit) {
2733 val snackbar = remember { SnackbarHostState() }
34+ var showStopConfirm by remember { mutableStateOf(false) }
35+ var showDisconnectConfirm by remember { mutableStateOf(false) }
2836
2937 LaunchedEffect(state.error) {
3038 state.error?.let {
31− snackbar.showSnackbar(it)
39+ snackbar.showSnackbar(
40+ message = it,
41+ actionLabel = "Dismiss",
42+ duration = SnackbarDuration.Short,
43+ )
44+ // Always clear the error from state after showing.
3245 dispatch(Event.DismissError)
3346 }
3447 }
3548
3649 BackHandler(enabled = state.screen != Screen.Connect) {
3750 when (state.screen) {
38− Screen.Chat -> dispatch(Event.NavigateToSessions)
39− Screen.Sessions -> dispatch(Event.NavigateToConnect)
51+ // Leaving mid-generation would silently drop the reply, so confirm first.
52+ Screen.Chat -> if (state.generating) showStopConfirm = true else dispatch(Event.NavigateToSessions)
53+ // Disconnecting drops the server session, so guard the back button.
54+ Screen.Sessions -> showDisconnectConfirm = true
4055 Screen.Connect -> Unit
4156 }
4257 }
4358
59+ if (showStopConfirm) {
60+ AlertDialog(
61+ onDismissRequest = { showStopConfirm = false },
62+ title = { Text("Stop generation?") },
63+ text = { Text("Leaving will stop the current generation.") },
64+ confirmButton = {
65+ TextButton(onClick = {
66+ showStopConfirm = false
67+ dispatch(Event.CancelGeneration)
68+ dispatch(Event.NavigateToSessions)
69+ }) {
70+ Text("Stop and leave")
71+ }
72+ },
73+ dismissButton = {
74+ TextButton(onClick = { showStopConfirm = false }) {
75+ Text("Stay")
76+ }
77+ },
78+ )
79+ }
80+
81+ if (showDisconnectConfirm) {
82+ AlertDialog(
83+ onDismissRequest = { showDisconnectConfirm = false },
84+ title = { Text("Disconnect?") },
85+ text = { Text("You will be disconnected from the server.") },
86+ confirmButton = {
87+ TextButton(onClick = {
88+ showDisconnectConfirm = false
89+ dispatch(Event.NavigateToConnect)
90+ }) {
91+ Text("Disconnect")
92+ }
93+ },
94+ dismissButton = {
95+ TextButton(onClick = { showDisconnectConfirm = false }) {
96+ Text("Cancel")
97+ }
98+ },
99+ )
100+ }
101+
44102 Scaffold(snackbarHost = { SnackbarHost(snackbar) }) { padding ->
45103 AnimatedContent(
46104 targetState = state.screen,
47105 modifier = Modifier.fillMaxSize().padding(padding),
48106 transitionSpec = {
49− (fadeIn() togetherWith fadeOut()).using(SizeTransform(clip = false))
107+ // Compare screen ordinals to decide slide direction: forward navigation
108+ // (Connect → Sessions → Chat) slides in from the right, back slides from the left.
109+ if (targetState.ordinal > initialState.ordinal) {
110+ slideInHorizontally { it } togetherWith slideOutHorizontally { -it }
111+ } else {
112+ slideInHorizontally { -it } togetherWith slideOutHorizontally { it }
113+ }
50114 },
51115 label = "screen",
52116 ) { screen ->
app/src/main/java/soy/iko/opencode/ui/components/MessageBubble.kt+296−27
@@ -3,6 +3,8 @@ package soy.iko.opencode.ui.components
33import android.content.ClipData
44import android.content.ClipboardManager
55import android.content.Context
6+import android.content.Intent
7+import android.net.Uri
68import android.widget.Toast
79import androidx.compose.foundation.ExperimentalFoundationApi
810import androidx.compose.foundation.background
@@ -13,14 +15,20 @@ import androidx.compose.foundation.layout.Column
1315import androidx.compose.foundation.layout.Row
1416import androidx.compose.foundation.layout.fillMaxWidth
1517import androidx.compose.foundation.layout.padding
18+import androidx.compose.foundation.layout.size
1619import androidx.compose.foundation.layout.widthIn
1720import androidx.compose.foundation.shape.RoundedCornerShape
21+import androidx.compose.foundation.text.selection.SelectionContainer
1822import androidx.compose.material.icons.Icons
1923import androidx.compose.material.icons.filled.Build
24+import androidx.compose.material.icons.filled.Check
2025import androidx.compose.material.icons.filled.ErrorOutline
2126import androidx.compose.material.icons.filled.ExpandLess
2227import androidx.compose.material.icons.filled.ExpandMore
2328import androidx.compose.material.icons.filled.Schedule
29+import androidx.compose.material.icons.filled.SmartToy
30+import androidx.compose.material3.DropdownMenu
31+import androidx.compose.material3.DropdownMenuItem
2432import androidx.compose.material3.Icon
2533import androidx.compose.material3.MaterialTheme
2634import androidx.compose.material3.Text
@@ -34,22 +42,30 @@ import androidx.compose.ui.Modifier
3442import androidx.compose.ui.draw.clip
3543import androidx.compose.ui.graphics.Color
3644import androidx.compose.ui.platform.LocalContext
45+import androidx.compose.ui.text.AnnotatedString
46+import androidx.compose.ui.text.LinkAnnotation
47+import androidx.compose.ui.text.SpanStyle
48+import androidx.compose.ui.text.TextLinkStyles
49+import androidx.compose.ui.text.buildAnnotatedString
50+import androidx.compose.ui.text.font.FontFamily
3751import androidx.compose.ui.text.font.FontStyle
52+import androidx.compose.ui.text.font.FontWeight
53+import androidx.compose.ui.text.withLink
54+import androidx.compose.ui.text.withStyle
55+import androidx.compose.ui.text.style.TextDecoration
3856import androidx.compose.ui.text.style.TextOverflow
3957import androidx.compose.ui.unit.dp
40−import java.text.SimpleDateFormat
41−import java.util.Date
42−import java.util.Locale
58+import androidx.compose.ui.unit.sp
4359import soy.iko.opencode.core.MessageStatus
4460import soy.iko.opencode.core.MessageView
4561import soy.iko.opencode.core.ToolView
4662import soy.iko.opencode.ui.theme.MonoStyle
4763
48−private val timeFormat = SimpleDateFormat("HH:mm", Locale.getDefault())
64+private val timeFormat = java.time.format.DateTimeFormatter.ofPattern("HH:mm")
4965
5066@OptIn(ExperimentalFoundationApi::class)
5167@Composable
52−fun MessageBubble(message: MessageView, modifier: Modifier = Modifier) {
68+fun MessageBubble(message: MessageView, modifier: Modifier = Modifier, onRetry: () -> Unit = {}, compactSpacing: Boolean = false) {
5369 val isUser = message.isUser
5470 val bubbleColor = if (isUser) MaterialTheme.colorScheme.primaryContainer else MaterialTheme.colorScheme.surfaceVariant
5571 val textColor = if (isUser) MaterialTheme.colorScheme.onPrimaryContainer else MaterialTheme.colorScheme.onSurface
@@ -62,13 +78,23 @@ fun MessageBubble(message: MessageView, modifier: Modifier = Modifier) {
6278 )
6379
6480 val context = LocalContext.current
81+ var showMenu by remember { mutableStateOf(false) }
6582
6683 Row(
67− modifier = modifier.fillMaxWidth().padding(horizontal = 12.dp, vertical = 4.dp),
84+ modifier = modifier.fillMaxWidth().padding(horizontal = 12.dp, vertical = if (compactSpacing) 1.dp else 4.dp),
6885 horizontalArrangement = if (isUser) Arrangement.End else Arrangement.Start,
6986 ) {
87+ if (!isUser) {
88+ Icon(
89+ Icons.Filled.SmartToy,
90+ contentDescription = null,
91+ tint = MaterialTheme.colorScheme.primary,
92+ modifier = Modifier.align(Alignment.Top).padding(end = 6.dp, top = 2.dp).size(28.dp),
93+ )
94+ }
7095 Column(
7196 modifier = Modifier
97+ .fillMaxWidth(0.85f)
7298 .widthIn(max = 320.dp)
7399 .clip(shape)
74100 .background(bubbleColor)
@@ -76,30 +102,72 @@ fun MessageBubble(message: MessageView, modifier: Modifier = Modifier) {
76102 onClick = {},
77103 onLongClick = {
78104 if (message.text.isNotEmpty()) {
79− copyToClipboard(context, message.text)
80− Toast.makeText(context, "Copied", Toast.LENGTH_SHORT).show()
105+ showMenu = true
81106 }
82107 },
83108 )
84109 .padding(horizontal = 14.dp, vertical = 10.dp),
85110 ) {
86− message.reasoning?.let { ReasoningBlock(it) }
111+ DropdownMenu(expanded = showMenu, onDismissRequest = { showMenu = false }) {
112+ DropdownMenuItem(
113+ text = { Text("Copy") },
114+ onClick = {
115+ copyToClipboard(context, message.text)
116+ Toast.makeText(context, "Copied", Toast.LENGTH_SHORT).show()
117+ showMenu = false
118+ },
119+ )
120+ DropdownMenuItem(
121+ text = { Text("Share") },
122+ onClick = {
123+ shareText(context, message.text)
124+ showMenu = false
125+ },
126+ )
127+ }
87128
88− message.tools.forEach { ToolRow(it) }
129+ message.reasoning?.let { ReasoningBlock(it, context) }
89130
90− if (message.text.isNotEmpty()) {
131+ // Give a quick overview of how many tools ran before listing them.
132+ if (message.tools.size > 1) {
91133 Text(
92− text = message.text + if (message.streaming) " ▌" else "",
93− color = textColor,
94− style = MaterialTheme.typography.bodyLarge,
134+ text = "${message.tools.size} tool calls",
135+ style = MaterialTheme.typography.labelSmall,
136+ color = MaterialTheme.colorScheme.onSurfaceVariant,
137+ modifier = Modifier.padding(bottom = 4.dp),
95138 )
96− } else if (message.streaming && message.tools.isEmpty() && message.reasoning == null) {
97− Text("▌", color = textColor, style = MaterialTheme.typography.bodyLarge)
139+ }
140+
141+ message.tools.forEach { ToolRow(it) }
142+
143+ SelectionContainer {
144+ if (message.text.isNotEmpty()) {
145+ val linkColor = MaterialTheme.colorScheme.primary
146+ // Tint the code background off the bubble's own foreground colour so it
147+ // contrasts on both bubble types (user bubbles use primaryContainer).
148+ val codeBackground = (if (isUser) MaterialTheme.colorScheme.onPrimaryContainer else MaterialTheme.colorScheme.onSurface).copy(alpha = 0.08f)
149+ val annotatedText = remember(message.text, message.streaming, linkColor, codeBackground, context) {
150+ buildLinkedText(message.text, message.streaming, linkColor, codeBackground, context)
151+ }
152+ Text(
153+ text = annotatedText,
154+ color = textColor,
155+ // Give assistant replies (often long, code-heavy) extra line
156+ // spacing for readability; user bubbles keep the default.
157+ style = if (isUser) {
158+ MaterialTheme.typography.bodyLarge
159+ } else {
160+ MaterialTheme.typography.bodyLarge.copy(lineHeight = 22.sp)
161+ },
162+ )
163+ } else if (message.streaming && message.tools.isEmpty() && message.reasoning == null) {
164+ Text("▌", color = textColor, style = MaterialTheme.typography.bodyLarge)
165+ }
98166 }
99167
100168 if (message.time > 0) {
101169 Text(
102− text = timeFormat.format(Date(message.time)),
170+ text = timeFormat.format(java.time.Instant.ofEpochMilli(message.time).atZone(java.time.ZoneId.systemDefault()).toLocalTime()),
103171 style = MaterialTheme.typography.labelSmall,
104172 color = mutedColor,
105173 modifier = Modifier.padding(top = 4.dp),
@@ -107,7 +175,7 @@ fun MessageBubble(message: MessageView, modifier: Modifier = Modifier) {
107175 }
108176
109177 if (isUser && message.status != MessageStatus.Sent) {
110− StatusLine(message.status)
178+ StatusLine(message.status, onRetry = onRetry)
111179 }
112180 }
113181 }
@@ -118,8 +186,172 @@ private fun copyToClipboard(context: Context, text: String) {
118186 clipboard.setPrimaryClip(ClipData.newPlainText("message", text))
119187}
120188
189+private val urlRegex = Regex("""https?://[^\s]+""")
190+
191+/**
192+ * Render [text] as an AnnotatedString. Triple-backtick fences split the text into
193+ * prose and code segments: code segments are rendered monospace on a tinted
194+ * [codeBackground] (fences dropped), while prose has each http(s) URL turned into a
195+ * clickable link. The streaming cursor (" ▌") is appended outside any annotation.
196+ */
197+private fun buildLinkedText(
198+ text: String,
199+ streaming: Boolean,
200+ linkColor: Color,
201+ codeBackground: Color,
202+ context: Context,
203+): AnnotatedString = buildAnnotatedString {
204+ // Odd-indexed segments sit between a pair of ``` fences, so they're code. An
205+ // unclosed fence leaves the trailing segment styled as code, mirroring markdown.
206+ val segments = text.split("```")
207+ segments.forEachIndexed { index, segment ->
208+ if (index % 2 == 1) {
209+ withStyle(SpanStyle(fontFamily = FontFamily.Monospace, background = codeBackground)) {
210+ append(segment)
211+ }
212+ } else {
213+ appendLinkedText(segment, linkColor, codeBackground, context)
214+ }
215+ }
216+ if (streaming) append(" ▌")
217+}
218+
219+/**
220+ * Append [text] to the builder. Double-asterisk pairs are rendered bold; within each
221+ * segment, single-backtick pairs are rendered as inline code (monospace, backticks
222+ * dropped), and the remaining prose has each http(s) URL turned into a clickable link.
223+ * Triple-backtick fences are already stripped upstream in [buildLinkedText], so only
224+ * inline spans reach here.
225+ */
226+private fun AnnotatedString.Builder.appendLinkedText(
227+ text: String,
228+ linkColor: Color,
229+ codeBackground: Color,
230+ context: Context,
231+) {
232+ // Bold is the outermost inline layer: split on ** first so a bold span can wrap
233+ // inline code and links inside it. Odd-indexed segments sit between a pair of **
234+ // markers, so they're bold (markers dropped); even segments are ordinary prose.
235+ // Only the non-bold (even) segments get single-* italic handling — splitting a bold
236+ // segment on "*" would misfire on the leftover asterisks from the ** markers.
237+ val segments = text.split("**")
238+ // An even segment count means an odd number of ** markers, so the final ** is unclosed:
239+ // its trailing segment isn't bold. Restore the literal ** and render it as ordinary prose.
240+ val unclosed = segments.size % 2 == 0
241+ segments.forEachIndexed { index, segment ->
242+ val isUnclosedTail = unclosed && index == segments.size - 1
243+ if (index % 2 == 1 && !isUnclosedTail) {
244+ withStyle(SpanStyle(fontWeight = FontWeight.Bold)) {
245+ appendInlineCode(segment, linkColor, codeBackground, context)
246+ }
247+ } else {
248+ if (isUnclosedTail) append("**")
249+ appendItalicText(segment, linkColor, codeBackground, context)
250+ }
251+ }
252+}
253+
254+/**
255+ * Append [text] to the builder, rendering single-asterisk pairs as italic. Double
256+ * asterisks are already consumed as bold upstream in [appendLinkedText], so only lone
257+ * `*` markers reach here. Each segment then flows through inline-code and URL handling.
258+ */
259+private fun AnnotatedString.Builder.appendItalicText(
260+ text: String,
261+ linkColor: Color,
262+ codeBackground: Color,
263+ context: Context,
264+) {
265+ // Odd-indexed segments sit between a pair of single asterisks, so they're italic
266+ // (markers dropped); even segments are ordinary prose.
267+ val segments = text.split("*")
268+ // An even segment count means an odd number of * markers, so the final * is unclosed:
269+ // its trailing segment isn't italic. Restore the literal * and render it as prose.
270+ val unclosed = segments.size % 2 == 0
271+ segments.forEachIndexed { index, segment ->
272+ val isUnclosedTail = unclosed && index == segments.size - 1
273+ if (index % 2 == 1 && !isUnclosedTail) {
274+ withStyle(SpanStyle(fontStyle = FontStyle.Italic)) {
275+ appendInlineCode(segment, linkColor, codeBackground, context)
276+ }
277+ } else {
278+ if (isUnclosedTail) append("*")
279+ appendInlineCode(segment, linkColor, codeBackground, context)
280+ }
281+ }
282+}
283+
284+/**
285+ * Append [text] to the builder, rendering single-backtick pairs as inline code
286+ * (monospace, backticks dropped) and turning each http(s) URL in the remaining prose
287+ * into a clickable link.
288+ */
289+private fun AnnotatedString.Builder.appendInlineCode(
290+ text: String,
291+ linkColor: Color,
292+ codeBackground: Color,
293+ context: Context,
294+) {
295+ // Odd-indexed segments sit between a pair of single backticks, so they're inline
296+ // code. Even segments are ordinary prose and get URL linking applied.
297+ val segments = text.split("`")
298+ // An even segment count means an odd number of backtick markers, so the final backtick
299+ // is unclosed: its trailing segment isn't code. Restore the literal ` and treat as prose.
300+ val unclosed = segments.size % 2 == 0
301+ segments.forEachIndexed { index, segment ->
302+ val isUnclosedTail = unclosed && index == segments.size - 1
303+ if (index % 2 == 1 && !isUnclosedTail) {
304+ withStyle(SpanStyle(fontFamily = FontFamily.Monospace, background = codeBackground)) {
305+ append(segment)
306+ }
307+ } else {
308+ if (isUnclosedTail) append("`")
309+ appendUrls(segment, linkColor, context)
310+ }
311+ }
312+}
313+
314+/** Append [text] to the builder, turning each http(s) URL into a clickable link. */
315+private fun AnnotatedString.Builder.appendUrls(
316+ text: String,
317+ linkColor: Color,
318+ context: Context,
319+) {
320+ val matches = urlRegex.findAll(text).toList()
321+ var cursor = 0
322+ for (match in matches) {
323+ append(text.substring(cursor, match.range.first))
324+ val url = match.value
325+ withLink(
326+ LinkAnnotation.Clickable(
327+ tag = url,
328+ styles = TextLinkStyles(
329+ SpanStyle(color = linkColor, textDecoration = TextDecoration.Underline),
330+ ),
331+ linkInteractionListener = {
332+ runCatching {
333+ context.startActivity(Intent(Intent.ACTION_VIEW, Uri.parse(url)))
334+ }
335+ },
336+ ),
337+ ) {
338+ append(url)
339+ }
340+ cursor = match.range.last + 1
341+ }
342+ append(text.substring(cursor))
343+}
344+
345+private fun shareText(context: Context, text: String) {
346+ val intent = Intent(Intent.ACTION_SEND).apply {
347+ type = "text/plain"
348+ putExtra(Intent.EXTRA_TEXT, text)
349+ }
350+ context.startActivity(Intent.createChooser(intent, null))
351+}
352+
121353@Composable
122−private fun ReasoningBlock(text: String) {
354+private fun ReasoningBlock(text: String, context: Context) {
123355 var expanded by remember { mutableStateOf(false) }
124356 Column(modifier = Modifier.padding(bottom = 6.dp)) {
125357 Row(
@@ -145,6 +377,17 @@ private fun ReasoningBlock(text: String) {
145377 color = MaterialTheme.colorScheme.onSurfaceVariant,
146378 modifier = Modifier.padding(top = 4.dp),
147379 )
380+ Text(
381+ text = "Copy",
382+ style = MaterialTheme.typography.labelSmall,
383+ color = MaterialTheme.colorScheme.primary,
384+ modifier = Modifier
385+ .padding(top = 4.dp)
386+ .clickable {
387+ copyToClipboard(context, text)
388+ Toast.makeText(context, "Copied", Toast.LENGTH_SHORT).show()
389+ },
390+ )
148391 }
149392 }
150393}
@@ -152,6 +395,13 @@ private fun ReasoningBlock(text: String) {
152395@Composable
153396private fun ToolRow(tool: ToolView) {
154397 var expanded by remember { mutableStateOf(false) }
398+ val baseColor = MaterialTheme.colorScheme.onSurfaceVariant
399+ val statusColor = when (tool.status) {
400+ "completed", "success" -> MaterialTheme.colorScheme.primary
401+ "error", "failed" -> MaterialTheme.colorScheme.error
402+ "pending", "running", "in_progress" -> MaterialTheme.colorScheme.onSurfaceVariant
403+ else -> MaterialTheme.colorScheme.onSurfaceVariant
404+ }
155405 Row(
156406 verticalAlignment = Alignment.CenterVertically,
157407 modifier = Modifier
@@ -166,14 +416,23 @@ private fun ToolRow(tool: ToolView) {
166416 modifier = Modifier.padding(end = 6.dp).widthIn(max = 16.dp),
167417 )
168418 Text(
169− text = buildString {
170− append(tool.name)
171− append(" · ")
172− append(tool.status)
173− if (expanded) tool.title?.let { append(" — "); append(it) }
419+ text = buildAnnotatedString {
420+ withStyle(SpanStyle(color = baseColor)) {
421+ append(tool.name)
422+ append(" · ")
423+ }
424+ withStyle(SpanStyle(color = statusColor)) {
425+ append(tool.status)
426+ }
427+ if (expanded) tool.title?.let {
428+ withStyle(SpanStyle(color = baseColor)) {
429+ append(" — ")
430+ append(it)
431+ }
432+ }
174433 },
175434 style = MonoStyle,
176− color = MaterialTheme.colorScheme.onSurfaceVariant,
435+ color = baseColor,
177436 maxLines = if (expanded) Int.MAX_VALUE else 1,
178437 overflow = TextOverflow.Ellipsis,
179438 modifier = Modifier.weight(1f),
@@ -188,11 +447,11 @@ private fun ToolRow(tool: ToolView) {
188447}
189448
190449@Composable
191−private fun StatusLine(status: MessageStatus) {
450+private fun StatusLine(status: MessageStatus, onRetry: () -> Unit) {
192451 val (icon, label, tint) = when (status) {
193452 MessageStatus.Pending -> Triple(Icons.Filled.Schedule, "Sending…", MaterialTheme.colorScheme.onPrimaryContainer)
194453 MessageStatus.Failed -> Triple(Icons.Filled.ErrorOutline, "Failed to send", MaterialTheme.colorScheme.error)
195− MessageStatus.Sent -> Triple(Icons.Filled.Schedule, "", Color.Unspecified)
454+ MessageStatus.Sent -> Triple(Icons.Filled.Check, "", Color.Unspecified)
196455 }
197456 Row(
198457 verticalAlignment = Alignment.CenterVertically,
@@ -206,5 +465,15 @@ private fun StatusLine(status: MessageStatus) {
206465 color = tint,
207466 modifier = Modifier.padding(start = 4.dp),
208467 )
468+ if (status == MessageStatus.Failed) {
469+ Text(
470+ text = "Retry",
471+ style = MaterialTheme.typography.labelSmall,
472+ color = MaterialTheme.colorScheme.primary,
473+ modifier = Modifier
474+ .padding(start = 12.dp)
475+ .clickable { onRetry() },
476+ )
477+ }
209478 }
210479}
app/src/main/java/soy/iko/opencode/ui/screens/ChatScreen.kt+400−33
@@ -1,43 +1,87 @@
11package soy.iko.opencode.ui.screens
22
3+import android.content.ClipData
4+import android.content.ClipboardManager
5+import android.content.Context
6+import android.widget.Toast
7+import androidx.compose.animation.AnimatedVisibility
8+import androidx.compose.animation.core.RepeatMode
9+import androidx.compose.animation.core.animateFloat
10+import androidx.compose.animation.core.infiniteRepeatable
11+import androidx.compose.animation.core.rememberInfiniteTransition
12+import androidx.compose.animation.core.tween
13+import androidx.compose.animation.fadeIn
14+import androidx.compose.animation.fadeOut
15+import androidx.compose.animation.scaleIn
16+import androidx.compose.animation.scaleOut
17+import androidx.compose.foundation.ExperimentalFoundationApi
18+import androidx.compose.foundation.background
19+import androidx.compose.foundation.clickable
20+import androidx.compose.foundation.combinedClickable
321import androidx.compose.foundation.layout.Arrangement
422import androidx.compose.foundation.layout.Box
23+import androidx.compose.foundation.layout.BoxScope
524import androidx.compose.foundation.layout.Column
625import androidx.compose.foundation.layout.Row
26+import androidx.compose.foundation.layout.Spacer
727import androidx.compose.foundation.layout.fillMaxSize
828import androidx.compose.foundation.layout.fillMaxWidth
29+import androidx.compose.foundation.layout.heightIn
930import androidx.compose.foundation.layout.PaddingValues
31+import androidx.compose.foundation.layout.WindowInsets
32+import androidx.compose.foundation.layout.ime
1033import androidx.compose.foundation.layout.imePadding
1134import androidx.compose.foundation.layout.padding
1235import androidx.compose.foundation.layout.size
1336import androidx.compose.foundation.lazy.LazyColumn
14−import androidx.compose.foundation.lazy.items
37+import androidx.compose.foundation.lazy.itemsIndexed
1538import androidx.compose.foundation.lazy.rememberLazyListState
39+import androidx.compose.foundation.shape.CircleShape
40+import androidx.compose.foundation.shape.RoundedCornerShape
41+import androidx.compose.foundation.text.KeyboardActions
42+import androidx.compose.foundation.text.KeyboardOptions
1643import androidx.compose.material.icons.Icons
1744import androidx.compose.material.icons.automirrored.filled.ArrowBack
1845import androidx.compose.material.icons.automirrored.filled.Send
46+import androidx.compose.material.icons.filled.ChatBubbleOutline
47+import androidx.compose.material.icons.filled.Close
1948import androidx.compose.material.icons.filled.KeyboardArrowDown
49+import androidx.compose.material.icons.filled.KeyboardArrowUp
2050import androidx.compose.material.icons.filled.Stop
2151import androidx.compose.material3.CircularProgressIndicator
2252import androidx.compose.material3.ExperimentalMaterial3Api
2353import androidx.compose.material3.FilledIconButton
54+import androidx.compose.material3.FilledTonalButton
2455import androidx.compose.material3.FloatingActionButton
2556import androidx.compose.material3.Icon
2657import androidx.compose.material3.IconButton
58+import androidx.compose.material3.LinearProgressIndicator
2759import androidx.compose.material3.MaterialTheme
2860import androidx.compose.material3.OutlinedTextField
2961import androidx.compose.material3.Scaffold
3062import androidx.compose.material3.Surface
3163import androidx.compose.material3.Text
64+import androidx.compose.material3.TextButton
3265import androidx.compose.material3.TopAppBar
3366import androidx.compose.runtime.Composable
3467import androidx.compose.runtime.LaunchedEffect
3568import androidx.compose.runtime.derivedStateOf
3669import androidx.compose.runtime.getValue
70+import androidx.compose.runtime.mutableStateOf
3771import androidx.compose.runtime.remember
3872import androidx.compose.runtime.rememberCoroutineScope
73+import androidx.compose.runtime.setValue
3974import androidx.compose.ui.Alignment
4075import androidx.compose.ui.Modifier
76+import androidx.compose.ui.draw.clip
77+import androidx.compose.ui.hapticfeedback.HapticFeedbackType
78+import androidx.compose.ui.platform.LocalClipboardManager
79+import androidx.compose.ui.platform.LocalContext
80+import androidx.compose.ui.platform.LocalDensity
81+import androidx.compose.ui.platform.LocalHapticFeedback
82+import androidx.compose.ui.text.font.FontStyle
83+import androidx.compose.ui.text.input.ImeAction
84+import androidx.compose.ui.text.style.TextAlign
4185import androidx.compose.ui.text.style.TextOverflow
4286import androidx.compose.ui.unit.dp
4387import kotlinx.coroutines.launch
@@ -45,11 +89,17 @@ import soy.iko.opencode.core.Event
4589import soy.iko.opencode.core.UiState
4690import soy.iko.opencode.ui.components.MessageBubble
4791
48−@OptIn(ExperimentalMaterial3Api::class)
92+@OptIn(ExperimentalMaterial3Api::class, ExperimentalFoundationApi::class)
4993@Composable
5094fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
5195 val listState = rememberLazyListState()
5296 val coroutineScope = rememberCoroutineScope()
97+ val context = LocalContext.current
98+
99+ // Hide the jump-to-bottom FAB while the keyboard is up: it would otherwise
100+ // sit over the top of the IME. Reading the ime inset bottom during composition
101+ // observes snapshot state, so this flips as the keyboard opens/closes.
102+ val isImeVisible = WindowInsets.ime.getBottom(LocalDensity.current) > 0
53103
54104 // Show a jump-to-bottom button once the user scrolls up away from the latest
55105 // message. derivedStateOf keeps this from recomposing on every pixel of scroll —
@@ -63,6 +113,22 @@ fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
63113 }
64114 }
65115
116+ // Offer a jump-to-top shortcut once the conversation is long and the user has
117+ // scrolled a few messages down — saves flick-scrolling back through history.
118+ val showScrollTop by remember {
119+ derivedStateOf { state.messages.size > 10 && listState.firstVisibleItemIndex > 3 }
120+ }
121+
122+ // Track how many messages the user has "seen" (i.e. was at the bottom for). While
123+ // they're scrolled up, any messages beyond that count are unseen — flag the FAB.
124+ var lastSeenCount by remember { mutableStateOf(0) }
125+ LaunchedEffect(showScrollButton, state.messages.size) {
126+ if (!showScrollButton) {
127+ lastSeenCount = state.messages.size
128+ }
129+ }
130+ val hasUnseenMessages = showScrollButton && state.messages.size > lastSeenCount
131+
66132 // Keep the newest content in view as it streams in, but only when the user is
67133 // already near the bottom. If they've scrolled up to read history, leave it alone.
68134 val lastLen = state.messages.lastOrNull()?.text?.length ?: 0
@@ -82,10 +148,20 @@ fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
82148 topBar = {
83149 TopAppBar(
84150 title = {
151+ val title = state.currentSessionTitle.ifEmpty { "Chat" }
85152 Text(
86− text = state.currentSessionTitle.ifEmpty { "Chat" },
153+ text = if (state.messages.isNotEmpty()) "$title (${state.messages.size})" else title,
87154 maxLines = 1,
88155 overflow = TextOverflow.Ellipsis,
156+ modifier = Modifier.combinedClickable(
157+ onClick = { dispatch(Event.NavigateToSessions) },
158+ onLongClick = {
159+ state.currentSessionId?.let { id ->
160+ copyToClipboard(context, id)
161+ Toast.makeText(context, "Session ID copied", Toast.LENGTH_SHORT).show()
162+ }
163+ },
164+ ),
89165 )
90166 },
91167 navigationIcon = {
@@ -107,6 +183,7 @@ fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
107183 ChatInputBar(
108184 draft = state.draft,
109185 generating = state.generating,
186+ loading = state.loading && state.messages.isEmpty(),
110187 onDraftChange = { dispatch(Event.DraftChanged(it)) },
111188 onSend = { dispatch(Event.SendMessage(state.draft)) },
112189 onStop = { dispatch(Event.CancelGeneration) },
@@ -117,43 +194,239 @@ fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
117194 if (!state.sseConnected && !state.generating) {
118195 ReconnectingBanner()
119196 }
197+ if (state.loading && state.messages.isEmpty()) {
198+ LinearProgressIndicator(Modifier.fillMaxWidth())
199+ }
200+ if (state.error != null && state.messages.isNotEmpty()) {
201+ ErrorBanner(
202+ message = state.error,
203+ onDismiss = { dispatch(Event.DismissError) },
204+ )
205+ }
120206 Box(Modifier.fillMaxWidth().weight(1f)) {
121207 if (state.messages.isEmpty() && !state.loading) {
122− Text(
123− text = "Send a message to start the conversation.",
124− style = MaterialTheme.typography.bodyMedium,
125− color = MaterialTheme.colorScheme.onSurfaceVariant,
208+ Column(
126209 modifier = Modifier.align(Alignment.Center).padding(32.dp),
127− )
210+ horizontalAlignment = Alignment.CenterHorizontally,
211+ ) {
212+ Icon(
213+ Icons.Filled.ChatBubbleOutline,
214+ contentDescription = null,
215+ modifier = Modifier.size(48.dp),
216+ tint = MaterialTheme.colorScheme.primary.copy(alpha = 0.5f),
217+ )
218+ Spacer(Modifier.size(16.dp))
219+ Text(
220+ text = "Send a message to start the conversation.",
221+ style = MaterialTheme.typography.bodyMedium,
222+ color = MaterialTheme.colorScheme.onSurfaceVariant,
223+ textAlign = TextAlign.Center,
224+ )
225+ }
128226 }
129227 LazyColumn(
130228 state = listState,
131229 modifier = Modifier.fillMaxSize(),
132230 contentPadding = PaddingValues(vertical = 8.dp),
133231 ) {
134− items(state.messages, key = { it.id }) { message ->
135− MessageBubble(message)
232+ itemsIndexed(state.messages, key = { _, message -> message.id }) { index, message ->
233+ // Insert a date separator whenever a message falls on a different
234+ // calendar day than the one before it (and above the first message).
235+ val previous = state.messages.getOrNull(index - 1)
236+ if (shouldShowDateSeparator(previous, message)) {
237+ DateSeparator(message.time)
238+ }
239+ val sameSenderAsPrevious = index > 0 && state.messages[index - 1].isUser == message.isUser
240+ MessageBubble(
241+ message,
242+ onRetry = { dispatch(Event.SendMessage(message.text)) },
243+ compactSpacing = sameSenderAsPrevious,
244+ )
245+ }
246+ // While the assistant is spinning up but hasn't emitted a streaming
247+ // message yet (only the user's message is present), show a typing
248+ // indicator so the wait doesn't look like a stall.
249+ if (state.generating && state.messages.none { it.streaming }) {
250+ item(key = "typing-indicator") {
251+ TypingIndicator()
252+ }
136253 }
137254 }
138− if (showScrollButton) {
139− FloatingActionButton(
140− onClick = {
141− coroutineScope.launch {
142− if (state.messages.isNotEmpty()) {
143− listState.animateScrollToItem(state.messages.size - 1)
144− }
255+ ScrollToBottomButton(
256+ visible = showScrollButton && !isImeVisible,
257+ showBadge = hasUnseenMessages,
258+ onClick = {
259+ coroutineScope.launch {
260+ if (state.messages.isNotEmpty()) {
261+ // When the typing indicator is showing it occupies the item
262+ // just past the last message, so scroll one further to reach it.
263+ val target = if (state.generating && state.messages.none { it.streaming })
264+ state.messages.size
265+ else
266+ state.messages.size - 1
267+ listState.animateScrollToItem(target)
145268 }
146− },
269+ }
270+ },
271+ )
272+ ScrollToTopButton(
273+ visible = showScrollTop,
274+ onClick = {
275+ coroutineScope.launch {
276+ listState.animateScrollToItem(0)
277+ }
278+ },
279+ )
280+ }
281+ }
282+ }
283+}
284+
285+// Extracted into a BoxScope extension so the top-level AnimatedVisibility overload
286+// resolves cleanly. Called inline inside a Column, the enclosing ColumnScope receiver
287+// would otherwise capture the ColumnScope.AnimatedVisibility overload and fail to compile.
288+@Composable
289+private fun BoxScope.ScrollToBottomButton(visible: Boolean, showBadge: Boolean, onClick: () -> Unit) {
290+ AnimatedVisibility(
291+ visible = visible,
292+ modifier = Modifier.align(Alignment.BottomEnd),
293+ enter = scaleIn() + fadeIn(),
294+ exit = scaleOut() + fadeOut(),
295+ ) {
296+ Box(Modifier.padding(16.dp)) {
297+ FloatingActionButton(
298+ onClick = onClick,
299+ modifier = Modifier.size(44.dp),
300+ ) {
301+ Icon(
302+ Icons.Filled.KeyboardArrowDown,
303+ contentDescription = "Scroll to latest message",
304+ )
305+ }
306+ if (showBadge) {
307+ Box(
308+ modifier = Modifier
309+ .align(Alignment.TopEnd)
310+ .size(8.dp)
311+ .clip(CircleShape)
312+ .background(MaterialTheme.colorScheme.error),
313+ )
314+ }
315+ }
316+ }
317+}
318+
319+// Mirror of ScrollToBottomButton, pinned to the top-start corner. Same BoxScope
320+// extension trick so the top-level AnimatedVisibility overload resolves cleanly.
321+@Composable
322+private fun BoxScope.ScrollToTopButton(visible: Boolean, onClick: () -> Unit) {
323+ AnimatedVisibility(
324+ visible = visible,
325+ modifier = Modifier.align(Alignment.TopStart),
326+ enter = scaleIn() + fadeIn(),
327+ exit = scaleOut() + fadeOut(),
328+ ) {
329+ Box(Modifier.padding(16.dp)) {
330+ FloatingActionButton(
331+ onClick = onClick,
332+ modifier = Modifier.size(44.dp),
333+ ) {
334+ Icon(
335+ Icons.Filled.KeyboardArrowUp,
336+ contentDescription = "Scroll to top",
337+ )
338+ }
339+ }
340+ }
341+}
342+
343+private fun copyToClipboard(context: Context, text: String) {
344+ val clipboard = context.getSystemService(Context.CLIPBOARD_SERVICE) as ClipboardManager
345+ clipboard.setPrimaryClip(ClipData.newPlainText("session", text))
346+}
347+
348+private val dateSeparatorFormat = java.time.format.DateTimeFormatter.ofPattern("MMM d")
349+
350+/** Convert an epoch-millis timestamp to the local calendar day it falls on. */
351+private fun localDayOf(epochMillis: Long): java.time.LocalDate =
352+ java.time.Instant.ofEpochMilli(epochMillis)
353+ .atZone(java.time.ZoneId.systemDefault())
354+ .toLocalDate()
355+
356+/**
357+ * A separator is shown above a message when it starts a new calendar day — i.e. the
358+ * first timestamped message, or one whose day differs from the previous message's.
359+ * Messages without a valid time (time <= 0) never get one.
360+ */
361+private fun shouldShowDateSeparator(previous: soy.iko.opencode.core.MessageView?, current: soy.iko.opencode.core.MessageView): Boolean {
362+ if (current.time <= 0) return false
363+ if (previous == null || previous.time <= 0) return true
364+ return localDayOf(previous.time) != localDayOf(current.time)
365+}
366+
367+/** Human-friendly day label: "Today", "Yesterday", or "MMM d" (e.g. "Jul 8"). */
368+private fun dateSeparatorLabel(day: java.time.LocalDate): String {
369+ val today = java.time.LocalDate.now()
370+ return when (day) {
371+ today -> "Today"
372+ today.minusDays(1) -> "Yesterday"
373+ else -> day.format(dateSeparatorFormat)
374+ }
375+}
376+
377+@Composable
378+private fun DateSeparator(epochMillis: Long) {
379+ Text(
380+ text = dateSeparatorLabel(localDayOf(epochMillis)),
381+ style = MaterialTheme.typography.labelSmall,
382+ color = MaterialTheme.colorScheme.onSurfaceVariant,
383+ textAlign = TextAlign.Center,
384+ modifier = Modifier.fillMaxWidth().padding(vertical = 8.dp),
385+ )
386+}
387+
388+// Assistant-style bubble with three sequentially-pulsing dots, shown as a stand-in
389+// until the real streaming reply starts. Mirrors MessageBubble's left-aligned,
390+// surfaceVariant look so it reads as an incoming message.
391+@Composable
392+private fun TypingIndicator() {
393+ val shape = RoundedCornerShape(topStart = 16.dp, topEnd = 16.dp, bottomStart = 4.dp, bottomEnd = 16.dp)
394+ val transition = rememberInfiniteTransition(label = "typing")
395+ Row(
396+ modifier = Modifier.fillMaxWidth().padding(horizontal = 12.dp, vertical = 4.dp),
397+ horizontalArrangement = Arrangement.Start,
398+ ) {
399+ Row(
400+ modifier = Modifier
401+ .clip(shape)
402+ .background(MaterialTheme.colorScheme.surfaceVariant)
403+ .padding(horizontal = 14.dp, vertical = 12.dp),
404+ verticalAlignment = Alignment.CenterVertically,
405+ horizontalArrangement = Arrangement.spacedBy(6.dp),
406+ ) {
407+ Text(
408+ text = "Thinking",
409+ style = MaterialTheme.typography.bodyMedium.copy(fontStyle = FontStyle.Italic),
410+ color = MaterialTheme.colorScheme.onSurfaceVariant,
411+ )
412+ Row(horizontalArrangement = Arrangement.spacedBy(4.dp)) {
413+ // Stagger the three dots so they fade in and out in sequence.
414+ listOf(0, 150, 300).forEach { staggerMs ->
415+ val alpha by transition.animateFloat(
416+ initialValue = 0.2f,
417+ targetValue = 1f,
418+ animationSpec = infiniteRepeatable(
419+ animation = tween(durationMillis = 600, delayMillis = staggerMs),
420+ repeatMode = RepeatMode.Reverse,
421+ ),
422+ label = "dot",
423+ )
424+ Box(
147425 modifier = Modifier
148− .align(Alignment.BottomEnd)
149− .padding(16.dp)
150− .size(44.dp),
151− ) {
152− Icon(
153− Icons.Filled.KeyboardArrowDown,
154− contentDescription = "Scroll to latest message",
155− )
156− }
426+ .size(6.dp)
427+ .clip(CircleShape)
428+ .background(MaterialTheme.colorScheme.onSurfaceVariant.copy(alpha = alpha)),
429+ )
157430 }
158431 }
159432 }
@@ -185,39 +458,133 @@ private fun ReconnectingBanner() {
185458 }
186459}
187460
461+@Composable
462+private fun ErrorBanner(message: String, onDismiss: () -> Unit) {
463+ Surface(
464+ modifier = Modifier.fillMaxWidth(),
465+ color = MaterialTheme.colorScheme.errorContainer,
466+ ) {
467+ Row(
468+ modifier = Modifier.fillMaxWidth().padding(start = 16.dp, top = 6.dp, bottom = 6.dp, end = 4.dp),
469+ verticalAlignment = Alignment.CenterVertically,
470+ horizontalArrangement = Arrangement.spacedBy(8.dp),
471+ ) {
472+ Text(
473+ text = message,
474+ style = MaterialTheme.typography.bodySmall,
475+ color = MaterialTheme.colorScheme.onErrorContainer,
476+ modifier = Modifier.weight(1f),
477+ )
478+ IconButton(onClick = onDismiss, modifier = Modifier.size(32.dp)) {
479+ Icon(
480+ Icons.Filled.Close,
481+ contentDescription = "Dismiss error",
482+ tint = MaterialTheme.colorScheme.onErrorContainer,
483+ modifier = Modifier.size(18.dp),
484+ )
485+ }
486+ }
487+ }
488+}
489+
188490@Composable
189491private fun ChatInputBar(
190492 draft: String,
191493 generating: Boolean,
494+ loading: Boolean,
192495 onDraftChange: (String) -> Unit,
193496 onSend: () -> Unit,
194497 onStop: () -> Unit,
195498) {
499+ val haptic = LocalHapticFeedback.current
500+ val clipboardManager = LocalClipboardManager.current
196501 Surface(tonalElevation = 3.dp) {
197− Row(
502+ Column(
198503 modifier = Modifier.fillMaxWidth().imePadding().padding(horizontal = 12.dp, vertical = 8.dp),
504+ ) {
505+ Row(
506+ modifier = Modifier.fillMaxWidth(),
199507 verticalAlignment = Alignment.Bottom,
200508 horizontalArrangement = Arrangement.spacedBy(8.dp),
201509 ) {
202510 OutlinedTextField(
203511 value = draft,
204512 onValueChange = onDraftChange,
205− modifier = Modifier.weight(1f),
513+ modifier = Modifier.weight(1f).heightIn(min = 48.dp),
514+ enabled = !loading,
206515 placeholder = { Text("Message opencode…") },
207516 maxLines = 5,
517+ // Offer a quick way to wipe the draft once there's something to clear
518+ // and we're not mid-generation (where the field is a poor place to fuss).
519+ trailingIcon = if (draft.isNotEmpty() && !generating) {
520+ {
521+ IconButton(
522+ onClick = { onDraftChange("") },
523+ modifier = Modifier.size(20.dp),
524+ ) {
525+ Icon(
526+ Icons.Filled.Close,
527+ contentDescription = "Clear draft",
528+ modifier = Modifier.size(16.dp),
529+ )
530+ }
531+ }
532+ } else {
533+ null
534+ },
535+ // Surface a running count once the draft gets long, so the user has a
536+ // sense of scale before sending a wall of text.
537+ supportingText = if (draft.length > 500) {
538+ {
539+ Text(
540+ text = "${draft.length} characters",
541+ style = MaterialTheme.typography.labelSmall,
542+ color = MaterialTheme.colorScheme.onSurfaceVariant,
543+ textAlign = TextAlign.End,
544+ modifier = Modifier.fillMaxWidth(),
545+ )
546+ }
547+ } else {
548+ null
549+ },
550+ keyboardOptions = KeyboardOptions(imeAction = ImeAction.Send),
551+ keyboardActions = KeyboardActions(onSend = {
552+ if (!generating && !loading && draft.isNotBlank()) onSend()
553+ }),
208554 )
209555 if (generating) {
210− FilledIconButton(onClick = onStop) {
211− Icon(Icons.Filled.Stop, contentDescription = "Stop generating")
556+ FilledTonalButton(onClick = onStop) {
557+ Icon(
558+ Icons.Filled.Stop,
559+ contentDescription = null,
560+ modifier = Modifier.size(18.dp),
561+ )
562+ Spacer(Modifier.size(6.dp))
563+ Text("Stop")
212564 }
213565 } else {
214566 FilledIconButton(
215− onClick = onSend,
216− enabled = draft.isNotBlank(),
567+ onClick = {
568+ haptic.performHapticFeedback(HapticFeedbackType.LongPress)
569+ onSend()
570+ },
571+ enabled = draft.isNotBlank() && !loading,
217572 ) {
218573 Icon(Icons.AutoMirrored.Filled.Send, contentDescription = "Send")
219574 }
220575 }
221576 }
577+ // With nothing typed yet, offer a one-tap way to drop in clipboard contents.
578+ if (draft.isEmpty() && !generating && !loading) {
579+ TextButton(
580+ onClick = {
581+ clipboardManager.getText()?.text?.let { onDraftChange(it) }
582+ },
583+ contentPadding = PaddingValues(0.dp),
584+ ) {
585+ Text("Paste")
586+ }
587+ }
588+ }
222589 }
223590}
app/src/main/java/soy/iko/opencode/ui/screens/ConnectScreen.kt+100−4
@@ -8,6 +8,7 @@ import androidx.compose.foundation.layout.fillMaxWidth
88import androidx.compose.foundation.layout.height
99import androidx.compose.foundation.layout.imePadding
1010import androidx.compose.foundation.layout.padding
11+import androidx.compose.foundation.layout.size
1112import androidx.compose.foundation.rememberScrollState
1213import androidx.compose.foundation.text.KeyboardActions
1314import androidx.compose.foundation.text.KeyboardOptions
@@ -15,18 +16,30 @@ import androidx.compose.foundation.verticalScroll
1516import androidx.compose.material.icons.Icons
1617import androidx.compose.material.icons.filled.Lock
1718import androidx.compose.material.icons.filled.Person
19+import androidx.compose.material.icons.filled.Terminal
20+import androidx.compose.material.icons.filled.Visibility
21+import androidx.compose.material.icons.filled.VisibilityOff
1822import androidx.compose.material3.Button
1923import androidx.compose.material3.CircularProgressIndicator
2024import androidx.compose.material3.Icon
25+import androidx.compose.material3.IconButton
2126import androidx.compose.material3.MaterialTheme
2227import androidx.compose.material3.OutlinedTextField
2328import androidx.compose.material3.Text
2429import androidx.compose.runtime.Composable
30+import androidx.compose.runtime.LaunchedEffect
31+import androidx.compose.runtime.getValue
32+import androidx.compose.runtime.mutableStateOf
33+import androidx.compose.runtime.remember
34+import androidx.compose.runtime.setValue
2535import androidx.compose.ui.Alignment
2636import androidx.compose.ui.Modifier
37+import androidx.compose.ui.focus.FocusRequester
38+import androidx.compose.ui.focus.focusRequester
2739import androidx.compose.ui.text.input.ImeAction
2840import androidx.compose.ui.text.input.KeyboardType
2941import androidx.compose.ui.text.input.PasswordVisualTransformation
42+import androidx.compose.ui.text.input.VisualTransformation
3043import androidx.compose.ui.text.style.TextAlign
3144import androidx.compose.ui.unit.dp
3245import soy.iko.opencode.core.Event
@@ -34,6 +47,15 @@ import soy.iko.opencode.core.UiState
3447
3548@Composable
3649fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
50+ val focusRequester = remember { FocusRequester() }
51+ var passwordVisible by remember { mutableStateOf(false) }
52+
53+ // Focus the server URL field when the screen opens, but only when it's empty
54+ // so we don't steal focus (and pop the keyboard) on a return visit.
55+ LaunchedEffect(Unit) {
56+ if (state.serverUrl.isEmpty()) focusRequester.requestFocus()
57+ }
58+
3759 Column(
3860 modifier = Modifier
3961 .fillMaxSize()
@@ -43,6 +65,13 @@ fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
4365 verticalArrangement = Arrangement.Center,
4466 horizontalAlignment = Alignment.CenterHorizontally,
4567 ) {
68+ Icon(
69+ Icons.Filled.Terminal,
70+ contentDescription = null,
71+ tint = MaterialTheme.colorScheme.primary,
72+ modifier = Modifier.size(72.dp),
73+ )
74+ Spacer(Modifier.height(16.dp))
4675 Text(
4776 text = "opencode",
4877 style = MaterialTheme.typography.displaySmall,
@@ -63,7 +92,7 @@ fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
6392 label = { Text("Server URL") },
6493 placeholder = { Text("http://192.168.1.10:4096") },
6594 singleLine = true,
66− modifier = Modifier.fillMaxWidth(),
95+ modifier = Modifier.fillMaxWidth().focusRequester(focusRequester),
6796 keyboardOptions = KeyboardOptions(
6897 keyboardType = KeyboardType.Uri,
6998 imeAction = if (state.authRequired) ImeAction.Next else ImeAction.Go,
@@ -72,6 +101,30 @@ fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
72101 enabled = !state.loading,
73102 )
74103
104+ // Point first-time users at the expected address format. Hidden once they
105+ // start typing so it doesn't compete with the field's own contents.
106+ if (state.serverUrl.isEmpty()) {
107+ Spacer(Modifier.height(6.dp))
108+ Text(
109+ text = "Enter the URL of your opencode server (e.g. http://192.168.1.10:4096)",
110+ style = MaterialTheme.typography.bodySmall,
111+ color = MaterialTheme.colorScheme.onSurfaceVariant,
112+ textAlign = TextAlign.Center,
113+ )
114+ }
115+
116+ // Nudge the user if the URL looks like it's missing a scheme — a bare host:port
117+ // won't connect. Informational (tertiary), not an error, since it's a heuristic.
118+ if (state.serverUrl.isNotBlank() && "://" !in state.serverUrl && !state.serverUrl.startsWith("http")) {
119+ Spacer(Modifier.height(6.dp))
120+ Text(
121+ text = "Tip: Add http:// if connecting to a local server",
122+ style = MaterialTheme.typography.bodySmall,
123+ color = MaterialTheme.colorScheme.tertiary,
124+ textAlign = TextAlign.Center,
125+ )
126+ }
127+
75128 if (state.authRequired) {
76129 Spacer(Modifier.height(12.dp))
77130 OutlinedTextField(
@@ -90,11 +143,27 @@ fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
90143 onValueChange = { dispatch(Event.PasswordChanged(it)) },
91144 label = { Text("Password") },
92145 singleLine = true,
93− visualTransformation = PasswordVisualTransformation(),
146+ visualTransformation = if (passwordVisible) VisualTransformation.None else PasswordVisualTransformation(),
94147 leadingIcon = { Icon(Icons.Filled.Lock, contentDescription = null) },
148+ trailingIcon = {
149+ IconButton(onClick = { passwordVisible = !passwordVisible }) {
150+ Icon(
151+ imageVector = if (passwordVisible) Icons.Filled.VisibilityOff else Icons.Filled.Visibility,
152+ contentDescription = if (passwordVisible) "Hide password" else "Show password",
153+ )
154+ }
155+ },
95156 modifier = Modifier.fillMaxWidth(),
96157 keyboardOptions = KeyboardOptions(keyboardType = KeyboardType.Password, imeAction = ImeAction.Go),
97− keyboardActions = KeyboardActions(onGo = { dispatch(Event.Connect) }),
158+ keyboardActions = KeyboardActions(onGo = {
159+ // Mirror the connect button's enabled condition so the keyboard's
160+ // Go action can't fire a request the form isn't ready for.
161+ if (!state.loading && state.serverUrl.isNotBlank() &&
162+ (!state.authRequired || state.username.isNotBlank() || state.password.isNotBlank())
163+ ) {
164+ dispatch(Event.Connect)
165+ }
166+ }),
98167 enabled = !state.loading,
99168 )
100169 Spacer(Modifier.height(4.dp))
@@ -109,7 +178,10 @@ fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
109178
110179 Button(
111180 onClick = { dispatch(Event.Connect) },
112− enabled = !state.loading && state.serverUrl.isNotBlank(),
181+ // When auth is required, keep the button disabled until at least one
182+ // credential is entered — otherwise it just triggers a validation error.
183+ enabled = !state.loading && state.serverUrl.isNotBlank() &&
184+ (!state.authRequired || state.username.isNotBlank() || state.password.isNotBlank()),
113185 modifier = Modifier.fillMaxWidth(),
114186 ) {
115187 if (state.loading) {
@@ -122,5 +194,29 @@ fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
122194 Text(if (state.authRequired) "Sign in" else "Connect")
123195 }
124196 }
197+
198+ // While connecting (including auto-connect on launch), echo the target URL
199+ // so the user knows what the spinner is waiting on.
200+ if (state.loading) {
201+ Spacer(Modifier.height(8.dp))
202+ Text(
203+ text = "Connecting to ${state.serverUrl}...",
204+ style = MaterialTheme.typography.bodySmall,
205+ color = MaterialTheme.colorScheme.onSurfaceVariant,
206+ textAlign = TextAlign.Center,
207+ )
208+ }
209+
210+ // Surface the error inline as well, so it stays visible while the user
211+ // corrects the form (the snackbar is transient).
212+ if (state.error != null && !state.loading) {
213+ Spacer(Modifier.height(8.dp))
214+ Text(
215+ text = state.error,
216+ style = MaterialTheme.typography.bodySmall,
217+ color = MaterialTheme.colorScheme.error,
218+ textAlign = TextAlign.Center,
219+ )
220+ }
125221 }
126222}
app/src/main/java/soy/iko/opencode/ui/screens/SessionsScreen.kt+240−18
@@ -1,22 +1,37 @@
11package soy.iko.opencode.ui.screens
22
3+import android.content.ClipData
4+import android.content.ClipboardManager
5+import android.content.Context
6+import android.widget.Toast
7+import androidx.compose.foundation.ExperimentalFoundationApi
8+import androidx.compose.foundation.background
39import androidx.compose.foundation.clickable
10+import androidx.compose.foundation.combinedClickable
411import androidx.compose.foundation.layout.Arrangement
12+import androidx.compose.foundation.layout.Box
513import androidx.compose.foundation.layout.Column
614import androidx.compose.foundation.layout.PaddingValues
715import androidx.compose.foundation.layout.Row
16+import androidx.compose.foundation.layout.Spacer
817import androidx.compose.foundation.layout.fillMaxSize
918import androidx.compose.foundation.layout.fillMaxWidth
19+import androidx.compose.foundation.layout.height
1020import androidx.compose.foundation.layout.padding
21+import androidx.compose.foundation.layout.size
1122import androidx.compose.foundation.lazy.LazyColumn
1223import androidx.compose.foundation.lazy.items
24+import androidx.compose.foundation.lazy.rememberLazyListState
25+import androidx.compose.foundation.shape.CircleShape
1326import androidx.compose.material.icons.Icons
1427import androidx.compose.material.icons.automirrored.filled.ArrowBack
1528import androidx.compose.material.icons.filled.Add
1629import androidx.compose.material.icons.filled.ChatBubbleOutline
30+import androidx.compose.material.icons.filled.ChevronRight
1731import androidx.compose.material.icons.filled.DeleteOutline
1832import androidx.compose.material.icons.filled.Refresh
1933import androidx.compose.material3.AlertDialog
34+import androidx.compose.material3.CircularProgressIndicator
2035import androidx.compose.material3.ExperimentalMaterial3Api
2136import androidx.compose.material3.ExtendedFloatingActionButton
2237import androidx.compose.material3.HorizontalDivider
@@ -24,17 +39,31 @@ import androidx.compose.material3.Icon
2439import androidx.compose.material3.IconButton
2540import androidx.compose.material3.LinearProgressIndicator
2641import androidx.compose.material3.MaterialTheme
42+import androidx.compose.material3.OutlinedTextField
2743import androidx.compose.material3.Scaffold
44+import androidx.compose.material3.Surface
45+import androidx.compose.material3.SwipeToDismissBox
46+import androidx.compose.material3.SwipeToDismissBoxValue
2847import androidx.compose.material3.Text
2948import androidx.compose.material3.TextButton
3049import androidx.compose.material3.TopAppBar
50+import androidx.compose.material3.rememberSwipeToDismissBoxState
51+import androidx.compose.material3.pulltorefresh.PullToRefreshBox
52+import androidx.compose.material3.pulltorefresh.rememberPullToRefreshState
3153import androidx.compose.runtime.Composable
54+import androidx.compose.runtime.LaunchedEffect
3255import androidx.compose.runtime.getValue
3356import androidx.compose.runtime.mutableStateOf
3457import androidx.compose.runtime.remember
3558import androidx.compose.runtime.setValue
3659import androidx.compose.ui.Alignment
3760import androidx.compose.ui.Modifier
61+import androidx.compose.ui.draw.clip
62+import androidx.compose.ui.graphics.Color
63+import androidx.compose.ui.hapticfeedback.HapticFeedbackType
64+import androidx.compose.ui.platform.LocalContext
65+import androidx.compose.ui.platform.LocalHapticFeedback
66+import androidx.compose.ui.text.style.TextAlign
3867import androidx.compose.ui.text.style.TextOverflow
3968import androidx.compose.ui.unit.dp
4069import soy.iko.opencode.core.Event
@@ -44,7 +73,25 @@ import soy.iko.opencode.core.UiState
4473@OptIn(ExperimentalMaterial3Api::class)
4574@Composable
4675fun SessionsScreen(state: UiState, dispatch: (Event) -> Unit) {
76+ val haptic = LocalHapticFeedback.current
4777 var pendingDeleteId by remember { mutableStateOf<String?>(null) }
78+ var isPullRefreshing by remember { mutableStateOf(false) }
79+ var searchQuery by remember { mutableStateOf("") }
80+ val sessionsListState = rememberLazyListState()
81+
82+ // Bring a newly created session (which lands at the top of the list) into view.
83+ // Only scroll when the list actually grows, not on every reload of the same sessions.
84+ var prevSize by remember { mutableStateOf(state.sessions.size) }
85+ LaunchedEffect(state.sessions.size) {
86+ if (state.sessions.size > prevSize) sessionsListState.animateScrollToItem(0)
87+ prevSize = state.sessions.size
88+ }
89+
90+ // The pull-to-refresh spinner should only reflect a user-initiated pull, not
91+ // every loading operation. Clear it once the underlying load completes.
92+ LaunchedEffect(state.loading) {
93+ if (!state.loading) isPullRefreshing = false
94+ }
4895
4996 pendingDeleteId?.let { deleteId ->
5097 val pendingSession = state.sessions.firstOrNull { it.id == deleteId }
@@ -54,6 +101,7 @@ fun SessionsScreen(state: UiState, dispatch: (Event) -> Unit) {
54101 text = { Text(pendingSession?.title ?: deleteId) },
55102 confirmButton = {
56103 TextButton(onClick = {
104+ haptic.performHapticFeedback(HapticFeedbackType.LongPress)
57105 dispatch(Event.DeleteSession(deleteId))
58106 pendingDeleteId = null
59107 }) {
@@ -71,7 +119,23 @@ fun SessionsScreen(state: UiState, dispatch: (Event) -> Unit) {
71119 Scaffold(
72120 topBar = {
73121 TopAppBar(
74− title = { Text("Sessions") },
122+ title = {
123+ Row(verticalAlignment = Alignment.CenterVertically) {
124+ // A quick at-a-glance health dot for the SSE stream: green when
125+ // connected, error-tinted when the stream is down (updates stall).
126+ Box(
127+ modifier = Modifier
128+ .size(8.dp)
129+ .clip(CircleShape)
130+ .background(
131+ if (state.sseConnected) Color(0xFF4CAF50)
132+ else MaterialTheme.colorScheme.error,
133+ ),
134+ )
135+ Spacer(Modifier.size(8.dp))
136+ Text(if (state.sessions.isNotEmpty()) "Sessions (${state.sessions.size})" else "Sessions")
137+ }
138+ },
75139 navigationIcon = {
76140 IconButton(onClick = { dispatch(Event.NavigateToConnect) }) {
77141 Icon(Icons.AutoMirrored.Filled.ArrowBack, contentDescription = "Disconnect")
@@ -85,30 +149,88 @@ fun SessionsScreen(state: UiState, dispatch: (Event) -> Unit) {
85149 )
86150 },
87151 floatingActionButton = {
88− ExtendedFloatingActionButton(
89− onClick = { dispatch(Event.CreateSession) },
90− icon = { Icon(Icons.Filled.Add, contentDescription = null) },
91− text = { Text("New session") },
92− )
152+ // Give the search results the full width while searching — the FAB would
153+ // otherwise cover the bottom rows and isn't relevant mid-search anyway.
154+ if (searchQuery.isEmpty()) {
155+ ExtendedFloatingActionButton(
156+ onClick = { dispatch(Event.CreateSession) },
157+ icon = { Icon(Icons.Filled.Add, contentDescription = null) },
158+ text = { Text("New session") },
159+ )
160+ }
93161 },
94162 ) { padding ->
95163 Column(Modifier.fillMaxSize().padding(padding)) {
96− if (state.loading) {
164+ if (!state.sseConnected) {
165+ ReconnectingBanner()
166+ }
167+ if (state.loading && !isPullRefreshing) {
97168 LinearProgressIndicator(Modifier.fillMaxWidth())
98169 }
170+ if (state.sessions.isNotEmpty()) {
171+ OutlinedTextField(
172+ value = searchQuery,
173+ onValueChange = { searchQuery = it },
174+ placeholder = { Text("Search sessions") },
175+ singleLine = true,
176+ modifier = Modifier
177+ .fillMaxWidth()
178+ .padding(horizontal = 16.dp, vertical = 8.dp),
179+ )
180+ }
181+ val filteredSessions = if (searchQuery.isEmpty()) {
182+ state.sessions
183+ } else {
184+ state.sessions.filter {
185+ it.title.contains(searchQuery, ignoreCase = true) ||
186+ it.id.contains(searchQuery, ignoreCase = true)
187+ }
188+ }
99189 when {
100190 state.sessions.isEmpty() && !state.loading -> EmptySessions()
101− else -> LazyColumn(
191+ filteredSessions.isEmpty() && searchQuery.isNotEmpty() -> NoSearchResults(searchQuery)
192+ else -> PullToRefreshBox(
193+ isRefreshing = isPullRefreshing,
194+ onRefresh = {
195+ isPullRefreshing = true
196+ dispatch(Event.LoadSessions)
197+ },
102198 modifier = Modifier.fillMaxSize(),
103− contentPadding = PaddingValues(bottom = 88.dp),
199+ state = rememberPullToRefreshState(),
104200 ) {
105− items(state.sessions, key = { it.id }) { session ->
106− SessionRow(
107− session = session,
108− onOpen = { dispatch(Event.SelectSession(session.id)) },
109− onDelete = { pendingDeleteId = session.id },
110− )
111− HorizontalDivider(color = MaterialTheme.colorScheme.outline.copy(alpha = 0.25f))
201+ LazyColumn(
202+ state = sessionsListState,
203+ modifier = Modifier.fillMaxSize(),
204+ contentPadding = PaddingValues(bottom = 88.dp),
205+ ) {
206+ items(filteredSessions, key = { it.id }) { session ->
207+ // animateItem() slides the remaining rows into place when one is
208+ // deleted (and fades new rows in) instead of snapping instantly.
209+ Column(modifier = Modifier.animateItem()) {
210+ val dismissState = rememberSwipeToDismissBoxState(
211+ // Only a swipe from the end (right-to-left) triggers deletion;
212+ // return false so the row snaps back and the confirm dialog decides.
213+ confirmValueChange = { value ->
214+ if (value == SwipeToDismissBoxValue.EndToStart) {
215+ pendingDeleteId = session.id
216+ }
217+ false
218+ },
219+ )
220+ SwipeToDismissBox(
221+ state = dismissState,
222+ enableDismissFromStartToEnd = false,
223+ backgroundContent = { SwipeDeleteBackground() },
224+ ) {
225+ SessionRow(
226+ session = session,
227+ onOpen = { dispatch(Event.SelectSession(session.id)) },
228+ onDelete = { pendingDeleteId = session.id },
229+ )
230+ }
231+ HorizontalDivider(color = MaterialTheme.colorScheme.outline.copy(alpha = 0.25f))
232+ }
233+ }
112234 }
113235 }
114236 }
@@ -116,12 +238,21 @@ fun SessionsScreen(state: UiState, dispatch: (Event) -> Unit) {
116238 }
117239}
118240
241+@OptIn(ExperimentalFoundationApi::class)
119242@Composable
120243private fun SessionRow(session: SessionView, onOpen: () -> Unit, onDelete: () -> Unit) {
244+ val context = LocalContext.current
121245 Row(
122246 modifier = Modifier
123247 .fillMaxWidth()
124− .clickable(onClick = onOpen)
248+ .background(MaterialTheme.colorScheme.surface)
249+ .combinedClickable(
250+ onClick = onOpen,
251+ onLongClick = {
252+ copyToClipboard(context, session.id)
253+ Toast.makeText(context, "Session ID copied", Toast.LENGTH_SHORT).show()
254+ },
255+ )
125256 .padding(start = 20.dp, end = 8.dp, top = 14.dp, bottom = 14.dp),
126257 verticalAlignment = Alignment.CenterVertically,
127258 ) {
@@ -146,20 +277,97 @@ private fun SessionRow(session: SessionView, onOpen: () -> Unit, onDelete: () ->
146277 overflow = TextOverflow.Ellipsis,
147278 )
148279 }
280+ Icon(
281+ Icons.Filled.ChevronRight,
282+ contentDescription = null,
283+ tint = MaterialTheme.colorScheme.onSurfaceVariant,
284+ modifier = Modifier.padding(horizontal = 4.dp),
285+ )
149286 IconButton(onClick = onDelete) {
150287 Icon(
151288 Icons.Filled.DeleteOutline,
152289 contentDescription = "Delete session",
153− tint = MaterialTheme.colorScheme.onSurfaceVariant,
290+ tint = MaterialTheme.colorScheme.error,
291+ )
292+ }
293+ }
294+}
295+
296+/** Red background with a trailing delete icon and label, revealed while swiping a session row away. */
297+@Composable
298+private fun SwipeDeleteBackground() {
299+ Box(
300+ modifier = Modifier
301+ .fillMaxSize()
302+ .background(MaterialTheme.colorScheme.error)
303+ .padding(horizontal = 24.dp),
304+ contentAlignment = Alignment.CenterEnd,
305+ ) {
306+ Row(verticalAlignment = Alignment.CenterVertically) {
307+ Icon(
308+ Icons.Filled.DeleteOutline,
309+ contentDescription = null,
310+ tint = MaterialTheme.colorScheme.onError,
311+ )
312+ Text(
313+ text = "Delete",
314+ style = MaterialTheme.typography.labelLarge,
315+ color = MaterialTheme.colorScheme.onError,
316+ modifier = Modifier.padding(start = 4.dp),
154317 )
155318 }
156319 }
157320}
158321
322+/** Shown at the top of the list while the SSE stream is down, since session updates stall too. */
323+@Composable
324+private fun ReconnectingBanner() {
325+ Surface(
326+ modifier = Modifier.fillMaxWidth(),
327+ color = MaterialTheme.colorScheme.tertiaryContainer,
328+ ) {
329+ Row(
330+ modifier = Modifier.fillMaxWidth().padding(horizontal = 16.dp, vertical = 6.dp),
331+ verticalAlignment = Alignment.CenterVertically,
332+ horizontalArrangement = Arrangement.spacedBy(8.dp),
333+ ) {
334+ CircularProgressIndicator(
335+ modifier = Modifier.size(14.dp),
336+ strokeWidth = 2.dp,
337+ color = MaterialTheme.colorScheme.onTertiaryContainer,
338+ )
339+ Text(
340+ text = "Reconnecting…",
341+ style = MaterialTheme.typography.bodySmall,
342+ color = MaterialTheme.colorScheme.onTertiaryContainer,
343+ )
344+ }
345+ }
346+}
347+
348+private fun copyToClipboard(context: Context, text: String) {
349+ val clipboard = context.getSystemService(Context.CLIPBOARD_SERVICE) as ClipboardManager
350+ clipboard.setPrimaryClip(ClipData.newPlainText("session", text))
351+}
352+
159353/** Shorten a session ID for display: "abc12345…" instead of the full UUID. */
160354private fun shortSessionId(id: String): String =
161355 if (id.length > 12) id.take(8) + "…" else id
162356
357+@Composable
358+private fun NoSearchResults(query: String) {
359+ Box(
360+ modifier = Modifier.fillMaxSize().padding(32.dp),
361+ contentAlignment = Alignment.Center,
362+ ) {
363+ Text(
364+ text = "No results for “$query”",
365+ style = MaterialTheme.typography.bodyMedium,
366+ color = MaterialTheme.colorScheme.onSurfaceVariant,
367+ )
368+ }
369+}
370+
163371@Composable
164372private fun EmptySessions() {
165373 Column(
@@ -167,6 +375,13 @@ private fun EmptySessions() {
167375 verticalArrangement = Arrangement.Center,
168376 horizontalAlignment = Alignment.CenterHorizontally,
169377 ) {
378+ Icon(
379+ Icons.Filled.ChatBubbleOutline,
380+ contentDescription = null,
381+ modifier = Modifier.size(64.dp),
382+ tint = MaterialTheme.colorScheme.primary.copy(alpha = 0.5f),
383+ )
384+ Spacer(Modifier.height(16.dp))
170385 Text(
171386 text = "No sessions yet",
172387 style = MaterialTheme.typography.titleMedium,
@@ -176,5 +391,12 @@ private fun EmptySessions() {
176391 style = MaterialTheme.typography.bodyMedium,
177392 color = MaterialTheme.colorScheme.onSurfaceVariant,
178393 )
394+ Spacer(Modifier.height(8.dp))
395+ Text(
396+ text = "Sessions you create on this server will appear here.",
397+ style = MaterialTheme.typography.bodySmall,
398+ color = MaterialTheme.colorScheme.onSurfaceVariant,
399+ textAlign = TextAlign.Center,
400+ )
179401 }
180402}
app/src/main/java/soy/iko/opencode/ui/theme/Theme.kt+23−2
@@ -1,6 +1,5 @@
11package soy.iko.opencode.ui.theme
22
3−import android.app.Activity
43import android.os.Build
54import androidx.compose.foundation.isSystemInDarkTheme
65import androidx.compose.material3.MaterialTheme
@@ -15,7 +14,18 @@ import androidx.compose.ui.platform.LocalContext
1514private val DarkColors = darkColorScheme(
1615 primary = Emerald,
1716 onPrimary = Color(0xFF0F1115),
17+ primaryContainer = Color(0xFF1A3D2A),
18+ onPrimaryContainer = Color(0xFFA8E6C1),
1819 secondary = EmeraldDark,
20+ onSecondary = Color(0xFFE6E9EF),
21+ secondaryContainer = Color(0xFF1A2A3D),
22+ onSecondaryContainer = Color(0xFFA8C5E6),
23+ tertiary = Color(0xFF6B7280),
24+ onTertiary = Color(0xFFE6E9EF),
25+ tertiaryContainer = Color(0xFF2A2A3D),
26+ onTertiaryContainer = Color(0xFFC5C5E6),
27+ errorContainer = Color(0xFF4D2626),
28+ onErrorContainer = Color(0xFFFFB4B4),
1929 background = DarkBackground,
2030 surface = DarkSurface,
2131 surfaceVariant = DarkSurfaceVariant,
@@ -25,7 +35,18 @@ private val DarkColors = darkColorScheme(
2535
2636private val LightColors = lightColorScheme(
2737 primary = EmeraldDark,
38+ primaryContainer = Color(0xFFD7F5E3),
39+ onPrimaryContainer = Color(0xFF0A3D22),
2840 secondary = Emerald,
41+ onSecondary = Color(0xFF1A1C1F),
42+ secondaryContainer = Color(0xFFD7E8F5),
43+ onSecondaryContainer = Color(0xFF0A2A4D),
44+ tertiary = Color(0xFF6B7280),
45+ onTertiary = Color(0xFFFFFFFF),
46+ tertiaryContainer = Color(0xFFE8E8F5),
47+ onTertiaryContainer = Color(0xFF2A2A4D),
48+ errorContainer = Color(0xFFFFE0E0),
49+ onErrorContainer = Color(0xFF660000),
2950 background = LightBackground,
3051 surface = LightSurface,
3152 surfaceVariant = LightSurfaceVariant,
@@ -36,7 +57,7 @@ private val LightColors = lightColorScheme(
3657@Composable
3758fun OpencodeTheme(
3859 darkTheme: Boolean = isSystemInDarkTheme(),
39− dynamicColor: Boolean = true,
60+ dynamicColor: Boolean = false,
4061 content: @Composable () -> Unit,
4162) {
4263 val context = LocalContext.current
shared/src/app.rs+14
@@ -297,16 +297,27 @@ impl App for OpencodeApp {
297297 // Store the field verbatim while typing; normalizing here would fight
298298 // the user's edits (e.g. eat the "://" as they type it). The URL is
299299 // normalized once, on Connect.
300+ // Clear any prior error as soon as the user edits a field, so a stale
301+ // message (e.g. "Invalid credentials") doesn't linger while they correct it.
300302 Event::ServerUrlChanged(url) => {
301303 model.server_url = url;
304+ if model.error.is_some() {
305+ model.error = None;
306+ }
302307 render()
303308 }
304309 Event::UsernameChanged(u) => {
305310 model.username = u;
311+ if model.error.is_some() {
312+ model.error = None;
313+ }
306314 render()
307315 }
308316 Event::PasswordChanged(p) => {
309317 model.password = p;
318+ if model.error.is_some() {
319+ model.error = None;
320+ }
310321 render()
311322 }
312323
@@ -341,6 +352,9 @@ impl App for OpencodeApp {
341352 title: display_title(&s.title),
342353 })
343354 .collect();
355+ // Impose a stable alphabetical order so the list doesn't shuffle
356+ // with whatever order the server happens to return sessions in.
357+ model.sessions.sort_by_key(|s| s.title.to_lowercase());
344358 }
345359 Err(e) => model.error = Some(e),
346360 }