diff --git a/src/common/util.ts b/src/common/util.ts index 191c907ab..6639beca4 100644 --- a/src/common/util.ts +++ b/src/common/util.ts @@ -1,12 +1,3 @@ -/** - * Split a string up to the delimiter. If the delimiter doesn't exist the first - * item will have all the text and the second item will be an empty string. - */ -export const split = (str: string, delimiter: string): [string, string] => { - const index = str.indexOf(delimiter) - return index !== -1 ? [str.substring(0, index).trim(), str.substring(index + 1)] : [str, ""] -} - /** * Appends an 's' to the provided string if count is greater than one; * otherwise the string is returned @@ -34,27 +25,6 @@ export const normalize = (url: string, keepTrailing = false): string => { return url.replace(/\/\/+/g, "/").replace(/\/+$/, keepTrailing ? "/" : "") } -/** - * Remove leading and trailing slashes. - */ -export const trimSlashes = (url: string): string => { - return url.replace(/^\/+|\/+$/g, "") -} - -/** - * Wrap the value in an array if it's not already an array. If the value is - * undefined return an empty array. - */ -export const arrayify = (value?: T | T[]): T[] => { - if (Array.isArray(value)) { - return value - } - if (typeof value === "undefined") { - return [] - } - return [value] -} - // TODO: Might make sense to add Error handling to the logger itself. export function logError(logger: { error: (msg: string) => void }, prefix: string, err: unknown): void { if (err instanceof Error) { diff --git a/src/node/app.ts b/src/node/app.ts index aece0432c..3e6e4bfa0 100644 --- a/src/node/app.ts +++ b/src/node/app.ts @@ -102,29 +102,6 @@ export const ensureAddress = (server: http.Server, protocol: string): URL | stri return addr } -/** - * Handles error events from the server. - * - * If the outlying Promise didn't resolve - * then we reject with the error. - * - * Otherwise, we log the error. - * - * We extracted into a function so that we could - * test this logic more easily. - */ -export const handleServerError = (resolved: boolean, err: Error, reject: (err: Error) => void) => { - // Promise didn't resolve earlier so this means it's an error - // that occurs before the server can successfully listen. - // Possibly triggered by listening on an invalid port or socket. - if (!resolved) { - reject(err) - } else { - // Promise resolved earlier so this is an unrelated error. - util.logError(logger, "http server error", err) - } -} - /** * Handles the error that occurs in the catch block * after we try fs.unlink(args.socket). diff --git a/src/node/constants.ts b/src/node/constants.ts index c85e0a7b0..bb6873dfa 100644 --- a/src/node/constants.ts +++ b/src/node/constants.ts @@ -3,8 +3,6 @@ import type { JSONSchemaForNPMPackageJsonFiles } from "@schemastore/package" import * as os from "os" import * as path from "path" -export const WORKBENCH_WEB_CONFIG_ID = "vscode-workbench-web-configuration" - export function getPackageJson(relativePath: string): JSONSchemaForNPMPackageJsonFiles { let pkg = {} try { @@ -21,7 +19,6 @@ export const vsRootPath = path.join(rootPath, "lib/vscode") const PACKAGE_JSON = "package.json" const pkg = getPackageJson(`${rootPath}/${PACKAGE_JSON}`) const codePkg = getPackageJson(`${vsRootPath}/${PACKAGE_JSON}`) || { version: "0.0.0" } -export const pkgName = pkg.name || "code-server" export const version = pkg.version || "development" export const commit = pkg.commit || "development" export const codeVersion = codePkg.version || "development" diff --git a/src/node/util.ts b/src/node/util.ts index e33ad95bf..1bf43cd89 100644 --- a/src/node/util.ts +++ b/src/node/util.ts @@ -426,15 +426,6 @@ export const enumToArray = (t: any): string[] => { return values } -/** - * For displaying all allowed options in an enum. - */ -// eslint-disable-next-line @typescript-eslint/no-explicit-any -export const buildAllowedMessage = (t: any): string => { - const values = enumToArray(t) - return `Allowed value${values.length === 1 ? " is" : "s are"} ${values.map((t) => `'${t}'`).join(", ")}` -} - /** * Return a promise that resolves with whether the socket path is active. */ diff --git a/test/unit/node/constants.test.ts b/test/unit/node/constants.test.ts index 38affbb87..2ed963f87 100644 --- a/test/unit/node/constants.test.ts +++ b/test/unit/node/constants.test.ts @@ -34,10 +34,6 @@ describe("constants", () => { jest.resetModules() }) - it("should provide the package name", () => { - expect(constants.pkgName).toBe(mockPackageJson.name) - }) - it("should provide the commit", () => { expect(constants.commit).toBe(mockPackageJson.commit) })