f03d5d94ed
Summary:
- Removed compilation on startup which is not required anymore after switching to plugin spec v2.
- Removed from Node process ("static" package) all the dependencies which are not required anymore: e.g. metro, babel etc.
- Plugin loading code from node process moved to browser process and made asyncronous.
Some expected benefits after these changes:
1) Reduced size of Flipper bundle (~4.5MB reduction for lzma package in my tests) as well as startup time. It's hard to say the exact startup time difference as it is very machine-dependent, and on my machine it was already fast ~1500ms (vs 5500ms for p95) and decreased by just 100ms. But I think we should definitely see some improvements on "launch time" analytics graph for p95/p99.
2) Plugin loading is async now and happens when UI is already shown, so perceptive startup time should be also better now.
3) All plugin loading code is now consolidated in "app/dispatcher/plugins.tsx" instead of being splitted between Node and Browser processes as before. So it will be easier to debug plugin loading.
4) Now it is possible to apply updates of plugins by simple refresh of browser window instead of full Electron process restart as before.
5) 60% less code in Node process. This is good because it is harder to debug changes in Node process than in Browser process, especially taking in account differences between dev/release builds. Because of this Node process often ended up broken after changes. Hopefully now it will be more stable.
Changelog: changed the way of plugin loading, and removed obsolete dependencies, which should reduce bundle size and startup time.
Reviewed By: passy
Differential Revision: D23682756
fbshipit-source-id: 8445c877234b41c73853cebe585e2fdb1638b2c9
119 lines
3.1 KiB
TypeScript
119 lines
3.1 KiB
TypeScript
/**
|
|
* Copyright (c) Facebook, Inc. and its affiliates.
|
|
*
|
|
* This source code is licensed under the MIT license found in the
|
|
* LICENSE file in the root directory of this source tree.
|
|
*
|
|
* @format
|
|
*/
|
|
|
|
import {Client} from 'fb-watchman';
|
|
import {v4 as uuid} from 'uuid';
|
|
import path from 'path';
|
|
|
|
export default class Watchman {
|
|
constructor(private rootDir: string) {}
|
|
|
|
private client?: Client;
|
|
private watch?: any;
|
|
private relativeRoot?: string;
|
|
|
|
async initialize(): Promise<void> {
|
|
if (this.client) {
|
|
return;
|
|
}
|
|
this.client = new Client();
|
|
this.client.setMaxListeners(250);
|
|
await new Promise((resolve, reject) => {
|
|
const onError = (err: Error) => {
|
|
this.client!.removeAllListeners('error');
|
|
reject(err);
|
|
this.client!.end();
|
|
delete this.client;
|
|
};
|
|
this.client!.once('error', onError);
|
|
this.client!.capabilityCheck(
|
|
{optional: [], required: ['relative_root']},
|
|
(error) => {
|
|
if (error) {
|
|
onError(error);
|
|
return;
|
|
}
|
|
this.client!.command(
|
|
['watch-project', this.rootDir],
|
|
(error, resp) => {
|
|
if (error) {
|
|
onError(error);
|
|
return;
|
|
}
|
|
if ('warning' in resp) {
|
|
console.warn(resp.warning);
|
|
}
|
|
this.watch = resp.watch;
|
|
this.relativeRoot = resp.relative_path;
|
|
resolve();
|
|
},
|
|
);
|
|
},
|
|
);
|
|
});
|
|
}
|
|
|
|
async startWatchFiles(
|
|
relativeDir: string,
|
|
handler: (resp: any) => void,
|
|
options: {excludes: string[]},
|
|
): Promise<void> {
|
|
if (!this.watch) {
|
|
throw new Error(
|
|
'Watchman is not initialized, please call "initialize" function and wait for the returned promise completion before calling "startWatchFiles".',
|
|
);
|
|
}
|
|
options = Object.assign({excludes: []}, options);
|
|
return new Promise((resolve, reject) => {
|
|
this.client!.command(['clock', this.watch], (error, resp) => {
|
|
if (error) {
|
|
return reject(error);
|
|
}
|
|
|
|
try {
|
|
const {clock} = resp;
|
|
|
|
const sub = {
|
|
expression: [
|
|
'allof',
|
|
['not', ['type', 'd']],
|
|
...options!.excludes.map((e) => [
|
|
'not',
|
|
['match', e, 'wholename'],
|
|
]),
|
|
],
|
|
fields: ['name'],
|
|
since: clock,
|
|
relative_root: this.relativeRoot
|
|
? path.join(this.relativeRoot, relativeDir)
|
|
: relativeDir,
|
|
};
|
|
|
|
const id = uuid();
|
|
|
|
this.client!.command(['subscribe', this.watch, id, sub], (error) => {
|
|
if (error) {
|
|
return reject(error);
|
|
}
|
|
this.client!.on('subscription', (resp) => {
|
|
if (resp.subscription !== id || !resp.files) {
|
|
return;
|
|
}
|
|
handler(resp);
|
|
});
|
|
resolve();
|
|
});
|
|
} catch (err) {
|
|
reject(err);
|
|
}
|
|
});
|
|
});
|
|
}
|
|
}
|