git.iko.soy/opencode-android
on master help sign in
swap
ab6a5da16 ahead, 0 behind
app/src/main/java/soy/iko/opencode/CoreViewModel.kt+14
since splitsnapshots
unifiedsplit
@@ -23,11 +23,23 @@ class CoreViewModel(app: Application) : AndroidViewModel(app) {
23 init {
23 init {
24 prefs.getString(KEY_URL, null)?.let { core.dispatch(Event.ServerUrlChanged(it)) }
24 prefs.getString(KEY_URL, null)?.let { core.dispatch(Event.ServerUrlChanged(it)) }
25 prefs.getString(KEY_USER, null)?.let { core.dispatch(Event.UsernameChanged(it)) }
25 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)) }
26
29
27 // On a fresh process after a configuration change, resume the SSE stream if
30 // On a fresh process after a configuration change, resume the SSE stream if
28 // the core is already connected (no-op otherwise — it guards on `connected`).
31 // the core is already connected (no-op otherwise — it guards on `connected`).
29 core.resumeStreamingIfConnected()
32 core.resumeStreamingIfConnected()
30
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+ }
42+
31 // Persist connection details once a connection succeeds.
43 // Persist connection details once a connection succeeds.
32 viewModelScope.launch {
44 viewModelScope.launch {
33 core.view.distinctUntilChangedBy { it.connected to it.serverUrl }.collect { state ->
45 core.view.distinctUntilChangedBy { it.connected to it.serverUrl }.collect { state ->
@@ -35,6 +47,7 @@ class CoreViewModel(app: Application) : AndroidViewModel(app) {
35 prefs.edit {
47 prefs.edit {
36 putString(KEY_URL, state.serverUrl)
48 putString(KEY_URL, state.serverUrl)
37 putString(KEY_USER, state.username)
49 putString(KEY_USER, state.username)
50+ putString(KEY_PASSWORD, state.password)
38 }
51 }
39 }
52 }
40 }
53 }
@@ -48,5 +61,6 @@ class CoreViewModel(app: Application) : AndroidViewModel(app) {
48 private companion object {
61 private companion object {
49 const val KEY_URL = "serverUrl"
62 const val KEY_URL = "serverUrl"
50 const val KEY_USER = "username"
63 const val KEY_USER = "username"
64+ const val KEY_PASSWORD = "password"
51 }
65 }
52}
66}
app/src/main/java/soy/iko/opencode/MainActivity.kt+7−1
@@ -2,6 +2,7 @@ package soy.iko.opencode
2
2
3import android.os.Bundle
3import android.os.Bundle
4import androidx.activity.ComponentActivity
4import androidx.activity.ComponentActivity
5+import androidx.activity.SystemBarStyle
5import androidx.activity.compose.setContent
6import androidx.activity.compose.setContent
6import androidx.activity.enableEdgeToEdge
7import androidx.activity.enableEdgeToEdge
7import androidx.compose.runtime.getValue
8import androidx.compose.runtime.getValue
@@ -12,7 +13,12 @@ import soy.iko.opencode.ui.theme.OpencodeTheme
12
13
13class MainActivity : ComponentActivity() {
14class MainActivity : ComponentActivity() {
14 override fun onCreate(savedInstanceState: Bundle?) {
15 override fun onCreate(savedInstanceState: Bundle?) {
15− enableEdgeToEdge()
16+ enableEdgeToEdge(
17+ statusBarStyle = SystemBarStyle.auto(
18+ lightScrim = 0xFF0F1115.toInt(),
19+ darkScrim = 0xFF0F1115.toInt(),
20+ ),
21+ )
16 super.onCreate(savedInstanceState)
22 super.onCreate(savedInstanceState)
17 setContent {
23 setContent {
18 OpencodeTheme {
24 OpencodeTheme {
app/src/main/java/soy/iko/opencode/core/HttpClient.kt+1
@@ -51,6 +51,7 @@ class HttpClient(
51 ): Result<HttpResp> = withContext(Dispatchers.IO) {
51 ): Result<HttpResp> = withContext(Dispatchers.IO) {
52 runCatching {
52 runCatching {
53 val builder = Request.Builder().url(url).method()
53 val builder = Request.Builder().url(url).method()
54+ builder.header("User-Agent", "opencode-android/1.0")
54 if (auth != null) builder.header("Authorization", auth)
55 if (auth != null) builder.header("Authorization", auth)
55 await(okhttp.newCall(builder.build()))
56 await(okhttp.newCall(builder.build()))
56 }
57 }
app/src/main/java/soy/iko/opencode/core/OpencodeCore.kt+198−38
@@ -1,6 +1,7 @@
1package soy.iko.opencode.core
1package soy.iko.opencode.core
2
2
3import kotlinx.coroutines.CoroutineScope
3import kotlinx.coroutines.CoroutineScope
4+import kotlinx.coroutines.Job
4import kotlinx.coroutines.delay
5import kotlinx.coroutines.delay
5import kotlinx.coroutines.flow.MutableStateFlow
6import kotlinx.coroutines.flow.MutableStateFlow
6import kotlinx.coroutines.flow.StateFlow
7import kotlinx.coroutines.flow.StateFlow
@@ -48,6 +49,31 @@ class OpencodeCore(
48 private var sse: SseClient? = null
49 private var sse: SseClient? = null
49 private var sseClosedIntentionally = false
50 private var sseClosedIntentionally = false
50 private var reconnectAttempts = 0
51 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>()
51
77
52 // ── Event entry point ──────────────────────────────────────────────────────
78 // ── Event entry point ──────────────────────────────────────────────────────
53 fun dispatch(event: Event) {
79 fun dispatch(event: Event) {
@@ -55,38 +81,69 @@ class OpencodeCore(
55 // Store the field verbatim while typing; normalizing here would fight
81 // Store the field verbatim while typing; normalizing here would fight
56 // the user's edits (e.g. eat the "://" as they type it). The URL is
82 // the user's edits (e.g. eat the "://" as they type it). The URL is
57 // normalized once, on Connect.
83 // normalized once, on Connect.
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.
58− is Event.ServerUrlChanged -> { serverUrl = event.url; emit() }
86+ is Event.ServerUrlChanged -> { serverUrl = event.url; if (error != null) error = null; emit() }
59− is Event.UsernameChanged -> { username = event.value; emit() }
87+ is Event.UsernameChanged -> { username = event.value; if (error != null) error = null; emit() }
60− is Event.PasswordChanged -> { password = event.value; emit() }
88+ is Event.PasswordChanged -> { password = event.value; if (error != null) error = null; emit() }
61
89
62 Event.Connect -> {
90 Event.Connect -> {
63 serverUrl = normalizeUrl(serverUrl)
91 serverUrl = normalizeUrl(serverUrl)
64 if (serverUrl.isEmpty()) { error = "Enter a server URL"; emit(); return }
92 if (serverUrl.isEmpty()) { error = "Enter a server URL"; emit(); return }
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.
65− loading = true; error = null; authRequired = false; emit()
98+ loading = true; error = null; emit()
99+ connectJob?.cancel()
66− scope.launch { probeHealth() }
100+ connectJob = scope.launch { probeHealth() }
67 }
101 }
68
102
103+ Event.LoadSessions -> {
104+ if (!connected) { error = "Not connected"; emit(); return }
105+ loading = true; emit()
106+ loadSessionsJob?.cancel()
69− Event.LoadSessions -> { loading = true; emit(); scope.launch { loadSessions() } }
107+ loadSessionsJob = scope.launch { loadSessions() }
108+ }
70
109
71 is Event.SelectSession -> {
110 is Event.SelectSession -> {
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.
72− currentSessionTitle = sessions.firstOrNull { it.id == event.id }?.title.orEmpty()
113+ val session = sessions.firstOrNull { it.id == event.id }
114+ if (session == null) { error = "Session no longer exists"; emit(); return }
115+ currentSessionTitle = session.title
73 currentSessionId = event.id
116 currentSessionId = event.id
74 messages.clear()
117 messages.clear()
118+ removedMessageIds.clear()
75 screen = Screen.Chat
119 screen = Screen.Chat
76 loading = true
120 loading = true
121+ generating = false
77 error = null
122 error = null
78 emit()
123 emit()
124+ connectJob?.cancel()
125+ abortJob?.cancel()
126+ sendMessageJob?.cancel()
127+ deleteSessionJob?.cancel()
128+ loadMessagesJob?.cancel()
79− scope.launch { loadMessages(event.id) }
129+ loadMessagesJob = scope.launch { loadMessages(event.id) }
80 }
130 }
81
131
132+ Event.CreateSession -> {
133+ if (!connected) { error = "Not connected"; emit(); return }
134+ loading = true; error = null; emit()
135+ createSessionJob?.cancel()
82− Event.CreateSession -> { loading = true; emit(); scope.launch { createSession() } }
136+ createSessionJob = scope.launch { createSession() }
137+ }
83
138
84 is Event.DeleteSession -> {
139 is Event.DeleteSession -> {
140+ if (!connected) { error = "Not connected"; emit(); return }
85 val id = event.id
141 val id = event.id
86 // Delete on the server first; only drop it from the UI once that
142 // Delete on the server first; only drop it from the UI once that
87 // succeeds, so a failed request leaves the session recoverable.
143 // succeeds, so a failed request leaves the session recoverable.
88− loading = true; emit()
144+ loading = true; error = null; emit()
145+ deleteSessionJob?.cancel()
89− scope.launch {
146+ deleteSessionJob = scope.launch {
90 http.delete("$serverUrl/session/$id", authHeader()).fold(
147 http.delete("$serverUrl/session/$id", authHeader()).fold(
91 onSuccess = { resp ->
148 onSuccess = { resp ->
92 if (resp.code in 200..299) {
149 if (resp.code in 200..299) {
@@ -119,17 +176,32 @@ class OpencodeCore(
119 messages.forEach { it.streaming = false }
176 messages.forEach { it.streaming = false }
120 emit()
177 emit()
121 val id = currentSessionId ?: return
178 val id = currentSessionId ?: return
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()
122− scope.launch { http.postJson("$serverUrl/session/$id/abort", "{}", authHeader()) }
182+ abortJob = scope.launch { http.postJson("$serverUrl/session/$id/abort", "{}", authHeader()) }
123 }
183 }
124
184
125 Event.NavigateToSessions -> {
185 Event.NavigateToSessions -> {
126 screen = Screen.Sessions
186 screen = Screen.Sessions
127 currentSessionId = null
187 currentSessionId = null
128 messages.clear()
188 messages.clear()
189+ removedMessageIds.clear()
190+ draft = ""
129 generating = false
191 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
130 loading = true
195 loading = true
131 emit()
196 emit()
197+ connectJob?.cancel()
198+ abortJob?.cancel()
199+ createSessionJob?.cancel()
200+ deleteSessionJob?.cancel()
201+ sendMessageJob?.cancel()
202+ loadMessagesJob?.cancel()
203+ loadSessionsJob?.cancel()
132− scope.launch { loadSessions() }
204+ loadSessionsJob = scope.launch { loadSessions() }
133 }
205 }
134
206
135 Event.NavigateToConnect -> {
207 Event.NavigateToConnect -> {
@@ -139,7 +211,18 @@ class OpencodeCore(
139 generating = false
211 generating = false
140 sessions.clear()
212 sessions.clear()
141 messages.clear()
213 messages.clear()
214+ removedMessageIds.clear()
142 currentSessionId = null
215 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()
143 closeSse()
226 closeSse()
144 emit()
227 emit()
145 }
228 }
@@ -153,26 +236,44 @@ class OpencodeCore(
153 if (connected && sse == null) startSse()
236 if (connected && sse == null) startSse()
154 }
237 }
155
238
156− 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+ }
157
251
158 // ── Effects ────────────────────────────────────────────────────────────────
252 // ── Effects ────────────────────────────────────────────────────────────────
159 private fun sendMessage(raw: String) {
253 private fun sendMessage(raw: String) {
160 val text = raw.trim()
254 val text = raw.trim()
161 val session = currentSessionId ?: return
255 val session = currentSessionId ?: return
162 if (text.isEmpty()) return
256 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
163 localSeq += 1
260 localSeq += 1
164 val user = MsgState("local-$localSeq", "user", System.currentTimeMillis()).apply {
261 val user = MsgState("local-$localSeq", "user", System.currentTimeMillis()).apply {
165 status = MessageStatus.Pending
262 status = MessageStatus.Pending
166 textParts["local"] = text
263 textParts["local"] = text
167 }
264 }
168 messages.add(user)
265 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
169 draft = ""
269 draft = ""
170 generating = true
270 generating = true
171 error = null
271 error = null
172 emit()
272 emit()
173
273
174 val body = Protocol.json.encodeToString(PromptBody.serializer(), PromptBody(listOf(TextInput(text = text))))
274 val body = Protocol.json.encodeToString(PromptBody.serializer(), PromptBody(listOf(TextInput(text = text))))
275+ sendMessageJob?.cancel()
175− scope.launch {
276+ sendMessageJob = scope.launch {
176 val result = http.postJson("$serverUrl/session/$session/prompt_async", body, authHeader())
277 val result = http.postJson("$serverUrl/session/$session/prompt_async", body, authHeader())
177 result.fold(
278 result.fold(
178 onSuccess = { resp ->
279 onSuccess = { resp ->
@@ -181,12 +282,16 @@ class OpencodeCore(
181 } else {
282 } else {
182 messages.firstOrNull { it.id == user.id }?.status = MessageStatus.Failed
283 messages.firstOrNull { it.id == user.id }?.status = MessageStatus.Failed
183 generating = false
284 generating = false
285+ messages.forEach { it.streaming = false }
286+ draft = savedDraft
184 error = "Server returned status ${resp.code}"
287 error = "Server returned status ${resp.code}"
185 }
288 }
186 },
289 },
187 onFailure = { e ->
290 onFailure = { e ->
188 messages.firstOrNull { it.id == user.id }?.status = MessageStatus.Failed
291 messages.firstOrNull { it.id == user.id }?.status = MessageStatus.Failed
189 generating = false
292 generating = false
293+ messages.forEach { it.streaming = false }
294+ draft = savedDraft
190 error = "Request failed: ${e.message}"
295 error = "Request failed: ${e.message}"
191 },
296 },
192 )
297 )
@@ -201,6 +306,10 @@ class OpencodeCore(
201 when {
306 when {
202 resp.code == 401 -> {
307 resp.code == 401 -> {
203 loading = false
308 loading = false
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
204 // Always reveal the auth fields so the user can see/edit
313 // Always reveal the auth fields so the user can see/edit
205 // their credentials — even when the ones supplied were wrong.
314 // their credentials — even when the ones supplied were wrong.
206 authRequired = true
315 authRequired = true
@@ -215,10 +324,14 @@ class OpencodeCore(
215 loading = true
324 loading = true
216 emit()
325 emit()
217 startSse()
326 startSse()
327+ // Route through loadSessionsJob so a prior in-flight loadSessions
328+ // (e.g. from a rapid double-connect) is cancelled first.
329+ loadSessionsJob?.cancel()
218− loadSessions()
330+ loadSessionsJob = scope.launch { loadSessions() }
219 }
331 }
220 else -> {
332 else -> {
221 loading = false
333 loading = false
334+ connected = false
222 error = "Server returned status ${resp.code}"
335 error = "Server returned status ${resp.code}"
223 emit()
336 emit()
224 }
337 }
@@ -226,6 +339,7 @@ class OpencodeCore(
226 },
339 },
227 onFailure = { e ->
340 onFailure = { e ->
228 loading = false
341 loading = false
342+ connected = false
229 error = "Connection failed: ${e.message}"
343 error = "Connection failed: ${e.message}"
230 emit()
344 emit()
231 },
345 },
@@ -238,6 +352,9 @@ class OpencodeCore(
238 if (resp.code in 200..299) {
352 if (resp.code in 200..299) {
239 sessions.clear()
353 sessions.clear()
240 sessions.addAll(Protocol.parseSessions(resp.body).map { SessionView(it.id, displayTitle(it.title)) })
354 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() }
241 } else {
358 } else {
242 error = "Server returned status ${resp.code}"
359 error = "Server returned status ${resp.code}"
243 }
360 }
@@ -255,6 +372,10 @@ class OpencodeCore(
255 if (session != null) {
372 if (session != null) {
256 if (sessions.none { it.id == session.id }) {
373 if (sessions.none { it.id == session.id }) {
257 sessions.add(0, SessionView(session.id, displayTitle(session.title)))
374 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() }
258 }
379 }
259 emit()
380 emit()
260 // SelectSession takes over the loading state (loads messages).
381 // SelectSession takes over the loading state (loads messages).
@@ -276,6 +397,7 @@ class OpencodeCore(
276 loading = false
397 loading = false
277 if (resp.code in 200..299) {
398 if (resp.code in 200..299) {
278 messages.clear()
399 messages.clear()
400+ removedMessageIds.clear()
279 messages.addAll(Protocol.parseMessages(resp.body).map(::messageFromWire))
401 messages.addAll(Protocol.parseMessages(resp.body).map(::messageFromWire))
280 generating = messages.any { it.streaming }
402 generating = messages.any { it.streaming }
281 } else {
403 } else {
@@ -286,6 +408,7 @@ class OpencodeCore(
286 onFailure = { e ->
408 onFailure = { e ->
287 if (!isCurrent(sessionId)) { loading = false; return }
409 if (!isCurrent(sessionId)) { loading = false; return }
288 loading = false
410 loading = false
411+ generating = false
289 error = "Request failed: ${e.message}"
412 error = "Request failed: ${e.message}"
290 emit()
413 emit()
291 },
414 },
@@ -314,6 +437,8 @@ class OpencodeCore(
314
437
315 private fun closeSse() {
438 private fun closeSse() {
316 sseClosedIntentionally = true
439 sseClosedIntentionally = true
440+ reconnectJob?.cancel()
441+ reconnectJob = null
317 sse?.close()
442 sse?.close()
318 sse = null
443 sse = null
319 }
444 }
@@ -324,7 +449,10 @@ class OpencodeCore(
324 // stream reconnects (onStateChange(true)) so a healthy server starts fresh.
449 // stream reconnects (onStateChange(true)) so a healthy server starts fresh.
325 val delayMs = (2000L * (1L shl reconnectAttempts)).coerceAtMost(30_000L)
450 val delayMs = (2000L * (1L shl reconnectAttempts)).coerceAtMost(30_000L)
326 if (delayMs < 30_000L) reconnectAttempts += 1
451 if (delayMs < 30_000L) reconnectAttempts += 1
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()
327− scope.launch {
455+ reconnectJob = scope.launch {
328 delay(delayMs)
456 delay(delayMs)
329 if (!sseClosedIntentionally && connected && screen != Screen.Connect) startSse()
457 if (!sseClosedIntentionally && connected && screen != Screen.Connect) startSse()
330 }
458 }
@@ -332,15 +460,20 @@ class OpencodeCore(
332
460
333 private fun handleServerEvent(json: String) {
461 private fun handleServerEvent(json: String) {
334 val event = Protocol.parseEvent(json) ?: return
462 val event = Protocol.parseEvent(json) ?: return
335− applyEvent(event)
463+ // Only re-render if the event actually changed our state; applyEvent returns
336− emit()
464+ // false for events that don't apply (other session, unknown message, …).
465+ if (applyEvent(event)) emit()
337 }
466 }
338
467
468+ /** Apply a server event to the model. Returns true iff any state actually changed. */
339− private fun applyEvent(event: Protocol.Event) {
469+ private fun applyEvent(event: Protocol.Event): Boolean {
340 when (event) {
470 when (event) {
341 is Protocol.Event.MessageUpdated -> {
471 is Protocol.Event.MessageUpdated -> {
342 val info = event.info
472 val info = event.info
343− 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
344 val streaming = info.time.completed == null && info.error == null
477 val streaming = info.time.completed == null && info.error == null
345 val existing = messages.firstOrNull { it.id == info.id }
478 val existing = messages.firstOrNull { it.id == info.id }
346 if (existing != null) {
479 if (existing != null) {
@@ -349,39 +482,59 @@ class OpencodeCore(
349 } else {
482 } else {
350 messages.add(MsgState(info.id, "assistant", info.time.created).apply { this.streaming = streaming })
483 messages.add(MsgState(info.id, "assistant", info.time.created).apply { this.streaming = streaming })
351 }
484 }
352− 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
353 }
487 }
354
488
355 is Protocol.Event.PartUpdated -> {
489 is Protocol.Event.PartUpdated -> {
356 val part = event.part
490 val part = event.part
357− val msg = messages.firstOrNull { it.id == part.messageId } ?: return
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.
358− when (part) {
494+ return when (part) {
359− is Protocol.Part.Text -> if (!part.synthetic) {
495+ is Protocol.Part.Text -> if (part.synthetic) {
496+ false
497+ } else {
360 msg.textParts[part.id] = pickText(part.text, event.delta, msg.textParts[part.id])
498 msg.textParts[part.id] = pickText(part.text, event.delta, msg.textParts[part.id])
499+ true
361 }
500 }
362− is Protocol.Part.Reasoning ->
501+ is Protocol.Part.Reasoning -> {
363 msg.reasoningParts[part.id] = pickText(part.text, event.delta, msg.reasoningParts[part.id])
502 msg.reasoningParts[part.id] = pickText(part.text, event.delta, msg.reasoningParts[part.id])
503+ true
504+ }
364− is Protocol.Part.Tool ->
505+ is Protocol.Part.Tool -> {
365 msg.toolParts[part.id] = ToolView(part.tool, part.status, part.title)
506 msg.toolParts[part.id] = ToolView(part.tool, part.status, part.title)
507+ true
508+ }
366 }
509 }
367 }
510 }
368
511
512+ is Protocol.Event.PartRemoved -> {
369− is Protocol.Event.PartRemoved -> messages.firstOrNull { it.id == event.messageId }?.remove(event.partId)
513+ val msg = messages.firstOrNull { it.id == event.messageId } ?: return false
514+ msg.remove(event.partId)
515+ return true
516+ }
370
517
518+ is Protocol.Event.MessageRemoved -> {
519+ // Remember the ID so a late MessageUpdated can't resurrect the message.
520+ removedMessageIds.add(event.messageId)
371− is Protocol.Event.MessageRemoved -> messages.removeAll { it.id == event.messageId }
521+ return messages.removeAll { it.id == event.messageId }
522+ }
372
523
373− is Protocol.Event.SessionIdle -> if (isCurrent(event.sessionId)) {
524+ is Protocol.Event.SessionIdle -> {
525+ if (!isCurrent(event.sessionId)) return false
374 generating = false
526 generating = false
375 messages.forEach { it.streaming = false }
527 messages.forEach { it.streaming = false }
528+ return true
376 }
529 }
377
530
378 is Protocol.Event.SessionError -> {
531 is Protocol.Event.SessionError -> {
379 val scoped = event.sessionId == null || isCurrent(event.sessionId)
532 val scoped = event.sessionId == null || isCurrent(event.sessionId)
380− if (scoped) {
533+ if (!scoped) return false
381− event.error?.let { error = it.message() }
534+ event.error?.let { error = it.message() }
382− generating = false
535+ generating = false
383− messages.forEach { it.streaming = false }
536+ messages.forEach { it.streaming = false }
384− }
537+ return true
385 }
538 }
386
539
387 is Protocol.Event.SessionUpserted -> {
540 is Protocol.Event.SessionUpserted -> {
@@ -392,19 +545,26 @@ class OpencodeCore(
392 } else {
545 } else {
393 sessions.add(0, SessionView(event.info.id, title))
546 sessions.add(0, SessionView(event.info.id, title))
394 }
547 }
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() }
395 if (isCurrent(event.info.id)) currentSessionTitle = title
551 if (isCurrent(event.info.id)) currentSessionTitle = title
552+ return true
396 }
553 }
397
554
398− is Protocol.Event.SessionDeleted -> event.sessionId?.let { id ->
555+ is Protocol.Event.SessionDeleted -> {
556+ val id = event.sessionId ?: return false
399− sessions.removeAll { it.id == id }
557+ val removed = sessions.removeAll { it.id == id }
400 if (isCurrent(id)) {
558 if (isCurrent(id)) {
401 currentSessionId = null
559 currentSessionId = null
402 messages.clear()
560 messages.clear()
403 screen = Screen.Sessions
561 screen = Screen.Sessions
562+ return true
404 }
563 }
564+ return removed
405 }
565 }
406
566
407− Protocol.Event.Other -> Unit
567+ Protocol.Event.Other -> return false
408 }
568 }
409 }
569 }
410
570
app/src/main/java/soy/iko/opencode/core/SseClient.kt+1
@@ -33,6 +33,7 @@ class SseClient(
33
33
34 fun connect() {
34 fun connect() {
35 val builder = Request.Builder().url(url).header("Accept", "text/event-stream")
35 val builder = Request.Builder().url(url).header("Accept", "text/event-stream")
36+ builder.header("User-Agent", "opencode-android/1.0")
36 if (auth != null) builder.header("Authorization", auth)
37 if (auth != null) builder.header("Authorization", auth)
37 source = EventSources.createFactory(client).newEventSource(builder.build(), listener)
38 source = EventSources.createFactory(client).newEventSource(builder.build(), listener)
38 }
39 }
app/src/main/java/soy/iko/opencode/ui/OpencodeApp.kt+71−7
@@ -2,18 +2,24 @@ package soy.iko.opencode.ui
2
2
3import androidx.activity.compose.BackHandler
3import androidx.activity.compose.BackHandler
4import androidx.compose.animation.AnimatedContent
4import androidx.compose.animation.AnimatedContent
5−import androidx.compose.animation.SizeTransform
5+import androidx.compose.animation.slideInHorizontally
6−import androidx.compose.animation.fadeIn
6+import androidx.compose.animation.slideOutHorizontally
7−import androidx.compose.animation.fadeOut
8import androidx.compose.animation.togetherWith
7import androidx.compose.animation.togetherWith
9import androidx.compose.foundation.layout.fillMaxSize
8import androidx.compose.foundation.layout.fillMaxSize
10import androidx.compose.foundation.layout.padding
9import androidx.compose.foundation.layout.padding
10+import androidx.compose.material3.AlertDialog
11import androidx.compose.material3.Scaffold
11import androidx.compose.material3.Scaffold
12+import androidx.compose.material3.SnackbarDuration
12import androidx.compose.material3.SnackbarHost
13import androidx.compose.material3.SnackbarHost
13import androidx.compose.material3.SnackbarHostState
14import androidx.compose.material3.SnackbarHostState
15+import androidx.compose.material3.Text
16+import androidx.compose.material3.TextButton
14import androidx.compose.runtime.Composable
17import androidx.compose.runtime.Composable
15import androidx.compose.runtime.LaunchedEffect
18import androidx.compose.runtime.LaunchedEffect
19+import androidx.compose.runtime.getValue
20+import androidx.compose.runtime.mutableStateOf
16import androidx.compose.runtime.remember
21import androidx.compose.runtime.remember
22+import androidx.compose.runtime.setValue
17import androidx.compose.ui.Modifier
23import androidx.compose.ui.Modifier
18import soy.iko.opencode.core.Event
24import soy.iko.opencode.core.Event
19import soy.iko.opencode.core.Screen
25import soy.iko.opencode.core.Screen
@@ -25,28 +31,86 @@ import soy.iko.opencode.ui.screens.SessionsScreen
25@Composable
31@Composable
26fun OpencodeApp(state: UiState, dispatch: (Event) -> Unit) {
32fun OpencodeApp(state: UiState, dispatch: (Event) -> Unit) {
27 val snackbar = remember { SnackbarHostState() }
33 val snackbar = remember { SnackbarHostState() }
34+ var showStopConfirm by remember { mutableStateOf(false) }
35+ var showDisconnectConfirm by remember { mutableStateOf(false) }
28
36
29 LaunchedEffect(state.error) {
37 LaunchedEffect(state.error) {
30 state.error?.let {
38 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.
32 dispatch(Event.DismissError)
45 dispatch(Event.DismissError)
33 }
46 }
34 }
47 }
35
48
36 BackHandler(enabled = state.screen != Screen.Connect) {
49 BackHandler(enabled = state.screen != Screen.Connect) {
37 when (state.screen) {
50 when (state.screen) {
51+ // Leaving mid-generation would silently drop the reply, so confirm first.
38− Screen.Chat -> dispatch(Event.NavigateToSessions)
52+ Screen.Chat -> if (state.generating) showStopConfirm = true else dispatch(Event.NavigateToSessions)
39− Screen.Sessions -> dispatch(Event.NavigateToConnect)
53+ // Disconnecting drops the server session, so guard the back button.
54+ Screen.Sessions -> showDisconnectConfirm = true
40 Screen.Connect -> Unit
55 Screen.Connect -> Unit
41 }
56 }
42 }
57 }
43
58
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+
44 Scaffold(snackbarHost = { SnackbarHost(snackbar) }) { padding ->
102 Scaffold(snackbarHost = { SnackbarHost(snackbar) }) { padding ->
45 AnimatedContent(
103 AnimatedContent(
46 targetState = state.screen,
104 targetState = state.screen,
47 modifier = Modifier.fillMaxSize().padding(padding),
105 modifier = Modifier.fillMaxSize().padding(padding),
48 transitionSpec = {
106 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+ }
50 },
114 },
51 label = "screen",
115 label = "screen",
52 ) { screen ->
116 ) { screen ->
app/src/main/java/soy/iko/opencode/ui/components/MessageBubble.kt+295−27
@@ -3,6 +3,8 @@ package soy.iko.opencode.ui.components
3import android.content.ClipData
3import android.content.ClipData
4import android.content.ClipboardManager
4import android.content.ClipboardManager
5import android.content.Context
5import android.content.Context
6+import android.content.Intent
7+import android.net.Uri
6import android.widget.Toast
8import android.widget.Toast
7import androidx.compose.foundation.ExperimentalFoundationApi
9import androidx.compose.foundation.ExperimentalFoundationApi
8import androidx.compose.foundation.background
10import androidx.compose.foundation.background
@@ -13,14 +15,20 @@ import androidx.compose.foundation.layout.Column
13import androidx.compose.foundation.layout.Row
15import androidx.compose.foundation.layout.Row
14import androidx.compose.foundation.layout.fillMaxWidth
16import androidx.compose.foundation.layout.fillMaxWidth
15import androidx.compose.foundation.layout.padding
17import androidx.compose.foundation.layout.padding
18+import androidx.compose.foundation.layout.size
16import androidx.compose.foundation.layout.widthIn
19import androidx.compose.foundation.layout.widthIn
17import androidx.compose.foundation.shape.RoundedCornerShape
20import androidx.compose.foundation.shape.RoundedCornerShape
21+import androidx.compose.foundation.text.selection.SelectionContainer
18import androidx.compose.material.icons.Icons
22import androidx.compose.material.icons.Icons
19import androidx.compose.material.icons.filled.Build
23import androidx.compose.material.icons.filled.Build
24+import androidx.compose.material.icons.filled.Check
20import androidx.compose.material.icons.filled.ErrorOutline
25import androidx.compose.material.icons.filled.ErrorOutline
21import androidx.compose.material.icons.filled.ExpandLess
26import androidx.compose.material.icons.filled.ExpandLess
22import androidx.compose.material.icons.filled.ExpandMore
27import androidx.compose.material.icons.filled.ExpandMore
23import androidx.compose.material.icons.filled.Schedule
28import 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
24import androidx.compose.material3.Icon
32import androidx.compose.material3.Icon
25import androidx.compose.material3.MaterialTheme
33import androidx.compose.material3.MaterialTheme
26import androidx.compose.material3.Text
34import androidx.compose.material3.Text
@@ -34,22 +42,30 @@ import androidx.compose.ui.Modifier
34import androidx.compose.ui.draw.clip
42import androidx.compose.ui.draw.clip
35import androidx.compose.ui.graphics.Color
43import androidx.compose.ui.graphics.Color
36import androidx.compose.ui.platform.LocalContext
44import 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
37import androidx.compose.ui.text.font.FontStyle
51import 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
38import androidx.compose.ui.text.style.TextOverflow
56import androidx.compose.ui.text.style.TextOverflow
39import androidx.compose.ui.unit.dp
57import androidx.compose.ui.unit.dp
40−import java.text.SimpleDateFormat
58+import androidx.compose.ui.unit.sp
41−import java.util.Date
42−import java.util.Locale
43import soy.iko.opencode.core.MessageStatus
59import soy.iko.opencode.core.MessageStatus
44import soy.iko.opencode.core.MessageView
60import soy.iko.opencode.core.MessageView
45import soy.iko.opencode.core.ToolView
61import soy.iko.opencode.core.ToolView
46import soy.iko.opencode.ui.theme.MonoStyle
62import soy.iko.opencode.ui.theme.MonoStyle
47
63
48−private val timeFormat = SimpleDateFormat("HH:mm", Locale.getDefault())
64+private val timeFormat = java.time.format.DateTimeFormatter.ofPattern("HH:mm")
49
65
50@OptIn(ExperimentalFoundationApi::class)
66@OptIn(ExperimentalFoundationApi::class)
51@Composable
67@Composable
52−fun MessageBubble(message: MessageView, modifier: Modifier = Modifier) {
68+fun MessageBubble(message: MessageView, modifier: Modifier = Modifier, onRetry: () -> Unit = {}, compactSpacing: Boolean = false) {
53 val isUser = message.isUser
69 val isUser = message.isUser
54 val bubbleColor = if (isUser) MaterialTheme.colorScheme.primaryContainer else MaterialTheme.colorScheme.surfaceVariant
70 val bubbleColor = if (isUser) MaterialTheme.colorScheme.primaryContainer else MaterialTheme.colorScheme.surfaceVariant
55 val textColor = if (isUser) MaterialTheme.colorScheme.onPrimaryContainer else MaterialTheme.colorScheme.onSurface
71 val textColor = if (isUser) MaterialTheme.colorScheme.onPrimaryContainer else MaterialTheme.colorScheme.onSurface
@@ -62,11 +78,20 @@ fun MessageBubble(message: MessageView, modifier: Modifier = Modifier) {
62 )
78 )
63
79
64 val context = LocalContext.current
80 val context = LocalContext.current
81+ var showMenu by remember { mutableStateOf(false) }
65
82
66 Row(
83 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),
68 horizontalArrangement = if (isUser) Arrangement.End else Arrangement.Start,
85 horizontalArrangement = if (isUser) Arrangement.End else Arrangement.Start,
69 ) {
86 ) {
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+ }
70 Column(
95 Column(
71 modifier = Modifier
96 modifier = Modifier
72 .fillMaxWidth(0.85f)
97 .fillMaxWidth(0.85f)
@@ -77,30 +102,72 @@ fun MessageBubble(message: MessageView, modifier: Modifier = Modifier) {
77 onClick = {},
102 onClick = {},
78 onLongClick = {
103 onLongClick = {
79 if (message.text.isNotEmpty()) {
104 if (message.text.isNotEmpty()) {
80− copyToClipboard(context, message.text)
105+ showMenu = true
81− Toast.makeText(context, "Copied", Toast.LENGTH_SHORT).show()
82 }
106 }
83 },
107 },
84 )
108 )
85 .padding(horizontal = 14.dp, vertical = 10.dp),
109 .padding(horizontal = 14.dp, vertical = 10.dp),
86 ) {
110 ) {
87− 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+ }
88
128
89− message.tools.forEach { ToolRow(it) }
129+ message.reasoning?.let { ReasoningBlock(it, context) }
90
130
131+ // Give a quick overview of how many tools ran before listing them.
91− if (message.text.isNotEmpty()) {
132+ if (message.tools.size > 1) {
92 Text(
133 Text(
93− text = message.text + if (message.streaming) " ▌" else "",
134+ text = "${message.tools.size} tool calls",
94− color = textColor,
135+ style = MaterialTheme.typography.labelSmall,
95− style = MaterialTheme.typography.bodyLarge,
136+ color = MaterialTheme.colorScheme.onSurfaceVariant,
137+ modifier = Modifier.padding(bottom = 4.dp),
96 )
138 )
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+ )
97− } else if (message.streaming && message.tools.isEmpty() && message.reasoning == null) {
163+ } else if (message.streaming && message.tools.isEmpty() && message.reasoning == null) {
98− Text("▌", color = textColor, style = MaterialTheme.typography.bodyLarge)
164+ Text("▌", color = textColor, style = MaterialTheme.typography.bodyLarge)
165+ }
99 }
166 }
100
167
101 if (message.time > 0) {
168 if (message.time > 0) {
102 Text(
169 Text(
103− text = timeFormat.format(Date(message.time)),
170+ text = timeFormat.format(java.time.Instant.ofEpochMilli(message.time).atZone(java.time.ZoneId.systemDefault()).toLocalTime()),
104 style = MaterialTheme.typography.labelSmall,
171 style = MaterialTheme.typography.labelSmall,
105 color = mutedColor,
172 color = mutedColor,
106 modifier = Modifier.padding(top = 4.dp),
173 modifier = Modifier.padding(top = 4.dp),
@@ -108,7 +175,7 @@ fun MessageBubble(message: MessageView, modifier: Modifier = Modifier) {
108 }
175 }
109
176
110 if (isUser && message.status != MessageStatus.Sent) {
177 if (isUser && message.status != MessageStatus.Sent) {
111− StatusLine(message.status)
178+ StatusLine(message.status, onRetry = onRetry)
112 }
179 }
113 }
180 }
114 }
181 }
@@ -119,8 +186,172 @@ private fun copyToClipboard(context: Context, text: String) {
119 clipboard.setPrimaryClip(ClipData.newPlainText("message", text))
186 clipboard.setPrimaryClip(ClipData.newPlainText("message", text))
120}
187}
121
188
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+
122@Composable
353@Composable
123−private fun ReasoningBlock(text: String) {
354+private fun ReasoningBlock(text: String, context: Context) {
124 var expanded by remember { mutableStateOf(false) }
355 var expanded by remember { mutableStateOf(false) }
125 Column(modifier = Modifier.padding(bottom = 6.dp)) {
356 Column(modifier = Modifier.padding(bottom = 6.dp)) {
126 Row(
357 Row(
@@ -146,6 +377,17 @@ private fun ReasoningBlock(text: String) {
146 color = MaterialTheme.colorScheme.onSurfaceVariant,
377 color = MaterialTheme.colorScheme.onSurfaceVariant,
147 modifier = Modifier.padding(top = 4.dp),
378 modifier = Modifier.padding(top = 4.dp),
148 )
379 )
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+ )
149 }
391 }
150 }
392 }
151}
393}
@@ -153,6 +395,13 @@ private fun ReasoningBlock(text: String) {
153@Composable
395@Composable
154private fun ToolRow(tool: ToolView) {
396private fun ToolRow(tool: ToolView) {
155 var expanded by remember { mutableStateOf(false) }
397 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+ }
156 Row(
405 Row(
157 verticalAlignment = Alignment.CenterVertically,
406 verticalAlignment = Alignment.CenterVertically,
158 modifier = Modifier
407 modifier = Modifier
@@ -167,14 +416,23 @@ private fun ToolRow(tool: ToolView) {
167 modifier = Modifier.padding(end = 6.dp).widthIn(max = 16.dp),
416 modifier = Modifier.padding(end = 6.dp).widthIn(max = 16.dp),
168 )
417 )
169 Text(
418 Text(
170− text = buildString {
419+ text = buildAnnotatedString {
420+ withStyle(SpanStyle(color = baseColor)) {
171− append(tool.name)
421+ append(tool.name)
172− append(" · ")
422+ append(" · ")
423+ }
424+ withStyle(SpanStyle(color = statusColor)) {
173− append(tool.status)
425+ append(tool.status)
426+ }
174− if (expanded) tool.title?.let { append(" — "); append(it) }
427+ if (expanded) tool.title?.let {
428+ withStyle(SpanStyle(color = baseColor)) {
429+ append(" — ")
430+ append(it)
431+ }
432+ }
175 },
433 },
176 style = MonoStyle,
434 style = MonoStyle,
177− color = MaterialTheme.colorScheme.onSurfaceVariant,
435+ color = baseColor,
178 maxLines = if (expanded) Int.MAX_VALUE else 1,
436 maxLines = if (expanded) Int.MAX_VALUE else 1,
179 overflow = TextOverflow.Ellipsis,
437 overflow = TextOverflow.Ellipsis,
180 modifier = Modifier.weight(1f),
438 modifier = Modifier.weight(1f),
@@ -189,11 +447,11 @@ private fun ToolRow(tool: ToolView) {
189}
447}
190
448
191@Composable
449@Composable
192−private fun StatusLine(status: MessageStatus) {
450+private fun StatusLine(status: MessageStatus, onRetry: () -> Unit) {
193 val (icon, label, tint) = when (status) {
451 val (icon, label, tint) = when (status) {
194 MessageStatus.Pending -> Triple(Icons.Filled.Schedule, "Sending…", MaterialTheme.colorScheme.onPrimaryContainer)
452 MessageStatus.Pending -> Triple(Icons.Filled.Schedule, "Sending…", MaterialTheme.colorScheme.onPrimaryContainer)
195 MessageStatus.Failed -> Triple(Icons.Filled.ErrorOutline, "Failed to send", MaterialTheme.colorScheme.error)
453 MessageStatus.Failed -> Triple(Icons.Filled.ErrorOutline, "Failed to send", MaterialTheme.colorScheme.error)
196− MessageStatus.Sent -> Triple(Icons.Filled.Schedule, "", Color.Unspecified)
454+ MessageStatus.Sent -> Triple(Icons.Filled.Check, "", Color.Unspecified)
197 }
455 }
198 Row(
456 Row(
199 verticalAlignment = Alignment.CenterVertically,
457 verticalAlignment = Alignment.CenterVertically,
@@ -207,5 +465,15 @@ private fun StatusLine(status: MessageStatus) {
207 color = tint,
465 color = tint,
208 modifier = Modifier.padding(start = 4.dp),
466 modifier = Modifier.padding(start = 4.dp),
209 )
467 )
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+ }
210 }
478 }
211}
479}
app/src/main/java/soy/iko/opencode/ui/screens/ChatScreen.kt+355−26
@@ -1,34 +1,57 @@
1package soy.iko.opencode.ui.screens
1package soy.iko.opencode.ui.screens
2
2
3+import android.content.ClipData
4+import android.content.ClipboardManager
5+import android.content.Context
6+import android.widget.Toast
3import androidx.compose.animation.AnimatedVisibility
7import 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
4import androidx.compose.animation.fadeIn
13import androidx.compose.animation.fadeIn
5import androidx.compose.animation.fadeOut
14import androidx.compose.animation.fadeOut
6import androidx.compose.animation.scaleIn
15import androidx.compose.animation.scaleIn
7import androidx.compose.animation.scaleOut
16import 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
8import androidx.compose.foundation.layout.Arrangement
21import androidx.compose.foundation.layout.Arrangement
9import androidx.compose.foundation.layout.Box
22import androidx.compose.foundation.layout.Box
10import androidx.compose.foundation.layout.BoxScope
23import androidx.compose.foundation.layout.BoxScope
11import androidx.compose.foundation.layout.Column
24import androidx.compose.foundation.layout.Column
12import androidx.compose.foundation.layout.Row
25import androidx.compose.foundation.layout.Row
26+import androidx.compose.foundation.layout.Spacer
13import androidx.compose.foundation.layout.fillMaxSize
27import androidx.compose.foundation.layout.fillMaxSize
14import androidx.compose.foundation.layout.fillMaxWidth
28import androidx.compose.foundation.layout.fillMaxWidth
29+import androidx.compose.foundation.layout.heightIn
15import androidx.compose.foundation.layout.PaddingValues
30import androidx.compose.foundation.layout.PaddingValues
31+import androidx.compose.foundation.layout.WindowInsets
32+import androidx.compose.foundation.layout.ime
16import androidx.compose.foundation.layout.imePadding
33import androidx.compose.foundation.layout.imePadding
17import androidx.compose.foundation.layout.padding
34import androidx.compose.foundation.layout.padding
18import androidx.compose.foundation.layout.size
35import androidx.compose.foundation.layout.size
19import androidx.compose.foundation.lazy.LazyColumn
36import androidx.compose.foundation.lazy.LazyColumn
20−import androidx.compose.foundation.lazy.items
37+import androidx.compose.foundation.lazy.itemsIndexed
21import androidx.compose.foundation.lazy.rememberLazyListState
38import androidx.compose.foundation.lazy.rememberLazyListState
39+import androidx.compose.foundation.shape.CircleShape
40+import androidx.compose.foundation.shape.RoundedCornerShape
22import androidx.compose.foundation.text.KeyboardActions
41import androidx.compose.foundation.text.KeyboardActions
23import androidx.compose.foundation.text.KeyboardOptions
42import androidx.compose.foundation.text.KeyboardOptions
24import androidx.compose.material.icons.Icons
43import androidx.compose.material.icons.Icons
25import androidx.compose.material.icons.automirrored.filled.ArrowBack
44import androidx.compose.material.icons.automirrored.filled.ArrowBack
26import androidx.compose.material.icons.automirrored.filled.Send
45import androidx.compose.material.icons.automirrored.filled.Send
46+import androidx.compose.material.icons.filled.ChatBubbleOutline
47+import androidx.compose.material.icons.filled.Close
27import androidx.compose.material.icons.filled.KeyboardArrowDown
48import androidx.compose.material.icons.filled.KeyboardArrowDown
49+import androidx.compose.material.icons.filled.KeyboardArrowUp
28import androidx.compose.material.icons.filled.Stop
50import androidx.compose.material.icons.filled.Stop
29import androidx.compose.material3.CircularProgressIndicator
51import androidx.compose.material3.CircularProgressIndicator
30import androidx.compose.material3.ExperimentalMaterial3Api
52import androidx.compose.material3.ExperimentalMaterial3Api
31import androidx.compose.material3.FilledIconButton
53import androidx.compose.material3.FilledIconButton
54+import androidx.compose.material3.FilledTonalButton
32import androidx.compose.material3.FloatingActionButton
55import androidx.compose.material3.FloatingActionButton
33import androidx.compose.material3.Icon
56import androidx.compose.material3.Icon
34import androidx.compose.material3.IconButton
57import androidx.compose.material3.IconButton
@@ -38,18 +61,27 @@ import androidx.compose.material3.OutlinedTextField
38import androidx.compose.material3.Scaffold
61import androidx.compose.material3.Scaffold
39import androidx.compose.material3.Surface
62import androidx.compose.material3.Surface
40import androidx.compose.material3.Text
63import androidx.compose.material3.Text
64+import androidx.compose.material3.TextButton
41import androidx.compose.material3.TopAppBar
65import androidx.compose.material3.TopAppBar
42import androidx.compose.runtime.Composable
66import androidx.compose.runtime.Composable
43import androidx.compose.runtime.LaunchedEffect
67import androidx.compose.runtime.LaunchedEffect
44import androidx.compose.runtime.derivedStateOf
68import androidx.compose.runtime.derivedStateOf
45import androidx.compose.runtime.getValue
69import androidx.compose.runtime.getValue
70+import androidx.compose.runtime.mutableStateOf
46import androidx.compose.runtime.remember
71import androidx.compose.runtime.remember
47import androidx.compose.runtime.rememberCoroutineScope
72import androidx.compose.runtime.rememberCoroutineScope
73+import androidx.compose.runtime.setValue
48import androidx.compose.ui.Alignment
74import androidx.compose.ui.Alignment
49import androidx.compose.ui.Modifier
75import androidx.compose.ui.Modifier
76+import androidx.compose.ui.draw.clip
50import androidx.compose.ui.hapticfeedback.HapticFeedbackType
77import 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
51import androidx.compose.ui.platform.LocalHapticFeedback
81import androidx.compose.ui.platform.LocalHapticFeedback
82+import androidx.compose.ui.text.font.FontStyle
52import androidx.compose.ui.text.input.ImeAction
83import androidx.compose.ui.text.input.ImeAction
84+import androidx.compose.ui.text.style.TextAlign
53import androidx.compose.ui.text.style.TextOverflow
85import androidx.compose.ui.text.style.TextOverflow
54import androidx.compose.ui.unit.dp
86import androidx.compose.ui.unit.dp
55import kotlinx.coroutines.launch
87import kotlinx.coroutines.launch
@@ -57,11 +89,17 @@ import soy.iko.opencode.core.Event
57import soy.iko.opencode.core.UiState
89import soy.iko.opencode.core.UiState
58import soy.iko.opencode.ui.components.MessageBubble
90import soy.iko.opencode.ui.components.MessageBubble
59
91
60−@OptIn(ExperimentalMaterial3Api::class)
92+@OptIn(ExperimentalMaterial3Api::class, ExperimentalFoundationApi::class)
61@Composable
93@Composable
62fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
94fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
63 val listState = rememberLazyListState()
95 val listState = rememberLazyListState()
64 val coroutineScope = rememberCoroutineScope()
96 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
65
103
66 // Show a jump-to-bottom button once the user scrolls up away from the latest
104 // Show a jump-to-bottom button once the user scrolls up away from the latest
67 // message. derivedStateOf keeps this from recomposing on every pixel of scroll —
105 // message. derivedStateOf keeps this from recomposing on every pixel of scroll —
@@ -75,6 +113,22 @@ fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
75 }
113 }
76 }
114 }
77
115
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+
78 // Keep the newest content in view as it streams in, but only when the user is
132 // Keep the newest content in view as it streams in, but only when the user is
79 // already near the bottom. If they've scrolled up to read history, leave it alone.
133 // already near the bottom. If they've scrolled up to read history, leave it alone.
80 val lastLen = state.messages.lastOrNull()?.text?.length ?: 0
134 val lastLen = state.messages.lastOrNull()?.text?.length ?: 0
@@ -94,10 +148,20 @@ fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
94 topBar = {
148 topBar = {
95 TopAppBar(
149 TopAppBar(
96 title = {
150 title = {
151+ val title = state.currentSessionTitle.ifEmpty { "Chat" }
97 Text(
152 Text(
98− text = state.currentSessionTitle.ifEmpty { "Chat" },
153+ text = if (state.messages.isNotEmpty()) "$title (${state.messages.size})" else title,
99 maxLines = 1,
154 maxLines = 1,
100 overflow = TextOverflow.Ellipsis,
155 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+ ),
101 )
165 )
102 },
166 },
103 navigationIcon = {
167 navigationIcon = {
@@ -119,6 +183,7 @@ fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
119 ChatInputBar(
183 ChatInputBar(
120 draft = state.draft,
184 draft = state.draft,
121 generating = state.generating,
185 generating = state.generating,
186+ loading = state.loading && state.messages.isEmpty(),
122 onDraftChange = { dispatch(Event.DraftChanged(it)) },
187 onDraftChange = { dispatch(Event.DraftChanged(it)) },
123 onSend = { dispatch(Event.SendMessage(state.draft)) },
188 onSend = { dispatch(Event.SendMessage(state.draft)) },
124 onStop = { dispatch(Event.CancelGeneration) },
189 onStop = { dispatch(Event.CancelGeneration) },
@@ -132,34 +197,86 @@ fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
132 if (state.loading && state.messages.isEmpty()) {
197 if (state.loading && state.messages.isEmpty()) {
133 LinearProgressIndicator(Modifier.fillMaxWidth())
198 LinearProgressIndicator(Modifier.fillMaxWidth())
134 }
199 }
200+ if (state.error != null && state.messages.isNotEmpty()) {
201+ ErrorBanner(
202+ message = state.error,
203+ onDismiss = { dispatch(Event.DismissError) },
204+ )
205+ }
135 Box(Modifier.fillMaxWidth().weight(1f)) {
206 Box(Modifier.fillMaxWidth().weight(1f)) {
136 if (state.messages.isEmpty() && !state.loading) {
207 if (state.messages.isEmpty() && !state.loading) {
137− Text(
208+ Column(
138− text = "Send a message to start the conversation.",
139− style = MaterialTheme.typography.bodyMedium,
140− color = MaterialTheme.colorScheme.onSurfaceVariant,
141 modifier = Modifier.align(Alignment.Center).padding(32.dp),
209 modifier = Modifier.align(Alignment.Center).padding(32.dp),
210+ horizontalAlignment = Alignment.CenterHorizontally,
142− )
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+ }
143 }
226 }
144 LazyColumn(
227 LazyColumn(
145 state = listState,
228 state = listState,
146 modifier = Modifier.fillMaxSize(),
229 modifier = Modifier.fillMaxSize(),
147 contentPadding = PaddingValues(vertical = 8.dp),
230 contentPadding = PaddingValues(vertical = 8.dp),
148 ) {
231 ) {
149− items(state.messages, key = { it.id }) { 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
150− MessageBubble(message)
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+ }
151 }
253 }
152 }
254 }
153 ScrollToBottomButton(
255 ScrollToBottomButton(
154− visible = showScrollButton,
256+ visible = showScrollButton && !isImeVisible,
257+ showBadge = hasUnseenMessages,
155 onClick = {
258 onClick = {
156 coroutineScope.launch {
259 coroutineScope.launch {
157 if (state.messages.isNotEmpty()) {
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
158− listState.animateScrollToItem(state.messages.size - 1)
266+ state.messages.size - 1
267+ listState.animateScrollToItem(target)
159 }
268 }
160 }
269 }
161 },
270 },
162 )
271 )
272+ ScrollToTopButton(
273+ visible = showScrollTop,
274+ onClick = {
275+ coroutineScope.launch {
276+ listState.animateScrollToItem(0)
277+ }
278+ },
279+ )
163 }
280 }
164 }
281 }
165 }
282 }
@@ -169,23 +286,149 @@ fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
169// resolves cleanly. Called inline inside a Column, the enclosing ColumnScope receiver
286// resolves cleanly. Called inline inside a Column, the enclosing ColumnScope receiver
170// would otherwise capture the ColumnScope.AnimatedVisibility overload and fail to compile.
287// would otherwise capture the ColumnScope.AnimatedVisibility overload and fail to compile.
171@Composable
288@Composable
172−private fun BoxScope.ScrollToBottomButton(visible: Boolean, onClick: () -> Unit) {
289+private fun BoxScope.ScrollToBottomButton(visible: Boolean, showBadge: Boolean, onClick: () -> Unit) {
173 AnimatedVisibility(
290 AnimatedVisibility(
174 visible = visible,
291 visible = visible,
175 modifier = Modifier.align(Alignment.BottomEnd),
292 modifier = Modifier.align(Alignment.BottomEnd),
176 enter = scaleIn() + fadeIn(),
293 enter = scaleIn() + fadeIn(),
177 exit = scaleOut() + fadeOut(),
294 exit = scaleOut() + fadeOut(),
178 ) {
295 ) {
296+ Box(Modifier.padding(16.dp)) {
179− FloatingActionButton(
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(
180− onClick = onClick,
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(
181 modifier = Modifier
400 modifier = Modifier
401+ .clip(shape)
402+ .background(MaterialTheme.colorScheme.surfaceVariant)
182− .padding(16.dp)
403+ .padding(horizontal = 14.dp, vertical = 12.dp),
183− .size(44.dp),
404+ verticalAlignment = Alignment.CenterVertically,
405+ horizontalArrangement = Arrangement.spacedBy(6.dp),
184 ) {
406 ) {
185− Icon(
407+ Text(
186− Icons.Filled.KeyboardArrowDown,
408+ text = "Thinking",
187− contentDescription = "Scroll to latest message",
409+ style = MaterialTheme.typography.bodyMedium.copy(fontStyle = FontStyle.Italic),
410+ color = MaterialTheme.colorScheme.onSurfaceVariant,
188 )
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(
425+ modifier = Modifier
426+ .size(6.dp)
427+ .clip(CircleShape)
428+ .background(MaterialTheme.colorScheme.onSurfaceVariant.copy(alpha = alpha)),
429+ )
430+ }
431+ }
189 }
432 }
190 }
433 }
191}
434}
@@ -215,35 +458,109 @@ private fun ReconnectingBanner() {
215 }
458 }
216}
459}
217
460
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+
218@Composable
490@Composable
219private fun ChatInputBar(
491private fun ChatInputBar(
220 draft: String,
492 draft: String,
221 generating: Boolean,
493 generating: Boolean,
494+ loading: Boolean,
222 onDraftChange: (String) -> Unit,
495 onDraftChange: (String) -> Unit,
223 onSend: () -> Unit,
496 onSend: () -> Unit,
224 onStop: () -> Unit,
497 onStop: () -> Unit,
225) {
498) {
226 val haptic = LocalHapticFeedback.current
499 val haptic = LocalHapticFeedback.current
500+ val clipboardManager = LocalClipboardManager.current
227 Surface(tonalElevation = 3.dp) {
501 Surface(tonalElevation = 3.dp) {
228− Row(
502+ Column(
229 modifier = Modifier.fillMaxWidth().imePadding().padding(horizontal = 12.dp, vertical = 8.dp),
503 modifier = Modifier.fillMaxWidth().imePadding().padding(horizontal = 12.dp, vertical = 8.dp),
504+ ) {
505+ Row(
506+ modifier = Modifier.fillMaxWidth(),
230 verticalAlignment = Alignment.Bottom,
507 verticalAlignment = Alignment.Bottom,
231 horizontalArrangement = Arrangement.spacedBy(8.dp),
508 horizontalArrangement = Arrangement.spacedBy(8.dp),
232 ) {
509 ) {
233 OutlinedTextField(
510 OutlinedTextField(
234 value = draft,
511 value = draft,
235 onValueChange = onDraftChange,
512 onValueChange = onDraftChange,
236− modifier = Modifier.weight(1f),
513+ modifier = Modifier.weight(1f).heightIn(min = 48.dp),
514+ enabled = !loading,
237 placeholder = { Text("Message opencode…") },
515 placeholder = { Text("Message opencode…") },
238 maxLines = 5,
516 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+ },
239 keyboardOptions = KeyboardOptions(imeAction = ImeAction.Send),
550 keyboardOptions = KeyboardOptions(imeAction = ImeAction.Send),
240 keyboardActions = KeyboardActions(onSend = {
551 keyboardActions = KeyboardActions(onSend = {
241− if (!generating && draft.isNotBlank()) onSend()
552+ if (!generating && !loading && draft.isNotBlank()) onSend()
242 }),
553 }),
243 )
554 )
244 if (generating) {
555 if (generating) {
245− FilledIconButton(onClick = onStop) {
556+ FilledTonalButton(onClick = onStop) {
557+ Icon(
558+ Icons.Filled.Stop,
246− Icon(Icons.Filled.Stop, contentDescription = "Stop generating")
559+ contentDescription = null,
560+ modifier = Modifier.size(18.dp),
561+ )
562+ Spacer(Modifier.size(6.dp))
563+ Text("Stop")
247 }
564 }
248 } else {
565 } else {
249 FilledIconButton(
566 FilledIconButton(
@@ -251,11 +568,23 @@ private fun ChatInputBar(
251 haptic.performHapticFeedback(HapticFeedbackType.LongPress)
568 haptic.performHapticFeedback(HapticFeedbackType.LongPress)
252 onSend()
569 onSend()
253 },
570 },
254− enabled = draft.isNotBlank(),
571+ enabled = draft.isNotBlank() && !loading,
255 ) {
572 ) {
256 Icon(Icons.AutoMirrored.Filled.Send, contentDescription = "Send")
573 Icon(Icons.AutoMirrored.Filled.Send, contentDescription = "Send")
257 }
574 }
258 }
575 }
259 }
576 }
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+ }
260 }
589 }
261}
590}
app/src/main/java/soy/iko/opencode/ui/screens/ConnectScreen.kt+100−4
@@ -8,6 +8,7 @@ import androidx.compose.foundation.layout.fillMaxWidth
8import androidx.compose.foundation.layout.height
8import androidx.compose.foundation.layout.height
9import androidx.compose.foundation.layout.imePadding
9import androidx.compose.foundation.layout.imePadding
10import androidx.compose.foundation.layout.padding
10import androidx.compose.foundation.layout.padding
11+import androidx.compose.foundation.layout.size
11import androidx.compose.foundation.rememberScrollState
12import androidx.compose.foundation.rememberScrollState
12import androidx.compose.foundation.text.KeyboardActions
13import androidx.compose.foundation.text.KeyboardActions
13import androidx.compose.foundation.text.KeyboardOptions
14import androidx.compose.foundation.text.KeyboardOptions
@@ -15,18 +16,30 @@ import androidx.compose.foundation.verticalScroll
15import androidx.compose.material.icons.Icons
16import androidx.compose.material.icons.Icons
16import androidx.compose.material.icons.filled.Lock
17import androidx.compose.material.icons.filled.Lock
17import androidx.compose.material.icons.filled.Person
18import 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
18import androidx.compose.material3.Button
22import androidx.compose.material3.Button
19import androidx.compose.material3.CircularProgressIndicator
23import androidx.compose.material3.CircularProgressIndicator
20import androidx.compose.material3.Icon
24import androidx.compose.material3.Icon
25+import androidx.compose.material3.IconButton
21import androidx.compose.material3.MaterialTheme
26import androidx.compose.material3.MaterialTheme
22import androidx.compose.material3.OutlinedTextField
27import androidx.compose.material3.OutlinedTextField
23import androidx.compose.material3.Text
28import androidx.compose.material3.Text
24import androidx.compose.runtime.Composable
29import 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
25import androidx.compose.ui.Alignment
35import androidx.compose.ui.Alignment
26import androidx.compose.ui.Modifier
36import androidx.compose.ui.Modifier
37+import androidx.compose.ui.focus.FocusRequester
38+import androidx.compose.ui.focus.focusRequester
27import androidx.compose.ui.text.input.ImeAction
39import androidx.compose.ui.text.input.ImeAction
28import androidx.compose.ui.text.input.KeyboardType
40import androidx.compose.ui.text.input.KeyboardType
29import androidx.compose.ui.text.input.PasswordVisualTransformation
41import androidx.compose.ui.text.input.PasswordVisualTransformation
42+import androidx.compose.ui.text.input.VisualTransformation
30import androidx.compose.ui.text.style.TextAlign
43import androidx.compose.ui.text.style.TextAlign
31import androidx.compose.ui.unit.dp
44import androidx.compose.ui.unit.dp
32import soy.iko.opencode.core.Event
45import soy.iko.opencode.core.Event
@@ -34,6 +47,15 @@ import soy.iko.opencode.core.UiState
34
47
35@Composable
48@Composable
36fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
49fun 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+
37 Column(
59 Column(
38 modifier = Modifier
60 modifier = Modifier
39 .fillMaxSize()
61 .fillMaxSize()
@@ -43,6 +65,13 @@ fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
43 verticalArrangement = Arrangement.Center,
65 verticalArrangement = Arrangement.Center,
44 horizontalAlignment = Alignment.CenterHorizontally,
66 horizontalAlignment = Alignment.CenterHorizontally,
45 ) {
67 ) {
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))
46 Text(
75 Text(
47 text = "opencode",
76 text = "opencode",
48 style = MaterialTheme.typography.displaySmall,
77 style = MaterialTheme.typography.displaySmall,
@@ -63,7 +92,7 @@ fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
63 label = { Text("Server URL") },
92 label = { Text("Server URL") },
64 placeholder = { Text("http://192.168.1.10:4096") },
93 placeholder = { Text("http://192.168.1.10:4096") },
65 singleLine = true,
94 singleLine = true,
66− modifier = Modifier.fillMaxWidth(),
95+ modifier = Modifier.fillMaxWidth().focusRequester(focusRequester),
67 keyboardOptions = KeyboardOptions(
96 keyboardOptions = KeyboardOptions(
68 keyboardType = KeyboardType.Uri,
97 keyboardType = KeyboardType.Uri,
69 imeAction = if (state.authRequired) ImeAction.Next else ImeAction.Go,
98 imeAction = if (state.authRequired) ImeAction.Next else ImeAction.Go,
@@ -72,6 +101,30 @@ fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
72 enabled = !state.loading,
101 enabled = !state.loading,
73 )
102 )
74
103
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+
75 if (state.authRequired) {
128 if (state.authRequired) {
76 Spacer(Modifier.height(12.dp))
129 Spacer(Modifier.height(12.dp))
77 OutlinedTextField(
130 OutlinedTextField(
@@ -90,11 +143,27 @@ fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
90 onValueChange = { dispatch(Event.PasswordChanged(it)) },
143 onValueChange = { dispatch(Event.PasswordChanged(it)) },
91 label = { Text("Password") },
144 label = { Text("Password") },
92 singleLine = true,
145 singleLine = true,
93− visualTransformation = PasswordVisualTransformation(),
146+ visualTransformation = if (passwordVisible) VisualTransformation.None else PasswordVisualTransformation(),
94 leadingIcon = { Icon(Icons.Filled.Lock, contentDescription = null) },
147 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+ },
95 modifier = Modifier.fillMaxWidth(),
156 modifier = Modifier.fillMaxWidth(),
96 keyboardOptions = KeyboardOptions(keyboardType = KeyboardType.Password, imeAction = ImeAction.Go),
157 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+ }),
98 enabled = !state.loading,
167 enabled = !state.loading,
99 )
168 )
100 Spacer(Modifier.height(4.dp))
169 Spacer(Modifier.height(4.dp))
@@ -109,7 +178,10 @@ fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
109
178
110 Button(
179 Button(
111 onClick = { dispatch(Event.Connect) },
180 onClick = { dispatch(Event.Connect) },
181+ // When auth is required, keep the button disabled until at least one
182+ // credential is entered — otherwise it just triggers a validation error.
112− enabled = !state.loading && state.serverUrl.isNotBlank(),
183+ enabled = !state.loading && state.serverUrl.isNotBlank() &&
184+ (!state.authRequired || state.username.isNotBlank() || state.password.isNotBlank()),
113 modifier = Modifier.fillMaxWidth(),
185 modifier = Modifier.fillMaxWidth(),
114 ) {
186 ) {
115 if (state.loading) {
187 if (state.loading) {
@@ -122,5 +194,29 @@ fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
122 Text(if (state.authRequired) "Sign in" else "Connect")
194 Text(if (state.authRequired) "Sign in" else "Connect")
123 }
195 }
124 }
196 }
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+ }
125 }
221 }
126}
222}
app/src/main/java/soy/iko/opencode/ui/screens/SessionsScreen.kt+217−18
@@ -1,7 +1,15 @@
1package soy.iko.opencode.ui.screens
1package soy.iko.opencode.ui.screens
2
2
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
3import androidx.compose.foundation.clickable
9import androidx.compose.foundation.clickable
10+import androidx.compose.foundation.combinedClickable
4import androidx.compose.foundation.layout.Arrangement
11import androidx.compose.foundation.layout.Arrangement
12+import androidx.compose.foundation.layout.Box
5import androidx.compose.foundation.layout.Column
13import androidx.compose.foundation.layout.Column
6import androidx.compose.foundation.layout.PaddingValues
14import androidx.compose.foundation.layout.PaddingValues
7import androidx.compose.foundation.layout.Row
15import androidx.compose.foundation.layout.Row
@@ -13,13 +21,17 @@ import androidx.compose.foundation.layout.padding
13import androidx.compose.foundation.layout.size
21import androidx.compose.foundation.layout.size
14import androidx.compose.foundation.lazy.LazyColumn
22import androidx.compose.foundation.lazy.LazyColumn
15import androidx.compose.foundation.lazy.items
23import androidx.compose.foundation.lazy.items
24+import androidx.compose.foundation.lazy.rememberLazyListState
25+import androidx.compose.foundation.shape.CircleShape
16import androidx.compose.material.icons.Icons
26import androidx.compose.material.icons.Icons
17import androidx.compose.material.icons.automirrored.filled.ArrowBack
27import androidx.compose.material.icons.automirrored.filled.ArrowBack
18import androidx.compose.material.icons.filled.Add
28import androidx.compose.material.icons.filled.Add
19import androidx.compose.material.icons.filled.ChatBubbleOutline
29import androidx.compose.material.icons.filled.ChatBubbleOutline
30+import androidx.compose.material.icons.filled.ChevronRight
20import androidx.compose.material.icons.filled.DeleteOutline
31import androidx.compose.material.icons.filled.DeleteOutline
21import androidx.compose.material.icons.filled.Refresh
32import androidx.compose.material.icons.filled.Refresh
22import androidx.compose.material3.AlertDialog
33import androidx.compose.material3.AlertDialog
34+import androidx.compose.material3.CircularProgressIndicator
23import androidx.compose.material3.ExperimentalMaterial3Api
35import androidx.compose.material3.ExperimentalMaterial3Api
24import androidx.compose.material3.ExtendedFloatingActionButton
36import androidx.compose.material3.ExtendedFloatingActionButton
25import androidx.compose.material3.HorizontalDivider
37import androidx.compose.material3.HorizontalDivider
@@ -27,21 +39,31 @@ import androidx.compose.material3.Icon
27import androidx.compose.material3.IconButton
39import androidx.compose.material3.IconButton
28import androidx.compose.material3.LinearProgressIndicator
40import androidx.compose.material3.LinearProgressIndicator
29import androidx.compose.material3.MaterialTheme
41import androidx.compose.material3.MaterialTheme
42+import androidx.compose.material3.OutlinedTextField
30import androidx.compose.material3.Scaffold
43import androidx.compose.material3.Scaffold
44+import androidx.compose.material3.Surface
45+import androidx.compose.material3.SwipeToDismissBox
46+import androidx.compose.material3.SwipeToDismissBoxValue
31import androidx.compose.material3.Text
47import androidx.compose.material3.Text
32import androidx.compose.material3.TextButton
48import androidx.compose.material3.TextButton
33import androidx.compose.material3.TopAppBar
49import androidx.compose.material3.TopAppBar
50+import androidx.compose.material3.rememberSwipeToDismissBoxState
34import androidx.compose.material3.pulltorefresh.PullToRefreshBox
51import androidx.compose.material3.pulltorefresh.PullToRefreshBox
35import androidx.compose.material3.pulltorefresh.rememberPullToRefreshState
52import androidx.compose.material3.pulltorefresh.rememberPullToRefreshState
36import androidx.compose.runtime.Composable
53import androidx.compose.runtime.Composable
54+import androidx.compose.runtime.LaunchedEffect
37import androidx.compose.runtime.getValue
55import androidx.compose.runtime.getValue
38import androidx.compose.runtime.mutableStateOf
56import androidx.compose.runtime.mutableStateOf
39import androidx.compose.runtime.remember
57import androidx.compose.runtime.remember
40import androidx.compose.runtime.setValue
58import androidx.compose.runtime.setValue
41import androidx.compose.ui.Alignment
59import androidx.compose.ui.Alignment
42import androidx.compose.ui.Modifier
60import androidx.compose.ui.Modifier
61+import androidx.compose.ui.draw.clip
62+import androidx.compose.ui.graphics.Color
43import androidx.compose.ui.hapticfeedback.HapticFeedbackType
63import androidx.compose.ui.hapticfeedback.HapticFeedbackType
64+import androidx.compose.ui.platform.LocalContext
44import androidx.compose.ui.platform.LocalHapticFeedback
65import androidx.compose.ui.platform.LocalHapticFeedback
66+import androidx.compose.ui.text.style.TextAlign
45import androidx.compose.ui.text.style.TextOverflow
67import androidx.compose.ui.text.style.TextOverflow
46import androidx.compose.ui.unit.dp
68import androidx.compose.ui.unit.dp
47import soy.iko.opencode.core.Event
69import soy.iko.opencode.core.Event
@@ -53,6 +75,23 @@ import soy.iko.opencode.core.UiState
53fun SessionsScreen(state: UiState, dispatch: (Event) -> Unit) {
75fun SessionsScreen(state: UiState, dispatch: (Event) -> Unit) {
54 val haptic = LocalHapticFeedback.current
76 val haptic = LocalHapticFeedback.current
55 var pendingDeleteId by remember { mutableStateOf<String?>(null) }
77 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+ }
56
95
57 pendingDeleteId?.let { deleteId ->
96 pendingDeleteId?.let { deleteId ->
58 val pendingSession = state.sessions.firstOrNull { it.id == deleteId }
97 val pendingSession = state.sessions.firstOrNull { it.id == deleteId }
@@ -80,7 +119,23 @@ fun SessionsScreen(state: UiState, dispatch: (Event) -> Unit) {
80 Scaffold(
119 Scaffold(
81 topBar = {
120 topBar = {
82 TopAppBar(
121 TopAppBar(
83− 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+ },
84 navigationIcon = {
139 navigationIcon = {
85 IconButton(onClick = { dispatch(Event.NavigateToConnect) }) {
140 IconButton(onClick = { dispatch(Event.NavigateToConnect) }) {
86 Icon(Icons.AutoMirrored.Filled.ArrowBack, contentDescription = "Disconnect")
141 Icon(Icons.AutoMirrored.Filled.ArrowBack, contentDescription = "Disconnect")
@@ -94,36 +149,87 @@ fun SessionsScreen(state: UiState, dispatch: (Event) -> Unit) {
94 )
149 )
95 },
150 },
96 floatingActionButton = {
151 floatingActionButton = {
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()) {
97− ExtendedFloatingActionButton(
155+ ExtendedFloatingActionButton(
98− onClick = { dispatch(Event.CreateSession) },
156+ onClick = { dispatch(Event.CreateSession) },
99− icon = { Icon(Icons.Filled.Add, contentDescription = null) },
157+ icon = { Icon(Icons.Filled.Add, contentDescription = null) },
100− text = { Text("New session") },
158+ text = { Text("New session") },
101− )
159+ )
160+ }
102 },
161 },
103 ) { padding ->
162 ) { padding ->
104 Column(Modifier.fillMaxSize().padding(padding)) {
163 Column(Modifier.fillMaxSize().padding(padding)) {
164+ if (!state.sseConnected) {
165+ ReconnectingBanner()
166+ }
105− if (state.loading) {
167+ if (state.loading && !isPullRefreshing) {
106 LinearProgressIndicator(Modifier.fillMaxWidth())
168 LinearProgressIndicator(Modifier.fillMaxWidth())
107 }
169 }
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+ }
108 when {
189 when {
109 state.sessions.isEmpty() && !state.loading -> EmptySessions()
190 state.sessions.isEmpty() && !state.loading -> EmptySessions()
191+ filteredSessions.isEmpty() && searchQuery.isNotEmpty() -> NoSearchResults(searchQuery)
110 else -> PullToRefreshBox(
192 else -> PullToRefreshBox(
111− isRefreshing = state.loading,
193+ isRefreshing = isPullRefreshing,
194+ onRefresh = {
195+ isPullRefreshing = true
112− onRefresh = { dispatch(Event.LoadSessions) },
196+ dispatch(Event.LoadSessions)
197+ },
113 modifier = Modifier.fillMaxSize(),
198 modifier = Modifier.fillMaxSize(),
114 state = rememberPullToRefreshState(),
199 state = rememberPullToRefreshState(),
115 ) {
200 ) {
116 LazyColumn(
201 LazyColumn(
202+ state = sessionsListState,
117 modifier = Modifier.fillMaxSize(),
203 modifier = Modifier.fillMaxSize(),
118 contentPadding = PaddingValues(bottom = 88.dp),
204 contentPadding = PaddingValues(bottom = 88.dp),
119 ) {
205 ) {
120− items(state.sessions, key = { it.id }) { session ->
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+ ) {
121− SessionRow(
225+ SessionRow(
122− session = session,
226+ session = session,
123− onOpen = { dispatch(Event.SelectSession(session.id)) },
227+ onOpen = { dispatch(Event.SelectSession(session.id)) },
124− onDelete = { pendingDeleteId = session.id },
228+ onDelete = { pendingDeleteId = session.id },
125− )
229+ )
230+ }
126− HorizontalDivider(color = MaterialTheme.colorScheme.outline.copy(alpha = 0.25f))
231+ HorizontalDivider(color = MaterialTheme.colorScheme.outline.copy(alpha = 0.25f))
232+ }
127 }
233 }
128 }
234 }
129 }
235 }
@@ -132,12 +238,21 @@ fun SessionsScreen(state: UiState, dispatch: (Event) -> Unit) {
132 }
238 }
133}
239}
134
240
241+@OptIn(ExperimentalFoundationApi::class)
135@Composable
242@Composable
136private fun SessionRow(session: SessionView, onOpen: () -> Unit, onDelete: () -> Unit) {
243private fun SessionRow(session: SessionView, onOpen: () -> Unit, onDelete: () -> Unit) {
244+ val context = LocalContext.current
137 Row(
245 Row(
138 modifier = Modifier
246 modifier = Modifier
139 .fillMaxWidth()
247 .fillMaxWidth()
248+ .background(MaterialTheme.colorScheme.surface)
249+ .combinedClickable(
140− .clickable(onClick = onOpen)
250+ onClick = onOpen,
251+ onLongClick = {
252+ copyToClipboard(context, session.id)
253+ Toast.makeText(context, "Session ID copied", Toast.LENGTH_SHORT).show()
254+ },
255+ )
141 .padding(start = 20.dp, end = 8.dp, top = 14.dp, bottom = 14.dp),
256 .padding(start = 20.dp, end = 8.dp, top = 14.dp, bottom = 14.dp),
142 verticalAlignment = Alignment.CenterVertically,
257 verticalAlignment = Alignment.CenterVertically,
143 ) {
258 ) {
@@ -162,20 +277,97 @@ private fun SessionRow(session: SessionView, onOpen: () -> Unit, onDelete: () ->
162 overflow = TextOverflow.Ellipsis,
277 overflow = TextOverflow.Ellipsis,
163 )
278 )
164 }
279 }
280+ Icon(
281+ Icons.Filled.ChevronRight,
282+ contentDescription = null,
283+ tint = MaterialTheme.colorScheme.onSurfaceVariant,
284+ modifier = Modifier.padding(horizontal = 4.dp),
285+ )
165 IconButton(onClick = onDelete) {
286 IconButton(onClick = onDelete) {
166 Icon(
287 Icon(
167 Icons.Filled.DeleteOutline,
288 Icons.Filled.DeleteOutline,
168 contentDescription = "Delete session",
289 contentDescription = "Delete session",
169− 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),
317+ )
318+ }
319+ }
320+}
321+
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,
170 )
343 )
171 }
344 }
172 }
345 }
173}
346}
174
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+
175/** Shorten a session ID for display: "abc12345…" instead of the full UUID. */
353/** Shorten a session ID for display: "abc12345…" instead of the full UUID. */
176private fun shortSessionId(id: String): String =
354private fun shortSessionId(id: String): String =
177 if (id.length > 12) id.take(8) + "…" else id
355 if (id.length > 12) id.take(8) + "…" else id
178
356
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+
179@Composable
371@Composable
180private fun EmptySessions() {
372private fun EmptySessions() {
181 Column(
373 Column(
@@ -199,5 +391,12 @@ private fun EmptySessions() {
199 style = MaterialTheme.typography.bodyMedium,
391 style = MaterialTheme.typography.bodyMedium,
200 color = MaterialTheme.colorScheme.onSurfaceVariant,
392 color = MaterialTheme.colorScheme.onSurfaceVariant,
201 )
393 )
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+ )
202 }
401 }
203}
402}
app/src/main/java/soy/iko/opencode/ui/theme/Theme.kt+23−1
@@ -14,7 +14,18 @@ import androidx.compose.ui.platform.LocalContext
14private val DarkColors = darkColorScheme(
14private val DarkColors = darkColorScheme(
15 primary = Emerald,
15 primary = Emerald,
16 onPrimary = Color(0xFF0F1115),
16 onPrimary = Color(0xFF0F1115),
17+ primaryContainer = Color(0xFF1A3D2A),
18+ onPrimaryContainer = Color(0xFFA8E6C1),
17 secondary = EmeraldDark,
19 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),
18 background = DarkBackground,
29 background = DarkBackground,
19 surface = DarkSurface,
30 surface = DarkSurface,
20 surfaceVariant = DarkSurfaceVariant,
31 surfaceVariant = DarkSurfaceVariant,
@@ -24,7 +35,18 @@ private val DarkColors = darkColorScheme(
24
35
25private val LightColors = lightColorScheme(
36private val LightColors = lightColorScheme(
26 primary = EmeraldDark,
37 primary = EmeraldDark,
38+ primaryContainer = Color(0xFFD7F5E3),
39+ onPrimaryContainer = Color(0xFF0A3D22),
27 secondary = Emerald,
40 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),
28 background = LightBackground,
50 background = LightBackground,
29 surface = LightSurface,
51 surface = LightSurface,
30 surfaceVariant = LightSurfaceVariant,
52 surfaceVariant = LightSurfaceVariant,
@@ -35,7 +57,7 @@ private val LightColors = lightColorScheme(
35@Composable
57@Composable
36fun OpencodeTheme(
58fun OpencodeTheme(
37 darkTheme: Boolean = isSystemInDarkTheme(),
59 darkTheme: Boolean = isSystemInDarkTheme(),
38− dynamicColor: Boolean = true,
60+ dynamicColor: Boolean = false,
39 content: @Composable () -> Unit,
61 content: @Composable () -> Unit,
40) {
62) {
41 val context = LocalContext.current
63 val context = LocalContext.current
shared/src/app.rs+14
@@ -297,16 +297,27 @@ impl App for OpencodeApp {
297 // Store the field verbatim while typing; normalizing here would fight
297 // Store the field verbatim while typing; normalizing here would fight
298 // the user's edits (e.g. eat the "://" as they type it). The URL is
298 // the user's edits (e.g. eat the "://" as they type it). The URL is
299 // normalized once, on Connect.
299 // 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.
300 Event::ServerUrlChanged(url) => {
302 Event::ServerUrlChanged(url) => {
301 model.server_url = url;
303 model.server_url = url;
304+ if model.error.is_some() {
305+ model.error = None;
306+ }
302 render()
307 render()
303 }
308 }
304 Event::UsernameChanged(u) => {
309 Event::UsernameChanged(u) => {
305 model.username = u;
310 model.username = u;
311+ if model.error.is_some() {
312+ model.error = None;
313+ }
306 render()
314 render()
307 }
315 }
308 Event::PasswordChanged(p) => {
316 Event::PasswordChanged(p) => {
309 model.password = p;
317 model.password = p;
318+ if model.error.is_some() {
319+ model.error = None;
320+ }
310 render()
321 render()
311 }
322 }
312
323
@@ -341,6 +352,9 @@ impl App for OpencodeApp {
341 title: display_title(&s.title),
352 title: display_title(&s.title),
342 })
353 })
343 .collect();
354 .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());
344 }
358 }
345 Err(e) => model.error = Some(e),
359 Err(e) => model.error = Some(e),
346 }
360 }