Skip to content

Commit 0d168e0

Browse files
committed
Fix copilot comments
1 parent cba3ae4 commit 0d168e0

3 files changed

Lines changed: 4 additions & 16 deletions

File tree

packages/server/api/src/app/ai/chat/ai-chat.service.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,7 @@ export const createChatContext = async (
141141
chatExpireTime,
142142
);
143143
const indexKey = userChatsIndexKey(userId, projectId);
144-
await cacheWrapper.addToSet(indexKey, chatId, chatExpireTime);
144+
await cacheWrapper.addToSet(indexKey, chatId);
145145
};
146146

147147
export const getChatContext = async (

packages/server/shared/src/lib/cache/memory-wrapper.ts

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -73,16 +73,12 @@ const getBufferAndDelete = async (key: string): Promise<Buffer | null> => {
7373
throw new Error('Not implemented');
7474
};
7575

76-
const addToSet = async (
77-
key: string,
78-
member: string,
79-
expireInSeconds: number = DEFAULT_EXPIRE_TIME / 1000,
80-
): Promise<void> => {
76+
const addToSet = async (key: string, member: string): Promise<void> => {
8177
const existing = (await getSerializedObject<string[]>(key)) ?? [];
8278
if (!existing.includes(member)) {
8379
existing.push(member);
8480
}
85-
await setSerializedObject(key, existing, expireInSeconds);
81+
await setSerializedObject(key, existing);
8682
};
8783

8884
const removeFromSet = async (key: string, member: string): Promise<void> => {

packages/server/shared/src/lib/cache/redis-wrapper.ts

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -59,17 +59,9 @@ async function setSerializedObject<T>(
5959
await setKey(key, JSON.stringify(obj), expireInSeconds);
6060
}
6161

62-
const addToSet = async (
63-
key: string,
64-
member: string,
65-
expireInSeconds?: number,
66-
): Promise<void> => {
62+
const addToSet = async (key: string, member: string): Promise<void> => {
6763
const redis = getRedisClient();
6864
await redis.sadd(key, member);
69-
70-
if (expireInSeconds && expireInSeconds > 0) {
71-
await redis.expire(key, expireInSeconds);
72-
}
7365
};
7466

7567
const removeFromSet = async (key: string, member: string): Promise<void> => {

0 commit comments

Comments
 (0)