Clarify selectNodes is required, not optional

[Copilot]

Addressed review feedback questioning why selectNodes was treated as optional in TreeRenderer.tsx.

Analysis

selectNodes is defined as required in TreeRendererProps:

export type TreeRendererProps = {
  selectNodes: (nodeIds: Array<string>, changeType: SelectionChangeType) => void;
  // ... other props
};

UseSelectionHandlerProps preserves this via Pick<TreeRendererProps, "selectNodes" | "rootNodes">, making selectNodes required throughout the type chain.

The original removal of ?? noopSelectNodes fallback was correct - no code changes needed.

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[changeset-bot]

⚠️ No Changeset found

Latest commit: f3cdc42

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR