[release-4.19] Remove incorrect comment in MCP Apps component

[openshift-cherrypick-robot]

This is an automated cherry-pick of #1911

/assign kyoto

Summary by CodeRabbit

• Chores
  • Code quality improvement: Cleaned up linting configuration in component logic.

---

Note: This release contains no end-user visible changes. It is a maintenance update focused on code quality.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: e7b498f1-dde4-4690-8950-73fa91cf3e5b

📥 Commits

Reviewing files that changed from the base of the PR and between aef9da9 and 0aa2af5.

📒 Files selected for processing (1)

• src/components/MCPApp.tsx

💤 Files with no reviewable changes (1)

• src/components/MCPApp.tsx

---

📝 Walkthrough

Walkthrough

The pull request removes an obsolete eslint suppression comment from a useEffect hook in MCPApp.tsx. The comment had been suppressing a warning about intentionally excluding theme from the dependency array, but since theme is now explicitly included, the suppression is no longer necessary.

Changes

Comment Cleanup

Layer / File(s)	Summary
Eslint Suppression Removal src/components/MCPApp.tsx	Eslint disable comment for theme dependency is removed; the useEffect dependency array explicitly includes theme, making the suppression unnecessary.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A comment once stood guard so tall,
But theme now marches in the hall,
No suppression needed anymore—
The linter's happy, conflict's o'er! ✨

🚥 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	The title accurately describes the main change: removing an incorrect comment from the MCP Apps component in the release-4.19 branch.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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

[kyoto]

/approve
/lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: kyoto

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [kyoto]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment