Skip to content

fix(test): move web-server-health ports below ephemeral range - #445

Merged
EyJunge1 merged 1 commit into
tickernelz:mainfrom
EyJunge1:fix/436-web-server-health-ports
Oct 27, 2026
Merged

EyJunge1 merged 1 commit into
tickernelz:mainfrom
EyJunge1:fix/436-web-server-health-ports

Conversation

@EyJunge1

Copy link
Copy Markdown
Collaborator

Summary

  • Move fixed takeover-test ports from 48747/48748 to 28747/28748 so they sit below Linux's default ephemeral range (32768–60999), preventing flaky neighbor-port binds during busy full-suite runs (#436).
  • Import missing log in connection-manager.ts so bun run typecheck / pre-commit pass again (broken on main after WAL index recovery).

Test plan

  • bun test tests/web-server-health.test.ts (6 pass)
  • Noted: full bun test without prior bun run build fails on dist-dependent contracts in a fresh worktree (unrelated)
  • Pre-commit typecheck + lint-staged passed on commit

Checklist

  • Branch is based on main (do not open PRs from commits already on main)
  • bun test passes (or N/A) — targeted file verified
  • bun run typecheck passes (or N/A)
  • bun run check passes (format + lint + typecheck; or N/A) — via pre-commit
  • Docs / README updated when behavior or contribution process changes — N/A

Fixed 48747/48748 sit inside Linux's default ephemeral range, so busy
full-suite runs can transiently hold the neighbor port and flake takeover.

Also import the missing `log` symbol in connection-manager so typecheck
passes (broken on main after WAL index recovery landed).

Closes #436
@EyJunge1
EyJunge1 merged commit c38deb1 into tickernelz:main Oct 27, 2026
2 of 7 checks passed
@EyJunge1 EyJunge1 mentioned this pull request Oct 27, 2026
2 of 6 tasks
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.

1 participant