Always set remoteControlEnabled to true for cloud agents

[cte]

Important

Always enable remote control for cloud agents in extension.ts by checking ROO_CODE_CLOUD_TOKEN.

• Behavior:
  • In activate() in extension.ts, always set remoteControlEnabled to true for cloud agents by checking process.env.ROO_CODE_CLOUD_TOKEN.
  • Logs the isCloudAgent status and socketBridgeUrl using cloudLogger.
• Logging:
  • Replaces inline logging with cloudLogger for consistency in extension.ts.

^{This description was created by for 52b8f27. You can customize this summary. It will automatically update as commits are pushed.}

[roomote]

Thank you for your contribution! I've reviewed the changes and have some suggestions for improvement. The implementation is clean and the core logic to force remote control for cloud agents makes sense.

[roomote]

Security consideration: The current check only validates that ROO_CODE_CLOUD_TOKEN exists and has length > 0, but doesn't validate if it's actually a valid token. Could someone potentially exploit this by setting a dummy value to force remote control enabled? Consider adding token validation or documenting why this simple check is sufficient for security.

[roomote]

Consider extracting this cloud agent detection logic into a utility function for better reusability and testability. For example:

You could add a new function in src/utils/remoteControl.ts or a similar utility file:

[roomote]

Consider adding a comment explaining what constitutes a cloud agent for future maintainers:

[roomote]

Good improvement using cloudLogger for consistency! This will help with debugging cloud-related issues.