Repository navigation
Reader 0.2: repair release blockers, consolidate ready issues and roadmap - #276
Open
andy-zen-dev wants to merge 5 commits into
Open
andy-zen-dev wants to merge 5 commits into
andy-zen-dev wants to merge 5 commits into
Conversation
Refs #273, #267, #269, #271. Integrate the exact reviewed implementations from #268 (50fbb9b), #270 (752f3c4) and #272 (610dc80) into one master-based delivery. Preserve the canonical public API and all historical artifacts. Add installable analyze/diagnostic/site commands, source-free CSV, reader labels, numeric fidelity, bounded input and private CLI errors. Keep the Pages deployment interlock and scientific acceptance gates unchanged. Add focused regressions and full-repository installed-package CI. No same-run merge acceptance; old PRs remain until this integration passes.
Refs #273. The full 736-test suite and Pages double build passed, but installed smoke used a temporary output outside the allowed repo/build tree. Put only derived site output under a unique disposable build child; keep installed imports and manuscript processing outside the checkout. Do not relax the renderer's output safety checks.
Refs #273. Preserve last valid site on write failures, fail closed when atomic replacement is unavailable, and reject competing first writes.
Refs #275, #269, #273. Preserve the existing gallery, Reader integration, and accepted current master histories. Correct the verified full renderer indentation and replace the stranded mkdir lock with stable nonblocking kernel locking. Add five bounded crash/special-file/inode regressions; all eleven publisher tests pass locally. Preserve existing tests and add installed-gallery deterministic CI. Reconcile owner delivery selection and the roadmap without changing scientific, licensing or deployment gates. Hosted full-suite, installed wheel and real-route visual acceptance are not claimed by this authoring commit. No same-run merge acceptance.
Collaborator
Author
Exact-head verification — 2026-10-18Head
The full hosted suite and installed-package checks are now verified, superseding the authoring-time NOT_RUN/PENDING scope. Later fresh acceptance must inspect the new diff and remaining presentation scope before merge; this authoring run does not self-approve or merge its repairs. No corpus/benchmark gate or Pages activation changed; M2 remains 0/5. |
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
1 participant
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.
Closes #275. Closes #269. Closes #273.
Delivered scope
One successor PR for the owner's 2026-10-18 cleanup. #268 and #272 are already merged, closing #267/#271; their accepted code and current master are preserved. The remaining executable product issues (#269/#273) are integrated here. #265 remains a separate administration/branch-deletion capability blocker.
site_renderer.pyindentation regression on its complete, hash-verified current file.project/ROADMAP.mdwith concrete acceptance. Reconcile the owner's delivery-first correction in AGENTS/manifest without changing scientific gates.Preservation and scope checks
Authoring head:
d81aa3a46c0125561a1ba610d51a6cf9f06df0c4.Parents preserve gallery
c411e29030e7e778f99cd9689441d348f75a7715, Reader62e6c078cb70ec77775007a1668fa0c3ede4f5fa, and current master9412ee54ab1a56c7c81f6ff4d290e39d83f49630. Native comparison is ahead-only, with no deleted files. The former README is retained byte-for-byte as RESEARCH_INDEX.md. No corpus/benchmark/showcase JSON, publication manifest or morphology/dictionary dependency pin changes.#270/#274 are retained as provenance pending successor acceptance; this is the single active repair path, already claimed from #273 through #275. They must not be merged independently with their old findings. No new branch, force-push, scheduler/admin/access/visibility change, registry publication or Pages activation.
Verification performed before authoring
74a79963b06777ab4b15145b541b7b06a59d7aedreproduced syntax failure; corrected file compiles and uploaded blob8a7c4f468a0a0290cd5a678bf08b44ac0cd482c0matches tested local bytes.f99fb10f4f19b3b363a3accbe68212feacf66c44reproduced two process-death failures.Required later acceptance
Inspect actual new exact-head CI: complete canonical suite, installed wheel outside checkout, installed gallery rebuild and deterministic canonical Pages build. Unknown/skipped/stale checks are not PASS. Inspect real desktop/mobile route behavior when permitted; visual acceptance is NOT_VERIFIED at creation. A static build is not DEPLOYED/LIVE_VERIFIED. Do not merge this substantively authored repair in the same run.
M2 remains 0/5. No new corpus admission, altered rounding/tolerance, fabricated metric or FantLab parity claim.