git.iko.soy/opencode-android
on cd89013 help sign in
app/src/main/java/soy/iko/opencode/CoreViewModel.kt−14
since splitsnapshots
unifiedsplit
@@ -23,23 +23,11 @@ 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)) }
29
26
30 // On a fresh process after a configuration change, resume the SSE stream if
27 // On a fresh process after a configuration change, resume the SSE stream if
31 // the core is already connected (no-op otherwise — it guards on `connected`).
28 // the core is already connected (no-op otherwise — it guards on `connected`).
32 core.resumeStreamingIfConnected()
29 core.resumeStreamingIfConnected()
33
30
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−
43 // Persist connection details once a connection succeeds.
31 // Persist connection details once a connection succeeds.
44 viewModelScope.launch {
32 viewModelScope.launch {
45 core.view.distinctUntilChangedBy { it.connected to it.serverUrl }.collect { state ->
33 core.view.distinctUntilChangedBy { it.connected to it.serverUrl }.collect { state ->
@@ -47,7 +35,6 @@ class CoreViewModel(app: Application) : AndroidViewModel(app) {
47 prefs.edit {
35 prefs.edit {
48 putString(KEY_URL, state.serverUrl)
36 putString(KEY_URL, state.serverUrl)
49 putString(KEY_USER, state.username)
37 putString(KEY_USER, state.username)
50− putString(KEY_PASSWORD, state.password)
51 }
38 }
52 }
39 }
53 }
40 }
@@ -61,6 +48,5 @@ class CoreViewModel(app: Application) : AndroidViewModel(app) {
61 private companion object {
48 private companion object {
62 const val KEY_URL = "serverUrl"
49 const val KEY_URL = "serverUrl"
63 const val KEY_USER = "username"
50 const val KEY_USER = "username"
64− const val KEY_PASSWORD = "password"
65 }
51 }
66}
52}
app/src/main/java/soy/iko/opencode/core/HttpClient.kt−1
@@ -51,7 +51,6 @@ 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")
55 if (auth != null) builder.header("Authorization", auth)
54 if (auth != null) builder.header("Authorization", auth)
56 await(okhttp.newCall(builder.build()))
55 await(okhttp.newCall(builder.build()))
57 }
56 }
app/src/main/java/soy/iko/opencode/core/OpencodeCore.kt+9−95
@@ -54,24 +54,6 @@ class OpencodeCore(
54 // Guards against overlapping session refreshes: rapid taps launch multiple
54 // Guards against overlapping session refreshes: rapid taps launch multiple
55 // loadSessions coroutines and the slowest-to-return would otherwise win.
55 // loadSessions coroutines and the slowest-to-return would otherwise win.
56 private var loadSessionsJob: Job? = null
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.
57 // Message IDs removed via SSE; a late MessageUpdated for one must not resurrect it.
76 private val removedMessageIds = mutableSetOf<String>()
58 private val removedMessageIds = mutableSetOf<String>()
77
59
@@ -96,12 +78,10 @@ class OpencodeCore(
96 // Leave authRequired as-is; probeHealth sets it from the response (true on
78 // 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.
79 // 401, false on success). Resetting it here just flickers the auth fields.
98 loading = true; error = null; emit()
80 loading = true; error = null; emit()
99− connectJob?.cancel()
100− connectJob = scope.launch { probeHealth() }
81+ scope.launch { probeHealth() }
101 }
82 }
102
83
103 Event.LoadSessions -> {
84 Event.LoadSessions -> {
104− if (!connected) { error = "Not connected"; emit(); return }
105 loading = true; emit()
85 loading = true; emit()
106 loadSessionsJob?.cancel()
86 loadSessionsJob?.cancel()
107 loadSessionsJob = scope.launch { loadSessions() }
87 loadSessionsJob = scope.launch { loadSessions() }
@@ -121,29 +101,17 @@ class OpencodeCore(
121 generating = false
101 generating = false
122 error = null
102 error = null
123 emit()
103 emit()
124− connectJob?.cancel()
125− abortJob?.cancel()
126− sendMessageJob?.cancel()
127− deleteSessionJob?.cancel()
128− loadMessagesJob?.cancel()
129− loadMessagesJob = scope.launch { loadMessages(event.id) }
104+ scope.launch { loadMessages(event.id) }
130 }
105 }
131
106
132− Event.CreateSession -> {
133− if (!connected) { error = "Not connected"; emit(); return }
134− loading = true; error = null; emit()
107+ Event.CreateSession -> { loading = true; error = null; emit(); scope.launch { createSession() } }
135− createSessionJob?.cancel()
136− createSessionJob = scope.launch { createSession() }
137− }
138
108
139 is Event.DeleteSession -> {
109 is Event.DeleteSession -> {
140− if (!connected) { error = "Not connected"; emit(); return }
141 val id = event.id
110 val id = event.id
142 // Delete on the server first; only drop it from the UI once that
111 // Delete on the server first; only drop it from the UI once that
143 // succeeds, so a failed request leaves the session recoverable.
112 // succeeds, so a failed request leaves the session recoverable.
144 loading = true; error = null; emit()
113 loading = true; error = null; emit()
145− deleteSessionJob?.cancel()
146− deleteSessionJob = scope.launch {
114+ scope.launch {
147 http.delete("$serverUrl/session/$id", authHeader()).fold(
115 http.delete("$serverUrl/session/$id", authHeader()).fold(
148 onSuccess = { resp ->
116 onSuccess = { resp ->
149 if (resp.code in 200..299) {
117 if (resp.code in 200..299) {
@@ -176,10 +144,7 @@ class OpencodeCore(
176 messages.forEach { it.streaming = false }
144 messages.forEach { it.streaming = false }
177 emit()
145 emit()
178 val id = currentSessionId ?: return
146 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()
182− abortJob = scope.launch { http.postJson("$serverUrl/session/$id/abort", "{}", authHeader()) }
147+ scope.launch { http.postJson("$serverUrl/session/$id/abort", "{}", authHeader()) }
183 }
148 }
184
149
185 Event.NavigateToSessions -> {
150 Event.NavigateToSessions -> {
@@ -187,19 +152,9 @@ class OpencodeCore(
187 currentSessionId = null
152 currentSessionId = null
188 messages.clear()
153 messages.clear()
189 removedMessageIds.clear()
154 removedMessageIds.clear()
190− draft = ""
191 generating = false
155 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
195 loading = true
156 loading = true
196 emit()
157 emit()
197− connectJob?.cancel()
198− abortJob?.cancel()
199− createSessionJob?.cancel()
200− deleteSessionJob?.cancel()
201− sendMessageJob?.cancel()
202− loadMessagesJob?.cancel()
203 loadSessionsJob?.cancel()
158 loadSessionsJob?.cancel()
204 loadSessionsJob = scope.launch { loadSessions() }
159 loadSessionsJob = scope.launch { loadSessions() }
205 }
160 }
@@ -213,16 +168,9 @@ class OpencodeCore(
213 messages.clear()
168 messages.clear()
214 removedMessageIds.clear()
169 removedMessageIds.clear()
215 currentSessionId = null
170 currentSessionId = null
216− draft = ""
217 // Reset to the default so a late-firing close callback from the old
171 // 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.
172 // SSE connection can't flash the reconnecting banner on the next connect.
219 sseConnected = true
173 sseConnected = true
220− connectJob?.cancel()
221− abortJob?.cancel()
222− createSessionJob?.cancel()
223− deleteSessionJob?.cancel()
224− sendMessageJob?.cancel()
225− loadMessagesJob?.cancel()
226 closeSse()
174 closeSse()
227 emit()
175 emit()
228 }
176 }
@@ -236,27 +184,13 @@ class OpencodeCore(
236 if (connected && sse == null) startSse()
184 if (connected && sse == null) startSse()
237 }
185 }
238
186
239− fun shutdown() {
187+ fun shutdown() = closeSse()
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− }
251
188
252 // ── Effects ────────────────────────────────────────────────────────────────
189 // ── Effects ────────────────────────────────────────────────────────────────
253 private fun sendMessage(raw: String) {
190 private fun sendMessage(raw: String) {
254 val text = raw.trim()
191 val text = raw.trim()
255 val session = currentSessionId ?: return
192 val session = currentSessionId ?: return
256 if (text.isEmpty()) return
193 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
260 localSeq += 1
194 localSeq += 1
261 val user = MsgState("local-$localSeq", "user", System.currentTimeMillis()).apply {
195 val user = MsgState("local-$localSeq", "user", System.currentTimeMillis()).apply {
262 status = MessageStatus.Pending
196 status = MessageStatus.Pending
@@ -272,8 +206,7 @@ class OpencodeCore(
272 emit()
206 emit()
273
207
274 val body = Protocol.json.encodeToString(PromptBody.serializer(), PromptBody(listOf(TextInput(text = text))))
208 val body = Protocol.json.encodeToString(PromptBody.serializer(), PromptBody(listOf(TextInput(text = text))))
275− sendMessageJob?.cancel()
276− sendMessageJob = scope.launch {
209+ scope.launch {
277 val result = http.postJson("$serverUrl/session/$session/prompt_async", body, authHeader())
210 val result = http.postJson("$serverUrl/session/$session/prompt_async", body, authHeader())
278 result.fold(
211 result.fold(
279 onSuccess = { resp ->
212 onSuccess = { resp ->
@@ -306,10 +239,6 @@ class OpencodeCore(
306 when {
239 when {
307 resp.code == 401 -> {
240 resp.code == 401 -> {
308 loading = false
241 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
313 // Always reveal the auth fields so the user can see/edit
242 // Always reveal the auth fields so the user can see/edit
314 // their credentials — even when the ones supplied were wrong.
243 // their credentials — even when the ones supplied were wrong.
315 authRequired = true
244 authRequired = true
@@ -324,14 +253,10 @@ class OpencodeCore(
324 loading = true
253 loading = true
325 emit()
254 emit()
326 startSse()
255 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()
330− loadSessionsJob = scope.launch { loadSessions() }
256+ loadSessions()
331 }
257 }
332 else -> {
258 else -> {
333 loading = false
259 loading = false
334− connected = false
335 error = "Server returned status ${resp.code}"
260 error = "Server returned status ${resp.code}"
336 emit()
261 emit()
337 }
262 }
@@ -339,7 +264,6 @@ class OpencodeCore(
339 },
264 },
340 onFailure = { e ->
265 onFailure = { e ->
341 loading = false
266 loading = false
342− connected = false
343 error = "Connection failed: ${e.message}"
267 error = "Connection failed: ${e.message}"
344 emit()
268 emit()
345 },
269 },
@@ -352,9 +276,6 @@ class OpencodeCore(
352 if (resp.code in 200..299) {
276 if (resp.code in 200..299) {
353 sessions.clear()
277 sessions.clear()
354 sessions.addAll(Protocol.parseSessions(resp.body).map { SessionView(it.id, displayTitle(it.title)) })
278 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() }
358 } else {
279 } else {
359 error = "Server returned status ${resp.code}"
280 error = "Server returned status ${resp.code}"
360 }
281 }
@@ -372,10 +293,6 @@ class OpencodeCore(
372 if (session != null) {
293 if (session != null) {
373 if (sessions.none { it.id == session.id }) {
294 if (sessions.none { it.id == session.id }) {
374 sessions.add(0, SessionView(session.id, displayTitle(session.title)))
295 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() }
379 }
296 }
380 emit()
297 emit()
381 // SelectSession takes over the loading state (loads messages).
298 // SelectSession takes over the loading state (loads messages).
@@ -482,7 +399,7 @@ class OpencodeCore(
482 } else {
399 } else {
483 messages.add(MsgState(info.id, "assistant", info.time.created).apply { this.streaming = streaming })
400 messages.add(MsgState(info.id, "assistant", info.time.created).apply { this.streaming = streaming })
484 }
401 }
485− info.error?.let { error = it.message(); messages.forEach { m -> m.streaming = false }; generating = false }
402+ info.error?.let { error = it.message(); generating = false }
486 return true
403 return true
487 }
404 }
488
405
@@ -545,9 +462,6 @@ class OpencodeCore(
545 } else {
462 } else {
546 sessions.add(0, SessionView(event.info.id, title))
463 sessions.add(0, SessionView(event.info.id, title))
547 }
464 }
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() }
551 if (isCurrent(event.info.id)) currentSessionTitle = title
465 if (isCurrent(event.info.id)) currentSessionTitle = title
552 return true
466 return true
553 }
467 }
app/src/main/java/soy/iko/opencode/core/SseClient.kt−1
@@ -33,7 +33,6 @@ 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")
37 if (auth != null) builder.header("Authorization", auth)
36 if (auth != null) builder.header("Authorization", auth)
38 source = EventSources.createFactory(client).newEventSource(builder.build(), listener)
37 source = EventSources.createFactory(client).newEventSource(builder.build(), listener)
39 }
38 }
app/src/main/java/soy/iko/opencode/ui/OpencodeApp.kt+2−55
@@ -7,19 +7,13 @@ 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
11import androidx.compose.material3.Scaffold
10import androidx.compose.material3.Scaffold
12import androidx.compose.material3.SnackbarDuration
11import androidx.compose.material3.SnackbarDuration
13import androidx.compose.material3.SnackbarHost
12import androidx.compose.material3.SnackbarHost
14import androidx.compose.material3.SnackbarHostState
13import androidx.compose.material3.SnackbarHostState
15−import androidx.compose.material3.Text
16−import androidx.compose.material3.TextButton
17import androidx.compose.runtime.Composable
14import androidx.compose.runtime.Composable
18import androidx.compose.runtime.LaunchedEffect
15import androidx.compose.runtime.LaunchedEffect
19−import androidx.compose.runtime.getValue
20−import androidx.compose.runtime.mutableStateOf
21import androidx.compose.runtime.remember
16import androidx.compose.runtime.remember
22−import androidx.compose.runtime.setValue
23import androidx.compose.ui.Modifier
17import androidx.compose.ui.Modifier
24import soy.iko.opencode.core.Event
18import soy.iko.opencode.core.Event
25import soy.iko.opencode.core.Screen
19import soy.iko.opencode.core.Screen
@@ -31,8 +25,6 @@ import soy.iko.opencode.ui.screens.SessionsScreen
31@Composable
25@Composable
32fun OpencodeApp(state: UiState, dispatch: (Event) -> Unit) {
26fun OpencodeApp(state: UiState, dispatch: (Event) -> Unit) {
33 val snackbar = remember { SnackbarHostState() }
27 val snackbar = remember { SnackbarHostState() }
34− var showStopConfirm by remember { mutableStateOf(false) }
35− var showDisconnectConfirm by remember { mutableStateOf(false) }
36
28
37 LaunchedEffect(state.error) {
29 LaunchedEffect(state.error) {
38 state.error?.let {
30 state.error?.let {
@@ -48,57 +40,12 @@ fun OpencodeApp(state: UiState, dispatch: (Event) -> Unit) {
48
40
49 BackHandler(enabled = state.screen != Screen.Connect) {
41 BackHandler(enabled = state.screen != Screen.Connect) {
50 when (state.screen) {
42 when (state.screen) {
51− // Leaving mid-generation would silently drop the reply, so confirm first.
52− Screen.Chat -> if (state.generating) showStopConfirm = true else dispatch(Event.NavigateToSessions)
43+ Screen.Chat -> dispatch(Event.NavigateToSessions)
53− // Disconnecting drops the server session, so guard the back button.
44+ Screen.Sessions -> dispatch(Event.NavigateToConnect)
54− Screen.Sessions -> showDisconnectConfirm = true
55 Screen.Connect -> Unit
45 Screen.Connect -> Unit
56 }
46 }
57 }
47 }
58
48
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−
102 Scaffold(snackbarHost = { SnackbarHost(snackbar) }) { padding ->
49 Scaffold(snackbarHost = { SnackbarHost(snackbar) }) { padding ->
103 AnimatedContent(
50 AnimatedContent(
104 targetState = state.screen,
51 targetState = state.screen,
app/src/main/java/soy/iko/opencode/ui/components/MessageBubble.kt+7−237
@@ -3,8 +3,6 @@ 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
8import android.widget.Toast
6import android.widget.Toast
9import androidx.compose.foundation.ExperimentalFoundationApi
7import androidx.compose.foundation.ExperimentalFoundationApi
10import androidx.compose.foundation.background
8import androidx.compose.foundation.background
@@ -15,7 +13,6 @@ import androidx.compose.foundation.layout.Column
15import androidx.compose.foundation.layout.Row
13import androidx.compose.foundation.layout.Row
16import androidx.compose.foundation.layout.fillMaxWidth
14import androidx.compose.foundation.layout.fillMaxWidth
17import androidx.compose.foundation.layout.padding
15import androidx.compose.foundation.layout.padding
18−import androidx.compose.foundation.layout.size
19import androidx.compose.foundation.layout.widthIn
16import androidx.compose.foundation.layout.widthIn
20import androidx.compose.foundation.shape.RoundedCornerShape
17import androidx.compose.foundation.shape.RoundedCornerShape
21import androidx.compose.foundation.text.selection.SelectionContainer
18import androidx.compose.foundation.text.selection.SelectionContainer
@@ -26,9 +23,6 @@ import androidx.compose.material.icons.filled.ErrorOutline
26import androidx.compose.material.icons.filled.ExpandLess
23import androidx.compose.material.icons.filled.ExpandLess
27import androidx.compose.material.icons.filled.ExpandMore
24import androidx.compose.material.icons.filled.ExpandMore
28import androidx.compose.material.icons.filled.Schedule
25import 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
32import androidx.compose.material3.Icon
26import androidx.compose.material3.Icon
33import androidx.compose.material3.MaterialTheme
27import androidx.compose.material3.MaterialTheme
34import androidx.compose.material3.Text
28import androidx.compose.material3.Text
@@ -42,17 +36,10 @@ import androidx.compose.ui.Modifier
42import androidx.compose.ui.draw.clip
36import androidx.compose.ui.draw.clip
43import androidx.compose.ui.graphics.Color
37import androidx.compose.ui.graphics.Color
44import androidx.compose.ui.platform.LocalContext
38import androidx.compose.ui.platform.LocalContext
45−import androidx.compose.ui.text.AnnotatedString
46−import androidx.compose.ui.text.LinkAnnotation
47import androidx.compose.ui.text.SpanStyle
39import androidx.compose.ui.text.SpanStyle
48−import androidx.compose.ui.text.TextLinkStyles
49import androidx.compose.ui.text.buildAnnotatedString
40import androidx.compose.ui.text.buildAnnotatedString
50−import androidx.compose.ui.text.font.FontFamily
51import androidx.compose.ui.text.font.FontStyle
41import androidx.compose.ui.text.font.FontStyle
52−import androidx.compose.ui.text.font.FontWeight
53−import androidx.compose.ui.text.withLink
54import androidx.compose.ui.text.withStyle
42import androidx.compose.ui.text.withStyle
55−import androidx.compose.ui.text.style.TextDecoration
56import androidx.compose.ui.text.style.TextOverflow
43import androidx.compose.ui.text.style.TextOverflow
57import androidx.compose.ui.unit.dp
44import androidx.compose.ui.unit.dp
58import androidx.compose.ui.unit.sp
45import androidx.compose.ui.unit.sp
@@ -65,7 +52,7 @@ private val timeFormat = java.time.format.DateTimeFormatter.ofPattern("HH:mm")
65
52
66@OptIn(ExperimentalFoundationApi::class)
53@OptIn(ExperimentalFoundationApi::class)
67@Composable
54@Composable
68−fun MessageBubble(message: MessageView, modifier: Modifier = Modifier, onRetry: () -> Unit = {}, compactSpacing: Boolean = false) {
55+fun MessageBubble(message: MessageView, modifier: Modifier = Modifier) {
69 val isUser = message.isUser
56 val isUser = message.isUser
70 val bubbleColor = if (isUser) MaterialTheme.colorScheme.primaryContainer else MaterialTheme.colorScheme.surfaceVariant
57 val bubbleColor = if (isUser) MaterialTheme.colorScheme.primaryContainer else MaterialTheme.colorScheme.surfaceVariant
71 val textColor = if (isUser) MaterialTheme.colorScheme.onPrimaryContainer else MaterialTheme.colorScheme.onSurface
58 val textColor = if (isUser) MaterialTheme.colorScheme.onPrimaryContainer else MaterialTheme.colorScheme.onSurface
@@ -78,20 +65,11 @@ fun MessageBubble(message: MessageView, modifier: Modifier = Modifier, onRetry:
78 )
65 )
79
66
80 val context = LocalContext.current
67 val context = LocalContext.current
81− var showMenu by remember { mutableStateOf(false) }
82
68
83 Row(
69 Row(
84− modifier = modifier.fillMaxWidth().padding(horizontal = 12.dp, vertical = if (compactSpacing) 1.dp else 4.dp),
70+ modifier = modifier.fillMaxWidth().padding(horizontal = 12.dp, vertical = 4.dp),
85 horizontalArrangement = if (isUser) Arrangement.End else Arrangement.Start,
71 horizontalArrangement = if (isUser) Arrangement.End else Arrangement.Start,
86 ) {
72 ) {
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− }
95 Column(
73 Column(
96 modifier = Modifier
74 modifier = Modifier
97 .fillMaxWidth(0.85f)
75 .fillMaxWidth(0.85f)
@@ -102,55 +80,21 @@ fun MessageBubble(message: MessageView, modifier: Modifier = Modifier, onRetry:
102 onClick = {},
80 onClick = {},
103 onLongClick = {
81 onLongClick = {
104 if (message.text.isNotEmpty()) {
82 if (message.text.isNotEmpty()) {
105− showMenu = true
83+ copyToClipboard(context, message.text)
84+ Toast.makeText(context, "Copied", Toast.LENGTH_SHORT).show()
106 }
85 }
107 },
86 },
108 )
87 )
109 .padding(horizontal = 14.dp, vertical = 10.dp),
88 .padding(horizontal = 14.dp, vertical = 10.dp),
110 ) {
89 ) {
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−
129 message.reasoning?.let { ReasoningBlock(it, context) }
90 message.reasoning?.let { ReasoningBlock(it, context) }
130
91
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−
141 message.tools.forEach { ToolRow(it) }
92 message.tools.forEach { ToolRow(it) }
142
93
143 SelectionContainer {
94 SelectionContainer {
144 if (message.text.isNotEmpty()) {
95 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(
96 Text(
153− text = annotatedText,
97+ text = message.text + if (message.streaming) " ▌" else "",
154 color = textColor,
98 color = textColor,
155 // Give assistant replies (often long, code-heavy) extra line
99 // Give assistant replies (often long, code-heavy) extra line
156 // spacing for readability; user bubbles keep the default.
100 // spacing for readability; user bubbles keep the default.
@@ -175,7 +119,7 @@ fun MessageBubble(message: MessageView, modifier: Modifier = Modifier, onRetry:
175 }
119 }
176
120
177 if (isUser && message.status != MessageStatus.Sent) {
121 if (isUser && message.status != MessageStatus.Sent) {
178− StatusLine(message.status, onRetry = onRetry)
122+ StatusLine(message.status)
179 }
123 }
180 }
124 }
181 }
125 }
@@ -186,170 +130,6 @@ private fun copyToClipboard(context: Context, text: String) {
186 clipboard.setPrimaryClip(ClipData.newPlainText("message", text))
130 clipboard.setPrimaryClip(ClipData.newPlainText("message", text))
187}
131}
188
132
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−
353@Composable
133@Composable
354private fun ReasoningBlock(text: String, context: Context) {
134private fun ReasoningBlock(text: String, context: Context) {
355 var expanded by remember { mutableStateOf(false) }
135 var expanded by remember { mutableStateOf(false) }
@@ -447,7 +227,7 @@ private fun ToolRow(tool: ToolView) {
447}
227}
448
228
449@Composable
229@Composable
450−private fun StatusLine(status: MessageStatus, onRetry: () -> Unit) {
230+private fun StatusLine(status: MessageStatus) {
451 val (icon, label, tint) = when (status) {
231 val (icon, label, tint) = when (status) {
452 MessageStatus.Pending -> Triple(Icons.Filled.Schedule, "Sending…", MaterialTheme.colorScheme.onPrimaryContainer)
232 MessageStatus.Pending -> Triple(Icons.Filled.Schedule, "Sending…", MaterialTheme.colorScheme.onPrimaryContainer)
453 MessageStatus.Failed -> Triple(Icons.Filled.ErrorOutline, "Failed to send", MaterialTheme.colorScheme.error)
233 MessageStatus.Failed -> Triple(Icons.Filled.ErrorOutline, "Failed to send", MaterialTheme.colorScheme.error)
@@ -465,15 +245,5 @@ private fun StatusLine(status: MessageStatus, onRetry: () -> Unit) {
465 color = tint,
245 color = tint,
466 modifier = Modifier.padding(start = 4.dp),
246 modifier = Modifier.padding(start = 4.dp),
467 )
247 )
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− }
478 }
248 }
479}
249}
app/src/main/java/soy/iko/opencode/ui/screens/ChatScreen.kt+15−270
@@ -1,23 +1,12 @@
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
7import androidx.compose.animation.AnimatedVisibility
3import 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
13import androidx.compose.animation.fadeIn
4import androidx.compose.animation.fadeIn
14import androidx.compose.animation.fadeOut
5import androidx.compose.animation.fadeOut
15import androidx.compose.animation.scaleIn
6import androidx.compose.animation.scaleIn
16import androidx.compose.animation.scaleOut
7import androidx.compose.animation.scaleOut
17−import androidx.compose.foundation.ExperimentalFoundationApi
18import androidx.compose.foundation.background
8import androidx.compose.foundation.background
19import androidx.compose.foundation.clickable
9import androidx.compose.foundation.clickable
20−import androidx.compose.foundation.combinedClickable
21import androidx.compose.foundation.layout.Arrangement
10import androidx.compose.foundation.layout.Arrangement
22import androidx.compose.foundation.layout.Box
11import androidx.compose.foundation.layout.Box
23import androidx.compose.foundation.layout.BoxScope
12import androidx.compose.foundation.layout.BoxScope
@@ -26,27 +15,21 @@ import androidx.compose.foundation.layout.Row
26import androidx.compose.foundation.layout.Spacer
15import androidx.compose.foundation.layout.Spacer
27import androidx.compose.foundation.layout.fillMaxSize
16import androidx.compose.foundation.layout.fillMaxSize
28import androidx.compose.foundation.layout.fillMaxWidth
17import androidx.compose.foundation.layout.fillMaxWidth
29−import androidx.compose.foundation.layout.heightIn
30import androidx.compose.foundation.layout.PaddingValues
18import androidx.compose.foundation.layout.PaddingValues
31−import androidx.compose.foundation.layout.WindowInsets
32−import androidx.compose.foundation.layout.ime
33import androidx.compose.foundation.layout.imePadding
19import androidx.compose.foundation.layout.imePadding
34import androidx.compose.foundation.layout.padding
20import androidx.compose.foundation.layout.padding
35import androidx.compose.foundation.layout.size
21import androidx.compose.foundation.layout.size
36import androidx.compose.foundation.lazy.LazyColumn
22import androidx.compose.foundation.lazy.LazyColumn
37−import androidx.compose.foundation.lazy.itemsIndexed
23+import androidx.compose.foundation.lazy.items
38import androidx.compose.foundation.lazy.rememberLazyListState
24import androidx.compose.foundation.lazy.rememberLazyListState
39import androidx.compose.foundation.shape.CircleShape
25import androidx.compose.foundation.shape.CircleShape
40−import androidx.compose.foundation.shape.RoundedCornerShape
41import androidx.compose.foundation.text.KeyboardActions
26import androidx.compose.foundation.text.KeyboardActions
42import androidx.compose.foundation.text.KeyboardOptions
27import androidx.compose.foundation.text.KeyboardOptions
43import androidx.compose.material.icons.Icons
28import androidx.compose.material.icons.Icons
44import androidx.compose.material.icons.automirrored.filled.ArrowBack
29import androidx.compose.material.icons.automirrored.filled.ArrowBack
45import androidx.compose.material.icons.automirrored.filled.Send
30import androidx.compose.material.icons.automirrored.filled.Send
46−import androidx.compose.material.icons.filled.ChatBubbleOutline
47import androidx.compose.material.icons.filled.Close
31import androidx.compose.material.icons.filled.Close
48import androidx.compose.material.icons.filled.KeyboardArrowDown
32import androidx.compose.material.icons.filled.KeyboardArrowDown
49−import androidx.compose.material.icons.filled.KeyboardArrowUp
50import androidx.compose.material.icons.filled.Stop
33import androidx.compose.material.icons.filled.Stop
51import androidx.compose.material3.CircularProgressIndicator
34import androidx.compose.material3.CircularProgressIndicator
52import androidx.compose.material3.ExperimentalMaterial3Api
35import androidx.compose.material3.ExperimentalMaterial3Api
@@ -61,7 +44,6 @@ import androidx.compose.material3.OutlinedTextField
61import androidx.compose.material3.Scaffold
44import androidx.compose.material3.Scaffold
62import androidx.compose.material3.Surface
45import androidx.compose.material3.Surface
63import androidx.compose.material3.Text
46import androidx.compose.material3.Text
64−import androidx.compose.material3.TextButton
65import androidx.compose.material3.TopAppBar
47import androidx.compose.material3.TopAppBar
66import androidx.compose.runtime.Composable
48import androidx.compose.runtime.Composable
67import androidx.compose.runtime.LaunchedEffect
49import androidx.compose.runtime.LaunchedEffect
@@ -75,13 +57,8 @@ import androidx.compose.ui.Alignment
75import androidx.compose.ui.Modifier
57import androidx.compose.ui.Modifier
76import androidx.compose.ui.draw.clip
58import androidx.compose.ui.draw.clip
77import androidx.compose.ui.hapticfeedback.HapticFeedbackType
59import 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
81import androidx.compose.ui.platform.LocalHapticFeedback
60import androidx.compose.ui.platform.LocalHapticFeedback
82−import androidx.compose.ui.text.font.FontStyle
83import androidx.compose.ui.text.input.ImeAction
61import androidx.compose.ui.text.input.ImeAction
84−import androidx.compose.ui.text.style.TextAlign
85import androidx.compose.ui.text.style.TextOverflow
62import androidx.compose.ui.text.style.TextOverflow
86import androidx.compose.ui.unit.dp
63import androidx.compose.ui.unit.dp
87import kotlinx.coroutines.launch
64import kotlinx.coroutines.launch
@@ -89,17 +66,11 @@ import soy.iko.opencode.core.Event
89import soy.iko.opencode.core.UiState
66import soy.iko.opencode.core.UiState
90import soy.iko.opencode.ui.components.MessageBubble
67import soy.iko.opencode.ui.components.MessageBubble
91
68
92−@OptIn(ExperimentalMaterial3Api::class, ExperimentalFoundationApi::class)
69+@OptIn(ExperimentalMaterial3Api::class)
93@Composable
70@Composable
94fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
71fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
95 val listState = rememberLazyListState()
72 val listState = rememberLazyListState()
96 val coroutineScope = rememberCoroutineScope()
73 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
103
74
104 // Show a jump-to-bottom button once the user scrolls up away from the latest
75 // Show a jump-to-bottom button once the user scrolls up away from the latest
105 // message. derivedStateOf keeps this from recomposing on every pixel of scroll —
76 // message. derivedStateOf keeps this from recomposing on every pixel of scroll —
@@ -113,12 +84,6 @@ fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
113 }
84 }
114 }
85 }
115
86
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
87 // 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.
88 // they're scrolled up, any messages beyond that count are unseen — flag the FAB.
124 var lastSeenCount by remember { mutableStateOf(0) }
89 var lastSeenCount by remember { mutableStateOf(0) }
@@ -148,20 +113,11 @@ fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
148 topBar = {
113 topBar = {
149 TopAppBar(
114 TopAppBar(
150 title = {
115 title = {
151− val title = state.currentSessionTitle.ifEmpty { "Chat" }
152 Text(
116 Text(
153− text = if (state.messages.isNotEmpty()) "$title (${state.messages.size})" else title,
117+ text = state.currentSessionTitle.ifEmpty { "Chat" },
154 maxLines = 1,
118 maxLines = 1,
155 overflow = TextOverflow.Ellipsis,
119 overflow = TextOverflow.Ellipsis,
156− modifier = Modifier.combinedClickable(
157− onClick = { dispatch(Event.NavigateToSessions) },
120+ modifier = Modifier.clickable { 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− ),
165 )
121 )
166 },
122 },
167 navigationIcon = {
123 navigationIcon = {
@@ -205,78 +161,33 @@ fun ChatScreen(state: UiState, dispatch: (Event) -> Unit) {
205 }
161 }
206 Box(Modifier.fillMaxWidth().weight(1f)) {
162 Box(Modifier.fillMaxWidth().weight(1f)) {
207 if (state.messages.isEmpty() && !state.loading) {
163 if (state.messages.isEmpty() && !state.loading) {
208− Column(
164+ Text(
165+ text = "Send a message to start the conversation.",
166+ style = MaterialTheme.typography.bodyMedium,
167+ color = MaterialTheme.colorScheme.onSurfaceVariant,
209 modifier = Modifier.align(Alignment.Center).padding(32.dp),
168 modifier = Modifier.align(Alignment.Center).padding(32.dp),
210− horizontalAlignment = Alignment.CenterHorizontally,
211− ) {
169+ )
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− }
226 }
170 }
227 LazyColumn(
171 LazyColumn(
228 state = listState,
172 state = listState,
229 modifier = Modifier.fillMaxSize(),
173 modifier = Modifier.fillMaxSize(),
230 contentPadding = PaddingValues(vertical = 8.dp),
174 contentPadding = PaddingValues(vertical = 8.dp),
231 ) {
175 ) {
232− itemsIndexed(state.messages, key = { _, message -> message.id }) { index, message ->
176+ items(state.messages, key = { it.id }) { message ->
233− // Insert a date separator whenever a message falls on a different
234− // calendar day than the one before it (and above the first message).
235− val previous = state.messages.getOrNull(index - 1)
236− if (shouldShowDateSeparator(previous, message)) {
237− DateSeparator(message.time)
238− }
239− val sameSenderAsPrevious = index > 0 && state.messages[index - 1].isUser == message.isUser
240− MessageBubble(
177+ MessageBubble(message)
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− }
253 }
178 }
254 }
179 }
255 ScrollToBottomButton(
180 ScrollToBottomButton(
256− visible = showScrollButton && !isImeVisible,
181+ visible = showScrollButton,
257 showBadge = hasUnseenMessages,
182 showBadge = hasUnseenMessages,
258 onClick = {
183 onClick = {
259 coroutineScope.launch {
184 coroutineScope.launch {
260 if (state.messages.isNotEmpty()) {
185 if (state.messages.isNotEmpty()) {
261− // When the typing indicator is showing it occupies the item
262− // just past the last message, so scroll one further to reach it.
263− val target = if (state.generating && state.messages.none { it.streaming })
264− state.messages.size
265− else
266− state.messages.size - 1
186+ listState.animateScrollToItem(state.messages.size - 1)
267− listState.animateScrollToItem(target)
268 }
187 }
269 }
188 }
270 },
189 },
271 )
190 )
272− ScrollToTopButton(
273− visible = showScrollTop,
274− onClick = {
275− coroutineScope.launch {
276− listState.animateScrollToItem(0)
277− }
278− },
279− )
280 }
191 }
281 }
192 }
282 }
193 }
@@ -316,123 +227,6 @@ private fun BoxScope.ScrollToBottomButton(visible: Boolean, showBadge: Boolean,
316 }
227 }
317}
228}
318
229
319−// Mirror of ScrollToBottomButton, pinned to the top-start corner. Same BoxScope
320−// extension trick so the top-level AnimatedVisibility overload resolves cleanly.
321−@Composable
322−private fun BoxScope.ScrollToTopButton(visible: Boolean, onClick: () -> Unit) {
323− AnimatedVisibility(
324− visible = visible,
325− modifier = Modifier.align(Alignment.TopStart),
326− enter = scaleIn() + fadeIn(),
327− exit = scaleOut() + fadeOut(),
328− ) {
329− Box(Modifier.padding(16.dp)) {
330− FloatingActionButton(
331− onClick = onClick,
332− modifier = Modifier.size(44.dp),
333− ) {
334− Icon(
335− Icons.Filled.KeyboardArrowUp,
336− contentDescription = "Scroll to top",
337− )
338− }
339− }
340− }
341−}
342−
343−private fun copyToClipboard(context: Context, text: String) {
344− val clipboard = context.getSystemService(Context.CLIPBOARD_SERVICE) as ClipboardManager
345− clipboard.setPrimaryClip(ClipData.newPlainText("session", text))
346−}
347−
348−private val dateSeparatorFormat = java.time.format.DateTimeFormatter.ofPattern("MMM d")
349−
350−/** Convert an epoch-millis timestamp to the local calendar day it falls on. */
351−private fun localDayOf(epochMillis: Long): java.time.LocalDate =
352− java.time.Instant.ofEpochMilli(epochMillis)
353− .atZone(java.time.ZoneId.systemDefault())
354− .toLocalDate()
355−
356−/**
357− * A separator is shown above a message when it starts a new calendar day — i.e. the
358− * first timestamped message, or one whose day differs from the previous message's.
359− * Messages without a valid time (time <= 0) never get one.
360− */
361−private fun shouldShowDateSeparator(previous: soy.iko.opencode.core.MessageView?, current: soy.iko.opencode.core.MessageView): Boolean {
362− if (current.time <= 0) return false
363− if (previous == null || previous.time <= 0) return true
364− return localDayOf(previous.time) != localDayOf(current.time)
365−}
366−
367−/** Human-friendly day label: "Today", "Yesterday", or "MMM d" (e.g. "Jul 8"). */
368−private fun dateSeparatorLabel(day: java.time.LocalDate): String {
369− val today = java.time.LocalDate.now()
370− return when (day) {
371− today -> "Today"
372− today.minusDays(1) -> "Yesterday"
373− else -> day.format(dateSeparatorFormat)
374− }
375−}
376−
377−@Composable
378−private fun DateSeparator(epochMillis: Long) {
379− Text(
380− text = dateSeparatorLabel(localDayOf(epochMillis)),
381− style = MaterialTheme.typography.labelSmall,
382− color = MaterialTheme.colorScheme.onSurfaceVariant,
383− textAlign = TextAlign.Center,
384− modifier = Modifier.fillMaxWidth().padding(vertical = 8.dp),
385− )
386−}
387−
388−// Assistant-style bubble with three sequentially-pulsing dots, shown as a stand-in
389−// until the real streaming reply starts. Mirrors MessageBubble's left-aligned,
390−// surfaceVariant look so it reads as an incoming message.
391−@Composable
392−private fun TypingIndicator() {
393− val shape = RoundedCornerShape(topStart = 16.dp, topEnd = 16.dp, bottomStart = 4.dp, bottomEnd = 16.dp)
394− val transition = rememberInfiniteTransition(label = "typing")
395− Row(
396− modifier = Modifier.fillMaxWidth().padding(horizontal = 12.dp, vertical = 4.dp),
397− horizontalArrangement = Arrangement.Start,
398− ) {
399− Row(
400− modifier = Modifier
401− .clip(shape)
402− .background(MaterialTheme.colorScheme.surfaceVariant)
403− .padding(horizontal = 14.dp, vertical = 12.dp),
404− verticalAlignment = Alignment.CenterVertically,
405− horizontalArrangement = Arrangement.spacedBy(6.dp),
406− ) {
407− Text(
408− text = "Thinking",
409− style = MaterialTheme.typography.bodyMedium.copy(fontStyle = FontStyle.Italic),
410− color = MaterialTheme.colorScheme.onSurfaceVariant,
411− )
412− Row(horizontalArrangement = Arrangement.spacedBy(4.dp)) {
413− // Stagger the three dots so they fade in and out in sequence.
414− listOf(0, 150, 300).forEach { staggerMs ->
415− val alpha by transition.animateFloat(
416− initialValue = 0.2f,
417− targetValue = 1f,
418− animationSpec = infiniteRepeatable(
419− animation = tween(durationMillis = 600, delayMillis = staggerMs),
420− repeatMode = RepeatMode.Reverse,
421− ),
422− label = "dot",
423− )
424− Box(
425− modifier = Modifier
426− .size(6.dp)
427− .clip(CircleShape)
428− .background(MaterialTheme.colorScheme.onSurfaceVariant.copy(alpha = alpha)),
429− )
430− }
431− }
432− }
433− }
434−}
435−
436@Composable
230@Composable
437private fun ReconnectingBanner() {
231private fun ReconnectingBanner() {
438 Surface(
232 Surface(
@@ -497,56 +291,19 @@ private fun ChatInputBar(
497 onStop: () -> Unit,
291 onStop: () -> Unit,
498) {
292) {
499 val haptic = LocalHapticFeedback.current
293 val haptic = LocalHapticFeedback.current
500− val clipboardManager = LocalClipboardManager.current
501 Surface(tonalElevation = 3.dp) {
294 Surface(tonalElevation = 3.dp) {
502− Column(
503− modifier = Modifier.fillMaxWidth().imePadding().padding(horizontal = 12.dp, vertical = 8.dp),
504− ) {
505 Row(
295 Row(
506− modifier = Modifier.fillMaxWidth(),
296+ modifier = Modifier.fillMaxWidth().imePadding().padding(horizontal = 12.dp, vertical = 8.dp),
507 verticalAlignment = Alignment.Bottom,
297 verticalAlignment = Alignment.Bottom,
508 horizontalArrangement = Arrangement.spacedBy(8.dp),
298 horizontalArrangement = Arrangement.spacedBy(8.dp),
509 ) {
299 ) {
510 OutlinedTextField(
300 OutlinedTextField(
511 value = draft,
301 value = draft,
512 onValueChange = onDraftChange,
302 onValueChange = onDraftChange,
513− modifier = Modifier.weight(1f).heightIn(min = 48.dp),
303+ modifier = Modifier.weight(1f),
514 enabled = !loading,
304 enabled = !loading,
515 placeholder = { Text("Message opencode…") },
305 placeholder = { Text("Message opencode…") },
516 maxLines = 5,
306 maxLines = 5,
517− // Offer a quick way to wipe the draft once there's something to clear
518− // and we're not mid-generation (where the field is a poor place to fuss).
519− trailingIcon = if (draft.isNotEmpty() && !generating) {
520− {
521− IconButton(
522− onClick = { onDraftChange("") },
523− modifier = Modifier.size(20.dp),
524− ) {
525− Icon(
526− Icons.Filled.Close,
527− contentDescription = "Clear draft",
528− modifier = Modifier.size(16.dp),
529− )
530− }
531− }
532− } else {
533− null
534− },
535− // Surface a running count once the draft gets long, so the user has a
536− // sense of scale before sending a wall of text.
537− supportingText = if (draft.length > 500) {
538− {
539− Text(
540− text = "${draft.length} characters",
541− style = MaterialTheme.typography.labelSmall,
542− color = MaterialTheme.colorScheme.onSurfaceVariant,
543− textAlign = TextAlign.End,
544− modifier = Modifier.fillMaxWidth(),
545− )
546− }
547− } else {
548− null
549− },
550 keyboardOptions = KeyboardOptions(imeAction = ImeAction.Send),
307 keyboardOptions = KeyboardOptions(imeAction = ImeAction.Send),
551 keyboardActions = KeyboardActions(onSend = {
308 keyboardActions = KeyboardActions(onSend = {
552 if (!generating && !loading && draft.isNotBlank()) onSend()
309 if (!generating && !loading && draft.isNotBlank()) onSend()
@@ -574,17 +331,5 @@ private fun ChatInputBar(
574 }
331 }
575 }
332 }
576 }
333 }
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− }
589 }
334 }
590}
335}
app/src/main/java/soy/iko/opencode/ui/screens/ConnectScreen.kt+2−49
@@ -101,30 +101,6 @@ fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
101 enabled = !state.loading,
101 enabled = !state.loading,
102 )
102 )
103
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−
128 if (state.authRequired) {
104 if (state.authRequired) {
129 Spacer(Modifier.height(12.dp))
105 Spacer(Modifier.height(12.dp))
130 OutlinedTextField(
106 OutlinedTextField(
@@ -155,15 +131,7 @@ fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
155 },
131 },
156 modifier = Modifier.fillMaxWidth(),
132 modifier = Modifier.fillMaxWidth(),
157 keyboardOptions = KeyboardOptions(keyboardType = KeyboardType.Password, imeAction = ImeAction.Go),
133 keyboardOptions = KeyboardOptions(keyboardType = KeyboardType.Password, imeAction = ImeAction.Go),
158− keyboardActions = KeyboardActions(onGo = {
134+ keyboardActions = KeyboardActions(onGo = { dispatch(Event.Connect) }),
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− }),
167 enabled = !state.loading,
135 enabled = !state.loading,
168 )
136 )
169 Spacer(Modifier.height(4.dp))
137 Spacer(Modifier.height(4.dp))
@@ -178,10 +146,7 @@ fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
178
146
179 Button(
147 Button(
180 onClick = { dispatch(Event.Connect) },
148 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.
183− enabled = !state.loading && state.serverUrl.isNotBlank() &&
149+ enabled = !state.loading && state.serverUrl.isNotBlank(),
184− (!state.authRequired || state.username.isNotBlank() || state.password.isNotBlank()),
185 modifier = Modifier.fillMaxWidth(),
150 modifier = Modifier.fillMaxWidth(),
186 ) {
151 ) {
187 if (state.loading) {
152 if (state.loading) {
@@ -195,18 +160,6 @@ fun ConnectScreen(state: UiState, dispatch: (Event) -> Unit) {
195 }
160 }
196 }
161 }
197
162
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
163 // Surface the error inline as well, so it stays visible while the user
211 // corrects the form (the snackbar is transient).
164 // corrects the form (the snackbar is transient).
212 if (state.error != null && !state.loading) {
165 if (state.error != null && !state.loading) {
app/src/main/java/soy/iko/opencode/ui/screens/SessionsScreen.kt+14−84
@@ -1,13 +1,7 @@
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
8import androidx.compose.foundation.background
3import androidx.compose.foundation.background
9import androidx.compose.foundation.clickable
4import androidx.compose.foundation.clickable
10−import androidx.compose.foundation.combinedClickable
11import androidx.compose.foundation.layout.Arrangement
5import androidx.compose.foundation.layout.Arrangement
12import androidx.compose.foundation.layout.Box
6import androidx.compose.foundation.layout.Box
13import androidx.compose.foundation.layout.Column
7import androidx.compose.foundation.layout.Column
@@ -21,8 +15,6 @@ import androidx.compose.foundation.layout.padding
21import androidx.compose.foundation.layout.size
15import androidx.compose.foundation.layout.size
22import androidx.compose.foundation.lazy.LazyColumn
16import androidx.compose.foundation.lazy.LazyColumn
23import androidx.compose.foundation.lazy.items
17import androidx.compose.foundation.lazy.items
24−import androidx.compose.foundation.lazy.rememberLazyListState
25−import androidx.compose.foundation.shape.CircleShape
26import androidx.compose.material.icons.Icons
18import androidx.compose.material.icons.Icons
27import androidx.compose.material.icons.automirrored.filled.ArrowBack
19import androidx.compose.material.icons.automirrored.filled.ArrowBack
28import androidx.compose.material.icons.filled.Add
20import androidx.compose.material.icons.filled.Add
@@ -58,12 +50,8 @@ import androidx.compose.runtime.remember
58import androidx.compose.runtime.setValue
50import androidx.compose.runtime.setValue
59import androidx.compose.ui.Alignment
51import androidx.compose.ui.Alignment
60import androidx.compose.ui.Modifier
52import androidx.compose.ui.Modifier
61−import androidx.compose.ui.draw.clip
62−import androidx.compose.ui.graphics.Color
63import androidx.compose.ui.hapticfeedback.HapticFeedbackType
53import androidx.compose.ui.hapticfeedback.HapticFeedbackType
64−import androidx.compose.ui.platform.LocalContext
65import androidx.compose.ui.platform.LocalHapticFeedback
54import androidx.compose.ui.platform.LocalHapticFeedback
66−import androidx.compose.ui.text.style.TextAlign
67import androidx.compose.ui.text.style.TextOverflow
55import androidx.compose.ui.text.style.TextOverflow
68import androidx.compose.ui.unit.dp
56import androidx.compose.ui.unit.dp
69import soy.iko.opencode.core.Event
57import soy.iko.opencode.core.Event
@@ -77,15 +65,6 @@ fun SessionsScreen(state: UiState, dispatch: (Event) -> Unit) {
77 var pendingDeleteId by remember { mutableStateOf<String?>(null) }
65 var pendingDeleteId by remember { mutableStateOf<String?>(null) }
78 var isPullRefreshing by remember { mutableStateOf(false) }
66 var isPullRefreshing by remember { mutableStateOf(false) }
79 var searchQuery by remember { mutableStateOf("") }
67 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
68
90 // The pull-to-refresh spinner should only reflect a user-initiated pull, not
69 // The pull-to-refresh spinner should only reflect a user-initiated pull, not
91 // every loading operation. Clear it once the underlying load completes.
70 // every loading operation. Clear it once the underlying load completes.
@@ -119,23 +98,7 @@ fun SessionsScreen(state: UiState, dispatch: (Event) -> Unit) {
119 Scaffold(
98 Scaffold(
120 topBar = {
99 topBar = {
121 TopAppBar(
100 TopAppBar(
122− title = {
101+ title = { Text("Sessions") },
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− },
139 navigationIcon = {
102 navigationIcon = {
140 IconButton(onClick = { dispatch(Event.NavigateToConnect) }) {
103 IconButton(onClick = { dispatch(Event.NavigateToConnect) }) {
141 Icon(Icons.AutoMirrored.Filled.ArrowBack, contentDescription = "Disconnect")
104 Icon(Icons.AutoMirrored.Filled.ArrowBack, contentDescription = "Disconnect")
@@ -149,15 +112,11 @@ fun SessionsScreen(state: UiState, dispatch: (Event) -> Unit) {
149 )
112 )
150 },
113 },
151 floatingActionButton = {
114 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()) {
155− ExtendedFloatingActionButton(
115+ ExtendedFloatingActionButton(
156− onClick = { dispatch(Event.CreateSession) },
116+ onClick = { dispatch(Event.CreateSession) },
157− icon = { Icon(Icons.Filled.Add, contentDescription = null) },
117+ icon = { Icon(Icons.Filled.Add, contentDescription = null) },
158− text = { Text("New session") },
118+ text = { Text("New session") },
159− )
119+ )
160− }
161 },
120 },
162 ) { padding ->
121 ) { padding ->
163 Column(Modifier.fillMaxSize().padding(padding)) {
122 Column(Modifier.fillMaxSize().padding(padding)) {
@@ -199,7 +158,6 @@ fun SessionsScreen(state: UiState, dispatch: (Event) -> Unit) {
199 state = rememberPullToRefreshState(),
158 state = rememberPullToRefreshState(),
200 ) {
159 ) {
201 LazyColumn(
160 LazyColumn(
202− state = sessionsListState,
203 modifier = Modifier.fillMaxSize(),
161 modifier = Modifier.fillMaxSize(),
204 contentPadding = PaddingValues(bottom = 88.dp),
162 contentPadding = PaddingValues(bottom = 88.dp),
205 ) {
163 ) {
@@ -238,21 +196,13 @@ fun SessionsScreen(state: UiState, dispatch: (Event) -> Unit) {
238 }
196 }
239}
197}
240
198
241−@OptIn(ExperimentalFoundationApi::class)
242@Composable
199@Composable
243private fun SessionRow(session: SessionView, onOpen: () -> Unit, onDelete: () -> Unit) {
200private fun SessionRow(session: SessionView, onOpen: () -> Unit, onDelete: () -> Unit) {
244− val context = LocalContext.current
245 Row(
201 Row(
246 modifier = Modifier
202 modifier = Modifier
247 .fillMaxWidth()
203 .fillMaxWidth()
248 .background(MaterialTheme.colorScheme.surface)
204 .background(MaterialTheme.colorScheme.surface)
249− .combinedClickable(
250− onClick = onOpen,
205+ .clickable(onClick = onOpen)
251− onLongClick = {
252− copyToClipboard(context, session.id)
253− Toast.makeText(context, "Session ID copied", Toast.LENGTH_SHORT).show()
254− },
255− )
256 .padding(start = 20.dp, end = 8.dp, top = 14.dp, bottom = 14.dp),
206 .padding(start = 20.dp, end = 8.dp, top = 14.dp, bottom = 14.dp),
257 verticalAlignment = Alignment.CenterVertically,
207 verticalAlignment = Alignment.CenterVertically,
258 ) {
208 ) {
@@ -287,13 +237,13 @@ private fun SessionRow(session: SessionView, onOpen: () -> Unit, onDelete: () ->
287 Icon(
237 Icon(
288 Icons.Filled.DeleteOutline,
238 Icons.Filled.DeleteOutline,
289 contentDescription = "Delete session",
239 contentDescription = "Delete session",
290− tint = MaterialTheme.colorScheme.error,
240+ tint = MaterialTheme.colorScheme.onSurfaceVariant,
291 )
241 )
292 }
242 }
293 }
243 }
294}
244}
295
245
296−/** Red background with a trailing delete icon and label, revealed while swiping a session row away. */
246+/** Red background with a trailing delete icon, revealed while swiping a session row away. */
297@Composable
247@Composable
298private fun SwipeDeleteBackground() {
248private fun SwipeDeleteBackground() {
299 Box(
249 Box(
@@ -303,19 +253,11 @@ private fun SwipeDeleteBackground() {
303 .padding(horizontal = 24.dp),
253 .padding(horizontal = 24.dp),
304 contentAlignment = Alignment.CenterEnd,
254 contentAlignment = Alignment.CenterEnd,
305 ) {
255 ) {
306− Row(verticalAlignment = Alignment.CenterVertically) {
307− Icon(
256+ Icon(
308− Icons.Filled.DeleteOutline,
257+ Icons.Filled.DeleteOutline,
309− contentDescription = null,
258+ contentDescription = null,
310− tint = MaterialTheme.colorScheme.onError,
259+ 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− )
260+ )
318− }
319 }
261 }
320}
262}
321
263
@@ -345,11 +287,6 @@ private fun ReconnectingBanner() {
345 }
287 }
346}
288}
347
289
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−
353/** Shorten a session ID for display: "abc12345…" instead of the full UUID. */
290/** Shorten a session ID for display: "abc12345…" instead of the full UUID. */
354private fun shortSessionId(id: String): String =
291private fun shortSessionId(id: String): String =
355 if (id.length > 12) id.take(8) + "…" else id
292 if (id.length > 12) id.take(8) + "…" else id
@@ -391,12 +328,5 @@ private fun EmptySessions() {
391 style = MaterialTheme.typography.bodyMedium,
328 style = MaterialTheme.typography.bodyMedium,
392 color = MaterialTheme.colorScheme.onSurfaceVariant,
329 color = MaterialTheme.colorScheme.onSurfaceVariant,
393 )
330 )
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− )
401 }
331 }
402}
332}
app/src/main/java/soy/iko/opencode/ui/theme/Theme.kt−22
@@ -14,18 +14,7 @@ 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),
19 secondary = EmeraldDark,
17 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),
29 background = DarkBackground,
18 background = DarkBackground,
30 surface = DarkSurface,
19 surface = DarkSurface,
31 surfaceVariant = DarkSurfaceVariant,
20 surfaceVariant = DarkSurfaceVariant,
@@ -35,18 +24,7 @@ private val DarkColors = darkColorScheme(
35
24
36private val LightColors = lightColorScheme(
25private val LightColors = lightColorScheme(
37 primary = EmeraldDark,
26 primary = EmeraldDark,
38− primaryContainer = Color(0xFFD7F5E3),
39− onPrimaryContainer = Color(0xFF0A3D22),
40 secondary = Emerald,
27 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),
50 background = LightBackground,
28 background = LightBackground,
51 surface = LightSurface,
29 surface = LightSurface,
52 surfaceVariant = LightSurfaceVariant,
30 surfaceVariant = LightSurfaceVariant,
shared/src/app.rs−3
@@ -352,9 +352,6 @@ impl App for OpencodeApp {
352 title: display_title(&s.title),
352 title: display_title(&s.title),
353 })
353 })
354 .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());
358 }
355 }
359 Err(e) => model.error = Some(e),
356 Err(e) => model.error = Some(e),
360 }
357 }