Skip to content

v1.4.0: enforce white three-line tables and in-place formula replacement - #2

Merged
AllenWang2005 merged 1 commit into
mainfrom
skill-v1.4.0
Jul 7, 2026
Merged

v1.4.0: enforce white three-line tables and in-place formula replacement#2
AllenWang2005 merged 1 commit into
mainfrom
skill-v1.4.0

Conversation

@AllenWang2005

Copy link
Copy Markdown
Owner

Fixes the two real-world failures observed in a delivered report:

  1. 表头带底纹(非白底三线表) — the table style's firstRow conditional shading (living in styles.xml, invisible in document.xml) survived into the delivered table. New recipe section + TABLE_SHADING (FAIL) auditor check that resolves the referenced table style's basedOn chain and tblStylePr conditional formats.
  2. 公式追加到段尾而非原位替换 — formulas were appended as italic duplicates (……如图 1 所示。F = 44.5 km²L = 15.4 kmL/J^(1/3) = 75.1). New In-Place Replacement Contract in the formula workflow + MATH_DUPLICATE (FAIL) auditor check.

Also added TABLE_RULES (WARN — missing top/bottom rules, row-to-row insideH, header rule not thinner), required reading of the three-line-table recipe whenever a document has tables, two new Must-Fix Audit items, tests for all three checks, and regenerated samples (the compliant sample now builds a real explicit three-line table).

CI steps run locally: byte-compile OK, 69/69 tests pass.

🤖 Generated with Claude Code

Driven by two real-world failures: table-style firstRow shading surviving
into a delivered three-line table, and formulas appended to paragraph ends
as italic duplicates instead of replacing the original text in place.

- Formula workflow: In-Place Replacement Contract (replace at exact
  position, original plain text must be gone, no appending/merging/strays)
- Three-line table recipe: white-background section (drop w:tblStyle,
  zero w:tblLook flags, clear every w:shd)
- Auditor: TABLE_SHADING (FAIL, resolves styles.xml basedOn/tblStylePr),
  TABLE_RULES (WARN, top/bottom/insideH/header-thickness), MATH_DUPLICATE
  (FAIL, OMML text still present as plain text in the same paragraph)
- Tests for all three checks; samples regenerated (compliant sample now
  builds a real explicit three-line table)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@AllenWang2005
AllenWang2005 merged commit 3de7d3c into main Jul 7, 2026
4 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.

1 participant