Skip to content

test(docx-core): characterize moveTo paragraph-mark Reject parity #985

Description

@stevenobiajulu

Independent dynamic Opus 5 review of PR #980 head25ff4870 found a pre-existing native Reject discrepancy: native rejectChanges PhaseA checks paragraph mark ins only, while comparison rejectAllChanges also resolves moveTo paragraph marks.

Synthetic reviewer control (no range markers): an empty moveTo-marked paragraph followed by text B produces two paragraphs [empty,B] natively versus one [B] through comparison AST. A plain ins-mark control agrees. This minimizes the differing predicates, but a repair must also test complete schema-valid move ranges/packages, not rely only on this reduced input.

  • Reproduce the predicate difference on main.
  • Add complete move-range controls for source/destination and Accept/Reject with native/AST parity.
  • Cover terminal and table-cell paragraphs, foreign authors and preserved range boundaries.
  • Decide bounded native mark handling without content-based paragraph deletion; run schemas, full gates and independent review.

The phase-A predicate is untouched by PR#980; its note-bearing reader regressions pass. No fix included in this PR. Separate from #982 cosmetic mark cleanup and #983 inherited reader field recalculation. Word remains unverified.

Ref: #941, PR #980 review N2.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions