Skip to content

Clarify required email-linked setup - #3

Merged
parasdoshicom merged 1 commit into
mainfrom
codex/clarify-linked-setup
Sep 8, 2026
Merged

parasdoshicom merged 1 commit into
mainfrom
codex/clarify-linked-setup

Conversation

@parasdoshicom

Copy link
Copy Markdown
Owner

The public setup copy still exposed legacy interactive linking commands, and Claude startup text could tell an unlinked user that analysis would continue. This patch makes the dashboard-generated email-linked command the only documented setup path, makes the startup instruction stop before analysis when unlinked, and names the exact JSON link field for each client.

Validation:

  • python3 -m unittest discover -s tests (148 tests)
  • python3 scripts/validate.py
  • git diff --check

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 8, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-08T18:24:52.140475Z 25df7d5 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@parasdoshicom
parasdoshicom merged commit 50f7364 into main Sep 8, 2026
6 checks passed
@parasdoshicom
parasdoshicom deleted the codex/clarify-linked-setup branch September 8, 2026 18:23
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