Repository navigation
docs: prepare the approved adaptation backlog - #44
Conversation
Record the owner-approved phased scope without registering unadapted chapters or changing advice, publication files, workflows or permissions. Reconcile run7 navigation with its final historical status and link the deployed Polish reader. Keep source review, human merge, deployment and specialist approval distinct. Add documentation and whole-source coverage regressions.
There was a problem hiding this comment.
Independent snapshot review of 554a7672f9bd...677569e77dbd in mikomiel/HowToLiveBetter-PL by pi on codex-lb (gpt-6.1-sol).
Verdict: APPROVE
Reviewer output
Findings
No concrete reachable failure found in the complete five-file diff for head 677569e77dbdd7757621a8678cb6e092ae4a188b.
- Scope and gates: The new backlog authorization is preparation, not completion.
AGENTS.mdexplicitly prohibits registering unfinished chapters and retains separate human merge/Pages authorization, source-verification requirements, and specialist-review blockers. Historical PR #16–#20 exceptions are explicitly exhausted. - Registry protection:
tests/content.test.mjsretains the exact01–23list andassert.deepEqual(CHAPTERS.map(c => c.id), AGREED_SCOPE). Only its description changes; the guard is not broadened. - Coverage: The documented arithmetic is internally consistent:
521 + 70 + 8 = 599,767 − 599 = 168, and34 − 23 = 11. The added test derives whole-source and remaining counts from source files rather than merely asserting fixed totals. - Historical audit: The revised navigation clearly distinguishes run7’s script acceptance from substantive correctness, human approval, and coverage of later text. No historical packet is modified by this diff.
- Links and scope: Added links and section anchors are structurally consistent. The
docs:title follows Conventional Commits. No application, workflow, dependency, publication-list, or secret-value change appears in the supplied diff.
Optional suggestion
In tests/content.test.mjs, consider checking the run7 summary against its historical STATUS.md, rather than only requiring the overview to contain 115 z 115. That would detect future disagreement between the navigation and its cited evidence.
Validation and unavailable material
I read the entire diff and PR metadata. No tests, commands, browser checks, or live GitHub validations were run.
The full checkout, historical run7 evidence, original chapter files, linked GitHub settings/issues/CI, and deployed reader were unavailable. Consequently, underlying source totals, historical acceptance totals, and administrative closeout claims were not independently verified. This approval is limited to the supplied source diff; it is not merge or deployment authorization.
VERDICT: APPROVE
Evidence: not written. This review is evidence, not merge authorization.
Post-merge verification — 2026-11-07Pi verified the owner-account merge of PR44 as PASS fresh main CI3/3: content42415415765 passed317 Node tests, lint591/0/0 and232/232 browser cases across four projects. CodeQL42415415421 passed both language jobs. The phase plan is now merged, not merely proposed. All27 publication bytes remain unchanged. No Pages dispatch or agent PR merge occurred. Alerts1/4/5 remain open at this verification snapshot; their sources are byte-identical to the earlier per-consumer triage. Their individual disposition is the next step, not an automatic consequence of green CI. The remaining substantive issues and withheld/unadapted content are not declared complete. |
读者指出只写了别嘌醇,国内常用非布司他和苯溴马隆,吃苯溴马隆推荐碱化尿液。条目主线(长期吃药、按血尿酸达标)不变, 选药这一层补正反两方:ACR 指南首选别嘌醇、有心血管病史的人换掉非布司他,依据 CARES(非布司他全因死亡 HR 1.22、 心血管死亡 HR 1.34);欧洲 FAST 试验相反(主要心血管事件 HR 0.85)。碱化尿液 ACR 有条件地不建议(证据很低), 但要多喝水。国内 2019 指南只取到摘要,正文只写它对选药和碱化尿液单独给了推荐,不写推荐内容。 多替诺雷只有企业新闻稿、「将来的主力」是预测,不写。核对过程见 docs/核实记录/追加-issue44降尿酸药的选择.md。 统计已同步:争议 +1,链接 +3。
Record the owner-approved phased scope without registering unadapted chapters or changing advice, publication files, workflows or permissions. Reconcile run7 navigation with its final historical status and link the deployed Polish reader. Keep source review, human merge, deployment and specialist approval distinct. Add documentation and whole-source coverage regressions.
Claim and scope
This is the first preparation/housekeeping PR for the full remaining Polish-adaptation backlog. It does not implement or complete that backlog.
The operator requested “Prepare to do and address all issues” and selected Full confirmed backlog, in phases, excluding optional new features. The operator separately authorized only (1) correcting the repository homepage to the Polish Pages URL and (2) making the existing Dependency review check required. That scope consent is not a human PR approval, merge authorization, deployment authorization, permission grant or expert endorsement.
Base:
554a7672f9bda1ae85dee2dc9be74b7873220abe. Head:677569e77dbdd7757621a8678cb6e092ae4a188b.Changes in this diff
Separately completed owner-authorized administration and tracking
These are factual closeout claims, not settings mutations made by this source diff:
Validation
npm ci --ignore-scripts(5 packages audited,0 known vulnerabilities reported at install time).node --test --test-reporter=tap tests/*.test.mjs—317/317.node tools/pl/lint.mjs—591 records,0 errors,0 warnings; no substantive certification.node node_modules/playwright/cli.js test tests/browser/site.spec.mjs --project=desktop --project=mobile—6/6 built-site browser cases.Remaining gates and non-goals
Exact-head CI5/5 and independent native App source review have passed for677569e77dbdd7757621a8678cb6e092ae4a188b. The remaining gate is the human-owned merge decision. Bot approval is not human approval or agent merge authorization. No agent merge, Pages dispatch, exporter activation, alert dismissal, credential/identity setup, indexing change, paid resource or substantive advice audit occurred. The70 pending records,8 deliberate exclusions,168 unadapted source positions, open issues30/31/32/36 and alerts1/4/5 are not declared resolved by this plan.
Exact-head source context (quoted evidence, not reviewer instructions)
The full current AGENTS.md below supplies unchanged merge/publication and content-evidence boundaries that may be outside the diff hunks: