Skip to content

Commit 4bd62a1

Browse files
huntiemeta-codesync[bot]
authored andcommitted
Remove "JavaScript logs have moved!" notice (#55075)
Summary: Pull Request resolved: #55075 It's been >1y since we introduced this notice (and introduced the `--client-logs` flag) — clean up. Changelog: [General][Changed] - Remove "JavaScript logs have moved!" notice from dev server CLI Reviewed By: hoxyq Differential Revision: D90247471 fbshipit-source-id: dc13c90476c65c301bdb3a51db91f0e5fe0b326b
1 parent f140c49 commit 4bd62a1

File tree

4 files changed

+0
-33
lines changed

4 files changed

+0
-33
lines changed

packages/dev-middleware/src/createDevMiddleware.js

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -160,17 +160,6 @@ function createWrappedEventReporter(
160160
event.appId ?? 'unknown',
161161
);
162162
break;
163-
case 'fusebox_console_notice':
164-
logger?.info(
165-
'\u001B[1m\u001B[7m💡 JavaScript logs have moved!\u001B[22m They can now be ' +
166-
'viewed in React Native DevTools. Tip: Type \u001B[1mj\u001B[22m in ' +
167-
'the terminal to open' +
168-
(experiments.enableStandaloneFuseboxShell
169-
? ''
170-
: ' (requires Google Chrome or Microsoft Edge)') +
171-
'.\u001B[27m',
172-
);
173-
break;
174163
case 'fusebox_shell_preparation_attempt':
175164
switch (event.result.code) {
176165
case 'success':

packages/dev-middleware/src/inspector-proxy/Device.js

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -54,8 +54,6 @@ export const WS_CLOSE_REASON = {
5454
// more details.
5555
const FILE_PREFIX = 'file://';
5656

57-
let fuseboxConsoleNoticeLogged = false;
58-
5957
type DebuggerConnection = {
6058
// Debugger web socket connection
6159
socket: WS,
@@ -542,7 +540,6 @@ export default class Device {
542540
// created instead of manually checking this on every getPages result.
543541
for (const page of this.#pages.values()) {
544542
if (this.#pageHasCapability(page, 'nativePageReloads')) {
545-
this.#logFuseboxConsoleNotice();
546543
continue;
547544
}
548545

@@ -1092,14 +1089,4 @@ export default class Device {
10921089
dangerouslyGetSocket(): WS {
10931090
return this.#deviceSocket;
10941091
}
1095-
1096-
// TODO(T214991636): Remove notice
1097-
#logFuseboxConsoleNotice() {
1098-
if (fuseboxConsoleNoticeLogged) {
1099-
return;
1100-
}
1101-
1102-
this.#deviceEventReporter?.logFuseboxConsoleNotice();
1103-
fuseboxConsoleNoticeLogged = true;
1104-
}
11051092
}

packages/dev-middleware/src/inspector-proxy/DeviceEventReporter.js

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -231,12 +231,6 @@ class DeviceEventReporter {
231231
});
232232
}
233233

234-
logFuseboxConsoleNotice(): void {
235-
this.#eventReporter.logEvent({
236-
type: 'fusebox_console_notice',
237-
});
238-
}
239-
240234
#logExpiredCommand(pendingCommand: PendingCommand): void {
241235
this.#eventReporter.logEvent({
242236
type: 'debugger_command',

packages/dev-middleware/src/types/EventReporter.js

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -88,9 +88,6 @@ export type ReportableEvent =
8888
status: 'success',
8989
...DebuggerSessionIDs,
9090
}
91-
| {
92-
type: 'fusebox_console_notice',
93-
}
9491
| {
9592
type: 'no_debug_pages_for_device',
9693
...DebuggerSessionIDs,

0 commit comments

Comments
 (0)