fix(helm): Rename mcp-server port from "mcp" to "mcp-server" for consistency (fixes #2021).

[junhaoliao]

Description

Rename the Kubernetes port name for the mcp-server component from "mcp" to "mcp-server" so it follows the same naming convention as every other component in the Helm chart (where the port name matches the component name).

Checklist

• The PR satisfies the contribution guidelines.
• This is a breaking change and that has been indicated in the PR title, OR this isn't a
  breaking change.
• Necessary docs have been updated, OR no docs need to be updated.

Validation performed

Ran helm template against the chart and confirmed:

• The rendered Deployment manifest shows name: "mcp-server" for the container port.
• The rendered Deployment manifest shows port: "mcp-server" in the readinessProbe.httpGet and livenessProbe.httpGet sections.
• The rendered Service manifest shows targetPort: "mcp-server".

Summary by CodeRabbit

• Chores
  • Updated Helm chart version to 0.2.0-dev.3.
  • Standardized port naming for the mcp server across deployment, health checks and service configuration to ensure consistent port references.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 504fa35 and 0b5ea1a.

📒 Files selected for processing (2)

• tools/deployment/package-helm/Chart.yaml
• tools/deployment/package-helm/templates/mcp-server-deployment.yaml

---

Walkthrough

Helm chart version bumped from 0.2.0-dev.1 to 0.2.0-dev.3 and port label renamed from "mcp" to "mcp-server" for mcp-server container and Service targetPort for naming consistency.

Changes

Cohort / File(s)	Summary
Chart metadata tools/deployment/package-helm/Chart.yaml	Chart version updated from 0.2.0-dev.1 → 0.2.0-dev.3.
Kubernetes templates tools/deployment/package-helm/templates/mcp-server-deployment.yaml, tools/deployment/package-helm/templates/mcp-server-service.yaml	Renamed port label from "mcp" to "mcp-server" in container port, readinessProbe port reference, and Service targetPort.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related issues

• Port naming consistency for mcp-server Kubernetes manifests: updated port label across deployment and service templates from mcp to mcp-server.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 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: renaming the mcp-server port from 'mcp' to 'mcp-server' for consistency, which aligns with all three affected files in the changeset.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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 and usage tips.}

[kirkrodrigues]

@junhaoliao Can we resolve the conflicts?

[junhaoliao]

assuming #2050 will be merged first