mirror of
https://github.com/tiennm99/ccs.git
synced 2026-10-12 20:15:40 +00:00
fix: treat remote-control as a Claude subcommand
This commit is contained in:
1 parent
cb7ab4eb6f
commit
db4b938c0e
2 files changed
+9
-5
No files matched your search
@@ -34,6 +34,7 @@ const CLAUDE_SUBCOMMANDS = new Set<string>([
|
|||||||
'plugin',
|
'plugin',
|
||||||
'plugins',
|
'plugins',
|
||||||
'project',
|
'project',
|
||||||
|
'remote-control',
|
||||||
'setup-token',
|
'setup-token',
|
||||||
'ultrareview',
|
'ultrareview',
|
||||||
'update',
|
'update',
|
||||||
|
|||||||
@@ -26,6 +26,7 @@ describe('isClaudeSubcommandInvocation', () => {
|
|||||||
'plugin',
|
'plugin',
|
||||||
'plugins',
|
'plugins',
|
||||||
'project',
|
'project',
|
||||||
|
'remote-control',
|
||||||
'setup-token',
|
'setup-token',
|
||||||
'ultrareview',
|
'ultrareview',
|
||||||
'update',
|
'update',
|
||||||
@@ -117,6 +118,9 @@ describe('stripClaudeSubcommandSessionArgs', () => {
|
|||||||
expect(stripClaudeSubcommandSessionArgs(['doctor', '--permission-mode=acceptEdits'])).toEqual([
|
expect(stripClaudeSubcommandSessionArgs(['doctor', '--permission-mode=acceptEdits'])).toEqual([
|
||||||
'doctor',
|
'doctor',
|
||||||
]);
|
]);
|
||||||
|
expect(
|
||||||
|
stripClaudeSubcommandSessionArgs(['remote-control', '--permission-mode', 'bypassPermissions'])
|
||||||
|
).toEqual(['remote-control']);
|
||||||
});
|
});
|
||||||
|
|
||||||
it('preserves --permission-mode for the agents subcommand (after)', () => {
|
it('preserves --permission-mode for the agents subcommand (after)', () => {
|
||||||
@@ -152,11 +156,7 @@ describe('stripClaudeSubcommandSessionArgs', () => {
|
|||||||
'--allow-dangerously-skip-permissions',
|
'--allow-dangerously-skip-permissions',
|
||||||
'agents',
|
'agents',
|
||||||
])
|
])
|
||||||
).toEqual([
|
).toEqual(['--dangerously-skip-permissions', '--allow-dangerously-skip-permissions', 'agents']);
|
||||||
'--dangerously-skip-permissions',
|
|
||||||
'--allow-dangerously-skip-permissions',
|
|
||||||
'agents',
|
|
||||||
]);
|
|
||||||
});
|
});
|
||||||
|
|
||||||
it('still strips --teammate-mode for the agents subcommand (not accepted by upstream)', () => {
|
it('still strips --teammate-mode for the agents subcommand (not accepted by upstream)', () => {
|
||||||
@@ -188,14 +188,17 @@ describe('subcommand passthrough — injectors short-circuit', () => {
|
|||||||
expect(appendThirdPartyWebSearchToolArgs(['agents'])).toEqual(['agents']);
|
expect(appendThirdPartyWebSearchToolArgs(['agents'])).toEqual(['agents']);
|
||||||
expect(appendThirdPartyWebSearchToolArgs(['doctor'])).toEqual(['doctor']);
|
expect(appendThirdPartyWebSearchToolArgs(['doctor'])).toEqual(['doctor']);
|
||||||
expect(appendThirdPartyWebSearchToolArgs(['mcp', 'list'])).toEqual(['mcp', 'list']);
|
expect(appendThirdPartyWebSearchToolArgs(['mcp', 'list'])).toEqual(['mcp', 'list']);
|
||||||
|
expect(appendThirdPartyWebSearchToolArgs(['remote-control'])).toEqual(['remote-control']);
|
||||||
});
|
});
|
||||||
|
|
||||||
it('appendThirdPartyImageAnalysisToolArgs returns args unchanged for subcommand invocations', () => {
|
it('appendThirdPartyImageAnalysisToolArgs returns args unchanged for subcommand invocations', () => {
|
||||||
expect(appendThirdPartyImageAnalysisToolArgs(['agents'])).toEqual(['agents']);
|
expect(appendThirdPartyImageAnalysisToolArgs(['agents'])).toEqual(['agents']);
|
||||||
|
expect(appendThirdPartyImageAnalysisToolArgs(['remote-control'])).toEqual(['remote-control']);
|
||||||
});
|
});
|
||||||
|
|
||||||
it('appendBrowserToolArgs returns args unchanged for subcommand invocations', () => {
|
it('appendBrowserToolArgs returns args unchanged for subcommand invocations', () => {
|
||||||
expect(appendBrowserToolArgs(['agents'])).toEqual(['agents']);
|
expect(appendBrowserToolArgs(['agents'])).toEqual(['agents']);
|
||||||
|
expect(appendBrowserToolArgs(['remote-control'])).toEqual(['remote-control']);
|
||||||
});
|
});
|
||||||
|
|
||||||
it('injectors still inject for non-subcommand interactive launches', () => {
|
it('injectors still inject for non-subcommand interactive launches', () => {
|
||||||
|
|||||||
Reference in new issue
Block a user