fix order

fix: aws starter template compatability
chore: add type parameter for testRequiredVariables
2024-06-26 16:44:21 -05:00 · 2024-06-26 16:34:52 -05:00 · 2024-06-26 19:00:42 +00:00 · 2024-06-26 17:59:12 +00:00 · 2024-06-26 17:21:39 +00:00 · 2024-06-26 16:01:08 +00:00
33 changed files with 1048 additions and 772 deletions
--- a/.icons/desktop.svg
+++ b/.icons/desktop.svg
@@ -0,0 +1,5 @@
+<svg width="32" height="32" viewBox="0 0 32 32" fill="none" xmlns="http://www.w3.org/2000/svg">
+<path d="M31 6V22C31 23.65 29.65 25 28 25H4C2.35 25 1 23.65 1 22V6C1 4.35 2.35 3 4 3H28C29.65 3 31 4.35 31 6Z" fill="#2197F3"/>
+<path d="M21 27H17V24C17 23.4478 16.5522 23 16 23C15.4478 23 15 23.4478 15 24V27H11C10.4478 27 10 27.4478 10 28C10 28.5522 10.4478 29 11 29H21C21.5522 29 22 28.5522 22 28C22 27.4478 21.5522 27 21 27Z" fill="#FFC10A"/>
+<path d="M31 17V22C31 23.65 29.65 25 28 25H4C2.35 25 1 23.65 1 22V17H31Z" fill="#3F51B5"/>
+</svg>
--- a/aws-region/main.test.ts
+++ b/aws-region/main.test.ts
@@ -22,13 +22,4 @@ describe("aws-region", async () => {
    });
    expect(state.outputs.value.value).toBe("us-west-2");
  });
-
-  it("set custom order for coder_parameter", async () => {
-    const order = 99;
-    const state = await runTerraformApply(import.meta.dir, {
-      coder_parameter_order: order.toString(),
-    });
-    expect(state.resources).toHaveLength(1);
-    expect(state.resources[0].instances[0].attributes.order).toBe(order);
-  });
 });
--- a/aws-region/main.tf
+++ b/aws-region/main.tf
@@ -51,12 +51,6 @@ variable "exclude" {
  type        = list(string)
 }

-variable "coder_parameter_order" {
-  type        = number
-  description = "The order determines the position of a template parameter in the UI/CLI presentation. The lowest order is shown first and parameters with equal order are sorted by name (ascending order)."
-  default     = null
-}
-
 locals {
  # This is a static list because the regions don't change _that_
  # frequently and including the `aws_regions` data source requires
@@ -182,7 +176,6 @@ data "coder_parameter" "region" {
  display_name = var.display_name
  description  = var.description
  default      = var.default == "" ? null : var.default
-  order        = var.coder_parameter_order
  mutable      = var.mutable
  dynamic "option" {
    for_each = { for k, v in local.regions : k => v if !(contains(var.exclude, k)) }
--- a/azure-region/main.test.ts
+++ b/azure-region/main.test.ts
@@ -22,13 +22,4 @@ describe("azure-region", async () => {
    });
    expect(state.outputs.value.value).toBe("westus");
  });
-
-  it("set custom order for coder_parameter", async () => {
-    const order = 99;
-    const state = await runTerraformApply(import.meta.dir, {
-      coder_parameter_order: order.toString(),
-    });
-    expect(state.resources).toHaveLength(1);
-    expect(state.resources[0].instances[0].attributes.order).toBe(order);
-  });
 });
--- a/azure-region/main.tf
+++ b/azure-region/main.tf
@@ -50,12 +50,6 @@ variable "exclude" {
  type        = list(string)
 }

