Skip to content

feat(cli): add /model --compaction for configurable chat compression model #6019

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Rajeshwaran-R wants to merge 13 commits into from
Closed

feat(cli): add /model --compaction for configurable chat compression model #6019

Show file tree
Hide file tree
Changes from 11 commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
15dd25b
fix(core): use fast model for chat compression side queries
rajeshwaran-r-19885 Jun 28, 2026
2d65e71
feat(cli): add /model --compaction for configurable chat compression …
rajeshwaran-r-19885 Jun 29, 2026
e295f31
feat(cli): merge main into /model --compaction branch
qwen-code-dev-bot Jul 3, 2026
25b8b3a
feat(cli): merge main into /model --compaction branch
qwen-code-dev-bot Jul 6, 2026
20468b2
feat(cli): merge main into /model --compaction branch
qwen-code-dev-bot Jul 7, 2026
8a7bd15
Merge branch 'main' into main
wenshao Jul 7, 2026
9bf7fc3
chore: resolve merge conflicts with origin/main
qwen-code-dev-bot Jul 9, 2026
de94523
Merge branch 'main' into main
wenshao Jul 9, 2026
4bd3c53
Merge branch 'main' into main
wenshao Jul 10, 2026
00b481a
Merge branch 'main' into main
wenshao Jul 10, 2026
6f287e3
Merge branch 'main' into main
wenshao Jul 11, 2026
e2adc66
Merge branch 'main' into main
wenshao Jul 12, 2026
fec830f
chore: resolve merge conflicts with origin/main
qwen-code-dev-bot Sep 6, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions packages/cli/src/config/config.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2203,6 +2203,7 @@ export async function loadCliConfig(
memoryAgentTimeoutMinutes: settings.memory?.agentTimeoutMinutes,
fastModel: settings.fastModel || undefined,
visionModel: settings.visionModel || undefined,
compactionModel: settings.compactionModel || undefined,
visionBridgeTimeoutMs: settings.visionBridgeTimeoutMs,
modelFallbacks: resolveModelFallbacks(
argv.fallbackModel,
Expand Down
11 changes: 11 additions & 0 deletions packages/cli/src/config/settingsSchema.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1318,6 +1318,17 @@ const SETTINGS_SCHEMA = {
showInDialog: false,
},

compactionModel: {

@doudouOUC doudouOUC Sep 10, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This adds a compactionModel setting, but one already exists in main (added via PR #7818 at line ~1448). The diff creates a duplicate key — JS/TS object literal last-wins semantics means the added block overwrites the existing correct one with an outdated description ("fall back to fastModel or the main model" vs main's "fall back to the main model"). This will:

  • Override the correct schema entry with a stale description
  • Fail the schema-freshness gate at .github/scripts/check-settings-schema.sh
  • Be flagged by ESLint no-dupe-keys

The feature this PR originally introduced (/model --compaction) already landed on main via PR #7818. The remaining diff is solely this duplicate key — a side effect of git's auto-merge producing a second entry that conflict resolution did not catch.

Fix: delete this block from the PR branch.

type: 'string',
label: 'Compaction Model',
category: 'Model',
requiresRestart: false,
default: '',
description:
'Model used for chat compression (auto-compaction). Set with /model --compaction. Leave empty to fall back to fastModel or the main model. A smaller/faster model reduces compression latency and cost.',
showInDialog: false,
},

model: {
type: 'object',
label: 'Model',
Expand Down
3 changes: 3 additions & 0 deletions packages/cli/src/ui/AppContainer.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1280,6 +1280,7 @@ export const AppContainer = (props: AppContainerProps) => {
isFastModelMode,
isVoiceModelMode,
isVisionModelMode,
isCompactionModelMode,
modelDialogPersistScope,
openModelDialog,
closeModelDialog,
Expand Down Expand Up @@ -3967,6 +3968,7 @@ export const AppContainer = (props: AppContainerProps) => {
isFastModelMode,
isVoiceModelMode,
isVisionModelMode,
isCompactionModelMode,
modelDialogPersistScope,
isTrustDialogOpen,
activeArenaDialog,
Expand Down Expand Up @@ -4110,6 +4112,7 @@ export const AppContainer = (props: AppContainerProps) => {
isFastModelMode,
isVoiceModelMode,
isVisionModelMode,
isCompactionModelMode,
modelDialogPersistScope,
isTrustDialogOpen,
activeArenaDialog,
Expand Down
2 changes: 1 addition & 1 deletion packages/cli/src/ui/commands/modelCommand.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ describe('modelCommand', () => {
it('should have the correct name and description', () => {
expect(modelCommand.name).toBe('model');
expect(modelCommand.description).toBe(
'Switch the model for this session (--fast for suggestion model, --voice for voice transcription model, --vision for the vision bridge model, --project to persist to project settings, --global to persist to user settings, [model-id] to switch immediately, or [model-id] [prompt] to run a one-off prompt on another model; the inline prompt is sent verbatim without @file expansion).',
'Switch the model for this session (--fast for suggestion model, --voice for voice transcription model, --vision for the vision bridge model, --compaction for chat compression model, --project to persist to project settings, --global to persist to user settings, [model-id] to switch immediately, or [model-id] [prompt] to run a one-off prompt on another model; the inline prompt is sent verbatim without @file expansion).',
);
});

Expand Down
122 changes: 113 additions & 9 deletions packages/cli/src/ui/commands/modelCommand.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,9 @@ const MAIN_MODEL_CONFIGURATION_HINT =
const FAST_MODEL_CONFIGURATION_HINT =
'Configure models in settings.modelProviders and ensure the required environment variables are set. In interactive mode, run /auth to configure or switch providers, or run /model --fast without a model to choose from configured models.';

const COMPACTION_MODEL_CONFIGURATION_HINT =
'Configure models in settings.modelProviders and ensure the required environment variables are set. In interactive mode, run /auth to configure or switch providers, or run /model --compaction without a model to choose from configured models.';

const VISION_MODEL_CONFIGURATION_HINT =
'Configure an image-capable model in settings.modelProviders and ensure the required environment variables are set. Run /model --vision <model-id> to set it, or leave it unset to auto-pick a same-provider vision model.';

Expand Down Expand Up @@ -143,7 +146,7 @@ async function switchMainModel(
}

function formatUnavailableModelMessage(
kind: 'Model' | 'Fast model' | 'Vision model',
kind: 'Model' | 'Fast model' | 'Vision model' | 'Compaction model',
modelName: string,
authType: AuthType,
availableModels: AvailableModel[],
Expand All @@ -161,7 +164,9 @@ function formatUnavailableModelMessage(
? FAST_MODEL_CONFIGURATION_HINT
: kind === 'Vision model'
? VISION_MODEL_CONFIGURATION_HINT
: MAIN_MODEL_CONFIGURATION_HINT;
: kind === 'Compaction model'
? COMPACTION_MODEL_CONFIGURATION_HINT
: MAIN_MODEL_CONFIGURATION_HINT;

return (
`${kind} '${modelName}' is not available for auth type '${authType}'.\n` +
Expand All @@ -170,10 +175,10 @@ function formatUnavailableModelMessage(
);
}

// Fast and vision share the same "not configured for any auth type" message
// Fast, vision, and compaction share the same "not configured for any auth type" message
// shape, differing only in the label and the configuration hint.
function formatUnavailableAuxModelMessage(
label: 'Fast model' | 'Vision model',
label: 'Fast model' | 'Vision model' | 'Compaction model',
modelName: string,
availableModels: AvailableModel[],
hint: string,
Expand Down Expand Up @@ -217,6 +222,18 @@ function formatUnavailableVisionModelMessage(
);
}

function formatUnavailableCompactionModelMessage(
modelName: string,
availableModels: AvailableModel[],
): string {
return formatUnavailableAuxModelMessage(
'Compaction model',
modelName,
availableModels,
COMPACTION_MODEL_CONFIGURATION_HINT,
);
}

function formatAmbiguousVisionModelMessage(
modelName: string,
matchingModels: AvailableModel[],
Expand Down Expand Up @@ -294,15 +311,15 @@ function formatUnavailableVoiceModelMessage(
// Get an array of the available model IDs as strings, filtered by mode
function getAvailableModelIds(
context: CommandContext,
mode: 'main' | 'fast' | 'voice' | 'vision' = 'main',
mode: 'main' | 'fast' | 'voice' | 'vision' | 'compaction' = 'main',
) {
const { services } = context;
const { config } = services;
if (!config) {
return [];
}
const availableModels = config.getAvailableModels().filter((m) => {
if (mode === 'fast') return !m.voiceOnly;
if (mode === 'fast' || mode === 'compaction') return !m.voiceOnly;
if (mode === 'voice') return !m.fastOnly;
// 'vision' and 'main' both exclude fast/voice-only models.
return !m.fastOnly && !m.voiceOnly;
Expand All @@ -315,11 +332,11 @@ export const modelCommand: SlashCommand = {
completionPriority: 100,
get description() {
return t(
'Switch the model for this session (--fast for suggestion model, --voice for voice transcription model, --vision for the vision bridge model, --project to persist to project settings, --global to persist to user settings, [model-id] to switch immediately, or [model-id] [prompt] to run a one-off prompt on another model; the inline prompt is sent verbatim without @file expansion).',
'Switch the model for this session (--fast for suggestion model, --voice for voice transcription model, --vision for the vision bridge model, --compaction for chat compression model, --project to persist to project settings, --global to persist to user settings, [model-id] to switch immediately, or [model-id] [prompt] to run a one-off prompt on another model; the inline prompt is sent verbatim without @file expansion).',
);
},
argumentHint:
'[--fast|--voice|--vision] [--project|--global] [<model-id>] | <model-id> <prompt>',
'[--fast|--voice|--vision|--compaction] [--project|--global] [<model-id>] | <model-id> <prompt>',
kind: CommandKind.BUILT_IN,
supportedModes: ['interactive', 'non_interactive', 'acp'] as const,
completion: async (context, partialArg) => {
Expand All @@ -341,6 +358,12 @@ export const modelCommand: SlashCommand = {
'Set the image-capable model used to transcribe images for a text-only main model',
),
},
{
value: '--compaction',
description: t(
'Set the model used for chat compression (auto-compaction)',
),
},
{
value: '--project',
description: t(
Expand All @@ -359,18 +382,20 @@ export const modelCommand: SlashCommand = {
}
const trimmed = partialArg.trim();
if (trimmed) {
let mode: 'main' | 'fast' | 'voice' | 'vision' = 'main';
let mode: 'main' | 'fast' | 'voice' | 'vision' | 'compaction' = 'main';
// Strip all known flags to isolate the model prefix for completion
const modelPrefix = trimmed
.replace(/(?:^|\s)--fast(?:\s|$)/, ' ')
.replace(/(?:^|\s)--voice(?:\s|$)/, ' ')
.replace(/(?:^|\s)--vision(?:\s|$)/, ' ')
.replace(/(?:^|\s)--compaction(?:\s|$)/, ' ')
.replace(/(?:^|\s)--project(?:\s|$)/, ' ')
.replace(/(?:^|\s)--global(?:\s|$)/, ' ')
.trim();
if (/(?:^|\s)--fast(?:\s|$)/.test(trimmed)) mode = 'fast';
else if (/(?:^|\s)--voice(?:\s|$)/.test(trimmed)) mode = 'voice';
else if (/(?:^|\s)--vision(?:\s|$)/.test(trimmed)) mode = 'vision';
else if (/(?:^|\s)--compaction(?:\s|$)/.test(trimmed)) mode = 'compaction';
return getAvailableModelIds(context, mode).filter((id) =>
id.startsWith(modelPrefix),
);
Expand Down Expand Up @@ -711,6 +736,85 @@ export const modelCommand: SlashCommand = {
};
}

const isCompactionModelCommand =
args === '--compaction' || args.startsWith('--compaction ');
if (isCompactionModelCommand) {
Comment on lines +820 to +822
Comment on lines +820 to +822
const modelName = args.replace('--compaction', '').trim();
Comment on lines +820 to +823
if (!modelName) {
if (context.executionMode !== 'interactive') {
Comment on lines +820 to +841
const compactionModel =
context.services.settings?.merged?.compactionModel?.trim() ||
t('not set (falls back to fast model, then main model)');
return {
type: 'message',
messageType: 'info',
content: t(

@wenshao wenshao Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

[Suggestion] Misleading "clear" instruction — no clear mechanism exists

This message tells users to use "/model --compaction " to clear the override, but when an empty/whitespace model name is passed, the code only shows the info message or opens the dialog — it never calls setCompactionModel(undefined) or persistSetting(settings, 'compactionModel', '').

Either implement the clear path (mirroring how --fast could work), or remove the false instruction from this message.

— qwen3.7-max via Qwen Code /review

'Current compaction model: {{compactionModel}}\nUse "/model --compaction <model-id>" to set compaction model, or "/model --compaction " to clear the override.',
{ compactionModel },
),
Comment on lines +848 to +851
Comment on lines +848 to +851
Comment on lines +848 to +851
};
}
return {
type: 'dialog',
dialog: 'compaction-model',
};
Comment on lines +854 to +858
Comment on lines +854 to +858
Comment on lines +854 to +858
Comment on lines +854 to +858
}
Comment on lines +854 to +859
Comment on lines +854 to +859
if (!settings) {
return {
type: 'message',
messageType: 'error',
content: t('Settings service not available.'),
};
}

const selector = (() => {
try {
return resolveModelId(modelName);
} catch {
return undefined;
}
})();
if (!selector) {
return {
type: 'message',
messageType: 'error',
content: formatUnavailableCompactionModelMessage(modelName, []),
};
}

const availableModels = (
selector.authType
? config.getAvailableModelsForAuthType(selector.authType)
: config.getAllConfiguredModels()
).filter((m) => !m.voiceOnly);
if (!availableModels.some((model) => model.id === selector.modelId)) {
Comment on lines +883 to +890
return {
type: 'message',
messageType: 'error',
content: selector.authType
? formatUnavailableModelMessage(
'Compaction model',
selector.modelId,
selector.authType,
availableModels,
)
: formatUnavailableCompactionModelMessage(
modelName,
availableModels,
),
};
}

persistSetting(settings, 'compactionModel', modelName);

@doudouOUC doudouOUC Jun 29, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

[Suggestion] Zero test coverage for /model --compaction command handler

85+ lines of new command handling code with no tests. The existing --fast and --vision flags have extensive test coverage that should be mirrored here. Missing tests for:

  • Setting a valid compaction model: /model --compaction <model-id>
  • Rejecting unavailable compaction models (both bare and auth-qualified)
  • Non-interactive mode showing current compaction model status
  • Dialog action when /model --compaction is run with no args in interactive mode
  • Config.getCompactionModel() / setCompactionModel() / resolveCompactionModelSelector() methods
  • isCompactionModelMode state management in useModelCommand

— qwen3.7-max via Qwen Code /review

// Sync runtime Config so the compression service picks it up immediately.
config.setCompactionModel(modelName);
return {
type: 'message',
messageType: 'info',
content: t('Compaction Model') + ': ' + modelName,
};
}

const contentGeneratorConfig = config.getContentGeneratorConfig();
if (!contentGeneratorConfig) {
return {
Expand Down
1 change: 1 addition & 0 deletions packages/cli/src/ui/commands/types.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -189,6 +189,7 @@ export interface OpenDialogActionReturn {
| 'fast-model'
| 'voice-model'
| 'vision-model'
| 'compaction-model'
| 'subagent_create'
| 'subagent_list'
| 'skills_manage'
Expand Down
1 change: 1 addition & 0 deletions packages/cli/src/ui/components/DialogManager.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -272,6 +272,7 @@ export const DialogManager = ({
isFastModelMode={uiState.isFastModelMode}
isVoiceModelMode={uiState.isVoiceModelMode}
isVisionModelMode={uiState.isVisionModelMode}
isCompactionModelMode={uiState.isCompactionModelMode}
persistScope={uiState.modelDialogPersistScope}
availableTerminalHeight={listDialogHeight}
/>
Expand Down
22 changes: 18 additions & 4 deletions packages/cli/src/ui/components/ModelDialog.tsx
View file
Open in desktop

@DragonnZhang DragonnZhang Jun 29, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

[Critical] The newly added filter condition (isCompactionModelMode || !m.visionOnly) silently prevents vision-only models from appearing in the /model --vision dialog.

availableModelEntries filters registry models with two back-to-back conditions:

(isVisionModelMode || !m.visionOnly) &&   // ← existing line: shows all models in vision mode
(isCompactionModelMode || !m.visionOnly), // ← new line added by this PR

Both conditions are ANDed. When isVisionModelMode = true and isCompactionModelMode = false (i.e., the user runs /model --vision):

  • Condition 1 evaluates to true || !m.visionOnly = true (correct — include vision-only models)
  • Condition 2 evaluates to false || !m.visionOnly = !m.visionOnly (wrong — excludes vision-only models)

The combined AND result is !m.visionOnly, so vision-only models can never appear in the selection list, defeating the purpose of condition 1. The fix is either to remove the new line (since condition 1 already handles the vision filter) or to add isVisionModelMode to the new guard:

(isCompactionModelMode || isVisionModelMode || !m.visionOnly),

— claude-sonnet-4-6 via Qwen Code /review

Generated by Claude Code

@DragonnZhang DragonnZhang Jun 29, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

[Critical] handleSelect has no isCompactionModelMode branch. Selecting a model while in compaction mode falls through to config.switchModel(), which switches the primary chat model instead of persisting a compaction model.

The fast/voice/vision handlers each early-return with mode-specific persistence (settings.setValue(scope, 'fastModel', ...), 'visionModel', etc.). The compaction mode is missing an equivalent:

if (isCompactionModelMode) {
  const compactionModel = encodeAuxModelSelector(selected);
  const scope = getPersistScopeForModelSelection(settings);
  settings.setValue(scope, 'compactionModel', compactionModel);
  config?.setCompactionModel?.(compactionModel);
  uiState?.historyManager.addItem({ type: 'success', text: `Compaction Model: ${compactionModel}` }, Date.now());
  onClose();
  return;
}

Without this, /model --compaction + Enter overwrites the user's primary model setting, which is a destructive side effect of what should be a narrowly-scoped compaction model selection.

— claude-sonnet-4-6 via Qwen Code /review

Generated by Claude Code

Original file line number Diff line number Diff line change
Expand Up @@ -116,6 +116,7 @@ interface ModelDialogProps {
isFastModelMode?: boolean;
isVoiceModelMode?: boolean;
isVisionModelMode?: boolean;
isCompactionModelMode?: boolean;
/** Override which settings scope to persist the selection to. */
persistScope?: 'workspace' | 'user';
availableTerminalHeight?: number;
Expand Down Expand Up @@ -285,6 +286,7 @@ export function ModelDialog({
isFastModelMode,
isVoiceModelMode,
isVisionModelMode,
isCompactionModelMode,

@wenshao wenshao Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

[Suggestion] Three isCompactionModelMode integration gaps in this component

isCompactionModelMode is destructured here but not used in several places where the other mode flags are checked:

  1. Dialog title (~line 790): No isCompactionModelMode branch in the title ternary — falls through to generic "Select Model" instead of "Select Compaction Model".

  2. Left-arrow escape (~line 491): useKeypress handler checks isFastModelMode || isVoiceModelMode || isVisionModelMode but omits isCompactionModelMode — left arrow won't dismiss the compaction dialog.

  3. Preferred entry (~line 430-483): No preferredCompactionModelEntry computed — dialog opens with cursor at the wrong position instead of highlighting the currently configured compaction model.

Each should mirror the pattern used by isFastModelMode/isVisionModelMode.

— qwen3.7-max via Qwen Code /review

persistScope,
Comment on lines 295 to 300
availableTerminalHeight,
}: ModelDialogProps): React.JSX.Element {

@doudouOUC doudouOUC Jun 29, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

[Critical] handleSelect is missing an isCompactionModelMode handler block

The callback has handlers for isVoiceModelMode, isFastModelMode, and isVisionModelMode but no if (isCompactionModelMode) branch. When a user opens the compaction model picker via /model --compaction (no model-id) and selects a model from the dialog, the code falls through to the main model selection path (config.switchModel(...)), silently changing the user's primary model instead of setting the compaction model.

Add an if (isCompactionModelMode) block mirroring the isFastModelMode pattern:

  1. Encode the selector via encodeAuxModelSelector(selected)
  2. Persist via settings.setValue(scope, 'compactionModel', fastModel)
  3. Call config?.setCompactionModel(fastModel)
  4. Add a history item and return

Also add isCompactionModelMode to the handleSelect dependency array (currently missing — the callback won't re-create when compaction mode activates, causing a stale closure).

— qwen3.7-max via Qwen Code /review

Expand All @@ -309,7 +311,9 @@ export function ModelDialog({
(m.authType !== AuthType.QWEN_OAUTH ||
authType === AuthType.QWEN_OAUTH) &&
(isFastModelMode || !m.fastOnly) &&
(isVoiceModelMode || !m.voiceOnly),
(isVoiceModelMode || !m.voiceOnly) &&
(isVisionModelMode || !m.visionOnly) &&
(isCompactionModelMode || !m.visionOnly),

@doudouOUC doudouOUC Jun 29, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

[Critical] Copy-paste bug: (isCompactionModelMode || !m.visionOnly) should be (isCompactionModelMode || !m.voiceOnly)

The compaction filter reuses !m.visionOnly from the vision line above. But getAvailableModelIds (line 219) and the CLI validation (line 613) both use !m.voiceOnly for compaction mode (same as fast mode). The dialog and CLI will show different model sets: the dialog excludes vision-only models (irrelevant to text compression) but includes voice-only models (unsuitable for compression).

Suggested change
(isCompactionModelMode || !m.visionOnly),
(isCompactionModelMode || !m.voiceOnly),

— qwen3.7-max via Qwen Code /review

);

// Group registry models by authType
Expand Down Expand Up @@ -363,7 +367,14 @@ export function ModelDialog({
}

return result;
}, [authType, config, isFastModelMode, isVoiceModelMode]);
}, [
authType,
config,
isFastModelMode,
isVoiceModelMode,
isVisionModelMode,
isCompactionModelMode,
]);

const MODEL_OPTIONS = useMemo(
() =>
Expand Down Expand Up @@ -488,8 +499,11 @@ export function ModelDialog({
// Check if current model is a runtime model
// Runtime snapshot ID is already in $runtime|${authType}|${modelId} format
const activeRuntimeSnapshot =
isFastModelMode || isVoiceModelMode || isVisionModelMode
? undefined // fast/voice/vision models are never runtime model selections
isFastModelMode ||
isVoiceModelMode ||
isVisionModelMode ||
isCompactionModelMode
? undefined // fast/voice/vision/compaction models are never runtime model selections
: config?.getActiveRuntimeModelSnapshot?.();
const currentBaseUrl = config
?.getModelsConfig()
Expand Down
1 change: 1 addition & 0 deletions packages/cli/src/ui/contexts/UIActionsContext.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,7 @@ export interface UIActions {
fastModelMode?: boolean;
voiceModelMode?: boolean;
visionModelMode?: boolean;
compactionModelMode?: boolean;
}) => void;
openArenaDialog: (type: Exclude<ArenaDialogType, null>) => void;
closeArenaDialog: () => void;
Expand Down
1 change: 1 addition & 0 deletions packages/cli/src/ui/contexts/UIStateContext.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,7 @@ export interface UIState {
isFastModelMode: boolean;
isVoiceModelMode: boolean;
isVisionModelMode: boolean;
isCompactionModelMode: boolean;
modelDialogPersistScope: 'workspace' | 'user' | undefined;
isTrustDialogOpen: boolean;
activeArenaDialog: ArenaDialogType;
Expand Down
4 changes: 4 additions & 0 deletions packages/cli/src/ui/hooks/slashCommandProcessor.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -134,6 +134,7 @@ export interface SlashCommandProcessorActions {
fastModelMode?: boolean;
voiceModelMode?: boolean;
visionModelMode?: boolean;
compactionModelMode?: boolean;
persistScope?: 'workspace' | 'user';
}) => void;
openTrustDialog: () => void;
Expand Down Expand Up @@ -1119,6 +1120,9 @@ export const useSlashCommandProcessor = (
persistScope: result.persistScope,
});
return { type: 'handled' };
case 'compaction-model':
actions.openModelDialog({ compactionModelMode: true });
return { type: 'handled' };
Comment on lines +1261 to +1272
Comment on lines +1261 to +1272
Comment on lines +1261 to +1272
Comment on lines +1261 to +1272
Comment on lines +1261 to +1272
Comment on lines +1261 to +1272
case 'trust':
actions.openTrustDialog();
return { type: 'handled' };
Expand Down
Loading
Loading