From eaea947318c4869c34b45c17e2ed0b7ed140ff0d Mon Sep 17 00:00:00 2001 From: Asher Date: Mon, 25 Feb 2019 15:18:44 -0600 Subject: [PATCH] Fix spdlog and node-pty in binary --- build/tasks.ts | 2 +- package.json | 1 - packages/protocol/package.json | 1 + packages/protocol/src/common/helpers.ts | 49 +++++++++---------------- packages/protocol/yarn.lock | 35 +++++++++++++++++- packages/server/package.json | 1 - packages/server/src/cli.ts | 1 - packages/server/yarn.lock | 16 +------- packages/vscode/src/fill/node-pty.ts | 4 +- packages/vscode/src/fill/spdlog.ts | 6 +-- yarn.lock | 23 +----------- 11 files changed, 58 insertions(+), 81 deletions(-) diff --git a/build/tasks.ts b/build/tasks.ts index 27b6d27..919dd7e 100644 --- a/build/tasks.ts +++ b/build/tasks.ts @@ -82,7 +82,7 @@ const buildServerBinaryCopy = register("build:server:binary:copy", async (runner const webOutputPath = path.join(pkgsPath, "web", "out"); const browserAppOutputPath = path.join(pkgsPath, "app", "browser", "out"); const nodePtyModule = path.join(pkgsPath, "protocol", "node_modules", "node-pty", "build", "Release", "pty.node"); - const spdlogModule = path.join(pkgsPath, "server", "node_modules", "spdlog", "build", "Release", "spdlog.node"); + const spdlogModule = path.join(pkgsPath, "protocol", "node_modules", "spdlog", "build", "Release", "spdlog.node"); if (!fs.existsSync(nodePtyModule)) { throw new Error("Could not find pty.node. Ensure all packages have been installed"); diff --git a/package.json b/package.json index f196b22..e0e4aa4 100644 --- a/package.json +++ b/package.json @@ -50,7 +50,6 @@ }, "dependencies": { "node-loader": "^0.6.0", - "spdlog": "^0.7.2", "trash": "^4.3.0", "webpack-merge": "^4.2.1" } diff --git a/packages/protocol/package.json b/packages/protocol/package.json index 46372ce..33e3e9e 100644 --- a/packages/protocol/package.json +++ b/packages/protocol/package.json @@ -5,6 +5,7 @@ "express": "^4.16.4", "google-protobuf": "^3.6.1", "node-pty": "^0.8.1", + "spdlog": "^0.7.2", "tslib": "^1.9.3", "ws": "^6.1.2" }, diff --git a/packages/protocol/src/common/helpers.ts b/packages/protocol/src/common/helpers.ts index 1a3f082..b121977 100644 --- a/packages/protocol/src/common/helpers.ts +++ b/packages/protocol/src/common/helpers.ts @@ -1,3 +1,4 @@ +/// import { ChildProcess, SpawnOptions, ForkOptions } from "child_process"; import { EventEmitter } from "events"; import { Socket } from "net"; @@ -7,9 +8,6 @@ import { logger } from "@coder/logger"; // tslint:disable no-any -declare var __non_webpack_require__: typeof require; -declare var __webpack_require__: typeof require; - export type ForkProvider = (modulePath: string, args: string[], options: ForkOptions) => ChildProcess; export interface Disposer extends IDisposable { @@ -26,6 +24,19 @@ interface ActiveEvalEmitter { * Helper class for server-side evaluations. */ export class EvalHelper { + // For any non-external modules that are not built in, we need to require and + // access them here. A require on the client-side won't work since that code + // won't exist on the server (and bloat the client with an unused import), and + // we can't manually import on the server-side and then call + // `__webpack_require__` on the client-side because Webpack stores modules by + // their paths which would require us to hard-code the path. These aren't + // required immediately so we have a chance to unpack the .node files and set + // their locations. + public modules = { + spdlog: require("spdlog") as typeof import("spdlog"), + pty: require("node-pty") as typeof import("node-pty"), + }; + /** * Some spawn code tries to preserve the env (the debug adapter for instance) * but the env is mostly blank (since we're in the browser), so we'll just @@ -38,29 +49,6 @@ export class EvalHelper { options.env = { ...process.env, ...options.env }; } } - - /** - * Try a non-webpack require, then a webpack require if that fails. - */ - public require(modulePath: string): any { - logger.info(`Attempting to require ${modulePath}`); - try { - return __non_webpack_require__(modulePath); - } catch (error) { /* Nothing. */ } - - logger.warn(`Non-webpack require failed for ${modulePath}`); - try { - return __webpack_require__(modulePath); - } catch (error) { /* Nothing. */ } - - logger.warn(`Webpack require failed for ${modulePath}`); - try { - return require(modulePath); - } catch (error) { - logger.error(`Failed to require ${modulePath}`); - throw error; - } - } } /** @@ -125,20 +113,17 @@ export class ActiveEvalHelper implements ActiveEvalEmitter { * Helper class for server-side active evaluations. */ export class ServerActiveEvalHelper extends ActiveEvalHelper implements EvalHelper { - private readonly evalHelper: EvalHelper; + private readonly evalHelper = new EvalHelper(); + public modules = this.evalHelper.modules; + public constructor(emitter: ActiveEvalEmitter, public readonly fork: ForkProvider) { super(emitter); - this.evalHelper = new EvalHelper(); } public preserveEnv(options: SpawnOptions | ForkOptions): void { this.evalHelper.preserveEnv(options); } - public require(modulePath: string): any { - return this.evalHelper.require(modulePath); - } - /** * If there is a callback ID, return a function that emits the callback event * on the active evaluation with that ID and all arguments passed to it. diff --git a/packages/protocol/yarn.lock b/packages/protocol/yarn.lock index eaff6bb..246fe4a 100644 --- a/packages/protocol/yarn.lock +++ b/packages/protocol/yarn.lock @@ -30,6 +30,13 @@ async-limiter@~1.0.0: resolved "https://registry.yarnpkg.com/async-limiter/-/async-limiter-1.0.0.tgz#78faed8c3d074ab81f22b4e985d79e8738f720f8" integrity sha512-jp/uFnooOiO+L211eZOoSyzpOITMXx1rBITauYykG3BRYPu8h0UcxsPNB04RR5vo4Tyz3+ay17tR6JVf9qzYWg== +bindings@^1.3.0: + version "1.4.0" + resolved "https://registry.yarnpkg.com/bindings/-/bindings-1.4.0.tgz#909efa49f2ebe07ecd3cb136778f665052040127" + integrity sha512-7znEVX22Djn+nYjxCWKDne0RRloa9XfYa84yk3s+HkE3LpDYZmhArYr9O9huBoHY3/oXispx5LorIX7Sl2CgSQ== + dependencies: + file-uri-to-path "1.0.0" + body-parser@1.18.3: version "1.18.3" resolved "https://registry.yarnpkg.com/body-parser/-/body-parser-1.18.3.tgz#5b292198ffdd553b3a0f20ded0592b956955c8b4" @@ -144,6 +151,11 @@ express@^4.16.4: utils-merge "1.0.1" vary "~1.1.2" +file-uri-to-path@1.0.0: + version "1.0.0" + resolved "https://registry.yarnpkg.com/file-uri-to-path/-/file-uri-to-path-1.0.0.tgz#553a7b8446ff6f684359c445f1e37a05dacc33dd" + integrity sha512-0Zt+s3L7Vf1biwWZ29aARiVYLx7iMGnEUl9x33fbB/j3jR81u/O2LbqK+Bm1CDSNDKVtJ/YjwY7TUd5SkeLQLw== + finalhandler@1.1.1: version "1.1.1" resolved "https://registry.yarnpkg.com/finalhandler/-/finalhandler-1.1.1.tgz#eebf4ed840079c83f4249038c9d703008301b105" @@ -231,12 +243,24 @@ mime@1.4.1: resolved "https://registry.yarnpkg.com/mime/-/mime-1.4.1.tgz#121f9ebc49e3766f311a76e1fa1c8003c4b03aa6" integrity sha512-KI1+qOZu5DcW6wayYHSzR/tXKCDC5Om4s1z2QJjDULzLcmf3DvzS7oluY4HCTrc+9FiKmWUgeNLg7W3uIQvxtQ== +minimist@0.0.8: + version "0.0.8" + resolved "https://registry.yarnpkg.com/minimist/-/minimist-0.0.8.tgz#857fcabfc3397d2625b8228262e86aa7a011b05d" + integrity sha1-hX/Kv8M5fSYluCKCYuhqp6ARsF0= + +mkdirp@^0.5.1: + version "0.5.1" + resolved "https://registry.yarnpkg.com/mkdirp/-/mkdirp-0.5.1.tgz#30057438eac6cf7f8c4767f38648d6697d75c903" + integrity sha1-MAV0OOrGz3+MR2fzhkjWaX11yQM= + dependencies: + minimist "0.0.8" + ms@2.0.0: version "2.0.0" resolved "https://registry.yarnpkg.com/ms/-/ms-2.0.0.tgz#5608aeadfc00be6c2901df5f9861788de0d597c8" integrity sha1-VgiurfwAvmwpAd9fmGF4jeDVl8g= -nan@2.12.1: +nan@2.12.1, nan@^2.8.0: version "2.12.1" resolved "https://registry.yarnpkg.com/nan/-/nan-2.12.1.tgz#7b1aa193e9aa86057e3c7bbd0ac448e770925552" integrity sha512-JY7V6lRkStKcKTvHO5NVSQRv+RV+FIL5pvDoLiAtSL9pKlC5x9PKQcZDsq7m4FO4d57mkhC6Z+QhAh3Jdk5JFw== @@ -342,6 +366,15 @@ setprototypeof@1.1.0: resolved "https://registry.yarnpkg.com/setprototypeof/-/setprototypeof-1.1.0.tgz#d0bd85536887b6fe7c0d818cb962d9d91c54e656" integrity sha512-BvE/TwpZX4FXExxOxZyRGQQv651MSwmWKZGqvmPcRIjDqWub67kTKuIMx43cZZrS/cBBzwBcNDWoFxt2XEFIpQ== +spdlog@^0.7.2: + version "0.7.2" + resolved "https://registry.yarnpkg.com/spdlog/-/spdlog-0.7.2.tgz#9298753d7694b9ee9bbfd7e01ea1e4c6ace1e64d" + integrity sha512-rHfWCaWMD4NindDnql6rc6kn7Bs8JR92jhiUpCl3D6v+jYcQ6GozMLig0RliOOR8st5mU+IHLZnr15fBys5x/Q== + dependencies: + bindings "^1.3.0" + mkdirp "^0.5.1" + nan "^2.8.0" + "statuses@>= 1.4.0 < 2": version "1.5.0" resolved "https://registry.yarnpkg.com/statuses/-/statuses-1.5.0.tgz#161c7dac177659fd9811f43771fa99381478628c" diff --git a/packages/server/package.json b/packages/server/package.json index ddc62f4..b068a7d 100644 --- a/packages/server/package.json +++ b/packages/server/package.json @@ -24,7 +24,6 @@ "node-netstat": "^1.6.0", "pem": "^1.14.1", "promise.prototype.finally": "^3.1.0", - "spdlog": "^0.7.2", "ws": "^6.1.2", "xhr2": "^0.1.4" }, diff --git a/packages/server/src/cli.ts b/packages/server/src/cli.ts index 070f0d1..492af59 100644 --- a/packages/server/src/cli.ts +++ b/packages/server/src/cli.ts @@ -80,7 +80,6 @@ export class Entry extends Command { if (!fs.existsSync(dataDir)) { fs.mkdirSync(dataDir); } - require("spdlog"); const logDir = path.join(dataDir, "logs", new Date().toISOString().replace(/[-:.TZ]/g, "")); process.env.VSCODE_LOGS = logDir; diff --git a/packages/server/yarn.lock b/packages/server/yarn.lock index 23add6b..1351aa3 100644 --- a/packages/server/yarn.lock +++ b/packages/server/yarn.lock @@ -437,11 +437,6 @@ binary-extensions@^1.0.0: resolved "https://registry.yarnpkg.com/binary-extensions/-/binary-extensions-1.12.0.tgz#c2d780f53d45bba8317a8902d4ceeaf3a6385b14" integrity sha512-DYWGk01lDcxeS/K9IHPGWfT8PsJmbXRtRd2Sx72Tnb8pcYZQFF1oSDb8hJtS1vhp212q1Rzi5dUf9+nq0o9UIg== -bindings@^1.3.0: - version "1.3.1" - resolved "https://registry.yarnpkg.com/bindings/-/bindings-1.3.1.tgz#21fc7c6d67c18516ec5aaa2815b145ff77b26ea5" - integrity sha512-i47mqjF9UbjxJhxGf+pZ6kSxrnI3wBLlnGI2ArWJ4r0VrvDS7ZYXkprq/pLaBWYq4GM0r4zdHY+NNRqEMU7uew== - bl@^1.0.0: version "1.2.2" resolved "https://registry.yarnpkg.com/bl/-/bl-1.2.2.tgz#a160911717103c07410cef63ef51b397c025af9c" @@ -2447,7 +2442,7 @@ mute-stream@0.0.7: resolved "https://registry.yarnpkg.com/mute-stream/-/mute-stream-0.0.7.tgz#3075ce93bc21b8fab43e1bc4da7e8115ed1e7bab" integrity sha1-MHXOk7whuPq0PhvE2n6BFe0ee6s= -nan@^2.8.0, nan@^2.9.2: +nan@^2.9.2: version "2.12.1" resolved "https://registry.yarnpkg.com/nan/-/nan-2.12.1.tgz#7b1aa193e9aa86057e3c7bbd0ac448e770925552" integrity sha512-JY7V6lRkStKcKTvHO5NVSQRv+RV+FIL5pvDoLiAtSL9pKlC5x9PKQcZDsq7m4FO4d57mkhC6Z+QhAh3Jdk5JFw== @@ -3327,15 +3322,6 @@ source-map@~0.1.38: dependencies: amdefine ">=0.0.4" -spdlog@^0.7.2: - version "0.7.2" - resolved "https://registry.yarnpkg.com/spdlog/-/spdlog-0.7.2.tgz#9298753d7694b9ee9bbfd7e01ea1e4c6ace1e64d" - integrity sha512-rHfWCaWMD4NindDnql6rc6kn7Bs8JR92jhiUpCl3D6v+jYcQ6GozMLig0RliOOR8st5mU+IHLZnr15fBys5x/Q== - dependencies: - bindings "^1.3.0" - mkdirp "^0.5.1" - nan "^2.8.0" - split-string@^3.0.1, split-string@^3.0.2: version "3.1.0" resolved "https://registry.yarnpkg.com/split-string/-/split-string-3.1.0.tgz#7cb09dda3a86585705c64b39a6466038682e8fe2" diff --git a/packages/vscode/src/fill/node-pty.ts b/packages/vscode/src/fill/node-pty.ts index 4da5934..311e5d1 100644 --- a/packages/vscode/src/fill/node-pty.ts +++ b/packages/vscode/src/fill/node-pty.ts @@ -15,11 +15,9 @@ class Pty implements nodePty.IPty { public constructor(file: string, args: string[] | string, options: nodePty.IPtyForkOptions) { this.ae = client.run((ae, file, args, options) => { - const nodePty = ae.require("node-pty") as typeof import("node-pty"); - ae.preserveEnv(options); - const ptyProc = nodePty.spawn(file, args, options); + const ptyProc = ae.modules.pty.spawn(file, args, options); let process = ptyProc.process; ae.emit("process", process); diff --git a/packages/vscode/src/fill/spdlog.ts b/packages/vscode/src/fill/spdlog.ts index 876dfa6..d3ec475 100644 --- a/packages/vscode/src/fill/spdlog.ts +++ b/packages/vscode/src/fill/spdlog.ts @@ -1,14 +1,12 @@ -/// import { RotatingLogger as NodeRotatingLogger } from "spdlog"; import { logger } from "@coder/logger"; import { client } from "@coder/ide/src/fill/client"; const ae = client.run((ae) => { - const spdlog = ae.require("spdlog") as typeof import("spdlog"); const loggers = new Map(); ae.on("new", (id: number, name: string, filePath: string, fileSize: number, fileCount: number) => { - const logger = new spdlog.RotatingLogger(name, filePath, fileSize, fileCount); + const logger = new ae.modules.spdlog.RotatingLogger(name, filePath, fileSize, fileCount); loggers.set(id, logger); }); @@ -19,7 +17,7 @@ const ae = client.run((ae) => { ae.on("errorLog", (id: number, message: string) => loggers.get(id)!.error(message)); ae.on("flush", (id: number) => loggers.get(id)!.flush()); ae.on("info", (id: number, message: string) => loggers.get(id)!.info(message)); - ae.on("setAsyncMode", (bufferSize: number, flushInterval: number) => spdlog.setAsyncMode(bufferSize, flushInterval)); + ae.on("setAsyncMode", (bufferSize: number, flushInterval: number) => ae.modules.spdlog.setAsyncMode(bufferSize, flushInterval)); ae.on("setLevel", (id: number, level: number) => loggers.get(id)!.setLevel(level)); ae.on("trace", (id: number, message: string) => loggers.get(id)!.trace(message)); ae.on("warn", (id: number, message: string) => loggers.get(id)!.warn(message)); diff --git a/yarn.lock b/yarn.lock index 45d802c..c8c1a6b 100644 --- a/yarn.lock +++ b/yarn.lock @@ -523,13 +523,6 @@ binary-extensions@^1.0.0: resolved "https://registry.yarnpkg.com/binary-extensions/-/binary-extensions-1.12.0.tgz#c2d780f53d45bba8317a8902d4ceeaf3a6385b14" integrity sha512-DYWGk01lDcxeS/K9IHPGWfT8PsJmbXRtRd2Sx72Tnb8pcYZQFF1oSDb8hJtS1vhp212q1Rzi5dUf9+nq0o9UIg== -bindings@^1.3.0: - version "1.4.0" - resolved "https://registry.yarnpkg.com/bindings/-/bindings-1.4.0.tgz#909efa49f2ebe07ecd3cb136778f665052040127" - integrity sha512-7znEVX22Djn+nYjxCWKDne0RRloa9XfYa84yk3s+HkE3LpDYZmhArYr9O9huBoHY3/oXispx5LorIX7Sl2CgSQ== - dependencies: - file-uri-to-path "1.0.0" - block-stream@*: version "0.0.9" resolved "https://registry.yarnpkg.com/block-stream/-/block-stream-0.0.9.tgz#13ebfe778a03205cfe03751481ebb4b3300c126a" @@ -1804,11 +1797,6 @@ file-loader@^3.0.1: loader-utils "^1.0.2" schema-utils "^1.0.0" -file-uri-to-path@1.0.0: - version "1.0.0" - resolved "https://registry.yarnpkg.com/file-uri-to-path/-/file-uri-to-path-1.0.0.tgz#553a7b8446ff6f684359c445f1e37a05dacc33dd" - integrity sha512-0Zt+s3L7Vf1biwWZ29aARiVYLx7iMGnEUl9x33fbB/j3jR81u/O2LbqK+Bm1CDSNDKVtJ/YjwY7TUd5SkeLQLw== - filesize@^3.6.1: version "3.6.1" resolved "https://registry.yarnpkg.com/filesize/-/filesize-3.6.1.tgz#090bb3ee01b6f801a8a8be99d31710b3422bb317" @@ -3367,7 +3355,7 @@ multicast-dns@^6.0.1: dns-packet "^1.3.1" thunky "^1.0.2" -nan@^2.10.0, nan@^2.8.0, nan@^2.9.2: +nan@^2.10.0, nan@^2.9.2: version "2.12.1" resolved "https://registry.yarnpkg.com/nan/-/nan-2.12.1.tgz#7b1aa193e9aa86057e3c7bbd0ac448e770925552" integrity sha512-JY7V6lRkStKcKTvHO5NVSQRv+RV+FIL5pvDoLiAtSL9pKlC5x9PKQcZDsq7m4FO4d57mkhC6Z+QhAh3Jdk5JFw== @@ -4774,15 +4762,6 @@ source-map@^0.6.0, source-map@^0.6.1, source-map@~0.6.0, source-map@~0.6.1: resolved "https://registry.yarnpkg.com/source-map/-/source-map-0.6.1.tgz#74722af32e9614e9c287a8d0bbde48b5e2f1a263" integrity sha512-UjgapumWlbMhkBgzT7Ykc5YXUT46F0iKu8SGXq0bcwP5dz/h0Plj6enJqjz1Zbq2l5WaqYnrVbwWOWMyF3F47g== -spdlog@^0.7.2: - version "0.7.2" - resolved "https://registry.yarnpkg.com/spdlog/-/spdlog-0.7.2.tgz#9298753d7694b9ee9bbfd7e01ea1e4c6ace1e64d" - integrity sha512-rHfWCaWMD4NindDnql6rc6kn7Bs8JR92jhiUpCl3D6v+jYcQ6GozMLig0RliOOR8st5mU+IHLZnr15fBys5x/Q== - dependencies: - bindings "^1.3.0" - mkdirp "^0.5.1" - nan "^2.8.0" - spdx-correct@^3.0.0: version "3.1.0" resolved "https://registry.yarnpkg.com/spdx-correct/-/spdx-correct-3.1.0.tgz#fb83e504445268f154b074e218c87c003cd31df4"