From e4b5e70c34eccdc4378101ecddab983cf169aadc Mon Sep 17 00:00:00 2001 From: icarus Date: Mon, 20 Oct 2025 08:21:18 +0800 Subject: [PATCH] refactor(ocr): update timestamp handling to use milliseconds Use dayjs().valueOf() instead of dayjs().unix() to get timestamps in milliseconds for consistency with the updated schema comment --- src/main/data/db/schemas/ocr/provider.ts | 2 +- src/main/services/ocr/OcrService.ts | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/main/data/db/schemas/ocr/provider.ts b/src/main/data/db/schemas/ocr/provider.ts index a7bb078613..47cb91c30e 100644 --- a/src/main/data/db/schemas/ocr/provider.ts +++ b/src/main/data/db/schemas/ocr/provider.ts @@ -39,7 +39,7 @@ export const ocrProviderTable = sqliteTable( */ config: text('config', { mode: 'json' }).$type().notNull(), - /** Timestamps. May not useful. */ + /** Unix timestamp (milliseconds since epoch) for creation and last update. */ ...createUpdateTimestamps }, (t) => [index('name').on(t.name)] diff --git a/src/main/services/ocr/OcrService.ts b/src/main/services/ocr/OcrService.ts index 53bb619c4a..bce1e3b01a 100644 --- a/src/main/services/ocr/OcrService.ts +++ b/src/main/services/ocr/OcrService.ts @@ -92,7 +92,7 @@ export class OcrService { throw new Error(`OCR provider ${update.id} not found`) } const found = providers[0] - const newProvider = { ...merge({}, found, update), updatedAt: dayjs().unix() } satisfies DbOcrProvider + const newProvider = { ...merge({}, found, update), updatedAt: dayjs().valueOf() } satisfies DbOcrProvider const [updated] = await dbService .getDb() .update(ocrProviderTable) @@ -114,7 +114,7 @@ export class OcrService { throw new Error(`OCR provider ${create.id} already exists`) } - const timestamp = dayjs().unix() + const timestamp = dayjs().valueOf() const newProvider = { ...create, createdAt: timestamp, @@ -137,7 +137,7 @@ export class OcrService { .where(eq(ocrProviderTable.id, provider.id)) .limit(1) - const timestamp = dayjs().unix() + const timestamp = dayjs().valueOf() if (providers.length === 0) { const newProvider = { ...provider,