Skip to content

fix: deterministic session reload for sync/open + android CI lint stability#8

Merged
ayagmar merged 1 commit into
masterfrom
fix/session-sync-reload-ux
Mar 23, 2026
Merged

fix: deterministic session reload for sync/open + android CI lint stability#8
ayagmar merged 1 commit into
masterfrom
fix/session-sync-reload-ux

Conversation

@ayagmar

@ayagmar ayagmar commented Mar 23, 2026

Copy link
Copy Markdown
Owner

Summary

  • allow opening the currently active session from Sessions (button stays enabled)
  • add deterministic runtime reload API in SessionController (reloadActiveSessionFromDisk)
  • force reload via switch_session(sessionFile) before manual and auto freshness sync loads
  • keep coherency warning visible and surface error when reload fails
  • adjust resume/open status message wording in SessionsViewModel
  • add test coverage for reload call counts and reload failure behavior
  • disable unstable GradleDependency lint check in app module to keep CI deterministic

Validation

  • ./gradlew ktlintCheck detekt test :app:check
  • (cd bridge && pnpm run check)

@github-actions

Copy link
Copy Markdown

Bridge Coverage (Vitest)

Metric Covered / Total Coverage
Lines 1741 / 2267 76.79%
Functions 115 / 124 92.74%
Branches 416 / 579 71.84%
Statements 1741 / 2267 76.79%

Source: bridge/coverage/coverage-summary.json

@github-actions

github-actions Bot commented Mar 23, 2026

Copy link
Copy Markdown

Kotlin Coverage

Overall Project 18.44% -0.36%
Files changed 29.9%

File Coverage
SessionController.kt 95.08% 🍏
ChatViewModel.kt 51.36% -0.39%
RpcSessionController.kt 24.85% -2.28%
SessionsViewModel.kt 2.77% -1.71%
PiTopBar.kt 0% -19.79%
SessionsScreen.kt 0% -0.59%

- allow reopening active sessions from the Sessions list

- force switch_session(sessionFile) before manual/auto sync timeline reloads

- keep coherency warning visible when reload fails

- add tests for reload call paths and failure handling

- disable unstable GradleDependency lint gate to keep CI green
@ayagmar
ayagmar force-pushed the fix/session-sync-reload-ux branch from e66110c to 7a3f40d Compare March 23, 2026 02:40
@ayagmar
ayagmar merged commit 8898c17 into master Mar 23, 2026
2 checks passed
@ayagmar ayagmar mentioned this pull request Mar 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant