List hub / Drawer sid re-login #62

New issue

Closed

opened 2026-08-04 19:40:37 +00:00 by TheAnachronism · 1 comment

TheAnachronism commented

2026-08-04 19:40:37 +00:00

Owner

Parent

Implements musts from docs/1.0-beta-readiness.md (polish Blocker seed #3 + architecture sid-retry must from 1.0 beta release readiness).

What to build

When the session id is dead, List hub headline loads and Drawer tree refresh recover the same way marks/Reader already do: transparent re-login + single retry via AccountRepository.withActiveClient (or the one shared equivalent). Ship a regression test at the best seam (prefer unit/controller).

Acceptance criteria

List hub browse recovers from session-expired / dead sid with one transparent re-login retry
Drawer tree refresh recovers the same way
Both paths use withActiveClient (or the one shared equivalent) — not bare activeApiClient without retry
Regression test fails before the fix and passes after
Marks/Reader session recovery behaviour remains intact

Blocked by

None — can start immediately.

## Parent Implements musts from [`docs/1.0-beta-readiness.md`](../src/branch/master/docs/1.0-beta-readiness.md) (polish Blocker seed #3 + architecture sid-retry must from [1.0 beta release readiness](https://git.anachronis.dev/TheAnachronism/tt-rss-viewer/issues/41)). ## What to build When the session id is dead, **List hub** headline loads and **Drawer tree** refresh recover the same way marks/Reader already do: transparent re-login + single retry via `AccountRepository.withActiveClient` (or the one shared equivalent). Ship a regression test at the best seam (prefer unit/controller). ## Acceptance criteria - [ ] List hub browse recovers from session-expired / dead sid with one transparent re-login retry - [ ] Drawer tree refresh recovers the same way - [ ] Both paths use `withActiveClient` (or the one shared equivalent) — not bare `activeApiClient` without retry - [ ] Regression test fails before the fix and passes after - [ ] Marks/Reader session recovery behaviour remains intact ## Blocked by - None — can start immediately.

TheAnachronism added the

ready-for-agent

label

2026-08-04 19:40:52 +00:00

TheAnachronism self-assigned this

2026-08-04 19:43:48 +00:00

TheAnachronism commented

2026-08-04 19:48:42 +00:00

Author

Owner

Done on master (d0171e9).

AccountHeadlinesApi.getHeadlines and DrawerNotifier.refresh now go through withActiveClient (same as Marks/Reader). Regressions: test/list/account_headlines_api_session_retry_test.dart, test/drawer/drawer_refresh_session_retry_test.dart.

Done on `master` (`d0171e9`). `AccountHeadlinesApi.getHeadlines` and `DrawerNotifier.refresh` now go through `withActiveClient` (same as Marks/Reader). Regressions: `test/list/account_headlines_api_session_retry_test.dart`, `test/drawer/drawer_refresh_session_retry_test.dart`.

TheAnachronism closed this issue

2026-08-04 19:48:42 +00:00

TheAnachronism referenced this issue from a commit

2026-08-04 19:54:27 +00:00

Fix Fresh Unread patches and List/Drawer sid re-login (#61, #62).

TheAnachronism referenced this issue from a commit

2026-08-04 21:14:22 +00:00

Fix Fresh Unread patches and List/Drawer sid re-login (#61, #62).

TheAnachronism removed their assignment

2026-08-06 14:35:35 +00:00

TheAnachronism self-assigned this

2026-08-06 14:35:47 +00:00