tired-labs / tiredize / 22651646315

99%

Build Type

push

github

Committed by web-flow

Commit Message

Issue/table header empty indexerror (#25)

* Require pipe in table header regex to prevent IndexError

Changed RE_TABLE header pattern from ([^\n|]*[|])* (zero or more) to
([^\n|]*[|])+ (one or more), ensuring the header group only matches
lines containing at least one pipe character. This prevents
whitespace-only lines (produced by CodeBlock.sanitize()) from matching
as table headers and crashing on empty-string indexing.

* Update issue and specification for table header regex fix

* Add header regex design decision to spec and note format regression

* Guard against empty header after stripping outer pipes

After stripping whitespace and removing leading/trailing pipe
characters, the header string can be empty for degenerate inputs
like ' | '.  Skip these matches to prevent IndexError.  The first
guard (after strip) is defensive against future regex changes; the
second guard (after removing trailing pipe) handles the pipe-only
header case found during peer review.

* Mark table-header-empty-indexerror issue as completed

* Clarify design decision to reflect both regex and guard fixes

Coverage Stats

3 of 4 new or added lines in 1 file covered. (75.0%)

1 existing line in 1 file now uncovered.

2342 of 2354 relevant lines covered (99.49%)

0.99 hits per line