test(cli): make stale-port status fixture deterministic

[yansigit]

Summary

Allocate the refused runtime-record port after binding the deliberately occupied configured-port listener, and assert the two ports differ. Reallocate per test so the occupied listener cannot reuse a previously released shared port and silently invert the fixture.

Test fixture only; no user documentation change needed.

Verification

bun test tests/cli/cli-status-json.test.ts: 47 passed.

All runtime checks used a fresh temporary OPENCODEX_HOME and alternate port; production config fingerprint and backup inventory remained unchanged. Full root-suite and review-readiness gates have not been completed for this head; this is intentionally a draft.

Checklist

• Scope stays focused and avoids unrelated cleanup.
• Added/updated regression coverage or verified existing coverage for the affected behavior.
• Docs or release notes were updated when needed.
• Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults.

Review readiness

• Local CI green.
• Branch on the latest dev commit.
• All correct Codex and CodeRabbit findings fixed.
• Ready-for-review confirmation.

Co-authored-by: SB Yoon 44089734+yansigit@users.noreply.github.com
Co-authored-by: Yumi automation@sbyoon.com

Review readiness checklist

This PR stays in draft until every box below is ticked. Tick all four boxes once the requirements are met:

• All CI tests are green on my local testing.

• I pushed my PR to the latest dev commit.

• I resolved all correct Codex and CodeRabbit findings.

• My PR is ready for review.

[coderabbitai]

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

• Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

---

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.}

[github-actions]

✅ Deterministic PR hygiene checks passed.

[github-actions]

⏳ DRAFT

• review readiness checklist open (0/4 boxes ticked).

What to do

• Tick all four boxes in the PR description once you're done (currently 0/4).

Review readiness checklist

• ⬜ All CI tests are green on my local testing.
• ⬜ I pushed my PR to the latest dev commit.
• ⬜ I resolved all correct Codex and CodeRabbit findings.
• ⬜ My PR is ready for review.

0/4 boxes ticked.

This PR stays in draft until every box above is ticked.

Hygiene

✅ Deterministic PR hygiene checks passed.

[lidge-jun]

리뷰 · 우선순위 47 / 80

이 PR은 tests/cli/cli-status-json.test.ts의 stale-port status fixture만 고칩니다. 지금 dev HEAD c15662855에서는 beforeAll로 한 번 잡은 freePort를 여러 테스트가 재사용합니다. occupied listener가 그 포트를 다시 bind하면, “거절되어야 하는 runtime-port”와 “일부러 점유한 configured-port”가 같아져 fixture가 조용히 뒤집힐 수 있습니다. 이 변경은 allocateFreePort()를 beforeEach마다 돌리고, occupied를 bind한 뒤에 recordedPort를 다시 할당하며, 두 포트가 다르다고 assert합니다. 제품 CLI/status 코드는 안 건드립니다. types/config 분할과 무관합니다.

라인 tests/cli/cli-status-json.test.ts · allocateFreePort - listen(0)으로 ephemeral port를 잡고 바로 close합니다. 공유 freePort 전역을 없앱니다.
라인 tests/cli/cli-status-json.test.ts · beforeEach - 테스트마다 새 freePort를 받습니다. 이전 케이스가 포트를 반납한 뒤의 재사용 독을 끊습니다.
라인 tests/cli/cli-status-json.test.ts · recordedPort after occupied - occupied bind 이후에 거절용 포트를 다시 고르고 not.toBe(occupiedPort)로 잠급니다. 이 assert가 없으면 레이스가 다시 숨어듭니다.
경로 본문 검증 - bun test tests/cli/cli-status-json.test.ts 47 passed를 보고했습니다. draft·readiness 미완입니다.

메인테이너의 판단이 필요한 지점

• 같은 beforeAll freePort 패턴이 다른 CLI 테스트에 있으면 후속으로 묶을지
• draft를 바로 머지 열에 태울지, 포커스 테스트 증거만으로 ready할지

너의 추천
fixture determinism 수정으로 방향이 맞습니다. 포커스 테스트가 초록이면 draft 해제 후 머지하세요. 런타임 동작 변경 없습니다.

이 댓글은 grok-bot이 작성했습니다

[lidge-jun]

Landed on dev in 164bd2b via #4053, which carried four reviewed changes as one sequential integration branch. Your commit is preserved in the squash body with a Co-authored-by trailer, so the contribution stays attributed to you.

Verification at the merged head e52116316: Cross-platform CI run 34265891731 finished with 19 jobs successful and 0 failures, bun x tsc --noEmit exit 0, and the changed test files pass. An independent review confirmed the stacked tree is byte-identical to the union of the carried diffs.

Closing this PR because the change is already on dev. Thank you.

[lidge-jun]

Landed via #4053 at 164bd2b