Skip to content

Naprawa projekcji rewizji znaków akapitu z bezpieczną kontrolą pokrycia - #204

Merged
mikolaj92 merged 1 commit into
mainfrom
feature/6311-paragraph-revision-coverage
Oct 2, 2026
Merged

mikolaj92 merged 1 commit into
mainfrom
feature/6311-paragraph-revision-coverage

Conversation

@PSyron

@PSyron PSyron commented Sep 30, 2026 •

Copy link
Copy Markdown
Collaborator

Rewizje znaków akapitu nie miały adresowalnych potwierdzeń w projekcji przeglądu, przez co dokumenty zawierające wyłącznie takie rewizje mogły być odrzucane przez konsumenta.

Zmiana wystawia oddzielne, zerowej długości potwierdzenia strukturalne, także dla pustych akapitów. Nie liczy historii formatowania jako brakującej granicy. Usunięcie granicy niepustego akapitu oznacza niekompletną projekcję, dopóki nie istnieje mapowanie semantycznego połączenia do zakresów źródłowych. Nie zmienia mechaniki zaakceptowania rewizji ani nie dodaje reguł prawnych.

Powiązane issue: https://github.com/mikolaj92/Temida/issues/6311. Lokalna naprawa została przetestowana w pełnym pipeline Argusa na bazie używanej przez demo; draft jest przeniesiony na aktualny main. Test operatora nadal trwa. Nie jest to zgoda na merge ani wdrożenie.

Walidacja finalnego SHA 9f3c7dde281986e2f3e03beeffe3aaa2b8bf30e0: Ruff przeszedł; pełny zestaw testów na aktualnej bazie main: 256 passed. Test konsumenta Dike z tym kodem i poprawionym ReviewKit: 8 passed. Zależny draft: mikolaj92/reviewkit#368.

@mikolaj92

Copy link
Copy Markdown
Owner

User-directed manual library verification

Exact tested/reviewed HEAD 9f3c7dde281986e2f3e03beeffe3aaa2b8bf30e0.

  • Independently reran complete declared contract: Ruff PASS + 256 project tests PASS,61.49s, existing5PyMuPDF warnings retained.
  • Review evidence retained honestly: first exact-model request hit output limit with NO final verdict; second terminal REQUEST_CHANGES was preserved. Concrete claims were investigated rather than ignored.
  • Actual existing XML mutation owner + runnable 3-case differential probe refuted the findings: retained text with deleted paragraph mark really merges (Before After) and is INCOMPLETE; all-deleted/empty paragraph leaves After and is COMPLETE; count mismatch is explicitly fail-closed.
  • Evidence-enriched independent exact-model GLM-5.3-Flash-EXL3 adjudication → PASS, zero remaining concrete logic/security findings. No source/threshold/production envelope changes to force approval; named false rejections recorded in Lokay #1484.
  • Zero-width addressable structural receipts; formatting history not a missing boundary; unsupported nonempty semantic joins remain blocked.
  • Current owner instructions distinguish valuable library fixes204/368 from rejected Temida6312 YAML/main-backport. This library merge does NOT merge6312 or invent UI/operator acceptance; tags/separate experimental integration remain separate work.

Manual owner-authorized recovery, NOT autonomous Lokay delivery evidence.

@mikolaj92
mikolaj92 merged commit 53aaaf4 into main Oct 2, 2026
2 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