[lexical-table] Bug Fix: Prevent error if pasted table has empty row

[amanharwara]

Description

Currently, if a table was pasted that has an empty row, the $tableTransform would throw an error. This is because in $computeTableMapSkipCellCheck, an array for a row is only initialized inside the loop that goes through its children. This causes there to be a gap in the gridMap since an array for the empty row never gets initialized, which then causes gridMap[i] to be undefined and throw an error.

This PR fixes this by making sure an empty array is initialized at the index of the empty row, which in turn will be filled with the necessary amount of cells by $tableTransform

Test plan

Added e2e test to cover this scenario

Before

After

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
lexical	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 16, 2025 5:04pm
lexical-playground	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 16, 2025 5:04pm

[github-actions]

size-limit report 📦

Path	Size
lexical - cjs	29.03 KB (0%)
lexical - esm	28.87 KB (0%)
@lexical/rich-text - cjs	37.97 KB (0%)
@lexical/rich-text - esm	30.92 KB (0%)
@lexical/plain-text - cjs	36.52 KB (0%)
@lexical/plain-text - esm	28.13 KB (0%)
@lexical/react - cjs	39.79 KB (0%)
@lexical/react - esm	32.28 KB (0%)

[etrepum]

I don't think this is correct regarding rowspan, but you can initialize startMapRow here which will have the same effect and should be correct (at least with regard to that constraint). You'll have to remove the initialization below but I can't put a suggestion on code you didn't change in the review

Suggested change

	if (row.getChildrenSize() === 0) {
	tableMap[rowIdx] = [];
	}
	const startMapRow = getMapRow(rowIdx);