mirror of
https://github.com/microsoft/vscode.git
synced 2025-12-24 12:19:20 +00:00
* chore: bump electron@15.3.0 * chore: bump node@16.x * chore: enable render process reuse * Revert "watcher - use `type` property for crash reporter location" This reverts commitbfa488dd8f. * Revert "watcher - enable crash reports on linux (#136264)" This reverts commitaf261488f8. * chore: enable crashpad on linux * chore: bump electron@15.3.1 * chore: update api changes * chore: bump @vscode/sqlite3@5.0.3 * spec: skip non-context aware module unittests * chore: fix perf hook integration with node environment * fix: adopt fs api changes * chore: fix integration tests * chore: bump electron@15.3.2 * chore: bump electron@16.0.0 * temp(macOS): kill test instances in OSS * Revert "temp(macOS): kill test instances in OSS" This reverts commit b0d796c8d64227a5ec012c060bbbee7d9da27568. * chore: update chromium version for clang downloader * some 💄 changes * align with changes * adopt more fs.rm * 💄 * chore: bump @vscode/sqlite3@5.0.4 * fix layers check to account for duplicated types from node.js * update todo for type casts * smoke - fix compile issue * chore: update module cache * watcher - fix unhandled rejection (fix #137416) * ci: update node version * enable stack dumping * update electron types to 16.x * chore: bump @vscode/sqlite3@5.0.5 Refs https://github.com/microsoft/vscode/issues/137496 * fix layer issue * add `AbortSignal` to core types * chore: update linux compile flags Refs797723ec83* ci: fix linux build * ci: update github ci cache * ci: fix remote build in github ci * ci: better fix for remote build * chore: bump azure cache * chore: fix merge conflict * :chore: update to electron@16.0.2 * chore: bump @vscode/sqlite3@5.0.7 * ci: update to gcc-4.9 for remote Refs https://github.com/microsoft/vscode/issues/137659 * ci: switch to buster for linux arm Refs https://github.com/microsoft/vscode/issues/137927 * ci: fix build on linux arm64 * ci: fix arm client compiler toolchain Refs #137927 * chore: bump electron@16.0.3 * ci: fix compile flags for the c toolchain * chore: bump electron@16.0.4 * Add experimental dark mode flag (#139109) * Add experimental dark mode flag * Apply PR feedback * chore: bump electron@16.0.6 * chore: bump electron@16.0.7 Fixes https://github.com/microsoft/vscode/issues/138792 Fixes https://github.com/microsoft/vscode/issues/139300 * chore: experimental highlight API * smoke - fix compile issue * FIXME: custom ELECTRON_RUN_AS_NODE with node worker * Revert "chore: bump electron@16.0.7" This reverts commit5fd01cf37c. * Revert "Revert "chore: bump electron@16.0.7"" This reverts commita7f1b73951. * chore: fix github linux workflow * chore: address review feedback * chore: bump electron@16.0.8 * ci: revert to stretch distro for linux arm Refs https://github.com/microsoft/vscode/issues/137927 * ci: force build * chore: update yarn.lock * address feedback * Revert "FIXME: custom ELECTRON_RUN_AS_NODE with node worker" This reverts commit7b48fa3732. * ci: fix remote folder build * chore: fix github linux ci * 🆙 `versionSpec` Co-authored-by: Benjamin Pasero <benjamin.pasero@microsoft.com> Co-authored-by: Raymond Zhao <raymondzhao@microsoft.com>
305 lines
10 KiB
JavaScript
305 lines
10 KiB
JavaScript
"use strict";
|
|
/*---------------------------------------------------------------------------------------------
|
|
* Copyright (c) Microsoft Corporation. All rights reserved.
|
|
* Licensed under the MIT License. See License.txt in the project root for license information.
|
|
*--------------------------------------------------------------------------------------------*/
|
|
Object.defineProperty(exports, "__esModule", { value: true });
|
|
const ts = require("typescript");
|
|
const fs_1 = require("fs");
|
|
const path_1 = require("path");
|
|
const minimatch_1 = require("minimatch");
|
|
//
|
|
// #############################################################################################
|
|
//
|
|
// A custom typescript checker for the specific task of detecting the use of certain types in a
|
|
// layer that does not allow such use. For example:
|
|
// - using DOM globals in common/node/electron-main layer (e.g. HTMLElement)
|
|
// - using node.js globals in common/browser layer (e.g. process)
|
|
//
|
|
// Make changes to below RULES to lift certain files from these checks only if absolutely needed
|
|
//
|
|
// #############################################################################################
|
|
//
|
|
// Types we assume are present in all implementations of JS VMs (node.js, browsers)
|
|
// Feel free to add more core types as you see needed if present in node.js and browsers
|
|
const CORE_TYPES = [
|
|
'require',
|
|
'setTimeout',
|
|
'clearTimeout',
|
|
'setInterval',
|
|
'clearInterval',
|
|
'console',
|
|
'log',
|
|
'info',
|
|
'warn',
|
|
'error',
|
|
'group',
|
|
'groupEnd',
|
|
'table',
|
|
'assert',
|
|
'Error',
|
|
'String',
|
|
'throws',
|
|
'stack',
|
|
'captureStackTrace',
|
|
'stackTraceLimit',
|
|
'TextDecoder',
|
|
'TextEncoder',
|
|
'encode',
|
|
'decode',
|
|
'self',
|
|
'trimStart',
|
|
'trimEnd',
|
|
'trimLeft',
|
|
'trimRight',
|
|
'queueMicrotask',
|
|
'Array',
|
|
'Uint8Array',
|
|
'Uint16Array',
|
|
'Uint32Array',
|
|
'Int8Array',
|
|
'Int16Array',
|
|
'Int32Array',
|
|
'Float32Array',
|
|
'Float64Array',
|
|
'Uint8ClampedArray',
|
|
'BigUint64Array',
|
|
'BigInt64Array',
|
|
'btoa',
|
|
'atob',
|
|
'AbortSignal',
|
|
'MessageChannel',
|
|
'MessagePort'
|
|
];
|
|
// Types that are defined in a common layer but are known to be only
|
|
// available in native environments should not be allowed in browser
|
|
const NATIVE_TYPES = [
|
|
'NativeParsedArgs',
|
|
'INativeEnvironmentService',
|
|
'AbstractNativeEnvironmentService',
|
|
'INativeWindowConfiguration',
|
|
'ICommonNativeHostService'
|
|
];
|
|
const RULES = [
|
|
// Tests: skip
|
|
{
|
|
target: '**/vs/**/test/**',
|
|
skip: true // -> skip all test files
|
|
},
|
|
// Common: vs/base/common/platform.ts
|
|
{
|
|
target: '**/vs/base/common/platform.ts',
|
|
allowedTypes: [
|
|
...CORE_TYPES,
|
|
// Safe access to postMessage() and friends
|
|
'MessageEvent',
|
|
'data'
|
|
],
|
|
disallowedTypes: NATIVE_TYPES,
|
|
disallowedDefinitions: [
|
|
'lib.dom.d.ts',
|
|
'@types/node' // no node.js
|
|
]
|
|
},
|
|
// Common: vs/platform/environment/common/*
|
|
{
|
|
target: '**/vs/platform/environment/common/*.ts',
|
|
allowedTypes: CORE_TYPES,
|
|
disallowedTypes: [ /* Ignore native types that are defined from here */],
|
|
disallowedDefinitions: [
|
|
'lib.dom.d.ts',
|
|
'@types/node' // no node.js
|
|
]
|
|
},
|
|
// Common: vs/platform/windows/common/windows.ts
|
|
{
|
|
target: '**/vs/platform/windows/common/windows.ts',
|
|
allowedTypes: CORE_TYPES,
|
|
disallowedTypes: [ /* Ignore native types that are defined from here */],
|
|
disallowedDefinitions: [
|
|
'lib.dom.d.ts',
|
|
'@types/node' // no node.js
|
|
]
|
|
},
|
|
// Common: vs/platform/native/common/native.ts
|
|
{
|
|
target: '**/vs/platform/native/common/native.ts',
|
|
allowedTypes: CORE_TYPES,
|
|
disallowedTypes: [ /* Ignore native types that are defined from here */],
|
|
disallowedDefinitions: [
|
|
'lib.dom.d.ts',
|
|
'@types/node' // no node.js
|
|
]
|
|
},
|
|
// Common: vs/workbench/api/common/extHostExtensionService.ts
|
|
{
|
|
target: '**/vs/workbench/api/common/extHostExtensionService.ts',
|
|
allowedTypes: [
|
|
...CORE_TYPES,
|
|
// Safe access to global
|
|
'global'
|
|
],
|
|
disallowedTypes: NATIVE_TYPES,
|
|
disallowedDefinitions: [
|
|
'lib.dom.d.ts',
|
|
'@types/node' // no node.js
|
|
]
|
|
},
|
|
// Common
|
|
{
|
|
target: '**/vs/**/common/**',
|
|
allowedTypes: CORE_TYPES,
|
|
disallowedTypes: NATIVE_TYPES,
|
|
disallowedDefinitions: [
|
|
'lib.dom.d.ts',
|
|
'@types/node' // no node.js
|
|
]
|
|
},
|
|
// Browser
|
|
{
|
|
target: '**/vs/**/browser/**',
|
|
allowedTypes: CORE_TYPES,
|
|
disallowedTypes: NATIVE_TYPES,
|
|
allowedDefinitions: [
|
|
'@types/node/stream/consumers.d.ts' // node.js started to duplicate types from lib.dom.d.ts so we have to account for that
|
|
],
|
|
disallowedDefinitions: [
|
|
'@types/node' // no node.js
|
|
]
|
|
},
|
|
// Browser (editor contrib)
|
|
{
|
|
target: '**/src/vs/editor/contrib/**',
|
|
allowedTypes: CORE_TYPES,
|
|
disallowedTypes: NATIVE_TYPES,
|
|
disallowedDefinitions: [
|
|
'@types/node' // no node.js
|
|
]
|
|
},
|
|
// node.js
|
|
{
|
|
target: '**/vs/**/node/**',
|
|
allowedTypes: [
|
|
...CORE_TYPES,
|
|
// --> types from node.d.ts that duplicate from lib.dom.d.ts
|
|
'URL',
|
|
'protocol',
|
|
'hostname',
|
|
'port',
|
|
'pathname',
|
|
'search',
|
|
'username',
|
|
'password'
|
|
],
|
|
disallowedDefinitions: [
|
|
'lib.dom.d.ts' // no DOM
|
|
]
|
|
},
|
|
// Electron (sandbox)
|
|
{
|
|
target: '**/vs/**/electron-sandbox/**',
|
|
allowedTypes: CORE_TYPES,
|
|
disallowedDefinitions: [
|
|
'@types/node' // no node.js
|
|
]
|
|
},
|
|
// Electron (renderer): skip
|
|
{
|
|
target: '**/vs/**/electron-browser/**',
|
|
skip: true // -> supports all types
|
|
},
|
|
// Electron (main)
|
|
{
|
|
target: '**/vs/**/electron-main/**',
|
|
allowedTypes: [
|
|
...CORE_TYPES,
|
|
// --> types from electron.d.ts that duplicate from lib.dom.d.ts
|
|
'Event',
|
|
'Request'
|
|
],
|
|
disallowedDefinitions: [
|
|
'lib.dom.d.ts' // no DOM
|
|
]
|
|
}
|
|
];
|
|
const TS_CONFIG_PATH = (0, path_1.join)(__dirname, '../../', 'src', 'tsconfig.json');
|
|
let hasErrors = false;
|
|
function checkFile(program, sourceFile, rule) {
|
|
checkNode(sourceFile);
|
|
function checkNode(node) {
|
|
var _a, _b;
|
|
if (node.kind !== ts.SyntaxKind.Identifier) {
|
|
return ts.forEachChild(node, checkNode); // recurse down
|
|
}
|
|
const text = node.getText(sourceFile);
|
|
if ((_a = rule.allowedTypes) === null || _a === void 0 ? void 0 : _a.some(allowed => allowed === text)) {
|
|
return; // override
|
|
}
|
|
if ((_b = rule.disallowedTypes) === null || _b === void 0 ? void 0 : _b.some(disallowed => disallowed === text)) {
|
|
const { line, character } = sourceFile.getLineAndCharacterOfPosition(node.getStart());
|
|
console.log(`[build/lib/layersChecker.ts]: Reference to '${text}' violates layer '${rule.target}' (${sourceFile.fileName} (${line + 1},${character + 1})`);
|
|
hasErrors = true;
|
|
return;
|
|
}
|
|
const checker = program.getTypeChecker();
|
|
const symbol = checker.getSymbolAtLocation(node);
|
|
if (symbol) {
|
|
const declarations = symbol.declarations;
|
|
if (Array.isArray(declarations)) {
|
|
DeclarationLoop: for (const declaration of declarations) {
|
|
if (declaration) {
|
|
const parent = declaration.parent;
|
|
if (parent) {
|
|
const parentSourceFile = parent.getSourceFile();
|
|
if (parentSourceFile) {
|
|
const definitionFileName = parentSourceFile.fileName;
|
|
if (rule.allowedDefinitions) {
|
|
for (const allowedDefinition of rule.allowedDefinitions) {
|
|
if (definitionFileName.indexOf(allowedDefinition) >= 0) {
|
|
continue DeclarationLoop;
|
|
}
|
|
}
|
|
}
|
|
if (rule.disallowedDefinitions) {
|
|
for (const disallowedDefinition of rule.disallowedDefinitions) {
|
|
if (definitionFileName.indexOf(disallowedDefinition) >= 0) {
|
|
const { line, character } = sourceFile.getLineAndCharacterOfPosition(node.getStart());
|
|
console.log(`[build/lib/layersChecker.ts]: Reference to '${text}' from '${disallowedDefinition}' violates layer '${rule.target}' (${sourceFile.fileName} (${line + 1},${character + 1})`);
|
|
hasErrors = true;
|
|
return;
|
|
}
|
|
}
|
|
}
|
|
}
|
|
}
|
|
}
|
|
}
|
|
}
|
|
}
|
|
}
|
|
}
|
|
function createProgram(tsconfigPath) {
|
|
const tsConfig = ts.readConfigFile(tsconfigPath, ts.sys.readFile);
|
|
const configHostParser = { fileExists: fs_1.existsSync, readDirectory: ts.sys.readDirectory, readFile: file => (0, fs_1.readFileSync)(file, 'utf8'), useCaseSensitiveFileNames: process.platform === 'linux' };
|
|
const tsConfigParsed = ts.parseJsonConfigFileContent(tsConfig.config, configHostParser, (0, path_1.resolve)((0, path_1.dirname)(tsconfigPath)), { noEmit: true });
|
|
const compilerHost = ts.createCompilerHost(tsConfigParsed.options, true);
|
|
return ts.createProgram(tsConfigParsed.fileNames, tsConfigParsed.options, compilerHost);
|
|
}
|
|
//
|
|
// Create program and start checking
|
|
//
|
|
const program = createProgram(TS_CONFIG_PATH);
|
|
for (const sourceFile of program.getSourceFiles()) {
|
|
for (const rule of RULES) {
|
|
if ((0, minimatch_1.match)([sourceFile.fileName], rule.target).length > 0) {
|
|
if (!rule.skip) {
|
|
checkFile(program, sourceFile, rule);
|
|
}
|
|
break;
|
|
}
|
|
}
|
|
}
|
|
if (hasErrors) {
|
|
process.exit(1);
|
|
}
|