refactor(plexus): migrate Node to functional component (#3392)

[hharshhsaini]

Which problem is this PR solving?

• Resolves [UI Refactor] [plexus] Migrate Node to Functional Component #3392

Description of the changes

• Migrated the Node component from a class-based component to a functional component with React.memo()
• Converted Node class component extending React.PureComponent to a functional component
• Wrapped the component with React.memo() to preserve the shallow prop comparison behavior of PureComponent
• Destructured props directly in the function signature
• Removed the render() method and converted to direct return statement
• Maintained generic type support using double type assertion pattern: as unknown as <T = {}>(props: TProps<T>) => React.ReactElement | null
• Preserved all existing functionality including conditional rendering logic
• No changes to component logic or behavior, only structural refactoring

How was this change tested?

• All existing plexus tests pass (18 tests)
• TypeScript compilation successful
• Prettier formatting verified
• Build successful for plexus package
• Minimal diff: 18 insertions, 18 deletions

Proof -

Checklist

• I have read https://github.com/jaegertracing/jaeger/blob/main/CONTRIBUTING_GUIDELINES.md
• I have signed all commits
• I have added unit tests for the new functionality
• I have run lint and test steps successfully: make lint test

AI Usage in this PR (choose one)

See AI Usage Policy.

• None: No AI tools were used in creating this PR
• Light: AI provided minor assistance (formatting, simple suggestions)
• Moderate: AI helped with code generation or debugging specific parts
• Heavy: AI generated most or all of the code changes

[github-actions]

Hi @hharshhsaini, thanks for your contribution! To ensure quality reviews, we limit how many concurrent open PRs new contributors can open.

This PR is currently on hold (Status: 9/1 open). We will automatically move this into the review queue once your existing PRs are merged or closed.

Please see our Contributing Guidelines for details on our tiered quota policy.

[greedy-wudpeckr]

@hharshhsaini , i think for the issues whom you are creating PR's , someone has already created PR's for them , kindly look that first.

@yurishkuro PTAL .

[hharshhsaini]

@hharshhsaini , i think for the issues whom you are creating PR's , someone has already created PR's for them , kindly look that first.

@yurishkuro PTAL .

@greedy-wudpeckr
@yurishkuro I did notice the existing PRs, but since they haven’t been merged yet and seem to be on hold, I submitted this PR as an alternative implementation. Please let me know if you’d prefer I close this in favor of another PR.

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[jkowall]

@hharshhsaini please address copilot comments.

[github-actions]

PR quota unlocked!

@hharshhsaini, this PR has been moved out of the waiting room and into the active review queue:

• Open: 7
• Limit: 3

Thank you for your patience.

[codecov]

Codecov Report

❌ Patch coverage is 0% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.68%. Comparing base (c043a42) to head (e4eb5ff).
⚠️ Report is 27 commits behind head on main.

Files with missing lines	Patch %	Lines
packages/plexus/src/Digraph/Node.tsx	0.00%	11 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3531      +/-   ##
==========================================
+ Coverage   88.60%   88.68%   +0.08%     
==========================================
  Files         299      300       +1     
  Lines        9484     9560      +76     
  Branches     2500     2449      -51     
==========================================
+ Hits         8403     8478      +75     
- Misses       1078     1079       +1     
  Partials        3        3              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[hharshhsaini]

@hharshhsaini please address copilot comments.

@jkowall the export typing Copilot suggestion (as typeof Node) and rebased on jaegertracing:main. All tests and type checks are passing now .

[jkowall]

Looks good, thanks @hharshhsaini