Skip to content

Commit ff679b3

Browse files
committed
missing files after 1.60.0 upgrade
1 parent 9cfbc29 commit ff679b3

File tree

2 files changed

+47
-25
lines changed

2 files changed

+47
-25
lines changed

build/lib/extensions.js

Lines changed: 18 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -218,11 +218,27 @@ const webBuiltInExtensions = productJson.webBuiltInExtensions || [];
218218
* Loosely based on `getExtensionKind` from `src/vs/workbench/services/extensions/common/extensionManifestPropertiesService.ts`
219219
*/
220220
function isWebExtension(manifest) {
221+
if (Boolean(manifest.browser)) {
222+
return true;
223+
}
224+
if (Boolean(manifest.main)) {
225+
return false;
226+
}
227+
// neither browser nor main
221228
if (typeof manifest.extensionKind !== 'undefined') {
222229
const extensionKind = Array.isArray(manifest.extensionKind) ? manifest.extensionKind : [manifest.extensionKind];
223-
return (extensionKind.indexOf('web') >= 0);
230+
if (extensionKind.indexOf('web') >= 0) {
231+
return true;
232+
}
233+
}
234+
if (typeof manifest.contributes !== 'undefined') {
235+
for (const id of ['debuggers', 'terminal', 'typescriptServerPlugins']) {
236+
if (manifest.contributes.hasOwnProperty(id)) {
237+
return false;
238+
}
239+
}
224240
}
225-
return (!Boolean(manifest.main) || Boolean(manifest.browser));
241+
return true;
226242
}
227243
function packageLocalExtensionsStream(forWeb) {
228244
const localExtensionsDescriptions = (glob.sync('extensions/*/package.json')

build/lib/util.js

Lines changed: 29 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
*--------------------------------------------------------------------------------------------*/
55
'use strict';
66
Object.defineProperty(exports, "__esModule", { value: true });
7-
exports.buildWebNodePaths = exports.getElectronVersion = exports.streamToPromise = exports.versionStringToNumber = exports.filter = exports.rebase = exports.getVersion = exports.ensureDir = exports.rreddir = exports.rimraf = exports.rewriteSourceMappingURL = exports.stripSourceMappingURL = exports.loadSourcemaps = exports.cleanNodeModules = exports.skipDirectories = exports.toFileUri = exports.setExecutableBit = exports.fixWin32DirectoryPermissions = exports.incremental = void 0;
7+
exports.buildWebNodePaths = exports.acquireWebNodePaths = exports.getElectronVersion = exports.streamToPromise = exports.versionStringToNumber = exports.filter = exports.rebase = exports.getVersion = exports.ensureDir = exports.rreddir = exports.rimraf = exports.rewriteSourceMappingURL = exports.stripSourceMappingURL = exports.loadSourcemaps = exports.cleanNodeModules = exports.skipDirectories = exports.toFileUri = exports.setExecutableBit = exports.fixWin32DirectoryPermissions = exports.incremental = void 0;
88
const es = require("event-stream");
99
const debounce = require("debounce");
1010
const _filter = require("gulp-filter");
@@ -274,31 +274,37 @@ function getElectronVersion() {
274274
return target;
275275
}
276276
exports.getElectronVersion = getElectronVersion;
277+
function acquireWebNodePaths() {
278+
var _a;
279+
const root = path.join(__dirname, '..', '..');
280+
const webPackageJSON = path.join(root, '/remote/web', 'package.json');
281+
const webPackages = JSON.parse(fs.readFileSync(webPackageJSON, 'utf8')).dependencies;
282+
const nodePaths = {};
283+
for (const key of Object.keys(webPackages)) {
284+
const packageJSON = path.join(root, 'node_modules', key, 'package.json');
285+
const packageData = JSON.parse(fs.readFileSync(packageJSON, 'utf8'));
286+
let entryPoint = (_a = packageData.browser) !== null && _a !== void 0 ? _a : packageData.main;
287+
// On rare cases a package doesn't have an entrypoint so we assume it has a dist folder with a min.js
288+
if (!entryPoint) {
289+
console.warn(`No entry point for ${key} assuming dist/${key}.min.js`);
290+
entryPoint = `dist/${key}.min.js`;
291+
}
292+
// Remove any starting path information so it's all relative info
293+
if (entryPoint.startsWith('./')) {
294+
entryPoint = entryPoint.substr(2);
295+
}
296+
else if (entryPoint.startsWith('/')) {
297+
entryPoint = entryPoint.substr(1);
298+
}
299+
nodePaths[key] = entryPoint;
300+
}
301+
return nodePaths;
302+
}
303+
exports.acquireWebNodePaths = acquireWebNodePaths;
277304
function buildWebNodePaths(outDir) {
278305
const result = () => new Promise((resolve, _) => {
279-
var _a;
280306
const root = path.join(__dirname, '..', '..');
281-
const webPackageJSON = path.join(root, '/remote/web', 'package.json');
282-
const webPackages = JSON.parse(fs.readFileSync(webPackageJSON, 'utf8')).dependencies;
283-
const nodePaths = {};
284-
for (const key of Object.keys(webPackages)) {
285-
const packageJSON = path.join(root, 'node_modules', key, 'package.json');
286-
const packageData = JSON.parse(fs.readFileSync(packageJSON, 'utf8'));
287-
let entryPoint = (_a = packageData.browser) !== null && _a !== void 0 ? _a : packageData.main;
288-
// On rare cases a package doesn't have an entrypoint so we assume it has a dist folder with a min.js
289-
if (!entryPoint) {
290-
console.warn(`No entry point for ${key} assuming dist/${key}.min.js`);
291-
entryPoint = `dist/${key}.min.js`;
292-
}
293-
// Remove any starting path information so it's all relative info
294-
if (entryPoint.startsWith('./')) {
295-
entryPoint = entryPoint.substr(2);
296-
}
297-
else if (entryPoint.startsWith('/')) {
298-
entryPoint = entryPoint.substr(1);
299-
}
300-
nodePaths[key] = entryPoint;
301-
}
307+
const nodePaths = acquireWebNodePaths();
302308
// Now we write the node paths to out/vs
303309
const outDirectory = path.join(root, outDir, 'vs');
304310
fs.mkdirSync(outDirectory, { recursive: true });

0 commit comments

Comments
 (0)