fix: search children of disabled tree nodes

[nrps9909]

Summary

• keep traversing children when a tree node is disabled or selectable={false}
• activate the first matching selectable descendant during search
• cover keyboard selection below both disabled and non-selectable parents

Fixes #652.

Why

getFirstMatchingNode previously used continue for a disabled or non-selectable node. That skipped the node's entire subtree, so an enabled matching child could be rendered in search results without becoming active or selectable through Enter.

The updated traversal only excludes the parent itself as a candidate. It still descends into the parent's children, while the existing Enter handler continues to enforce disabled and selectable guards on the active node.

Verification

• Exact-base regression: both new cases failed because no active node was found
• Focused search-input suite: 10/10 tests passed
• Full suite: 12/12 suites, 186/186 tests, 15/15 snapshots passed
• npm run tsc
• npm run lint (0 errors; 6 existing warnings)
• npm run compile
• git diff --check

AI assistance disclosure: Codex was used to trace the active-node traversal, construct the exact-base regressions, implement the focused fix, and run the verification commands. I reviewed and verified the source diff, failure reproduction, and test results.

Summary by CodeRabbit

• Bug Fixes
  • 修复搜索结果选择逻辑：即使父节点被禁用或不可选择，仍可搜索并选中其可选择的子节点。
  • 优化树形选项的匹配行为，避免不必要地跳过整个子树。

[vercel]

@nrps9909 is attempting to deploy a commit to the React Component Team on Vercel.

A member of the Team first needs to authorize it.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 0bc661af-ec0c-4b97-b8e8-fdd46c934ea5

📥 Commits

Reviewing files that changed from the base of the PR and between 7035d29 and a8e4a76.

📒 Files selected for processing (2)

• src/OptionList.tsx
• tests/Select.SearchInput.spec.js

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.

---

Walkthrough

Changes

搜索结果子节点选择

Layer / File(s)	Summary
匹配节点搜索与键盘验证 src/OptionList.tsx, tests/Select.SearchInput.spec.js	getFirstMatchingNode 在跳过 disabled 或 selectable === false 节点自身后继续检查子节点。参数化测试验证搜索结果可聚焦，并可通过回车选择可选子节点。

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to a8e4a

This localized change allows search to reach selectable descendants beneath disabled or non-selectable parents, with the documented regression and validation checks passing; no actionable merge-blocking risk remains.

Suggested reviewers: afc163

Poem

小兔敲键盘，
父节点虽不可选，
子节点仍显现。
搜索找到目标后，
回车轻轻完成选择。

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	标题准确描述了主要修复：搜索会继续检查 disabled 节点的子节点。标题未明确提及 selectable=false，但仍与核心变更直接相关。
Linked Issues check	✅ Passed	代码仅排除 disabled 或 selectable=false 节点自身，并继续递归搜索其子节点。可选择的匹配子节点可以获得焦点并通过 Enter 选中，符合问题 [#652] 的核心要求。现有搜索结果导航逻辑无需额外修改。
Out of Scope Changes check	✅ Passed	变更仅涉及搜索遍历逻辑和对应的键盘选择测试。所有变更都直接支持问题 [#652]，未发现无关代码或测试修改。
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 2…

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 2 files.

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}