-variable "coder_parameter_order" {
-  type        = number
-  description = "The order determines the position of a template parameter in the UI/CLI presentation. The lowest order is shown first and parameters with equal order are sorted by name (ascending order)."
-  default     = null
-}
-
 locals {
  # Note: Options are limited to 64 regions, some redundant regions have been removed.
  all_regions = {
@@ -315,7 +309,6 @@ data "coder_parameter" "region" {
  display_name = var.display_name
  description  = var.description
  default      = var.default == "" ? null : var.default
-  order        = var.coder_parameter_order
  mutable      = var.mutable
  icon         = "/icon/azure.png"
  dynamic "option" {
--- a/code-server/main.tf
+++ b/code-server/main.tf
@@ -95,12 +95,6 @@ variable "use_cached" {
  default     = false
 }

-variable "extensions_dir" {
-  type        = string
-  description = "Override the directory to store extensions in."
-  default     = ""
-}
-
 resource "coder_script" "code-server" {
  agent_id     = var.agent_id
  display_name = "code-server"
@@ -116,7 +110,6 @@ resource "coder_script" "code-server" {
    SETTINGS : replace(jsonencode(var.settings), "\"", "\\\""),
    OFFLINE : var.offline,
    USE_CACHED : var.use_cached,
-    EXTENSIONS_DIR : var.extensions_dir,
  })
  run_on_start = true

--- a/code-server/run.sh
+++ b/code-server/run.sh
@@ -6,16 +6,10 @@ CODE='\033[36;40;1m'
 RESET='\033[0m'
 CODE_SERVER="${INSTALL_PREFIX}/bin/code-server"

-# Set extension directory
-EXTENSION_ARG=""
-if [ -n "${EXTENSIONS_DIR}" ]; then
-  EXTENSION_ARG="--extensions-dir=${EXTENSIONS_DIR}"
-fi
-
 function run_code_server() {
  echo "👷 Running code-server in the background..."
  echo "Check logs at ${LOG_PATH}!"
-  $CODE_SERVER "$EXTENSION_ARG" --auth none --port "${PORT}" --app-name "${APP_NAME}" > "${LOG_PATH}" 2>&1 &
+  $CODE_SERVER --auth none --port "${PORT}" --app-name "${APP_NAME}" > "${LOG_PATH}" 2>&1 &
 }

 # Check if the settings file exists...
@@ -63,7 +57,7 @@ for extension in "$${EXTENSIONLIST[@]}"; do
    continue
  fi
  printf "🧩 Installing extension $${CODE}$extension$${RESET}...\n"
-  output=$($CODE_SERVER "$EXTENSION_ARG" --install-extension "$extension")
+  output=$($CODE_SERVER --install-extension "$extension")
  if [ $? -ne 0 ]; then
    echo "Failed to install extension: $extension: $output"
    exit 1
--- a/dotfiles/README.md
+++ b/dotfiles/README.md
@@ -18,16 +18,3 @@ module "dotfiles" {
  agent_id = coder_agent.example.id
 }
 ```
-
-## Setting a default dotfiles repository
-
-You can set a default dotfiles repository for all users by setting the `default_dotfiles_uri` variable:
-
-```tf
-module "dotfiles" {
-  source               = "registry.coder.com/modules/dotfiles/coder"
-  version              = "1.0.12"
-  agent_id             = coder_agent.example.id
-  default_dotfiles_uri = "https://github.com/coder/dotfiles"
-}
-```
--- a/dotfiles/main.test.ts
+++ b/dotfiles/main.test.ts
@@ -18,23 +18,4 @@ describe("dotfiles", async () => {
    });
    expect(state.outputs.dotfiles_uri.value).toBe("");
  });
-
-  it("set a default dotfiles_uri", async () => {
-    const default_dotfiles_uri = "foo";
-    const state = await runTerraformApply(import.meta.dir, {
-      agent_id: "foo",
-      default_dotfiles_uri,
-    });
-    expect(state.outputs.dotfiles_uri.value).toBe(default_dotfiles_uri);
-  });
-
-  it("set custom order for coder_parameter", async () => {
-    const order = 99;
-    const state = await runTerraformApply(import.meta.dir, {
-      agent_id: "foo",
-      coder_parameter_order: order.toString(),
-    });
-    expect(state.resources).toHaveLength(2);
-    expect(state.resources[0].instances[0].attributes.order).toBe(order);
-  });
 });
--- a/dotfiles/main.tf
+++ b/dotfiles/main.tf
@@ -14,24 +14,11 @@ variable "agent_id" {
  description = "The ID of a Coder agent."
 }

-variable "default_dotfiles_uri" {
-  type        = string
-  description = "The default dotfiles URI if the workspace user does not provide one."
-  default     = ""
-}
-
-variable "coder_parameter_order" {
-  type        = number
-  description = "The order determines the position of a template parameter in the UI/CLI presentation. The lowest order is shown first and parameters with equal order are sorted by name (ascending order)."
-  default     = null
-}
-
 data "coder_parameter" "dotfiles_uri" {
  type         = "string"
  name         = "dotfiles_uri"
  display_name = "Dotfiles URL (optional)"
-  order        = var.coder_parameter_order
-  default      = var.default_dotfiles_uri
+  default      = ""
  description  = "Enter a URL for a [dotfiles repository](https://dotfiles.github.io) to personalize your workspace"
  mutable      = true
  icon         = "/icon/dotfiles.svg"
@@ -53,4 +40,4 @@ resource "coder_script" "personalize" {
 output "dotfiles_uri" {
  description = "Dotfiles URI"
  value       = data.coder_parameter.dotfiles_uri.value
-}
+}
--- a/exoscale-instance-type/main.test.ts
+++ b/exoscale-instance-type/main.test.ts
@@ -31,13 +31,4 @@ describe("exoscale-instance-type", async () => {
      });
    }).toThrow('default value "gpu3.huge" must be defined as one of options');
  });
-
-  it("set custom order for coder_parameter", async () => {
-    const order = 99;
-    const state = await runTerraformApply(import.meta.dir, {
-      coder_parameter_order: order.toString(),
-    });
-    expect(state.resources).toHaveLength(1);
-    expect(state.resources[0].instances[0].attributes.order).toBe(order);
-  });
 });
--- a/exoscale-instance-type/main.tf
+++ b/exoscale-instance-type/main.tf
@@ -56,12 +56,6 @@ variable "exclude" {
  type        = list(string)
 }

-variable "coder_parameter_order" {
-  type        = number
-  description = "The order determines the position of a template parameter in the UI/CLI presentation. The lowest order is shown first and parameters with equal order are sorted by name (ascending order)."
-  default     = null
-}
-
 locals {
  # https://www.exoscale.com/pricing/

@@ -263,7 +257,6 @@ data "coder_parameter" "instance_type" {
  display_name = var.display_name
  description  = var.description
  default      = var.default == "" ? null : var.default
-  order        = var.coder_parameter_order
  mutable      = var.mutable
  dynamic "option" {
    for_each = [for k, v in concat(
--- a/exoscale-zone/main.test.ts
+++ b/exoscale-zone/main.test.ts
@@ -22,13 +22,4 @@ describe("exoscale-zone", async () => {
    });
    expect(state.outputs.value.value).toBe("at-vie-1");
  });
-
-  it("set custom order for coder_parameter", async () => {
-    const order = 99;
-    const state = await runTerraformApply(import.meta.dir, {
-      coder_parameter_order: order.toString(),
-    });
-    expect(state.resources).toHaveLength(1);
-    expect(state.resources[0].instances[0].attributes.order).toBe(order);
-  });
 });
--- a/exoscale-zone/main.tf
+++ b/exoscale-zone/main.tf
@@ -51,11 +51,6 @@ variable "exclude" {
  type        = list(string)
 }

-variable "coder_parameter_order" {
-  type        = number
-  description = "The order determines the position of a template parameter in the UI/CLI presentation. The lowest order is shown first and parameters with equal order are sorted by name (ascending order)."
-  default     = null
-}

 locals {
  # This is a static list because the zones don't change _that_
@@ -99,7 +94,6 @@ data "coder_parameter" "zone" {
  display_name = var.display_name
  description  = var.description
  default      = var.default == "" ? null : var.default
-  order        = var.coder_parameter_order
  mutable      = var.mutable
  dynamic "option" {
    for_each = { for k, v in local.zones : k => v if !(contains(var.exclude, k)) }
--- a/gcp-region/main.test.ts
+++ b/gcp-region/main.test.ts
@@ -40,13 +40,4 @@ describe("gcp-region", async () => {
    });
    expect(state.outputs.value.value).toBe("us-west2-b");
  });
-
-  it("set custom order for coder_parameter", async () => {
-    const order = 99;
-    const state = await runTerraformApply(import.meta.dir, {
-      coder_parameter_order: order.toString(),
-    });
-    expect(state.resources).toHaveLength(1);
-    expect(state.resources[0].instances[0].attributes.order).toBe(order);
-  });
 });
--- a/gcp-region/main.tf
+++ b/gcp-region/main.tf
@@ -63,12 +63,6 @@ variable "single_zone_per_region" {
  type        = bool
 }

-variable "coder_parameter_order" {
-  type        = number
-  description = "The order determines the position of a template parameter in the UI/CLI presentation. The lowest order is shown first and parameters with equal order are sorted by name (ascending order)."
-  default     = null
-}
-
 locals {
  zones = {
    # US Central
@@ -721,7 +715,6 @@ data "coder_parameter" "region" {
  icon         = "/icon/gcp.png"
  mutable      = var.mutable
  default      = var.default != null && var.default != "" && (!var.gpu_only || try(local.zones[var.default].gpu, false)) ? var.default : null
-  order        = var.coder_parameter_order
  dynamic "option" {
    for_each = {
      for k, v in local.zones : k => v
--- a/git-clone/README.md
+++ b/git-clone/README.md
@@ -50,106 +50,3 @@ data "coder_git_auth" "github" {
  id = "github"
 }
 ```
-
-## GitHub clone with branch name
-
-To GitHub clone with a specific branch like `feat/example`
-
-```tf
-# Prompt the user for the git repo URL
-data "coder_parameter" "git_repo" {
-  name         = "git_repo"
-  display_name = "Git repository"
-  default      = "https://github.com/coder/coder/tree/feat/example"
-}
-
-# Clone the repository for branch `feat/example`
-module "git_clone" {
-  source   = "registry.coder.com/modules/git-clone/coder"
-  version  = "1.0.11"
-  agent_id = coder_agent.example.id
-  url      = data.coder_parameter.git_repo.value
-}
-
-# Create a code-server instance for the cloned repository
-module "code-server" {
-  source   = "registry.coder.com/modules/code-server/coder"
-  version  = "1.0.11"
-  agent_id = coder_agent.example.id
-  order    = 1
-  folder   = "/home/${local.username}/${module.git_clone.folder_name}"
-}
-
-# Create a Coder app for the website
-resource "coder_app" "website" {
-  agent_id     = coder_agent.example.id
-  order        = 2
-  slug         = "website"
-  external     = true
-  display_name = module.git_clone.folder_name
-  url          = module.git_clone.web_url
-  icon         = module.git_clone.git_provider != "" ? "/icon/${module.git_clone.git_provider}.svg" : "/icon/git.svg"
-  count        = module.git_clone.web_url != "" ? 1 : 0
-}
-```
-
-Configuring `git-clone` for a self-hosted GitHub Enterprise Server running at `github.example.com`
-
-```tf
-module "git-clone" {
-  source   = "registry.coder.com/modules/git-clone/coder"
-  version  = "1.0.11"
-  agent_id = coder_agent.example.id
-  url      = "https://github.example.com/coder/coder/tree/feat/example"
-  git_providers = {
-    "https://github.example.com/" = {
-      provider = "github"
-    }
-  }
-}
-```
-
-## GitLab clone with branch name
-
-To GitLab clone with a specific branch like `feat/example`
-
-```tf
-module "git-clone" {
-  source   = "registry.coder.com/modules/git-clone/coder"
-  version  = "1.0.11"
-  agent_id = coder_agent.example.id
-  url      = "https://gitlab.com/coder/coder/-/tree/feat/example"
-}
-```
-
-Configuring `git-clone` for a self-hosted GitLab running at `gitlab.example.com`
-
-```tf
-module "git-clone" {
-  source   = "registry.coder.com/modules/git-clone/coder"
-  version  = "1.0.11"
-  agent_id = coder_agent.example.id
-  url      = "https://gitlab.example.com/coder/coder/-/tree/feat/example"
-  git_providers = {
-    "https://gitlab.example.com/" = {
-      provider = "gitlab"
-    }
-  }
-}
-```
-
-## Git clone with branch_name set
-
-Alternatively, you can set the `branch_name` attribute to clone a specific branch.
-
-For example, to clone the `feat/example` branch:
-
-```tf
-module "git-clone" {
-  source      = "registry.coder.com/modules/git-clone/coder"
-  version     = "1.0.11"
-  agent_id    = coder_agent.example.id
-  url         = "https://github.com/coder/coder"
-  branch_name = "feat/example"
-}
-```
--- a/git-clone/main.test.ts
+++ b/git-clone/main.test.ts
@@ -36,196 +36,4 @@ describe("git-clone", async () => {
      "Cloning fake-url to ~/fake-url...",
    ]);
  });
-
-  it("repo_dir should match repo name for https", async () => {
-    const url = "https://github.com/coder/coder.git";
-    const state = await runTerraformApply(import.meta.dir, {
-      agent_id: "foo",
-      base_dir: "/tmp",
-      url,
-    });
-    expect(state.outputs.repo_dir.value).toEqual("/tmp/coder");
-    expect(state.outputs.folder_name.value).toEqual("coder");
-    expect(state.outputs.clone_url.value).toEqual(url);
-    expect(state.outputs.web_url.value).toEqual(url);
-    expect(state.outputs.branch_name.value).toEqual("");
-  });
-
-  it("repo_dir should match repo name for https without .git", async () => {
-    const url = "https://github.com/coder/coder";
-    const state = await runTerraformApply(import.meta.dir, {
-      agent_id: "foo",
-      base_dir: "/tmp",
-      url,
-    });
-    expect(state.outputs.repo_dir.value).toEqual("/tmp/coder");
-    expect(state.outputs.clone_url.value).toEqual(url);
-    expect(state.outputs.web_url.value).toEqual(url);
-    expect(state.outputs.branch_name.value).toEqual("");
-  });
-
-  it("repo_dir should match repo name for ssh", async () => {
-    const url = "git@github.com:coder/coder.git";
-    const state = await runTerraformApply(import.meta.dir, {
-      agent_id: "foo",
-      base_dir: "/tmp",
-      url,
-    });
-    expect(state.outputs.repo_dir.value).toEqual("/tmp/coder");
-    expect(state.outputs.git_provider.value).toEqual("");
-    expect(state.outputs.clone_url.value).toEqual(url);
-    const https_url = "https://github.com/coder/coder.git";
-    expect(state.outputs.web_url.value).toEqual(https_url);
-    expect(state.outputs.branch_name.value).toEqual("");
-  });
-
-  it("branch_name should not include query string", async () => {
-    const state = await runTerraformApply(import.meta.dir, {
-      agent_id: "foo",
-      url: "https://gitlab.com/mike.brew/repo-tests.log/-/tree/feat/branch?ref_type=heads",
-    });
-    expect(state.outputs.repo_dir.value).toEqual("~/repo-tests.log");
-    expect(state.outputs.folder_name.value).toEqual("repo-tests.log");
-    const https_url = "https://gitlab.com/mike.brew/repo-tests.log";
-    expect(state.outputs.clone_url.value).toEqual(https_url);
-    expect(state.outputs.web_url.value).toEqual(https_url);
-    expect(state.outputs.branch_name.value).toEqual("feat/branch");
-  });
-
-  it("branch_name should not include fragments", async () => {
-    const state = await runTerraformApply(import.meta.dir, {
-      agent_id: "foo",
-      base_dir: "/tmp",
-      url: "https://gitlab.com/mike.brew/repo-tests.log/-/tree/feat/branch#name",
-    });
-    expect(state.outputs.repo_dir.value).toEqual("/tmp/repo-tests.log");
-    const https_url = "https://gitlab.com/mike.brew/repo-tests.log";
-    expect(state.outputs.clone_url.value).toEqual(https_url);
-    expect(state.outputs.web_url.value).toEqual(https_url);
-    expect(state.outputs.branch_name.value).toEqual("feat/branch");
-  });
-
-  it("gitlab url with branch should match", async () => {
-    const state = await runTerraformApply(import.meta.dir, {
-      agent_id: "foo",
-      base_dir: "/tmp",
-      url: "https://gitlab.com/mike.brew/repo-tests.log/-/tree/feat/branch",
-    });
-    expect(state.outputs.repo_dir.value).toEqual("/tmp/repo-tests.log");
-    expect(state.outputs.git_provider.value).toEqual("gitlab");
-    const https_url = "https://gitlab.com/mike.brew/repo-tests.log";
-    expect(state.outputs.clone_url.value).toEqual(https_url);
-    expect(state.outputs.web_url.value).toEqual(https_url);
-    expect(state.outputs.branch_name.value).toEqual("feat/branch");
-  });
-
-  it("github url with branch should match", async () => {
-    const state = await runTerraformApply(import.meta.dir, {
-      agent_id: "foo",
-      base_dir: "/tmp",
-      url: "https://github.com/michaelbrewer/repo-tests.log/tree/feat/branch",
-    });
-    expect(state.outputs.repo_dir.value).toEqual("/tmp/repo-tests.log");
-    expect(state.outputs.git_provider.value).toEqual("github");
-    const https_url = "https://github.com/michaelbrewer/repo-tests.log";
-    expect(state.outputs.clone_url.value).toEqual(https_url);
-    expect(state.outputs.web_url.value).toEqual(https_url);
-    expect(state.outputs.branch_name.value).toEqual("feat/branch");
-  });
-
-  it("self-host git url with branch should match", async () => {
-    const state = await runTerraformApply(import.meta.dir, {
-      agent_id: "foo",
-      base_dir: "/tmp",
-      url: "https://git.example.com/example/project/-/tree/feat/example",
-      git_providers: `
-      {
-        "https://git.example.com/" = {
-          provider = "gitlab"
-        }
-      }`,
-    });
-    expect(state.outputs.repo_dir.value).toEqual("/tmp/project");
-    expect(state.outputs.git_provider.value).toEqual("gitlab");
-    const https_url = "https://git.example.com/example/project";
-    expect(state.outputs.clone_url.value).toEqual(https_url);
-    expect(state.outputs.web_url.value).toEqual(https_url);
-    expect(state.outputs.branch_name.value).toEqual("feat/example");
-  });
-
-  it("handle unsupported git provider configuration", async () => {
-    const t = async () => {
-      await runTerraformApply(import.meta.dir, {
-        agent_id: "foo",
-        url: "foo",
-        git_providers: `
-        {
-          "https://git.example.com/" = {
-            provider = "bitbucket"
-          }
-        }`,
-      });
-    };
-    expect(t).toThrow('Allowed values for provider are "github" or "gitlab".');
-  });
-
-  it("handle unknown git provider url", async () => {
-    const url = "https://git.unknown.com/coder/coder";
-    const state = await runTerraformApply(import.meta.dir, {
-      agent_id: "foo",
-      base_dir: "/tmp",
-      url,
-    });
-    expect(state.outputs.repo_dir.value).toEqual("/tmp/coder");
-    expect(state.outputs.clone_url.value).toEqual(url);
-    expect(state.outputs.web_url.value).toEqual(url);
-    expect(state.outputs.branch_name.value).toEqual("");
-  });
-
-  it("runs with github clone with switch to feat/branch", async () => {
-    const state = await runTerraformApply(import.meta.dir, {
-      agent_id: "foo",
-      url: "https://github.com/michaelbrewer/repo-tests.log/tree/feat/branch",
-    });
-    const output = await executeScriptInContainer(state, "alpine/git");
-    expect(output.exitCode).toBe(0);
-    expect(output.stdout).toEqual([
-      "Creating directory ~/repo-tests.log...",
-      "Cloning https://github.com/michaelbrewer/repo-tests.log to ~/repo-tests.log on branch feat/branch...",
-    ]);
-  });
-
-  it("runs with gitlab clone with switch to feat/branch", async () => {
-    const state = await runTerraformApply(import.meta.dir, {
-      agent_id: "foo",
-      url: "https://gitlab.com/mike.brew/repo-tests.log/-/tree/feat/branch",
-    });
-    const output = await executeScriptInContainer(state, "alpine/git");
-    expect(output.exitCode).toBe(0);
-    expect(output.stdout).toEqual([
-      "Creating directory ~/repo-tests.log...",
-      "Cloning https://gitlab.com/mike.brew/repo-tests.log to ~/repo-tests.log on branch feat/branch...",
-    ]);
-  });
-
-  it("runs with github clone with branch_name set to feat/branch", async () => {
-    const url = "https://github.com/michaelbrewer/repo-tests.log";
-    const branch_name = "feat/branch";
-    const state = await runTerraformApply(import.meta.dir, {
-      agent_id: "foo",
-      url,
-      branch_name,
-    });
-    expect(state.outputs.repo_dir.value).toEqual("~/repo-tests.log");
-    expect(state.outputs.clone_url.value).toEqual(url);
-    expect(state.outputs.web_url.value).toEqual(url);
-    expect(state.outputs.branch_name.value).toEqual(branch_name);
-
-    const output = await executeScriptInContainer(state, "alpine/git");
-    expect(output.exitCode).toBe(0);
-    expect(output.stdout).toEqual([
-      "Creating directory ~/repo-tests.log...",
-      "Cloning https://github.com/michaelbrewer/repo-tests.log to ~/repo-tests.log on branch feat/branch...",
-    ]);
-  });
 });
