refactor(auth): Atlas is login/logout, Codex is connect/disconnect#118
Merged
Aayam Bansal (aayambansal) merged 1 commit intoJul 7, 2026
Merged
Conversation
Splits the two account types onto distinct, symmetric verb pairs to remove the long-standing ambiguity where both 'connect login' and 'login' meant the Atlas account: - Atlas (managed account, wallet, sync) keeps the top-level login / logout / status / sync / devices commands. The back-compat 'connect' umbrella that aliased them is removed. - Codex (ChatGPT subscription) now owns the connect / disconnect verbs. 'connect [codex]' signs in with ChatGPT (reusing runCodexAuthFlow); 'disconnect [codex]' is a NEW command that clears the local openai-codex OAuth credential and best-effort revokes it server-side — the missing counterpart to sign-in. 'keys signin' still works. Updates every user-facing string and prompt that told users to run 'openscience connect login' / 'connect sync' to the top-level 'openscience login' / 'sync' (the connect subcommands no longer exist), including the canonical initialize-atlas-graph SKILL.md so it stays in sync with the embedded copy.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Aayam Bansal (aayambansal)
deleted the
refactor/auth-atlas-login-codex-connect
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Splits the two account types onto distinct, symmetric verb pairs and removes the ambiguity where both
connect loginandloginmeant the Atlas account.Before
login/logout/status/sync/devicesand aconnectumbrella that re-aliased all of them (connect login,connect logout, …).keys signinto sign in, and no first-class disconnect.After
login/logout(plusstatus/sync/devices), all top-level. Theconnectumbrella is removed.connect/disconnect:openscience connect [codex]— sign in with a ChatGPT subscription (reusesrunCodexAuthFlow).openscience disconnect [codex]— new: clears the localopenai-codexOAuth credential and best-effort revokes it server-side (DELETE /api/keys/openai-codex) — the counterpart sign-out that was missing.keys signinstill works.[service]defaults tocodexand only acceptscodextoday, so bothconnectandconnect codexwork.Reference-string cleanup
Every user-facing string/prompt that told users to run
openscience connect login/connect syncnow points at the top-levelopenscience login/sync(theconnectsubcommands no longer exist) — including the error inprovider.ts, theskill.tshint,research.txt, theinitialize-atlas-graphskill, and the frontend dev helper. The canonicalinitialize-atlas-graph/SKILL.mdis updated too so the embedded-vs-canonical drift test stays green.Verification
typecheck+prettierclean; full backend suite (929 tests) green;--helpconfirmsconnect/disconnectare Codex andlogin/logout/sync/devicesremain Atlas.