Repository navigation
Conversation
This was referenced Oct 23, 2026
aaron-seq
added this to the
Reading workspace reliability milestone
ChatGPT Codex Connector
added 5 commits
November 2, 2026 11:13
Keep the request deadline active through success and error body reads. Preserve timeout classification without retrying, and leave parsing failures outside the transient network retry policy. Fixes #51
Stop scheduling summary chunks after a fatal section error while already-started siblings settle. Preserve the original rejection and cover deferred sibling completion through the mapper and worker. Fixes #49
Check for an existing cleanup alarm before creating one so ordinary worker wakes do not restart its daily delay. Keep listener registration synchronous and contain alarm read/create failures for a later retry. Cover restart timing, absent alarms, blocked reads, rejection recovery, and scheduling despite unavailable settings. Fixes #50
Read the original DOM with cached computed styles and exclude hidden or unwanted content without changing the source page. Keep block boundaries and inline words intact. Add unit and real Chromium regressions for hidden text, adjacent blocks, inline content, and an unchanged source DOM. Fixes #52
added 5 commits
November 2, 2026 11:16
Route options updates and confirmed resets through the worker so every UI shares one preferences writer. Queue initialization, updates, and replacement writes without allowing a failure to block later edits. Cover overlapping top-level and nested edits, initialization/reset ordering, read/write recovery, and real two-context browser persistence. Inject failed save reads in the worker while preserving page-load errors. Fixes #46
Keep a stale popup from replacing the sibling language saved in Settings. Preserve the intentional two-field swap, with unit payload and native storage browser regressions. Refs #46
Read only the first direct summary of collapsed details and reject selected article containers hidden by a collapsed ancestor. Keep content inside a visible first summary readable without changing the source DOM. Preserve blank lines around rendered paragraphs so summary chunking keeps whole paragraphs. Add red-first unit and Chromium details regressions and an actual extraction-to-chunkContent browser check. Refs #52
Keep the product checkpoint concise and link the requirement-to-regression evidence, validation limits, milestone, and tooling follow-up. Document worker-owned settings writes and the explicit reset message.
aaron-seq
changed the title
Preserve the contributor commit for #49 and its scheduling documentation. Resolve the equivalent stop-flag implementation using the tested version, and consolidate duplicate regressions into the existing deferred mapper and worker cases.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
The reading UI crowded source context, controls and results. This keeps the source beside the tools, adds restrained light/dark themes and local SVG icons, and restores keyboard access to tabs and summary-length controls.
The seven reliability fixes preserve overlapping settings edits, isolate document/chat responses, stop queued summary work after failure, preserve cleanup alarms, bound response-body reads, and extract visible text with paragraph boundaries. Review caught and corrected stale popup language payloads, independent summary restoration and collapsed disclosures.
Community PR #53 addresses the same #49 failure. Its commit
4d31f67is incorporated with contributor history; equivalent tests are consolidated into the retained deferred mapper/worker regressions.Closes #36. Closes #46. Closes #47. Closes #48. Closes #49. Closes #50. Closes #51. Closes #52.
Validation
Head:
eb721b8; basebd0e0f7includes merged #42–#44. Primary-session local checks used source9c3db4b; later documentation/consolidation commits leave runtime, tests and configuration unchanged.npm run verify: lint/typecheck and 361 tests pass.npm run test:coverage: 95.01% statements / 85.16% branches / 95.60% functions / 97.02% lines; floors retained.npm run test:e2e: 33 unpacked-Chromium tests pass, with intercepted providers, native two-context storage and delayed-response races.Current-head CI covers Node 20/22/24, Chromium, ancestry and packaging; prior same-source CI passed. Final artifact status/inspection is linked from the bug board. Optional Codecov authentication still fails (log, #45); coverage gates pass.
Requirements, regressions and validation limits.
Risks and rollback
No runtime dependency, permission, schema, provider/model default or deployed service changes. Already-started sibling requests may finish after a section failure. Alarm timing is simulated. No live-provider, native toolbar/OS, screen-reader, minimum-browser or real-book performance claim. PDF/OCR and semantic retrieval remain roadmap work.
This remains a draft for review. Revert and reload to roll back; no data migration.
Screenshots
Reading summary · Dark Chat · Previous interface