fix: assistants -> configs -> agents #8808
Merged
Conversation
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
dosubot
bot
added
the
size:M
|
✅ Review Complete Code Review Summary |
…allin/configs-assistants-cleanup-7
|
✅ Review Complete Code Review Summary |
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
5 issues found across 9 files
Prompt for AI agents (all 5 issues)
Understand the root cause of the following 5 issues and fix them.
<file name="docs/guides/cli.mdx">
<violation number="1" location="docs/guides/cli.mdx:57">
Update the example to reference the new fallback slug `continuedev/default-config` so the documentation matches the CLI behavior.</violation>
</file>
<file name="extensions/cli/src/configLoader.ts">
<violation number="1" location="extensions/cli/src/configLoader.ts:190">
The warning still says “falling back to default agent,” even though the changed code now loads the default config, leading to misleading logs.</violation>
<violation number="2" location="extensions/cli/src/configLoader.ts:375">
The function now fetches `continuedev/default-config`, but the thrown error still claims the default agent failed, resulting in an inaccurate error message.</violation>
</file>
<file name="extensions/cli/vitest.setup.ts">
<violation number="1" location="extensions/cli/vitest.setup.ts:17">
The fetch mock now watches for `default-cli-config`, but the CLI still requests `continuedev/default-config`, so the fallback request is no longer stubbed and tests will hit the network instead of the mock.</violation>
</file>
<file name="extensions/cli/spec/config-loading.md">
<violation number="1" location="extensions/cli/spec/config-loading.md:40">
The spec now claims the CLI falls back to `continuedev/default-cli-config`, but the implementation continues to load `continuedev/default-config`, so the documentation is incorrect.</violation>
</file>
Reply to cubic to teach it or ask questions. Re-run a review with @cubic-dev-ai review this PR
Patrick-Erichsen
previously approved these changes
Nov 20, 2025
Patrick-Erichsen
approved these changes
Nov 21, 2025
Contributor
|
🎉 This PR is included in version 1.6.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Contributor
|
🎉 This PR is included in version 1.36.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Description
More cases of "assistants" -> "configs" and migrating configs to agents. I created https://hub.continue.dev/continuedev/tidy-up-markdown-agent to make this work
Summary by cubic
Standardized terminology by switching from “assistants” to “configs” and updating workflows to use “agents.” Also changed the default fallback slug from continuedev/default-agent to continuedev/default-cli-config across CLI, docs, specs, and tests for consistency.
Refactors
Migration
Written for commit 011fdc4. Summary will update automatically on new commits.