@copilot Do TRUSTED_DOMAINS_ARG similar to EXTENSION_ARG, try keep it as similar as possible please.

@copilot Do TRUSTED_DOMAINS_ARG similar to EXTENSION_ARG, try keep it as similar as possible please.

Done! I've refactored TRUSTED_DOMAINS_ARG to match the EXTENSION_ARG pattern exactly:

• Moved TRUSTED_DOMAINS_ARG setup to the top of the script (like EXTENSION_ARG)
• Simplified run_code_server() function to use consistent command pattern
• Removed conditional logic in command execution
• Both arguments are now included consistently in all $CODE_SERVER calls

All tests still pass and the implementation is much cleaner. Changes in commite438265.

@copilot Remove the trusted domain tests, as we don't do anything similar for extensions, and Copilot can't test them due to firewall.

@copilot Remove the trusted domain tests, as we don't do anything similar for extensions, and Copilot can't test them due to firewall.

Done! Removed bothtrusted_domains_empty andtrusted_domains_with_values test blocks from the tftest.hcl file. The module now follows the same testing pattern as extensions - the functionality works but isn't explicitly tested due to firewall restrictions. All remaining tests pass. Changes in commit829a692.

@copilot We can't just comma-merge the list, we have to in Bash split by comma and add the argument each time.

Copilot said: Based on my analysis of the code-serverBased on my analysis of the code-server repository, for the --link-protection-trusted-domains flag, you should use the argument multiple times rather than providing a comma-separated list.Here's what I found in the code:Type Definition: The flag is defined as type: "string[]" in the options configurationParsing Logic: When parsing string[] type arguments, the code specifically handles multiple instances of the same flag by pushing each value into an arrayTest Evidence: The test cases show this pattern clearly:TypeScriptconst args = parse(["--proxy-domain", "*.coder.com", "--proxy-domain", "coder.com", "--proxy-domain", "coder.org"])expect(args).toEqual({  "proxy-domain": ["*.coder.com", "coder.com", "coder.org"],})Correct usage:bashcode-server --link-protection-trusted-domains domain.com --link-protection-trusted-domains domain.netNot:bashcode-server --link-protection-trusted-domains domain.com,domain.netThe parsing logic in src/node/cli.ts handles string[] types by creating an array and pushing each occurrence of the flag as a separate value, which is the standard Unix/Linux command-line convention for multi-value arguments.

@copilot We can't just comma-merge the list, we have to in Bash split by comma and add the argument each time.

...

Fixed! The script now splits the comma-separated domains in Bash and creates multiple--link-protection-trusted-domains flag instances as required by code-server. Instead of--link-protection-trusted-domains=domain1,domain2,domain3, it now generates--link-protection-trusted-domains=domain1 --link-protection-trusted-domains=domain2 --link-protection-trusted-domains=domain3. All tests pass. Changes in commit350d595.