Overhaul Compose UI with sidebar navigation and component architecture by linroid · Pull Request #48 · linroid/Ketch

linroid · 2026-02-12T05:25:37Z

Summary

Restructure monolithic 1,348-line App.kt into 25+ modular components with proper separation of concerns
Add left sidebar navigation with status filters (All, Downloading, Paused, Completed, Failed) and live task count badges
Introduce custom Material 3 dark/light theme with brand color palette and state-specific colors
Add collapsible toggle icons (speed limit, priority, schedule, task info) with animated expandable panels
Add bottom status bar showing backend info, connection state, and real-time download speed
Extract centralized AppState class for UI state management
Add 65 unit tests for format utilities and status filter matching logic

Test plan

Verify sidebar navigation filters tasks correctly by status
Verify collapsible panels expand/collapse properly (only one at a time)
Verify dark and light theme switching works
Verify status bar shows correct backend info and speeds
Run unit tests: ./gradlew :app:shared:allTests
Test on Desktop, Android, and Web platforms

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Restructure the monolithic 1,348-line App.kt into a modular component architecture with proper separation of concerns: - Add Motrix-style left sidebar navigation with status filters and task counts - Introduce custom Material 3 dark/light theme with brand color palette - Extract centralized AppState for UI state management - Split download list items into composable components (progress, actions, toggles) - Add collapsible toggle icons for speed limit, priority, schedule, and task info - Add bottom status bar showing backend info and download speed - Add comprehensive tests for format utilities and status filter logic Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 401b5ea3d1

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

chatgpt-codex-connector · 2026-02-12T05:29:04Z

app/shared/src/commonMain/kotlin/com/linroid/kdown/app/ui/AppShell.kt

