test gap: SEQ Table identifier coverage missing in canonical 5-run form tests (#104 follow-up)

[kiki830621]

Problem

From verification of PsychQuant/che-word-mcp#104 (verify report: comment-4346442620):

「SEQ Table identifier completely uncovered by tests. Issue body explicitly lists 7 SEQ Table paragraphs in reproducer; tests only use sequenceName: "Figure". Parser is identifier-transparent (dispatchParse doesn't read identifier string), so logic should work, but no defensive assertion exists.」

— Source: team:requirements + team:devils-advocate + codex (gpt-5.5)

#104's reproducer (docs/...碩士論文-rescue-swift-v317.docx) contains both SEQ Figure (12 occurrences) and SEQ Table (7 occurrences). The fix in commits 537de62 + 58fe4f9 is identifier-transparent — FieldParser.dispatchParse reads the SEQ identifier from instrText after detection, so Figure passing should imply Table passes too. But no test exercises a non-Figure identifier, so the assumption is unverified.

Type

test gap (P3)

Acceptance criteria

• Add a test that constructs a canonical 5-run paragraph with instrText = " SEQ Table \\* ARABIC " (or hand-built equivalent) and asserts FieldParser.parse(...) returns 1 ParsedField with .sequence(seq) where seq.identifier == "Table".
• OR: extend an existing parametric test (if any) to iterate over ["Figure", "Table", "Equation"] identifiers.

Why low priority

Parser logic is genuinely identifier-transparent (verified by reading FieldParser.dispatchParse and SequenceField.parse(instrText:)). Real-world risk of regression is near-zero unless a future refactor introduces identifier-specific code paths. But the asymmetry violates "test what the issue body claims to fix" — surfaced as part of #104's verify discipline.

Related: PsychQuant/che-word-mcp#104

---

Current Status

Phase: implemented
Last updated: 2026-05-02 by idd-implement

Key Decisions

• Implemented as focused canonical 5-run SEQ Table parser coverage in PR Cover canonical sequence field edge cases #37.
• No production behavior change was needed; parser remains identifier-transparent.

Scope Changes

• (none)

Blocking

• Awaiting IDD verify / PR review.

Commits

• 11860b5 test: cover canonical sequence field edge cases

[kiki830621]

Implementation Plan

Scope

Add canonical 5-run parser coverage for non-Figure SEQ identifiers.

Plan

• Extend Issue104FieldParserCanonicalFormTests with a canonical 5-run SEQ Table \\* ARABIC case.
• Assert FieldParser.parse(paragraph:) returns exactly one .sequence field whose identifier is Table.

[kiki830621]

Implementation Complete

Checklist (synced from TaskList)

• Add canonical 5-run SEQ Table parser coverage -> commit 11860b5
• Assert parsed sequence identifier is Table -> commit 11860b5

Changes

• 11860b5: added testFieldParserHandlesCanonical5RunSEQTableIdentifier() in Issue104FieldParserCanonicalFormTests.
• PR: Cover canonical sequence field edge cases #37

Files Changed

Sources/OOXMLSwift/Parsing/FieldParser.swift       |   3 +-
Tests/OOXMLSwiftTests/Issue104FieldParserCanonicalFormTests.swift | 164 +++++++++++++++++++++

Verification

• swift test --filter Issue104FieldParserCanonicalFormTests -> passed
• swift test -> 834 tests, 1 skipped, 0 failures

Scope Compliance

Only canonical SEQ parser/update coverage and the requested truncated-span doc comment were changed.

Next: verification pending