Skip to content

Fix Codex readiness after ChatGPT login - #2

Merged
Vincent (vincent-ycw) merged 1 commit into
mainfrom
fix/chatgpt-readiness-race
Aug 21, 2026
Merged

Vincent (vincent-ycw) merged 1 commit into
mainfrom
fix/chatgpt-readiness-race

Conversation

@vincent-ycw

@vincent-ycw Vincent (vincent-ycw) commented Aug 21, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • wait for the authoritative account update after device login completion
  • transition connected state through the shared credential-validation path
  • prevent duplicate account updates from firing readiness callbacks twice
  • add regression coverage for the observed App Server event ordering

Verification

  • npm run typecheck
  • npm test (482 passed, 32 skipped)
  • npm run test:integration (27 passed, 30 skipped)
  • npm run test:chaos (28 passed)
  • npm run docs:check
  • git diff --check

Live verification

A fresh Render deployment should confirm Codex becomes ready after ChatGPT device login without requiring a service restart.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@coderabbitai

coderabbitai Bot commented Aug 21, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 0f73e0ad-9099-48f8-a95e-11677b38a78a


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

Comment @coderabbitai help to get the list of available commands.

@vincent-ycw
Vincent (vincent-ycw) merged commit f15f78a into main Aug 21, 2026
3 checks passed
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