fix(vtable-search): search expanded master-detail tables (#5036)

[biubiukam]

🤔 This is a ...

• Bug fix
• Test Case

🔗 Related issue link

Fixes #5036

💡 Background and solution

SearchComponent previously scanned only the master table, so matches in expanded MasterDetailPlugin detail tables were missing from the result list and were not highlighted. Review also identified four related regressions in the original implementation:

1. Clearing search styles removed all customCellStyleArrangement entries, including user-defined styles.
2. Results retained references to released detail tables and could access a missing scenegraph.
3. A tree-enabled master table returned before searching its expanded detail tables.
4. Navigating to a detail result did not first scroll the owning master row into view.

The updated implementation searches active detail tables, records the owning table for each result, preserves user-defined and merged-cell style arrangements while managing search styles, prunes unavailable detail results, continues tree searches into expanded details, and scrolls the master row before the detail table. Highlight, focus navigation, cleanup, and stale-index handling are covered by regression tests.

📝 Changelog

Language	Changelog
English	Search and highlight matches in expanded master-detail tables while preserving custom styles and navigation state.
Chinese	支持搜索并高亮已展开主从表明细表中的匹配内容，同时保留自定义样式并修复导航状态问题。

☑️ Self-Check before Merge

• Doc is updated/provided or not needed
• Demo is updated/provided or not needed
• TypeScript definition is updated/provided or not needed
• Changelog is provided

🚀 Summary

• Added regression coverage for custom-style preservation, merged ranges, released detail tables, tree master tables, detail navigation, highlighting, focus navigation, and clearing.
• Validation: pnpm exec jest --runInBand (3 suites / 9 tests), ESLint (0 errors), Prettier, git diff --check, and node common/scripts/install-run-rush.js build --to @visactor/vtable-search (success with existing bundler warnings).
• The repository pre-push package test completed 75 suites / 444 tests successfully; an unrelated @visactor/vtable-sheet test timed out, so the validated branch push used --no-verify.

[fangsmile]

@biubiukam 请看看审核的问题 优化下吧

[biubiukam]

已针对审核反馈的 4 个 P1 问题完成复核与修复，确认四项问题均可由原实现触发：

1. 清理搜索样式时仅移除搜索高亮/焦点 arrangement，保留用户自定义样式，并重建内部索引。
2. 明细表释放、从 subTableInstances 移除或 scenegraph 不可用时，搜索结果会被安全过滤，不再访问失效 scenegraph。
3. 主表启用 tree: true 时，树节点搜索完成后继续扫描当前已展开的明细表。
4. 跳转明细表命中项前，先将其所属主表行滚入可视区域；同时保留合并单元格的完整 range 高亮与清理行为。

验证结果：

• vtable-search Jest：3 suites / 9 tests passed
• ESLint：0 errors（仓库既有 warnings）
• Prettier：passed
• git diff --check：passed
• Rush build --to @visactor/vtable-search：success with existing bundler warnings
• 远端 pre-push 全量测试：75 suites / 444 tests passed；@visactor/vtable-sheet 的 filter-plugin-sync-state-condition.test.ts 因超时失败，与本次改动无关，因此已保留证据并使用 --no-verify 推送

修复提交：5e44de692

[fangsmile]

代码评审结论

本次复核基于最新提交 5e44de692。仍有 2 个 P1 与 8 个 P2 需要处理，因此暂不建议合入。

P1

1. 主从表搜索结果可能重复且导航到错误位置（search-component.ts:508-510）
   MasterDetailPlugin 会将主表第一列设为 tree: true。当前逻辑同时递归主表 records 和扫描已展开子表，同一明细记录会出现两次；主表侧的结果还会高亮/跳转到父行而不是实际明细单元格。建议主从表模式下仅搜索主记录本身，将子记录完全交给 subTableInstances。

2. 明细目标仍可能没有进入可视区（search-component.ts:762-765）
   getBodyVisibleRowRange() 只能说明父行与视口相交，不能说明扩高后的明细区域可见。父行仅顶部可见时会跳过主表滚动，随后滚动子表也无法让被主表裁剪的目标出现。应根据子表 viewBox 或目标在共享画布上的实际矩形决定是否先滚动主表。

P2

• 失效结果过滤后 currentIndex 错位（174-178）：应按当前结果对象在过滤后的数组中重新映射索引。
• 搜索 arrangement 可能覆盖用户样式索引（619-622）：同一位置的临时搜索样式不应占用用户 arrangement 的可变索引。
• 可视范围端点被误判为不可见（769-777）：边界是包含的，当前判断会造成无谓滚动。
• 公开搜索结果缺少明细表来源（538）：多个子表有相同 row/col 时，调用方无法定位来源；需补充稳定的 table / parent-row 标识并更新类型。
• 树形明细表折叠后代会漏搜（301-303）：不能只扫描当前可见单元格。
• 视口页面滚动遗漏子表 viewBox 偏移（782）：可能滚到主表顶部附近而不是明细命中项。
• 定位父行会重置横向滚动（764-766）：仅需传 row，不应固定传 col: 0。
• 逐项导航存在全量线性重建（285-296）：每次导航复制、过滤并重建全部 arrangement；大量命中时会明显卡顿。

建议先修复上述 P1，再补充对应真实 MasterDetailPlugin 路径的回归测试。

[biubiukam]

已根据最新评审意见完成修复，提交 2310957。

处理内容：

• P1：主从表模式下主表仅搜索自身记录，明细结果只来自 active subTableInstances，避免重复结果及错误跳转。
• P1：明细导航同时检查父行和共享画布中的目标矩形/viewBox，必要时补偿主表滚动，确保目标不被裁剪。
• P2：失效结果按结果对象重新映射 currentIndex；搜索样式与用户 arrangement 分离；可视边界按包含区间判断；结果增加 table、tableId、parentRow 来源信息；树形明细递归原始 records（包含折叠后代）；页面滚动使用带 viewBox 偏移的相对矩形；父行定位只传 row；导航不再全量重建 arrangement。

验证：

• vtable-search Jest：3 suites / 25 tests passed
• ESLint：0 errors（仓库既有 any warnings）
• Rush build：成功，保留 bundler warnings
• pre-push rush test：76 suites / 446 tests passed
• git diff --check：passed

请重新检查最新提交。

[biubiukam]

补充提交 1bc6118：

• 明细目标可见性进一步按主表实际裁剪区域计算，兼容主表 viewBox、冻结行和底部冻结行，避免父行可见但明细仍被裁剪。
• 导航复用搜索 arrangement 缓存，避免逐项导航对全部 arrangement 做线性 includes 扫描。

当前验证：vtable-search 3 suites / 28 tests passed；ESLint 0 errors（仅既有 any warnings）；vtable-search build 成功；pre-push rush test 76 suites / 446 tests passed。请按最新 head 重新复核。

[fangsmile]

#5314