Rename RemoteNodeAPI tp RemoteServerContext
Reviewed By: mweststrate Differential Revision: D32921160 fbshipit-source-id: 9d6b723427d7a819f2fc1b7168f6d1077dc56bd9
This commit is contained in:
committed by
Facebook GitHub Bot
parent
0e785fb18d
commit
4cb80a452f
@@ -112,7 +112,7 @@ test('Correct top level API exposed', () => {
|
||||
"NormalizedMenuEntry",
|
||||
"Notification",
|
||||
"PluginClient",
|
||||
"RemoteNodeAPI",
|
||||
"RemoteServerContext",
|
||||
]
|
||||
`);
|
||||
});
|
||||
|
||||
@@ -39,7 +39,7 @@ export {
|
||||
setFlipperLibImplementation as _setFlipperLibImplementation,
|
||||
FileDescriptor,
|
||||
FileEncoding,
|
||||
RemoteNodeAPI,
|
||||
RemoteServerContext,
|
||||
} from './plugin/FlipperLib';
|
||||
export {
|
||||
MenuEntry,
|
||||
|
||||
@@ -28,7 +28,7 @@ export interface FileDescriptor {
|
||||
path?: string;
|
||||
}
|
||||
|
||||
export type RemoteNodeAPI = {
|
||||
export type RemoteServerContext = {
|
||||
childProcess: {
|
||||
exec(
|
||||
command: string,
|
||||
@@ -134,7 +134,7 @@ export interface FlipperLib {
|
||||
homePath: string;
|
||||
appPath: string;
|
||||
};
|
||||
removeNodeAPI: RemoteNodeAPI;
|
||||
remoteServerContext: RemoteServerContext;
|
||||
}
|
||||
|
||||
export let flipperLibInstance: FlipperLib | undefined;
|
||||
|
||||
@@ -389,7 +389,7 @@ export function createMockFlipperLib(options?: StartPluginOptions): FlipperLib {
|
||||
appPath: process.cwd(),
|
||||
homePath: `/dev/null`,
|
||||
},
|
||||
removeNodeAPI: {
|
||||
remoteServerContext: {
|
||||
childProcess: {
|
||||
exec: jest.fn(),
|
||||
},
|
||||
|
||||
@@ -7,7 +7,10 @@
|
||||
* @format
|
||||
*/
|
||||
|
||||
import {_setFlipperLibImplementation, RemoteNodeAPI} from 'flipper-plugin';
|
||||
import {
|
||||
_setFlipperLibImplementation,
|
||||
RemoteServerContext,
|
||||
} from 'flipper-plugin';
|
||||
import type {
|
||||
BufferEncoding,
|
||||
ExecOptions,
|
||||
@@ -68,17 +71,12 @@ export function initializeFlipperLibImplementation(
|
||||
appPath: renderHost.serverConfig.paths.appPath,
|
||||
homePath: renderHost.serverConfig.paths.homePath,
|
||||
},
|
||||
removeNodeAPI: {
|
||||
remoteServerContext: {
|
||||
childProcess: {
|
||||
exec: (async (
|
||||
exec: async (
|
||||
command: string,
|
||||
options?: ExecOptions & {encoding?: BufferEncoding},
|
||||
) =>
|
||||
renderHost.flipperServer.exec(
|
||||
'node-api-exec',
|
||||
command,
|
||||
options,
|
||||
)) as RemoteNodeAPI['childProcess']['exec'],
|
||||
) => renderHost.flipperServer.exec('node-api-exec', command, options),
|
||||
},
|
||||
fs: {
|
||||
access: async (path: string, mode?: number) =>
|
||||
@@ -95,7 +93,7 @@ export function initializeFlipperLibImplementation(
|
||||
'node-api-fs-mkdir',
|
||||
path,
|
||||
options,
|
||||
)) as RemoteNodeAPI['fs']['mkdir'],
|
||||
)) as RemoteServerContext['fs']['mkdir'],
|
||||
copyFile: async (src: string, dest: string, flags?: number) =>
|
||||
renderHost.flipperServer.exec(
|
||||
'node-api-fs-copyFile',
|
||||
|
||||
@@ -262,7 +262,7 @@ export default class AllocationsPlugin extends FlipperDevicePlugin<
|
||||
}
|
||||
|
||||
async init() {
|
||||
await getFlipperLib().removeNodeAPI.childProcess.exec(
|
||||
await getFlipperLib().remoteServerContext.childProcess.exec(
|
||||
'adb forward tcp:6000 localfilesystem:/data/local/debugger-socket',
|
||||
);
|
||||
await this.connectToDebugApi();
|
||||
|
||||
@@ -81,7 +81,7 @@ export default class KaiOSGraphs extends FlipperDevicePlugin<State, any, any> {
|
||||
|
||||
async init() {
|
||||
try {
|
||||
await getFlipperLib().removeNodeAPI.childProcess.exec('adb root');
|
||||
await getFlipperLib().remoteServerContext.childProcess.exec('adb root');
|
||||
} catch (e) {
|
||||
console.error('Error obtaining root on the device', e);
|
||||
}
|
||||
|
||||
@@ -33,7 +33,9 @@ const DEV_TOOLS_PORT = 8097; // hardcoded in RN
|
||||
async function findGlobalDevTools(): Promise<string | undefined> {
|
||||
try {
|
||||
const {stdout: basePath} =
|
||||
await getFlipperLib().removeNodeAPI.childProcess.exec('npm root -g');
|
||||
await getFlipperLib().remoteServerContext.childProcess.exec(
|
||||
'npm root -g',
|
||||
);
|
||||
const devToolsPath = path.join(
|
||||
basePath.trim(),
|
||||
'react-devtools',
|
||||
|
||||
Reference in New Issue
Block a user