--- a/git-clone/main.tf
+++ b/git-clone/main.tf
@@ -25,50 +25,8 @@ variable "agent_id" {
  type        = string
 }

-variable "git_providers" {
-  type = map(object({
-    provider = string
-  }))
-  description = "A mapping of URLs to their git provider."
-  default = {
-    "https://github.com/" = {
-      provider = "github"
-    },
-    "https://gitlab.com/" = {
-      provider = "gitlab"
-    },
-  }
-  validation {
-    error_message = "Allowed values for provider are \"github\" or \"gitlab\"."
-    condition     = alltrue([for provider in var.git_providers : contains(["github", "gitlab"], provider.provider)])
-  }
-}
-
-variable "branch_name" {
-  description = "The branch name to clone. If not provided, the default branch will be cloned."
-  type        = string
-  default     = ""
-}
-
 locals {
-  # Remove query parameters and fragments from the URL
-  url = replace(replace(var.url, "/\\?.*/", ""), "/#.*/", "")
-
-  # Find the git provider based on the URL and determine the tree path
-  provider_key = try(one([for key in keys(var.git_providers) : key if startswith(local.url, key)]), null)
-  provider     = try(lookup(var.git_providers, local.provider_key).provider, "")
-  tree_path    = local.provider == "gitlab" ? "/-/tree/" : local.provider == "github" ? "/tree/" : ""
-
-  # Remove tree and branch name from the URL
-  clone_url = var.branch_name == "" && local.tree_path != "" ? replace(local.url, "/${local.tree_path}.*/", "") : local.url
-  # Extract the branch name from the URL
-  branch_name = var.branch_name == "" && local.tree_path != "" ? replace(replace(local.url, local.clone_url, ""), "/.*${local.tree_path}/", "") : var.branch_name
-  # Extract the folder name from the URL
-  folder_name = replace(basename(local.clone_url), ".git", "")
-  # Construct the path to clone the repository
-  clone_path = var.base_dir != "" ? join("/", [var.base_dir, local.folder_name]) : join("/", ["~", local.folder_name])
-  # Construct the web URL
-  web_url = startswith(local.clone_url, "git@") ? replace(replace(local.clone_url, ":", "/"), "git@", "https://") : local.clone_url
+  clone_path = var.base_dir != "" ? join("/", [var.base_dir, replace(basename(var.url), ".git", "")]) : join("/", ["~", replace(basename(var.url), ".git", "")])
 }

 output "repo_dir" {
@@ -76,37 +34,11 @@ output "repo_dir" {
  description = "Full path of cloned repo directory"
 }

-output "git_provider" {
-  value       = local.provider
-  description = "The git provider of the repository"
-}
-
-output "folder_name" {
-  value       = local.folder_name
-  description = "The name of the folder that will be created"
-}
-
-output "clone_url" {
-  value       = local.clone_url
-  description = "The exact Git repository URL that will be cloned"
-}
-
-output "web_url" {
-  value       = local.web_url
-  description = "Git https repository URL (may be invalid for unsupported providers)"
-}
-
-output "branch_name" {
-  value       = local.branch_name
-  description = "Git branch name (may be empty)"
-}
-
 resource "coder_script" "git_clone" {
  agent_id = var.agent_id
  script = templatefile("${path.module}/run.sh", {
-    CLONE_PATH = local.clone_path,
-    REPO_URL : local.clone_url,
-    BRANCH_NAME : local.branch_name,
+    CLONE_PATH = local.clone_path
+    REPO_URL : var.url,
  })
  display_name       = "Git Clone"
  icon               = "/icon/git.svg"
--- a/git-clone/run.sh
+++ b/git-clone/run.sh
@@ -2,7 +2,6 @@

 REPO_URL="${REPO_URL}"
 CLONE_PATH="${CLONE_PATH}"
-BRANCH_NAME="${BRANCH_NAME}"
 # Expand home if it's specified!
 CLONE_PATH="$${CLONE_PATH/#\~/$${HOME}}"

@@ -34,13 +33,8 @@ fi
 # Check if the directory is empty
 # and if it is, clone the repo, otherwise skip cloning
 if [ -z "$(ls -A "$CLONE_PATH")" ]; then
-  if [ -z "$BRANCH_NAME" ]; then
-    echo "Cloning $REPO_URL to $CLONE_PATH..."
-    git clone "$REPO_URL" "$CLONE_PATH"
-  else
-    echo "Cloning $REPO_URL to $CLONE_PATH on branch $BRANCH_NAME..."
-    git clone "$REPO_URL" -b "$BRANCH_NAME" "$CLONE_PATH"
-  fi
+  echo "Cloning $REPO_URL to $CLONE_PATH..."
+  git clone "$REPO_URL" "$CLONE_PATH"
 else
  echo "$CLONE_PATH already exists and isn't empty, skipping clone!"
  exit 0
--- a/git-commit-signing/README.md
+++ b/git-commit-signing/README.md
@@ -19,7 +19,7 @@ This module has a chance of conflicting with the user's dotfiles / the personali
 ```tf
 module "git-commit-signing" {
  source   = "registry.coder.com/modules/git-commit-signing/coder"
-  version  = "1.0.11"
+  version  = "1.0.9"
  agent_id = coder_agent.example.id
 }
 ```
--- a/git-config/main.test.ts
+++ b/git-config/main.test.ts
@@ -1,99 +0,0 @@
-import { describe, expect, it } from "bun:test";
-import {
-  runTerraformApply,
-  runTerraformInit,
-  testRequiredVariables,
-} from "../test";
-
-describe("git-config", async () => {
-  await runTerraformInit(import.meta.dir);
-
-  testRequiredVariables(import.meta.dir, {
-    agent_id: "foo",
-  });
-
-  it("can run apply allow_username_change and allow_email_change disabled", async () => {
-    const state = await runTerraformApply(import.meta.dir, {
-      agent_id: "foo",
-      allow_username_change: "false",
-      allow_email_change: "false",
-    });
-
-    const resources = state.resources;
-    expect(resources).toHaveLength(3);
-    expect(resources).toMatchObject([
-      { type: "coder_workspace", name: "me" },
-      { type: "coder_env", name: "git_author_name" },
-      { type: "coder_env", name: "git_commmiter_name" },
-    ]);
-  });
-
-  it("can run apply allow_email_change enabled", async () => {
-    const state = await runTerraformApply(import.meta.dir, {
-      agent_id: "foo",
-      allow_email_change: "true",
-    });
-
-    const resources = state.resources;
-    expect(resources).toHaveLength(5);
-    expect(resources).toMatchObject([
-      { type: "coder_parameter", name: "user_email" },
-      { type: "coder_parameter", name: "username" },
-      { type: "coder_workspace", name: "me" },
-      { type: "coder_env", name: "git_author_name" },
-      { type: "coder_env", name: "git_commmiter_name" },
-    ]);
-  });
-
-  it("can run apply allow_email_change enabled", async () => {
-    const state = await runTerraformApply(
-      import.meta.dir,
-      {
-        agent_id: "foo",
-        allow_username_change: "false",
-        allow_email_change: "false",
-      },
-      { CODER_WORKSPACE_OWNER_EMAIL: "foo@emai.com" },
-    );
-
-    const resources = state.resources;
-    expect(resources).toHaveLength(5);
-    expect(resources).toMatchObject([
-      { type: "coder_workspace", name: "me" },
-      { type: "coder_env", name: "git_author_email" },
-      { type: "coder_env", name: "git_author_name" },
-      { type: "coder_env", name: "git_commmiter_email" },
-      { type: "coder_env", name: "git_commmiter_name" },
-    ]);
-  });
-
-  it("set custom order for coder_parameter for both fields", async () => {
-    const order = 20;
-    const state = await runTerraformApply(import.meta.dir, {
-      agent_id: "foo",
-      allow_username_change: "true",
-      allow_email_change: "true",
-      coder_parameter_order: order.toString(),
-    });
-    expect(state.resources).toHaveLength(5);
-    // user_email order is the same as the order
-    expect(state.resources[0].instances[0].attributes.order).toBe(order);
-    // username order is incremented by 1
-    // @ts-ignore: Object is possibly 'null'.
-    expect(state.resources[1].instances[0]?.attributes.order).toBe(order + 1);
-  });
-
-  it("set custom order for coder_parameter for just username", async () => {
-    const order = 30;
-    const state = await runTerraformApply(import.meta.dir, {
-      agent_id: "foo",
-      allow_email_change: "false",
-      allow_username_change: "true",
-      coder_parameter_order: order.toString(),
-    });
-    expect(state.resources).toHaveLength(4);
-    // user_email was not created
-    // username order is incremented by 1
-    expect(state.resources[0].instances[0].attributes.order).toBe(order + 1);
-  });
-});
--- a/git-config/main.tf
+++ b/git-config/main.tf
@@ -26,11 +26,6 @@ variable "allow_email_change" {
  default     = false
 }

-variable "coder_parameter_order" {
-  type        = number
-  description = "The order determines the position of a template parameter in the UI/CLI presentation. The lowest order is shown first and parameters with equal order are sorted by name (ascending order)."
-  default     = null
-}

 data "coder_workspace" "me" {}

@@ -39,7 +34,6 @@ data "coder_parameter" "user_email" {
  name         = "user_email"
  type         = "string"
  default      = ""
-  order        = var.coder_parameter_order != null ? var.coder_parameter_order + 0 : null
  description  = "Git user.email to be used for commits. Leave empty to default to Coder user's email."
  display_name = "Git config user.email"
  mutable      = true
@@ -50,7 +44,6 @@ data "coder_parameter" "username" {
  name         = "username"
  type         = "string"
  default      = ""
-  order        = var.coder_parameter_order != null ? var.coder_parameter_order + 1 : null
  description  = "Git user.name to be used for commits. Leave empty to default to Coder user's Full Name."
  display_name = "Full Name for Git config"
  mutable      = true
@@ -72,12 +65,10 @@ resource "coder_env" "git_author_email" {
  agent_id = var.agent_id
  name     = "GIT_AUTHOR_EMAIL"
  value    = coalesce(try(data.coder_parameter.user_email[0].value, ""), data.coder_workspace.me.owner_email)
-  count    = data.coder_workspace.me.owner_email != "" ? 1 : 0
 }

 resource "coder_env" "git_commmiter_email" {
  agent_id = var.agent_id
  name     = "GIT_COMMITTER_EMAIL"
  value    = coalesce(try(data.coder_parameter.user_email[0].value, ""), data.coder_workspace.me.owner_email)
-  count    = data.coder_workspace.me.owner_email != "" ? 1 : 0
 }
--- a/jetbrains-gateway/README.md
+++ b/jetbrains-gateway/README.md
@@ -14,7 +14,7 @@ This module adds a JetBrains Gateway Button to open any workspace with a single
 ```tf
 module "jetbrains_gateway" {
  source         = "registry.coder.com/modules/jetbrains-gateway/coder"
-  version        = "1.0.11"
+  version        = "1.0.9"
  agent_id       = coder_agent.example.id
  agent_name     = "example"
  folder         = "/home/coder/example"
@@ -27,12 +27,12 @@ module "jetbrains_gateway" {

 ## Examples

-### Add GoLand and WebStorm as options with the default set to GoLand
+### Add GoLand and WebStorm with the default set to GoLand

 ```tf
 module "jetbrains_gateway" {
  source         = "registry.coder.com/modules/jetbrains-gateway/coder"
-  version        = "1.0.11"
+  version        = "1.0.9"
  agent_id       = coder_agent.example.id
  agent_name     = "example"
  folder         = "/home/coder/example"
@@ -41,37 +41,6 @@ module "jetbrains_gateway" {
 }
 ```

-### Use the latest release version
-
-```tf
-module "jetbrains_gateway" {
-  source         = "registry.coder.com/modules/jetbrains-gateway/coder"
-  version        = "1.0.11"
-  agent_id       = coder_agent.example.id
-  agent_name     = "example"
-  folder         = "/home/coder/example"
-  jetbrains_ides = ["GO", "WS"]
-  default        = "GO"
-  latest         = true
-}
-```
-
-### Use the latest EAP version
-
-```tf
-module "jetbrains_gateway" {
-  source         = "registry.coder.com/modules/jetbrains-gateway/coder"
-  version        = "1.0.11"
-  agent_id       = coder_agent.example.id
-  agent_name     = "example"
-  folder         = "/home/coder/example"
-  jetbrains_ides = ["GO", "WS"]
-  default        = "GO"
-  latest         = true
-  channel        = "eap"
-}
-```
-
 ## Supported IDEs

 This module and JetBrains Gateway support the following JetBrains IDEs:
--- a/jetbrains-gateway/main.tf
+++ b/jetbrains-gateway/main.tf
@@ -6,10 +6,6 @@ terraform {
      source  = "coder/coder"
      version = ">= 0.17"
    }
-    http = {
-      source  = "hashicorp/http"
-      version = ">= 3.0"
-    }
  }
 }

@@ -50,22 +46,6 @@ variable "coder_parameter_order" {
  default     = null
 }

-variable "latest" {
-  type        = bool
-  description = "Whether to fetch the latest version of the IDE."
-  default     = false
-}
-
-variable "channel" {
-  type        = string
-  description = "JetBrains IDE release channel. Valid values are release and eap."
-  default     = "release"
-  validation {
-    condition     = can(regex("^(release|eap)$", var.channel))
-    error_message = "The channel must be either release or eap."
-  }
-}
-
 variable "jetbrains_ide_versions" {
  type = map(object({
    build_number = string
@@ -74,36 +54,36 @@ variable "jetbrains_ide_versions" {
  description = "The set of versions for each jetbrains IDE"
  default = {
    "IU" = {
-      build_number = "241.14494.240"
-      version      = "2024.1"
+      build_number = "233.14808.21"
+      version      = "2023.3.5"
    }
    "PS" = {
-      build_number = "241.14494.237"
-      version      = "2024.1"
+      build_number = "233.14808.18"
+      version      = "2023.3.5"
    }
    "WS" = {
-      build_number = "241.14494.235"
-      version      = "2024.1"
+      build_number = "233.14475.40"
+      version      = "2023.3.4"
    }
    "PY" = {
-      build_number = "241.14494.241"
-      version      = "2024.1"
+      build_number = "233.14475.56"
+      version      = "2023.3.4"
    }
    "CL" = {
-      build_number = "241.14494.288"
-      version      = "2024.1"
+      build_number = "233.14475.31"
+      version      = "2023.3.4"
    }
    "GO" = {
-      build_number = "241.14494.238"
-      version      = "2024.1"
+      build_number = "233.14808.20"
+      version      = "2023.3.5"
    }
    "RM" = {
-      build_number = "241.14494.234"
-      version      = "2024.1"
+      build_number = "233.14808.14"
+      version      = "2023.3.5"
    }
    "RD" = {
-      build_number = "241.14494.307"
-      version      = "2024.1"
+      build_number = "233.14475.66"
+      version      = "2023.3.4"
    }
  }
  validation {
@@ -140,11 +120,6 @@ variable "jetbrains_ides" {
  }
 }

-data "http" "jetbrains_ide_versions" {
-  for_each = var.latest ? toset(var.jetbrains_ides) : toset([])
-  url      = "https://data.services.jetbrains.com/products/releases?code=${each.key}&latest=true&type=${var.channel}"
-}
-
 locals {
  jetbrains_ides = {
    "GO" = {
@@ -153,7 +128,6 @@ locals {
      identifier    = "GO",
      build_number  = var.jetbrains_ide_versions["GO"].build_number,
      download_link = "https://download.jetbrains.com/go/goland-${var.jetbrains_ide_versions["GO"].version}.tar.gz"
-      version       = var.jetbrains_ide_versions["GO"].version
    },
    "WS" = {
      icon          = "/icon/webstorm.svg",
@@ -161,7 +135,6 @@ locals {
      identifier    = "WS",
      build_number  = var.jetbrains_ide_versions["WS"].build_number,
      download_link = "https://download.jetbrains.com/webstorm/WebStorm-${var.jetbrains_ide_versions["WS"].version}.tar.gz"
-      version       = var.jetbrains_ide_versions["WS"].version
    },
    "IU" = {
      icon          = "/icon/intellij.svg",
@@ -169,7 +142,6 @@ locals {
      identifier    = "IU",
      build_number  = var.jetbrains_ide_versions["IU"].build_number,
      download_link = "https://download.jetbrains.com/idea/ideaIU-${var.jetbrains_ide_versions["IU"].version}.tar.gz"
-      version       = var.jetbrains_ide_versions["IU"].version
    },
    "PY" = {
      icon          = "/icon/pycharm.svg",
@@ -177,7 +149,6 @@ locals {
      identifier    = "PY",
      build_number  = var.jetbrains_ide_versions["PY"].build_number,
      download_link = "https://download.jetbrains.com/python/pycharm-professional-${var.jetbrains_ide_versions["PY"].version}.tar.gz"
-      version       = var.jetbrains_ide_versions["PY"].version
    },
    "CL" = {
      icon          = "/icon/clion.svg",
@@ -185,7 +156,6 @@ locals {
      identifier    = "CL",
      build_number  = var.jetbrains_ide_versions["CL"].build_number,
      download_link = "https://download.jetbrains.com/cpp/CLion-${var.jetbrains_ide_versions["CL"].version}.tar.gz"
-      version       = var.jetbrains_ide_versions["CL"].version
    },
    "PS" = {
      icon          = "/icon/phpstorm.svg",
@@ -193,7 +163,6 @@ locals {
      identifier    = "PS",
      build_number  = var.jetbrains_ide_versions["PS"].build_number,
      download_link = "https://download.jetbrains.com/webide/PhpStorm-${var.jetbrains_ide_versions["PS"].version}.tar.gz"
-      version       = var.jetbrains_ide_versions["PS"].version
    },
    "RM" = {
      icon          = "/icon/rubymine.svg",
@@ -201,7 +170,6 @@ locals {
      identifier    = "RM",
      build_number  = var.jetbrains_ide_versions["RM"].build_number,
      download_link = "https://download.jetbrains.com/ruby/RubyMine-${var.jetbrains_ide_versions["RM"].version}.tar.gz"
-      version       = var.jetbrains_ide_versions["RM"].version
    }
    "RD" = {
      icon          = "/icon/rider.svg",
@@ -209,18 +177,8 @@ locals {
      identifier    = "RD",
      build_number  = var.jetbrains_ide_versions["RD"].build_number,
      download_link = "https://download.jetbrains.com/rider/JetBrains.Rider-${var.jetbrains_ide_versions["RD"].version}.tar.gz"
-      version       = var.jetbrains_ide_versions["RD"].version
    }
  }
-
-  icon          = try(lookup(local.jetbrains_ides, data.coder_parameter.jetbrains_ide.value).icon, "/icon/gateway.svg")
-  json_data     = var.latest ? jsondecode(data.http.jetbrains_ide_versions[data.coder_parameter.jetbrains_ide.value].response_body) : {}
-  key           = var.latest ? keys(local.json_data)[0] : ""
-  display_name  = local.jetbrains_ides[data.coder_parameter.jetbrains_ide.value].name
-  identifier    = data.coder_parameter.jetbrains_ide.value
-  download_link = var.latest ? local.json_data[local.key][0].downloads.linux.link : local.jetbrains_ides[data.coder_parameter.jetbrains_ide.value].download_link
-  build_number  = var.latest ? local.json_data[local.key][0].build : local.jetbrains_ides[data.coder_parameter.jetbrains_ide.value].build_number
-  version       = var.latest ? local.json_data[local.key][0].version : var.jetbrains_ide_versions[data.coder_parameter.jetbrains_ide.value].version
 }

 data "coder_parameter" "jetbrains_ide" {
@@ -235,9 +193,9 @@ data "coder_parameter" "jetbrains_ide" {
  dynamic "option" {
    for_each = var.jetbrains_ides
    content {
-      icon  = local.jetbrains_ides[option.value].icon
-      name  = local.jetbrains_ides[option.value].name
-      value = option.value
+      icon  = lookup(local.jetbrains_ides, option.value).icon
+      name  = lookup(local.jetbrains_ides, option.value).name
+      value = lookup(local.jetbrains_ides, option.value).identifier
    }
  }
 }
@@ -247,8 +205,8 @@ data "coder_workspace" "me" {}
 resource "coder_app" "gateway" {
  agent_id     = var.agent_id
  slug         = "gateway"
-  display_name = try(lookup(data.coder_parameter.jetbrains_ide.option, data.coder_parameter.jetbrains_ide.value).name, "JetBrains IDE")
-  icon         = try(lookup(data.coder_parameter.jetbrains_ide.option, data.coder_parameter.jetbrains_ide.value).icon, "/icon/gateway.svg")
+  display_name = try(lookup(local.jetbrains_ides, data.coder_parameter.jetbrains_ide.value).name, "JetBrains IDE")
+  icon         = try(lookup(local.jetbrains_ides, data.coder_parameter.jetbrains_ide.value).icon, "/icon/gateway.svg")
  external     = true
  order        = var.order
  url = join("", [
@@ -263,36 +221,36 @@ resource "coder_app" "gateway" {
    "&token=",
    "$SESSION_TOKEN",
    "&ide_product_code=",
-    data.coder_parameter.jetbrains_ide.value,
+    local.jetbrains_ides[data.coder_parameter.jetbrains_ide.value].identifier,
    "&ide_build_number=",
-    local.build_number,
+    local.jetbrains_ides[data.coder_parameter.jetbrains_ide.value].build_number,
    "&ide_download_link=",
-    local.download_link,
+    local.jetbrains_ides[data.coder_parameter.jetbrains_ide.value].download_link
  ])
 }

 output "identifier" {
-  value = local.identifier
+  value = data.coder_parameter.jetbrains_ide.value
 }

-output "display_name" {
-  value = local.display_name
+output "name" {
+  value = coder_app.gateway.display_name
 }

 output "icon" {
-  value = local.icon
+  value = coder_app.gateway.icon
 }

 output "download_link" {
-  value = local.download_link
+  value = lookup(local.jetbrains_ides, data.coder_parameter.jetbrains_ide.value).download_link
 }

 output "build_number" {
-  value = local.build_number
+  value = lookup(local.jetbrains_ides, data.coder_parameter.jetbrains_ide.value).build_number
 }

 output "version" {
-  value = local.version
+  value = var.jetbrains_ide_versions[data.coder_parameter.jetbrains_ide.value].version
 }

 output "url" {
--- a/package-lock.json
+++ b/package-lock.json
@@ -0,0 +1,263 @@
+{
+  "name": "modules",
+  "lockfileVersion": 3,
+  "requires": true,
+  "packages": {
+    "": {
+      "name": "modules",
+      "devDependencies": {
+        "bun-types": "^1.0.18",
+        "gray-matter": "^4.0.3",
+        "marked": "^12.0.0",
+        "prettier-plugin-sh": "^0.13.1",
+        "prettier-plugin-terraform-formatter": "^1.2.1"
+      },
+      "peerDependencies": {
+        "typescript": "^5.3.3"
+      }
+    },
+    "node_modules/@types/node": {
+      "version": "20.12.14",
+      "resolved": "https://registry.npmjs.org/@types/node/-/node-20.12.14.tgz",
+      "integrity": "sha512-scnD59RpYD91xngrQQLGkE+6UrHUPzeKZWhhjBSa3HSkwjbQc38+q3RoIVEwxQGRw3M+j5hpNAM+lgV3cVormg==",
+      "dev": true,
+      "dependencies": {
+        "undici-types": "~5.26.4"
+      }
+    },
+    "node_modules/@types/ws": {
+      "version": "8.5.10",
+      "resolved": "https://registry.npmjs.org/@types/ws/-/ws-8.5.10.tgz",
+      "integrity": "sha512-vmQSUcfalpIq0R9q7uTo2lXs6eGIpt9wtnLdMv9LVpIjCA/+ufZRozlVoVelIYixx1ugCBKDhn89vnsEGOCx9A==",
+      "dev": true,
+      "dependencies": {
+        "@types/node": "*"
+      }
+    },
+    "node_modules/argparse": {
+      "version": "1.0.10",
+      "resolved": "https://registry.npmjs.org/argparse/-/argparse-1.0.10.tgz",
+      "integrity": "sha512-o5Roy6tNG4SL/FOkCAN6RzjiakZS25RLYFrcMttJqbdd8BWrnA+fGz57iN5Pb06pvBGvl5gQ0B48dJlslXvoTg==",
+      "dev": true,
+      "dependencies": {
+        "sprintf-js": "~1.0.2"
+      }
+    },
+    "node_modules/bun-types": {
+      "version": "1.1.16",
+      "resolved": "https://registry.npmjs.org/bun-types/-/bun-types-1.1.16.tgz",
+      "integrity": "sha512-LpAh8dQe4NKvhSW390Rkftw0ume0moSkRm575e1JZ1PwI/dXjbXyjpntq+2F0bVW1FV7V6B8EfWx088b+dNurw==",
+      "dev": true,
+      "dependencies": {
+        "@types/node": "~20.12.8",
+        "@types/ws": "~8.5.10"
+      }
+    },
+    "node_modules/esprima": {
+      "version": "4.0.1",
+      "resolved": "https://registry.npmjs.org/esprima/-/esprima-4.0.1.tgz",
+      "integrity": "sha512-eGuFFw7Upda+g4p+QHvnW0RyTX/SVeJBDM/gCtMARO0cLuT2HcEKnTPvhjV6aGeqrCB/sbNop0Kszm0jsaWU4A==",
+      "dev": true,
+      "bin": {
+        "esparse": "bin/esparse.js",
+        "esvalidate": "bin/esvalidate.js"
+      },
+      "engines": {
+        "node": ">=4"
+      }
+    },
+    "node_modules/extend-shallow": {
+      "version": "2.0.1",
+      "resolved": "https://registry.npmjs.org/extend-shallow/-/extend-shallow-2.0.1.tgz",
+      "integrity": "sha512-zCnTtlxNoAiDc3gqY2aYAWFx7XWWiasuF2K8Me5WbN8otHKTUKBwjPtNpRs/rbUZm7KxWAaNj7P1a/p52GbVug==",
+      "dev": true,
+      "dependencies": {
+        "is-extendable": "^0.1.0"
+      },
+      "engines": {
+        "node": ">=0.10.0"
+      }
+    },
+    "node_modules/gray-matter": {
+      "version": "4.0.3",
+      "resolved": "https://registry.npmjs.org/gray-matter/-/gray-matter-4.0.3.tgz",
+      "integrity": "sha512-5v6yZd4JK3eMI3FqqCouswVqwugaA9r4dNZB1wwcmrD02QkV5H0y7XBQW8QwQqEaZY1pM9aqORSORhJRdNK44Q==",
+      "dev": true,
+      "dependencies": {
+        "js-yaml": "^3.13.1",
+        "kind-of": "^6.0.2",
+        "section-matter": "^1.0.0",
+        "strip-bom-string": "^1.0.0"
+      },
+      "engines": {
+        "node": ">=6.0"
+      }
+    },
+    "node_modules/is-extendable": {
+      "version": "0.1.1",
+      "resolved": "https://registry.npmjs.org/is-extendable/-/is-extendable-0.1.1.tgz",
+      "integrity": "sha512-5BMULNob1vgFX6EjQw5izWDxrecWK9AM72rugNr0TFldMOi0fj6Jk+zeKIt0xGj4cEfQIJth4w3OKWOJ4f+AFw==",
+      "dev": true,
+      "engines": {
+        "node": ">=0.10.0"
+      }
+    },
+    "node_modules/js-yaml": {
+      "version": "3.14.1",
+      "resolved": "https://registry.npmjs.org/js-yaml/-/js-yaml-3.14.1.tgz",
+      "integrity": "sha512-okMH7OXXJ7YrN9Ok3/SXrnu4iX9yOk+25nqX4imS2npuvTYDmo/QEZoqwZkYaIDk3jVvBOTOIEgEhaLOynBS9g==",
+      "dev": true,
+      "dependencies": {
+        "argparse": "^1.0.7",
+        "esprima": "^4.0.0"
+      },
+      "bin": {
+        "js-yaml": "bin/js-yaml.js"
+      }
+    },
+    "node_modules/kind-of": {
+      "version": "6.0.3",
+      "resolved": "https://registry.npmjs.org/kind-of/-/kind-of-6.0.3.tgz",
+      "integrity": "sha512-dcS1ul+9tmeD95T+x28/ehLgd9mENa3LsvDTtzm3vyBEO7RPptvAD+t44WVXaUjTBRcrpFeFlC8WCruUR456hw==",
+      "dev": true,
+      "engines": {
+        "node": ">=0.10.0"
+      }
+    },
+    "node_modules/marked": {
+      "version": "12.0.2",
+      "resolved": "https://registry.npmjs.org/marked/-/marked-12.0.2.tgz",
+      "integrity": "sha512-qXUm7e/YKFoqFPYPa3Ukg9xlI5cyAtGmyEIzMfW//m6kXwCy2Ps9DYf5ioijFKQ8qyuscrHoY04iJGctu2Kg0Q==",
+      "dev": true,
+      "bin": {
+        "marked": "bin/marked.js"
+      },
+      "engines": {
+        "node": ">= 18"
+      }
+    },
+    "node_modules/mvdan-sh": {
+      "version": "0.10.1",
+      "resolved": "https://registry.npmjs.org/mvdan-sh/-/mvdan-sh-0.10.1.tgz",
+      "integrity": "sha512-kMbrH0EObaKmK3nVRKUIIya1dpASHIEusM13S4V1ViHFuxuNxCo+arxoa6j/dbV22YBGjl7UKJm9QQKJ2Crzhg==",
+      "dev": true
+    },
+    "node_modules/prettier": {
+      "version": "3.3.2",
+      "resolved": "https://registry.npmjs.org/prettier/-/prettier-3.3.2.tgz",
+      "integrity": "sha512-rAVeHYMcv8ATV5d508CFdn+8/pHPpXeIid1DdrPwXnaAdH7cqjVbpJaT5eq4yRAFU/lsbwYwSF/n5iNrdJHPQA==",
+      "dev": true,
+      "peer": true,
+      "bin": {
+        "prettier": "bin/prettier.cjs"
+      },
+      "engines": {
+        "node": ">=14"
+      },
+      "funding": {
+        "url": "https://github.com/prettier/prettier?sponsor=1"
+      }
+    },
+    "node_modules/prettier-plugin-sh": {
+      "version": "0.13.1",
+      "resolved": "https://registry.npmjs.org/prettier-plugin-sh/-/prettier-plugin-sh-0.13.1.tgz",
+      "integrity": "sha512-ytMcl1qK4s4BOFGvsc9b0+k9dYECal7U29bL/ke08FEUsF/JLN0j6Peo0wUkFDG4y2UHLMhvpyd6Sd3zDXe/eg==",
+      "dev": true,
+      "dependencies": {
+        "mvdan-sh": "^0.10.1",
+        "sh-syntax": "^0.4.1"
+      },
+      "engines": {
+        "node": ">=16.0.0"
+      },
+      "funding": {
+        "url": "https://opencollective.com/unts"
+      },
+      "peerDependencies": {
+        "prettier": "^3.0.0"
+      }
+    },
+    "node_modules/prettier-plugin-terraform-formatter": {
+      "version": "1.2.1",
+      "resolved": "https://registry.npmjs.org/prettier-plugin-terraform-formatter/-/prettier-plugin-terraform-formatter-1.2.1.tgz",
+      "integrity": "sha512-rdzV61Bs/Ecnn7uAS/vL5usTX8xUWM+nQejNLZxt3I1kJH5WSeLEmq7LYu1wCoEQF+y7Uv1xGvPRfl3lIe6+tA==",
+      "dev": true,
+      "peerDependencies": {
+        "prettier": ">= 1.16.0"
+      },
+      "peerDependenciesMeta": {
+        "prettier": {
+          "optional": true
+        }
+      }
+    },
+    "node_modules/section-matter": {
+      "version": "1.0.0",
+      "resolved": "https://registry.npmjs.org/section-matter/-/section-matter-1.0.0.tgz",
+      "integrity": "sha512-vfD3pmTzGpufjScBh50YHKzEu2lxBWhVEHsNGoEXmCmn2hKGfeNLYMzCJpe8cD7gqX7TJluOVpBkAequ6dgMmA==",
+      "dev": true,
+      "dependencies": {
+        "extend-shallow": "^2.0.1",
+        "kind-of": "^6.0.0"
+      },
+      "engines": {
+        "node": ">=4"
+      }
+    },
+    "node_modules/sh-syntax": {
+      "version": "0.4.2",
+      "resolved": "https://registry.npmjs.org/sh-syntax/-/sh-syntax-0.4.2.tgz",
+      "integrity": "sha512-/l2UZ5fhGZLVZa16XQM9/Vq/hezGGbdHeVEA01uWjOL1+7Ek/gt6FquW0iKKws4a9AYPYvlz6RyVvjh3JxOteg==",
+      "dev": true,
+      "dependencies": {
+        "tslib": "^2.6.2"
+      },
+      "engines": {
+        "node": ">=16.0.0"
+      },
+      "funding": {
+        "url": "https://opencollective.com/unts"
+      }
+    },
+    "node_modules/sprintf-js": {
+      "version": "1.0.3",
+      "resolved": "https://registry.npmjs.org/sprintf-js/-/sprintf-js-1.0.3.tgz",
+      "integrity": "sha512-D9cPgkvLlV3t3IzL0D0YLvGA9Ahk4PcvVwUbN0dSGr1aP0Nrt4AEnTUbuGvquEC0mA64Gqt1fzirlRs5ibXx8g==",
+      "dev": true
+    },
+    "node_modules/strip-bom-string": {
+      "version": "1.0.0",
+      "resolved": "https://registry.npmjs.org/strip-bom-string/-/strip-bom-string-1.0.0.tgz",
+      "integrity": "sha512-uCC2VHvQRYu+lMh4My/sFNmF2klFymLX1wHJeXnbEJERpV/ZsVuonzerjfrGpIGF7LBVa1O7i9kjiWvJiFck8g==",
+      "dev": true,
+      "engines": {
+        "node": ">=0.10.0"
+      }
+    },
+    "node_modules/tslib": {
+      "version": "2.6.3",
+      "resolved": "https://registry.npmjs.org/tslib/-/tslib-2.6.3.tgz",
+      "integrity": "sha512-xNvxJEOUiWPGhUuUdQgAJPKOOJfGnIyKySOc09XkKsgdUV/3E2zvwZYdejjmRgPCgcym1juLH3226yA7sEFJKQ==",
+      "dev": true
+    },
+    "node_modules/typescript": {
+      "version": "5.5.2",
+      "resolved": "https://registry.npmjs.org/typescript/-/typescript-5.5.2.tgz",
+      "integrity": "sha512-NcRtPEOsPFFWjobJEtfihkLCZCXZt/os3zf8nTxjVH3RvTSxjrCamJpbExGvYOF+tFHc3pA65qpdwPbzjohhew==",
+      "peer": true,
+      "bin": {
+        "tsc": "bin/tsc",
+        "tsserver": "bin/tsserver"
+      },
+      "engines": {
+        "node": ">=14.17"
+      }
+    },
+    "node_modules/undici-types": {
+      "version": "5.26.5",
+      "resolved": "https://registry.npmjs.org/undici-types/-/undici-types-5.26.5.tgz",
+      "integrity": "sha512-JlCMO+ehdEIKqlFxk6IfVoAUVmgz7cU7zD/h9XZ0qzeosSHmUJVOzSQvvYSYWXkFXC+IfLKSIffhv0sVZup6pA==",
+      "dev": true
+    }
+  }
+}
--- a/test.ts
+++ b/test.ts
@@ -29,8 +29,10 @@ export const runContainer = async (
  return containerID.trim();
 };

-// executeScriptInContainer finds the only "coder_script"
-// resource in the given state and runs it in a container.
+/**
+ * Finds the only "coder_script" resource in the given state and runs it in a
+ * container.
+ */
 export const executeScriptInContainer = async (
  state: TerraformState,
  image: string,
@@ -76,27 +78,22 @@ export const execContainer = async (
  };
 };

+type TerraformStateResource = {
+  type: string;
+  name: string;
+  provider: string;
+  instances: [{ attributes: Record<string, any> }];
+};
+
 export interface TerraformState {
  outputs: {
    [key: string]: {
      type: string;
      value: any;
    };
-  }
-  resources: [
-    {
-      type: string;
-      name: string;
-      provider: string;
-      instances: [
-        {
-          attributes: {
-            [key: string]: any;
-          };
-        },
-      ];
-    },
-  ];
+  };
+
+  resources: [TerraformStateResource, ...TerraformStateResource[]];
 }

 export interface CoderScriptAttributes {
@@ -105,10 +102,11 @@ export interface CoderScriptAttributes {
  url: string;
 }

-// findResourceInstance finds the first instance of the given resource
-// type in the given state. If name is specified, it will only find
-// the instance with the given name.
-export const findResourceInstance = <T extends "coder_script" | string>(
+/**
+ * finds the first instance of the given resource type in the given state. If
+ * name is specified, it will only find the instance with the given name.
+ */
+export const findResourceInstance = <T extends string>(
  state: TerraformState,
  type: T,
  name?: string,
@@ -131,12 +129,13 @@ export const findResourceInstance = <T extends "coder_script" | string>(
  return resource.instances[0].attributes as any;
 };

-// testRequiredVariables creates a test-case
-// for each variable provided and ensures that
-// the apply fails without it.
-export const testRequiredVariables = (
+/**
+ * Creates a test-case for each variable provided and ensures that the apply
+ * fails without it.
+ */
+export const testRequiredVariables = <TVars extends Record<string, string>>(
  dir: string,
-  vars: Record<string, string>,
+  vars: TVars,
 ) => {
  // Ensures that all required variables are provided.
  it("required variables", async () => {
@@ -165,15 +164,19 @@ export const testRequiredVariables = (
  });
 };

-// runTerraformApply runs terraform apply in the given directory
-// with the given variables. It is fine to run in parallel with
-// other instances of this function, as it uses a random state file.
-export const runTerraformApply = async (
+/**
+ * Runs terraform apply in the given directory with the given variables. It is
+ * fine to run in parallel with other instances of this function, as it uses a
+ * random state file.
+ */
+export const runTerraformApply = async <
+  TVars extends Readonly<Record<string, string>>,
+>(
  dir: string,
-  vars: Record<string, string>,
-  env: Record<string, string> = {},
+  vars: TVars,
 ): Promise<TerraformState> => {
  const stateFile = `${dir}/${crypto.randomUUID()}.tfstate`;
+  const env = {};
  Object.keys(vars).forEach((key) => (env[`TF_VAR_${key}`] = vars[key]));
  const proc = spawn(
    [
@@ -203,7 +206,9 @@ export const runTerraformApply = async (
  return JSON.parse(content);
 };

-// runTerraformInit runs terraform init in the given directory.
+/**
+ * Runs terraform init in the given directory.
+ */
 export const runTerraformInit = async (dir: string) => {
  const proc = spawn(["terraform", "init"], {
    cwd: dir,
@@ -221,5 +226,5 @@ export const createJSONResponse = (obj: object, statusCode = 200): Response => {
      "Content-Type": "application/json",
    },
    status: statusCode,
-  })
-}
+  });
+};
--- a/vscode-desktop/main.test.ts
+++ b/vscode-desktop/main.test.ts
@@ -21,7 +21,7 @@ describe("vscode-desktop", async () => {
      "vscode://coder.coder-remote/open?owner=default&workspace=default&token=$SESSION_TOKEN",
    );

-    const resources: any = state.resources;
+    const resources = state.resources;
    expect(resources[1].instances[0].attributes.order).toBeNull();
  });

@@ -31,7 +31,7 @@ describe("vscode-desktop", async () => {
      order: "22",
    });

-    const resources: any = state.resources;
+    const resources = state.resources;
    expect(resources[1].instances[0].attributes.order).toBe(22);
  });
 });
--- a/vscode-web/README.md
+++ b/vscode-web/README.md
@@ -14,7 +14,7 @@ Automatically install [Visual Studio Code Server](https://code.visualstudio.com/
 ```tf
 module "vscode-web" {
  source         = "registry.coder.com/modules/vscode-web/coder"
-  version        = "1.0.11"
+  version        = "1.0.10"
  agent_id       = coder_agent.example.id
  accept_license = true
 }
@@ -29,7 +29,7 @@ module "vscode-web" {
 ```tf
 module "vscode-web" {
  source         = "registry.coder.com/modules/vscode-web/coder"
-  version        = "1.0.11"
+  version        = "1.0.10"
  agent_id       = coder_agent.example.id
  install_prefix = "/home/coder/.vscode-web"
  folder         = "/home/coder"
@@ -42,7 +42,7 @@ module "vscode-web" {
 ```tf
 module "vscode-web" {
  source         = "registry.coder.com/modules/vscode-web/coder"
-  version        = "1.0.11"
+  version        = "1.0.10"
  agent_id       = coder_agent.example.id
  extensions     = ["github.copilot", "ms-python.python", "ms-toolsai.jupyter"]
  accept_license = true
@@ -56,7 +56,7 @@ Configure VS Code's [settings.json](https://code.visualstudio.com/docs/getstarte
 ```tf
 module "vscode-web" {
  source     = "registry.coder.com/modules/vscode-web/coder"
-  version    = "1.0.11"
+  version    = "1.0.10"
  agent_id   = coder_agent.example.id
  extensions = ["dracula-theme.theme-dracula"]
  settings = {
--- a/windows-rdp/README.md
+++ b/windows-rdp/README.md
@@ -0,0 +1,35 @@
+---
+display_name: Windows RDP
+description: RDP Server and Web Client powered by Devolutions
+icon: ../.icons/desktop.svg
+maintainer_github: coder
+verified: false
+tags: [windows, rdp, web, desktop]
+---
+
+# Windows RDP
+
+Enable Remote Desktop + a web based client on Windows workspaces, powered by [devolutions-gateway](https://github.com/Devolutions/devolutions-gateway)
+
+[![Web RDP on Windows](https://cdn.loom.com/sessions/thumbnails/a5d98c7007a7417fb572aba1acf8d538-with-play.gif)](https://www.loom.com/share/a5d98c7007a7417fb572aba1acf8d538)
+
+## Usage
+
+```tf
+module "windows_rdp" {
+  count = data.coder_workspace.me.start_count
+  source = "github.com/coder/modules//windows-rdp?ref=web-rdp"
+  agent_id = resource.coder_agent.main.id
+  resource_id = resource.google_compute_instance.dev[0].id
+}
+```
+
+## Tested on
+
+- ✅ GCP with Windows Server 2022: [Example template](https://gist.github.com/bpmct/18918b8cab9f20295e5c4039b92b5143)
+
+## Roadmap
+
+- [ ] Test on additional cloud providers
+- [ ] Automatically establish web RDP session when users click "web RDP"
+  > This may require forking [the webapp from devolutions-gateway](https://github.com/Devolutions/devolutions-gateway/tree/master/webapp), modifying `webapp/`, building, and specifying a new [static root path](https://github.com/Devolutions/devolutions-gateway/blob/a884cbb8ff313496fb3d4072e67ef75350c40c03/devolutions-gateway/tests/config.rs#L271). Ideally we can upstream this functionality.
--- a/windows-rdp/devolutions-patch.js
+++ b/windows-rdp/devolutions-patch.js
@@ -0,0 +1,410 @@
+// @ts-check
+/**
+ * @file Defines the custom logic for patching in UI changes/behavior into the
+ * base Devolutions Gateway Angular app.
+ *
+ * Defined as a JS file to remove the need to have a separate compilation step.
+ * It is highly recommended that you work on this file from within VS Code so
+ * that you can take advantage of the @ts-check directive and get some type-
+ * checking still.
+ *
+ * Other notes about the weird ways this file is set up:
+ * - A lot of the HTML selectors in this file will look nonstandard. This is
+ *   because they are actually custom Angular components.
+ * - It is strongly advised that you avoid template literals that use the
+ *   placeholder syntax via the dollar sign. The Terraform script looks for
+ *   these characters so that it can inject Coder-specific values, so any
+ *   template literal that uses the character actually needs to double up each
+ *   of them. There are already a few places in this file where it couldn't be
+ *   avoided, but avoiding this as much as possible will save you some headache.
+ * - All the CSS should be written via custom style tags and the !important
+ *   directive (as much as that is a bad idea most of the time). We do not
+ *   control the Angular app, so we have to modify things from afar to ensure
+ *   that as Angular's internal state changes, it doesn't modify its HTML nodes
+ *   in a way that causes our custom styles to get wiped away.
+ *
+ * @typedef {Readonly<{ querySelector: string; value: string; }>} FormFieldEntry
+ * @typedef {Readonly<Record<string, FormFieldEntry>>} FormFieldEntries
+ */
+
+/**
+ * The communication protocol to set Devolutions to.
+ */
+const PROTOCOL = "RDP";
+
+/**
+ * The hostname to use with Devolutions.
+ */
+const HOSTNAME = "localhost";
+
+/**
+ * How often to poll the screen for the main Devolutions form.
+ */
+const SCREEN_POLL_INTERVAL_MS = 500;
+
+/**
+ * The fields in the Devolutions sign-in form that should be populated with
+ * values from the Coder workspace.
+ *
+ * All properties should be defined as placeholder templates in the form
+ * VALUE_NAME. The Coder module, when spun up, should then run some logic to
+ * replace the template slots with actual values. These values should never
+ * change from within JavaScript itself.
+ *
+ * @satisfies {FormFieldEntries}
+ */
+const formFieldEntries = {
+  /** @readonly */
+  username: {
+    /** @readonly */
+    querySelector: "web-client-username-control input",
+
+    /** @readonly */
+    value: "${CODER_USERNAME}",
+  },
+
+  /** @readonly */
+  password: {
+    /** @readonly */
+    querySelector: "web-client-password-control input",
+
+    /** @readonly */
+    value: "${CODER_PASSWORD}",
+  },
+};
+
+/**
+ * Handles typing in the values for the input form. All values are written
+ * immediately, even though that would be physically impossible with a real
+ * keyboard.
+ *
+ * Note: this code will never break, but you might get warnings in the console
+ * from Angular about unexpected value changes. Angular patches over a lot of
+ * the built-in browser APIs to support its component change detection system.
+ * As part of that, it has validations for checking whether an input it
+ * previously had control over changed without it doing anything.
+ *
+ * But the only way to simulate a keyboard input is by setting the input's
+ * .value property, and then firing an input event. So basically, the inner
+ * value will change, which Angular won't be happy about, but then the input
+ * event will fire and sync everything back together.
+ *
+ * @param {HTMLInputElement} inputField
+ * @param {string} inputText
+ * @returns {Promise<void>}
+ */
+function setInputValue(inputField, inputText) {
+  return new Promise((resolve, reject) => {
+    // Adding timeout for input event, even though we'll be dispatching it
+    // immediately, just in the off chance that something in the Angular app
+    // intercepts it or stops it from propagating properly
+    const timeoutId = window.setTimeout(() => {
+      reject(new Error("Input event did not get processed correctly in time."));
+    }, 3_000);
+
+    const handleSuccessfulDispatch = () => {
+      window.clearTimeout(timeoutId);
+      inputField.removeEventListener("input", handleSuccessfulDispatch);
+      resolve();
+    };
+
+    inputField.addEventListener("input", handleSuccessfulDispatch);
+
+    // Code assumes that Angular will have an event handler in place to handle
+    // the new event
+    const inputEvent = new Event("input", {
+      bubbles: true,
+      cancelable: true,
+    });
+
+    inputField.value = inputText;
+    inputField.dispatchEvent(inputEvent);
+  });
+}
+
+/**
+ * Takes a Devolutions remote session form, auto-fills it with data, and then
+ * submits it.
+ *
+ * The logic here is more convoluted than it should be for two main reasons:
+ * 1. Devolutions' HTML markup has errors. There are labels, but they aren't
+ *    bound to the inputs they're supposed to describe. This means no easy hooks
+ *    for selecting the elements, unfortunately.
+ * 2. Trying to modify the .value properties on some of the inputs doesn't
+ *    work. Probably some combo of Angular data-binding and some inputs having
+ *    the readonly attribute. Have to simulate user input to get around this.
+ *
+ * @param {HTMLFormElement} myForm
+ * @returns {Promise<void>}
+ */
+async function autoSubmitForm(myForm) {
+  const setProtocolValue = () => {
+    /** @type {HTMLDivElement | null} */
+    const protocolDropdownTrigger = myForm.querySelector('div[role="button"]');
+    if (protocolDropdownTrigger === null) {
+      throw new Error("No clickable trigger for setting protocol value");
+    }
+
+    protocolDropdownTrigger.click();
+
+    // Can't use form as container for querying the list of dropdown options,
+    // because the elements don't actually exist inside the form. They're placed
+    // in the top level of the HTML doc, and repositioned to make it look like
+    // they're part of the form. Avoids CSS stacking context issues, maybe?
+    /** @type {HTMLLIElement | null} */
+    const protocolOption = document.querySelector(
+      'p-dropdownitem[ng-reflect-label="' + PROTOCOL + '"] li',
+    );
+
+    if (protocolOption === null) {
+      throw new Error(
+        "Unable to find protocol option on screen that matches desired protocol",
+      );
+    }
+
+    protocolOption.click();
+  };
+
+  const setHostname = () => {
+    /** @type {HTMLInputElement | null} */
+    const hostnameInput = myForm.querySelector("p-autocomplete#hostname input");
+
+    if (hostnameInput === null) {
+      throw new Error("Unable to find field for adding hostname");
+    }
+
+    return setInputValue(hostnameInput, HOSTNAME);
+  };
+
+  const setCoderFormFieldValues = async () => {
+    // The RDP form will not appear on screen unless the dropdown is set to use
+    // the RDP protocol
+    const rdpSubsection = myForm.querySelector("rdp-form");
+    if (rdpSubsection === null) {
+      throw new Error(
+        "Unable to find RDP subsection. Is the value of the protocol set to RDP?",
+      );
+    }
+
+    for (const { value, querySelector } of Object.values(formFieldEntries)) {
+      /** @type {HTMLInputElement | null} */
+      const input = document.querySelector(querySelector);
+
+      if (input === null) {
+        throw new Error(
+          'Unable to element that matches query "' + querySelector + '"',
+        );
+      }
+
+      await setInputValue(input, value);
+    }
+  };
+
+  const triggerSubmission = () => {
+    /** @type {HTMLButtonElement | null} */
+    const submitButton = myForm.querySelector(
+      'p-button[ng-reflect-type="submit"] button',
+    );
+
+    if (submitButton === null) {
+      throw new Error("Unable to find submission button");
+    }
+
+    if (submitButton.disabled) {
+      throw new Error(
+        "Unable to submit form because submit button is disabled. Are all fields filled out correctly?",
+      );
+    }
+
+    submitButton.click();
+  };
+
+  setProtocolValue();
+  await setHostname();
+  await setCoderFormFieldValues();
+  triggerSubmission();
+}
+
+/**
+ * Sets up logic for auto-populating the form data when the form appears on
+ * screen.
+ *
+ * @returns {void}
+ */
+function setupFormDetection() {
+  /** @type {HTMLFormElement | null} */
+  let formValueFromLastMutation = null;
+
+  /** @returns {void} */
+  const onDynamicTabMutation = () => {
+    /** @type {HTMLFormElement | null} */
+    const latestForm = document.querySelector("web-client-form > form");
+
+    // Only try to auto-fill if we went from having no form on screen to
+    // having a form on screen. That way, we don't accidentally override the
+    // form if the user is trying to customize values, and this essentially
+    // makes the script values function as default values
+    const mounted = formValueFromLastMutation === null && latestForm !== null;
+    if (mounted) {
+      autoSubmitForm(latestForm);
+    }
+
+    formValueFromLastMutation = latestForm;
+  };
+
+  /** @type {number | undefined} */
+  let pollingId = undefined;
+
+  /** @returns {void} */
+  const checkScreenForDynamicTab = () => {
+    const dynamicTab = document.querySelector("web-client-dynamic-tab");
+
+    // Keep polling until the main content container is on screen
+    if (dynamicTab === null) {
+      return;
+    }
+
+    window.clearInterval(pollingId);
+
+    // Call the mutation callback manually, to ensure it runs at least once
+    onDynamicTabMutation();
+
+    // Having the mutation observer is kind of an extra safety net that isn't
+    // really expected to run that often. Most of the content in the dynamic
+    // tab is being rendered through Canvas, which won't trigger any mutations
+    // that the observer can detect
+    const dynamicTabObserver = new MutationObserver(onDynamicTabMutation);
+    dynamicTabObserver.observe(dynamicTab, {
+      subtree: true,
+      childList: true,
+    });
+  };
+
+  pollingId = window.setInterval(
+    checkScreenForDynamicTab,
+    SCREEN_POLL_INTERVAL_MS,
+  );
+}
+
+/**
+ * Sets up custom styles for hiding default Devolutions elements that Coder
+ * users shouldn't need to care about.
+ *
+ * @returns {void}
+ */
+function setupAlwaysOnStyles() {
+  const styleId = "coder-patch--styles-always-on";
+  const existingContainer = document.querySelector("#" + styleId);
+  if (existingContainer) {
+    return;
+  }
+
+  const styleContainer = document.createElement("style");
+  styleContainer.id = styleId;
+  styleContainer.innerHTML = `
+    /* app-menu corresponds to the sidebar of the default view. */
+    app-menu {
+      display: none !important;
+    }
+  `;
+
+  document.head.appendChild(styleContainer);
+}
+
+function hideFormForInitialSubmission() {
+  const styleId = "coder-patch--styles-initial-submission";
+  const cssOpacityVariableName = "--coder-opacity-multiplier";
+
+  /** @type {HTMLStyleElement | null} */
+  let styleContainer = document.querySelector("#" + styleId);
+  if (!styleContainer) {
+    styleContainer = document.createElement("style");
+    styleContainer.id = styleId;
+    styleContainer.innerHTML = `
+      /*
+        Have to use opacity instead of visibility, because the element still
+        needs to be interactive via the script so that it can be auto-filled.
+      */
+      :root {
+        /*
+          Can be 0 or 1. Start off invisible to avoid risks of UI flickering,
+          but the rest of the function should be in charge of making the form
+          container visible again if something goes wrong during setup.
+
+          Double dollar sign needed to avoid Terraform script false positives
+        */
+        $${cssOpacityVariableName}: 0;
+      }
+
+      /*
+        web-client-form is the container for the main session form, while
+        the div is for the dropdown that is used for selecting the protocol.
+        The dropdown is not inside of the form for CSS styling reasons, so we
+        need to select both.
+       */
+      web-client-form,
+      body > div.p-overlay {
+        /*
+          Double dollar sign needed to avoid Terraform script false positives
+        */
+        opacity: calc(100% * var($${cssOpacityVariableName})) !important;
+      }
+    `;
+
+    document.head.appendChild(styleContainer);
+  }
+
+  // The root node being undefined should be physically impossible (if it's
+  // undefined, the browser itself is busted), but we need to do a type check
+  // here so that the rest of the function doesn't need to do type checks over
+  // and over.
+  const rootNode = document.querySelector(":root");
+  if (!(rootNode instanceof HTMLHtmlElement)) {
+    // Remove the container entirely because if the browser is busted, who knows
+    // if the CSS variables can be applied correctly. Better to have something
+    // be a bit more ugly/painful to use, than have it be impossible to use
+    styleContainer.remove();
+    return;
+  }
+
+  // It's safe to make the form visible preemptively because Devolutions
+  // outputs the Windows view through an HTML canvas that it overlays on top
+  // of the rest of the app. Even if the form isn't hidden at the style level,
+  // it will still be covered up.
+  const restoreOpacity = () => {
+    rootNode.style.setProperty(cssOpacityVariableName, "1");
+  };
+
+  // If this file gets more complicated, it might make sense to set up the
+  // timeout and event listener so that if one triggers, it cancels the other,
+  // but having restoreOpacity run more than once is a no-op for right now.
+  // Not a big deal if these don't get cleaned up.
+
+  // Have the form automatically reappear no matter what, so that if something
+  // does break, the user isn't left out to dry
+  window.setTimeout(restoreOpacity, 5_000);
+
+  /** @type {HTMLFormElement | null} */
+  const form = document.querySelector("web-client-form > form");
+  form?.addEventListener(
+    "submit",
+    () => {
+      // Not restoring opacity right away just to give the HTML canvas a little
+      // bit of time to get spun up and cover up the main form
+      window.setTimeout(restoreOpacity, 1_000);
+    },
+    { once: true },
+  );
+}
+
+// Always safe to call these immediately because even if the Angular app isn't
+// loaded by the time the function gets called, the CSS will always be globally
+// available for when Angular is finally ready
+setupAlwaysOnStyles();
+hideFormForInitialSubmission();
+
+if (document.readyState === "loading") {
+  document.addEventListener("DOMContentLoaded", setupFormDetection);
+} else {
+  setupFormDetection();
+}
--- a/windows-rdp/main.test.ts
+++ b/windows-rdp/main.test.ts
@@ -0,0 +1,72 @@
+import { describe, expect, it, test } from "bun:test";
+import {
+  executeScriptInContainer,
+  runTerraformApply,
+  runTerraformInit,
+  testRequiredVariables,
+} from "../test";
+
+type TestVariables = Readonly<{
+  agent_id: string;
+  resource_id: string;
+  admin_username?: string;
+  admin_password?: string;
+}>;
+
+describe("Web RDP", async () => {
+  await runTerraformInit(import.meta.dir);
+  testRequiredVariables<TestVariables>(import.meta.dir, {
+    agent_id: "foo",
+    resource_id: "bar",
+  });
+
+  it("Installs the Devolutions Gateway Angular app locally on the machine", async () => {
+    const state = await runTerraformApply<TestVariables>(import.meta.dir, {
+      agent_id: "foo",
+      resource_id: "bar",
+    });
+
+    throw new Error("Not implemented yet");
+  });
+
+  /**
+   * @todo Verify that the HTML file has been modified, and that the JS file is
+   * also part of the file system
+   */
+  it("Patches the Devolutions Angular app's .html file to include an import for the custom JS file", async () => {
+    const state = await runTerraformApply<TestVariables>(import.meta.dir, {
+      agent_id: "foo",
+      resource_id: "bar",
+    });
+
+    throw new Error("Not implemented yet");
+  });
+
+  it("Injects Terraform's username and password into the JS patch file", async () => {
+    throw new Error("Not implemented yet");
+
+    // Test that things work with the default username/password
+    const defaultState = await runTerraformApply<TestVariables>(
+      import.meta.dir,
+      {
+        agent_id: "foo",
+        resource_id: "bar",
+      },
+    );
+
+    const output = await executeScriptInContainer(defaultState, "alpine");
+
+    // Test that custom usernames/passwords are also forwarded correctly
+    const customUsername = "crouton";
+    const customPassword = "VeryVeryVeryVeryVerySecurePassword97!";
+    const customizedState = await runTerraformApply<TestVariables>(
+      import.meta.dir,
+      {
+        agent_id: "foo",
+        resource_id: "bar",
+        admin_username: customUsername,
+        admin_password: customPassword,
+      },
+    );
+  });
+});
--- a/windows-rdp/main.tf
+++ b/windows-rdp/main.tf
@@ -0,0 +1,173 @@
+terraform {
+  required_version = ">= 1.0"
+
+  required_providers {
+    coder = {
+      source  = "coder/coder"
+      version = ">= 0.17"
+    }
+  }
+}
+
+variable "agent_id" {
+  type        = string
+  description = "The ID of a Coder agent."
+}
+
+variable "resource_id" {
+  type        = string
+  description = "The ID of the primary Coder resource (e.g. VM)."
+}
+
+variable "admin_username" {
+  type    = string
+  default = "Administrator"
+}
+
+variable "admin_password" {
+  type      = string
+  default   = "coderRDP!"
+  sensitive = true
+}
+
+resource "coder_script" "windows-rdp" {
+  agent_id     = var.agent_id
+  display_name = "windows-rdp"
+  icon         = "https://svgur.com/i/158F.svg" # TODO: add to Coder icons
+  script       = <<EOF
+  function Set-AdminPassword {
+      param (
+          [string]$adminPassword
+      )
+      # Set admin password
+      Get-LocalUser -Name "${var.admin_username}" | Set-LocalUser -Password (ConvertTo-SecureString -AsPlainText $adminPassword -Force)
+      # Enable admin user
+      Get-LocalUser -Name "${var.admin_username}" | Enable-LocalUser
+  }
+
+  function Configure-RDP {
+      # Enable RDP
+      New-ItemProperty -Path 'HKLM:\System\CurrentControlSet\Control\Terminal Server' -Name "fDenyTSConnections" -Value 0 -PropertyType DWORD -Force
+      # Disable NLA
+      New-ItemProperty -Path 'HKLM:\System\CurrentControlSet\Control\Terminal Server\WinStations\RDP-Tcp' -Name "UserAuthentication" -Value 0 -PropertyType DWORD -Force
+      New-ItemProperty -Path 'HKLM:\System\CurrentControlSet\Control\Terminal Server\WinStations\RDP-Tcp' -Name "SecurityLayer" -Value 1 -PropertyType DWORD -Force
+      # Enable RDP through Windows Firewall
+      Enable-NetFirewallRule -DisplayGroup "Remote Desktop"
+  }
+
+  function Install-DevolutionsGateway {
+      # Define the module name and version
+      $moduleName = "DevolutionsGateway"
+      $moduleVersion = "2024.1.5"
+
+      Install-Module -Name $moduleName -RequiredVersion $moduleVersion -Force -Confirm:$false -SkipPublisherCheck
+      
+      try {
+          # Try to import the module directly
+          Import-Module $moduleName -ErrorAction Stop
+      } catch {
+          # If it fails, install and then import the module
+
+          # Construct the module path for system-wide installation
+          $moduleBasePath = "C:\Windows\system32\config\systemprofile\Documents\PowerShell\Modules\$moduleName\$moduleVersion"
+          $modulePath = Join-Path -Path $moduleBasePath -ChildPath "$moduleName.psd1"
+
+          # Import the module using the full path
+          Import-Module $modulePath
+      }
+
+      Install-DGatewayPackage
+
+      # Configure Devolutions Gateway
+      $Hostname = "localhost"
+      $HttpListener = New-DGatewayListener 'http://*:7171' 'http://*:7171'
+      $WebApp = New-DGatewayWebAppConfig -Enabled $true -Authentication None
+      $ConfigParams = @{
+          Hostname = $Hostname
+          Listeners = @($HttpListener)
+          WebApp = $WebApp
+      }
+      Set-DGatewayConfig @ConfigParams
+      New-DGatewayProvisionerKeyPair -Force
+
+      # Configure and start the Windows service
+      Set-Service 'DevolutionsGateway' -StartupType 'Automatic'
+      Start-Service 'DevolutionsGateway'
+  }
+
+  function Patch-Devolutions-HTML {
+      $root = "C:\Program Files\Devolutions\Gateway\webapp\client"
+      $devolutionsHtml = "$root\index.html"
+      $patch = '<script defer id="coder-patch" src="coder.js"></script>'
+      
+      # Always copy the file in case we change it.
+      @'
+  ${templatefile("${path.module}/devolutions-patch.js", {
+      CODER_USERNAME : var.admin_username,
+      CODER_PASSWORD : var.admin_password,
+  })}
+  '@ | Set-Content "$root\coder.js"
+
+      # Only inject the src if we have not before.
+      $isPatched = Select-String -Path "$devolutionsHtml" -Pattern "$patch" -SimpleMatch
+      if ($isPatched -eq $null) {
+          (Get-Content $devolutionsHtml).Replace('</app-root>', "</app-root>$patch") | Set-Content $devolutionsHtml
+      }
+  }
+
+  Set-AdminPassword -adminPassword "${var.admin_password}"
+  Configure-RDP
+  Install-DevolutionsGateway
+  Patch-Devolutions-HTML
+
+  EOF
+
+  run_on_start = true
+}
+
+resource "coder_app" "windows-rdp" {
+  agent_id     = var.agent_id
+  slug         = "web-rdp"
+  display_name = "Web RDP"
+  url          = "http://localhost:7171"
+  icon         = "https://svgur.com/i/158F.svg"
+  subdomain    = true
+
+  healthcheck {
+    url       = "http://localhost:7171"
+    interval  = 5
+    threshold = 15
+  }
+}
+
+resource "coder_app" "rdp-docs" {
+  agent_id     = var.agent_id
+  display_name = "Local RDP"
+  slug         = "rdp-docs"
+  icon         = "https://raw.githubusercontent.com/matifali/logos/main/windows.svg"
+  url          = "https://coder.com/docs/v2/latest/ides/remote-desktops#rdp-desktop"
+  external     = true
+}
+
+# For some reason this is not rendering, commented out for now
+# resource "coder_metadata" "rdp_details" {
+#   resource_id = var.resource_id
+#   daily_cost  = 0
+#   item {
+#     key   = "Host"
+#     value = "localhost"
+#   }
+#   item {
+#     key   = "Port"
+#     value = "3389"
+#   }
+#   item {
+#     key   = "Username"
+#     value = "Administrator"
+#   }
+#   item {
+#     key       = "Password"
+#     value     = var.admin_password
+#     sensitive = true
+#   }
+# }
Author	SHA1	Message	Date
Ben Potter	cb075aa035	fix order	2024-06-26 16:44:21 -05:00
Ben Potter	9864408643	fix: aws starter template compatability	2024-06-26 16:34:52 -05:00
Parkreiner	de00f6334f	chore: add type parameter for testRequiredVariables	2024-06-26 19:00:42 +00:00
Parkreiner	264584e673	fix: make comments for test helpers exportable	2024-06-26 17:59:12 +00:00
Parkreiner	83ecba2293	wip: commit current progress	2024-06-26 17:21:39 +00:00
Parkreiner	b2807640aa	wip: commit progress on main test file	2024-06-26 16:01:08 +00:00
Parkreiner	33d44fdf17	fix: remove unneeded any types	2024-06-26 16:00:57 +00:00
Parkreiner	f335cd343d	fix: update type definitions for helpers	2024-06-26 16:00:40 +00:00
Parkreiner	aebf095075	refactor: clean up patch logic for clarity	2024-06-26 14:37:14 +00:00
Parkreiner	b283ac3129	docs: fix misleading typo in comment	2024-06-25 21:54:13 +00:00
Parkreiner	5f418c3253	docs: add comments about necessary double dollar signs	2024-06-25 21:51:21 +00:00
Parkreiner	b09c4cb084	fix: speed up code for filling in form	2024-06-25 21:35:53 +00:00
Parkreiner	8aff87fdf7	fix: add logic for hiding the dropdown of protocol options	2024-06-25 21:20:42 +00:00
Parkreiner	f3c30abeb4	fix: make form hiding logic run on webpage opening	2024-06-25 21:03:02 +00:00
Parkreiner	a9a75b675f	fix: add more changes to opacity logic	2024-06-25 21:01:11 +00:00
Parkreiner	ef4c87e48e	fix: simplify code for hiding form	2024-06-25 20:45:39 +00:00
Parkreiner	1a0a8659cc	wip: update logic for hiding form to avoid whiffs	2024-06-25 20:40:44 +00:00
Parkreiner	c7a4fced4c	fix: update instanceof check	2024-06-25 20:15:18 +00:00
Parkreiner	5ec1b207d1	docs: remove now-inaccurate comment	2024-06-25 19:58:56 +00:00
Parkreiner	702271133f	fix: update HTML query selector	2024-06-25 19:57:48 +00:00
Parkreiner	652fc6b84f	refactor: clean up form code	2024-06-25 19:55:14 +00:00
Parkreiner	8195cf4453	wip: add current code for hiding Devolutions form	2024-06-25 19:48:44 +00:00
Parkreiner	d5cfadb4e7	fix: remove template literal dollar signs	2024-06-25 17:03:54 +00:00
Parkreiner	fba0f842a9	fix: remove regex search from Select-String	2024-06-24 21:47:01 +00:00
Parkreiner	14e3fc5b6b	fix: whitespace	2024-06-24 21:13:15 +00:00
Parkreiner	0b6975c266	fix: escape quotes	2024-06-24 20:41:45 +00:00
Parkreiner	d530d68b12	fix: more money, more problems	2024-06-24 20:28:44 +00:00
Parkreiner	047ccd67ca	fix: dolla dolla	2024-06-24 20:24:49 +00:00
Parkreiner	c7aa8253e3	fix: dolla dolla	2024-06-24 20:22:25 +00:00
Parkreiner	452f41aa86	fix: add parenthesis	2024-06-24 20:17:31 +00:00
Parkreiner	29209d546e	fix: update typo in powershell script Co-authored-by: Asher <ash@coder.com>	2024-06-24 20:13:11 +00:00
Parkreiner	aab5e55663	fix: update script frequency	2024-06-24 20:10:22 +00:00
Parkreiner	ff96b3f653	wip: commit current progress for devolutions patch	2024-06-24 20:07:39 +00:00
Parkreiner	20795aa2b6	chore: add script file for overriding Devolutions	2024-06-24 19:37:31 +00:00
Ben	b93471a381	chore: add admin username	2024-04-24 22:39:24 +00:00
Ben	53083a5718	add more context on auto login	2024-04-06 20:46:50 +00:00
Ben	7de78d2ef5	add tags	2024-04-06 20:36:55 +00:00
Ben	89135671b2	fix module usage	2024-04-06 20:34:06 +00:00
Ben	ac648cc0a9	add thumbnail	2024-04-06 20:32:52 +00:00
Ben	748a180ac3	add temp link to example template	2024-04-06 20:18:58 +00:00
Ben	ec922c7c3d	remove metadata for now	2024-04-06 20:13:50 +00:00
Ben	9f8eee55b2	rename script	2024-04-06 20:11:59 +00:00
Ben	0e7644b284	remove count	2024-04-06 20:05:57 +00:00
Ben	bf06e8d3ac	fix agent id	2024-04-06 20:04:28 +00:00
Ben	12fd16f701	add metadata and local instructions	2024-04-06 20:01:57 +00:00
Ben	1197e6bf0d	fix port typo	2024-04-06 19:51:51 +00:00
Ben	c5c521fabd	feat: add web RDP module	2024-04-06 19:48:37 +00:00