+  val filteredTasks by remember(
+    sortedTasks, appState.statusFilter
+  ) {
+    derivedStateOf {


Recompute filtered task data on state transitions

filteredTasks is wrapped in remember { derivedStateOf { ... } } but the predicate reads taskStates, which is a plain MutableMap and not Compose snapshot state. When a task changes from e.g. Downloading to Completed (without being added/removed), this derived value is not invalidated, so the filtered list (and similarly the count/summary derived blocks below) can stay stale until some unrelated key changes like switching filters.

Useful? React with 👍 / 👎.

@codex fix it

Summary

Fixed AppShell to rebuild taskStates from current sortedTasks each composition instead of mutating a long-lived state map, which naturally prunes removed task IDs and prevents stale deleted-task state from lingering. app/shared/src/commonMain/kotlin/com/linroid/kdown/app/ui/AppShell.ktL63-L68

Updated filteredTasks to use remember(sortedTasks, taskStates, appState.statusFilter) so filtering correctly recomputes when task state transitions happen without list membership changes. app/shared/src/commonMain/kotlin/com/linroid/kdown/app/ui/AppShell.ktL70-L82

Updated derived sidebar/action summaries (taskCounts, hasActive, hasPaused) to depend on current taskStates, keeping counts and batch-action visibility in sync with live state changes. app/shared/src/commonMain/kotlin/com/linroid/kdown/app/ui/AppShell.ktL84-L103

Testing

✅ curl -I https://developer.android.com/jetpack/compose/state | head

❌ ./gradlew :app:shared:allTests (failed: Gradle could not resolve org.gradle.toolchains.foojay-resolver-convention:1.0.0 from configured plugin repositories)

Committed on branch work with commit 75019fb, and I also created the follow-up PR via the make_pr tool.

View task →

chatgpt-codex-connector · 2026-02-12T05:29:04Z

app/shared/src/commonMain/kotlin/com/linroid/kdown/app/ui/AppShell.kt

+  sortedTasks.forEach { task ->
+    val state by task.state.collectAsState()
+    taskStates[task.taskId] = state


Prune removed tasks from the cached state map

taskStates only ever inserts/updates entries and never removes IDs that disappeared from sortedTasks. After a task is deleted, its last state remains in the map, so sidebar counts, batch-action visibility, and speed/active aggregates can include tasks that no longer exist.

Useful? React with 👍 / 👎.

@codex fix it

Summary

Updated AppShell to stop mutating a long-lived cached taskStates map during composition; it now rebuilds a fresh taskId -> DownloadState map from current sortedTasks, which naturally drops removed tasks and avoids stale deleted entries. app/shared/src/commonMain/kotlin/com/linroid/kdown/app/ui/AppShell.ktL63-L67

Updated filteredTasks derivation to be remembered with explicit dependencies (sortedTasks, taskStates, and statusFilter) so filter results recompute correctly when task states transition without list membership changes. app/shared/src/commonMain/kotlin/com/linroid/kdown/app/ui/AppShell.ktL69-L85

Updated task-count and batch-action summary derivations (taskCounts, hasActive, hasPaused) to key off current taskStates, ensuring sidebar counts and action visibility stay in sync with live state transitions. app/shared/src/commonMain/kotlin/com/linroid/kdown/app/ui/AppShell.ktL87-L106

Testing

⚠️ ./gradlew :app:shared:allTests (failed in this environment because Gradle could not resolve org.gradle.toolchains.foojay-resolver-convention:1.0.0 from configured plugin repositories).

✅ curl -I https://plugins.gradle.org/m2/ (web check executed successfully).

View task →

github-actions · 2026-02-12T05:32:42Z

Test Results

978 tests +222 978 ✅ +222 12s ⏱️ ±0s
69 suites + 6 0 💤 ± 0
69 files + 6 0 ❌ ± 0

Results for commit fd0770e. ± Comparison against base commit eebc6a9.

♻️ This comment has been updated with latest results.

- Include Idle state in Downloading filter so sub-counts sum to All - Use mutableStateMapOf for reactive task state tracking and clean stale entries - Use Card onClick instead of Modifier.clickable for proper ripple clipping - Reorder sidebar filters: All, Downloading, Completed, Paused, Failed - Add copy-to-clipboard button on URL row in task info panel Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Calculate bytesPerSecond in DownloadCoordinator.buildContext using elapsed time delta (sampled every 500ms) and pass it to DownloadProgress. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Replace four independent AnimatedVisibility blocks with a single AnimatedContent keyed on the expanded panel enum, so panel transitions crossfade smoothly without intermediate height changes. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

linroid and others added 4 commits February 12, 2026 11:55

Replace JVM with Desktop and WebAssembly with Web in README

6f4d177

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Add Web (WasmJs) badge to README

ba271b5

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Add Apache-2.0 LICENSE file

5579297

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

chatgpt-codex-connector bot reviewed Feb 12, 2026

View reviewed changes

linroid and others added 3 commits February 12, 2026 13:32

Fix download speed always showing 0 B/s

ae7e2f6

Calculate bytesPerSecond in DownloadCoordinator.buildContext using elapsed time delta (sampled every 500ms) and pass it to DownloadProgress. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

linroid merged commit fd8ba1c into main Feb 12, 2026

linroid deleted the ui-overhaul branch February 12, 2026 08:52

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Overhaul Compose UI with sidebar navigation and component architecture#48

Overhaul Compose UI with sidebar navigation and component architecture#48
linroid merged 7 commits intomainfrom
ui-overhaul

linroid commented Feb 12, 2026 •

edited

Loading

Uh oh!

chatgpt-codex-connector bot left a comment

Uh oh!

chatgpt-codex-connector bot Feb 12, 2026

Uh oh!

linroid Feb 12, 2026

Uh oh!

chatgpt-codex-connector bot Feb 12, 2026

Uh oh!

chatgpt-codex-connector bot Feb 12, 2026

Uh oh!

linroid Feb 12, 2026

Uh oh!

chatgpt-codex-connector bot Feb 12, 2026

Uh oh!

github-actions bot commented Feb 12, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

linroid commented Feb 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Test plan

Uh oh!

chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

chatgpt-codex-connector bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

linroid Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

chatgpt-codex-connector bot Feb 12, 2026

Choose a reason for hiding this comment

Summary

Uh oh!

chatgpt-codex-connector bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

linroid Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

chatgpt-codex-connector bot Feb 12, 2026

Choose a reason for hiding this comment

Summary

Uh oh!

github-actions bot commented Feb 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Test Results

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

linroid commented Feb 12, 2026 •

edited

Loading

github-actions bot commented Feb 12, 2026 •

edited

Loading