Skip to content

fix(cli): respect existing config during onboarding#12260

Open
cyphercodes wants to merge 1 commit intocontinuedev:mainfrom
cyphercodes:fix-cli-onboarding-existing-config-12258
Open

fix(cli): respect existing config during onboarding#12260
cyphercodes wants to merge 1 commit intocontinuedev:mainfrom
cyphercodes:fix-cli-onboarding-existing-config-12258

Conversation

@cyphercodes
Copy link
Copy Markdown

@cyphercodes cyphercodes commented Apr 29, 2026

Description

Fixes #12258.

  • Skip CLI onboarding prompts when the default ~/.continue/config.yaml already exists, allowing the normal config loader to use an existing local config.
  • Mark onboarding complete after a successful explicit --config load so one-time config selection is remembered.
  • Add unit coverage for both local default config skipping and successful --config completion marking.

AI Code Review

  • Team members only: AI review runs automatically when PR is opened or marked ready for review
  • Team members can also trigger a review by commenting @continue-review

Checklist

  • I've read the contributing guide
  • The relevant docs, if any, have been updated or created
  • The relevant tests, if any, have been updated or created

Screen recording or screenshot

N/A — CLI onboarding behavior is covered by unit tests.

Tests

  • npm test -- src/onboarding.test.ts
  • npx prettier --check src/onboarding.ts src/onboarding.test.ts
  • npm run lint
  • npm run build
  • git diff --check

Summary by cubic

Skips CLI onboarding when a local config exists and persists successful --config loads so users aren't prompted again.

  • Bug Fixes
    • Skip interactive onboarding if ~/.continue/config.yaml exists; use the normal loader.
    • After a successful --config load, mark onboarding complete so the choice persists.

Written for commit adf565f. Summary will update on new commits. Review in cubic

@cyphercodes cyphercodes requested a review from a team as a code owner April 29, 2026 23:43
@cyphercodes cyphercodes requested review from sestinj and removed request for a team April 29, 2026 23:43
@dosubot dosubot Bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Apr 29, 2026
@github-actions
Copy link
Copy Markdown
Contributor


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

@cyphercodes cyphercodes force-pushed the fix-cli-onboarding-existing-config-12258 branch from de9eeef to adf565f Compare April 29, 2026 23:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S This PR changes 10-29 lines, ignoring generated files.

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

CLI: onboarding prompts even when ~/.continue/config.yaml already exists

1 participant