Skip to content

Conversation

@joshka-oai
Copy link
Collaborator

@joshka-oai joshka-oai commented Jan 30, 2026

Problem

OpenAI employees were sent to the public GitHub issue flow after /feedback, which is the wrong follow-up path internally.

Mental model

After feedback upload completes, we render a follow-up link/message. That link should be audience-aware but must not change the upload pipeline itself.

Non-goals

  • Changing how feedback is captured or uploaded
  • Changing external user behavior

Tradeoffs

We detect employees via the authenticated account email suffix (@openai.com). If the email is unavailable (e.g., API key auth), we default to the external behavior.

Architecture

  • Introduce FeedbackAudience and thread it from App -> ChatWidget -> FeedbackNoteView
  • Gate internal messaging/links on FeedbackAudience::OpenAiEmployee
  • Internal follow-up link is now http://go/codex-feedback-internal
  • External GitHub URL remains byte-for-byte identical

Observability

No new telemetry; this only changes rendered follow-up instructions.

Tests

  • just fmt
  • cargo test -p codex-tui --lib

Introduce a FeedbackAudience enum derived from the authenticated account
email and thread it into the feedback follow-up view. When the account
email ends with @openai.com, route users to the internal
http://go/codex-feedback-internal link and suggest sharing
go/codex-feedback/<thread-id> with a short problem description.

External users keep the exact existing GitHub behavior, including the
prepopulated bug template URL and the follow-up hint to mention the
thread ID. Tests assert the external URL exactly and cover the employee
routing.

Testing: just fmt; cargo test -p codex-tui --lib
@joshka-oai joshka-oai enabled auto-merge (squash) January 30, 2026 02:01
@joshka-oai joshka-oai merged commit 36f2fe8 into main Jan 30, 2026
32 checks passed
@joshka-oai joshka-oai deleted the joshka/feedback-internal-link branch January 30, 2026 02:12
@github-actions github-actions bot locked and limited conversation to collaborators Jan 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants