Merge ecb29c4 into 1b80244

Author: code-asher

.eslintrc.yaml

@@ -36,8 +36,8 @@ rules:
   import/order:
     [error, { alphabetize: { order: "asc" }, groups: [["builtin", "external", "internal"], "parent", "sibling"] }]
   no-async-promise-executor: off
-  # This isn't a real module, just types, which apparently doesn't resolve.
-  import/no-unresolved: [error, { ignore: ["express-serve-static-core"] }]
+  # These aren't real modules, just types, which apparently don't resolve.
+  import/no-unresolved: [error, { ignore: ["express-serve-static-core", "vscode"] }]
 
 settings:
   # Does not work with CommonJS unfortunately.

CHANGELOG.md

@@ -39,7 +39,7 @@ VS Code v0.00.0
 
 ### New Features
 
-- item
+- Add `VSCODE_PROXY_URI` env var for use in the terminal and extensions (#1510)
 
 ### Bug Fixes
 

ci/dev/postinstall.sh

@@ -3,15 +3,20 @@ set -euo pipefail
 
 main() {
   cd "$(dirname "$0")/../.."
+  source ./ci/lib.sh
 
-  echo "Installing code-server test dependencies..."
+  pushd test
+  echo "Installing dependencies for $PWD"
+  yarn install
+  popd
 
-  cd test
+  pushd test/e2e/extensions/test-extension
+  echo "Installing dependencies for $PWD"
   yarn install
-  cd ..
+  popd
 
-  cd vendor
-  echo "Installing vendor dependencies..."
+  pushd vendor
+  echo "Installing dependencies for $PWD"
 
   # * We install in 'modules' instead of 'node_modules' because VS Code's extensions
   # use a webpack config which cannot differentiate between its own node_modules
@@ -32,6 +37,8 @@ main() {
 
   # Finally, run the vendor `postinstall`
   yarn run postinstall
+
+  popd
 }
 
 main "$@"

ci/dev/test-e2e.sh

@@ -6,6 +6,11 @@ main() {
 
   source ./ci/lib.sh
 
+  pushd test/e2e/extensions/test-extension
+  echo "Building test extension"
+  yarn build
+  popd
+
   local dir="$PWD"
   if [[ ! ${CODE_SERVER_TEST_ENTRY-} ]]; then
     echo "Set CODE_SERVER_TEST_ENTRY to test another build of code-server"

ci/dev/test-unit.sh

@@ -3,12 +3,16 @@ set -euo pipefail
 
 main() {
   cd "$(dirname "$0")/../.."
-  cd test/unit/node/test-plugin
+  source ./ci/lib.sh
+
+  echo "Building test plugin"
+  pushd test/unit/node/test-plugin
   make -s out/index.js
+  popd
+
   # We must keep jest in a sub-directory. See ../../test/package.json for more
   # information. We must also run it from the root otherwise coverage will not
   # include our source files.
-  cd "$OLDPWD"
   CS_DISABLE_PLUGINS=true ./test/node_modules/.bin/jest "$@"
 }
 

src/common/util.ts

@@ -66,33 +66,42 @@ export const resolveBase = (base?: string): string => {
   return normalize(url.pathname)
 }
 
+let options: Options
+
 /**
  * Get options embedded in the HTML or query params.
  */
 export const getOptions = <T extends Options>(): T => {
-  let options: T
-  try {
-    options = JSON.parse(document.getElementById("coder-options")!.getAttribute("data-settings")!)
-  } catch (error) {
-    options = {} as T
-  }
+  if (!options) {
+    try {
+      options = JSON.parse(document.getElementById("coder-options")!.getAttribute("data-settings")!)
+    } catch (error) {
+      console.error(error)
+      options = {} as T
+    }
 
-  // You can also pass options in stringified form to the options query
-  // variable. Options provided here will override the ones in the options
-  // element.
-  const params = new URLSearchParams(location.search)
-  const queryOpts = params.get("options")
-  if (queryOpts) {
-    options = {
-      ...options,
-      ...JSON.parse(queryOpts),
+    // You can also pass options in stringified form to the options query
+    // variable. Options provided here will override the ones in the options
+    // element.
+    const params = new URLSearchParams(location.search)
+    const queryOpts = params.get("options")
+    if (queryOpts) {
+      try {
+        options = {
+          ...options,
+          ...JSON.parse(queryOpts),
+        }
+      } catch (error) {
+        // Don't fail if the query parameters are malformed.
+        console.error(error)
+      }
     }
-  }
 
-  options.base = resolveBase(options.base)
-  options.csStaticBase = resolveBase(options.csStaticBase)
+    options.base = resolveBase(options.base)
+    options.csStaticBase = resolveBase(options.csStaticBase)
+  }
 
-  return options
+  return options as T
 }
 
 /**

test/e2e/extensions.test.ts

@@ -0,0 +1,12 @@
+import { describe, test } from "./baseFixture"
+
+describe("Extensions", true, () => {
+  // This will only work if the test extension is loaded into code-server.
+  test("should have access to VSCODE_PROXY_URI", async ({ codeServerPage }) => {
+    const address = await codeServerPage.address()
+
+    await codeServerPage.executeCommandViaMenus("code-server: Get proxy URI")
+
+    await codeServerPage.page.waitForSelector(`text=${address}/proxy/{port}`)
+  })
+})

test/e2e/extensions/test-extension/.gitignore

@@ -0,0 +1 @@
+/extension.js

test/e2e/extensions/test-extension/extension.ts

@@ -0,0 +1,13 @@
+import * as vscode from "vscode"
+
+export function activate(context: vscode.ExtensionContext) {
+  context.subscriptions.push(
+    vscode.commands.registerCommand("codeServerTest.proxyUri", () => {
+      if (process.env.VSCODE_PROXY_URI) {
+        vscode.window.showInformationMessage(process.env.VSCODE_PROXY_URI)
+      } else {
+        vscode.window.showErrorMessage("No proxy URI was set")
+      }
+    }),
+  )
+}

test/e2e/extensions/test-extension/package.json

@@ -0,0 +1,29 @@
+{
+  "name": "code-server-extension",
+  "description": "code-server test extension",
+  "version": "0.0.1",
+  "publisher": "cdr",
+  "activationEvents": [
+    "onCommand:codeServerTest.proxyUri"
+  ],
+  "engines": {
+    "vscode": "^1.56.0"
+  },
+  "main": "./extension.js",
+  "contributes": {
+    "commands": [
+      {
+        "command": "codeServerTest.proxyUri",
+        "title": "Get proxy URI",
+        "category": "code-server"
+      }
+    ]
+  },
+  "devDependencies": {
+    "@types/vscode": "^1.56.0",
+    "typescript": "^4.0.5"
+  },
+  "scripts": {
+    "build": "tsc extension.ts"
+  }
+}