Checklist

• I've thoroughly read the latest contribution guidelines.
• I've rebased my branch onto main before creating this PR.
• My commit messages follow conventional spec
• I've added tests to cover my changes (if applicable).
• I've verified that all new and existing tests have passed locally for mobile, tablet, and desktop screen sizes.
• My change requires documentation updates.
• I've updated the documentation accordingly.

Description

Addresses review feedback on PR #2425 to improve type safety and API semantics.

Key Changes

ProjectData includePaths consolidation – Merged separate includePaths: URI[] and includePathsAsStrings: ProjectFolder[] arrays into single structure:

interface ProjectData {
  includePaths?: Array<{
    uri: URI
    folder: ProjectFolder
  }>
}

Eliminates possibility of arrays getting out of sync and reduces cognitive overhead.

Node lookup semantics – Renamed findNodeByElementFqn to findNodeByModelFqn and changed return type from NodeId | null to enriched node object:

export const findNodeByModelFqn = <T extends NodeWithData>(
  xynodes: T[],
  elementFqn: Fqn,
): (T & { data: { modelFqn: Fqn } }) | null

Callers now get full node with guaranteed modelFqn property instead of just ID string.

Auto-unfocus timing – Changed AUTO_UNFOCUS_DELAY from 0 to 3000ms. Previous value made conditional check always false, disabling intended behavior.

Test cleanup – Removed autogenerated test file with no actual assertions.

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