feat(setup): remove AI provider configuration step from onboarding wizard#863
feat(setup): remove AI provider configuration step from onboarding wizard#863
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2da0abd39f
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| {safeStepIndex === STEP.WELCOME && <WelcomeContent />} | ||
| {safeStepIndex === STEP.RUNTIME && <RuntimeContent onStatusChange={setRuntimeChecksPassed} />} | ||
| {safeStepIndex === STEP.PROVIDER && ( | ||
| <ProviderContent | ||
| providers={providers} | ||
| selectedProvider={selectedProvider} | ||
| onSelectProvider={setSelectedProvider} | ||
| apiKey={apiKey} | ||
| onApiKeyChange={setApiKey} | ||
| onConfiguredChange={setProviderConfigured} | ||
| /> | ||
| )} | ||
| {safeStepIndex === STEP.INSTALLING && ( | ||
| <InstallingContent |
There was a problem hiding this comment.
Update E2E coverage for onboarding flow change
/workspace/ClawX/AGENTS.md states that any user-visible UI change must include or update an Electron E2E spec in the same PR, but this commit changes the setup wizard flow by removing a step and only modifies src/pages/Setup/index.tsx. I checked the commit diff and no tests/e2e/*.spec.ts files were updated, so step progression and skip/navigation behavior introduced here can regress without automated detection.
Useful? React with 👍 / 👎.
| { | ||
| id: 'installing', | ||
| title: t('steps.installing.title'), |
There was a problem hiding this comment.
Sync setup docs after removing provider step
/workspace/ClawX/AGENTS.md requires doc sync for functional changes across README.md, README.zh-CN.md, and README.ja-JP.md, but this onboarding change was not mirrored there (for example, README.md still lists an AI Provider wizard step in the First Launch section). This leaves user-facing setup instructions inconsistent with the actual UI flow and will cause onboarding confusion.
Useful? React with 👍 / 👎.
…docs update (ValueCell-ai#863 ValueCell-ai#900 ValueCell-ai#901 ValueCell-ai#905 ValueCell-ai#910) Remove provider setup step, speed up initial loading, remove brave-web-search, update READMEs for LaTeX, add agent context files, strip first run sections.
No description provided.