Skip to content

test: stabilize relay ops hello timeout#55

Merged
freshtechbro merged 1 commit into
mainfrom
codex/release-0-0-29-relay-handshake-fix
May 16, 2026
Merged

test: stabilize relay ops hello timeout#55
freshtechbro merged 1 commit into
mainfrom
codex/release-0-0-29-relay-handshake-fix

Conversation

@freshtechbro

Copy link
Copy Markdown
Owner

Summary

  • arm the ops error and close listeners before the shortened relay hello timeout can fire
  • preserves the explicit ops timeout error, 1011 close assertion, and readiness cleanup assertions

Testing

  • npm run test -- tests/relay-server.test.ts -t "fails silent ops hello handshakes explicitly and clears relay readiness"
  • npm run test -- tests/relay-server.test.ts
  • npm run lint
  • npm run typecheck
  • npm run test
  • npm run build
  • npm run extension:build
  • npm run version:check
  • node scripts/audit-zombie-files.mjs
  • node scripts/docs-drift-check.mjs
  • node scripts/chrome-store-compliance-check.mjs
  • ./skills/opendevbrowser-best-practices/scripts/validate-skill-assets.sh
  • NPM_CONFIG_CACHE=/tmp/odb-npx-cache-help-flag npx opendevbrowser --help
  • NPM_CONFIG_CACHE=/tmp/odb-npx-cache-help-command npx opendevbrowser help

Arm the ops error and close listeners immediately after the shortened hello-timeout path sends ops_hello, so CI cannot miss the explicit timeout frame while still asserting relay readiness clears and the socket closes.

Co-authored-by: Codex <noreply@openai.com>
@freshtechbro freshtechbro merged commit 996a0b1 into main May 16, 2026
8 checks passed
@freshtechbro freshtechbro deleted the codex/release-0-0-29-relay-handshake-fix branch May 16, 2026 14:08
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