mirror of
https://github.com/tiennm99/ccs.git
synced 2026-10-12 20:15:40 +00:00
fix(droid): add structural exec passthrough for ccsd
This commit is contained in:
1 parent
cfdf1d8718
commit
5963ba973e
5 files changed
+296
-18
No files matched your search
@@ -167,6 +167,16 @@ CCS also persists Droid's active model selector in `~/.factory/settings.json`
|
|||||||
(`model: custom:<alias>`). This avoids passing `-m` argv in interactive mode,
|
(`model: custom:<alias>`). This avoids passing `-m` argv in interactive mode,
|
||||||
which Droid treats as queued prompt text.
|
which Droid treats as queued prompt text.
|
||||||
|
|
||||||
|
CCS supports structural Droid command passthrough after profile selection:
|
||||||
|
|
||||||
|
```bash
|
||||||
|
ccsd codex exec --skip-permissions-unsafe "fix failing tests"
|
||||||
|
ccsd codex --skip-permissions-unsafe "fix failing tests" # auto-routed to: droid exec ...
|
||||||
|
```
|
||||||
|
|
||||||
|
If you pass exec-only flags without a prompt (for example `--skip-permissions-unsafe`),
|
||||||
|
Droid `exec` will return its native "No prompt provided" usage guidance.
|
||||||
|
|
||||||
Dashboard parity: `ccs config` -> `Factory Droid`
|
Dashboard parity: `ccs config` -> `Factory Droid`
|
||||||
|
|
||||||
### Per-Profile Target Defaults
|
### Per-Profile Target Defaults
|
||||||
@@ -187,6 +197,7 @@ Built-in CLIProxy providers also work with Droid alias/target override:
|
|||||||
ccsd codex
|
ccsd codex
|
||||||
ccsd agy
|
ccsd agy
|
||||||
ccs codex --target droid
|
ccs codex --target droid
|
||||||
|
ccsd codex exec --auto high "triage this bug report"
|
||||||
```
|
```
|
||||||
|
|
||||||
Dashboard parity:
|
Dashboard parity:
|
||||||
|
|||||||
+28
-18
@@ -62,6 +62,7 @@ import {
|
|||||||
DroidReasoningFlagError,
|
DroidReasoningFlagError,
|
||||||
resolveDroidReasoningRuntime,
|
resolveDroidReasoningRuntime,
|
||||||
} from './targets/droid-reasoning-runtime';
|
} from './targets/droid-reasoning-runtime';
|
||||||
|
import { routeDroidCommandArgs } from './targets/droid-command-router';
|
||||||
|
|
||||||
// Version and Update check utilities
|
// Version and Update check utilities
|
||||||
import { getVersion } from './utils/version';
|
import { getVersion } from './utils/version';
|
||||||
@@ -718,26 +719,35 @@ async function main(): Promise<void> {
|
|||||||
let targetRemainingArgs = remainingArgs;
|
let targetRemainingArgs = remainingArgs;
|
||||||
let droidReasoningOverride: string | number | undefined;
|
let droidReasoningOverride: string | number | undefined;
|
||||||
if (resolvedTarget === 'droid') {
|
if (resolvedTarget === 'droid') {
|
||||||
try {
|
const droidRoute = routeDroidCommandArgs(remainingArgs);
|
||||||
const runtime = resolveDroidReasoningRuntime(remainingArgs, process.env.CCS_THINKING);
|
targetRemainingArgs = droidRoute.argsForDroid;
|
||||||
targetRemainingArgs = runtime.argsWithoutReasoningFlags;
|
|
||||||
droidReasoningOverride = runtime.reasoningOverride;
|
|
||||||
|
|
||||||
if (runtime.duplicateDisplays.length > 0) {
|
if (droidRoute.mode === 'interactive') {
|
||||||
console.error(
|
try {
|
||||||
warn(
|
const runtime = resolveDroidReasoningRuntime(remainingArgs, process.env.CCS_THINKING);
|
||||||
`[!] Multiple reasoning flags detected. Using first occurrence: ${runtime.sourceDisplay || '<first-flag>'}`
|
targetRemainingArgs = runtime.argsWithoutReasoningFlags;
|
||||||
)
|
droidReasoningOverride = runtime.reasoningOverride;
|
||||||
);
|
|
||||||
|
if (runtime.duplicateDisplays.length > 0) {
|
||||||
|
console.error(
|
||||||
|
warn(
|
||||||
|
`[!] Multiple reasoning flags detected. Using first occurrence: ${runtime.sourceDisplay || '<first-flag>'}`
|
||||||
|
)
|
||||||
|
);
|
||||||
|
}
|
||||||
|
} catch (error) {
|
||||||
|
if (error instanceof DroidReasoningFlagError) {
|
||||||
|
console.error(fail(error.message));
|
||||||
|
console.error(' Examples: --thinking low, --thinking 8192, --thinking off');
|
||||||
|
console.error(' Codex alias: --effort medium|high|xhigh');
|
||||||
|
process.exit(1);
|
||||||
|
}
|
||||||
|
throw error;
|
||||||
}
|
}
|
||||||
} catch (error) {
|
} else if (droidRoute.autoPrependedExec && process.stdout.isTTY) {
|
||||||
if (error instanceof DroidReasoningFlagError) {
|
console.error(
|
||||||
console.error(fail(error.message));
|
info('Detected Droid exec-only flags. Routing as: droid exec <flags> [prompt]')
|
||||||
console.error(' Examples: --thinking low, --thinking 8192, --thinking off');
|
);
|
||||||
console.error(' Codex alias: --effort medium|high|xhigh');
|
|
||||||
process.exit(1);
|
|
||||||
}
|
|
||||||
throw error;
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -338,6 +338,14 @@ Run ${color('ccs config', 'command')} for web dashboard`.trim();
|
|||||||
['ccsd glm', 'Same as above (alias)'],
|
['ccsd glm', 'Same as above (alias)'],
|
||||||
['ccsd codex', 'Run built-in CLIProxy Codex profile on Droid'],
|
['ccsd codex', 'Run built-in CLIProxy Codex profile on Droid'],
|
||||||
['ccsd agy', 'Run built-in CLIProxy Antigravity profile on Droid'],
|
['ccsd agy', 'Run built-in CLIProxy Antigravity profile on Droid'],
|
||||||
|
[
|
||||||
|
'ccsd codex exec --skip-permissions-unsafe "fix failing tests"',
|
||||||
|
'Pass through Droid exec mode',
|
||||||
|
],
|
||||||
|
[
|
||||||
|
'ccsd codex --skip-permissions-unsafe "fix failing tests"',
|
||||||
|
'Auto-routes to Droid exec when exec-only flags are detected',
|
||||||
|
],
|
||||||
[
|
[
|
||||||
'ccs cliproxy create my-codex --provider codex --target droid',
|
'ccs cliproxy create my-codex --provider codex --target droid',
|
||||||
'Create CLIProxy variant with Droid as default target',
|
'Create CLIProxy variant with Droid as default target',
|
||||||
@@ -393,6 +401,10 @@ Run ${color('ccs config', 'command')} for web dashboard`.trim();
|
|||||||
['--effort <level>', 'Codex alias for reasoning effort (medium/high/xhigh)'],
|
['--effort <level>', 'Codex alias for reasoning effort (medium/high/xhigh)'],
|
||||||
['--effort xhigh', 'Pin Codex effort to xhigh for this run'],
|
['--effort xhigh', 'Pin Codex effort to xhigh for this run'],
|
||||||
['', ''],
|
['', ''],
|
||||||
|
['Droid exec:', 'Use native Droid flag: --reasoning-effort <level>'],
|
||||||
|
['', 'CCS auto-maps --thinking/--effort to --reasoning-effort in droid exec mode.'],
|
||||||
|
['', 'For interactive droid sessions, CCS applies reasoning via Droid BYOK model config.'],
|
||||||
|
['', ''],
|
||||||
['Note:', 'Extended thinking allocates compute for step-by-step reasoning'],
|
['Note:', 'Extended thinking allocates compute for step-by-step reasoning'],
|
||||||
['', 'before responding.'],
|
['', 'before responding.'],
|
||||||
['', 'Providers: agy/gemini use --thinking, codex uses --effort (or --thinking alias).'],
|
['', 'Providers: agy/gemini use --thinking, codex uses --effort (or --thinking alias).'],
|
||||||
|
|||||||
@@ -0,0 +1,159 @@
|
|||||||
|
/**
|
||||||
|
* Droid command router
|
||||||
|
*
|
||||||
|
* Determines whether profile args should launch Droid interactive mode
|
||||||
|
* (`droid [prompt...]`) or command mode (`droid <subcommand> ...`).
|
||||||
|
*
|
||||||
|
* Also normalizes CCS legacy reasoning aliases for `droid exec`:
|
||||||
|
* - --effort / --thinking -> --reasoning-effort
|
||||||
|
*/
|
||||||
|
|
||||||
|
export type DroidCommandMode = 'interactive' | 'command';
|
||||||
|
|
||||||
|
export interface DroidCommandRoute {
|
||||||
|
mode: DroidCommandMode;
|
||||||
|
argsForDroid: string[];
|
||||||
|
command?: string;
|
||||||
|
autoPrependedExec: boolean;
|
||||||
|
}
|
||||||
|
|
||||||
|
const DROID_SUBCOMMANDS = new Set([
|
||||||
|
'exec',
|
||||||
|
'mcp',
|
||||||
|
'plugin',
|
||||||
|
'daemon',
|
||||||
|
'search',
|
||||||
|
'find',
|
||||||
|
'ssh',
|
||||||
|
'computer',
|
||||||
|
'update',
|
||||||
|
'help',
|
||||||
|
]);
|
||||||
|
|
||||||
|
// Exec-only long flags from Factory Droid CLI help.
|
||||||
|
const DROID_EXEC_ONLY_LONG_FLAGS = new Set([
|
||||||
|
'--output-format',
|
||||||
|
'--input-format',
|
||||||
|
'--file',
|
||||||
|
'--auto',
|
||||||
|
'--skip-permissions-unsafe',
|
||||||
|
'--session-id',
|
||||||
|
'--model',
|
||||||
|
'--reasoning-effort',
|
||||||
|
'--enabled-tools',
|
||||||
|
'--disabled-tools',
|
||||||
|
'--cwd',
|
||||||
|
'--tag',
|
||||||
|
'--log-group-id',
|
||||||
|
'--list-tools',
|
||||||
|
]);
|
||||||
|
|
||||||
|
function getLongFlagToken(arg: string): string {
|
||||||
|
const eqIndex = arg.indexOf('=');
|
||||||
|
return eqIndex >= 0 ? arg.slice(0, eqIndex) : arg;
|
||||||
|
}
|
||||||
|
|
||||||
|
function isExplicitSubcommand(arg: string | undefined): boolean {
|
||||||
|
return !!arg && DROID_SUBCOMMANDS.has(arg);
|
||||||
|
}
|
||||||
|
|
||||||
|
function hasExecOnlyFlagsAtFront(args: string[]): boolean {
|
||||||
|
for (let i = 0; i < args.length; i++) {
|
||||||
|
const arg = args[i];
|
||||||
|
if (arg === '--') return false;
|
||||||
|
|
||||||
|
// CCS legacy aliases may appear before exec-only flags; skip their values when present.
|
||||||
|
if (arg === '--effort' || arg === '--thinking') {
|
||||||
|
const possibleValue = args[i + 1];
|
||||||
|
if (possibleValue && !possibleValue.startsWith('-')) {
|
||||||
|
i += 1;
|
||||||
|
}
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
if (arg.startsWith('--effort=') || arg.startsWith('--thinking=')) {
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!arg.startsWith('-')) return false;
|
||||||
|
if (!arg.startsWith('--')) continue; // short flags are ambiguous at root (`-r` is resume)
|
||||||
|
|
||||||
|
const flagToken = getLongFlagToken(arg);
|
||||||
|
if (DROID_EXEC_ONLY_LONG_FLAGS.has(flagToken)) {
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
function normalizeLegacyReasoningAliasesForExec(args: string[]): string[] {
|
||||||
|
const normalized: string[] = [];
|
||||||
|
|
||||||
|
for (let i = 0; i < args.length; i++) {
|
||||||
|
const arg = args[i];
|
||||||
|
|
||||||
|
if (arg === '--effort' || arg === '--thinking') {
|
||||||
|
const value = args[i + 1];
|
||||||
|
if (value && !value.startsWith('-')) {
|
||||||
|
normalized.push('--reasoning-effort', value);
|
||||||
|
i += 1;
|
||||||
|
} else {
|
||||||
|
// Keep invalid/missing-value form so Droid can surface native validation.
|
||||||
|
normalized.push(arg);
|
||||||
|
}
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (arg.startsWith('--effort=')) {
|
||||||
|
normalized.push(`--reasoning-effort=${arg.slice('--effort='.length)}`);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (arg.startsWith('--thinking=')) {
|
||||||
|
normalized.push(`--reasoning-effort=${arg.slice('--thinking='.length)}`);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
normalized.push(arg);
|
||||||
|
}
|
||||||
|
|
||||||
|
return normalized;
|
||||||
|
}
|
||||||
|
|
||||||
|
export function routeDroidCommandArgs(args: string[]): DroidCommandRoute {
|
||||||
|
if (args.length === 0) {
|
||||||
|
return {
|
||||||
|
mode: 'interactive',
|
||||||
|
argsForDroid: [],
|
||||||
|
autoPrependedExec: false,
|
||||||
|
};
|
||||||
|
}
|
||||||
|
|
||||||
|
if (isExplicitSubcommand(args[0])) {
|
||||||
|
const command = args[0];
|
||||||
|
const argsForDroid =
|
||||||
|
command === 'exec' ? normalizeLegacyReasoningAliasesForExec(args) : [...args];
|
||||||
|
return {
|
||||||
|
mode: 'command',
|
||||||
|
command,
|
||||||
|
argsForDroid,
|
||||||
|
autoPrependedExec: false,
|
||||||
|
};
|
||||||
|
}
|
||||||
|
|
||||||
|
if (hasExecOnlyFlagsAtFront(args)) {
|
||||||
|
const argsWithExec = ['exec', ...args];
|
||||||
|
return {
|
||||||
|
mode: 'command',
|
||||||
|
command: 'exec',
|
||||||
|
argsForDroid: normalizeLegacyReasoningAliasesForExec(argsWithExec),
|
||||||
|
autoPrependedExec: true,
|
||||||
|
};
|
||||||
|
}
|
||||||
|
|
||||||
|
return {
|
||||||
|
mode: 'interactive',
|
||||||
|
argsForDroid: [...args],
|
||||||
|
autoPrependedExec: false,
|
||||||
|
};
|
||||||
|
}
|
||||||
@@ -0,0 +1,86 @@
|
|||||||
|
import { describe, expect, it } from 'bun:test';
|
||||||
|
import { routeDroidCommandArgs } from '../../../src/targets/droid-command-router';
|
||||||
|
|
||||||
|
describe('droid-command-router', () => {
|
||||||
|
it('keeps interactive mode for plain profile launches', () => {
|
||||||
|
const route = routeDroidCommandArgs([]);
|
||||||
|
|
||||||
|
expect(route.mode).toBe('interactive');
|
||||||
|
expect(route.argsForDroid).toEqual([]);
|
||||||
|
expect(route.autoPrependedExec).toBe(false);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('keeps explicit droid subcommands untouched', () => {
|
||||||
|
const route = routeDroidCommandArgs(['mcp', '--help']);
|
||||||
|
|
||||||
|
expect(route.mode).toBe('command');
|
||||||
|
expect(route.command).toBe('mcp');
|
||||||
|
expect(route.argsForDroid).toEqual(['mcp', '--help']);
|
||||||
|
expect(route.autoPrependedExec).toBe(false);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('auto-prepends exec for exec-only flags provided after profile', () => {
|
||||||
|
const route = routeDroidCommandArgs(['--skip-permissions-unsafe']);
|
||||||
|
|
||||||
|
expect(route.mode).toBe('command');
|
||||||
|
expect(route.command).toBe('exec');
|
||||||
|
expect(route.argsForDroid).toEqual(['exec', '--skip-permissions-unsafe']);
|
||||||
|
expect(route.autoPrependedExec).toBe(true);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('does not auto-prepend exec for root help flag', () => {
|
||||||
|
const route = routeDroidCommandArgs(['--help']);
|
||||||
|
|
||||||
|
expect(route.mode).toBe('interactive');
|
||||||
|
expect(route.argsForDroid).toEqual(['--help']);
|
||||||
|
expect(route.autoPrependedExec).toBe(false);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('normalizes --effort alias to --reasoning-effort for explicit exec', () => {
|
||||||
|
const route = routeDroidCommandArgs(['exec', '--effort', 'xhigh', 'fix test flake']);
|
||||||
|
|
||||||
|
expect(route.mode).toBe('command');
|
||||||
|
expect(route.command).toBe('exec');
|
||||||
|
expect(route.argsForDroid).toEqual([
|
||||||
|
'exec',
|
||||||
|
'--reasoning-effort',
|
||||||
|
'xhigh',
|
||||||
|
'fix test flake',
|
||||||
|
]);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('normalizes --thinking alias when exec is auto-prepended', () => {
|
||||||
|
const route = routeDroidCommandArgs(['--auto', 'high', '--thinking=medium', 'summarize logs']);
|
||||||
|
|
||||||
|
expect(route.mode).toBe('command');
|
||||||
|
expect(route.command).toBe('exec');
|
||||||
|
expect(route.argsForDroid).toEqual([
|
||||||
|
'exec',
|
||||||
|
'--auto',
|
||||||
|
'high',
|
||||||
|
'--reasoning-effort=medium',
|
||||||
|
'summarize logs',
|
||||||
|
]);
|
||||||
|
expect(route.autoPrependedExec).toBe(true);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('still auto-prepends exec when --effort appears before exec-only flags', () => {
|
||||||
|
const route = routeDroidCommandArgs([
|
||||||
|
'--effort',
|
||||||
|
'xhigh',
|
||||||
|
'--skip-permissions-unsafe',
|
||||||
|
'fix flaky test',
|
||||||
|
]);
|
||||||
|
|
||||||
|
expect(route.mode).toBe('command');
|
||||||
|
expect(route.command).toBe('exec');
|
||||||
|
expect(route.argsForDroid).toEqual([
|
||||||
|
'exec',
|
||||||
|
'--reasoning-effort',
|
||||||
|
'xhigh',
|
||||||
|
'--skip-permissions-unsafe',
|
||||||
|
'fix flaky test',
|
||||||
|
]);
|
||||||
|
expect(route.autoPrependedExec).toBe(true);
|
||||||
|
});
|
||||||
|
});
|
||||||
Reference in new issue
Block a user