Skip to content

refactor: extract OPDS screen orchestration hook - #97

Merged
jdluu merged 1 commit into
mainfrom
refactor/opds-screen-orchestration
Aug 29, 2026
Merged

refactor: extract OPDS screen orchestration hook#97
jdluu merged 1 commit into
mainfrom
refactor/opds-screen-orchestration

Conversation

@jdluu

@jdluu jdluu commented Aug 29, 2026

Copy link
Copy Markdown
Owner

Summary

  • Extracts modal selection and saved-catalog refresh state into a dedicated hook
  • Moves disconnect and best-effort save orchestration behind a focused interface
  • Reduces OpdsCatalogScreenContainer to composition and data wiring
  • Adds focused hook tests

Scope

Closes #96. Refactor-only and within the active feature freeze. Existing IPC, data flow, callbacks, and rendered behavior are preserved.

Validation

  • Focused OPDS tests: 5 hook tests passed; OPDS suite 171 passed
  • Full frontend suite: 290 passed
  • npx tsc -b: passed
  • Direct Biome check: passed
  • pnpm build: passed
  • git diff --check: passed

Android runtime testing was not part of this code slice. No adb commands were run for this PR.

Move modal selection, saved-catalog refresh, and cross-feature callbacks out of the screen container while preserving existing data flow.\n\nCloses #96
@jdluu
jdluu merged commit 80ee97e into main Aug 29, 2026
3 checks passed
@jdluu
jdluu deleted the refactor/opds-screen-orchestration branch August 29, 2026 20:46
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.

refactor: extract OPDS screen orchestration hook

1 participant