fix: address automated review feedback
This commit is contained in:
1 parent
254438247e
commit
aded62977e
4 files changed
+22
-6
No files matched your search
@@ -214,10 +214,26 @@ export const createSessionQueryPersister = (
|
||||
remainingBytes -= queryBytes;
|
||||
}
|
||||
|
||||
return {
|
||||
const buildCandidate = (queryCount: number): PersistedClient => ({
|
||||
...limitedClient,
|
||||
clientState: { ...limitedClient.clientState, queries: selectedQueries },
|
||||
};
|
||||
clientState: { ...limitedClient.clientState, queries: selectedQueries.slice(0, queryCount) },
|
||||
});
|
||||
let minimumQueryCount = 0;
|
||||
let maximumQueryCount = selectedQueries.length;
|
||||
|
||||
while (minimumQueryCount < maximumQueryCount) {
|
||||
const queryCount = Math.ceil((minimumQueryCount + maximumQueryCount) / 2);
|
||||
const candidate = buildCandidate(queryCount);
|
||||
if (measureSerializedBytes(candidate) <= queryPersistenceMaxCacheBytes) {
|
||||
minimumQueryCount = queryCount;
|
||||
} else {
|
||||
maximumQueryCount = queryCount - 1;
|
||||
}
|
||||
}
|
||||
|
||||
const candidate = buildCandidate(minimumQueryCount);
|
||||
if (measureSerializedBytes(candidate) > queryPersistenceMaxCacheBytes) return undefined;
|
||||
return candidate;
|
||||
};
|
||||
|
||||
const persistNow = (client: PersistedClient) => {
|
||||
|
||||
@@ -50,7 +50,7 @@ const overview = `<!doctype html>
|
||||
export const GET = () =>
|
||||
new Response(overview, {
|
||||
headers: {
|
||||
"Cache-Control": "public, max-age=31536000, immutable",
|
||||
"Cache-Control": "public, max-age=3600",
|
||||
"Content-Type": "text/html; charset=utf-8",
|
||||
},
|
||||
});
|
||||
@@ -4,7 +4,7 @@ const videoBase64 =
|
||||
export const GET = () =>
|
||||
new Response(Buffer.from(videoBase64, "base64"), {
|
||||
headers: {
|
||||
"Cache-Control": "public, max-age=31536000, immutable",
|
||||
"Cache-Control": "public, max-age=3600",
|
||||
"Content-Type": "video/mp4",
|
||||
},
|
||||
});
|
||||
@@ -51,7 +51,7 @@ export const immichRouter = createTRPCRouter({
|
||||
.concat(createImmichIntegrationMiddleware("query"))
|
||||
.query(async ({ ctx, input }) => {
|
||||
if (ctx.integration.kind === "mock") {
|
||||
if (input.limit === undefined) return mockWidgetData.immichAlbums;
|
||||
if (input.limit === undefined) return [...mockWidgetData.immichAlbums];
|
||||
return mockWidgetData.immichAlbums.slice(0, input.limit);
|
||||
}
|
||||
const innerHandler = immichAlbumsRequestHandler.handler(
|
||||
|
||||
Reference in new issue
Block a user