feat(cliproxy): add variant port isolation #184
+1,999 −244
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
- add port field to VariantConfig in types and unified-config-types - implement getNextAvailablePort() to allocate unique ports (8318-8417) - add getConfigPathForPort() for per-variant config files (config-{port}.yaml) - update session-tracker with port-specific lock files (sessions-{port}.json) - pass port to unregisterSession() for proper cleanup - show port column in `ccs cliproxy list` output - clean up port-specific files on variant removal Each CLIProxy variant now runs on its own port, enabling true concurrent execution without session conflicts. - Add session lock cleanup on variant deletion (deleteSessionLockForPort) - Handle EPERM in isProcessRunning() to avoid false stale lock detection - Improve port exhaustion error with actionable guidance
Added baseProvider prop to ProviderEditor for model filtering. Variants now use their parent provider (e.g., 'agy') for filtering instead of the variant name (e.g., 'g3'), showing all available models.
Add 85 new tests covering port isolation feature: - Port allocation unit tests (getNextAvailablePort, boundary conditions) - Session tracker port-specific tests (sessions-{port}.json handling) - Config generator port tests (config-{port}.yaml handling) - Edge case tests (port exhaustion, stale cleanup, legacy migration) Update existing session-tracker tests to use port parameter for non-default port operations (testPort=18317), fixing test failures caused by port isolation changes. - Use service layer for variant updates in PUT /api/cliproxy - Remove dead code from route-helpers.ts - Add port parameter to session tracking for isolation - Use getCcsDir() instead of os.homedir() for path consistency
…shboard - Resolve settings path from variant configuration instead of fixed pattern - Add port and model fields to Variant interface - Display port badge in ProviderEditor header for variants
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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.
Summary
portfield toVariantConfigfor per-variant port allocationgetNextAvailablePort()to allocate unique ports (8318-8417)getConfigPathForPort()for per-variant config files (config-{port}.yaml)sessions-{port}.json)unregisterSession()for proper cleanupccs cliproxy listoutputEach CLIProxy variant now runs on its own port, enabling true concurrent execution without session conflicts.
Test plan
ccs cliproxy createccs cliproxy listshows port column