refactor(database): ♻️ Use dates instead of strings in database

CPlusPatch · CPlusPatch · commit 6056a6622c0d · 2025-12-11T04:03:57.000+01:00
diff --git a/cli/user/delete.ts b/cli/user/delete.ts
@@ -31,7 +31,9 @@ export const deleteUserCommand = defineCommand({
         console.info(`About to delete user ${chalk.gray(user.data.username)}!`);
         console.info(`Username: ${chalk.blue(user.data.username)}`);
         console.info(`Display Name: ${chalk.blue(user.data.displayName)}`);
-        console.info(`Created At: ${chalk.blue(user.data.createdAt)}`);
+        console.info(
+            `Created At: ${chalk.blue(user.data.createdAt.toISOString())}`,
+        );
         console.info(
             `Instance: ${chalk.blue(user.data.instance?.baseUrl || "Local")}`,
         );
diff --git a/packages/api/routes/api/v1/accounts/[id]/index.test.ts b/packages/api/routes/api/v1/accounts/[id]/index.test.ts
@@ -44,7 +44,7 @@ describe("/api/v1/accounts/:id", () => {
             avatar: expect.any(String),
             header: expect.any(String),
             locked: users[0].data.isLocked,
-            created_at: new Date(users[0].data.createdAt).toISOString(),
+            created_at: users[0].data.createdAt.toISOString(),
             followers_count: 0,
             following_count: 0,
             statuses_count: 5,
diff --git a/packages/api/routes/api/v1/accounts/verify_credentials/index.test.ts b/packages/api/routes/api/v1/accounts/verify_credentials/index.test.ts
@@ -48,9 +48,7 @@ describe("/api/v1/accounts/verify_credentials", () => {
             expect(data.moved).toBeNull();
             expect(data.suspended).toBe(false);
             expect(data.limited).toBe(false);
-            expect(data.created_at).toBe(
-                new Date(users[0].data.createdAt).toISOString(),
-            );
+            expect(data.created_at).toBe(users[0].data.createdAt.toISOString());
             expect(data.last_status_at).toBeNull();
             expect(data.statuses_count).toBe(0);
             expect(data.followers_count).toBe(0);
diff --git a/packages/api/routes/api/v1/markers/index.ts b/packages/api/routes/api/v1/markers/index.ts
@@ -98,7 +98,7 @@ export default apiRoute((app) => {
                     markers.home = {
                         last_read_id: found.noteId,
                         version: totalCount,
-                        updated_at: new Date(found.createdAt).toISOString(),
+                        updated_at: found.createdAt.toISOString(),
                     };
                 }
             }
@@ -124,7 +124,7 @@ export default apiRoute((app) => {
                     markers.notifications = {
                         last_read_id: found.notificationId,
                         version: totalCount,
-                        updated_at: new Date(found.createdAt).toISOString(),
+                        updated_at: found.createdAt.toISOString(),
                     };
                 }
             }
@@ -212,9 +212,7 @@ export default apiRoute((app) => {
                 markers.home = {
                     last_read_id: homeId,
                     version: totalCount,
-                    updated_at: new Date(
-                        insertedMarker.createdAt,
-                    ).toISOString(),
+                    updated_at: insertedMarker.createdAt.toISOString(),
                 };
             }
 
@@ -242,9 +240,7 @@ export default apiRoute((app) => {
                 markers.notifications = {
                     last_read_id: notificationsId,
                     version: totalCount,
-                    updated_at: new Date(
-                        insertedMarker.createdAt,
-                    ).toISOString(),
+                    updated_at: insertedMarker.createdAt.toISOString(),
                 };
             }
 
diff --git a/packages/api/routes/api/v2/filters/[id]/index.ts b/packages/api/routes/api/v2/filters/[id]/index.ts
@@ -184,9 +184,7 @@ export default apiRoute((app) => {
                     title,
                     context: ctx ?? [],
                     filterAction: filter_action,
-                    expireAt: new Date(
-                        Date.now() + (expires_in ?? 0),
-                    ).toISOString(),
+                    expireAt: new Date(Date.now() + (expires_in ?? 0)),
                 })
                 .where(and(eq(Filters.userId, user.id), eq(Filters.id, id)));
 
@@ -244,9 +242,7 @@ export default apiRoute((app) => {
                     id: updatedFilter.id,
                     title: updatedFilter.title,
                     context: updatedFilter.context,
-                    expires_at: updatedFilter.expireAt
-                        ? new Date(updatedFilter.expireAt).toISOString()
-                        : null,
+                    expires_at: updatedFilter.expireAt?.toISOString() || null,
                     filter_action: updatedFilter.filterAction,
                     keywords: updatedFilter.keywords.map((keyword) => ({
                         id: keyword.id,
diff --git a/packages/api/routes/api/v2/filters/index.ts b/packages/api/routes/api/v2/filters/index.ts
@@ -59,9 +59,7 @@ export default apiRoute((app) => {
                     id: filter.id,
                     title: filter.title,
                     context: filter.context,
-                    expires_at: filter.expireAt
-                        ? new Date(Date.now() + filter.expireAt).toISOString()
-                        : null,
+                    expires_at: filter.expireAt?.toISOString() || null,
                     filter_action: filter.filterAction,
                     keywords: filter.keywords.map((keyword) => ({
                         id: keyword.id,
@@ -147,9 +145,7 @@ export default apiRoute((app) => {
                         title,
                         context: ctx,
                         filterAction: filter_action,
-                        expireAt: new Date(
-                            Date.now() + (expires_in ?? 0),
-                        ).toISOString(),
+                        expireAt: new Date(Date.now() + (expires_in ?? 0)),
                         userId: user.id,
                     })
                     .returning()
diff --git a/packages/api/routes/oauth/revoke.test.ts b/packages/api/routes/oauth/revoke.test.ts
@@ -17,8 +17,8 @@ const token = await Token.insert({
     id: randomUUIDv7(),
     clientId: application.id,
     accessToken: "test-access-token",
-    expiresAt: new Date(Date.now() + 3600 * 1000).toISOString(),
-    createdAt: new Date().toISOString(),
+    expiresAt: new Date(Date.now() + 3600 * 1000),
+    createdAt: new Date(),
     scopes: application.data.scopes,
     userId: users[0].id,
 });
diff --git a/packages/api/routes/oauth/sso/[issuer]/callback.ts b/packages/api/routes/oauth/sso/[issuer]/callback.ts
@@ -278,7 +278,7 @@ export default apiRoute((app) => {
             await db.insert(AuthorizationCodes).values({
                 clientId: flow.client.id,
                 code,
-                expiresAt: new Date(Date.now() + 10 * 60 * 1000).toISOString(), // 10 minutes
+                expiresAt: new Date(Date.now() + 10 * 60 * 1000), // 10 minutes
                 redirectUri: flow.clientRedirectUri ?? undefined,
                 userId: user.id,
                 scopes: flow.clientScopes ?? [],
diff --git a/packages/api/routes/oauth/token.test.ts b/packages/api/routes/oauth/token.test.ts
@@ -24,7 +24,7 @@ const authorizationCode = (
             code: randomString(10),
             redirectUri: application.data.redirectUris[0],
             userId: users[0].id,
-            expiresAt: new Date(Date.now() + 300 * 1000).toISOString(),
+            expiresAt: new Date(Date.now() + 300 * 1000),
         })
         .returning()
 )[0];
diff --git a/packages/api/routes/oauth/token.ts b/packages/api/routes/oauth/token.ts
@@ -101,7 +101,7 @@ export default apiRoute((app) => {
 
             if (
                 !authorizationCode ||
-                new Date(authorizationCode.expiresAt).getTime() < Date.now()
+                authorizationCode.expiresAt.getTime() < Date.now()
             ) {
                 return context.json(
                     {
@@ -131,8 +131,7 @@ export default apiRoute((app) => {
                     ...token.toApi(),
                     expires_in: token.data.expiresAt
                         ? Math.floor(
-                              (new Date(token.data.expiresAt).getTime() -
-                                  Date.now()) /
+                              (token.data.expiresAt.getTime() - Date.now()) /
                                   1000,
                           )
                         : null,
diff --git a/packages/api/routes/well-known/versia.ts b/packages/api/routes/well-known/versia.ts
@@ -73,9 +73,7 @@ export default apiRoute((app) =>
                         "/inbox",
                         config.http.base_url,
                     ).toString(),
-                    created_at: new Date(
-                        firstUser?.data.createdAt ?? 0,
-                    ).toISOString(),
+                    created_at: firstUser?.data.createdAt.toISOString(),
                     extensions: {
                         "pub.versia:instance_messaging": {
                             endpoint: new URL(
diff --git a/packages/kit/api.ts b/packages/kit/api.ts
@@ -137,7 +137,7 @@ export const checkRouteNeedsChallenge = async (
     // Expire the challenge
     await db
         .update(Challenges)
-        .set({ expiresAt: new Date().toISOString() })
+        .set({ expiresAt: new Date() })
         .where(eq(Challenges.id, challenge_id));
 };
 
diff --git a/packages/kit/db/like.ts b/packages/kit/db/like.ts
@@ -153,7 +153,7 @@ export class Like extends BaseInterface<typeof Likes, LikeType> {
                 this.data.liker.uri ? new URL(this.data.liker.uri) : null,
             ).href,
             type: "pub.versia:likes/Like",
-            created_at: new Date(this.data.createdAt).toISOString(),
+            created_at: this.data.createdAt.toISOString(),
             liked: this.data.liked.uri
                 ? new URL(this.data.liked.uri).href
                 : new URL(`/notes/${this.data.liked.id}`, config.http.base_url)
diff --git a/packages/kit/db/note.ts b/packages/kit/db/note.ts
@@ -513,7 +513,7 @@ export class Note extends BaseInterface<typeof Notes, NoteTypeWithRelations> {
             reblogId: this.id,
             visibility,
             sensitive: false,
-            updatedAt: new Date().toISOString(),
+            updatedAt: new Date(),
             clientId: null,
             uri: uri?.href,
         });
@@ -979,7 +979,7 @@ export class Note extends BaseInterface<typeof Notes, NoteTypeWithRelations> {
                 authorId: author.id,
                 visibility: "public",
                 uri,
-                createdAt: new Date(created_at).toISOString(),
+                createdAt: new Date(created_at),
             }));
 
         const attachments = await Promise.all(
@@ -1150,7 +1150,7 @@ export class Note extends BaseInterface<typeof Notes, NoteTypeWithRelations> {
             in_reply_to_id: data.replyId || null,
             in_reply_to_account_id: data.reply?.authorId || null,
             account: this.author.toApi(userFetching?.id === data.authorId),
-            created_at: new Date(data.createdAt).toISOString(),
+            created_at: data.createdAt.toISOString(),
             application: data.client
                 ? new Client(data.client).toApi()
                 : undefined,
@@ -1200,9 +1200,7 @@ export class Note extends BaseInterface<typeof Notes, NoteTypeWithRelations> {
                       (n) => n?.toApi(userFetching),
                   )) ?? null)
                 : null,
-            edited_at: data.updatedAt
-                ? new Date(data.updatedAt).toISOString()
-                : null,
+            edited_at: data.updatedAt.toISOString(),
             reactions,
             text: data.contentSource,
         };
@@ -1246,7 +1244,7 @@ export class Note extends BaseInterface<typeof Notes, NoteTypeWithRelations> {
         const status = this.data;
         return new VersiaEntities.Note({
             type: "Note",
-            created_at: new Date(status.createdAt).toISOString(),
+            created_at: status.createdAt.toISOString(),
             id: status.id,
             author: this.author.uri.href,
             uri: this.getUri().href,
diff --git a/packages/kit/db/notification.ts b/packages/kit/db/notification.ts
@@ -186,7 +186,7 @@ export class Notification extends BaseInterface<
 
         return {
             account: account.toApi(),
-            created_at: new Date(this.data.createdAt).toISOString(),
+            created_at: this.data.createdAt.toISOString(),
             id: this.data.id,
             type: this.data.type,
             status: this.data.status
diff --git a/packages/kit/db/reaction.ts b/packages/kit/db/reaction.ts
@@ -217,7 +217,7 @@ export class Reaction extends BaseInterface<typeof Reactions, ReactionType> {
                 this.data.authorId,
                 this.data.author.uri ? new URL(this.data.author.uri) : null,
             ).href,
-            created_at: new Date(this.data.createdAt).toISOString(),
+            created_at: this.data.createdAt.toISOString(),
             id: this.id,
             object: this.data.note.uri
                 ? new URL(this.data.note.uri).href
diff --git a/packages/kit/db/token.ts b/packages/kit/db/token.ts
@@ -160,9 +160,7 @@ export class Token extends BaseInterface<typeof Tokens, TokenType> {
             access_token: this.data.accessToken,
             token_type: "Bearer",
             scope: this.data.scopes.join(" "),
-            created_at: Math.floor(
-                new Date(this.data.createdAt).getTime() / 1000,
-            ),
+            created_at: Math.floor(this.data.createdAt.getTime() / 1000),
         };
     }
 }
diff --git a/packages/kit/db/user.ts b/packages/kit/db/user.ts
@@ -475,7 +475,7 @@ export class User extends BaseInterface<typeof Users, UserWithRelations> {
                         isNull(Users.instanceId),
                         gte(
                             Notes.createdAt,
-                            new Date(Date.now() - milliseconds).toISOString(),
+                            new Date(Date.now() - milliseconds),
                         ),
                     ),
                 )
@@ -803,7 +803,7 @@ export class User extends BaseInterface<typeof Users, UserWithRelations> {
         }
 
         await user.update({
-            createdAt: new Date(created_at).toISOString(),
+            createdAt: new Date(created_at),
             endpoints: {
                 inbox,
                 outbox: collections.outbox,
@@ -940,7 +940,7 @@ export class User extends BaseInterface<typeof Users, UserWithRelations> {
             publicKey: keys.public_key,
             fields: [],
             privateKey: keys.private_key,
-            updatedAt: new Date().toISOString(),
+            updatedAt: new Date(),
             source: {
                 language: "en",
                 note: "",
@@ -1116,7 +1116,7 @@ export class User extends BaseInterface<typeof Users, UserWithRelations> {
             avatar: this.getAvatarUrl().proxied,
             header: this.getHeaderUrl()?.proxied ?? "",
             locked: user.isLocked,
-            created_at: new Date(user.createdAt).toISOString(),
+            created_at: user.createdAt.toISOString(),
             followers_count:
                 user.isHidingCollections && !isOwnAccount
                     ? 0
@@ -1177,7 +1177,7 @@ export class User extends BaseInterface<typeof Users, UserWithRelations> {
                     remote: false,
                 },
             },
-            created_at: new Date(user.createdAt).toISOString(),
+            created_at: user.createdAt.toISOString(),
             collections: {
                 featured: new URL(
                     `/users/${user.id}/featured`,
diff --git a/packages/kit/tables/migrations/0053_lively_hellfire_club.sql b/packages/kit/tables/migrations/0053_lively_hellfire_club.sql
@@ -0,0 +1,45 @@
+ALTER TABLE "AuthorizationCodes" ALTER COLUMN "expires_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "AuthorizationCodes" ALTER COLUMN "created_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "AuthorizationCodes" ALTER COLUMN "created_at" SET DEFAULT now();--> statement-breakpoint
+ALTER TABLE "Challenges" ALTER COLUMN "expires_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "Challenges" ALTER COLUMN "expires_at" SET DEFAULT NOW() + INTERVAL '5 minutes';--> statement-breakpoint
+ALTER TABLE "Challenges" ALTER COLUMN "created_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "Challenges" ALTER COLUMN "created_at" SET DEFAULT now();--> statement-breakpoint
+ALTER TABLE "Filters" ALTER COLUMN "expires_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "Filters" ALTER COLUMN "created_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "Filters" ALTER COLUMN "created_at" SET DEFAULT now();--> statement-breakpoint
+ALTER TABLE "Flags" ALTER COLUMN "created_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "Flags" ALTER COLUMN "created_at" SET DEFAULT now();--> statement-breakpoint
+ALTER TABLE "Likes" ALTER COLUMN "created_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "Likes" ALTER COLUMN "created_at" SET DEFAULT now();--> statement-breakpoint
+ALTER TABLE "Markers" ALTER COLUMN "created_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "Markers" ALTER COLUMN "created_at" SET DEFAULT now();--> statement-breakpoint
+ALTER TABLE "ModNotes" ALTER COLUMN "created_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "ModNotes" ALTER COLUMN "created_at" SET DEFAULT now();--> statement-breakpoint
+ALTER TABLE "ModTags" ALTER COLUMN "created_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "ModTags" ALTER COLUMN "created_at" SET DEFAULT now();--> statement-breakpoint
+ALTER TABLE "Notes" ALTER COLUMN "created_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "Notes" ALTER COLUMN "created_at" SET DEFAULT now();--> statement-breakpoint
+ALTER TABLE "Notes" ALTER COLUMN "updated_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "Notes" ALTER COLUMN "updated_at" SET DEFAULT now();--> statement-breakpoint
+ALTER TABLE "Notifications" ALTER COLUMN "created_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "Notifications" ALTER COLUMN "created_at" SET DEFAULT now();--> statement-breakpoint
+ALTER TABLE "PushSubscriptions" ALTER COLUMN "created_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "PushSubscriptions" ALTER COLUMN "created_at" SET DEFAULT now();--> statement-breakpoint
+ALTER TABLE "PushSubscriptions" ALTER COLUMN "updated_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "PushSubscriptions" ALTER COLUMN "updated_at" SET DEFAULT now();--> statement-breakpoint
+ALTER TABLE "Reaction" ALTER COLUMN "created_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "Reaction" ALTER COLUMN "created_at" SET DEFAULT now();--> statement-breakpoint
+ALTER TABLE "Reaction" ALTER COLUMN "updated_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "Reaction" ALTER COLUMN "updated_at" SET DEFAULT now();--> statement-breakpoint
+ALTER TABLE "Relationships" ALTER COLUMN "created_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "Relationships" ALTER COLUMN "created_at" SET DEFAULT now();--> statement-breakpoint
+ALTER TABLE "Relationships" ALTER COLUMN "updated_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "Relationships" ALTER COLUMN "updated_at" SET DEFAULT now();--> statement-breakpoint
+ALTER TABLE "Tokens" ALTER COLUMN "expires_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "Tokens" ALTER COLUMN "created_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "Tokens" ALTER COLUMN "created_at" SET DEFAULT now();--> statement-breakpoint
+ALTER TABLE "Users" ALTER COLUMN "created_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "Users" ALTER COLUMN "created_at" SET DEFAULT now();--> statement-breakpoint
+ALTER TABLE "Users" ALTER COLUMN "updated_at" SET DATA TYPE timestamp (3) with time zone;--> statement-breakpoint
+ALTER TABLE "Users" ALTER COLUMN "updated_at" SET DEFAULT now();
diff --git a/packages/kit/tables/migrations/meta/0053_snapshot.json b/packages/kit/tables/migrations/meta/0053_snapshot.json
diff --git a/packages/kit/tables/migrations/meta/_journal.json b/packages/kit/tables/migrations/meta/_journal.json
diff --git a/packages/kit/tables/schema.ts b/packages/kit/tables/schema.ts
diff --git a/packages/tests/index.ts b/packages/tests/index.ts
diff --git a/utils/challenges.ts b/utils/challenges.ts
diff --git a/utils/rss.ts b/utils/rss.ts
