Remove old config cleanup

These should be long gone.  The old config would not affect the plugin
going forward anyway because the syntax is going to change to support
multiple deployment.

Author: code-asher

src/sshConfig.test.ts

@@ -138,76 +138,6 @@ Host *
   })
 })
 
-it("removes old coder SSH config and adds the new one", async () => {
-  const existentSSHConfig = `Host coder-vscode--*
-  ConnectTimeout=0
-  HostName coder.something
-  LogLevel ERROR
-  ProxyCommand command
-  StrictHostKeyChecking=no
-  UserKnownHostsFile=/dev/null`
-  mockFileSystem.readFile.mockResolvedValueOnce(existentSSHConfig)
-
-  const sshConfig = new SSHConfig(sshFilePath, mockFileSystem)
-  await sshConfig.load()
-  await sshConfig.update({
-    Host: "coder-vscode--*",
-    ProxyCommand: "some-command-here",
-    ConnectTimeout: "0",
-    StrictHostKeyChecking: "no",
-    UserKnownHostsFile: "/dev/null",
-    LogLevel: "ERROR",
-  })
-
-  const expectedOutput = `# --- START CODER VSCODE ---
-Host coder-vscode--*
-  ConnectTimeout 0
-  LogLevel ERROR
-  ProxyCommand some-command-here
-  StrictHostKeyChecking no
-  UserKnownHostsFile /dev/null
-# --- END CODER VSCODE ---`
-
-  expect(mockFileSystem.writeFile).toBeCalledWith(sshFilePath, expectedOutput, {
-    encoding: "utf-8",
-    mode: 384,
-  })
-})
-
-it("it does not remove a user-added block that only matches the host of an old coder SSH config", async () => {
-  const existentSSHConfig = `Host coder-vscode--*
-  ForwardAgent=yes`
-  mockFileSystem.readFile.mockResolvedValueOnce(existentSSHConfig)
-
-  const sshConfig = new SSHConfig(sshFilePath, mockFileSystem)
-  await sshConfig.load()
-  await sshConfig.update({
-    Host: "coder-vscode--*",
-    ProxyCommand: "some-command-here",
-    ConnectTimeout: "0",
-    StrictHostKeyChecking: "no",
-    UserKnownHostsFile: "/dev/null",
-    LogLevel: "ERROR",
-  })
-
-  const expectedOutput = `Host coder-vscode--*
-  ForwardAgent=yes
-
-# --- START CODER VSCODE ---
-Host coder-vscode--*
-  ConnectTimeout 0
-  LogLevel ERROR
-  ProxyCommand some-command-here
-  StrictHostKeyChecking no
-  UserKnownHostsFile /dev/null
-# --- END CODER VSCODE ---`
-
-  expect(mockFileSystem.writeFile).toBeCalledWith(sshFilePath, expectedOutput, {
-    encoding: "utf-8",
-    mode: 384,
-  })
-})
-
 it("override values", async () => {
   mockFileSystem.readFile.mockRejectedValueOnce("No file found")
   const sshConfig = new SSHConfig(sshFilePath, mockFileSystem)

src/sshConfig.ts

@@ -103,9 +103,6 @@ export class SSHConfig {
   }
 
   async update(values: SSHValues, overrides: Record<string, string> = defaultSSHConfigResponse) {
-    // We should remove this in March 2023 because there is not going to have
-    // old configs
-    this.cleanUpOldConfig()
     const block = this.getBlock()
     const newBlock = this.buildBlock(values, overrides)
     if (block) {
@@ -116,16 +113,6 @@ export class SSHConfig {
     await this.save()
   }
 
-  private async cleanUpOldConfig() {
-    const raw = this.getRaw()
-    const oldConfig = raw.split("\n\n").find((config) => config.startsWith("Host coder-vscode--*"))
-    // Perform additional sanity check that the block also contains a
-    // ProxyCommand, otherwise it might be a different block.
-    if (oldConfig && oldConfig.includes(" ProxyCommand ")) {
-      this.raw = raw.replace(oldConfig, "")
-    }
-  }
-
   private getBlock(): Block | undefined {
     const raw = this.getRaw()
     const startBlockIndex = raw.indexOf(this.startBlockComment)