Skip to content

fix(docx-io): strip local comment tracking tokens - #419

Draft
loc01227175513 wants to merge 1 commit into
cicero-im:claude/fix-docx-validator-Ou7FOfrom
loc01227175513:codex/fix-pr57-local-token
Draft

fix(docx-io): strip local comment tracking tokens#419
loc01227175513 wants to merge 1 commit into
cicero-im:claude/fix-docx-validator-Ou7FOfrom
loc01227175513:codex/fix-pr57-local-token

Conversation

@loc01227175513

Copy link
Copy Markdown

Summary

  • strip DOCX comment tracking tokens from local comment fallback document content
  • strip tokens recursively from local rich comment bodies during re-import
  • add regression tests for plain text and rich comment body re-import paths

Verification

  • npm exec --yes bun -- test packages/docx-io/src/lib/applyDocxTracking.spec.ts packages/docx-io/src/lib/parseDocxTracking.spec.ts
  • yarn biome check packages/docx-io/src/lib/importComments.ts packages/docx-io/src/lib/applyDocxTracking.spec.ts
  • yarn turbo build --filter=./packages/docx-io

Notes:

  • yarn turbo typecheck --filter=./packages/docx-io still hits the same TypeScript internal Debug Failure on the unmodified PR head baseline under Node v22.15.0.
  • yarn workspace @platejs/docx-io lint still reports many pre-existing package diagnostics outside the touched files.

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