Skip to content

fix: detect ChatGPT temporary chat state - #1

Draft
Alan-512 wants to merge 18 commits into
masterfrom
codex/cross-vendor-temporary-chat
Draft

fix: detect ChatGPT temporary chat state#1
Alan-512 wants to merge 18 commits into
masterfrom
codex/cross-vendor-temporary-chat

Conversation

@Alan-512

@Alan-512 Alan-512 commented Aug 11, 2026

Copy link
Copy Markdown
Owner

Summary

  • Recognize ChatGPT temporary mode from the active Turn off temporary chat toggle label.
  • Read conversationId/conversation_id query parameters when the temporary chat route does not expose a normal /c/ path.
  • Keep Turn on temporary chat inactive and add lifecycle regression coverage.

Root cause

The exporter only mounted when getActiveConversationId() returned an ID. The active ChatGPT temporary toggle could match the mode text but lacked the state attributes required by the detector, and the temporary conversation ID could remain only in the URL query.

Validation

  • node --test tests/content-ui-lifecycle.test.js (15 passed)
  • node --check content.js
  • node --check inject.js
  • node --check tests/content-ui-lifecycle.test.js
  • Manifest JSON/resource validation
  • git diff --check

@Alan-512 Alan-512 changed the title feat: support cross-vendor temporary chats fix: detect ChatGPT temporary chat state Aug 18, 2026
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