Skip to content

fix: integrate reader navigation, shared panels and PDF import - #15

Merged
Eoic merged 3 commits into
developmentfrom
fix/reader-integration
Oct 10, 2026
Merged

Eoic merged 3 commits into
developmentfrom
fix/reader-integration

Conversation

@Eoic

@Eoic Eoic commented Oct 10, 2026

Copy link
Copy Markdown
Member

Summary

Integrate the complete reader workstream: accurate EPUB progress and covers, uninterrupted navigation, overlay controls, client-owned reader sheets, browser PDF import, and removal of pre-release compatibility shims. Record the current-contract policy in CODING_STYLE.md.

Update the client, reader and server submodules to the linked commits. Also capture the existing docs and website development revisions in the workspace; their component changes are already pushed.

Component PRs and merge order

  1. Reader: fix: improve reader progress, navigation and host-owned panels reader#6
  2. Server: refactor: remove legacy library payload promotion server#14
  3. Client: fix: integrate reader sheets, PDF imports and current data contracts client#46 (pins the reader commit)
  4. Merge this coordinating workspace PR after the component PRs.

Use merges that preserve the recorded component commits. No versions, releases or deployments are changed.

Validation

  • Workspace committed-reference check passed: client manifest/lock and reader gitlink agree.
  • Workspace tooling: 18 tests and ShellCheck passed.
  • Reader: 170 tests, analysis, example checks, packaged worker check and browser smoke passed.
  • Client: 1,620 tests passed with 19 skipped; 45 final focused regressions and 23 tests against the pushed reader dependency passed. Analysis, formatting, bootstrap checks and production web build passed.
  • Server: 13 focused library validation/sync tests plus Ruff and Mypy passed.
  • Real-book sessions ran in desktop Chrome and the actual client on an iOS simulator. Component reports describe evidence, reproduction and untested targets; live authenticated synchronization and physical-device certification were not performed.

All local source changes are committed. Ignored local overrides, private books, credentials and build artifacts remain untracked by Git. Remote CI status is reported separately by the linked PR checks.

Copilot AI balanced review requested due to automatic review settings October 10, 2026 18:25

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-10T18:27:33.787691Z 784753b PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@Eoic
Eoic merged commit b3936c5 into development Oct 10, 2026
4 checks passed
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.

2 participants