Skip to content

Fail when DOCX templates contain tracked changes or comments - #84

Merged
nonprofittechy merged 1 commit into
mainfrom
83-detect-track-changes
Sep 1, 2026
Merged

Fail when DOCX templates contain tracked changes or comments#84
nonprofittechy merged 1 commit into
mainfrom
83-detect-track-changes

Conversation

@nonprofittechy

@nonprofittechy nonprofittechy commented Sep 1, 2026

Copy link
Copy Markdown
Member

Closes #83

Summary of Changes

  • Detect review markup in DOCX templates, including tracked changes (w:ins, w:del, w:moveTo, w:moveFrom) and embedded comments (w:commentRangeStart, w:commentRangeEnd, w:commentReference, word/comments.xml).
  • Introduced finding code EG130 (MessageId.DOCX_REVIEW_MARKUP) with error severity by default so that templates with unaccepted/unrejected review markup fail validation.
  • Added --docx-review-markup / --no-docx-review-markup CLI option (enabled by default).
  • Added comprehensive unit and CLI test coverage for review markup detection and CLI flags.

Tested this manually across a folder with 133 DOCX files, 8 of which had active track change records (flagged correctly) and 6 of which had track changes turned on, but no unaccepted changes (by design, not an error).

@nonprofittechy
nonprofittechy merged commit 86406d0 into main Sep 1, 2026
8 checks passed
@nonprofittechy
nonprofittechy deleted the 83-detect-track-changes branch September 1, 2026 19:48
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.

Warn when a document has tracked changes turned on?

1 participant