feat: support Atlas Cloud environment configuration - #265
feat: support Atlas Cloud environment configuration#265binyangzhu000-sudo wants to merge 4 commits into
Conversation
Signed-off-by: binyangzhu000-sudo <224954946+binyangzhu000-sudo@users.noreply.github.com>
|
Resolved the merge conflict with current The resolution preserves both the Atlas Cloud settings tests and the new upstream context-window tests. The online PR diff remains limited to the original four core files. Validation:
The exact-head |
Three conflicts, all from upstream refactors this branch overlapped: - model-capabilities.ts: upstream replaced the NON_MULTIMODAL_MODELS set with supportsMultimodal(model, mode), inferring from "-vision". Took the new mechanism and dropped this branch's list, having first checked it has no remaining consumers. The Atlas model id stays in DEEPSEEK_V4_MODELS, which still drives the thinking-mode default. - index.ts: both sides added exports; kept both. - settings.ts: upstream hoisted baseURL into a local. Folded the Atlas default into that local instead of the return site, so baseURL now mirrors how model already handles useAtlasCloudDefaults. Signed-off-by: binyangzhu000-sudo <binyangzhu000@gmail.com>
|
Synced with
const baseURL =
trimString(env.BASE_URL) ||
(useAtlasCloudDefaults ? ATLASCLOUD_BASE_URL : "") ||
defaults.baseURL;Verification — |
Single conflict in model-capabilities.ts, where upstream added deepseek-flash to DEEPSEEK_V4_MODELS in the same row this branch adds the Atlas-prefixed deepseek-ai/deepseek-v4-pro. Both entries are kept. Checked the auto-merged side too: the ATLASCLOUD_API_KEY fallback in resolveSettingsSources survived intact, and ATLASCLOUD_DEFAULT_MODEL still names a model Atlas serves. npm run typecheck clean across workspaces. npm test -> 282/283, 370/374 and 59/59. The four failures (cli-args.test.ts, and three tool-permission/image cases in packages/core) reproduce identically on a clean upstream/main worktree, so they are pre-existing rather than from this branch.
|
Synced with I also checked the auto-merged side, since Verification on the merged branch:
The four failures are No conflicts remain. |
Summary
ATLASCLOUD_API_KEYas an opt-in credential when no explicitAPI_KEYis configuredDEEPCODE_*settings precedenceValidation
npm run generate && npm run checknpm test(596 passed, 1 skipped)createOpenAIClient()(123 models; default model present)createOpenAIClient()(non-empty response)git diff --checkNo README, public documentation, logo, dependency, or lockfile changes.