Skip to content

Conversation

@Scra3
Copy link
Member

@Scra3 Scra3 commented Feb 10, 2026

Summary

  • Replace hardcoded port numbers (39310-39331) with dynamically assigned ports using getAvailablePort() helper
  • Fixes flaky EADDRINUSE test failure in CI (PR #1456)

Test plan

  • All 483 mcp-server tests pass locally with coverage
  • Lint passes (0 errors)
  • CI passes

🤖 Generated with Claude Code

alban bertolini and others added 2 commits February 10, 2026 11:38
Replace hardcoded port numbers with dynamically assigned ports to avoid
EADDRINUSE errors in CI environments where ports may already be in use.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@qltysh
Copy link

qltysh bot commented Feb 10, 2026

Qlty

Coverage Impact

This PR will not change total coverage.

🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

alban bertolini and others added 2 commits February 10, 2026 11:51
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@Scra3 Scra3 merged commit 1fce1ee into main Feb 10, 2026
27 checks passed
@Scra3 Scra3 deleted the fix/mcp-server-test-random-ports branch February 10, 2026 13:41
forest-bot added a commit that referenced this pull request Feb 10, 2026
## @forestadmin/mcp-server [1.7.7](https://github.com/ForestAdmin/agent-nodejs/compare/@forestadmin/mcp-server@1.7.6...@forestadmin/mcp-server@1.7.7) (2026-02-10)

### Bug Fixes

* **mcp-server:** add diagnostic logging for 500 errors ([#1456](#1456)) ([1e61fd7](1e61fd7))
* **mcp-server:** use random available ports in server tests ([#1457](#1457)) ([1fce1ee](1fce1ee))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants