Skip to content

Commit 535ea15

Browse files
perf(web): discovered local servers compile without manual memoization
Co-Authored-By: Claude Opus 5.5 <[email protected]>
1 parent 70d60e0 commit 535ea15

2 files changed

Lines changed: 10 additions & 14 deletions

File tree

‎apps/web/reactCompiledFiles.ts‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -96,4 +96,5 @@ export const REACT_COMPILED_FILES = [
9696
"apps/web/src/components/pullRequest/PullRequestDetailPanel.tsx",
9797
"apps/web/src/components/ThreadTerminalDrawer.tsx",
9898
"apps/web/src/components/settings/SettingsPanels.tsx",
99+
"apps/web/src/components/preview/useDiscoveredLocalServers.ts",
99100
] as const;

‎apps/web/src/components/preview/useDiscoveredLocalServers.ts‎

Lines changed: 9 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import type { DiscoveredLocalServer } from "@t3tools/contracts";
22
import { isLoopbackHost } from "@t3tools/shared/preview";
3-
import { useMemo } from "react";
43

54
import type { EnvironmentId } from "@t3tools/contracts";
65
import { resolveDiscoveredServerUrl } from "~/browser/browserTargetResolver";
@@ -25,19 +24,15 @@ export function useDiscoveredLocalServers(
2524
): ReadonlyArray<PreviewableServer> {
2625
const scannerState = useDiscoveredPortsState(input.environmentId, input.configuredUrls);
2726

28-
return useMemo(
29-
() =>
30-
mergeServers({
31-
scanner: scannerState.servers.map((server) => ({
32-
...server,
33-
url: resolveDiscoveredServerUrl(input.environmentId, server.url),
34-
requestedUrl: server.url,
35-
})),
36-
configuredUrls: input.configuredUrls ?? [],
37-
configuredUrlProbing: scannerState.configuredUrlProbing,
38-
}),
39-
[input.environmentId, scannerState, input.configuredUrls],
40-
);
27+
return mergeServers({
28+
scanner: scannerState.servers.map((server) => ({
29+
...server,
30+
url: resolveDiscoveredServerUrl(input.environmentId, server.url),
31+
requestedUrl: server.url,
32+
})),
33+
configuredUrls: input.configuredUrls ?? [],
34+
configuredUrlProbing: scannerState.configuredUrlProbing,
35+
});
4136
}
4237

4338
export function mergeServers(input: {

0 commit comments

Comments
 (0)