git.iko.soy/opencode-android
on master help sign in
app/src/main/java/soy/iko/opencode/CoreViewModel.kt+5
since splitsnapshots
unifiedsplit
@@ -23,6 +23,9 @@ 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`).
@@ -44,6 +47,7 @@ class CoreViewModel(app: Application) : AndroidViewModel(app) {
44 prefs.edit {
47 prefs.edit {
45 putString(KEY_URL, state.serverUrl)
48 putString(KEY_URL, state.serverUrl)
46 putString(KEY_USER, state.username)
49 putString(KEY_USER, state.username)
50+ putString(KEY_PASSWORD, state.password)
47 }
51 }
48 }
52 }
49 }
53 }
@@ -57,5 +61,6 @@ class CoreViewModel(app: Application) : AndroidViewModel(app) {
57 private companion object {
61 private companion object {
58 const val KEY_URL = "serverUrl"
62 const val KEY_URL = "serverUrl"
59 const val KEY_USER = "username"
63 const val KEY_USER = "username"
64+ const val KEY_PASSWORD = "password"
60 }
65 }
61}
66}
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+69−6
@@ -60,6 +60,18 @@ class OpencodeCore(
60 // Tracks the in-flight loadMessages coroutine. Rapidly switching sessions launches
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).
61 // several; cancel the prior one so only the newest fetch runs (saves needless requests).
62 private var loadMessagesJob: Job? = null
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
63 // Message IDs removed via SSE; a late MessageUpdated for one must not resurrect it.
75 // Message IDs removed via SSE; a late MessageUpdated for one must not resurrect it.
64 private val removedMessageIds = mutableSetOf<String>()
76 private val removedMessageIds = mutableSetOf<String>()
65
77
@@ -84,10 +96,12 @@ class OpencodeCore(
84 // Leave authRequired as-is; probeHealth sets it from the response (true on
96 // Leave authRequired as-is; probeHealth sets it from the response (true on
85 // 401, false on success). Resetting it here just flickers the auth fields.
97 // 401, false on success). Resetting it here just flickers the auth fields.
86 loading = true; error = null; emit()
98 loading = true; error = null; emit()
99+ connectJob?.cancel()
87− scope.launch { probeHealth() }
100+ connectJob = scope.launch { probeHealth() }
88 }
101 }
89
102
90 Event.LoadSessions -> {
103 Event.LoadSessions -> {
104+ if (!connected) { error = "Not connected"; emit(); return }
91 loading = true; emit()
105 loading = true; emit()
92 loadSessionsJob?.cancel()
106 loadSessionsJob?.cancel()
93 loadSessionsJob = scope.launch { loadSessions() }
107 loadSessionsJob = scope.launch { loadSessions() }
@@ -107,22 +121,29 @@ class OpencodeCore(
107 generating = false
121 generating = false
108 error = null
122 error = null
109 emit()
123 emit()
124+ connectJob?.cancel()
125+ abortJob?.cancel()
126+ sendMessageJob?.cancel()
127+ deleteSessionJob?.cancel()
110 loadMessagesJob?.cancel()
128 loadMessagesJob?.cancel()
111 loadMessagesJob = scope.launch { loadMessages(event.id) }
129 loadMessagesJob = scope.launch { loadMessages(event.id) }
112 }
130 }
113
131
114 Event.CreateSession -> {
132 Event.CreateSession -> {
133+ if (!connected) { error = "Not connected"; emit(); return }
115 loading = true; error = null; emit()
134 loading = true; error = null; emit()
116 createSessionJob?.cancel()
135 createSessionJob?.cancel()
117 createSessionJob = scope.launch { createSession() }
136 createSessionJob = scope.launch { createSession() }
118 }
137 }
119
138
120 is Event.DeleteSession -> {
139 is Event.DeleteSession -> {
140+ if (!connected) { error = "Not connected"; emit(); return }
121 val id = event.id
141 val id = event.id
122 // 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
123 // succeeds, so a failed request leaves the session recoverable.
143 // succeeds, so a failed request leaves the session recoverable.
124 loading = true; error = null; emit()
144 loading = true; error = null; emit()
145+ deleteSessionJob?.cancel()
125− scope.launch {
146+ deleteSessionJob = scope.launch {
126 http.delete("$serverUrl/session/$id", authHeader()).fold(
147 http.delete("$serverUrl/session/$id", authHeader()).fold(
127 onSuccess = { resp ->
148 onSuccess = { resp ->
128 if (resp.code in 200..299) {
149 if (resp.code in 200..299) {
@@ -155,7 +176,10 @@ class OpencodeCore(
155 messages.forEach { it.streaming = false }
176 messages.forEach { it.streaming = false }
156 emit()
177 emit()
157 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()
158− scope.launch { http.postJson("$serverUrl/session/$id/abort", "{}", authHeader()) }
182+ abortJob = scope.launch { http.postJson("$serverUrl/session/$id/abort", "{}", authHeader()) }
159 }
183 }
160
184
161 Event.NavigateToSessions -> {
185 Event.NavigateToSessions -> {
@@ -165,9 +189,16 @@ class OpencodeCore(
165 removedMessageIds.clear()
189 removedMessageIds.clear()
166 draft = ""
190 draft = ""
167 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
168 loading = true
195 loading = true
169 emit()
196 emit()
197+ connectJob?.cancel()
198+ abortJob?.cancel()
170 createSessionJob?.cancel()
199 createSessionJob?.cancel()
200+ deleteSessionJob?.cancel()
201+ sendMessageJob?.cancel()
171 loadMessagesJob?.cancel()
202 loadMessagesJob?.cancel()
172 loadSessionsJob?.cancel()
203 loadSessionsJob?.cancel()
173 loadSessionsJob = scope.launch { loadSessions() }
204 loadSessionsJob = scope.launch { loadSessions() }
@@ -186,7 +217,11 @@ class OpencodeCore(
186 // Reset to the default so a late-firing close callback from the old
217 // Reset to the default so a late-firing close callback from the old
187 // SSE connection can't flash the reconnecting banner on the next connect.
218 // SSE connection can't flash the reconnecting banner on the next connect.
188 sseConnected = true
219 sseConnected = true
220+ connectJob?.cancel()
221+ abortJob?.cancel()
189 createSessionJob?.cancel()
222 createSessionJob?.cancel()
223+ deleteSessionJob?.cancel()
224+ sendMessageJob?.cancel()
190 loadMessagesJob?.cancel()
225 loadMessagesJob?.cancel()
191 closeSse()
226 closeSse()
192 emit()
227 emit()
@@ -201,13 +236,27 @@ class OpencodeCore(
201 if (connected && sse == null) startSse()
236 if (connected && sse == null) startSse()
202 }
237 }
203
238
204− 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+ }
205
251
206 // ── Effects ────────────────────────────────────────────────────────────────
252 // ── Effects ────────────────────────────────────────────────────────────────
207 private fun sendMessage(raw: String) {
253 private fun sendMessage(raw: String) {
208 val text = raw.trim()
254 val text = raw.trim()
209 val session = currentSessionId ?: return
255 val session = currentSessionId ?: return
210 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
211 localSeq += 1
260 localSeq += 1
212 val user = MsgState("local-$localSeq", "user", System.currentTimeMillis()).apply {
261 val user = MsgState("local-$localSeq", "user", System.currentTimeMillis()).apply {
213 status = MessageStatus.Pending
262 status = MessageStatus.Pending
@@ -223,7 +272,8 @@ class OpencodeCore(
223 emit()
272 emit()
224
273
225 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()
226− scope.launch {
276+ sendMessageJob = scope.launch {
227 val result = http.postJson("$serverUrl/session/$session/prompt_async", body, authHeader())
277 val result = http.postJson("$serverUrl/session/$session/prompt_async", body, authHeader())
228 result.fold(
278 result.fold(
229 onSuccess = { resp ->
279 onSuccess = { resp ->
@@ -256,6 +306,10 @@ class OpencodeCore(
256 when {
306 when {
257 resp.code == 401 -> {
307 resp.code == 401 -> {
258 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
259 // Always reveal the auth fields so the user can see/edit
313 // Always reveal the auth fields so the user can see/edit
260 // their credentials — even when the ones supplied were wrong.
314 // their credentials — even when the ones supplied were wrong.
261 authRequired = true
315 authRequired = true
@@ -277,6 +331,7 @@ class OpencodeCore(
277 }
331 }
278 else -> {
332 else -> {
279 loading = false
333 loading = false
334+ connected = false
280 error = "Server returned status ${resp.code}"
335 error = "Server returned status ${resp.code}"
281 emit()
336 emit()
282 }
337 }
@@ -284,6 +339,7 @@ class OpencodeCore(
284 },
339 },
285 onFailure = { e ->
340 onFailure = { e ->
286 loading = false
341 loading = false
342+ connected = false
287 error = "Connection failed: ${e.message}"
343 error = "Connection failed: ${e.message}"
288 emit()
344 emit()
289 },
345 },
@@ -316,6 +372,10 @@ class OpencodeCore(
316 if (session != null) {
372 if (session != null) {
317 if (sessions.none { it.id == session.id }) {
373 if (sessions.none { it.id == session.id }) {
318 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() }
319 }
379 }
320 emit()
380 emit()
321 // SelectSession takes over the loading state (loads messages).
381 // SelectSession takes over the loading state (loads messages).
@@ -422,7 +482,7 @@ class OpencodeCore(
422 } else {
482 } else {
423 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 })
424 }
484 }
425− info.error?.let { error = it.message(); generating = false }
485+ info.error?.let { error = it.message(); messages.forEach { m -> m.streaming = false }; generating = false }
426 return true
486 return true
427 }
487 }
428
488
@@ -485,6 +545,9 @@ class OpencodeCore(
485 } else {
545 } else {
486 sessions.add(0, SessionView(event.info.id, title))
546 sessions.add(0, SessionView(event.info.id, title))
487 }
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() }
488 if (isCurrent(event.info.id)) currentSessionTitle = title
551 if (isCurrent(event.info.id)) currentSessionTitle = title
489 return true
552 return true
490 }
553 }
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+55−2
@@ -7,13 +7,19 @@ import androidx.compose.animation.slideOutHorizontally
7import androidx.compose.animation.togetherWith
7import androidx.compose.animation.togetherWith
8import androidx.compose.foundation.layout.fillMaxSize
8import androidx.compose.foundation.layout.fillMaxSize
9import androidx.compose.foundation.layout.padding
9import androidx.compose.foundation.layout.padding
10+import androidx.compose.material3.AlertDialog
10import androidx.compose.material3.Scaffold
11import androidx.compose.material3.Scaffold
11import androidx.compose.material3.SnackbarDuration
12import 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,6 +31,8 @@ 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 {
@@ -40,12 +48,57 @@ fun OpencodeApp(state: UiState, dispatch: (Event) -> Unit) {
40
48
41 BackHandler(enabled = state.screen != Screen.Connect) {
49 BackHandler(enabled = state.screen != Screen.Connect) {
42 when (state.screen) {
50 when (state.screen) {
51+ // Leaving mid-generation would silently drop the reply, so confirm first.
43− Screen.Chat -> dispatch(Event.NavigateToSessions)
52+ Screen.Chat -> if (state.generating) showStopConfirm = true else dispatch(Event.NavigateToSessions)
44− Screen.Sessions -> dispatch(Event.NavigateToConnect)
53+ // Disconnecting drops the server session, so guard the back button.
54+ Screen.Sessions -> showDisconnectConfirm = true
45 Screen.Connect -> Unit
55 Screen.Connect -> Unit
46 }
56 }
47 }
57 }
48
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+
49 Scaffold(snackbarHost = { SnackbarHost(snackbar) }) { padding ->
102 Scaffold(snackbarHost = { SnackbarHost(snackbar) }) { padding ->
50 AnimatedContent(
103 AnimatedContent(
51 targetState = state.screen,
104 targetState = state.screen,
app/src/main/java/soy/iko/opencode/ui/components/MessageBubble.kt+225−5
@@ -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,6 +15,7 @@ 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
18import androidx.compose.foundation.text.selection.SelectionContainer
21import androidx.compose.foundation.text.selection.SelectionContainer
@@ -23,6 +26,9 @@ import androidx.compose.material.icons.filled.ErrorOutline
23import androidx.compose.material.icons.filled.ExpandLess
26import androidx.compose.material.icons.filled.ExpandLess
24import androidx.compose.material.icons.filled.ExpandMore
27import androidx.compose.material.icons.filled.ExpandMore
25import 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
26import androidx.compose.material3.Icon
32import androidx.compose.material3.Icon
27import androidx.compose.material3.MaterialTheme
33import androidx.compose.material3.MaterialTheme
28import androidx.compose.material3.Text
34import androidx.compose.material3.Text
@@ -36,10 +42,17 @@ import androidx.compose.ui.Modifier
36import androidx.compose.ui.draw.clip
42import androidx.compose.ui.draw.clip
37import androidx.compose.ui.graphics.Color
43import androidx.compose.ui.graphics.Color
38import 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
39import androidx.compose.ui.text.SpanStyle
47import androidx.compose.ui.text.SpanStyle
48+import androidx.compose.ui.text.TextLinkStyles
40import androidx.compose.ui.text.buildAnnotatedString
49import androidx.compose.ui.text.buildAnnotatedString
50+import androidx.compose.ui.text.font.FontFamily
41import 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
42import androidx.compose.ui.text.withStyle
54import androidx.compose.ui.text.withStyle
55+import androidx.compose.ui.text.style.TextDecoration
43import androidx.compose.ui.text.style.TextOverflow
56import androidx.compose.ui.text.style.TextOverflow
44import androidx.compose.ui.unit.dp
57import androidx.compose.ui.unit.dp
45import androidx.compose.ui.unit.sp
58import androidx.compose.ui.unit.sp
@@ -52,7 +65,7 @@ private val timeFormat = java.time.format.DateTimeFormatter.ofPattern("HH:mm")
52
65
53@OptIn(ExperimentalFoundationApi::class)
66@OptIn(ExperimentalFoundationApi::class)
54@Composable
67@Composable
55−fun MessageBubble(message: MessageView, modifier: Modifier = Modifier, onRetry: () -> Unit = {}) {
68+fun MessageBubble(message: MessageView, modifier: Modifier = Modifier, onRetry: () -> Unit = {}, compactSpacing: Boolean = false) {
56 val isUser = message.isUser
69 val isUser = message.isUser
57 val bubbleColor = if (isUser) MaterialTheme.colorScheme.primaryContainer else MaterialTheme.colorScheme.surfaceVariant
70 val bubbleColor = if (isUser) MaterialTheme.colorScheme.primaryContainer else MaterialTheme.colorScheme.surfaceVariant
58 val textColor = if (isUser) MaterialTheme.colorScheme.onPrimaryContainer else MaterialTheme.colorScheme.onSurface
71 val textColor = if (isUser) MaterialTheme.colorScheme.onPrimaryContainer else MaterialTheme.colorScheme.onSurface
@@ -65,11 +78,20 @@ fun MessageBubble(message: MessageView, modifier: Modifier = Modifier, onRetry:
65 )
78 )
66
79
67 val context = LocalContext.current
80 val context = LocalContext.current
81+ var showMenu by remember { mutableStateOf(false) }
68
82
69 Row(
83 Row(
70− 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),
71 horizontalArrangement = if (isUser) Arrangement.End else Arrangement.Start,
85 horizontalArrangement = if (isUser) Arrangement.End else Arrangement.Start,
72 ) {
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+ }
73 Column(
95 Column(
74 modifier = Modifier
96 modifier = Modifier
75 .fillMaxWidth(0.85f)
97 .fillMaxWidth(0.85f)
@@ -80,21 +102,55 @@ fun MessageBubble(message: MessageView, modifier: Modifier = Modifier, onRetry:
80 onClick = {},
102 onClick = {},
81 onLongClick = {
103 onLongClick = {
82 if (message.text.isNotEmpty()) {
104 if (message.text.isNotEmpty()) {
83− copyToClipboard(context, message.text)
105+ showMenu = true
84− Toast.makeText(context, "Copied", Toast.LENGTH_SHORT).show()
85 }
106 }
86 },
107 },
87 )
108 )
88 .padding(horizontal = 14.dp, vertical = 10.dp),
109 .padding(horizontal = 14.dp, vertical = 10.dp),
89 ) {
110 ) {
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+ }
128+
90 message.reasoning?.let { ReasoningBlock(it, context) }
129 message.reasoning?.let { ReasoningBlock(it, context) }
91
130
131+ // Give a quick overview of how many tools ran before listing them.
132+ if (message.tools.size > 1) {
133+ Text(
134+ text = "${message.tools.size} tool calls",
135+ style = MaterialTheme.typography.labelSmall,
136+ color = MaterialTheme.colorScheme.onSurfaceVariant,
137+ modifier = Modifier.padding(bottom = 4.dp),
138+ )
139+ }
140+
92 message.tools.forEach { ToolRow(it) }
141 message.tools.forEach { ToolRow(it) }
93
142
94 SelectionContainer {
143 SelectionContainer {
95 if (message.text.isNotEmpty()) {
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+ }
96 Text(
152 Text(
97− text = message.text + if (message.streaming) " ▌" else "",
153+ text = annotatedText,
98 color = textColor,
154 color = textColor,
99 // Give assistant replies (often long, code-heavy) extra line
155 // Give assistant replies (often long, code-heavy) extra line
100 // spacing for readability; user bubbles keep the default.
156 // spacing for readability; user bubbles keep the default.
@@ -130,6 +186,170 @@ private fun copyToClipboard(context: Context, text: String) {
130 clipboard.setPrimaryClip(ClipData.newPlainText("message", text))
186 clipboard.setPrimaryClip(ClipData.newPlainText("message", text))
131}
187}
132
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+
133@Composable
353@Composable
134private fun ReasoningBlock(text: String, context: Context) {
354private fun ReasoningBlock(text: String, context: Context) {
135 var expanded by remember { mutableStateOf(false) }
355 var expanded by remember { mutableStateOf(false) }
app/src/main/java/soy/iko/opencode/ui/screens/ChatScreen.kt+145−11
@@ -1,5 +1,9 @@
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
4import androidx.compose.animation.core.RepeatMode
8import androidx.compose.animation.core.RepeatMode
5import androidx.compose.animation.core.animateFloat
9import androidx.compose.animation.core.animateFloat
@@ -10,8 +14,10 @@ import androidx.compose.animation.fadeIn
10import androidx.compose.animation.fadeOut
14import androidx.compose.animation.fadeOut
11import androidx.compose.animation.scaleIn
15import androidx.compose.animation.scaleIn
12import androidx.compose.animation.scaleOut
16import androidx.compose.animation.scaleOut
17+import androidx.compose.foundation.ExperimentalFoundationApi
13import androidx.compose.foundation.background
18import androidx.compose.foundation.background
14import androidx.compose.foundation.clickable
19import androidx.compose.foundation.clickable
20+import androidx.compose.foundation.combinedClickable
15import androidx.compose.foundation.layout.Arrangement
21import androidx.compose.foundation.layout.Arrangement
16import androidx.compose.foundation.layout.Box
22import androidx.compose.foundation.layout.Box
17import androidx.compose.foundation.layout.BoxScope
23import androidx.compose.foundation.layout.BoxScope
@@ -20,6 +26,7 @@ import androidx.compose.foundation.layout.Row
20import androidx.compose.foundation.layout.Spacer
26import androidx.compose.foundation.layout.Spacer
21import androidx.compose.foundation.layout.fillMaxSize
27import androidx.compose.foundation.layout.fillMaxSize
22import androidx.compose.foundation.layout.fillMaxWidth
28import androidx.compose.foundation.layout.fillMaxWidth
29+import androidx.compose.foundation.layout.heightIn
23import androidx.compose.foundation.layout.PaddingValues
30import androidx.compose.foundation.layout.PaddingValues
24import androidx.compose.foundation.layout.WindowInsets
31import androidx.compose.foundation.layout.WindowInsets
25import androidx.compose.foundation.layout.ime
32import androidx.compose.foundation.layout.ime
@@ -36,8 +43,10 @@ import androidx.compose.foundation.text.KeyboardOptions
36import androidx.compose.material.icons.Icons
43import androidx.compose.material.icons.Icons
37import androidx.compose.material.icons.automirrored.filled.ArrowBack
44import androidx.compose.material.icons.automirrored.filled.ArrowBack
38import androidx.compose.material.icons.automirrored.filled.Send
45import androidx.compose.material.icons.automirrored.filled.Send
46+import androidx.compose.material.icons.filled.ChatBubbleOutline
39import androidx.compose.material.icons.filled.Close
47import androidx.compose.material.icons.filled.Close
40import androidx.compose.material.icons.filled.KeyboardArrowDown
48import androidx.compose.material.icons.filled.KeyboardArrowDown
49+import androidx.compose.material.icons.filled.KeyboardArrowUp
41import androidx.compose.material.icons.filled.Stop
50import androidx.compose.material.icons.filled.Stop
42import androidx.compose.material3.CircularProgressIndicator
51import androidx.compose.material3.CircularProgressIndicator
43import androidx.compose.material3.ExperimentalMaterial3Api
52import androidx.compose.material3.ExperimentalMaterial3Api
@@ -52,6 +61,7 @@ import androidx.compose.material3.OutlinedTextField
52import androidx.compose.material3.Scaffold
61import androidx.compose.material3.Scaffold
53import androidx.compose.material3.Surface
62import androidx.compose.material3.Surface
54import androidx.compose.material3.Text
63import androidx.compose.material3.Text
64+import androidx.compose.material3.TextButton
55import androidx.compose.material3.TopAppBar
65import androidx.compose.material3.TopAppBar
56import androidx.compose.runtime.Composable
66import androidx.compose.runtime.Composable
57import androidx.compose.runtime.LaunchedEffect
67import androidx.compose.runtime.LaunchedEffect
@@ -65,6 +75,8 @@ import androidx.compose.ui.Alignment
65import androidx.compose.ui.Modifier
75import androidx.compose.ui.Modifier
66import androidx.compose.ui.draw.clip
76import androidx.compose.ui.draw.clip
67import 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
68import androidx.compose.ui.platform.LocalDensity
80import androidx.compose.ui.platform.LocalDensity
69import androidx.compose.ui.platform.LocalHapticFeedback
81import androidx.compose.ui.platform.LocalHapticFeedback
70import androidx.compose.ui.text.font.FontStyle
82import androidx.compose.ui.text.font.FontStyle
@@ -77,11 +89,12 @@ import soy.iko.opencode.core.Event
77import soy.iko.opencode.core.UiState
89import soy.iko.opencode.core.UiState
78import soy.iko.opencode.ui.components.MessageBubble
90import soy.iko.opencode.ui.components.MessageBubble
79
91
80−@OptIn(ExperimentalMaterial3Api::class)
92+@OptIn(ExperimentalMaterial3Api::class, ExperimentalFoundationApi::class)
81@Composable
93@Composable
82fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
94fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
83 val listState = rememberLazyListState()
95 val listState = rememberLazyListState()
84 val coroutineScope = rememberCoroutineScope()
96 val coroutineScope = rememberCoroutineScope()
97+ val context = LocalContext.current
85
98
86 // Hide the jump-to-bottom FAB while the keyboard is up: it would otherwise
99 // Hide the jump-to-bottom FAB while the keyboard is up: it would otherwise
87 // sit over the top of the IME. Reading the ime inset bottom during composition
100 // sit over the top of the IME. Reading the ime inset bottom during composition
@@ -100,6 +113,12 @@ fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
100 }
113 }
101 }
114 }
102
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+
103 // Track how many messages the user has "seen" (i.e. was at the bottom for). While
122 // Track how many messages the user has "seen" (i.e. was at the bottom for). While
104 // they're scrolled up, any messages beyond that count are unseen — flag the FAB.
123 // they're scrolled up, any messages beyond that count are unseen — flag the FAB.
105 var lastSeenCount by remember { mutableStateOf(0) }
124 var lastSeenCount by remember { mutableStateOf(0) }
@@ -129,11 +148,20 @@ fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
129 topBar = {
148 topBar = {
130 TopAppBar(
149 TopAppBar(
131 title = {
150 title = {
151+ val title = state.currentSessionTitle.ifEmpty { "Chat" }
132 Text(
152 Text(
133− text = state.currentSessionTitle.ifEmpty { "Chat" },
153+ text = if (state.messages.isNotEmpty()) "$title (${state.messages.size})" else title,
134 maxLines = 1,
154 maxLines = 1,
135 overflow = TextOverflow.Ellipsis,
155 overflow = TextOverflow.Ellipsis,
156+ modifier = Modifier.combinedClickable(
136− modifier = Modifier.clickable { dispatch(Event.NavigateToSessions) },
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+ ),
137 )
165 )
138 },
166 },
139 navigationIcon = {
167 navigationIcon = {
@@ -177,12 +205,24 @@ fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
177 }
205 }
178 Box(Modifier.fillMaxWidth().weight(1f)) {
206 Box(Modifier.fillMaxWidth().weight(1f)) {
179 if (state.messages.isEmpty() && !state.loading) {
207 if (state.messages.isEmpty() && !state.loading) {
180− Text(
208+ Column(
181− text = "Send a message to start the conversation.",
182− style = MaterialTheme.typography.bodyMedium,
183− color = MaterialTheme.colorScheme.onSurfaceVariant,
184 modifier = Modifier.align(Alignment.Center).padding(32.dp),
209 modifier = Modifier.align(Alignment.Center).padding(32.dp),
210+ horizontalAlignment = Alignment.CenterHorizontally,
185− )
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+ }
186 }
226 }
187 LazyColumn(
227 LazyColumn(
188 state = listState,
228 state = listState,
@@ -196,9 +236,11 @@ fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
196 if (shouldShowDateSeparator(previous, message)) {
236 if (shouldShowDateSeparator(previous, message)) {
197 DateSeparator(message.time)
237 DateSeparator(message.time)
198 }
238 }
239+ val sameSenderAsPrevious = index > 0 && state.messages[index - 1].isUser == message.isUser
199 MessageBubble(
240 MessageBubble(
200 message,
241 message,
201 onRetry = { dispatch(Event.SendMessage(message.text)) },
242 onRetry = { dispatch(Event.SendMessage(message.text)) },
243+ compactSpacing = sameSenderAsPrevious,
202 )
244 )
203 }
245 }
204 // While the assistant is spinning up but hasn't emitted a streaming
246 // While the assistant is spinning up but hasn't emitted a streaming
@@ -216,11 +258,25 @@ fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
216 onClick = {
258 onClick = {
217 coroutineScope.launch {
259 coroutineScope.launch {
218 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
219− listState.animateScrollToItem(state.messages.size - 1)
266+ state.messages.size - 1
267+ listState.animateScrollToItem(target)
220 }
268 }
221 }
269 }
222 },
270 },
223 )
271 )
272+ ScrollToTopButton(
273+ visible = showScrollTop,
274+ onClick = {
275+ coroutineScope.launch {
276+ listState.animateScrollToItem(0)
277+ }
278+ },
279+ )
224 }
280 }
225 }
281 }
226 }
282 }
@@ -260,6 +316,35 @@ private fun BoxScope.ScrollToBottomButton(visible: Boolean, showBadge: Boolean,
260 }
316 }
261}
317}
262
318
319+// Mirror of ScrollToBottomButton, pinned to the top-start corner. Same BoxScope
320+// extension trick so the top-level AnimatedVisibility overload resolves cleanly.
321+@Composable
322+private fun BoxScope.ScrollToTopButton(visible: Boolean, onClick: () -> Unit) {
323+ AnimatedVisibility(
324+ visible = visible,
325+ modifier = Modifier.align(Alignment.TopStart),
326+ enter = scaleIn() + fadeIn(),
327+ exit = scaleOut() + fadeOut(),
328+ ) {
329+ Box(Modifier.padding(16.dp)) {
330+ FloatingActionButton(
331+ onClick = onClick,
332+ modifier = Modifier.size(44.dp),
333+ ) {
334+ Icon(
335+ Icons.Filled.KeyboardArrowUp,
336+ contentDescription = "Scroll to top",
337+ )
338+ }
339+ }
340+ }
341+}
342+
343+private fun copyToClipboard(context: Context, text: String) {
344+ val clipboard = context.getSystemService(Context.CLIPBOARD_SERVICE) as ClipboardManager
345+ clipboard.setPrimaryClip(ClipData.newPlainText("session", text))
346+}
347+
263private val dateSeparatorFormat = java.time.format.DateTimeFormatter.ofPattern("MMM d")
348private val dateSeparatorFormat = java.time.format.DateTimeFormatter.ofPattern("MMM d")
264
349
265/** Convert an epoch-millis timestamp to the local calendar day it falls on. */
350/** Convert an epoch-millis timestamp to the local calendar day it falls on. */
@@ -412,19 +497,56 @@ private fun ChatInputBar(
412 onStop: () -> Unit,
497 onStop: () -> Unit,
413) {
498) {
414 val haptic = LocalHapticFeedback.current
499 val haptic = LocalHapticFeedback.current
500+ val clipboardManager = LocalClipboardManager.current
415 Surface(tonalElevation = 3.dp) {
501 Surface(tonalElevation = 3.dp) {
416− Row(
502+ Column(
417 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(),
418 verticalAlignment = Alignment.Bottom,
507 verticalAlignment = Alignment.Bottom,
419 horizontalArrangement = Arrangement.spacedBy(8.dp),
508 horizontalArrangement = Arrangement.spacedBy(8.dp),
420 ) {
509 ) {
421 OutlinedTextField(
510 OutlinedTextField(
422 value = draft,
511 value = draft,
423 onValueChange = onDraftChange,
512 onValueChange = onDraftChange,
424− modifier = Modifier.weight(1f),
513+ modifier = Modifier.weight(1f).heightIn(min = 48.dp),
425 enabled = !loading,
514 enabled = !loading,
426 placeholder = { Text("Message opencode…") },
515 placeholder = { Text("Message opencode…") },
427 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+ },
428 keyboardOptions = KeyboardOptions(imeAction = ImeAction.Send),
550 keyboardOptions = KeyboardOptions(imeAction = ImeAction.Send),
429 keyboardActions = KeyboardActions(onSend = {
551 keyboardActions = KeyboardActions(onSend = {
430 if (!generating && !loading && draft.isNotBlank()) onSend()
552 if (!generating && !loading && draft.isNotBlank()) onSend()
@@ -452,5 +574,17 @@ private fun ChatInputBar(
452 }
574 }
453 }
575 }
454 }
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+ }
455 }
589 }
456}
590}
app/src/main/java/soy/iko/opencode/ui/screens/ConnectScreen.kt+37−2
@@ -113,6 +113,18 @@ fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
113 )
113 )
114 }
114 }
115
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+
116 if (state.authRequired) {
128 if (state.authRequired) {
117 Spacer(Modifier.height(12.dp))
129 Spacer(Modifier.height(12.dp))
118 OutlinedTextField(
130 OutlinedTextField(
@@ -143,7 +155,15 @@ fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
143 },
155 },
144 modifier = Modifier.fillMaxWidth(),
156 modifier = Modifier.fillMaxWidth(),
145 keyboardOptions = KeyboardOptions(keyboardType = KeyboardType.Password, imeAction = ImeAction.Go),
157 keyboardOptions = KeyboardOptions(keyboardType = KeyboardType.Password, imeAction = ImeAction.Go),
146− 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+ }),
147 enabled = !state.loading,
167 enabled = !state.loading,
148 )
168 )
149 Spacer(Modifier.height(4.dp))
169 Spacer(Modifier.height(4.dp))
@@ -158,7 +178,10 @@ fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
158
178
159 Button(
179 Button(
160 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.
161− enabled = !state.loading && state.serverUrl.isNotBlank(),
183+ enabled = !state.loading && state.serverUrl.isNotBlank() &&
184+ (!state.authRequired || state.username.isNotBlank() || state.password.isNotBlank()),
162 modifier = Modifier.fillMaxWidth(),
185 modifier = Modifier.fillMaxWidth(),
163 ) {
186 ) {
164 if (state.loading) {
187 if (state.loading) {
@@ -172,6 +195,18 @@ fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
172 }
195 }
173 }
196 }
174
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+
175 // Surface the error inline as well, so it stays visible while the user
210 // Surface the error inline as well, so it stays visible while the user
176 // corrects the form (the snackbar is transient).
211 // corrects the form (the snackbar is transient).
177 if (state.error != null && !state.loading) {
212 if (state.error != null && !state.loading) {
app/src/main/java/soy/iko/opencode/ui/screens/SessionsScreen.kt+75−9
@@ -1,7 +1,13 @@
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
3import androidx.compose.foundation.background
8import androidx.compose.foundation.background
4import androidx.compose.foundation.clickable
9import androidx.compose.foundation.clickable
10+import androidx.compose.foundation.combinedClickable
5import androidx.compose.foundation.layout.Arrangement
11import androidx.compose.foundation.layout.Arrangement
6import androidx.compose.foundation.layout.Box
12import androidx.compose.foundation.layout.Box
7import androidx.compose.foundation.layout.Column
13import androidx.compose.foundation.layout.Column
@@ -15,6 +21,8 @@ import androidx.compose.foundation.layout.padding
15import androidx.compose.foundation.layout.size
21import androidx.compose.foundation.layout.size
16import androidx.compose.foundation.lazy.LazyColumn
22import androidx.compose.foundation.lazy.LazyColumn
17import 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
18import androidx.compose.material.icons.Icons
26import androidx.compose.material.icons.Icons
19import androidx.compose.material.icons.automirrored.filled.ArrowBack
27import androidx.compose.material.icons.automirrored.filled.ArrowBack
20import androidx.compose.material.icons.filled.Add
28import androidx.compose.material.icons.filled.Add
@@ -50,8 +58,12 @@ import androidx.compose.runtime.remember
50import androidx.compose.runtime.setValue
58import androidx.compose.runtime.setValue
51import androidx.compose.ui.Alignment
59import androidx.compose.ui.Alignment
52import androidx.compose.ui.Modifier
60import androidx.compose.ui.Modifier
61+import androidx.compose.ui.draw.clip
62+import androidx.compose.ui.graphics.Color
53import androidx.compose.ui.hapticfeedback.HapticFeedbackType
63import androidx.compose.ui.hapticfeedback.HapticFeedbackType
64+import androidx.compose.ui.platform.LocalContext
54import androidx.compose.ui.platform.LocalHapticFeedback
65import androidx.compose.ui.platform.LocalHapticFeedback
66+import androidx.compose.ui.text.style.TextAlign
55import androidx.compose.ui.text.style.TextOverflow
67import androidx.compose.ui.text.style.TextOverflow
56import androidx.compose.ui.unit.dp
68import androidx.compose.ui.unit.dp
57import soy.iko.opencode.core.Event
69import soy.iko.opencode.core.Event
@@ -65,6 +77,15 @@ fun SessionsScreen(state: UiState, dispatch: (Event) -> Unit) {
65 var pendingDeleteId by remember { mutableStateOf<String?>(null) }
77 var pendingDeleteId by remember { mutableStateOf<String?>(null) }
66 var isPullRefreshing by remember { mutableStateOf(false) }
78 var isPullRefreshing by remember { mutableStateOf(false) }
67 var searchQuery by remember { mutableStateOf("") }
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+ }
68
89
69 // The pull-to-refresh spinner should only reflect a user-initiated pull, not
90 // The pull-to-refresh spinner should only reflect a user-initiated pull, not
70 // every loading operation. Clear it once the underlying load completes.
91 // every loading operation. Clear it once the underlying load completes.
@@ -98,7 +119,23 @@ fun SessionsScreen(state: UiState, dispatch: (Event) -> Unit) {
98 Scaffold(
119 Scaffold(
99 topBar = {
120 topBar = {
100 TopAppBar(
121 TopAppBar(
101− 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+ },
102 navigationIcon = {
139 navigationIcon = {
103 IconButton(onClick = { dispatch(Event.NavigateToConnect) }) {
140 IconButton(onClick = { dispatch(Event.NavigateToConnect) }) {
104 Icon(Icons.AutoMirrored.Filled.ArrowBack, contentDescription = "Disconnect")
141 Icon(Icons.AutoMirrored.Filled.ArrowBack, contentDescription = "Disconnect")
@@ -162,6 +199,7 @@ fun SessionsScreen(state: UiState, dispatch: (Event) -> Unit) {
162 state = rememberPullToRefreshState(),
199 state = rememberPullToRefreshState(),
163 ) {
200 ) {
164 LazyColumn(
201 LazyColumn(
202+ state = sessionsListState,
165 modifier = Modifier.fillMaxSize(),
203 modifier = Modifier.fillMaxSize(),
166 contentPadding = PaddingValues(bottom = 88.dp),
204 contentPadding = PaddingValues(bottom = 88.dp),
167 ) {
205 ) {
@@ -200,13 +238,21 @@ fun SessionsScreen(state: UiState, dispatch: (Event) -> Unit) {
200 }
238 }
201}
239}
202
240
241+@OptIn(ExperimentalFoundationApi::class)
203@Composable
242@Composable
204private fun SessionRow(session: SessionView, onOpen: () -> Unit, onDelete: () -> Unit) {
243private fun SessionRow(session: SessionView, onOpen: () -> Unit, onDelete: () -> Unit) {
244+ val context = LocalContext.current
205 Row(
245 Row(
206 modifier = Modifier
246 modifier = Modifier
207 .fillMaxWidth()
247 .fillMaxWidth()
208 .background(MaterialTheme.colorScheme.surface)
248 .background(MaterialTheme.colorScheme.surface)
249+ .combinedClickable(
209− .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+ )
210 .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),
211 verticalAlignment = Alignment.CenterVertically,
257 verticalAlignment = Alignment.CenterVertically,
212 ) {
258 ) {
@@ -241,13 +287,13 @@ private fun SessionRow(session: SessionView, onOpen: () -> Unit, onDelete: () ->
241 Icon(
287 Icon(
242 Icons.Filled.DeleteOutline,
288 Icons.Filled.DeleteOutline,
243 contentDescription = "Delete session",
289 contentDescription = "Delete session",
244− tint = MaterialTheme.colorScheme.onSurfaceVariant,
290+ tint = MaterialTheme.colorScheme.error,
245 )
291 )
246 }
292 }
247 }
293 }
248}
294}
249
295
250−/** Red background with a trailing delete icon, revealed while swiping a session row away. */
296+/** Red background with a trailing delete icon and label, revealed while swiping a session row away. */
251@Composable
297@Composable
252private fun SwipeDeleteBackground() {
298private fun SwipeDeleteBackground() {
253 Box(
299 Box(
@@ -257,11 +303,19 @@ private fun SwipeDeleteBackground() {
257 .padding(horizontal = 24.dp),
303 .padding(horizontal = 24.dp),
258 contentAlignment = Alignment.CenterEnd,
304 contentAlignment = Alignment.CenterEnd,
259 ) {
305 ) {
306+ Row(verticalAlignment = Alignment.CenterVertically) {
260− Icon(
307+ Icon(
261− Icons.Filled.DeleteOutline,
308+ Icons.Filled.DeleteOutline,
262− contentDescription = null,
309+ contentDescription = null,
263− tint = MaterialTheme.colorScheme.onError,
310+ tint = MaterialTheme.colorScheme.onError,
264− )
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+ }
265 }
319 }
266}
320}
267
321
@@ -291,6 +345,11 @@ private fun ReconnectingBanner() {
291 }
345 }
292}
346}
293
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+
294/** 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. */
295private fun shortSessionId(id: String): String =
354private fun shortSessionId(id: String): String =
296 if (id.length > 12) id.take(8) + "…" else id
355 if (id.length > 12) id.take(8) + "…" else id
@@ -332,5 +391,12 @@ private fun EmptySessions() {
332 style = MaterialTheme.typography.bodyMedium,
391 style = MaterialTheme.typography.bodyMedium,
333 color = MaterialTheme.colorScheme.onSurfaceVariant,
392 color = MaterialTheme.colorScheme.onSurfaceVariant,
334 )
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+ )
335 }
401 }
336}
402}
app/src/main/java/soy/iko/opencode/ui/theme/Theme.kt+10
@@ -17,6 +17,11 @@ private val DarkColors = darkColorScheme(
17 primaryContainer = Color(0xFF1A3D2A),
17 primaryContainer = Color(0xFF1A3D2A),
18 onPrimaryContainer = Color(0xFFA8E6C1),
18 onPrimaryContainer = Color(0xFFA8E6C1),
19 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),
20 tertiaryContainer = Color(0xFF2A2A3D),
25 tertiaryContainer = Color(0xFF2A2A3D),
21 onTertiaryContainer = Color(0xFFC5C5E6),
26 onTertiaryContainer = Color(0xFFC5C5E6),
22 errorContainer = Color(0xFF4D2626),
27 errorContainer = Color(0xFF4D2626),
@@ -33,6 +38,11 @@ private val LightColors = lightColorScheme(
33 primaryContainer = Color(0xFFD7F5E3),
38 primaryContainer = Color(0xFFD7F5E3),
34 onPrimaryContainer = Color(0xFF0A3D22),
39 onPrimaryContainer = Color(0xFF0A3D22),
35 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),
36 tertiaryContainer = Color(0xFFE8E8F5),
46 tertiaryContainer = Color(0xFFE8E8F5),
37 onTertiaryContainer = Color(0xFF2A2A4D),
47 onTertiaryContainer = Color(0xFF2A2A4D),
38 errorContainer = Color(0xFFFFE0E0),
48 errorContainer = Color(0xFFFFE0E0),