Compare commits
1 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| ad1242ad67 |
@@ -0,0 +1,4 @@
|
|||||||
|
## 2025-06-03 - Avoid TextEncoder for string byte length
|
||||||
|
|
||||||
|
**Learning:** `new TextEncoder().encode(text).byteLength` allocates a `Uint8Array` for the entire string, which causes significant memory allocations and is ~100x slower for large JSON payloads in Bun/Node.
|
||||||
|
**Action:** Use `Buffer.byteLength(text)` when available (with a fallback to `TextEncoder` for browser compatibility) to compute string byte lengths without memory allocation overhead.
|
||||||
+3
-5
@@ -307,10 +307,6 @@ function PrivateApp() {
|
|||||||
[],
|
[],
|
||||||
);
|
);
|
||||||
|
|
||||||
const handleExcalidrawAPI = useCallback((api: ExcalidrawImperativeAPI) => {
|
|
||||||
excalidrawApiRef.current = api;
|
|
||||||
}, []);
|
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<div className="app-shell" ref={appShellRef}>
|
<div className="app-shell" ref={appShellRef}>
|
||||||
{toastMessage && (
|
{toastMessage && (
|
||||||
@@ -336,7 +332,9 @@ function PrivateApp() {
|
|||||||
onSceneChange={handleSceneChange}
|
onSceneChange={handleSceneChange}
|
||||||
onSelectionStateChange={handleCodeBlockSelectionChange}
|
onSelectionStateChange={handleCodeBlockSelectionChange}
|
||||||
onEditorActivity={scheduleThemeTokenSync}
|
onEditorActivity={scheduleThemeTokenSync}
|
||||||
onExcalidrawAPI={handleExcalidrawAPI}
|
onExcalidrawAPI={(api) => {
|
||||||
|
excalidrawApiRef.current = api;
|
||||||
|
}}
|
||||||
renderEmbeddable={renderCodeBlockEmbeddable}
|
renderEmbeddable={renderCodeBlockEmbeddable}
|
||||||
/>
|
/>
|
||||||
<CodeBlockSidebar
|
<CodeBlockSidebar
|
||||||
|
|||||||
@@ -30,24 +30,20 @@ function CodeBlockEditor({
|
|||||||
|
|
||||||
useEffect(() => {
|
useEffect(() => {
|
||||||
let cancelled = false;
|
let cancelled = false;
|
||||||
|
void renderHighlightedCodeBlockHtml(draft.code, draft.language)
|
||||||
const timeoutId = setTimeout(() => {
|
.then((nextHtml) => {
|
||||||
void renderHighlightedCodeBlockHtml(draft.code, draft.language)
|
if (!cancelled) {
|
||||||
.then((nextHtml) => {
|
setHtml(nextHtml);
|
||||||
if (!cancelled) {
|
}
|
||||||
setHtml(nextHtml);
|
})
|
||||||
}
|
.catch(() => {
|
||||||
})
|
if (!cancelled) {
|
||||||
.catch(() => {
|
setHtml(null);
|
||||||
if (!cancelled) {
|
}
|
||||||
setHtml(null);
|
});
|
||||||
}
|
|
||||||
});
|
|
||||||
}, 100);
|
|
||||||
|
|
||||||
return () => {
|
return () => {
|
||||||
cancelled = true;
|
cancelled = true;
|
||||||
clearTimeout(timeoutId);
|
|
||||||
};
|
};
|
||||||
}, [draft.code, draft.language]);
|
}, [draft.code, draft.language]);
|
||||||
|
|
||||||
|
|||||||
+8
-1
@@ -75,7 +75,14 @@ export function normalizeScene(input: unknown): ScenePayload {
|
|||||||
}
|
}
|
||||||
|
|
||||||
export function parseJsonBody<T = unknown>(text: string): T {
|
export function parseJsonBody<T = unknown>(text: string): T {
|
||||||
if (new TextEncoder().encode(text).byteLength > MAX_SCENE_BYTES) {
|
// Optimization: TextEncoder.encode creates a new Uint8Array and allocates memory for the entire
|
||||||
|
// string, which is slow for large JSON payloads. Buffer.byteLength counts bytes without allocation.
|
||||||
|
const byteLength =
|
||||||
|
typeof Buffer !== "undefined"
|
||||||
|
? Buffer.byteLength(text)
|
||||||
|
: new TextEncoder().encode(text).byteLength;
|
||||||
|
|
||||||
|
if (byteLength > MAX_SCENE_BYTES) {
|
||||||
throw new HttpError(413, "Payload too large");
|
throw new HttpError(413, "Payload too large");
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user